Question

I am kinda new to Android and it would improve my application a lof if I coul keep several OnClickListenres in one class. What I am thiking of is something like this :

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);
                }
        };
    }
}

So I can then reference it in other class B like this

button1.setOnClickListener(OnClickListeners.open);

Any though how to do it?
Android SDK seems to be against me as I can figure it out now for about 2 days now…

Thanks for any advices and help

Answer 1

There is a sleek way to consolidate all your anonymous classes into one and switch on the view. This works best if you know ahead of time which buttons will be using the 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

        }

    }
}