Best way to implement View.OnClickListener in Android

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

Наиболее распространенный способ реализовать это - позволить подклассу Activity реализовать OnClickListener , что-то вроде этого:

public class ActivityMain extends Activity implements View.OnClickListener
{   
    @Override
    public void onClick(View view)
    {
        switch (view.getId())
        {
            //handle multiple view click events
        }
    }
}

Мне нравится реализовывать это так: создать закрытый класс внутри Activity-подкласса и позволить этому внутреннему классу реализовать OnClickListener:

public class ActivityMain extends Activity implements View.OnClickListener
{
    private class ClickListener implements View.OnClickListener
    {   
        @Override
        public void onClick(View view)
        {
            switch (view.getId())
            {
                //handle multiple view click events
            }
        }
    }
}

Таким образом код кажется более организованным и простым в обслуживании.

Более того, говоря об отношениях "Is-a", "Has-a", последнее кажется хорошей практикой, потому что теперь подкласс Activity будет иметь отношения "Has-a" с ClickListener . В то время как в первом методе мы бы говорили, что наш Activity-подкласс "Is-a" ClickListener, что не совсем верно.

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

Кроме того, о добавлении тега onClick в xml не может быть и речи.

Итак, как на самом деле лучше всего реализовать ClickListener?

Пожалуйста, не предлагайте никаких библиотек, таких как RoboGuice или ButterKnife и т.д.

Обновить:

Я хотел бы поделиться подходом, который я, наконец, принял.

Я непосредственно реализую прослушиватель в Activity / Fragment .

Что касается дизайна ООП. Подход "ЕСТЬ-А" не дает никаких практических преимуществ и даже занимает больше памяти. Учитывая количество вложенных классов (и нагрузку на память), которые мы будем создавать для каждого аналогичного прослушивателя, которого мы реализуем, этого подхода явно следует избегать.

Во-первых, Android не определяет наилучших практик в отношении регистрации прослушивателей кликов. Это полностью зависит от вашего варианта использования.

Правильная реализация View.OnClickListener интерфейса для Activity . Поскольку Android настоятельно рекомендует реализацию интерфейса снова и снова, будь то Activity или фрагмент.

Теперь, как вы описали :

public class ActivityMain extends Activity implements View.OnClickListener
{
    private class ClickListener implements View.OnClickListener
    {   
        @Override
        public void onClick(View view)
        {
            switch (view.getId())
            {
                //handle multiple view click events
            }
        }
    }
}

Это ваш подход. Теперь это ваш способ реализации, и в этом нет ничего плохого, если вас не беспокоят затраты памяти. Но какая польза от создания внутреннего класса и реализации View.OnClickListener, если вы можете просто реализовать это в основном классе, что также может привести к ясности и простоте кода, которые вам нужны.

Так что это просто обсуждение, а не поиск наилучшего возможного решения для реализации представления.OnClickListener потому что, если вы придерживаетесь практической точки зрения каждого, вы выберете решение, которое является простым и экономичным по объему памяти.

Поэтому я бы предпочел обычный способ. Это упрощает работу. Проверьте код ниже:

@Override
public void onClick(View view)
{
    switch (view.getId())
    {
        //handle multiple view click events
    }
}

P.S : Ваш подход определенно увеличит количество строк кода : P ;)

Прежде всего, давайте разберемся с основами..

Реализуя интерфейс, ваш класс не становится таким .. как вы сказали:

"Наш подкласс Activity " - это " ClickListener, что не совсем верно".

Ваш класс может иметь отношение "Is-a" только в том случае, если он расширяется, в данном случае an Activity. Реализация интерфейса означает, что он может вести себя так, как интерфейс, установивший свой контракт.

Пример:

класс Peter расширяет Human .. означает, что Peter - человек..

класс Peter также может реализовать программист, музыкант, муж и т.д. Это означает, что Питер может вести себя так, как описано выше.

Что касается наилучшей практики, вы могли бы создать совершенно отдельный класс, который реализует OnClickListener следующим образом:

class MyListener implements View.OnClickListener{

  @Override
public void onClick(View view) {
        // do whatever you want here based on the view being passed
    }

}

И в вашем основном, Activity вы могли бы создать экземпляр MyListener и вызвать onClick() и передать в нем свое представление:

MyListener listener = new MyListener();

Button b = null;

@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    button = (Button)findViewById(R.id.button);
    listener.onClick(button);
   }

Я использую button.setOnClickListener(this); where my Activity implements View.OnClickListener, а затем получаю идентификатор Button отдельным методом. Пример смотрите ниже.:

public class MyActivity extends ActionBarActivity implements View.OnClickListener {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.YOUR_LAYOUT);

        ...

        Button myFirstButton = (Button) findViewById(R.id.YOUR_FIRST_BUTTON);
        myFirstButton.setOnClickListener(this);

        Button mySecondButton = (Button) findViewById(R.id.YOUR_SECOND_BUTTON);
        mySecondButton.setOnClickListener(this);

        ...

    }

    ...

    @Override
    public void onClick(View v) {
        Button b = (Button) v;
        switch(b.getId()) {
            case R.id.YOUR_FIRST_BUTTON:
                // Do something
                break;
            case R.id.YOUR_SECOND_BUTTON:
                // Do something
                break;
            ...
        }
    }

    ...

}

Здесь вы можете создать объект btnClickListner, и после этого вы будете вызывать этот объект btnCLickLisner, когда когда-либо захотите выполнить действия oncleck для кнопок..

Давайте предположим, что в моей деятельности у меня есть от 5 до 10 кнопок, и писать каждую кнопку отдельно onclick listner - плохая идея. Итак, чтобы преодолеть это, мы можем использовать следующее..

зарегистрируйте свои кнопки

Button button1 = (Button)findViewById(R.id.button1);
Button button2 = (Button)findViewById(R.id.button2);
Button button3 = (Button)findViewById(R.id.button3);
Button button4 = (Button)findViewById(R.id.button4);
Button button5 = (Button)findViewById(R.id.button5);

Здесь я устанавливаю список onclick для своих кнопок после щелчка

button1.setOnClickListener(btnClickListner);
button2.setOnClickListener(btnClickListner);
button3.setOnClickListener(btnClickListner);
button4.setOnClickListener(btnClickListner);
button5.setOnClickListener(btnClickListner);

Вот реализация btnClick Listner

View.OnClickListener btnClickListner = new OnClickListener()
    {
  @Override
        public void onClick( View v )
        {
            // TODO Auto-generated method stub
            if( button1.getId() == v.getId() )
            {
                //Do Button1 click operations here

            }
            else if( button2.getId() == v.getId() )
            {

               // Do Button2 click operations here

            }
            else if( button3.getId() == v.getId() )
            {
                 // Do Button3 click operations here

            }
            else if( button4.getId() == v.getId() )
            {
                // Do Button4 click operations here

            }
            else if( button5.getId() == v.getId() )
            {
                // Do Button5 click operations here
            }

        }

     }