Question

我试图构建这个测验程序，其中一个人被问到一个问题，然后必须回答，这是一个两个人的游戏，他们只是通过所有问题，游戏结束。

到目前为止我设法做的是将数组（strings.xml文件中的数组）加载到arraylist然后从0到arraylist.size的随机id然后显示该字符串的通用代码（问题）然后删除随机ID并采用新的。

但是当我运行arraylist中的所有id时程序崩溃，我的代码看起来像这样：http://pastebin.com/MaEj49BL

重要的是这个：

public void gameCode()
{
    setContentView(R.layout.game);
    if (mList == null) 
{
     String[] mQuestions = getResources().getStringArray(R.array.mQuestions);
     mList = new ArrayList();
     Collections.addAll(mList, mQuestions);
}
     else if (mList.isEmpty()) 
{
     m = (TextView)findViewById(R.id.textView1);
         m.setText("You've run out of questions!");
}
if (wList == null) 
{
   String[] wQuestions = getResources().getStringArray(R.array.wQuestions);
   wList = new ArrayList();
   Collections.addAll(wList, wQuestions);
} 
else if (wList.isEmpty()) 
{
    w = (TextView)findViewById(R.id.textView1);
            w.setText("You've run out of questions!");
}

    //Takes a random ID from the arraylist
    int rndm = generator.nextInt(mList.size());
    int rndw = generator.nextInt(wList.size());


    //Sets the Strings to a random number from one of the array lists
    String qMan = (String) mList.get(rndm);
    String qWoman = (String) wList.get(rndw);
    if(i == 0)
        {
            //Defines that w is textView1 with the value of qWoman
            w = (TextView)findViewById(R.id.textView1);
            w.setText(qWoman);
            wList.remove(rndw);
            i = 1;
            final Button buttonNext = (Button) findViewById(R.id.buttonNext);
            buttonNext.setOnClickListener(new View.OnClickListener() 
            {
                public void onClick(View v) 
                {
                    gameCode();
                }
            });
        }
    else
        {

            m = (TextView)findViewById(R.id.textView1);
            m.setText(qMan);
            mList.remove(rndm);
            i = 0;
            final Button buttonNext = (Button) findViewById(R.id.buttonNext);
            buttonNext.setOnClickListener(new View.OnClickListener() 
            {
                public void onClick(View v) 
                {
                    gameCode();
                }
            });
       }

}

}

来自pastebin的源代码中有注释。现在我只需要将文本更改为“你已经用完了问题”

Answer 1

您正在检查2个数组是否为空，但是当它们是......时，您不会将逻辑的其余部分短路...

如果mList和wList为空，则应该绕过方法的其余部分，或者至少在尝试从每个数组中获取下一个随机问题之前检查mList.size() > 0 and wList.size()>0。

Answer 2

使用null安全的CollectionUtils.isEmpty（）

Android / Java：检查arraylist是否为空，然后启动另一个活动

2 个答案: