S
Ток что проверил)
Там инструкция даже есть. И сервисный файлик само собой)
Я это понимаю как SA/DevOps.
Мне интересно больше критика кода)
Size: a a a
2020 August 21
V
O
Но мне важно мнение насчёт кода. Я понимаю что первый проект и там много ужасного - но что в первую очередь плохо?
Сделайте нормальное логирование через, например, slf4j+logback. e.printStackTrace() вызывают ужас
V
Адрес сменится - и каждый раз только перекомпиляция
S
Спасибо за совет! Я даже как-то не подумал
S
Сделайте нормальное логирование через, например, slf4j+logback. e.printStackTrace() вызывают ужас
Было в планах, но slf4j пока неосилил)
ch
Ток что проверил)
просмотри весь свой код и исправь все так чтобы он соответствовал приницпу единственной ответственности, какого фига у тебя в SessionHandler происходит проверка на /start, почему запросы в БД разбросаны по всему проекту
ch
https://github.com/rubenlagus/TelegramBots/tree/master/telegrambots-extensions вот этот проект вообще содзан для того чтобы обработку команда не писать самому а переложить все на руки библиотеки
O
getBot_username - в джаве обычно camelCase, у вас как-то странно смешано
static final переменные - это константы, UPPER_CASE
static final переменные - это константы, UPPER_CASE
S
просмотри весь свой код и исправь все так чтобы он соответствовал приницпу единственной ответственности, какого фига у тебя в SessionHandler происходит проверка на /start, почему запросы в БД разбросаны по всему проекту
Спасибо за критику, буду исправляться
O
Было в планах, но slf4j пока неосилил)
осильте, если это проект для резюме. Одна из базовых вещей в коммерческой разработке
да и несложно там
да и несложно там
S
осильте, если это проект для резюме. Одна из базовых вещей в коммерческой разработке
да и несложно там
да и несложно там
Да, вообще планировал как раз послушать критику и исправить по советам тех у кого опыт. Это для резюме планировалось.
ch
все что связано с телеграммом в отдельный пакет с название телеграм и так далее
S
осильте, если это проект для резюме. Одна из базовых вещей в коммерческой разработке
да и несложно там
да и несложно там
Обязательно :)
S
getBot_username - в джаве обычно camelCase, у вас как-то странно смешано
static final переменные - это константы, UPPER_CASE
static final переменные - это константы, UPPER_CASE
Понял. Да, косяк.
Название этого сгегерировала идея насколько помню а дальше я забил
Название этого сгегерировала идея насколько помню а дальше я забил
S
все что связано с телеграммом в отдельный пакет с название телеграм и так далее
Спасибо)
i
Ребят, вопрос к тем кто бэк пишет
i
i think it's okay
Ребят, вопрос к тем кто бэк пишет
У меня есть боди, в котором должно быть только integer
т.е код сам нормализует значение в Integer.
Разраб говорит, что мол - ну и ок, но он это специально не делал.
Я вот думаю - это реально "ок" и похуй - типа мы оба знаем и ладно.
Или доебаться и попросить чтоб сделал "нормально"
{
"rating": 5,
"serviceId": 38,
"text": 20,
"userId": 121
}
но если я отправлю в запросе все значения как string - ошибки не будет.т.е код сам нормализует значение в Integer.
Разраб говорит, что мол - ну и ок, но он это специально не делал.
Я вот думаю - это реально "ок" и похуй - типа мы оба знаем и ладно.
Или доебаться и попросить чтоб сделал "нормально"
VP
Если єто код чисто для Вас - тогда пох.
Но вообще - говняная практика писать код и надеться что его никто не увидит.
Присто вьі "вкидьіваете" следующих разработчиков - осознано увеличиваете время изучения код для других!
Но вообще - говняная практика писать код и надеться что его никто не увидит.
Присто вьі "вкидьіваете" следующих разработчиков - осознано увеличиваете время изучения код для других!
VP
RequestBody{
private Integer rating;
private Integer serviceId;
private Integer text;
private Integer userId;
}
Вам ведь не тяжело так написать? А потом через Optional чекнуть валидность значения или повесить валидатор на бади?
private Integer rating;
private Integer serviceId;
private Integer text;
private Integer userId;
}
Вам ведь не тяжело так написать? А потом через Optional чекнуть валидность значения или повесить валидатор на бади?