이승원 seminar0 과제#34
Conversation
| .gradle/ | ||
| build/ | ||
|
|
||
| .DS_Store No newline at end of file |
There was a problem hiding this comment.
git 마지막 newline 을 넣어주는 것이 좋은 convention 입니다.
| binderFactory: WebDataBinderFactory?, | ||
| ): User { | ||
| TODO() | ||
| val accessToken = webRequest.getHeader("Authorization")?.removePrefix("Bearer ") ?: throw AuthenticateException() |
There was a problem hiding this comment.
HttpHeaders.AUTHORIZATION을 사용하면 하드코딩을 줄일 수 있습니다.
| } catch (ex: Exception) { | ||
| when(ex) { | ||
| is SignUpBadUsernameException, is SignUpBadPasswordException -> { | ||
| ResponseEntity.status(400).build() |
There was a problem hiding this comment.
status() 는 Int 대신 HttpStatus 타입도 받는데요. HttpStatus.BAD_REQUEST 를 이용하면 하드코딩을 줄일 수 있습니다.
| TODO() | ||
| return try { | ||
| userService.signUp(request.username, request.password, request.image) | ||
| ResponseEntity.status(200).build() |
There was a problem hiding this comment.
status(200) 대신 ok()를 쓸 수 있습니다.
| when(ex) { | ||
| is AuthenticateException -> { | ||
| ResponseEntity.status(401).build() | ||
| } | ||
| else -> throw ex | ||
| } |
There was a problem hiding this comment.
요기선 when 보다 단순 if 가 더 나으려나 싶기도 하네요
| @Service | ||
| class UserServiceImpl( | ||
| private val userRepository: UserRepository, | ||
| private var id: Long = 0 |
There was a problem hiding this comment.
DB 를 안 쓰시고 이렇게 id 로 구현하신 점이 흥미로운데요.
어쩌면 설정에 DB 세팅이 없어서, DB 가 없으니 자체 구현해야겠다고 생각하신 것일 수 있겠으나, Spring Boot 기본 설정에 의해 DB 를 명시하지 않았으면 인메모리 DB 인 H2 가 사용되게 됩니다.
이렇게 개발하셨어도 테스트는 성공하지만, 사실 요청이 동시에 빠르게 많이 들어오면 분명 동시성 이슈가 생깁니다. (조만간의 세미나 회차에서 더 다뤄질 거 같네요.) DB 를 안 쓰고 이런 식으로 구현하려면, 최소한 AtomicLong 을 사용하여야 합니다.
서버에서 변경이 가능한 프로퍼티인 var을 쓸 때는 항상 주의해야 합니다.
| } else if (userRepository.findByUsername(username) != null) { | ||
| throw SignUpUsernameConflictException() | ||
| } else { | ||
| val user = UserEntity(id++, username, password, image) |
There was a problem hiding this comment.
여기서 ++ 을 하는 것이 동시성 이슈를 만드는 부분입니다.
DB 를 안 써서 이렇게 구현하더라도 AtomicLong 의 incrementAndGet()을 사용하거나 해야합니다.
PFCJeong
force-pushed
the
master
branch
2 times, most recently
from
1a8e813 to
f420cff
Compare
PFCJeong
force-pushed
the
master
branch
3 times, most recently
from
633df04 to
66e14db
Compare
2 participants
청강입니다!