|
26 / 26 / 6
Регистрация: 09.02.2011
Сообщений: 71
|
|
Покритикуйте код14.06.2017, 10:15. Показов 831. Ответов 13
Метки нет (Все метки)
Всем привет.
У меня есть обычный POJO-класс с полями и класс для работы с данной коллекцией объектов. Посоветуйте плс, как улучшить код в стиле Java8. Вот линк на гитхаб: https://github.com/VictorSem/P... oductJava8
0
|
|
| 14.06.2017, 10:15 | |
|
Ответы с готовыми решениями:
13
Покритикуйте код моего сокет сервера для игрового чата Покритикуйте код Покритикуйте новичка |
|
238 / 237 / 142
Регистрация: 03.02.2011
Сообщений: 1,437
|
||
| 14.06.2017, 12:48 | ||
0
|
||
|
26 / 26 / 6
Регистрация: 09.02.2011
Сообщений: 71
|
|
| 14.06.2017, 13:44 [ТС] | |
|
В смысле "как понимать". Есть код, я хочу чтобы посмотрели код, указали на слабые места, что можно улучшить/удалить/добавить и т.п.
0
|
|
|
33 / 33 / 20
Регистрация: 18.03.2016
Сообщений: 101
|
||||||
| 19.06.2017, 03:05 | ||||||
|
А не могли бы на сайт код копировать. Вот в таком формате (спойлер под каждый файл):
Кликните здесь для просмотра всего текста
0
|
||||||
|
|
|||||||||||
| 19.06.2017, 08:53 | |||||||||||
|
I. Мне не нравится ваш класс
Product.
II. Класс ProductCollections
Кликните здесь для просмотра всего текста
ProductCollection например (см. Макконнела или Мартина). Модификаторы доступа вообще жесть: у вас приватный метод products возвращает стрим от аррэйлиста, выдаваемого публичным методом getProdCollection, который по сути геттер. Да еще к тому же и вызывается только там. Отсюда вопрос: а в таком ключе он вообще нужен (риторический если что)? Методы сортировки sortBy-bla-bla: они не используются. Зачем они:а) вообще б) конкретно в этом классе (это вообще не его ответственность -- снова см. любого из двух авторов у кого описано разделение ответственностей) ? Итого: грустно. Три класса, два из которых демонстрируют полную непродуманность процесса. Доводите до ума.
1
|
|||||||||||
|
378 / 370 / 114
Регистрация: 30.06.2010
Сообщений: 1,445
|
|
| 20.06.2017, 21:01 | |
|
я бы сказал еще что вызов метода get() у Optional - bad practice
0
|
|
|
|
|
| 21.06.2017, 08:26 | |
|
0
|
|
|
3639 / 2971 / 918
Регистрация: 05.07.2013
Сообщений: 14,220
|
||||
| 21.06.2017, 10:12 | ||||
|
1
|
||||
|
|
||
| 21.06.2017, 11:51 | ||
|
xoraxax, увидел, спасибо.
Добавлено через 49 минут Идея в том, что подобные сеттеры: 1) делают код нетривиальным для чтения, ибо, если доков к методу нет например, сразу неочевидно какое состояние объекта возвращается: до или после (а тесты могут и не показывать) 2) ломают javabeans конвенцию 3) фреймворки могут не принять такие сеттеры (следствие 2)
0
|
||
|
3639 / 2971 / 918
Регистрация: 05.07.2013
Сообщений: 14,220
|
||
| 21.06.2017, 12:24 | ||
|
0
|
||
|
378 / 370 / 114
Регистрация: 30.06.2010
Сообщений: 1,445
|
|||||
| 21.06.2017, 16:43 | |||||
|
я бы сказал еще что вызов метода get() у Optional - bad practice
3 - не встречал, но по идее не должно, return value может игнорится, хотя... если только фреймворк не жестко следует javabeans Добавлено через 15 секунд я бы сказал еще что вызов метода get() у Optional - bad practice 3 - не встречал, но по идее не должно, return value может игнорится, хотя... если только фреймворк не жестко следует javabeans
0
|
|||||
|
4575 / 2774 / 491
Регистрация: 28.04.2012
Сообщений: 8,765
|
||
| 21.06.2017, 19:13 | ||
|
0
|
||
|
3639 / 2971 / 918
Регистрация: 05.07.2013
Сообщений: 14,220
|
||
| 21.06.2017, 20:52 | ||
|
0
|
||
|
378 / 370 / 114
Регистрация: 30.06.2010
Сообщений: 1,445
|
||
| 21.06.2017, 21:14 | ||
|
0
|
||
| 21.06.2017, 21:14 | |
|
Помогаю со студенческими работами здесь
14
Алгоритм: Трамвайные билеты. Покритикуйте Задачи на строки и числа. Решение. Покритикуйте Задачи на числа. Решение. Покритикуйте. (часть №1) Покритикуйте пожалуйста резюме Junior Java Developer Покритикуйте код Искать еще темы с ответами Или воспользуйтесь поиском по форуму: |
|
Новые блоги и статьи
|
|||
|
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве
8Observer8 02.02.2026
Содержание блога
Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .
|
Философия технологии
iceja 01.02.2026
На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .
|
SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf
8Observer8 01.02.2026
Содержание блога
В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .
|
SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли
8Observer8 30.01.2026
Содержание блога
Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .
|
|
SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm
8Observer8 30.01.2026
Содержание блога
Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .
|
SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров
8Observer8 29.01.2026
Содержание блога
Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .
|
Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT
bedvit 28.01.2026
Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования.
Часть библиотеки BedvitCOM
Использованы. . .
|
SDL3 для Android: Загрузка PNG с альфа-каналом с помощью SDL_LoadPNG (без SDL3_image)
8Observer8 28.01.2026
Содержание блога
SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .
|