Java.util.Scanner, выпуск с закрытием

Question

Итак, проблема с этим кодом заключается в том, что если он итерации по циклу более одного раза, закрытие сканера приводит к тому, что код пренебрегает сканированием для дальнейшего ввода пользователем. Я понимаю, что проблема должна быть внутренней в классе Scanner, но есть ли у кого-нибудь способ закрыть сканер безопаснее? (Это может помочь скопировать код и попробовать ввести «55» или «j», например, если вы не осознаете проблему, которую я сейчас имею).

public static void main(String[] args){ 
    int numCom = 0, count = 0; 
    boolean valid = false; 

    do{ 
     Scanner num = new Scanner(System.in); 
     try{ 
      if(count == 0){ 
       System.out.println("Okay, how many adversaries would you like to be put up against? (between 1-8)"); 
       count++; 
      }else{ 
       System.out.println("Error!\nPlease enter a number between 1-8."); 
      } 
      numCom = num.nextInt(); 
     }catch(Exception e){} 
     if(numCom < 9 && numCom > 0){ 
      valid = true; 
     } 
     num.close(); 
    }while(!valid); 
}//main() 

Answer 1

Создайте и закройте сканер вне цикла. Как это:

public static void main(String[] args) { 
     int numCom = 0, count = 0; 
     boolean valid = false; 
     Scanner num = new Scanner(System.in); 

     do { 
      try { 
       if (count == 0) { 
        System.out.println("Okay, how many adversaries would you like to be put up against? (between 1-8)"); 
        count++; 
       } else { 
        System.out.println("Error!\nPlease enter a number between 1-8."); 
       } 
       numCom = num.nextInt(); 
      } catch (Exception e) {} 
      if (numCom < 9 && numCom > 0) { 
       valid = true; 
      } 
     } while (!valid); 
     num.close(); 
    } //main() 

Answer 2

Try This: Java 7+ http://ideone.com/dxtXmz

public static void main(String[] args){ 
    int numCom = 0, count = 0; 
    boolean valid = false; 
     try (Scanner num = new Scanner(System.in)) { 
      do{ 
       try{ 
        if(count == 0){ 
         System.out.println("Okay, how many adversaries would you like to be put up against? (between 1-8)"); 
         count++; 
        }else{ 
         System.out.println("Error!\nPlease enter a number between 1-8."); 
        } 
        numCom = num.nextInt(); 
       }catch(Exception e){} 
       if(numCom < 9 && numCom > 0){ 
        valid = true; 
       } 
      }while(!valid); 
     } 
}//main() 

ИЛИ

Try This: http://ideone.com/LaN6kF

public static void main(String[] args){ 
int numCom = 0, count = 0; 
boolean valid = false; 

    Scanner num = new Scanner(System.in); 
do{ 
    try{ 
     if(count == 0){ 
      System.out.println("Okay, how many adversaries would you like to be put up against? (between 1-8)"); 
      count++; 
     }else{ 
      System.out.println("Error!\nPlease enter a number between 1-8."); 
     } 
     numCom = num.nextInt(); 
    }catch(Exception e){} 
    if(numCom < 9 && numCom > 0){ 
     valid = true; 
    } 
}while(!valid); 

    num.close(); 
}//main() 

В основном ресурс: сканер: выделяется только один раз и повторно использовать. , и когда это не требуется, его закрытие.

Answer 3

Если вам нужно обрабатывать строки, а также, попробуйте это один:http://ideone.com/f7IMwD

public static void main(String[] args) { 
     int numCom = 0, count = 0; 
     boolean valid = false; 
     try (Scanner num = new Scanner(System.in)) { 
      do { 
       try { 
        if (count == 0) { 
         System.out.println("Okay, how many adversaries would you like to be put up against? (between 1-8)"); 
         count++; 
        } else { 
         System.out.println("Error!\nPlease enter a number between 1-8."); 
        } 
        String line = num.nextLine(); 
        numCom = Integer.parseInt(line); 
        if (numCom < 9 && numCom > 0) { 
         valid = true; 
        } 
       } catch (NumberFormatException e) { 
        valid = false; 
       } 

      } while (!valid); 
     } 
    }//main() 

Answer 4

Что вам нужно, чтобы убедиться, что только сканер закрыт, а не в System.in. Вызов scanner.close() в свою очередь вызывает закрытие источника, который вы предоставляете в конструкторе (System.in) в этом случае. У вас есть два варианта. Более элегантный один будет использовать http://commons.apache.org/proper/commons-io/javadocs/api-release/org/apache/commons/io/input/CloseShieldInputStream.html

Scanner num = new Scanner(new CloseShieldInputStream(System.in)); 

чистое решение ява будет

public class Scan { 
private static final class MyShieldedSystemIn extends InputStream { 
    @Override 
    public int read(byte[] b) throws IOException { 
     return System.in.read(b); 
    } 

    @Override 
    public int read(byte[] b, int off, int len) throws IOException { 
     return System.in.read(b, off, len); 
    } 

    @Override 
    public void close() throws IOException { 
     // do nothing 
     System.out.println("ignoring call to close()"); 
    } 

    @Override 
    public int read() throws IOException { 
     return System.in.read(); 
    } 
} 
final static InputStream SSI = new MyShieldedSystemIn(); 
public static void main(String[] args){ 
    int numCom = 0, count = 0; 
    boolean valid = false; 


    do{ 
     Scanner num = new Scanner(SSI); 
     try{ 
      if(count == 0){ 
       System.out.println("Okay, how many adversaries would you like to be put up against? (between 1-8)"); 
       count++; 
      }else{ 
       System.out.println("Error!\nPlease enter a number between 1-8."); 
      } 
      numCom = num.nextInt(); 
     }catch(Exception e){} 
     if(numCom < 9 && numCom > 0){ 
      valid = true; 
     } 
     num.close(); 
    }while(!valid); 
}//main() 

}

Answer 5

При закрытии Scanner, он закроет свой источник входного сигнала, который в этот случай System.in. Как только вы закрыли System.in, вы больше не можете читать ввод. Не закрывайте Scanner. У него нет никаких ресурсов, которые необходимо закрыть, если вы не хотите закрыть источник ввода, который в этом случае у вас нет. Если вы удалите вызов num.close();, ваш код будет работать правильно.

Редактировать: Тем не менее, вы делаете тяжелую работу. Длина вашего кода может быть вдвое без потери функциональности:

int numAdversaries; 
System.out.println("Okay, how many adversaries would you like to be put up against? (between 1-8)"); 
for (;;) { 
    try { 
     numAdversaries = new Scanner(System.in).nextInt(); 
     if (numAdversaries >= 1 && numAdversaries <= 8) break; 
    } catch (Exception e) {} 
    System.out.println("Error!\nPlease enter a number between 1-8."); 
}