我有 5 个字符串,有些正在动态变化,但有些不能

i have 5 strings some are changing dynamically but some can't

实际上我是编程新手,但我尝试了一个简单的数学游戏。在这个程序中,我让 s1、s2、s3、s4 和 s5 动态生成。但只有 s1、s2 正在改变,如果方法再次调用,s3、s4、s5 不会改变……请帮助我。

public class MainActivity extends Activity implements OnClickListener {
Button b1, b2, b3;
ImageView i, i2, i3;
TextView t1, t2, t3;
String s1, s2, s3, s4, s5;

int m, r2, r3, r4, r5, r6, r7, r8, r9, r10, r11, r12, r13, n[];

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);
    b1 = (Button) findViewById(R.id.button1);
    b2 = (Button) findViewById(R.id.button2);
    b3 = (Button) findViewById(R.id.button3);

    i = (ImageView) findViewById(R.id.imageView1);
    t1 = (TextView) findViewById(R.id.textView1);
    t2 = (TextView) findViewById(R.id.textView2);
    t3 = (TextView) findViewById(R.id.textView3);
    b1.setOnClickListener(this);
    b2.setOnClickListener(this);
    b3.setOnClickListener(this);
    imp();

}

public void imp() {
    Random r1 = new Random();
    r2 = r1.nextInt(15) + 1;
    r3 = r1.nextInt(15) + 1;
    r4 = r2 + r3;
    r5 = r2 - r3;
    r6 = r2 * r3;

    n = new int[] { r4, r5, r6 };

    while (r9 == r11 || r9 == r13 || r11 == r13) {
        r8 = r1.nextInt(3) + 0;
        r9 = n[r8];
        r10 = r1.nextInt(3) + 0;
        r11 = n[r10];
        r12 = r1.nextInt(3) + 0;
        r13 = n[r12];
    }
    s1 = Integer.toString(r2);
    s2 = Integer.toString(r3);
    s3 = Integer.toString(r9);
    s4 = Integer.toString(r11);
    s5 = Integer.toString(r13);
    t1.setText(s1);
    t2.setText(s2);
    b1.setText(s3);
    b2.setText(s4);
    b3.setText(s5);

    int[] images = { R.drawable.plus, R.drawable.minus, R.drawable.multiply };

    Random random = new Random();
    m = images[random.nextInt(images.length)];
    i.setImageResource(m);
}

public void click(int k, Button b) {
    if (m == R.drawable.plus && k == r4 || m == R.drawable.minus && k == r5
            || m == R.drawable.multiply && k == r6) {

        imp();

    } else {
        t3.setText("FINISH");
        t3.setVisibility(View.VISIBLE);
        b.setPressed(true);
        b1.setClickable(false);
        b2.setClickable(false);
        b3.setClickable(false);

    }
}

@Override
public void onClick(View v) {
    switch (v.getId()) {
    case R.id.button1:
        click(r9, b1);

        break;
    case R.id.button2:
        click(r11, b2);
        break;
    case R.id.button3:
        click(r13, b3);
        break;
    }
  }
}

布局为

<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
android:layout_width="fill_parent"
android:layout_height="fill_parent"
android:gravity="center_horizontal|center_vertical"
android:orientation="vertical" >

<LinearLayout
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    android:orientation="horizontal" >

<TextView
    android:id="@+id/textView1"
    android:layout_width="wrap_content"
    android:layout_height="46dp"
    android:layout_weight="0.97"
    android:editable="false"
    android:text="Large Text"
    android:gravity="center"
    android:textAppearance="?android:attr/textAppearanceLarge" />

<ImageView
    android:layout_weight="1"
    android:id="@+id/imageView1"
    android:layout_width="67dp"
    android:layout_height="29dp"
    android:src="@drawable/plus" />

<TextView
    android:id="@+id/textView2"
    android:layout_width="wrap_content"
    android:layout_height="match_parent"
    android:layout_weight="1.00"
    android:text="Large Text"
    android:gravity="center"
    android:textAppearance="?android:attr/textAppearanceLarge" />

</LinearLayout>

<LinearLayout
    android:paddingTop="50dp"
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    android:orientation="horizontal" >
 <Button
      android:layout_weight="1"
    android:id="@+id/button1"
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:text="Button" />
<Button
     android:layout_weight="1"
    android:id="@+id/button2"
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:text="Button" />

<Button
     android:layout_weight="1"
    android:id="@+id/button3"
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:text="Button" />

</LinearLayout>

<TextView
android:id="@+id/textView3"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:paddingTop="50dp"
android:text="Large Text"
android:textAppearance="?android:attr/textAppearanceLarge"
android:textSize="50dp" 
android:visibility="invisible"/>

</LinearLayout>

看看你的 while 循环,你第二次 运行 通过那个方法,条件不可能为真,因为上次的结果被记住了。对于这些答案,我建议使用局部变量而不是字段。您可以通过将 r9、r11 和 r13 中的每一个设置为 0 来继续使用字段;但是,没有充分的理由让它们成为字段。在设置按钮文本之前,您只需要这些变量。现在,对于局部变量,您确实需要将它们初始化为 0,默认情况下它们没有像字段那样的值。

如果您不介意我也主动提出一些建议,我建议您使用更具描述性的变量名称。例如:

while (result1 == result2 || result1 == result3 || result2 == result3) {
    randomIndex1 = random.nextInt(3) + 0;
    result1 = resultArray[randomIndex1];
    randomIndex2 = random.nextInt(3) + 0;
    result2 = resultArray[randomIndex2];
    randomIndex3 = random.nextInt(3) + 0;
    result3 = resultArray[randomIndex3];
}

这会导致其他人更容易理解代码。 (以及未来的自己!)