JavaRush/Помощь по Java задачам/Задача "Самые частые байты" - com.javarush.task.task18.t...

41 уровень

29.03.2020
807просмотров
10комментариев

Задача "Самые частые байты" - com.javarush.task.task18.task1803

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

Решен

Друзья, помогите, пожалуйста, разобраться, что не так с кодом. На мой взгляд, вывод корректный, но валидатор не принимает: По пункту: "В консоль через пробел должны выводиться все байты из файла с максимальным количеством повторов" появляется вот такая рекомендация от ментора: "Ты выводишь все байты встречающиеся в файле, а нужно только байты с максимальным количеством повторов." Хотя я не вывожу все байты. У меня содержимое файла такое: 00001111123456789абвгдеёжзиклмнопрстуфхцчшщъыьэюяqqqqqqqqqq Вывод в консоль при этом вот такой: 48 49 113 Код привожу ниже. Заранее спасибо!

Ввести с консоли имя файла.
Найти байт или байты с максимальным количеством повторов.
Вывести их на экран через пробел.
Закрыть поток ввода-вывода.

Требования:

Программа должна считывать имя файла с консоли.
Для чтения из файла используй поток FileInputStream.
В консоль через пробел должны выводиться все байты из файла с максимальным количеством повторов.
Данные в консоль должны выводится в одну строку.
Поток чтения из файла должен быть закрыт.

00001111123456789абвгдеёжзиклмнопрстуфхцчшщъыьэюяqqqqqqqqqq

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

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

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

Артем Стукалов

Уровень 23

Expert

27 августа 2020, 21:20

Задача - упоротая, условие упоротое. Решил через 2 ArrayList с тучей if-else. Получилось ~62 строчки кода. Через мапу принципиально не пошел, т.к. все через нее решают, выходит так же +- 60 строчек кода. Красивее и лаконичнее решать через двумерные массивы ИМХО

IceBerg

Уровень 41

Master

29 марта 2020, 13:42

А разве вывод должен быть такой?

Вывод в консоль при этом вот такой: 48 49 113

Твоя программа выводит все байты, которые встречаются больше одного раза.

barracuda

Уровень 41

Expert

29 марта 2020, 17:49

Спасибо за ответ! Да, в данном случае больше одного. Так же я пробовал пройти валидацию с тем же кодом, только с условием "больше двух раз"... Тоже не прошёл ) В комментах к задаче пишут, что в условии задачи имеется ввиду, что "В консоль, через пробел, нужно вывести все байты имеющие более 2-х повторений."

IceBerg

Уровень 41

Master

29 марта 2020, 18:27

решение

Ничего такого, только байты с максимальным количеством повторений, даже если 200 раз повторился, но есть который 201, то выводить только тот, что 201

barracuda

Уровень 41

Expert

29 марта 2020, 19:45

Ну вроде бы так и должно быть. Но это не бьется с условием задачи: "3. В консоль через пробел должны выводиться все байты из файла с максимальным количеством повторов." Что в таком случае нужно выводить через пробел, если мы выводим только тот байт, который встретился 201 раз?

barracuda

Уровень 41

Expert

29 марта 2020, 19:49

Круто! Я решил! Спасибо за помощь, без вас было бы тяжело!!!

JustinianJudge в Mega City One

Master

29 марта 2020, 13:41

как минимум, убери статические переменные list и map. Подобные переменные нужно создавать в нужном методе и передавать как аргументы в вызываемые методы. Поля класса это значимые характеристики объекта. Список продуктов это заметка в твоем блокноте, но никак не поле класса - твоя значимая характеристика, которая должна быть в паспорте наравне с именем и возрастом + твоя программа вообще ничего не выводит при файлах:

на файле

вывела

Тестируй хорошенько на разных примерах

barracuda

Уровень 41

Expert

29 марта 2020, 17:52

Спасибо ) Ну в случае с "123" я не знаю, что должно выводиться. Ведь максимальное число повторов для всех байтов = 0. То есть повторов не. Либо нужно ничего не выводить, либо вывести все байты. Скорее всего , что все, тогда у меня баг. В случае с "123344" - действительно косяк. Буду проверять. Точнее, удалю код и сделаю все по-другому. Если что, вернусь к Вам! ))) ============ PS: по поводу статических переменных - убрал. Спасибо. Они там появились, когда в ходе одной из версий моего решения у меня было несколько методов, которые должны были их использовать, поэтому вынес их как статические переменные класса.

JustinianJudge в Mega City One

Master

29 марта 2020, 17:58

решение

как использовать объекты в нескольких методах я выше написал. В случае 123 должны вывестись все байты, для простоты понимания, можешь заменить повторы, "сколько раз байт встречается в файле" Может условия конечно поменяли, но когда я сдавал, было именно так Сам посуди, даже по твоей логике, если ни один байт не повторяется, количество повторов - 0. У трех байт количество повторов 0 Максимальное количество повторов - 0 по условию задачи, нужно вывести байты с максимальным количеством повторов. Значит должны вывестись все байты. 0 это тоже число

barracuda

Уровень 41

Expert

29 марта 2020, 19:50

Да, я так и сделал. И решил. Спасибо, помогло!!!