Как сделать кликабельное меню из кнопок с помощью switch?

Question

[tema86]

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

Рабочий метод:

public class main extends AppCompatActivity {

    Button b1,b2,b3;


    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        LinearLayout l1 = new LinearLayout(this);
        l1.setOrientation(LinearLayout.VERTICAL);
        LinearLayout.LayoutParams l2 = new LinearLayout.LayoutParams(ViewGroup.LayoutParams.MATCH_PARENT, ViewGroup.LayoutParams.MATCH_PARENT);
        setContentView(l1,l2);


        b1 = (Button) findViewById(R.id.b1);
        b2 = (Button) findViewById(R.id.b2);
        b3 = (Button) findViewById(R.id.b3);


        LinearLayout.LayoutParams l3 = new LinearLayout.LayoutParams(200, ViewGroup.LayoutParams.WRAP_CONTENT);
        

        b1 = new Button(this);
        b1.setText("Button 1");
        l1.addView(b1,l3);

        b2= new Button(this);
        b2.setText("Button 2");
        l1.addView(b2,l3);

        b3 = new Button(this);
        b3.setText("Button 3");
        l1.addView(b3,l3);


        b1.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                Intent intent = new Intent(main.this,m2.class);
                startActivity(intent);
            }
        });

        b2.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                Intent intent = new Intent(main.this,m3.class);
                startActivity(intent);
            }
        });
        b3.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                Intent intent = new Intent(main.this,m4.class);
                startActivity(intent);
            }
        });
    }

и не рабочий:

public class main extends AppCompatActivity {

    Button b1,b2,b3;


    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        LinearLayout l1 = new LinearLayout(this);
        l1.setOrientation(LinearLayout.VERTICAL);
        LinearLayout.LayoutParams l2 = new LinearLayout.LayoutParams(ViewGroup.LayoutParams.MATCH_PARENT, ViewGroup.LayoutParams.MATCH_PARENT);
        setContentView(l1,l2);


        b1 = (Button) findViewById(R.id.b1);
        b2 = (Button) findViewById(R.id.b2);
        b3 = (Button) findViewById(R.id.b3);


        LinearLayout.LayoutParams l3 = new LinearLayout.LayoutParams(200, ViewGroup.LayoutParams.WRAP_CONTENT);
        

        b1 = new Button(this);
        b1.setText("Button 1");
        l1.addView(b1,l3);

        b2= new Button(this);
        b2.setText("Button 2");
        l1.addView(b2,l3);

        b3 = new Button(this);
        b3.setText("Button 3");
        l1.addView(b3,l3);


       View.OnClickListener onClickListener = (new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                switch(v.getID()) {
                    case R.id.b1:
                            Intent intent = new Intent(main.this,m2.class);
                            startActivity(intent);
                            break;
                    case R.id.b2:
                            Intent intent1 = new Intent(main.this,m3.class);
                            startActivity(intent1);
                            break;
                    case R.id.b3:
                            Intent intent2 = new Intent(main.this,m4.class);
                            startActivity(intent2);
                            break;
                }
            }
        });
    
        b1.setOnClickListener(onClickListener);
        b2.setOnClickListener(onClickListener);
        b3.setOnClickListener(onClickListener);

}

В чем может быть проблема? спасибо

Comments

[automatik]

а в чём логика b1 = (Button) findViewById(R.id.b1), после которого пишете b2= new Button(this)?

[tema86]

значит метод findView надо вообще убрать, или поставить после созданных кнопок?

[viktormarkov]

tema86: метод findViewById здесь вообще не нужен

Answer 1

[viktormarkov]

Сначала Вы присваиваете переменным b1, b2, b3 объекты Button, созданные методом findViewById

b1 = (Button) findViewById(R.id.b1);
        b2 = (Button) findViewById(R.id.b2);
        b3 = (Button) findViewById(R.id.b3);

Потом этим же переменным присваиваете новые объекты Button

b1 = new Button(this);
....

Соответственно, эти переменные ссылаются на совершенно другие кнопки, id которых сгенерирован автоматически.
Поэтому в switch условие и не срабатывает

Comments

[Денис Загаевский]

Только findViewById ничего не создает.

[viktormarkov]

