JavaRush/Помощь по Java задачам/"Убедись, что учитывается весь id, а не его часть."

41 уровень

28.06.2019
791просмотров
10комментариев

"Убедись, что учитывается весь id, а не его часть."

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

Решен

Что бы это значило? В IDEA все работает, правда я свои примеры использую.

Считать с консоли имя файла.
Найти в файле информацию, которая относится к заданному 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; import java.io.*; import java.util.*; /* Поиск данных внутри файла */ public class Solution { public static void main(String[] args) throws IOException{ BufferedReader b=new BufferedReader(new InputStreamReader(System.in)); FileInputStream f=new FileInputStream(b.readLine()); BufferedReader b2=new BufferedReader(new InputStreamReader(f)); String s=""; do {s=b2.readLine(); String[] s2=s.split(" "); if (s2[0].equals(args[0])) System.out.println(s); } while (!s.isEmpty()); b.close(); f.close(); b2.close(); } }

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

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

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

JustinianJudge в Mega City One

Master

28 июня 2019, 23:08

Я понимаю что джава каждый учит для своих целей, но коли уж это джава, а не питон,пхп или ассемблер с скалой, то нужно знать и применять основные правила. https://google.github.io/styleguide/javaguide.html https://medium.com/coding-skills/clean-code-101-meaningful-names-and-functions-bf450456d90c не забывая регулярно нажимать CTRL+SHIFT+L Как вариант хотя бы так:

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

        String id = args[0];
        String line;

        do {
            line = bufferedReader.readLine();
            String currentId = line.split(" ")[0];
            if (currentId.equals(id)) {
                System.out.println(line);
            }
        } while (!line.isEmpty());

        reader.close();
        inputStream.close();
        bufferedReader.close();
    }
}

s для временной строковой переменной, а s2 для массива строк, это уже слишком, по такой логике s3 будет числом, а s4 объектом типа Кот. Мы должны писать код так, чтобы он отвечал на существующие вопросы, а не создавал дополнительные. При этом отмечу и похвалю, что FileInputStream в отдельную строку вынесен, есть много любителей это с BufferedReader в одну мега-строку писать..21:9 монитора не хватит такое прочитать

Николай

Уровень 41

29 июня 2019, 09:44

Да, так красивее. :-) Но я и не собирался изначально кому-то этот код показывать. Идея была в том, чтобы оперативно написать код в несколько строчек для "внутреннего использования", пройти валидацию и двигаться дальше. Но пока не получается. :-( Мне то как раз понятно - s строка, s2 тоже что-то строчное:-)

JustinianJudge в Mega City One

Master

29 июня 2019, 10:28

полезный

Я понимаю, и ты абсолютно прав - главное движение вперед, перфекционизм губителен, но... нужно формировать привычки, есть такая книжка Clean code Роберта Мартина, читай и бери на вооружение. Это дело привычки, время затраченное на программу будет такое же, а вот время на дебаг и приведение кода в работоспособное состояние - ты будешь экономить и экономить очень много.

Николай

Уровень 41

29 июня 2019, 10:46

Это да. Когда я смотрю свои программы, которые написаны несколько месяцев назад, не понимаю, что там вообще делается. В основном, кстати, из-за отсутствия комментариев. Согласен, что все, к чему будешь потом возвращаться, нужно делать по правилам.

JustinianJudge в Mega City One

Master

29 июня 2019, 10:50

Комментарии упаси тебя бог. Комменты это признак того что место коду в мусорнике, в джава коде за редким исключением комменты не применяют.

JustinianJudge в Mega City One

Master

28 июня 2019, 22:59

Странно даже, айдишник у тебя должно находить верный, но вот логика цикла очень непривычная и странная. От того и непривычная, что не рабочая.

1257   кцукцукцу
126    цукцукц
125    цукцу33
1      232323

Для такого файла, аргумент запуска программы: 125 Что твой код выдает?

Николай

Уровень 41

29 июня 2019, 09:24

Так третью строчку и выдаст. Первую не выдаст. Я переделал код на IndexOf(args[0])==0, и так тоже валидацию не проходит - то есть строку, начинающуюся с id, выдавать и не должен. А чем логика цикла странная? Мне как раз представляется простой и понятной.

JustinianJudge в Mega City One

Master

29 июня 2019, 10:20

решение

У меня твой первоначальный вариант отрабатывает так: "125 цукцу33 Exception in thread "main" java.lang.NullPointerException at JavaRush.Николай.main(Николай.java:16)" из-за того что ты сначала считываешь строку - s = b2.readLine(); потом ее сплитишь - String[] s2 = s.split(" "); а потом обращаешься к первому элементу массиву который должен получится после сплита пустой строки и только после этого всего проверяешь, а не пустая ли строка, может выйти. Интересно почему у тебя эксепшен не выскакивал.

Николай

Уровень 41

29 июня 2019, 10:39

Вообще говоря, выскакивал, но я не обратил внимания, только после твоего замечания увидел. :-) Спасибо, переделал на исполнение цикла при условии, что b2.ready() и ошибка ушла - валидатор принял.

JustinianJudge в Mega City One

Master

29 июня 2019, 10:50

) Отличная новость, удачи!