ReSQL.ru
     
powered by simpleCommunicator - 2.0.56     © 2025 Programmizd 02
Мобильная версия    Контакт    Правила    FAQ    Помощь
Целевая тема:
Создать новую тему:
Автор:
Закрыть
Цитировать
Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр
Форумы / Java [игнор отключен] [закрыт для гостей] / Покритикуйте/оцените spring-boot приложение
12 сообщений из 37, страница 2 из 2
  2  все
Покритикуйте/оцените spring-boot приложение
    #39666745
Фотография Petro123
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Petro123
Участник
А в чем необходимость иметь одинаковый логин?
...
Рейтинг: 0 / 0
27.06.2018, 17:42
| Ответить | Цитировать | Написать  
Покритикуйте/оцените spring-boot приложение
    #39666747
Фотография Valentin Kolesnikov
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Valentin Kolesnikov
Участник
Petro123А в чем необходимость иметь одинаковый логин?

Это может быть нужно чтобы не удалять пользователя а деактивировать.

С уважением, Валентин
...
Рейтинг: 0 / 0
27.06.2018, 17:46
| Ответить | Цитировать | Написать  
Покритикуйте/оцените spring-boot приложение
    #39666750
andreykaT
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
andreykaT
Участник
TsyklopValentin Kolesnikov,
каким образом может быть пользователи с одинаковыми логинами если проверка есть? как они там окажутся?
никак. воитель об этом не подумал )) во-первых, я согласен (можно отбросить вообще весь остальной код т.к. комментировать его можно бесконечно), может быть случай когда логины одинаковы. НО. у юзера должен быть УИД по которому он идентифицируется и это либо логин либо имя либо еще что то либо набор неких параметров дающих уникальный ключ. тут же я подозреваю, тс банально глубоко не анализировал задачу

Валентин, по конкретно этому случаю (другие обсуждать можно вечность) у вас банально нет консистенси.
вы говорите может быть много одинаковых логинов у разных юзеров но при логине вы берете первый из списка?)) а как же другие юзеры? т.е. есть маша с логином А есть даша с логином А есть саша с логином А. кто то из них (скажем саша) вводит логин А и... логинится под машей? )) я не уверен что это корректное поведение. а если учесть что гетОлл вам вообще не гарантирует ордеринг )) у вас будут логиниться рандомно с разными кредами от разных юзеров. замечательно. ))
...
Рейтинг: 0 / 0
27.06.2018, 17:47
| Ответить | Цитировать | Написать  
Покритикуйте/оцените spring-boot приложение
    #39666753
Фотография Petro123
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Petro123
Участник
andreykaTНО. у юзера должен быть УИД по которому он идентифицируется и это либо логин либо имя либо еще что то либоразумеется.
Если логин с повтором, тогда это не логин))))))
...
Рейтинг: 0 / 0
27.06.2018, 17:51
| Ответить | Цитировать | Написать  
Покритикуйте/оцените spring-boot приложение
    #39666755
Фотография Valentin Kolesnikov
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Valentin Kolesnikov
Участник
andreykaTTsyklopValentin Kolesnikov,
каким образом может быть пользователи с одинаковыми логинами если проверка есть? как они там окажутся?
никак. воитель об этом не подумал )) во-первых, я согласен (можно отбросить вообще весь остальной код т.к. комментировать его можно бесконечно), может быть случай когда логины одинаковы. НО. у юзера должен быть УИД по которому он идентифицируется и это либо логин либо имя либо еще что то либо набор неких параметров дающих уникальный ключ. тут же я подозреваю, тс банально глубоко не анализировал задачу

Валентин, по конкретно этому случаю (другие обсуждать можно вечность) у вас банально нет консистенси.
вы говорите может быть много одинаковых логинов у разных юзеров но при логине вы берете первый из списка?)) а как же другие юзеры? т.е. есть маша с логином А есть даша с логином А есть саша с логином А. кто то из них (скажем саша) вводит логин А и... логинится под машей? )) я не уверен что это корректное поведение. а если учесть что гетОлл вам вообще не гарантирует ордеринг )) у вас будут логиниться рандомно с разными кредами от разных юзеров. замечательно. ))

Согласен тут нужно переделывать.

В приложении оставили только facebook login. E-mail логин остался как устаревший.

С уважением, Валентин
...
Рейтинг: 0 / 0
27.06.2018, 17:52
| Ответить | Цитировать | Написать  
Покритикуйте/оцените spring-boot приложение
    #39666757
andreykaT
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
andreykaT
Участник
Valentin KolesnikovPetro123А в чем необходимость иметь одинаковый логин?

