Булевая петля в сочетании с JOptionPane Java

Question

Я хотел бы поделиться своим кодом. Но в моем университете код тестируется на «обман».

Но вот мой код в более простой форме.

public static String readin() { 
    boolean error = false; 
    do { 
     string stringin; 
     stringin = JOptionPane.showInputDialog(null, "Please enter a number"); 
     switch (stringin.length()) { 
      case 0: 
       JOptionPane.showMessageDialog(null, "Error Please repeat"); 
       error = true; 

      case 1: 
       return stringin; 
     } 
     return null; 
    } while (error == true); 
} 

Этот код действительно в самой простой форме. Я знаю, что для этого случая было бы разумнее установить время, пока JOptionPane пуст или что-то еще. Так как в моем коде есть 12 различных ошибок. Я хочу использовать boolean. Пожалуйста: нулевой символ возврата никогда не встретится в реальном коде.

Но реальная проблема у меня есть: она отлично работает, кроме того: если он повторяет цикл, он не дает мне возможность набрать новую строку. Как я могу это сделать?

Также я прошу прощения за свои ошибки на английском языке.

EDIT: Все ваши исправления устраняют мои проблемы! Большое спасибо! Мне нравится этот форум!

Answer 1

Попробуйте с перерыв перед тем case 1 (@Tuxxy_Thang) и удалить return null; перед тем while (error); и положить после.

public static String readin(){ 
    boolean error=false; 
    do{ 
     string stringin; 
     stringin=JOptionPane.showInputDialog(null,"Please enter a number"); 
     switch (stringin.length()){ 
      case 0: JOptionPane.showMessageDialog(null, "Error Please repeat"); 
       error=true; 
       break; 
      case 1: return stringin; 
     } 
    } while (error); 
    return null; 
} 

Answer 2

Вы не должны проверить в то время как условия, как вы хотите, чтобы пользователь повторить again.Return только если у вас есть правильное значение еще раз спросить

public static String readin() { 
     while (true) { 
      String stringin = JOptionPane.showInputDialog(null, "Please enter a number"); 
      switch (stringin.length()) { 
       case 0: 
        JOptionPane.showMessageDialog(null, "Error Please repeat"); 
        break;//is important in switch cases 
       case 1: 
        return stringin; 
      } 
     } 
    } 

Answer 3

Есть две причины:

• return null следует переместить в конец.
• Там должен быть break заявления в конце первого случая

Я дал измененный код и он повторяет цикл, если пользователь не вводит ничего для JOptionPane:

public static String readin() { 
    boolean error = false; 
    do { 
     String stringin; 
     stringin = JOptionPane.showInputDialog(null, 
       "Please enter a number"); 
     switch (stringin.length()) { 
     case 0: 
      JOptionPane.showMessageDialog(null, "Error Please repeat"); 
      error = true; 
      break;      // **added** 
     case 1: 
      return stringin; 
     } 
    } while (error == true); 
    return null; // Moved here. It will return if user entered more than 1 letter. 
} 

Answer 4

вы делаете простую работу трудно, использовать классы Java должным образом, и вы будете иметь более легкое время вещей

public static String readin() 
{ 
    Boolean gettingNumber = true; 
    String stringin = null; 

    while (gettingNumber) 
    { 
     stringin = JOptionPane.showInputDialog(null, "Please enter a number"); 

     try 
     { 
      Integer number = Integer.parseInt(stringin); 
      gettingNumber = false; 
     } 
     catch (NumberFormatException exception) 
     { 
      // no need to do anything 
     } 

    } 

    System.out.println("returning [" + stringin + "]"); 
    return(stringin); 
} 

Я добавил это потому, что я вы указали, что вы просите пользователя ввести номер, но ваш код разрешит пользователю вводить какой-либо один символ независимо от того, является ли он действительной цифрой или нет, и это то, что вы вернете, из своего исходного метода.