Критика проекта "гостевой книги" на технологии Spring

@Calorii · Регистрация: 13.11.2013

Студворк — интернет-сервис помощи студентам

Добрый день, недавно возникла необходимость создания "гостевой книги" на технологии Spring. Так как сам новичок, в спешном порядке принялся изучать, в качестве "view" я выбрал JSF, так как хотел заодно познакомиться и с этой технологией.
Вот что получилось: https://github.com/PolyakovSergey/guestbook.git
Очень хотелось бы услышать критику или дельный совет, заранее спасибо.

А вот и требования к гостевой книге.

Гостевая книга предоставляет возможность пользователям сайта оставлять сообщения на сайте.
Все данные введенные пользователем сохраняются в БД PostgreSQL, так же в базе данных
сохраняются данные о IP пользователя и его браузере.
Форма добавления записи в гостевую книгу должна иметь следующие поля:
User Name (цифры и буквы латинского алфавита) – обязательное поле
E-mail (формат email) — обязательное поле
Homepage (формат url) – необязательное поле
CAPTCHA (цифры и буквы латинского алфавита) – изображение и обязательное поле
Text (непосредственно сам текст сообщения, HTML тэги недопустимы) – обязательное поле
Сообщения должны выводится в виде таблицы, с возможностью сортировки по следующим
полям: User Name, e-mail, и дата добавления (как в порядке убывания, так и в обратном).
Сообщения должны разбиваться на страницы по 25 сообщений на каждой. Сортировка по
умолчанию – LIFO.
При написании проекта следует обратить внимание на защиту от XSS атак и SQL –инъекций.
Приветствуется создания простейшего дизайна с использованием
Разработка должна идти с использованием Spring

@KEKCoGEN · 06.08.2014, 01:17

В целом выглядит неплохо. С технологией JSF я незнаком, поэтому комментировать небуду. Могу сказать пару слов о спринге.

* У всех ваших entity всегда есть id поэтому его можно убрать в суперкласс и наследовать сущности у этого класса чтобы убрать повторяющиеся id.

* Добавлять create_date можно в лисенере (@EntityListeners(class=MyListener.class )) в методе с аннотацией @PrePersist (http://docs.jboss.org/hibernat... eners.html) чтобы это происходило автоматически (так же если например положить поле create_date в базовый класс, оно появится у всех сущностей (что неплохо, знать когда что создалось) и будет автоматически заполнятся при создании а так же при апдейте (аннотация @PreUpdate)

* В UserRepository вам необязательно прописывать @Query на метод findByName т.к у Spring data есть конвенция имен и следуя этой конвенции название метода findByName запустит тот же самый запрос что и у вас (http://docs.spring.io/spring-d... ories.html таблица 2.3)

* Разбивка сообщений на страницы реализованна не самым лучшим способом. При большом кол-ве сообщений, вы будите делать выборку всех сообщений из базы и затем выводить только их часть. То есть если у вас будет 100500 страниц сообщений, 99.999% пользователей никогда не будут смотреть их все. Выходит вы просто так вытащили очень много информации из базы данных (и затратили на это кучу времени). Для разбивки на страницы, spring data поддерживает реализацию Page. Для этого вам просто нужно передать в метод findAll объект PageRequest это может выглядить к примеру так

Java
1
2
3
4
    public Page<Message> getAll() {
        PageRequest p = new PageRequest(0, 10);
        return messageRepository.findAll(p);
    }

Так же можно задавать и поле сортировки.

* Все методы работы с БД лучше помечать аннотацией @Transactional чтобы в случае исключения, можно было сделать rollback

* В проекте отсутствуют обработки каких либо исключений в БД. В случае если таковые будут, результат зависит от того как обработка проводится в JSF. Чтобы управлять этим процессом, используйте ExceptionMapper

-------------------------------

Насчет XSS и SQL инъекций ничего не скажу т.к проект не запускал, но вроде классы валидаторов должны справлятся с этой работой.

В целом проект сделан довольно аккуратно.

Новые блоги и статьи Все статьи Все блоги /
Новый CodeBlocs. Версия 25.03 palva 04.01.2026 Оказывается, недавно вышла новая версия CodeBlocks за номером 25. 03. Когда-то давно я возился с только что вышедшей тогда версией 20. 03. С тех пор я давно снёс всё с компьютера и забыл. Теперь. . .	Модель микоризы: классовый агентный подход anaschu 02.01.2026 Раньше это было два гриба и бактерия. Теперь три гриба, растение. И на уровне агентов добавится между грибами или бактериями взаимодействий. До того я пробовал подход через многомерные массивы,. . .	Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .
PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .

@Calorii 0 / 0 / 0 Регистрация: 13.11.2013 Сообщений: 16

	Критика проекта "гостевой книги" на технологии Spring 05.08.2014, 19:30. Показов 2003. Ответов 1 Метки нет (Все метки) Добрый день, недавно возникла необходимость создания "гостевой книги" на технологии Spring. Так как сам новичок, в спешном порядке принялся изучать, в качестве "view" я выбрал JSF, так как хотел заодно познакомиться и с этой технологией. Вот что получилось: https://github.com/PolyakovSergey/guestbook.git Очень хотелось бы услышать критику или дельный совет, заранее спасибо. А вот и требования к гостевой книге. Гостевая книга предоставляет возможность пользователям сайта оставлять сообщения на сайте. Все данные введенные пользователем сохраняются в БД PostgreSQL, так же в базе данных сохраняются данные о IP пользователя и его браузере. Форма добавления записи в гостевую книгу должна иметь следующие поля: User Name (цифры и буквы латинского алфавита) – обязательное поле E-mail (формат email) — обязательное поле Homepage (формат url) – необязательное поле CAPTCHA (цифры и буквы латинского алфавита) – изображение и обязательное поле Text (непосредственно сам текст сообщения, HTML тэги недопустимы) – обязательное поле Сообщения должны выводится в виде таблицы, с возможностью сортировки по следующим полям: User Name, e-mail, и дата добавления (как в порядке убывания, так и в обратном). Сообщения должны разбиваться на страницы по 25 сообщений на каждой. Сортировка по умолчанию – LIFO. При написании проекта следует обратить внимание на защиту от XSS атак и SQL –инъекций. Приветствуется создания простейшего дизайна с использованием Разработка должна идти с использованием Spring 0

Критика проекта "гостевой книги" на технологии Spring

Решение