ReSQL.ru
     
powered by simpleCommunicator - 2.0.59     © 2025 Programmizd 02
Мобильная версия    Контакт    Правила    FAQ    Помощь
Целевая тема:
Создать новую тему:
Автор:
Закрыть
Цитировать
Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр
Форумы / Java [игнор отключен] [закрыт для гостей] / Начал знакомство со spring. Поругайте код
25 сообщений из 75, страница 1 из 3
  1  все
Начал знакомство со spring. Поругайте код
    #39962123
bobo96
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
bobo96
Гость
Здравствуйте! Если кого не затруднит, поругайте код! Только начал знакомиться с фраймворком, хотелось бы сразу на берегу понимать, если что-то делаю не так. Пока что просто поднял сервер и настроил выборку из БД.
https://github.com/bobo766/spring
Далее хочу прикрутить авторизацию и снова приду с тем же вопросом)))
Заранее благодарю всех неравнодушных!!
...
Рейтинг: 0 / 0
26.05.2020, 12:09
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962163
Фотография crutchmaster
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
crutchmaster
Участник
bobo96,

Типичный helloworld с curd и имена пакетов сраным транслитом.
...
Рейтинг: 0 / 0
26.05.2020, 12:47
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962188
PetroNotC Sharp
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
PetroNotC Sharp
Участник
bobo96,
Думаешь все тут скачивают что то себе на комп чтобы ругать или смотреть?
Из 100 один скачивает но не ругает. Один ругает. И один не скачивает, а ругает если дают на блюдечке и с тегом src
)))
...
Рейтинг: 0 / 0
26.05.2020, 13:15
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962249
mad_nazgul
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
mad_nazgul
Участник
bobo96,

Зачем так сложно?!

Т.е. BaseResponse как бы не особо нужен.
ИМХО ошибки и Http коды пока лучше доверит Spring Framework

И если делаете "простой CRUD", то может сразу будете использовать Sprin Data REST ?!
...
Рейтинг: 0 / 0
26.05.2020, 14:23
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962360
Фотография asv79
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
asv79
Участник
Код конечно печальный))
в репе почему <User,Integer>
почему findById принимает long а в дто поле Long

Далее метод findById вообще не нужен он и так есть по дефолту
далее ты используешь интрефейс репы вместо ее имплементации-
тоесть тебе нужен сервис слой которой имплементирует этот интерфейс
...
Рейтинг: 0 / 0
26.05.2020, 17:25
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962362
Фотография asv79
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
asv79
Участник
Тоесть у тебя даже есть такой сервис класс - но ты его не используешь
...
Рейтинг: 0 / 0
26.05.2020, 17:28
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962376
Фотография asv79
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
asv79
Участник
@Transactional на findById)

судя по классу CustomUserDetailsService

ты слабо понимаешь ООП механику ,тоесть ты зачем то используешь ЮзерРепу в этом классе,при этом ты наследуешься от этого класса где этот метод переопределен
...
Рейтинг: 0 / 0
26.05.2020, 17:42
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962381
Фотография asv79
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
asv79
Участник
Советую отложить пока spring
в нем ничего сложного нет
а вот основы ООП нужно подятнуть,так как на лицо полное непонимание механики интерфейсов- а это как бы суть Spring
...
Рейтинг: 0 / 0
26.05.2020, 17:46
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962399
dakeiras
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
dakeiras
Участник
asv79
Код конечно печальный))
в репе почему <User,Integer>
почему findById принимает long а в дто поле Long

Далее метод findById вообще не нужен он и так есть по дефолту
далее ты используешь интрефейс репы вместо ее имплементации-
тоесть тебе нужен сервис слой которой имплементирует этот интерфейс


Кстати, как лучше, а то я изучал это лет 15 назад и уже не помню.
Примитив или обёртка?
...
Рейтинг: 0 / 0
26.05.2020, 18:20
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962406
Фотография asv79
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
asv79
Участник
dakeiras
asv79
Код конечно печальный))
в репе почему <User,Integer>
почему findById принимает long а в дто поле Long

Далее метод findById вообще не нужен он и так есть по дефолту
далее ты используешь интрефейс репы вместо ее имплементации-
тоесть тебе нужен сервис слой которой имплементирует этот интерфейс


Кстати, как лучше, а то я изучал это лет 15 назад и уже не помню.
Примитив или обёртка?


Там должен быть Long ,вроде потому что long не может быть null
я просто помню что у меня изза этого проект какой то сломался
тоесть в качестве id примитив нельзя
...
Рейтинг: 0 / 0
26.05.2020, 18:27
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962411
dakeiras
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
dakeiras
Участник
asv79
dakeiras
пропущено...


Кстати, как лучше, а то я изучал это лет 15 назад и уже не помню.
Примитив или обёртка?


Там должен быть Long ,вроде потому что long не может быть null
я просто помню что у меня изза этого проект какой то сломался
тоесть в качестве id примитив нельзя

спс, хорошо что я обычно обёртку использую.
...
Рейтинг: 0 / 0
26.05.2020, 18:32
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962426
Фотография asv79
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
asv79
Участник
dakeiras,

обертка почти всегда лучше чем примитив,а на реальных проектах мы все заворачиваем в Optional)
...
Рейтинг: 0 / 0
26.05.2020, 18:56
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962449
bobo96
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
bobo96
Гость
То, что код печальный, это я сам понимаю))
Причесывать буду, не вопрос.
В целом скажите, сама суть авторизации basic + token хоть верно сделана ?))) Если честно, некоторые моменты пока не до конца понятны...

asv79

