Подскажите, что не так, пожалуйста?

Спасибо, мне в целом решение не нужно, мне не ясно, что именно не так в моей программе!

Я показал свой пример, как бы у Вас слишком замудренно)

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

Олег Викторович, 10 не терялось никогда. Не понимаю, с чего вы так решили.

Я - да! А вы?

Спасибо. Но моя программа как раз и реализует метод nextInt() типа Scanner. Т.е. это и сделано в коде, который я представил. Scanner, мягко говоря, совсем неудачный класс, вообще его не хочется использовать. Тут представилась интересная задача, да валидатор жаль рубит. Повторюсь, я писал выше, мне не нужно решение, мне нужен ответ, что именно не так в моем коде, что валидатор его не пропускает. Ошибки я не нашел.

Функционал nextInt() типа Scanner выполняет свою функцию даже хуже, чем код, который я сделал, например, он не подходит для типов Long и BigInteger, а код в программе вполне подойдет.

Наверное, правильно говорить "метод nextInt() класса Scanner".

1212121212 вот вам число как бы четное. 222222222 111111112 1212121212 такой набор не выводит верно. Поработайте еще над функционалом.

посмотрите на тестовый файл input1 вверху этой беседы. видите там 2 пустых строки внизу? метод nextInt() класса Scanner их "проигнорирует" а ваш код на данных этого файла вылетит с ошибкой. вот такое изменение может быть вас спасет:

if ( s.length() != 0 && (temp = Integer.parseInt(s)) %2 == 0)
    array.add(temp);

обязательно двойной && не & иначе все равно будет вылет при пустой строке. возможно валидация не проходит из-за этого... то что написал Евгений вашим кодом обрабатывается корректно. кстати хорошо что числа хранятся в байтовом виде иначе ваша практика чтения по байтам дала бы сбой, почему вы решили использовать поток(FileInputStream), а не ридер(FileReader)? а еще лучше и легче BufferedReader:

import java.io.*;
import java.util.ArrayList;
import java.util.Collections;

/*
Сортировка четных чисел из файла
*/

public class Solution {
    public static void main(String[] args) throws IOException {
        ArrayList<Integer> array = new ArrayList<>();
        BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(System.in));
        InputStream inStream = new FileInputStream(bufferedReader.readLine());
        bufferedReader.close();
        BufferedReader bfr = new BufferedReader(new  InputStreamReader(inStream));
        String str;
        while (null!=(str= bfr.readLine())) {
            if(str.length() != 0) {
                int temp = Integer.parseInt(str);
                if(temp%2==0)
                    array.add(temp);
            }
        }
        inStream.close();
        bfr.close();
        Collections.sort(array);
        for(Integer i:array)
            System.out.println(i);
    }
}

InputStream можно конечно не создавать отдельной строкой, а сразу в InputStreamReader прописать но тогда валидатор может ругнуться на несоблюдение 2го и последнего условий

Код не отрабатывает как надо. Специально проверил еще раз. 222222222 111111112 1212121212 для такого набора вывод Вывод 111111112 222222222 корректно вывести все 3 числа. в этой задаче вообще нужно добавить все числа и отсортировать и вывести только те что делятся без остатка на 2. Зачем мудрить?

точно, когда проверяется конец ввода числа, не проверяется конец файла, и последнее число не добавляется!

public static void main(String[] args) throws IOException {
        ArrayList<Integer> array = new ArrayList<>();
        BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(System.in));
        InputStream inStream = new FileInputStream(bufferedReader.readLine());
        bufferedReader.close();
        String s = "";
        char c;
        int temp;
        while (inStream.available() > 0) {
            c = (char) inStream.read();
            if (c == '\r')
                if (s.length() != 0 && (temp = Integer.parseInt(s)) % 2 == 0)
                    array.add(temp);
            s = s + c;
            if (c == '\n')
                s = "";
        }
        inStream.close();
        if (s.length() != 0 && (temp = Integer.parseInt(s)) % 2 == 0)
            array.add(temp);
        Collections.sort(array);
        for (Integer integer : array)
            System.out.println(integer);
    }
}

Спасибо, помогли отловить ошибку с пробелом. Это классно, но , к сожалению, в целом не помогло. Исправил ошибку с пустой строкой и чтением пробела. Однако, валидатор все также режет решение. Вот мой текущий код. Всем будут благодарен, если прогоните на своем компе и поможете найти в нем bug. package com.javarush.task.task13.task1326; import java.io.*; import java.util.ArrayList; import java.util.Collections; import java.util.List; import java.util.Scanner; /* Сортировка четных чисел из файла */ public class Solution { public static void main(String[] args) throws IOException { ArrayList<Integer> array = new ArrayList<>(); BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(System.in)); try { InputStream inStream = new FileInputStream(bufferedReader.readLine()); String s = ""; char c; while (inStream.available() > 0){ int temp = Integer.MAX_VALUE; c = (char) inStream.read(); try { if(c != '\r' || c != '\n' ){ if (c == '\r') { if ( (temp = Integer.parseInt(s)) %2 == 0){ array.add(temp); } } if (c != ' ') { s = s + c; } if (c == '\n') { s = ""; } } } catch (NumberFormatException e) { } } inStream.close(); } catch (FileNotFoundException e) { } Collections.sort(array); for (Integer integer : array) { System.out.println(integer); } } }