Это может быть нужно чтобы не удалять пользователя а деактивировать.

С уважением, Валентин

User user = repo.findByLoginAndDeletedIsNull(String login);
параллельно вешаем констрейнт на базу что в ней не может жить больше одного юзера с конкретным логином и делетед нулл.

вот и весь код.
...
Рейтинг: 0 / 0
27.06.2018, 17:52
| Ответить | Цитировать | Написать  
Покритикуйте/оцените spring-boot приложение
    #39666758
Фотография Petro123
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Petro123
Участник
Valentin Kolesnikovне удалять пользователя а деактивировать.если бизнесу надо (не программисту!!) то вводите в Модель понятие Уволенный юзверь.
...
Рейтинг: 0 / 0
27.06.2018, 17:53
| Ответить | Цитировать | Написать  
Покритикуйте/оцените spring-boot приложение
    #39666772
Андрей Панфилов
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Андрей Панфилов
Участник
maytonОпечатка скорее. Там code и expirationDate должны тестится на не пустое.не... там половина тестов таких. Вероятнее всего чтобы оно через побольше методов проходило и получить "внушительное" покрытие кода тестами
...
Рейтинг: 0 / 0
27.06.2018, 18:15
| Ответить | Цитировать | Написать  
Покритикуйте/оцените spring-boot приложение
    #39666776
Фотография mayton
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
mayton
Участник
Возможно готовили рефакторинг и перестарались. Тут КМК важнее закрепить текущее поведение.
...
Рейтинг: 0 / 0
27.06.2018, 18:21
| Ответить | Цитировать | Написать  
Покритикуйте/оцените spring-boot приложение
    #39746243
Фотография Valentin Kolesnikov
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Valentin Kolesnikov
Участник
Недавно перешёл на spring-boot 2.1.1

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

С уважением, Валентин
...
Рейтинг: 0 / 0
11.12.2018, 18:37
| Ответить | Цитировать | Написать  
Покритикуйте/оцените spring-boot приложение
    #39746547
pavel_nv
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
pavel_nv
Участник
Valentin Kolesnikov,

как-то странно Optional используется
Код: java
1.
2.
3.
4.
5.
6.
7.
    ExerciseResponseDTO findOne(Long id) {
        final Exercise exercise = exerciseRepository.findById(id).orElse(null);
        if (exercise == null) {
            throw new ResourceNotFoundException("Exercise with id " + id + " not found.");
        }
        return exerciseToDto(exercise);
    }


вместо
Код: java
1.
2.
3.
4.
    ExerciseResponseDTO findOne(Long id) {
        final Optional<Exercise> exercise = exerciseRepository.findById(id);
        return exercise.map(this::exerciseToDto).orElseThrow(() -> new ExerciseNotFoundException(id));
    }
...
Рейтинг: 0 / 0
12.12.2018, 11:45
| Ответить | Цитировать | Написать  
Покритикуйте/оцените spring-boot приложение
    #39746856
Фотография Valentin Kolesnikov
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Valentin Kolesnikov
Участник
pavel_nvValentin Kolesnikov,

как-то странно Optional используется
Код: java
1.
2.
3.
4.
5.
6.
7.
    ExerciseResponseDTO findOne(Long id) {
        final Exercise exercise = exerciseRepository.findById(id).orElse(null);
        if (exercise == null) {
            throw new ResourceNotFoundException("Exercise with id " + id + " not found.");
        }
        return exerciseToDto(exercise);
    }


вместо
Код: java
1.
2.
3.
4.
    ExerciseResponseDTO findOne(Long id) {
        final Optional<Exercise> exercise = exerciseRepository.findById(id);
        return exercise.map(this::exerciseToDto).orElseThrow(() -> new ExerciseNotFoundException(id));
    }



Можно переделать, согласен.

С уважением, Валентин
...
Рейтинг: 0 / 0
12.12.2018, 17:57
| Ответить | Цитировать | Написать  
12 сообщений из 37, страница 2 из 2
  2  все
Форумы / Java [игнор отключен] [закрыт для гостей] / Покритикуйте/оцените spring-boot приложение
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение
NoSQL.ru       Новости, Чат       Фотоальбом: Участники, Встречи
Закрыть
start [/forum/topic.php?fid=59&msg=39746243&tid=2121599]:
 0ms
get settings:
 8ms
get forum list:
 11ms
check forum access:
 3ms
check topic access:
 3ms
track hit:
 81ms
get topic data:
 9ms
get forum data:
 3ms
get page messages:
 50ms
get tp. blocked users:
 1ms
others: 12ms
total:  181ms
созд. / загр.: 181ms
x
x
Закрыть


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