Бесконечный цикл while в Java

У вас нет input = keyboard.nextLine(); во втором цикле while.

Вы можете реорганизовать свой код, чтобы запрашивать новый ввод только при возникновении ошибки. Итак, сразу после sysout "ОШИБКА..."

Дополнительно: я бы сделал это по-другому. «Ошибка = истина» в начале немного сбивает с толку, потому что ошибки может и не быть.

Вы могли бы, например, написать метод с именем tryProcessLine, который считывает входные данные и возвращает true, если все в порядке, и false, если произошла ошибка, а затем просто выполните что-то вроде while(!tryProcessLine()){ }

Рабочий пример ниже:

import java.io.IOException;
import java.util.Scanner;

public class Methods {

  private static int qoh;

  public static void main(String args[]) throws IOException {

    while (!tryProcessLine()) {
        System.out.println("error... Trying again");
    }

    System.out.println("succeeded! Result: " + qoh);

  }

  public static boolean tryProcessLine() {

    String input = "";

    Scanner keyboard = new Scanner(System.in);

    System.out.print("\nEnter Quantity on Hand: ");

    input = keyboard.nextLine();

    try {
        qoh = Integer.valueOf(input);

        if (qoh < 0 || qoh > 500) {
          System.out.println("\n**ERROR06** - Quantity on hand must be between 0 and 500");
          return false;
        } else {
          return true;
        }
    } catch (NumberFormatException e) {
        System.out.println("\n**ERROR06** - Quantity on hand must be numeric");
        return false;
    }
  }
}

Проблема именно в этом разделе:

                        while (error==true)
                        {
                            if (Character.isLetter(input.charAt(0)))
                            {
                                System.out.println("\n**ERROR06** - Quantity on hand must be between 0 and 500");
                                error=true;
                                System.out.println(qoh);
                                System.out.println(input);
                            }
                            else
                            {
                                qoh = Integer.parseInt(input);
                                error=false;
                            }
                        }

Как только у вас есть буква в первой позиции, этот цикл никогда не может завершиться. Он проверяет, стоит ли буква на первой позиции (есть), печатает ее и повторяет. Попробуйте изменить на:

                            while (error==true)
                            {
                                if (Character.isLetter(input.charAt(0)))
                                {
                                    System.out.println("\n**ERROR06** - Quantity on hand must be between 0 and 500");
                                    error=false;

                                    ...

Кроме того, еще пара вещей:

while (error == true) можно сократить до while(error).

Кроме того, Integer.parseInt выдаст NumberFormatException, если ввод не является целым числом — вам нужно поймать и обработать это.

Кроме того, зачем вообще нужен второй цикл? Похоже, предполагается только проверка ввода - если это так, вы можете переместить эту логику в первый цикл и исключить второй. Используйте циклы только для вещей, которые должны повторяться (например, ввод пользователем входных данных). Нет необходимости многократно проверять один и тот же вход.

Бесконечный цикл возникает из-за того, что второй цикл while неоднократно проверяет, является ли первый символ в строке (input.charAt(0)) буквой. Если предположить, что результат этой проверки верен, цикл никогда не завершится.

Ваш код можно упростить до чего-то вроде:

Integer qty = null;

while (scanner.hasNext() && qty == null) {
  String line = scanner.next();
  try {
    qty = Integer.parseInt(line);
  } catch(NumberFormatException ex) {
    System.err.println("Warning: Ignored non-integer value: " + line);
  }
}

if (qty == null) {
  System.err.println("Warning: No quantity specified.");
}

Если это символ, вы допускаете, что ошибка все еще = true, что заставляет этот цикл продолжаться вечно, вы никогда не возвращаетесь к началу и не читаете другую строку.

Вот некоторый код, который делает то, что вы хотите, и структурирован немного лучше.

public class ScanInfo {

  Scanner keyboard = new Scanner(System.in);

  public ScanInfo(){
    String line = getLineFromConsole();
    while(null != line && !"quit".equals(line)){
      if(isValidInput(line)){
        int validNumber = Integer.parseInt(line);
        System.out.println("I recieved valid input: "+validNumber);
      }else{
        System.out.println("\n**ERROR06** - Quantity on hand must be between 0 and 500");
      }
      line = getLineFromConsole();
    }

  }

  private boolean isValidInput(String line){
    //basic sanity
    if(null == line || line.length() < 1){
      return false;
    }


    try {
      int number = Integer.parseInt(line);

      return (number >= 0 && number <= 500);

    } catch (NumberFormatException e) {
      return false;
    }

  }


  public static void main(String[] args) {
    new ScanInfo();

  }

  public String getLineFromConsole(){
    System.out.print("\nEnter Quantity on Hand: ");
    return keyboard.nextLine();

  }

}