我试图以编程方式设置5个(五个)按钮的ID,并每次对其进行随机播放。
随机播放没有问题。
当我使用2(两种)随机播放的方法并设置5个按钮的id并设置每个按钮的文本时,会出现错误。
我知道这可能是一团糟,因为这是我第一次尝试for-each循环。
请帮忙。谢谢。
Button b1, b2, b3, b4, b5;
Button[] buttons = { b1, b2, b3, b4, b5 };
public void shuffleButtons() {
Integer[] Id = { R.id.bChoice1, R.id.bChoice2, R.id.bChoice3,
R.id.bChoice4, R.id.bChoice5 };
ArrayList<Integer> buttonId = new ArrayList<Integer>(Arrays.asList(Id));
Collections.shuffle(buttonId);
for (int x = 0; x < 5; x++) {
for (Button b : buttons) {
b = (Button) findViewById(buttonId.get(x));
}
}
}
public void setButtonTxt() {
for (Button b : buttons) {
for (int x = 0; x <= buttons.length; x++) {
b.setText(textList.get(x));
}
}
}
最佳答案
简而言之,我猜您的问题是x <= buttons.length
应该在您的for循环中为x < buttons.length
。但是,您似乎要使用当前代码将所有5个按钮都设置为textList.get(5)
。
为什么不尝试使用XML中的预设按钮?
<Button android:id="@+id/bChoice1" ... />
<!-- etc -->
<Button android:id="@+id/bChoice5" ... />
然后在后面的代码中,将文本随机化,而不是两者都随机化?
Button b1 = (Button)findViewById(R.id.bChoice1);
Button b2 = (Button)findViewById(R.id.bChoice2);
Button b3 = (Button)findViewById(R.id.bChoice3);
Button b4 = (Button)findViewById(R.id.bChoice4);
Button b5 = (Button)findViewById(R.id.bChoice5);
// assuming textList is an ArrayList of text items
// the next two lines will randomize your textList order
// so you don't need to do it yourself and much less
// error-prone
long seed = System.nanoTime();
Collections.shuffle(textList, new Random(seed));
b1.setText(textList.get(0));
b2.setText(textList.get(1));
b3.setText(textList.get(2));
b4.setText(textList.get(3));
b5.setText(textList.get(4));