Всем привет!
В параллель с прохождением уровней на javarush реализую свои структуры данных (уже добрался до деревьев)
https://github.com/derevyannykh-aleksei/Structures_java/tree/master/src
Буду рад услышать любые отзывы о коде, замечания и дополнения.
Спасибо.
Alex
51 уровень
Code review
Комментарии (5)
- популярные
- новые
- старые
Для того, чтобы оставить комментарий Вы должны авторизоваться
Андрей Святогоров
14 ноября 2019, 10:56
Добавил тебе пару пулов в репозиторий, с коментами, постарался объяснить, почему что и как, плю закинул задачу одну.
0
Alex
14 ноября 2019, 21:02
Спасибо, мне пришло уведомление на пулы. посмотрел их по диагонали. Про гит игнор, сам толком не разобрался, поэтому просто добавлял все ) спасибо еще раз как разберусь с другими задачами все гляну и отвечу тут
0
Андрей Святогоров
15 ноября 2019, 13:30
github автоматически генерирует .gitignore там только нужно галочку поставить и выбрать нужный язык, в целом нормально покрывает все что необходимо заигнорить
0
Justinian Judge в Mega City One Master
16 октября 2019, 23:12
Для 9-го уровня более чем.
Из таких замечаний, методы есть очень большие и перегруженные, разбивай на более маленькие, думай о том, как можно использовать один и тот же код (хелпер метод) в нескольких методах.
Нейминг местами.
Ну и главное, у тебя мессиво все в одном пакете. Неструктурные структуры получаются.
Каждая структура данных, свой пакет.
Иначе в твоем коде поди разбери, где у тебя утильный класс, где интерфейс, где класс хранящий данные, ноды, сегменты, структуры все в перемешку.
По тестам, обычно пишут более конкретные с отображением в имени Например добавитьВПустой список. Добавить в средину полного, удалениеСГоловыСписка, удалениеПоследнегоЭлемента.
В принципе можно группировать конечно, но у тебя совсем все общее.
По алгоритмам есть вопросы, но..ты продолжай, это в любом случае опыт. Первая стадия, это когда ты в принципе можешь выполнить задание хоть как-то.
Потом набираешься опыта, и уже начинаешь по другому писать.
Так что продолжай. После того как сам написал, смотри варианты как пишут другие, гугль в помощь, статьи, книги, гитхаб. Сравнивай с своим кодом, думай что можно в нем изменить.
У тебя активная подписка на джава раше?
Какой у тебя план изучения джавы сейчас?
+1
Alex
18 октября 2019, 10:09
Спасибо.
"У тебя активная подписка на джава раше?"
да
"Какой у тебя план изучения джавы сейчас?"
ближайший план посмотреть что такое Spring и дальнейшее прохождение уровней на javajush
0