|
|
|
Java Spring (Home Finance)
|
|||
|---|---|---|---|
|
#18+
Добрый день! По специальности я не программист, пытаюсь самостоятельно изучать Java Spring, в данный момент пишу проект "Домашняя бухгалтерия" (пока конечно это только начальная стадия написания, но то, что написано, уже работает, что не может не радовать:-)). Если не трудно, прошу посмотреть то, что я уже написала, и дать рекомендации/советы/комментарии. Буду благодарна за полезную конструктивную критику!:-) Ссылка на Github: https://github.com/FeschenkoNatalia/Spring ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 03.12.2015, 17:18 |
|
||
|
Java Spring (Home Finance)
|
|||
|---|---|---|---|
|
#18+
очень много бесполезных интерфейсов (например, TransactionService.java) ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 03.12.2015, 17:22 |
|
||
|
Java Spring (Home Finance)
|
|||
|---|---|---|---|
|
#18+
http://www.javapractices.com/topic/TopicAction.do?Id=205 SpringDBConfig -> хардкод testdb. Если база встроеная, конечно, пофигу. Но чаще она внешная. И почему test? TransactionController -> Много хардкода в showAddTransactionForm() web.xml, вроде не сильно нужен? Service вылился в бесполезный класс. transactionform.jsp какой-то монстрячий вышел. Это, в целом, косяк JSP. Трудно пере-использовать код. В целом - чистенько и аккуратненько. Это большой плюс. Но особо ревьювить нечего - обычный CRUD. Я бы посоветовал начать смотреть Spring Data, JPA, Spring Boot, Thymeleaf. Возможно, jOOQ. Потому что куча кода в DAO для обычного CRUD это подход из прошлого десятилетия. Да и war такому приложению не шибко нужен. Можно было было jar-ником запаковать в standalone модуль. Make jar not war! ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 03.12.2015, 17:36 |
|
||
|
Java Spring (Home Finance)
|
|||
|---|---|---|---|
|
#18+
grasoff.netочень много бесполезных интерфейсов (например, TransactionService.java) А я тоже JDK Proxy люблю, а не CGLIB. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 03.12.2015, 17:37 |
|
||
|
Java Spring (Home Finance)
|
|||
|---|---|---|---|
|
#18+
BlazkowiczMake jar not war! Смотрю тут доклад по Spring Boot. Просто кладезь! авторEvery time you use a field injection a unit test dies. http://olivergierke.de/2013/11/why-field-injection-is-evil/ ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 03.12.2015, 19:35 |
|
||
|
Java Spring (Home Finance)
|
|||
|---|---|---|---|
|
#18+
Natalia_141277, Код: html 1. 2. убрать из страниц в каталоге transactions и перенести это безобразие в header.jsp наряду с открывающим тегом body ну и заодно закрывающие теги body и html перенести в footer.jsp дабы не плодить на выходе невалидные HTML страницы ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 03.12.2015, 20:04 |
|
||
|
Java Spring (Home Finance)
|
|||
|---|---|---|---|
|
#18+
BlazkowiczBlazkowiczMake jar not war! Смотрю тут доклад по Spring Boot. Просто кладезь! авторEvery time you use a field injection a unit test dies. http://olivergierke.de/2013/11/why-field-injection-is-evil/ Ни слова не увидел там о буте. Или о каком докладе идет речь? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 04.12.2015, 09:14 |
|
||
|
Java Spring (Home Finance)
|
|||
|---|---|---|---|
|
#18+
JonnySKSНи слова не увидел там о буте. Или о каком докладе идет речь? Доклад тут оффтопик: ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 04.12.2015, 10:15 |
|
||
|
Java Spring (Home Finance)
|
|||
|---|---|---|---|
|
#18+
Natalia_141277, добрый день! В данной статье на хабре описывается пример реализации домашней бухгалтерии http://habrahabr.ru/company/haulmont/blog/235617/. Там же есть ссылка на исходники. Чтобы по-простому развернуть и посмотреть у себя проект, скачайте платформу (все на JAVA, в т.ч. с использованием Spring) и студию: https://www.cuba-platform.ru/download. Там правда надо зарегистрироваться, но это все быстро делается. После запуска студии в Samples будет проект akkount. Откроете -там будут доступны все исходники, которые можно просматривать как из студии, так и из распространенных IDE. Надеюсь это поможет Вам в Ваших начинаниях:) ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 15.12.2015, 15:39 |
|
||
|
Java Spring (Home Finance)
|
|||
|---|---|---|---|
|
#18+
Добрый день! Спасибо большое всем!:-) Ваши комментарии/замечания/рекомендации были очень полезны для меня, как для начинающей.:-) В последующем буду сообщать о прогрессе/ходе работ. Надеюсь, с Вашей помощью все получится!:-) ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 15.12.2015, 16:03 |
|
||
|
|
start [/forum/topic.php?fid=59&msg=39120130&tid=2124554]: |
0ms |
get settings: |
9ms |
get forum list: |
16ms |
check forum access: |
3ms |
check topic access: |
3ms |
track hit: |
141ms |
get topic data: |
11ms |
get forum data: |
2ms |
get page messages: |
54ms |
get tp. blocked users: |
1ms |
| others: | 214ms |
| total: | 454ms |
| 0 / 0 |
