[BE] 이근표 강화하기 #2

rootTiket · 2024-05-29T00:46:32Z

❓ 어떤 기능을 구현하였나요?

✅ 회원가입 기능 구현

POST users/register

결과 사진

회원가입 기능을 구현합니다. 사용자의 이메일, 이름, 비밀번호를 입력받아 회원가입을 진행합니다.
중복 로그인 시 예외를 처리했습니다.

✅ 로그인 기능 구현

POST users/login

결과 사진

로그인 기능을 구현합니다. 사용자의 이메일, 비밀번호를 입력받아 로그인을 진행합니다. 만약 아이디나 비밀번호가 틀릴경우 예외를 처리했습니다.

✅ 강화기능 구현

POST /enhance

결과 사진

강화권 소진 시 예외

강화기능을 구현합니다. requestBody의 강화권 사용 유무에 따라 달라지는 확률을 처리했습니다.
확률 강화권을 모두 사용했을경우 예외메시지도 따로 처리했습니다.

✅ 아이템 생성 구현

POST /items

결과 사진

아이템 생성을 구현합니다. 아이템 이름을 요청받아 db에 저장합니다. 로그인을 하지 않았을 경우 예외를, 이미 아이템을 가지고 있을 경우 예외를 각각 처리하였습니다.

✅ 아이템 조회 구현

GET /items

결과 사진

아이템 조회를 구현합니다.

✅ 아이템Top10 조회 구현

GET /items/top10

결과 사진

아이템 Top10 조회를 구현합니다.

🤔 구현 중 무엇을 고민했을까요?

강화로직을 어떻게 분리해야 할까?

최초 구현 코드

public EnhanceResponse executeEnhance(Authentication authentication, EnhanceRequest request) {
        User user = userRepository.findByEmail(authentication.getName()).orElseThrow(UserNotFoundException::new);
        Weapon weapon = weaponRepository.findByUser(user).orElseThrow(WeaponNotFoundException::new);
        Random random = new Random();
        Integer increase = ((isUsedIncreaseProbability(user,request.useIncreaseProbability()))? 30 : 0);
        WeaponResponseMessage weaponResponseMessage;
        Integer level = weapon.getLevel();
        Level currentLevel = getValidLevel(level);
        if (random.nextInt(100) < currentLevel.getSuccessRate()+increase) {
            level = weapon.getLevel() + 1;
            weaponResponseMessage = UPGRADE_SUCCESS;
        } else {
            weaponResponseMessage = UPGRADE_FAILED;
            if (random.nextInt(100) < Math.min(currentLevel.getDestroyRate(), MAX_DESTROY_RATE)) {
                weaponResponseMessage = LEVEL_DOWN;
                level = weapon.getLevel() - 1;
                if (random.nextInt(100) < 50) {
                    weaponResponseMessage = DESTROYED;
                    level = 0;
                }
            }
        }
        weaponRepository.save(Weapon.updateWeapon(weapon, level));
        return new EnhanceResponse(weaponResponseMessage, level);
    }

아무리 봐도 읽기 쉽지 않은 코드

기존 강화는 weaponEnhance(아이템 관련 로직을 처리하는 함수) 에 모두 구현되어 있었습니다.

함수 분리

