feat : auth 유효 검증 api 구현 #108
Conversation
|
뭐야 아니네;; 이거 조건이 넘 헷갈리는데 로비가 존재하는 경우 false인데 로비가 존재하지만 게임이 존재하지 않는 경우 true이면 로비만 존재하는 경우 false가 맞는건가요? 그리고 로비와 게임이 존재하는데 이름도 존재하면 true인건가요?? 그러면 로비가 존재하고 게임이 없거나, 있다면 valid해야하는 건가요? |
저도 이거 헷갈리던데.... 우선 로비 존재시 게임 존재 안할 경우 일단 로비가 살아있고 해당 auth의 유저가 방에 존재할때 true로 한 이유는 이 기능의 이유가 튕기거나 나가졌을때를 위함인데 이때 false일 경우 로비에서 튕길 경우 돌아갈 방법이 없기 때문이라고 생각해서 true로 반환하였습니다. |
| @@ -81,4 +81,16 @@ public GameResultResponse findResult(final String code) { | |||
| } | |||
| return GameResultResponse.from(game); | |||
| } | |||
|
|
|||
| public boolean isValid(final String code, final String name) { | |||
There was a problem hiding this comment.
false 조건을
로비 가없을때,
로비에서 게임시작전 플레이어(이름)가 없을때,
게임이 있을때 플레이어(이름)가 없는경우
라고 생각하면되는거지?
There was a problem hiding this comment.
여러번 번복해서 죄송한데, game exists 라는 네이밍이 개인적으로는 어색한 것 같아요. 사실 로비가 있는것도 게임이 생성된 것이니 괜찮은 것 같기도 하지만 딱 한번에 와닿지는 않는 느낌?
There was a problem hiding this comment.
네이밍이라고 하면 api말하시는건가요? 아니면 controller의 isGameExist말하시는 건가요? 저도 하면서 exists보다는 auth의 유효성 검사니까 valid가 더 맞는거 같던데...
There was a problem hiding this comment.
|
|
||
| public boolean isValid(final String code, final String name) { | ||
| Optional<Lobby> lobby = lobbyRepository.findById(code); | ||
| Optional<Game> game = gameRepository.findById(code); |
There was a problem hiding this comment.
이거 조건문 이후에 추가하면 얼리 리턴되서 굳이 안가져올수도 있을 것 같아용
There was a problem hiding this comment.
| if (!game.isPresent()) { | ||
| return lobby.get().isParticipantExist(name); | ||
| } | ||
| return game.get().isPlayerExist(name); |
There was a problem hiding this comment.
역시 개발자는 코드로 이야기하나보네요 ㅋㅋㅋ 이건 이해가 확실히 되네요.
| @@ -81,4 +81,16 @@ public GameResultResponse findResult(final String code) { | |||
| } | |||
| return GameResultResponse.from(game); | |||
| } | |||
|
|
|||
| public boolean isValid(final String code, final String name) { | |||
There was a problem hiding this comment.
여러번 번복해서 죄송한데, game exists 라는 네이밍이 개인적으로는 어색한 것 같아요. 사실 로비가 있는것도 게임이 생성된 것이니 괜찮은 것 같기도 하지만 딱 한번에 와닿지는 않는 느낌?
* feat : auth 유효 검증 api 구현 * refactor : game 조회 얼리 리턴 이후로 변경 * feat : exist 네이밍 valid로 변경
close #107
여러가지 조건을 더 추가하였습니다.