java.lang.StringIndexOutOfBoundsException 和 showInputDialog

Question

这是一个家庭作业问题。我正在开发一个程序，该程序使用JOptionPane 来计算选票，直到输入 Q 或 q 并且计票停止。然后它会启动一个showConfirmDialog，询问用户是否真的想退出。程序的主要任务已经完成，但是其中一条指令指出，在 showInputDialog 上，任何不是 Y、y、N、n、Q 或 q 的条目都将被忽略。

我遇到了这个问题，因为出于某种原因，无论何时单击对话框上的任何按钮，程序都会关闭。该程序的其余部分工作正常，但是当我运行它时，它会给出一个编译错误通知，我最终遇到了这个问题。

这是我的代码。知道这里发生了什么吗？感谢您的任何帮助。

编辑 - 你知道，System.out.println("HI"); 行就在那里，所以我可以确定问题是否出在该位置的 continue 语句的整个 if 语句上。

编辑 #2 - 我使用您的建议（至少我如何解释它）在第一个程序（现在是 VoteCount2）下方发布了我对代码的编辑。

编辑 #3 - 我重新格式化并尝试听从您的建议。如果我做错了什么，我很抱歉，但我不知道该怎么办。

编辑 #4 - 这是我的工作方式..

Do { String voteString = ""; 
voteString = JOptionPane.showInputDialog(null, "Enter 'Y' to vote yes, 'N' to vote no, or 'Q' to quit voting", "Vote Now!", JOptionPane.OK_CANCEL_OPTION);
 if (voteString == null || voteString.length() == 0) continue;
 else if(voteString.toLowerCase().equals("y")) 
yesVotes++;
 else if (voteString.toLowerCase().equals("n")) 
noVotes++;

旧程序的其余部分

           package javaapplication16;

           import javax.swing.JOptionPane;

           public class VoteCount {

           public static void main(String[] args) {

           int yesVotes = 0, noVotes = 0, totalVotes = 0;
           totalVotes = yesVotes + noVotes;
           char vote;
           int reply = 1;
           int option = 0;
           {
           do {
           String voteString = JOptionPane.showInputDialog(null, "Enter 'Y' to vote yes, 'N'  to vote no, or 'Q' to quit voting", "Vote Now!", JOptionPane.OK_CANCEL_OPTION);
           vote = voteString.charAt(0);
           if (vote == 'Y' || vote == 'y')
           yesVotes++;
           if (vote == 'N' || vote == 'n')
           noVotes++;
           if (option == JOptionPane.OK_OPTION)
           System.out.println("HI");
           if (option == JOptionPane.CANCEL_OPTION)
           continue;
           if (option == JOptionPane.CLOSED_OPTION)
           continue;
             {
            }

           if (vote == 'Q' || vote == 'q')
           {
           reply = JOptionPane.showConfirmDialog(null, "Quit?", "Warning!", JOptionPane.YES_NO_OPTION);
           if (reply == JOptionPane.YES_OPTION)
              JOptionPane.showMessageDialog(null, "yes " + yesVotes + " no " + noVotes + " total " + totalVotes);
           else if (reply != JOptionPane.YES_OPTION);
    continue;
            }
           } while (reply != JOptionPane.YES_OPTION);

           }
        }
    }


package javaapplication16;

import javax.swing.JOptionPane;

public class VoteCount {

public static void main(String[] args) {

int yesVotes = 0, noVotes = 0, totalVotes = 0;
totalVotes = yesVotes + noVotes;
char vote;
    int reply = 1;
{
    do {
 String voteString = JOptionPane.showInputDialog(null, "Enter 'Y' to vote yes, 'N' to vote no, or 'Q' to quit voting", "Vote Now!", JOptionPane.OK_CANCEL_OPTION);
 vote = voteString.charAt(0);
 if (voteString == null || voteString.length() == 0)
vote = 'q';
if (voteString.toLowerCase().equals("y"))
     yesVotes++;
if (voteString.toLowerCase().equals("n"))
        noVotes++;

        {
    }

if (voteString.toLowerCase().equals("q"))
{
    reply = JOptionPane.showConfirmDialog(null, "Quit?", "Warning!", JOptionPane.YES_NO_OPTION);
    if (reply == JOptionPane.YES_OPTION)
        JOptionPane.showMessageDialog(null, "yes " + yesVotes + " no " + noVotes + " total " + totalVotes);
    else if (reply != JOptionPane.YES_OPTION);
continue;
        }
    } while (reply != JOptionPane.YES_OPTION);

    }
    }
}

Answer 1

无论字符串中是否有任何内容（例如单击按钮时），您都尝试从字符串中获取一个字符。一种选择是先检查按钮是否按下。

另一种方法是对照"y" 或"n" 检查小写字符串，例如：

if (voteString.toLowerCase().equals("y")) {
    // etc.

当用户退出对话框时，您仍然需要处理null 返回值。如果他们选择“确定”，则它不会是null，但它将为空，从而导致越界异常。

Answer 2

我在这里看到的唯一问题是，如果您按“取消”，则会出现 NullPointerException，因为在这种情况下对话框将返回 null 作为输入，但您仍然尝试获取第一个字符。否则它应该（并且确实——我编译并运行它）工作得很好。

看看这两行：

String voteString = JOptionPane.showInputDialog(null, "Enter 'Y' to vote yes, 'N'  to vote no, or 'Q' to quit voting", "Vote Now!", JOptionPane.OK_CANCEL_OPTION);
vote = voteString.charAt(0);

有两个可能的问题。首先，假设用户按下“取消”。在这种情况下，showInputDialog() 返回null，所以voteString 是null。这意味着只要调用voteString.charAt(0) 就会抛出NullPointerException。您必须首先检查null - 即，

String voteString = JOptionPane.showInputDialog(null, "Enter 'Y' to vote yes, 'N'  to vote no, or 'Q' to quit voting", "Vote Now!", JOptionPane.OK_CANCEL_OPTION);
if (voteString == null) {
    // User has cancelled; let's quit
    System.exit(0);
}

第二个问题是如果您不输入任何内容就按“确定”会发生什么。在这种情况下，voteString 的长度为零，调用charAt(0) 将抛出IndexOutOfBoundsException，因为索引 0 处没有字符。同样，您需要检查这一点。添加到前面的示例：

String voteString = JOptionPane.showInputDialog(null, "Enter 'Y' to vote yes, 'N'  to vote no, or 'Q' to quit voting", "Vote Now!", JOptionPane.OK_CANCEL_OPTION);
if (voteString == null) {
    // User has cancelled; let's quit
    System.exit(0);
} else if (voteString.length() == 0) {
    // This will just skip to the next go-round of the do/while loop, and prompt again
    continue;
}

Answer 3

一些事情：

正如其他人所提到的，您没有正确处理没有输入文本的情况。具体来说，如果您单击“确定”，则 voteString 将是该字段中的任何内容（即空字符串）。如果单击取消，则 voteString 将为空。

如果检测到以下任一情况，您可以通过将 voteString 设置为默认值来处理此问题：

if (voteString == null || voteString .equals("")) voteString = "q";

其他一些事情：您永远不会更新 totalVotes。

你可以使用

vote = Character.toLowerCase(voteString.charAt(0))

获取输入的字符。您甚至可以使用不同类型的对话框（允许选择是/否/取消按钮），而不是输入文本。

如果您可以使用 switch(vote) 块在输入的文本之间进行选择。