爲什麼第一次點擊後隨機按鈕遊戲存在缺陷？

Question

我有一個隨機值應用於四個隨機按鈕的程序。其中一個隨機值id顯示在四個按鈕的頂部。如果用戶點擊包含顯示號碼的按鈕（loadG4），他們會得到一個點。我遇到的問題是，一旦用戶點擊了正確的按鈕並生成了新的數字，rbvalue按鈕可能並不全都與loadG4的值不同。從我的代碼中可以看出，它們不應該是平等的。下面的代碼：

int score = 0; 
Random random = new Random(); 
int rbselector = random.nextInt(4); //These four initiations are out onCreate. 
int loadG4 = random.nextInt(10); 


final Button[] selectrb = new Button[4]; 
    selectrb[0] = rb1; 
    selectrb[1] = rb2; 
    selectrb[2] = rb3; 
    selectrb[3] = rb4; 


    final Button loseStarter4 = (Button) findViewById(R.id.Starter4); 
    loseStarter4.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View v) { 
      infoG4.setVisibility(View.GONE); 
      loseStarter4.setVisibility(View.GONE); 
      rb1.setVisibility(View.VISIBLE); 
      rb2.setVisibility(View.VISIBLE); 
      rb3.setVisibility(View.VISIBLE); 
      rb4.setVisibility(View.VISIBLE); 


      rbselector = random.nextInt(4); 

      final TextView number = (TextView) findViewById(R.id.number); 
      number.setText(""+ loadG4); 


      for(int allrbA=0; allrbA<4; allrbA++) { 
       int rbvalue = random.nextInt(9); 
       if (rbvalue == loadG4) { 
        rbvalue=9; 
       } 
       selectrb[allrbA].setText(""+rbvalue); 
      } 
      selectrb[rbselector].setText(""+ loadG4); 


       for (int allrbA = 0; allrbA < 4; allrbA++) { 
       selectrb[allrbA].setOnClickListener(new View.OnClickListener() { 
        @Override 
        public void onClick(View v) { 
          Button clicked = (Button) v; 
          String clickVal = (String) clicked.getText(); 
          int finalClick = Integer.valueOf(clickVal); 

          if (finalClick == loadG4) { 
           score++; 
           for (int allrbA = 0; allrbA < 4; allrbA++) { 
            int rbvalue = random.nextInt(9); 
            if (rbvalue == loadG4) { 
             rbvalue=9; 
            } 
            selectrb[allrbA].setText("" + rbvalue); 
           } 
           int loadG4 = random.nextInt(10); 
           number.setText("" + loadG4); 
           int rbselector = random.nextInt(4); 
           selectrb[rbselector].setText("" + loadG4); 
          } 
        } 
       }); 
       } 

我有它這樣，如果一個rbvalue等於loadG4，它被設置爲= 9。出於某種原因，之後的第一個正確的點擊有可能是一個rbvalue可以等於loadG4。

此外，loadG4的值在這裏也不會改變。爲什麼是這樣？

感謝任何人都可以通過這個解釋並解釋爲什麼a）rbvalue在第一次正確點擊後並不總是與loadG4不同，b）爲什麼loadG4保持相同的數字。

非常感謝提前。

Answer 1

在你的代碼中，你正在初始化loadG4和rbselector。相反，你應該使用全局變量。

//Declare this globally. Not in oncreate. 
int score = 0; 
Random random = new Random(); 
int rbselector = random.nextInt(4); 
int loadG4 = random.nextInt(10); 


final Button[] selectrb = new Button[4]; 
selectrb[0] = rb1; 
selectrb[1] = rb2; 
selectrb[2] = rb3; 
selectrb[3] = rb4; 


final Button loseStarter4 = (Button) findViewById(R.id.Starter4); 
loseStarter4.setOnClickListener(new View.OnClickListener() { 
    @Override 
    public void onClick(View v) { 
     infoG4.setVisibility(View.GONE); 
     loseStarter4.setVisibility(View.GONE); 
     rb1.setVisibility(View.VISIBLE); 
     rb2.setVisibility(View.VISIBLE); 
     rb3.setVisibility(View.VISIBLE); 
     rb4.setVisibility(View.VISIBLE); 


     rbselector = random.nextInt(4); 

     final TextView number = (TextView) findViewById(R.id.number); 
     number.setText(""+ loadG4); 


     for(int allrbA=0; allrbA<4; allrbA++) { 
      int rbvalue = random.nextInt(9); 
      if (rbvalue == loadG4) { 
       rbvalue+=1; 
      } 
      selectrb[allrbA].setText(""+rbvalue); 
     } 
     selectrb[rbselector].setText(""+ loadG4); 


      for (int allrbA = 0; allrbA < 4; allrbA++) { 
      selectrb[allrbA].setOnClickListener(new View.OnClickListener() { 
       @Override 
       public void onClick(View v) { 
         Button clicked = (Button) v; 
         String clickVal = (String) clicked.getText(); 
         int finalClick = Integer.valueOf(clickVal); 

         if (finalClick == loadG4) { 
          score++; 
          loadG4 = random.nextInt(10); //removed declaration 
          for (int allrbA = 0; allrbA < 4; allrbA++) { 
           int rbvalue = random.nextInt(9); 
           if (rbvalue == loadG4) { 
            rbvalue+=1; // better method to prevent duplicates for 9 
           } 
           selectrb[allrbA].setText("" + rbvalue); 
          } 
          number.setText("" + loadG4); 
          rbselector = random.nextInt(4); 
          selectrb[rbselector].setText("" + loadG4); 
         } 
       } 
      }); 
      } 

現在你的代碼應該工作。