makeText не работал над моей первой игрой в студии Android

Question

Я создал свою первую математическую игру, все работает нормально, но makeText не работает. Когда я нажимаю на ответ, ничего не происходит, не получается "Well Done!" или "Sorry that's wrong".

import android.os.Bundle; 
import android.app.Activity; 
import android.view.View; 
import android.widget.Button; 
import android.widget.TextView; 
import android.widget.Toast; 


public class GameActivity extends Activity implements View.OnClickListener { 

    int correctAnswer; 
    Button buttonObjectChoice1; 
    Button buttonObjectChoice2; 
    Button buttonObjectChoice3; 


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



//ovde inicijalizujem sve varijable 


     int partA = 9; 
     int partB = 9; 
     int correctAnswer = partA * partB; 
     int wrongAnswer1 = correctAnswer + 1; 
     int wrongAnswer2 = correctAnswer - 1; 

    /*ovde povezujem objekte sa odgovarajucim UI-em*/ 


     TextView textObjectPartA = (TextView) findViewById(R.id.textPartA); 
     TextView textObjectPartB = (TextView) findViewById(R.id.textPartB); 
     Button buttonObjectChoice1 =(Button) findViewById(R.id.buttonChoice1); 
     Button buttonObjectChoice2 =(Button) findViewById(R.id.buttonChoice2); 
     Button buttonObjectChoice3 =(Button) findViewById(R.id.buttonChoice3); 



/*sada cu koristiti setText metodu za moje nove objekte 
da pokazem vrednost na Ui dizajnu. 
Just like when we output to the console in the exercise - 
Expressions in Java, only now we use setText method 
to put the values in our variables onto the actual UI. */ 


     textObjectPartA.setText("" + partA); 
     textObjectPartB.setText("" + partB); 


//which button receives which answer, at this stage is arbitrary. 


     buttonObjectChoice1.setText("" + correctAnswer); 
     buttonObjectChoice2.setText("" + wrongAnswer1); 
     buttonObjectChoice3.setText("" + wrongAnswer2); 



    } 


    @Override 
    public void onClick(View v) { 


//declare a new int to be used in all the cases 


     int answerGiven=0; 
     switch (v.getId()) { 
      case R.id.buttonChoice1: 


//initialize a new int with the value contained in buttonObjectChoice1 
//Remember we put it there ourselves previously 


       answerGiven = Integer.parseInt("" + buttonObjectChoice1.getText()); 


//is it the right answer? 



       if(answerGiven==correctAnswer) 
       {//yay it's the right answer 
        Toast.makeText(getApplicationContext(), "Well done!", 
          Toast.LENGTH_LONG).show(); 
       } 
       else{ 



//uh oh! 



        Toast.makeText(getApplicationContext(),"Sorry that's wrong", Toast.LENGTH_LONG).show(); 
       } 
       break; 

      case R.id.buttonChoice2: 


//same as previous case but using the next button 


       answerGiven = Integer.parseInt("" + buttonObjectChoice2.getText()); 
       if(answerGiven==correctAnswer) { 
        Toast.makeText(getApplicationContext(), "Well done!", Toast.LENGTH_LONG).show(); 
       } 
       else{ 
        Toast.makeText(getApplicationContext(),"Sorry that's wrong", Toast.LENGTH_LONG).show(); 
       } 
       break; 
      case R.id.buttonChoice3: 

       //same as previous case but using the next button 
       answerGiven = Integer.parseInt("" + buttonObjectChoice3.getText()); 
       if(answerGiven==correctAnswer) { 
        Toast.makeText(getApplicationContext(), "Well done!", Toast.LENGTH_LONG).show(); 
       } 
       else    { 
        Toast.makeText(getApplicationContext(),"Sorry that's wrong", Toast.LENGTH_LONG).show(); 
       } 
       break; 
     } 




    } 



} 

Answer 1

Я знаю, почему это не работает. Поскольку вы новичок в Android, я также дам вам несколько советов.

Во-первых, вы случайно определили 2 объекта Button для каждой из ваших кнопок. Прежде всего, определить их как переменные в верхней части Активность:

Button buttonObjectChoice1; 
Button buttonObjectChoice2; 
Button buttonObjectChoice3; 

Тогда можно определить их как локальные переменные внутри OnCreate():

Button buttonObjectChoice1 =(Button) findViewById(R.id.buttonChoice1); 
    Button buttonObjectChoice2 =(Button) findViewById(R.id.buttonChoice2); 
    Button buttonObjectChoice3 =(Button) findViewById(R.id.buttonChoice3); 

Вы должны иметь только ваши кнопки в качестве элемента ИЛИ локального переменные, не оба. Для того, чтобы сохранить их как только переменные-члены, установить их идентификаторы таким образом:

buttonObjectChoice1 =(Button) findViewById(R.id.buttonChoice1); 
    buttonObjectChoice2 =(Button) findViewById(R.id.buttonChoice2); 
    buttonObjectChoice3 =(Button) findViewById(R.id.buttonChoice3); 

Если вы заметили, что не отображается ваш Тост, ваш первый заказ бизнеса должен быть, чтобы проверить, что ваш метод OnClick() на самом деле является вообще называется. Вы можете сделать это, поставив точку останова в первой строке OnClick() или вставив оператор журнала в первую строку OnClick(). Нажмите кнопку, и если ваш журнал никогда не появится, вы теперь знаете, что это проблема с настройкой OnClick, а не с Тостами.

Недостаточно использовать инструмент View.OnClickListener для правильного использования метода OnClick. Вам также необходимо подключить этот прослушиватель к каждому элементу представления, которому вы хотите «слушать». Вы можете сделать это после того, как вы называете findViewById() для каждой из кнопок:

buttonObjectChoice1.setOnClickListener(this); 
    buttonObjectChoice2.setOnClickListener(this); 
    buttonObjectChoice3.setOnClickListener(this); 

Вы передаете в this, потому что текущая активность является тот, который определяет слушателя.