Конец файла в программе ловится другим оператором, а именно предикатором inStream.available() > 0. Дело не в этом, а в гадском валидаторе. Хотя ребята вверху и помогли мне отловить пару багов. Пришлось и исключения перехватить до кучи.

Спасибо, что заморочились. Сейчас уже отрабатывает. Чуть выше новый код. Однако, валидатор все равно на страже.

Вы верное подметили. Исправил - да не помогло валидатор пройти. Чуть выше новый код. В ответе предыдущему оратору.

if(c != '\r' || c != '\n' )

равносильно

if(true)

уберите этот if вообще вы опять наступаете на те же грабли, у вас сигналом окончания ввода числа является "\n" но в конце файла это не так.. вот же я специально чуть чуть ваш код причесал и слегка дополнил, разве что игнор пробела, я только сейчас допилил.

import java.io.*;
import java.util.ArrayList;
import java.util.Collections;

/*
Сортировка четных чисел из файла
*/

public class TempSolution {
    public static void main(String[] args) throws IOException {
        ArrayList<Integer> array = new ArrayList<>();
        BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(System.in));
        InputStream inStream = new FileInputStream(bufferedReader.readLine());
        bufferedReader.close();
        String s = "";
        char c;
        int temp;
        while (inStream.available() > 0) {
            c = (char) inStream.read();
            if (c == '\r' || c == ' ')
                if (s.length() != 0 && (temp = Integer.parseInt(s)) % 2 == 0)
                    array.add(temp);
            s = s + c;
            if (c == '\n'|| c == ' ')
                s = "";
        }
        inStream.close();
        if (s.length() != 0 && (temp = Integer.parseInt(s)) % 2 == 0)
            array.add(temp);
        Collections.sort(array);
        for (Integer integer : array)
            System.out.println(integer);
    }
}

Ну раз хочешь помучаться с байтами, вот попробуй так. Меньше вложенных if и правильный порядок проверок.

public class Solution {
    public static void main(String[] args) throws IOException {
        ArrayList<Integer> array = new ArrayList<>();
        BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(System.in));
        try {
            InputStream inStream = new FileInputStream(bufferedReader.readLine());

            String s = "";
            char c;
            while (inStream.available() > 0) {
                int temp;
                c = (char) inStream.read();
                try {
                    if (c != '\r' && c != '\n' && c != ' ') {
                        s += c;
                    }
                    if (c == '\r' || inStream.available() == 0) {
                        if (s.length() > 0 && ( temp = Integer.parseInt(s) ) % 2 == 0) {
                            array.add(temp);
                        }
                    }
                    if (c == '\n') {
                        s = "";
                    }
                } catch (NumberFormatException e) {
                }
            }
            inStream.close();
        } catch (FileNotFoundException e) {
        }
        Collections.sort(array);
        for (Integer integer : array) {
            System.out.println(integer);
        }
    }
}

Ну да. Ну или так если надо сохранить всё а вывести только четное.

public class Solution {
	public static void main(String[] args) throws IOException {
		ArrayList<Integer> array = new ArrayList<>();
		BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(System.in));
		InputStream inStream = new FileInputStream(bufferedReader.readLine());
		bufferedReader.close();
		String s = "";
		char c;
		while (inStream.available() > 0) {
			c = (char) inStream.read();
			if (c > 47 && c < 59) {
				s += c;
				if (inStream.available()==0)
					array.add(Integer.parseInt(s));
			} else {
				if (s.length()>0) {
					array.add(Integer.parseInt(s));
				}
				s = "";
			}
		}
		inStream.close();
		Collections.sort(array);
		for (Integer num : array) {
			if (num%2==0)
			System.out.println(num);
		}
	}
}

Владимир, спасибо вам, еще раз. Однако замечу, что сигналом об окончании файла в моем коде является предикатор (inStream.available() > 0). Он прекрасно справляется с обходом граблей, которые вы мне прочите. Однако, я загрузил ваш код и попытался проверить решение с ним. Увы, он оставил те же две ошибки: Программа должна выводить данные на экран. Программа должна вывести на экран все четные числа, считанные из файла, отсортированные по возрастанию.

каким списком проверяете?

Комбинацией 5 8 -2 11 3 -5 2 10 0 222222222 111111112 1212121212 Вы видите там и пустые строки. Но программа с самого начала выдавала верные, как друзья царя, данные. Вопрос в том, что не нравится валидатору.

Константин, валидатор выдает те же самые ошибки. Я загрузил ваше решение.

Евгений, с вашим кодом стало лучше. Плюс вам, осталась только одна ошибка Программа должна вывести на экран все четные числа, считанные из файла, отсортированные по возрастанию.

Евгений, интересно, что убралась ошибка "Программа должна выводить данные на экран" Хоть убей не понятно, что именно в вашем коде заставило валидатор её убрать.

Евгений, спасибо. Я расширил ваш диапазон символов ASCII до минуса и задача прошла валидатор. Вам спасибо огромное.

Да точно про минус я запамятовал). Рад был помочь).