Хранение нескольких OnClickListeners в одном классе

Question

Я немного новичок в Android, и это улучшило бы мое приложение, если бы я мог хранить несколько OnClickListenres в одном классе. То, о чем я думаю, - это что-то вроде этого:

Public class OnClickListeners {

public Button.OnClickListener open;
public Button.OnClickListener doSomethingElse;
public Button.OnClickListener etc;

    public OnClickListeners() {
        open = new Button.OnClickListener()
        {
                public void onClick(View view)
                {
                    DetailList.SetId(view.getId());
                    Intent intent = new Intent(view.getContext(), DetailList.class);
                    startActivityForResult(intent, 100);
                }
        };
    }
}

Так что я могу затем сослаться на него в другом классе B, как это

button1.setOnClickListener(OnClickListeners.open);

Кто-нибудь, как это сделать? Android SDK, кажется, против меня, поскольку я могу понять это в течение приблизительно 2 дней сейчас ...

Спасибо за любые советы и помощь

Answer 1

Существует элегантный способ объединить все ваши анонимные классы в один и включить представление. Это работает лучше всего, если вы заранее знаете, какие кнопки будут использовать clicklistener:

public class AndroidTestClickListenerActivity extends Activity {

    /** Called when the activity is first created. */
    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);

        Button button1 = (Button) findViewById(R.id.button1);
        button1.setOnClickListener(new MyClickListener());

        Button button2 = (Button) findViewById(R.id.button2);
        button2.setOnClickListener(new MyClickListener());

        Button button3 = (Button) findViewById(R.id.button3);
        button3.setOnClickListener(new MyClickListener());
    }


}

class MyClickListener implements View.OnClickListener {
    @Override
    public void onClick(View arg0) {
        switch (arg0.getId()) {
            case R.id.button1:
                // do soemthign for button1
                break;
            case R.id.button2:
                // do something for button2
                break;
            case R.id.button3:
                // do something for button3
                break;
            default:
                // do something for any other button

        }

    }
}

Answer 2

Поскольку ни одно из решений на самом деле не выполнило того, чего я хотел достичь - мне потребовался второй (или несколько) onClickListener, который не переопределял onClickListeners, которые уже были назначены элементу управления .

Вот класс Java, который я написал для этой цели:

https://gist.github.com/kosiara/c090dcd684ec6fb2ac42#file-doubleclicklistenerimagebutton-java

public class DoubleClickListenerImageButton extends ImageButton {

    View.OnClickListener mSecondOnClickListener;

    public DoubleClickListenerImageButton(Context context) {
       super(context);
    }

    [...]

    public void setSecondOnClickListener(View.OnClickListener l) {
        mSecondOnClickListener = l;
    }

    @Override
    public boolean performClick() {
       if (mSecondOnClickListener != null)
          mSecondOnClickListener.onClick(this);
       return super.performClick();
    }

    @Override
    public boolean performContextClick() {
       if (mSecondOnClickListener != null)
          mSecondOnClickListener.onClick(this);
       return super.performContextClick();
    }
}

Answer 3

Вы можете написать

button1.setOnClickListener(new OnClickListeners().open);  

вместо этого, но это кажется странной архитектурой для меня. Я бы посоветовал вам сохранить 1 слушателя в 1 файле, имея их всех в 1 пакете, и использовать как

button1.setOnClickListener(new OpenListener());  

Answer 4

Проблема вашего подхода в том, что обычно слушатели должны манипулировать некоторыми данными, которые являются частью класса, в котором находятся элементы пользовательского интерфейса.

Если вы берете слушателей и помещаете их вОтдельный класс, вам также нужно будет предоставить множество ссылок на объекты, где находятся данные для манипуляции.Это создаст много взаимозависимых классов, что не очень хорошо.

ИМХО самый чистый способ - использовать анонимные внутренние классы.

Answer 5

Вы можете, но вы должны объявить OnClickListener как статический, если вы хотите использовать его таким образом.

public static Button.OnClickListener openListener = new Button.OnClickListener() {
     public void onClick(View view) {

     }
};

Тогда вы можете использовать:

button1.setOnClickListener(OnClickListeners.openListener);

Какотметил другой пользователь - такой подход больше всего похож на плохой.Вы должны обработать прослушиватели представления в том же представлении, а затем, возможно, вызвать другой метод, например openActivity ().Я бы не стал этого делать - вы также открываете действие из другого действия, это, вероятно, не будет работать вообще или испортит стек истории действий