search

Cha0z / springBootHomeWork

0 stars 1 forks source link

Code review #6 Web Components #6

Open hjvfyfyfy opened 5 years ago

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/b0295001194db2204dacbf258dd2032e29059048/src/main/resources/WEB_INF/shop.jsp# Water Pants це лісти а це не гарно їх виводити в такому вигляді. Тут було б добре щось придумати. Також є яка проблема ти тут виводиш всі поля з модельки, ти втрачаєш контроли на тим що ти виводиш, наприклад тобі не треба виводити ID (дуже рідко ID з бази виводиться на UI). Гарна проктика контролювати те що ти виводиш на UI.

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/b0295001194db2204dacbf258dd2032e29059048/src/main/java/com/homework/home/models/Message.java#L10

немає часу відправки повідомлення. Час завжди треба добавляти в модельки, це необхідно для дебагу і значно спрощує життя коли коли щось травляється

hjvfyfyfy commented 5 years ago

FYI по сути і тебе 2 завдання одне про jsp і друге про WebSockets, яку використовують спільні модельки, сервіси і тд, код можна булоб розбити на 3 проекта. А потім зробити імпорт через dependency, При зустрічі обговоримо

hjvfyfyfy commented 5 years ago

Щодо бази, напевно я трохи пізно пишу це але може требуло б використати H2 як базу, щоб не морочити собі голову

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/b0295001194db2204dacbf258dd2032e29059048/src/main/java/com/homework/home/web/controller/SocketEndPoint.java#L33

Немає перевірки на те що юзер з таким ніком вже є в системі

hjvfyfyfy commented 5 years ago

Тобі требуло щось своє дописати а не тільки копіпастити, а розібратися додати щось своє