Все информацию выводит, но не проходит валидацию.

Вопрос по задаче

Решен

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

Считать с консоли имя файла.
Найти в файле информацию, которая относится к заданному id, и вывести ее на экран в виде, в котором она записана в файле.
Программа запускается с одним параметром: id (int).
Закрыть потоки.

В файле данные разделены пробелом и хранятся в следующей последовательности:
id productName price quantity
где id - int.
productName - название товара, может содержать пробелы, String.
price - цена, double.
quantity - количество, int.

Информация по каждому товару хранится в отдельной строке.

Пример содержимого файла:
194 хлеб 12.6 25 195 масло сливочное 52.2 12 196 молоко 22.9 19

Пример вывода для id = 195:
195 масло сливочное 52.2 12

Требования:

Программа должна считать имя файла с консоли.
В методе main создай для файла поток для чтения.
Программа должна найти в файле и вывести информацию о id, который передается первым параметром.
Поток для чтения из файла должен быть закрыт.

package com.javarush.task.task18.task1822; /* Поиск данных внутри файла id productName price quantity где id - int. productName - название товара, может содержать пробелы, String. price - цена, double. quantity - количество, int. */ import java.io.BufferedReader; import java.io.FileInputStream; import java.io.IOException; import java.io.InputStreamReader; public class Solution { public static void main(String[] args) throws IOException { FileInputStream fileInputStream = new FileInputStream(new BufferedReader(new InputStreamReader(System.in)).readLine()); String str = ""; while (fileInputStream.available() > 0) { for (int i = fileInputStream.read(); i != 10 && fileInputStream.available() > 0; i = fileInputStream.read()) str += (char) i; if (str.startsWith((args[0] + " "))) break; str = ""; } System.out.println(str.trim()); fileInputStream.close(); } }

Комментарии (7)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Igor Davydov

Уровень 23

4 ноября 2018, 17:30

решение

Забыл выложить, просто поменял немного решение.

public class Solution {
    public static void main(String[] args) throws IOException {
        BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(System.in));
        String filename = bufferedReader.readLine();
        FileInputStream fileInputStream = new FileInputStream(filename);
        bufferedReader = new BufferedReader(new InputStreamReader(fileInputStream, "UTF-8"));
        System.out.println(findString(bufferedReader, args[0]));

        fileInputStream.close();
    }

    public static String findString(BufferedReader br, String id) throws IOException {
        String result;
        while ((result = br.readLine()) != null) {
            if (result.startsWith(id + " ")) {
                break;
            }
        }
        return result;
    }
}

Hokan.Vit

Уровень 20

2 ноября 2018, 10:25

ИМХО: попытался разобраться в первоначальном варианте кода)):

for (int i = fileInputStream.read(); i != 10 && fileInputStream.available() > 0; i = fileInputStream.read())

например строка: 123 abcd ef 45.67 89 a) 123 - id // id длиной от 1 символа и до 10 // 2147483647 (максимальное значение int) b) abcd ef - это productName // .. может содержать пробелы с)45.67 -- price d) 89 - quantity ещё до начала выполнения цикла fileInputStream.available() = 20 (в строке 20 символов считать read-ом можно будет 20 раз ) в цикле происходит следующее(читаем по байту) 0) в объявлении цикла: считываем read() --> i = 1 (считали первый символ строки) проверяем условие--> available = 19 (1 символ считали, осталось 19), определяем для i --> i = 2 (считали второй символ) 1) первый шаг цикла: строка --> str="2" // первый символ строки потеряли 2) второй шаг цикла: --> i = 3 str ="23" 3) третий шаг: --> i = " " str = "23 " 4) тут начинается интересное и удивительное!)) символ a если его рассматривать как char при считывании read() прочитается как byte --> i = 97 'a' в byte будет 97 ... следует отметить! представление символа будет зависеть от кодировки и от самого символа, например для некоторых языков потребуется несколько байт для представления одного символа тут может одного байта считанного read() не хватить для однозначного определения самого символа ... .... .... а по поводу второго варианта: условие

data != 10 && data != -1

в случае чтения файла из нескольких строк. каждая строка будет оканчиваться переводом строки на новую, и тогда даже после прочтения первой строки data=10 цикл while прекратится, остальные строки просто не прочитаются...

Сергеев Виктор

Уровень 40

Master

1 ноября 2018, 18:21

потестируйте на файле в котором больше 1 строки Критика по коду (можно не читать): FileInputStream fileInputStream = new FileInputStream(new BufferedReader(new InputStreamReader(System.in)).readLine()); - это ужасная конструкция, это как предложение, вложенное в предложение, которое вложено в предложение, которое что-то рассказывает и вложено в предложение, которое вложено в предложение, чувствуете как неудобно читать? Вот так и с вашим кодом. for (int i = fileInputStream.read(); i != 10 && fileInputStream.available() > 0; i = fileInputStream.read()) это примерно в туже степь, не пытаешься все сжать, явное, лучше чем не явное. Есть некоторые шаблоны, к которым уже многие привыкли и которые легко читаются и узнаются с первого взгляда. Чтение файла один из них. Используйте фигурные скобки, чтобы ЯВНО указать что вы пытались сделать, код читается и понимается с ними лучше. Без них приходится гадать "так задумано или он просто забыл?"

Igor Davydov

Уровень 23

2 ноября 2018, 05:42

Прислушался к вашему комментарию и немного переделал код, надеюсь, так он стал действительно более читаем)

public class Solution {
    public static void main(String[] args) throws IOException {
        BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(System.in));
        FileInputStream fileInputStream = new FileInputStream(bufferedReader.readLine());

        System.out.println(findString(fileInputStream, args[0]));

        bufferedReader.close();
        fileInputStream.close();
    }

    public static String findString(FileInputStream fileInputStream, String id) throws IOException {
        String result = "";
        while (fileInputStream.available() > 0) {
            int data = fileInputStream.read();
            while (data != 10 && data != -1) { // проверка на перенос и конец файла
                result += (char) data;
                data = fileInputStream.read();
            }
            if (result.startsWith(id + " ")){
                break;
            }
            result = "";
        }
        return result;
    }
}

Согласен, что так выглядит более читабельно, чем первый вариант. Решил поэкспериментировать) Для тестирования использовался файл с таким содержимым: 1 a 2.50 40 5 uo 6.40 20 17 irj 9.50 30 24 ohu 10.0 43 78 i k t 39.96 12 124 asdrtji 95.20 69 Возможно, по виду не совсем подходящий, но по формату и типу данных он подходит) При передаче через args[0] 78, выводилась именно эта строка "78 i k t 39.96 12" без ничего лишнего. Предыдущий код работал так же и тестировалась не одна строка) Проблема по прежнему актуальна.

Артем Divertitto Senior Android-разработчи в United Tech

Master

2 ноября 2018, 11:14

Попробуй для теста использовать русские слова)

Igor Davydov

Уровень 23

2 ноября 2018, 11:15

Ага, вот тут у меня пошло что-то не так) Спасибо, будем думать)

Артем Divertitto Senior Android-разработчи в United Tech

Master

2 ноября 2018, 11:17

Попробуй линиями считывать, вместо:

result += (char) data;