fix: timetableLecture 오류 수정 #1096

Soundbar91 · 2024-11-27T12:01:24Z

🔥 연관 이슈

close timetableLecture 오류 수정 #1095

🚀 작업 내용

기존 데이터에 , , ,와 같은 데이터가 있으면 인덱스가 터지는 버그를 고쳤습니다.
- ,구분자로 데이터를 자르고 공백을 지운 상태로 시간과 매핑을 하는데, 인덱스가 강의 장소 배열보다 넘치게 되는 경우가 발생해서 처리를 했습니다.
리펙토링 과정에서 frame 권한 여부 확인을 frame id와 user id와 매칭하고 있어서 수정했습니다.

💬 리뷰 중점사항

github-actions · 2024-11-27T12:06:30Z

Unit Test Results

339 tests 338 ✔️ 1m 29s ⏱️
  41 suites     1 💤
  41 files     0 ❌

Results for commit 92230fb.

♻️ This comment has been updated with latest results.

seongjae6751

고생하셨습니다!

duehee

오늘 고생이 많으시네용.. :p

kwoo28

수고 많으셨습니다! 어려운 사항을 다루느라 항상 고생이 많으신것같습니다..

kwoo28 · 2024-11-27T13:50:33Z

src/main/java/in/koreatech/koin/domain/timetableV2/dto/response/TimetableLectureResponse.java