private EnhanceResponse enhanceWeapon(Weapon weapon, Integer increase) {
        Random random = new Random();
        Integer level = weapon.getLevel();
        Level currentLevel = getValidLevel(level);

        if (random.nextInt(100) < currentLevel.getSuccessRate() + increase) {
            return new EnhanceResponse(UPGRADE_SUCCESS, level + 1);
        } else {
            return handleFailure(random, level, currentLevel);
        }
    }

    private EnhanceResponse handleFailure(Random random, Integer level, Level currentLevel) {
        if (random.nextInt(100) < Math.min(currentLevel.getDestroyRate(), MAX_DESTROY_RATE)) {
            level--;
            if (random.nextInt(100) < 50) {
                return new EnhanceResponse(DESTROYED, 0);
            } else {
                return new EnhanceResponse(LEVEL_DOWN, level);
            }
        }
        return new EnhanceResponse(UPGRADE_FAILED, level);
    }

    private Integer calculateIncreaseProbability(User user, EnhanceRequest request) {
        return isUsedIncreaseProbability(user, request.useIncreaseProbability()) ? 30 : 0;
    }

    public Boolean isUsedIncreaseProbability(User user, Boolean useIncreaseProbability) {
        if (useIncreaseProbability && user.getIncreasingProbability() <= 0) {
            throw new NoItemException();
        } else if (useIncreaseProbability) {
            userRepository.save(User.decreaseItem(user));
            return true;
        } else {
            return false;
        }
    }

가독성도, 유지보수도 어려운 코드 인 것 같아 확률을 얻어오는 함수, 강화를 담당하는 함수, 강화실패시 로직을 처리하는 함수를 분리하자는 생각으로 리펙토링을 진행하였습니다.

클래스 분리

// weaponService
public EnhanceResponse executeEnhance(Authentication authentication, EnhanceRequest request) {
        User user = userRepository.findByEmail(authentication.getName()).orElseThrow(UserNotFoundException::new);
        Weapon weapon = weaponRepository.findByUser(user).orElseThrow(WeaponNotFoundException::new);
        Integer increase = calculateIncreaseProbability(user, request);
        EnhanceResponse result = weapon.enhance(increase);
        weaponRepository.save(weapon);
        return new EnhanceResponse(result.weaponResponseMessage(), result.level());
    }

// weaponEntity
public EnhanceResponse enhance(Integer increaseProbability) {
        Random random = new Random();
        Level currentLevel = Level.getLevelByNumber(this.level).get();

        if (random.nextInt(RANDOM_BOUND) < currentLevel.getSuccessRate() + increaseProbability) {
            this.level++;
            return new EnhanceResponse(UPGRADE_SUCCESS, this.level);
        } else {
            return handleFailure(random, currentLevel);
        }
    }

    private EnhanceResponse handleFailure(Random random, Level currentLevel) {
        if (random.nextInt(RANDOM_BOUND) < Math.min(currentLevel.getDestroyRate(), MAX_DESTROY_RATE)) {
            if (random.nextInt(RANDOM_BOUND) < 50) {
                this.level = 0;
                return new EnhanceResponse(DESTROYED, this.level);
            } else {
                this.level--;
                return new EnhanceResponse(LEVEL_DOWN, this.level);
            }
        } else {
            return new EnhanceResponse(UPGRADE_FAILED, this.level);}
    }

💭 어차피 강화는 무기(아이템)의 수치를 변경하는 것 아닌가? 클래스를 분리해보자!

💭 아쉬운 점은?

스트림, 람다를 공부 하고 있어 최대한 활용해보려고 노력했습니다. 또한 기존이라면 서비스에서 처리해야 할 것 같다 하는 부분도 더 고민해 보며 객체에서 변경 가능한가? , 이걸 서비스에서 처리하는게 맞나? 를 고민해보며 코드를 짰던 것 같습니다.
클래스 분리 한 강화 코드에서 반환 형태를 ResponseDto로 넘기는 부분은 코드의 의미가 명확하지 못해서 다른 클래스로 전달하여 처리하는 로직으로 수정하려고 합니다.

hoonyworld

고생하셨습니다!

hoonyworld · 2024-05-30T10:19:47Z

src/main/java/leets/enhance/domain/user/domain/User.java

+
+@Getter
+@NoArgsConstructor
+@AllArgsConstructor


매개변수가 있는 생성자를 대체하기 위해 정적 팩토리 메서드를 사용하신 것이라면, 해당 어노테이션은 필요가 없는 어노테이션이라 생각되는데 혹시 어떤 이유로 사용하셨을까요?

