Добавление чисел в массив из .txt-файла

Question

Привет, ребята, я довольно новичок в java, и у меня возникает эта проблема при попытке добавить числа из файла compact.txt в массив. Вот мой код:

public void compactArray(){ 
    try{ 
     Scanner scan = new Scanner(new File("compact.txt")); 
     while(scan.hasNextInt()){ 
      num++; 
     } 
     int [] a = new int[num]; 
     Scanner in = new Scanner(new File("compact.txt")); 
     while(counter < num){ 
      a[counter] = in.nextInt(); 
      counter++; 
     } 
     System.out.println(Arrays.toString(a)); 
    }catch(IOException bob){ 
     bob.getMessage(); 
    } 
} 

Проблема с этим кодом в том, что он никогда не перестает работать. Сначала мой код читает файл compact.txt, а затем подсчитывает количество чисел, которое он должен определить размер массива. Затем я делаю другую переменную сканера, чтобы добавить числа из файла compact.txt в массив. Я использую переменную счетчика как способ остановки, когда желаемое количество чисел добавляется в массив a. Я не слишком уверен, в чем проблема, но он продолжает работать и не попадает в строку, где предполагается распечатать массив. Может кто-то, пожалуйста, помогите мне. Огромное спасибо.

Answer 1

Вы должны позвонить

scan.nextInt(); 

в первом цикле. Вы никогда не двигаете курсором, в результате вы продолжаете читать первый элемент.

Однако решение, которое требуется решить, требуется дважды через ваш набор данных. Возможно, вы захотите использовать ArrayList, который представляет собой массив, размер которого можно изменить. Таким образом, вам не нужно будет сначала считать ваш файл.

Answer 2

изменение

while(scan.hasNextInt()){ here is the problem, This loop never move to next integer. You need to call to scan.nextInt() required to move next integer 
      num++; 
} 

в

while(scan.hasNextInt()){ 
     scan.nextInt(); 
     num++; 
} 

Answer 3

Вы делаете что-то неправильно там: Вы должны использовать только один Scanner объект для этого.

Более конкретно, что происходит в вашем случае, является следующее: вы проверяете, имеет ли сканер следующий int в while(scan.hasNextInt()){, но вы никогда не читаете этот int. Так что это будет цикл навсегда.

Правильный рабочий код будет:

public void compactArray(){ 
    List<Integer> ints = new ArrayList<>(); 
    try{ 
     Scanner scan = new Scanner(new File("compact.txt")); 
     while(scan.hasNextInt()){ 
      ints.add(in.nextInt()); 
     } 
    }catch(IOException ex){ 
     ex.getMessage(); 
    } 
    System.out.println(Arrays.toString(ints.toArray(new int[ints.size()]))); 
} 

Я также изменил следующие моменты вашего кода:

• Внутренне теперь List<Integer> используется для хранения целых чисел. Из-за этого больше не нужно делать подсчет!
• Дал исключение значимое имя.
• В System.out.println теперь List<Integer> сначала преобразуется в массив, а затем отображается String -представление.