|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
Это монолит на spring-boot. Есть админка на javascript. Из особенностей - большое число юнит тестов. https://github.com/javadev/pt-backend С уважением, Валентин ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 08:48 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
Самая сложная для понимания часть - это javascript. https://github.com/javadev/pt-backend/blob/master/src/main/resources/static/scripts/views/programs.js С уважением, Валентин ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 11:12 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
Значит идеальное приложение, если нет коментариев :) С уважением, Валентин ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 14:50 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
Valentin KolesnikovЗначит идеальное приложение, если нет коментариев :) С уважением, Валентин Скорее оно такое объемное что нужно время на изучение кода. А со временем бывают трудности. ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 14:54 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
Valentin Kolesnikov, а каким образом контролируется процесс установки скриптиков отсюда? https://github.com/javadev/pt-backend/tree/master/src/main/resources/db/migration ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 15:08 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
maytonValentin Kolesnikov, а каким образом контролируется процесс установки скриптиков отсюда? https://github.com/javadev/pt-backend/tree/master/src/main/resources/db/migration Добавлена зависимость для flyway и настрока для application.properties: Код: plaintext 1. 2. 3.
Код: xml 1. 2. 3. 4. 5.
С уважением, Валентин ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 15:29 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
Valentin Kolesnikov, +1 Но по тестам там смотреть особо нечего. Тесты - это не продуктив и главное чтоб они работали. ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 15:34 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
1. в корне проекта насрано 2. flyway - разделения по релизам нет, т.е. миграции выполнить можно только один раз Valentin KolesnikovИз особенностей - большое число юнит тестов 3. юнит-тесты просто на грани фантастики: Код: java 1. 2. 3. 4. 5. 6.
Вы JDK чтоли тестируете чтобы конструктор null не возвращал? ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 15:52 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
ну тесты то тестами - смотря что чел хочет оттестировать. это раз а два. структура проекта мне кажется это трэш несколько удивительна. зачем мешать сущности и простите контроллеры? вроде есть стандартная структура типа папка с сервисами, папка (пакаж) с репозиториями, пакаж с контроллером(ми) и тд. а тут просто в кучу навалено по какой-то своей неведомой структуре (я надеюсь что она всё же есть). видел как то как челы внутри контроллера фигачили коннекшн к базе данных, один раз, а другой раз фигачили коннекшн уже прям в сущности. и ее передавали контроллеру. ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 16:03 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
Андрей Панфилов1. в корне проекта насрано 2. flyway - разделения по релизам нет, т.е. миграции выполнить можно только один раз Valentin KolesnikovИз особенностей - большое число юнит тестов 3. юнит-тесты просто на грани фантастики: Код: java 1. 2. 3. 4. 5. 6.
Вы JDK чтоли тестируете чтобы конструктор null не возвращал? а... глянул. ну угу. тест ради теста. некоторые вон на геттеры и сеттеры (созданные ломбоком) тесты пишут. ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 16:04 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
andreykaTну тесты то тестами - смотря что чел хочет оттестировать. это раз а два. структура проекта мне кажется это трэш несколько удивительна. зачем мешать сущности и простите контроллеры? вроде есть стандартная структура типа папка с сервисами, папка (пакаж) с репозиториями, пакаж с контроллером(ми) и тд. а тут просто в кучу навалено по какой-то своей неведомой структуре (я надеюсь что она всё же есть). видел как то как челы внутри контроллера фигачили коннекшн к базе данных, один раз, а другой раз фигачили коннекшн уже прям в сущности. и ее передавали контроллеру. Это концепция package by feature. Для разбивки приложения на отдельные модули/приложения в будущем. С уважжением, Валентин ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 16:06 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
Опечатка скорее. Там code и expirationDate должны тестится на не пустое. ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 16:07 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
Вытянул себе, чисто для просмотра кода и проекта от более опытного разработчика нежели я :) Но еще не смотрел. Действительно нужно время... ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 16:11 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
Код: java 1. 2. 3. 4. 5. 6. 7. 8. 9. 10.
мне кажется просто поля помечать автоваерами было бы проще т.к. инжектов может быть и не один. будете конструктор на 10 полей создавать? Код: java 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21. 22. 23. 24. 25.
просто первый попавшийся копипаст: если креейт то креейт зачем скрещивать с апдейтом? да еще и так? 1) почему ин-юзер? 2) почему сущность называется inUserEmail а там стопитцот полей? включая ин_юзер? и емейл? 3) что за именование? String user_name; String device_id; камелкейс - не? 4) юзайте маппер называйте поля одинаково будет всем хорошо. 5) зачем писать реально модифицирующий данные код в СУЩНОСТИ?? (назад в п.2) 6) inUser.setD_level(userRequestDTO.getLevel() == null ? null : "" + userRequestDTO.getLevel()); -- Што это штооо? 6) вы слышали что нибудь о книжке клин код? 7) вы слышали что нибудь об аббревиатуре SOLID? 8) вы слышали что нибудь о книге Java Head First? ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 16:39 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
Valentin KolesnikovandreykaTну тесты то тестами - смотря что чел хочет оттестировать. это раз а два. структура проекта мне кажется это трэш несколько удивительна. зачем мешать сущности и простите контроллеры? вроде есть стандартная структура типа папка с сервисами, папка (пакаж) с репозиториями, пакаж с контроллером(ми) и тд. а тут просто в кучу навалено по какой-то своей неведомой структуре (я надеюсь что она всё же есть). видел как то как челы внутри контроллера фигачили коннекшн к базе данных, один раз, а другой раз фигачили коннекшн уже прям в сущности. и ее передавали контроллеру. Это концепция package by feature. Для разбивки приложения на отдельные модули/приложения в будущем. С уважжением, Валентин очень тяжело шарить по пакеджам где больше пяти классов, очень тяжело читать методы где больше пары десятков строк, очень тяжело читать наборы параметров что передаешь в методы когда их больше пяти. очень тяжело разбираться в том что делает метод если он делает всё. ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 16:42 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
...очень тяжело шарить по пакетам среди классов если все классы делают мягко говоря разные вещи. вы можете бить по пакаджам никто ж вас не ограничивает в уровнях вложенности? ) ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 16:44 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
Код: java 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19.
я конечно вас критиковать не хочу, но простите, что здесь происходит? вам надо найти юзера по логину? у вас может быть так что в системе более одного юзера с одним логином? )) а почему вы делаете тогда последний ретерн так? ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 16:50 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
andreykaT, Wow! Wow! Осади коней! ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 16:56 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
andreykaT Код: java 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19.
я конечно вас критиковать не хочу, но простите, что здесь происходит? вам надо найти юзера по логину? у вас может быть так что в системе более одного юзера с одним логином? )) а почему вы делаете тогда последний ретерн так? Да, может быть несколько пользователей с одинаковым логином в базе данных. С уважением, Валентин ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 16:59 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
што? ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 17:05 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
...и вы берете всегда последнего? )) ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 17:05 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
andreykaT...и вы берете всегда последнего? )) Приложение не даст создать несколко пользователей с одним логином. В будущем возможна деактивация пользователей без удаления. С уважением, Валентин ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 17:09 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
Valentin KolesnikovДа, может быть несколько пользователей с одинаковым логином в базе данных. С уважением, Валентин Valentin KolesnikovandreykaT...и вы берете всегда последнего? )) Приложение не даст создать несколко пользователей с одним логином. В будущем возможна деактивация пользователей без удаления. С уважением, Валентин да? ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 17:11 |
|
Покритикуйте/оцените spring-boot приложение
|
|||
---|---|---|---|
#18+
andreykaTValentin KolesnikovДа, может быть несколько пользователей с одинаковым логином в базе данных. С уважением, Валентин Valentin Kolesnikovпропущено... Приложение не даст создать несколко пользователей с одним логином. В будущем возможна деактивация пользователей без удаления. С уважением, Валентин да? В базе данных может быть несколько пользователей с одинаковым логином, а в приложении есть проверка при создании. С уважением, Валентин ... |
|||
:
Нравится:
Не нравится:
|
|||
27.06.2018, 17:20 |
|
|
start [/forum/topic.php?fid=59&msg=39666719&tid=2121599]: |
0ms |
get settings: |
11ms |
get forum list: |
15ms |
check forum access: |
3ms |
check topic access: |
3ms |
track hit: |
53ms |
get topic data: |
12ms |
get forum data: |
3ms |
get page messages: |
73ms |
get tp. blocked users: |
1ms |
others: | 340ms |
total: | 514ms |
0 / 0 |