구현된 정적 팩토리는 유저가 가지고 있는 확률아이템의 개수를 감소시키는 메서드 였습니다!

hoonyworld · 2024-05-30T10:29:05Z

src/main/java/leets/enhance/domain/user/presentation/UserController.java

+import static org.springframework.http.HttpStatus.OK;
+
+@RestController
+@EnableJpaAuditing


@EnableJpaAuditing를 컨트롤러에 어노테이션을 붙여주신 이유가 궁금합니다!

저는 JpaAuditingConfig라는 auditing 기능을 담당하는 클래스를 만들고, 이 클래스에 해당 어노테이션을 붙여주는데 그 이유는 다음과 같습니다.
한 번 참고해 보시면 좋을 것 같아요!

https://hellorennon.tistory.com/9

지적 감사합니다! 수정하였습니다!

hoonyworld · 2024-05-30T10:32:32Z

src/main/java/leets/enhance/domain/weapon/Level.java

UserResponseMessage enum 에서는 @Getter와 @AllArgsConstructor로 처리해주셨는데, 해당 클래스에서는 직접 getter와 생성자를 만드신 이유가 궁금합니다

서비스로직 중 level 객체에서 정보를 가져와야 하는 부분이 있어 필요에 의해 만들었습니다!

KoungQ

고생하셨습니당

KoungQ · 2024-05-30T10:44:50Z

src/main/java/leets/enhance/domain/user/application/UserService.java

+    public User checkValidUser(LoginRequest loginRequest) {
+        if (!(userRepository.findByEmail(loginRequest.email()).isPresent())) {
+            throw new InvalidIdException();
+        } else if (!(userRepository.findByPassword(loginRequest.password())).isPresent()) {


해싱도 하면 좋을 거 같아요 :)

누락된것 같습니다 감사합니다!

KoungQ · 2024-05-30T10:48:51Z

src/main/java/leets/enhance/domain/weapon/Level.java

+        public static Optional<Level> getLevelByNumber(int levelNumber) {
+                return Arrays.stream(values())
+                        .filter(level -> level.getLevel() == levelNumber)
+                        .findFirst();


7레벨 넘게 된다면 리턴 값이 null 일 것 같아요

로직 분리 중 누락되었네요 감사합니다

KoungQ · 2024-05-30T10:51:57Z

src/main/java/leets/enhance/domain/weapon/application/WeaponService.java

+    private static final int MAX_DESTROY_RATE = 50;
+
+    public void createWeapon(Authentication authentication, CreateWeaponRequest createWeaponRequest) {
+        User user = userRepository.findByEmail(authentication.getName()).orElseThrow(UserNotFoundException::new);


User 객체를 가져오는 로직이 메서드마다 동일하게 존재하는데 컨트롤러 내부에서 하나의 메서드로 뺴두고 호출하면 중복도 피하고 가독성 면에서도 좋을 것 같아요!

참고하겠습니다 !

KoungQ · 2024-05-30T10:55:05Z

src/main/java/leets/enhance/global/error/GlobalExceptionHandler.java

+@Slf4j
+@RestControllerAdvice
+@RequiredArgsConstructor
+public class GlobalExceptionHandler {
+    private static final String LOG_FORMAT = "Class : {}, Code : {}, Message : {}";
+    @ExceptionHandler(ServiceException.class)
+    public ResponseDto handleApplicationException(ServiceException ex) {
+        log.error(LOG_FORMAT, ex.getClass().getSimpleName(), ex.getErrorCode(), ex.getMessage());
+        return ResponseDto.of(ex.getErrorCode(), ex.getMessage());
+    }
+}


덕분에 저도 컨트롤러를 간결하게 작성할 수 있었어요
에러는 다른 곳에서 핸들링 하는 게 보기 좋네요

감사합니다 :)

leejuae · 2024-05-30T11:21:43Z

src/main/java/leets/enhance/domain/weapon/application/WeaponService.java

+        if (useIncreaseProbability && user.getIncreasingProbability() <= 0) {
+            throw new NoItemException();
+        } else if (useIncreaseProbability) {
+            userRepository.save(User.decreaseItem(user));
+            return true;
+        } else {
+            return false;
+        }
+    }


얼리리턴 형식으로 바꿀 수 있을거 같아요!

leejuae · 2024-05-30T11:22:15Z

src/main/java/leets/enhance/domain/weapon/domain/Weapon.java

+            if (random.nextInt(RANDOM_BOUND) < 50) {
+                this.level = 0;
+                return new EnhanceResponse(DESTROYED, this.level);
+            } else {
+                this.level--;
+                return new EnhanceResponse(LEVEL_DOWN, this.level);
+            }
+        } else {
+            return new EnhanceResponse(UPGRADE_FAILED, this.level);}
+    }