Денис Загаевский: пожалуйста, поправьте, если не прав.
Метод findViewById возвращает View, соответствующий данному id. Судя по вышеизложенному коду, метод возвращает null, так как в родительском View эллементов нет.
А что касается идентификатора, то правильнее говорить, что id всегда генерируется автоматически, а мы только декларируем ссылку (R.id.*** или +id/***) для прямого доступа к id, который хранится в классе R.
Так будет вернее?
Просто я сам новичок. На практике понимаю, а как оно внутри действует, требует уточнения

[Денис Загаевский]

Да.
Нет. Айди может генерироваться, обычно так и бывает. А если руками создавать вью, то можно назначит любой айди, вью с которым ещё нет в родителе.

[viktormarkov]

Денис Загаевский: спасибо!

[tema86]

viktormarkov: ставил метод findView после созданных кнопок, все равно клик не отрабатывает.

[tema86]

viktormarkov: выходит ошибка java.lang.NullPointerException: Attempt to invoke virtual method 'void android.widget.Button.setOnClickListener(android.view.View$OnClickListener)' on a null object reference
и приложение вылетает.

[viktormarkov]

tema86: после строчек b1 = new Button(this); и т.д. Вам нужно прописать id для каждой кнопки. Это можно сделать напрямую b1.setId(123); b2.setId(456), но лучше в папку values создать xml файл с именем ids и прописать все id там (stackoverflow.com/questions/8937380/how-to-set-id-...

Потом в switch Вы проверяете те id, которые назначили кнопкам, к примеру
case 123: break; case 456: break

[viktormarkov]

tema86: а ошибка после перестановки вылетает именно из-за того, что findViewById не находит кнопок с данным id в LinearLayout. Этот метод здесь лишний вообще

[Денис Загаевский]

tema86: viktormarkov: просто не надо создавать вьюхи из кода, если не понимаете, что делаете.

[tema86]

viktormarkov: сделал все как вы сказали, приложение запускается без ошибок, но попрежнему перехода в друие активити по клику нет.

[tema86]

viktormarkov: метод findByView удалил

[tema86]

viktormarkov: id создавал через ids.xml

[viktormarkov]

tema86: Продублировал код у себя - все работает

public class Main extends AppCompatActivity {

    Button b1, b2, b3;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        LinearLayout l1 = new LinearLayout(this);
        l1.setOrientation(LinearLayout.VERTICAL);
        LinearLayout.LayoutParams l2 = new LinearLayout.LayoutParams(ViewGroup.LayoutParams.MATCH_PARENT, ViewGroup.LayoutParams.MATCH_PARENT);
        setContentView(l1,l2);


        LinearLayout.LayoutParams l3 = new LinearLayout.LayoutParams(200, ViewGroup.LayoutParams.WRAP_CONTENT);


        b1 = new Button(this);
        b1.setText("Button 1");
        b1.setId(R.id.button1);
        l1.addView(b1,l3);

        b2= new Button(this);
        b2.setText("Button 2");
        b2.setId(R.id.button2);
        l1.addView(b2,l3);

        b3 = new Button(this);
        b3.setText("Button 3");
        b3.setId(R.id.button3);
        l1.addView(b3,l3);


        View.OnClickListener onClickListener = new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                switch(v.getId()) {
                    case R.id.button1:

                        Toast.makeText(Main.this, "button 1", Toast.LENGTH_SHORT).show();

                        break;
                    case R.id.button2:

                        Toast.makeText(Main.this, "button 2", Toast.LENGTH_SHORT).show();

                        break;
                    case R.id.button3:

                        Toast.makeText(Main.this, "button 3", Toast.LENGTH_SHORT).show();

                        break;
                }
            }
        };

        b1.setOnClickListener(onClickListener);
        b2.setOnClickListener(onClickListener);
        b3.setOnClickListener(onClickListener);
    }
}

res/values/ids.xml

<?xml version="1.0" encoding="utf-8"?>
<resources>
    <item type="id" name="button1" />
    <item type="id" name="button2" />
    <item type="id" name="button3" />
</resources>

[tema86]

viktormarkov: Прошу прощения, все работает, сделал ids.xml, но не создал setID.
спасибо Огромное за помощь.