@@ -83,7 +82,25 @@ public record ClassInfo(
            public static List<ClassInfo> of(String classTime, String classPlace) {
                // 정규 강의인 경우 강의 장소가 없기 때문에 바로 반환
                if (classPlace == null) {
-                    return List.of(new ClassInfo(parseClassTimes(classTime), null));
+                    String[] split = classTime.substring(1, classTime.length() - 1).trim().split(",\\s*");


R

classTime이 null값이면 NPE발생할것같아요!

정규 강의인 경우 getClassTime 메소드를 사용해서 null 체크를 하고 있습니다 !

커스텀인 경우에는 발생하지않나요?? 이번에 stage에서 생긴 Cannot invoke "java.util.Collection.toArray()" because "c" is null 오류가 이 부분같다고 생각이 드는데.. 지금 로컬에서 테스트해봤는데, 커스텀 생성에 class_time값을 null을 주면 똑같은 오류 발생하고있습니다!

커스텀인 경우 class_time이 null이 되면 안되는 것을 dto에서 어노테이션을 통해 검증을 해야하는데, 정규 강의의 경우 class_time를 null로 추기 때문에 발생하는 문제라고 생각합니다.

음.. 그러면 pr에 수정된 코드는 해결된건가요?? 일단 승인하겠습니다!

kwoo28 · 2024-11-27T14:01:08Z

src/main/java/in/koreatech/koin/domain/timetableV2/dto/response/TimetableLectureResponse.java

@@ -83,7 +82,25 @@ public record ClassInfo(
            public static List<ClassInfo> of(String classTime, String classPlace) {
                // 정규 강의인 경우 강의 장소가 없기 때문에 바로 반환


R

이번 PR내용은 아니긴한데.. 정규강의도 강의 장소 추가가 가능합니다!
관련 슬랙 스레드 링크 : https://bcsdlab.slack.com/archives/C06NEM6EY3W/p1728732635463599

이 부분은 주석 설명이 잘못된 것 같습니다.
장소가 없는 경우에 시간과 매칭을 할 수 없기 때문에 관련해서 작성한 부분입니다.

kwoo28 · 2024-11-27T14:05:11Z

src/main/java/in/koreatech/koin/domain/timetableV2/dto/response/TimetableLectureResponse.java

@@ -83,7 +82,25 @@ public record ClassInfo(
            public static List<ClassInfo> of(String classTime, String classPlace) {
                // 정규 강의인 경우 강의 장소가 없기 때문에 바로 반환
                if (classPlace == null) {


C

혹시 classPlace를 굳이 null로 조건문을 달아서 따로 로직 처리하는 이유가 있나요?

강의 시간은 무조건 현재 존재하지만, 장소는 존재하지 않는 경우가 존재합니다.
초기 정규 강의는 classPlace가 null이기 때문에 체크를 하지 않으면 에러가 반환이 되서 조건문을 달았습니다.

kwoo28 · 2024-11-27T14:22:03Z

src/main/java/in/koreatech/koin/domain/timetableV2/dto/response/TimetableLectureResponse.java

@@ -83,7 +82,25 @@ public record ClassInfo(
            public static List<ClassInfo> of(String classTime, String classPlace) {


A

전체적인 of()메서드의 로직부분에 대한 제안해봅니다.

classTime 반복문 내에서 classPlace관련 로직을 다루는것보다, 각각 시간과 장소에 대한 로직을 분리해서 for문으로 돌리는게 좋지않을까요??

제가 생각한방식은 처음에 class_time관련된 부분은 -1구분자로 나눠서 2차원 배열로 만든 후에, class_time과 class_place중에 max_size를 알아낸 후에, 그 max_size크기 만큼 각각 class_time과 class_place에서 for문 돌리면서 쌍을 맞추는 방식은 어떨까요? (이 부분 무시하셔도 될것같습니다..)

일단 지금은 에러 수정사항이니 이 부분은 리팩토링으로 두면 어떨까 생각합니다!

넵 리펙토링으로 남기겠습니다.

duehee

수정하신 부분, 특히 구분자를 통해 강의를 구분하는 로직이 조금 더 확실해진 거 같네요.
고생하셨습니당~

duehee · 2024-11-28T03:58:35Z

src/main/java/in/koreatech/koin/domain/timetableV2/controller/TimetableApiV2.java

+            lecture_id가 있는 경우 class_infos, professor은 null, grades는 '0'으로 입력해야합니다.\n
+            lecture_id가 없는 경우 class_infos, professor, grades을 선택적으로 입력합니다.


A

설명이 자세해져서 좋은 거 같아요!

duehee · 2024-11-28T03:59:54Z

src/main/java/in/koreatech/koin/domain/timetableV2/service/TimetableLectureService.java

@@ -70,7 +72,7 @@ private TimetableLectureResponse getTimetableLectureResponse(Integer userId, Tim
    public void deleteTimetableLectures(List<Integer> request, Integer userId) {
        request.stream()
            .map(timetableLectureRepositoryV2::getById)
-            .peek(lecture -> validateUserAuthorization(lecture.getTimetableFrame().getId(), userId))
+            .peek(lecture -> validateUserAuthorization(lecture.getTimetableFrame().getUser().getId(), userId))


A

이 부분을 제대로 안 읽은 리뷰어의 잘못..👀

Soundbar91 added 2 commits November 27, 2024 20:44

fix: 인덱스 에러 수정

32f8439

fix: 인덱스 에러 수정

16e835e

Soundbar91 added 버그 정상적으로 동작하지 않는 문제상황입니다. Team User 유저 팀에서 작업할 이슈입니다 labels Nov 27, 2024

Soundbar91 requested review from seongjae6751, daheeParkk, kwoo28 and duehee November 27, 2024 12:01

Soundbar91 self-assigned this Nov 27, 2024

seongjae6751 approved these changes Nov 27, 2024

View reviewed changes

duehee approved these changes Nov 27, 2024

View reviewed changes

Soundbar91 added 4 commits November 27, 2024 21:55

fix: 정규 강의 분할 반환

af1c2d6

fix: 롤백

c4181da

test: 테스트 수정

2bb9bc0

fix: 로직 수정

a794707

kwoo28 requested changes Nov 27, 2024

View reviewed changes

fix: 롤백

fc2a5a9

kwoo28 approved these changes Nov 27, 2024

View reviewed changes

Soundbar91 changed the title ~~fix: timetableLecture 인덱스 에러 수정~~ fix: timetableLecture 오류 수정 Nov 27, 2024

Soundbar91 added 5 commits November 28, 2024 01:23

fix: deleteTimetableLectures 검증 오류 수정

be89d9f

fix: 강의 장소가 없는 강의 정보가 두개 이상 들어왔을 때 오류 해결

b639197

fix: 빈 문자열 에러 수정

429c192

chore: 스웨거 설명 수정

0b5eb09

chore: todo 작성

d41fcd9

duehee approved these changes Nov 28, 2024

View reviewed changes

Soundbar91 added 2 commits November 28, 2024 15:49

fix: class_place "null" 저장 수정

3de2275

chore: 미사용 import 삭제

92230fb

Soundbar91 merged commit a3c3ae4 into develop Nov 28, 2024
4 checks passed

Soundbar91 deleted the fix/1095-timetable-lecture branch November 28, 2024 07:18

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: timetableLecture 오류 수정 #1096

fix: timetableLecture 오류 수정 #1096

Soundbar91 commented Nov 27, 2024 •

edited

Loading

github-actions bot commented Nov 27, 2024 •

edited

Loading

seongjae6751 left a comment

duehee left a comment

kwoo28 left a comment

kwoo28 Nov 27, 2024

Soundbar91 Nov 27, 2024

kwoo28 Nov 27, 2024 •

edited

Loading

Soundbar91 Nov 27, 2024

kwoo28 Nov 27, 2024

kwoo28 Nov 27, 2024

Soundbar91 Nov 27, 2024

kwoo28 Nov 27, 2024

Soundbar91 Nov 27, 2024

kwoo28 Nov 27, 2024 •

edited

Loading

Soundbar91 Nov 27, 2024

duehee left a comment

duehee Nov 28, 2024

duehee Nov 28, 2024

		@@ -83,7 +82,25 @@ public record ClassInfo(
		public static List<ClassInfo> of(String classTime, String classPlace) {
		// 정규 강의인 경우 강의 장소가 없기 때문에 바로 반환

		lecture_id가 있는 경우 class_infos, professor은 null, grades는 '0'으로 입력해야합니다.\n
		lecture_id가 없는 경우 class_infos, professor, grades을 선택적으로 입력합니다.

fix: timetableLecture 오류 수정 #1096

fix: timetableLecture 오류 수정 #1096

Conversation

Soundbar91 commented Nov 27, 2024 • edited Loading

🔥 연관 이슈

🚀 작업 내용

💬 리뷰 중점사항

github-actions bot commented Nov 27, 2024 • edited Loading

Unit Test Results

seongjae6751 left a comment

Choose a reason for hiding this comment

duehee left a comment

Choose a reason for hiding this comment

kwoo28 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

R

Choose a reason for hiding this comment

kwoo28 Nov 27, 2024 • edited Loading

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

R

Choose a reason for hiding this comment

Choose a reason for hiding this comment

C

Choose a reason for hiding this comment

kwoo28 Nov 27, 2024 • edited Loading

Choose a reason for hiding this comment

A

Choose a reason for hiding this comment

duehee left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

A

Choose a reason for hiding this comment

A

Soundbar91 commented Nov 27, 2024 •

edited

Loading

github-actions bot commented Nov 27, 2024 •

edited

Loading

kwoo28 Nov 27, 2024 •

edited

Loading

kwoo28 Nov 27, 2024 •

edited

Loading