이것도 얼리리턴 형식으로 바꾸면 더 깔끔할 것 같습니다!

leejuae

최고!

rootTiket added 15 commits May 20, 2024 23:51

feat: responseDto 생성 및 초기세팅

2712c22

feat: 에러 처리 세팅

ac6278b

feat: JWT 세팅

99840a0

feat: user엔티티 생성

086a45b

feat: 유저 예외 구현

bf86a34

feat: 회원가입 로직 구현 , email 중복 체크 구현

d4b3a33

fix: 테이블 이름은 user에서 users 로 변경

09b1d8d

feat: 로그인 구현

6c1c09a

refactor: 명확하게 메서드 이름을 변경

2e49f51

feat: 강화기능 구현

64b3773

feat: 확률증가 아이템 사용로직 추가

1f69da7

feat: 아이템 조회 기능 구현

bca6370

feat: 상위 10개 아이템 조회기능 구현

490ddba

refactor: 강화기능 함수 분할

db1f685

refactor: 강화기능 함수 클래스 이동

0d35529

rootTiket self-assigned this May 29, 2024

rootTiket requested review from jj0526, jiixon, 1winhyun and huncozyboy May 29, 2024 01:04

rootTiket changed the title ~~[BE] 이근표 강화게임~~ [BE] 이근표 강화하기 May 29, 2024

hoonyworld approved these changes May 30, 2024

View reviewed changes

KoungQ reviewed May 30, 2024

View reviewed changes

refactor: 불필요한 어노테이션 삭제

8026001

leejuae reviewed May 30, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[BE] 이근표 강화하기 #2

[BE] 이근표 강화하기 #2

rootTiket commented May 29, 2024 •

edited

Loading

hoonyworld left a comment

hoonyworld May 30, 2024

rootTiket May 30, 2024

hoonyworld May 30, 2024

rootTiket May 30, 2024

hoonyworld May 30, 2024

rootTiket May 30, 2024

KoungQ left a comment

KoungQ May 30, 2024

rootTiket May 30, 2024

KoungQ May 30, 2024

rootTiket May 30, 2024

KoungQ May 30, 2024

rootTiket May 30, 2024

KoungQ May 30, 2024

rootTiket May 30, 2024

leejuae May 30, 2024

leejuae May 30, 2024

leejuae left a comment

[BE] 이근표 강화하기 #2

Are you sure you want to change the base?

[BE] 이근표 강화하기 #2

Conversation

rootTiket commented May 29, 2024 • edited Loading

❓ 어떤 기능을 구현하였나요?

✅ 회원가입 기능 구현

✅ 로그인 기능 구현

✅ 강화기능 구현

✅ 아이템 생성 구현

✅ 아이템 조회 구현

✅ 아이템Top10 조회 구현

🤔 구현 중 무엇을 고민했을까요?

💭 아쉬운 점은?

hoonyworld left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

KoungQ left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

leejuae left a comment

Choose a reason for hiding this comment

rootTiket commented May 29, 2024 •

edited

Loading