ReSQL.ru
     
powered by simpleCommunicator - 2.0.49     © 2025 Programmizd 02
Мобильная версия    Контакт    Правила    FAQ    Помощь
Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр
Форумы / Java [игнор отключен] [закрыт для гостей] / Поругайте код
10 сообщений из 10, страница 1 из 1
Поругайте код
    #39801844
bobo96
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
bobo96
Гость
Здравствуйте!
Пытаюсь вникнуть в spring, сваял первый "проект")) Это просто скелет с минимальным функционалом.
Сейчас это авторизация + доступ к одному разделу для авторизированного пользователя. + по-мелочи: своя структура возвращаемого json и т.п.
То-есть на мой нубский взгляд, это та минимальная болванка, на основании которой можно двигаться дальше))
Гитхаб - https://github.com/bobo196/springdemo
Если кого не затруднит посмотрите и если что не так, расскажите пожалуйста)
Спасибо!
...
Рейтинг: 0 / 0
16.04.2019, 07:39
| Ответить | Цитировать | Написать  
Поругайте код
    #39801961
Tsyklop
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Tsyklop
Гость
Код: java
1.
2.
3.
Functions
                .getLogger(JwtAuthenticationEntryPoint.class)
                .info("User {} login successfully", loginRequest.getLogin());



Зачем и почему?

Это делается так:

Код: java
1.
private static final Logger LOGGER = LoggerFactory.getLogger(AuthController.class);
...
Рейтинг: 0 / 0
16.04.2019, 11:03
| Ответить | Цитировать | Написать  
Поругайте код
    #39801967
Tsyklop
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Tsyklop
Гость
Все классы которые служат как ответ или как данные для запроса, к примеру LoginRequest - это POJO. Было бы хорошо это объединить в одном пакете и в конце дописывать DTO.
...
Рейтинг: 0 / 0
16.04.2019, 11:05
| Ответить | Цитировать | Написать  
Поругайте код
    #39802069
lleming
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
lleming
Участник
Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
public class Functions {

    public static int getHttpCodeFromDesc(HttpStatus status) {
        String error = status.toString();
        HttpStatus httpStatus = HttpStatus.valueOf(error.substring(error.indexOf(" ") + 1));
        return httpStatus.value();
    }

    public static Logger getLogger(Class<?> clazz) {
        return LoggerFactory.getLogger(clazz);
    }

}



Про getLogger уже было.
Что за метод getHttpCodeFromDesc http code from what ? If it gets http code from something why there is expression "+1"

строка status.toString() смотрим код toString()

Код: java
1.
return this.value + " " + name();


сначала строим строку с разделителем пробелом, а в след строке делим опять по пробелу
error.substring(error.indexOf(" ") + 1)
и выводим то с чего начали.
...
Рейтинг: 0 / 0
16.04.2019, 13:09
| Ответить | Цитировать | Написать  
Поругайте код
    #39802522
bobo96
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
bobo96
Гость
Спасибо вам, все понял)
...
Рейтинг: 0 / 0
17.04.2019, 09:54
| Ответить | Цитировать | Написать  
Поругайте код
    #39802610
lleming
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
lleming
Участник
AuthController.java

Не рекомендуется делать field injection. Это провоцирует вместо того чтобы думать при внесении дополнительного фукнционала, просто докидывать еще поля в итоге имеем контроллер с 15 полями заижекченными. Такой сложно протестировать.

из предыдущего моего сообщения
If it gets http code from something why there is expression "+1"

это потому что сразу не увидел что +1 относить к substring а не к http status code + 1, и не понял, что тоже тащем та признак нехорошего кода.
...
Рейтинг: 0 / 0
17.04.2019, 11:27
| Ответить | Цитировать | Написать  
Поругайте код
    #39802780
Фотография Герой дня
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Герой дня
Участник
используй lombok с аннотацией @Slf4j над классом, а внутри класса появится поле log, которое можно использовать стандартно типа:

log.error()

и тп
...
Рейтинг: 0 / 0
17.04.2019, 13:49
| Ответить | Цитировать | Написать  
Поругайте код
    #39802829
bobo96
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
bobo96
Гость
llemingAuthController.java

Не рекомендуется делать field injection. Это провоцирует вместо того чтобы думать при внесении дополнительного фукнционала, просто докидывать еще поля в итоге имеем контроллер с 15 полями заижекченными.
Вот тут не понял, что вы имеете в виду :( Как правильно, можно пример ?
За остальное спасибо)
...
Рейтинг: 0 / 0
17.04.2019, 14:32
| Ответить | Цитировать | Написать  
Поругайте код
    #39802832
bobo96
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
bobo96
Гость
Герой дняиспользуй lombok с аннотацией @Slf4j над классом, а внутри класса появится поле log, которое можно использовать стандартно типа:

log.error()

и тп
Я попробовал логгер Slf4j, он мне каким-то замороченным показался..
...
Рейтинг: 0 / 0
17.04.2019, 14:34
| Ответить | Цитировать | Написать  
Поругайте код
    #39802868
lleming
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
lleming
Участник
bobo96llemingAuthController.java

Не рекомендуется делать field injection. Это провоцирует вместо того чтобы думать при внесении дополнительного фукнционала, просто докидывать еще поля в итоге имеем контроллер с 15 полями заижекченными.
Вот тут не понял, что вы имеете в виду :( Как правильно, можно пример ?
За остальное спасибо)

bobo96llemingAuthController.java

Не рекомендуется делать field injection. Это провоцирует вместо того чтобы думать при внесении дополнительного фукнционала, просто докидывать еще поля в итоге имеем контроллер с 15 полями заижекченными.
Вот тут не понял, что вы имеете в виду :( Как правильно, можно пример ?
За остальное спасибо)

constructor injection. Помимо того сразу видно и в глаза бросается некрасиво в конструкторе 15 полей.

Кроме того когда мало зависимостей то в тестах можно прям без мокито инлайн анонимнынми классами инжектить.
...
Рейтинг: 0 / 0
17.04.2019, 15:09
| Ответить | Цитировать | Написать  
10 сообщений из 10, страница 1 из 1
Форумы / Java [игнор отключен] [закрыт для гостей] / Поругайте код
Целевая тема:
Создать новую тему:
Автор:
Закрыть
Цитировать
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение
NoSQL.ru       Новости, Чат       Фотоальбом: Участники, Встречи
Закрыть
start [/forum/topic.php?fid=59&msg=39802829&tid=2121365]:
 0ms
get settings:
 13ms
get forum list:
 17ms
check forum access:
 4ms
check topic access:
 4ms
track hit:
 48ms
get topic data:
 13ms
get forum data:
 2ms
get page messages:
 51ms
get tp. blocked users:
 1ms
others: 234ms
total:  387ms
созд. / загр.: 387ms
x
x
Закрыть


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