Почему код работает по разному?

Question

[Koshkin_I]

Суть в принципе в заголовке.

String filename;
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        while (!(filename= reader.readLine()).equals("exit")){
            list.add(filename);
                     System.out.println(Arrays.toString(filename.split(".part")));
        }
        System.out.println(list);

выдает такой массив:
[uyfd.aufv.part23, uyfd.aufv.part5, uyfd.aufv.part11]
а если закоментить строку System.out... в цикле while то будет так:
[uyfd.aufv.part23, , uyfd.aufv.part5, , uyfd.aufv.part11, ]
Я так понимаю 3 пустых элемента, но от куда берутся и почему не понимаю.

Comments

[Dmitry Roo]

Добавьте входные данные к вопросу

[Koshkin_I]

Какие данные? Это часть другой задачи. Просто тут нашел притыку.
С клавы вводятся имена файлов с суффиксом [.partN] в произвольном порядке. Их надо отсортировать по N. Тут System.out'ы просто для лога. Сортировку в этой части вообще не затрагиваю. Озадачил сам list, точнее разница вариантов.

[Dmitry Roo]

Koshkin_I, входящие данные - это, видимо, то, что вы с клавиатуры вводите.
И еще неплохо бы целиком весь код добавить, а не какой-то кусочек.

[Koshkin_I]

Полный код добавил уже в другом ответе ну и тут:

public static  ArrayList<String> list = new ArrayList<>();
    public static void main(String[] args) throws IOException {
     //   ArrayList<String>  list = new ArrayList<>();
        String filename ="";
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        while (!(filename= reader.readLine()).equals("exit")){
            list.add(filename);
            //                    System.out.println(Arrays.toString(filename.split(".part")));
        }
        System.out.println(list);
   list = (ArrayList<String>) list.stream().sorted((String s1, String s2)->{
          var st1 = s1.split(".part");
            System.out.println(Arrays.toString(st1));
          var st2 =s2.split(".part");
            System.out.println(Arrays.toString(st2));
          var n2 =Integer.parseInt(st2[st2.length-1]);
var n1 = Integer.parseInt(st1[st1.length-1]);
          return n2 - n1;
        }).collect(Collectors.toList());
        System.out.println(list);
    }

Вводил:
uyfd.aufv.part23
uyfd.aufv.part5
uyfd.aufv.part11
exit

Когда та строка раскомментирована - всё сортируется.

[Dmitry Roo]

Koshkin_I, проблема, похоже, не в коде. У меня не воспроизводится во всяком случае.
Либо вы просто enter нажимаете два раза, либо что-то не договариваете.

[Koshkin_I]

Dmitry Roo, да нет, вроде всё как есть описал. Еще раз ща запустил - снова те же расклады.

[Dmitry Roo]

Koshkin_I, попробуйте сделать ввод данных не с клавиатуры.
Либо из файла, либо захардкодив строку.
Вдруг с консолью какая-то проблема...

[Koshkin_I]

Dmitry Roo, ) Во как. А так работает. О чем это может говорить?

[Dmitry Roo]

Koshkin_I, похоже на какую-то путанницу с cr/lf (новая строка/возврат корретки).

Answer 1

[Дмитрий Кинаш]

Если "если закоментить строку System.out... в цикле while", то для в остальном идентичного кода и идентичных входных данных результат будет идентичным.

Следовательно или входящие данные различны, или код выполняется в различных условиях. Даже бегло взглянув на код видно, что данные добавляются в неизвестный список, о котором ничего не известно - кто и где его создал, была ли очистка перед чтением входящего потока?

Comments

[Koshkin_I]

Вот полный код. Все System`ы - логи.

public static  ArrayList<String> list = new ArrayList<>();
    public static void main(String[] args) throws IOException {
     //   ArrayList<String>  list = new ArrayList<>();
        String filename ="";
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        while (!(filename= reader.readLine()).equals("exit")){
            list.add(filename);
            //                    System.out.println(Arrays.toString(filename.split(".part")));
        }
        System.out.println(list);
   list = (ArrayList<String>) list.stream().sorted((String s1, String s2)->{
          var st1 = s1.split(".part");
            System.out.println(Arrays.toString(st1));
          var st2 =s2.split(".part");
            System.out.println(Arrays.toString(st2));
          var n2 =Integer.parseInt(st2[st2.length-1]);
var n1 = Integer.parseInt(st1[st1.length-1]);
          return n2 - n1;
        }).collect(Collectors.toList());
        System.out.println(list);
    }

Данные одинаковые. Много раз повторял. Выходы идентичные для каждой ситуации. Окончательный sout выдает отсортированный список если строка в цикле не закомментирована и ошибку в противном случае

Answer 2

[GavriKos]

Я вот нифига не знаю как там в джаве устроены потоки ввода-вывода, но ОЧЕНЬ ПОХОЖЕ, что в BufferedReader ваш попадает результат System.out.println.

Для очень быстрой проверки замените
System.out.println(Arrays.toString(filename.split(".part")));
на что то вроде
System.out.println("WTF");
и посмотрите что будет в массиве

Answer 3

[Koshkin_I]

Решен. Может кому пригодится.

Dmitry Roo Dmitry Roo Куратор тега Java
Koshkin_I, похоже на какую-то путанницу с cr/lf (новая строка/возврат корретки).

Не знаю насколько это костыльно, но проблему решает проверка !isEmpty() перед добавлением в list в цикле.