0

你好，有一个小问题，我不知道出了什么问题......代码片段：

do{
      String third = JOptionPane.showInputDialog(null, "Please enter Year ["+day+"/"+month+"/YY?]");
      year = Integer.parseInt (third);
      validChoice = validYear(year);
   }while(!validChoice);

do{
      String goUS = JOptionPane.showInputDialog(null, "Do you want to switch the format to US? [Y/N]");
      goUS = goUS.toUpperCase();
      char conversion = goUS.charAt(0);
      validChoice = validOption(conversion);


      switch(conversion){
          case 'Y':

              if(year < 10)
               {
                  date.append(month).append("/").append(day).append("/" + "200").append(year);
               }else{
                  date.append(month).append("/").append(day).append("/" + "20").append(year);
              }
              JOptionPane.showMessageDialog(null, date);
              break;

          case 'N':

              if(year < 10)
               {
                  date.append(day).append("/").append(month).append("/" + "200").append(year);
               }else{
                  date.append(day).append("/").append(month).append("/" + "20").append(year);
              }
              JOptionPane.showMessageDialog(null, date);
              break;

          default:
              JOptionPane.showMessageDialog(null, "Invalid answer! Use Y/N", "Error", JOptionPane.ERROR_MESSAGE);
       }}while(!validChoice);

//// METHODS:

public static boolean validYear (int year){
    boolean isValid = true;

    if(year < 1 || year > 99)
     {
      isValid = false;
     }

    return isValid;
}

public static boolean validOption (char conversion){
    boolean isValid = true;

    if(conversion != 'y' || conversion != 'n')
     {
      isValid = false;
     }

    return isValid;
}

第一部分，关于年份，以及它的相关方法，如果输入错误，它会很好地循环。

第二部分，“goUS”，该方法远程相同，检查字符是否为 y/n - 它在检查和所有方面工作正常，但在正确显示日期点击 OK 后，它开始goUS 再次循环，答案开始从前一个显示中连接起来，依此类推。我希望应用程序在显示日期后结束。

任何指出我的缺陷的帮助将不胜感激。

4

3 回答 3

1

好吧，您validOption检查小写字符，但您将其传递给大写字符。

最灵活的解决方案是在与预期值进行比较之前对案例进行规范化，例如

public static boolean validOption (char conversion) {
    conversion = Character.toLowerCase(conversion)
    return conversion == 'y' || conversion == 'n';
}
于 2013-07-25T12:23:42.653 回答
0

你的validOption()工作不正常。循环其余部分的工作与此无关，但终止取决于正确的返回值。

您在那里检查小写字符，但在其他地方检查大写。您在有效性检查之前将检查的字符更改为大写。也将字符更改为大写validOption()。

于 2013-07-25T11:51:59.440 回答
0

问题似乎是由于输入字符的大小写，您validOption在主方法中检查小写并检查大写。我相信您的输入是大写的，并且validOption返回 false。

更改validOption为：

public static boolean validOption (char conversion) {
    return conversion == 'Y' || conversion == 'N';
}

甚至使用java.lang.Character toLower或toUpper方法。

于 2013-07-25T12:17:49.933 回答