далее ты используешь интрефейс репы вместо ее имплементации-
тоесть тебе нужен сервис слой которой имплементирует этот интерфейс

Тут не понял, о чем речь :( Можно более простым языком, для таких, как я ?)
...
Рейтинг: 0 / 0
26.05.2020, 19:23
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962460
Фотография asv79
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
asv79
Участник
bobo96

Тут не понял, о чем речь :( Можно более простым языком, для таких, как я ?)

проблема в том,что вы идете в спринг не понимая основных принципов java,поэтому у тебя возник такой вопрос

если говорить относительно представленнного кода там должны быть слои
DAO,SERVICE,CONTROLLER

попутно есть классы по работе с репозиторием- обычно это интерфейсы,которые нужно имплементировать

тоесть дао слой это интрерфейс

сервис это логика
котнроллер дергает сервис

в представленном коде есть все но используется неправильно и написано неправильно
...
Рейтинг: 0 / 0
26.05.2020, 19:44
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962476
bobo96
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
bobo96
Гость
Странно, вроде делал по офф докам.. Не все конечно, но большинство.
...
Рейтинг: 0 / 0
26.05.2020, 20:06
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962478
dakeiras
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
dakeiras
Участник
asv79
dakeiras,

обертка почти всегда лучше чем примитив,а на реальных проектах мы все заворачиваем в Optional)

Optional крутая тема, особенно Optional.ofNullable(foo).isPresent().
...
Рейтинг: 0 / 0
26.05.2020, 20:07
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962482
bobo96
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
bobo96
Гость
asv79, не покажешь на каком-нибудь небольшом примере, что именно у меня не так и как надо правильно ?
...
Рейтинг: 0 / 0
26.05.2020, 20:09
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962523
Фотография asv79
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
asv79
Участник
dakeiras
asv79
dakeiras,

обертка почти всегда лучше чем примитив,а на реальных проектах мы все заворачиваем в Optional)

Optional крутая тема, особенно Optional.ofNullable(foo).isPresent().

вообще это используется немного не так)
а через map
сейчас показать не могу ,но если надо напиши на почту покажу пример
...
Рейтинг: 0 / 0
26.05.2020, 21:17
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962530
Фотография asv79
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
asv79
Участник
bobo96
asv79, не покажешь на каком-нибудь небольшом примере, что именно у меня не так и как надо правильно ?

к сожалению все что я пишу в последнее время является коммерческой собственностью и публичных репозиториев у меня нет

но если очень хочется пиши мне на почту я могу провести некий мастер класс с примерами через тимс
...
Рейтинг: 0 / 0
26.05.2020, 21:24
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962544
bobo96
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
bobo96
Гость
asv79, спасибо, если не шутишь о_О Меня в любом случае твои готовые продукты не интересуют в том плане, что вряд ли я там что-то пойму))) Примеров типа хеллоу ворлд будет достаточно. На почту завтра отпишу.
...
Рейтинг: 0 / 0
26.05.2020, 22:17
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962643
Фотография Valentin Kolesnikov
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Valentin Kolesnikov
Участник
Можете посмотреть мой репозиторий.

Java 14, spring boot 2.3.0, flyway, JavaScript.

https://github.com/javadev/pt-backend

Хорошего вам дня!
...
Рейтинг: 0 / 0
27.05.2020, 07:27
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962649
bobo96
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
bobo96
Гость
Valentin Kolesnikov, спасибо!
...
Рейтинг: 0 / 0
27.05.2020, 07:56
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962826
mad_nazgul
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
mad_nazgul
Участник
bobo96
asv79, спасибо, если не шутишь о_О Меня в любом случае твои готовые продукты не интересуют в том плане, что вряд ли я там что-то пойму))) Примеров типа хеллоу ворлд будет достаточно. На почту завтра отпишу.


Для вас уже специально все сделали.
Вот

<:o)
...
Рейтинг: 0 / 0
27.05.2020, 14:29
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962845
Фотография mayton
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
mayton
Участник
Это ... прекрасно.

Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
@AllArgsConstructor
@NoArgsConstructor
@Accessors(chain = true)
@Getter
@Setter
@Entity
@Table (name = "goal_parameter", schema = "ptcore")
@DynamicInsert
public class GoalParameter {
...
Рейтинг: 0 / 0
27.05.2020, 15:01
| Ответить | Цитировать | Написать  
Начал знакомство со spring. Поругайте код
    #39962866
mad_nazgul
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
mad_nazgul
Участник
mayton
Это ... прекрасно.

Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
@AllArgsConstructor
@NoArgsConstructor
@Accessors(chain = true)
@Getter
@Setter
@Entity
@Table (name = "goal_parameter", schema = "ptcore")
@DynamicInsert
public class GoalParameter {



Больше половины аннотаций это lombok.
Который можно заменить на Kotlin :-)
Ну или Java 14.
...
Рейтинг: 0 / 0
27.05.2020, 15:27
| Ответить | Цитировать | Написать  
25 сообщений из 75, страница 1 из 3
  1  все
Форумы / Java [игнор отключен] [закрыт для гостей] / Начал знакомство со spring. Поругайте код
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение
NoSQL.ru       Новости, Чат       Фотоальбом: Участники, Встречи
Закрыть
start [/forum/topic.php?fid=59&msg=39962406&tid=2120791]:
 0ms
get settings:
 11ms
get forum list:
 13ms
check forum access:
 4ms
check topic access:
 4ms
track hit:
 62ms
get topic data:
 13ms
get forum data:
 3ms
get page messages:
 62ms
get tp. blocked users:
 1ms
others: 228ms
total:  401ms
созд. / загр.: 401ms
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]