refactor,fix: 2회차 멘토링 기반 리팩토링

[chcch529]

✨ 설명

#67

1. (작업 파일)

• src/main/java/com/oronaminc/join/global/exception/ErrorException.java
• src/main/java/com/oronaminc/join/question/api/QuestionController.java
• src/main/java/com/oronaminc/join/room/util/CodeGenerator.java
• src/main/java/com/oronaminc/join/websocket/common/EventType.java

2. (작업 내용 요약)

2회차 멘토링 기반 수정 사항

1. 예외 발생 시 예외 메시지 구체화

    public static ErrorException of(ErrorCode errorCode, String message, Object... args) {
        String errorMessage = createMessage(message, args);
        return new ErrorException(errorCode, errorMessage);
    }

기존 ErrorException에 메소드를 추가하였습니다. 필요 시 해당 메소드 사용하여서 어떤 값으로 인해 예외가 발생했는지 확인할 수 있도록 하시면 될 것 같습니다.

2. EventType enum 추가

event를 string으로 처리하던 것을 enum을 사용하도록 리팩토링 했습니다.

3. jpql 스타일 통일

소문자 사용으로 통일시켜 놨습니다. (다수에 맞춤)

4. commons-lang

    public static String generateCode(int length) {

        return RandomStringUtils.random(length, CHAR_POOL);
    }

유틸리티 클래스 사용하도록 리팩토링 했습니다.

💬 리뷰

• 제가 이 부분 작업 후 pr 작성까지만 해놓고 올리진 않았다는 것을 뒤늦게 알았습니다 ㅎㅎ..
• 추가로 QuestionController에서 roomId를 pathvariable이 아닌 requestparam으로 받고 있던 거 함께 수정했습니다!

[rjswjddn]

코드 생성 부분 바꿔주셔서 감사합니다. 고생하셨습니다!

[min0962]

고생하셨습니다!!

[chw0912]

예외 발생시 메시지와 매개변수를 받아 예외처리가 더 명확해져 좋은 방법인 것 같습니다!
고생하셨습니다!