Начал знакомство со spring. Поругайте код / Java

2.0.30

Планшетная версия Контакт Правила FAQ Помощь

Гость

Нов. | Гор. | Избр.

Действия ...

Доб. в избранное
Игнор. тему
Прикреп. тему
Пометить прочит. / непрочит.
Фильтр:
Сообщения автора темы
Сообщение содержит вложения
Сообщение содержит картинки
Сообщение содержит видеоклипы
Сообщение содержит аудиоклипы
Сообщение содержит картинки или видео 18+

Форумы / Java [игнор отключен] [закрыт для гостей] / Начал знакомство со spring. Поругайте код / 25 сообщений из 75, страница 1 из 3

все

26.05.2020, 12:09

#39962123

bobo96

Гость

Начал знакомство со spring. Поругайте код

Здравствуйте! Если кого не затруднит, поругайте код! Только начал знакомиться с фраймворком, хотелось бы сразу на берегу понимать, если что-то делаю не так. Пока что просто поднял сервер и настроил выборку из БД.
https://github.com/bobo766/spring
Далее хочу прикрутить авторизацию и снова приду с тем же вопросом)))
Заранее благодарю всех неравнодушных!!

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 12:47

#39962163

crutchmaster

Участник

Откуда: оттуда.
Сообщения: 2 141
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

bobo96,

Типичный helloworld с curd и имена пакетов сраным транслитом.

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 13:15

#39962188

PetroNotC Sharp

Участник

Сообщения: 10 098
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

bobo96,
Думаешь все тут скачивают что то себе на комп чтобы ругать или смотреть?
Из 100 один скачивает но не ругает. Один ругает. И один не скачивает, а ругает если дают на блюдечке и с тегом src
)))

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 14:23

#39962249

mad_nazgul

Участник

Сообщения: 3 848
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

bobo96,

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

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

И если делаете "простой CRUD", то может сразу будете использовать Sprin Data REST ?!

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 17:25

#39962360

asv79

Участник

Откуда: Тверь
Сообщения: 3 277
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

Код конечно печальный))
в репе почему <User,Integer>
почему findById принимает long а в дто поле Long

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

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 17:28

#39962362

asv79

Участник

Откуда: Тверь
Сообщения: 3 277
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

Тоесть у тебя даже есть такой сервис класс - но ты его не используешь

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 17:42

#39962376

asv79

Участник

Откуда: Тверь
Сообщения: 3 277
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

@Transactional на findById)

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

ты слабо понимаешь ООП механику ,тоесть ты зачем то используешь ЮзерРепу в этом классе,при этом ты наследуешься от этого класса где этот метод переопределен

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 17:46

#39962381

asv79

Участник

Откуда: Тверь
Сообщения: 3 277
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

Советую отложить пока spring
в нем ничего сложного нет
а вот основы ООП нужно подятнуть,так как на лицо полное непонимание механики интерфейсов- а это как бы суть Spring

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 18:20

#39962399

dakeiras

Участник

Сообщения: 502
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

asv79

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

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 18:27

#39962406

asv79

Участник

Откуда: Тверь
Сообщения: 3 277
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

dakeiras

asv79

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

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

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 18:32

#39962411

dakeiras

Участник

Сообщения: 502
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

asv79

dakeiras

пропущено...

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

спс, хорошо что я обычно обёртку использую.

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 18:56

#39962426

asv79

Участник

Откуда: Тверь
Сообщения: 3 277
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

dakeiras,

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

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 19:23

#39962449

bobo96

Гость

Начал знакомство со spring. Поругайте код

То, что код печальный, это я сам понимаю))
Причесывать буду, не вопрос.
В целом скажите, сама суть авторизации basic + token хоть верно сделана ?))) Если честно, некоторые моменты пока не до конца понятны...

asv79

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

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

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 19:44

#39962460

asv79

Участник

Откуда: Тверь
Сообщения: 3 277
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

bobo96

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

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

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

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

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

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

в представленном коде есть все но используется неправильно и написано неправильно

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 20:06

#39962476

bobo96

Гость

Начал знакомство со spring. Поругайте код

Странно, вроде делал по офф докам.. Не все конечно, но большинство.

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 20:07

#39962478

dakeiras

Участник

Сообщения: 502
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

asv79

dakeiras,

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

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

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 20:09

#39962482

bobo96

Гость

Начал знакомство со spring. Поругайте код

asv79, не покажешь на каком-нибудь небольшом примере, что именно у меня не так и как надо правильно ?

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 21:17

#39962523

asv79

Участник

Откуда: Тверь
Сообщения: 3 277
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

dakeiras

asv79

dakeiras,

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

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

вообще это используется немного не так)
а через map
сейчас показать не могу ,но если надо напиши на почту покажу пример

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 21:24

#39962530

asv79

Участник

Откуда: Тверь
Сообщения: 3 277
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

bobo96

asv79, не покажешь на каком-нибудь небольшом примере, что именно у меня не так и как надо правильно ?

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

но если очень хочется пиши мне на почту я могу провести некий мастер класс с примерами через тимс

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

26.05.2020, 22:17

#39962544

bobo96

Гость

Начал знакомство со spring. Поругайте код

asv79, спасибо, если не шутишь о_О Меня в любом случае твои готовые продукты не интересуют в том плане, что вряд ли я там что-то пойму))) Примеров типа хеллоу ворлд будет достаточно. На почту завтра отпишу.

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

27.05.2020, 07:27

#39962643

Valentin Kolesnikov

Участник

Сообщения: 3 546
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

Можете посмотреть мой репозиторий.

Java 14, spring boot 2.3.0, flyway, JavaScript.

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

Хорошего вам дня!

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

27.05.2020, 07:56

#39962649

bobo96

Гость

Начал знакомство со spring. Поругайте код

Valentin Kolesnikov, спасибо!

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

27.05.2020, 14:29

#39962826

mad_nazgul

Участник

Сообщения: 3 848
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

bobo96

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

<:o)

...

Рейтинг:

0 / 0

| Ответить | Цитировать | Написать

27.05.2020, 15:01

#39962845

mayton

Участник

Откуда: loopback
Сообщения: 53 422
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

Это ... прекрасно.

Код: 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:27

#39962866

mad_nazgul

Участник

Сообщения: 3 848
Рейтинг: 0 / 0

Начал знакомство со spring. Поругайте код

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

| Ответить | Цитировать | Написать

все

Написать

Автор*:

Ввести пароль для входа

Тема*:

Сообщение

Данное сообщение тематическое

Сообщение содержит картинки или видео 18+

Автор:

ВНИМАНИЕ! На данном подфоруме действуют строгие правила. Удостоверьтесь, что ваше сообщение соответствует им!

Форум или тема закрыты для гостей. Необходима авторизация!

Загрузить последнюю сохраненную версию

Вставить как галерею

Максимальный размер вложений: 4,0 МБ, аудио/видео: 8,0 МБ. Картинки большего размера ужимаются, если возможно.

Введите код, изображенный на картинке. Если код нечитаемый, кликните картинку, чтобы загрузить другой вариант.

Отправляя сообщение, я выражаю свое согласие с правилами форума и принимаю пользовательское соглашение.

Читали тему (0):

Читали форум (0):

Пользователи онлайн (0):

start [/forum/topic.php?fid=59&mobile=1&tid=2120791]:	0ms
get settings:	23ms
get forum list:	13ms
check forum access:	4ms
check topic access:	4ms
track hit:	45ms
get topic data:	11ms
get forum data:	3ms
get page messages:	440ms
get tp. blocked users:	1ms
others:	305ms

total:	849ms