Fix [#162] 로딩뷰 수정 #165
Fix [#162] 로딩뷰 수정 #165
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| } | ||
| } | ||
| } | ||
|
|
||
| private func setHierarchy() { | ||
| view.addSubviews(homeCollectionView) | ||
| } |
There was a problem hiding this comment.
코드 리뷰 요약:
hasAppearedBefore변수가 사용되지 않고 있으므로, 관련 코드를 제거해도 될 것으로 보입니다.showLoadingView()메서드 역시 사용되지 않고 있으므로, 삭제하는 것이 좋을 것 같습니다.- 주석이 없어서 코드의 의도를 명확히 이해하기 어려울 수 있습니다. 각 기능에 대한 주석 추가는 도움이 될 수 있습니다.
- 함수와 변수 이름에서 더 명확한 의미 전달을 위해 네이밍 규칙을 통일하고, 의미있는 이름을 선택할 수 있습니다.
- 코딩 스타일 일관성을 유지하는 것이 중요합니다. 들여쓰기 및 스타일을 일관성 있게 유지해야 합니다.
개선 사항:
- 사용되지 않는 변수 및 메서드 제거
- 적절한 주석 추가
- 네이밍 규칙 준수
- 코딩 스타일 일관성 유지
위의 내용을 참고하여 코드를 개선하시기 바랍니다.
| } | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
주요 코드 리뷰:
-
showLoadingView메서드에서 로딩 뷰를 보여주고 숨기는 로직이 있습니다. 그러나 애니메이션 블록 내부에 아무 작업도 수행하지 않는 부분이 있습니다. 이렇게 하는 대신 완료 핸들러 클로저에 필요한 작업을 추가하는 것이 좋습니다. -
beforeIndex == 2및beforeIndex == 3일 때selectedIndex == 0인 경우showLoadingView가 호출됩니다. 이런 동작이 의도된 것인지, 혹은 조건의 변경이 필요한지 확인해야 합니다. -
applyFontColorAttributes메서드는 선택된 뷰 컨트롤러의 탭 바 아이템에 폰트 색상 속성을 적용합니다. 이 방법이 올바르게 동작하는지 확인하고 없는 경우에 필요한 스타일을 적용해야 합니다. -
코드 상
tabBarController에 대한 액세스 권한을 확인하고 필요 시 안전한 옵셔널 연산자 사용을 검토해야 합니다. -
특정 인덱스 값이 하드코딩되어 있으므로 추후에 변경 시 코드 수정이 필요할 수 있습니다. 해당 값을 상수로 정의하여 관리하는 것이 좋습니다.
-
코드 중복을 줄이기 위해 공통으로 사용되는 부분을 메서드로 분리하고 재사용성을 높일 수 있습니다.
-
코드 주석을 추가하여 각 섹션이 어떤 동작을 수행하는지 명확하게 설명해야 합니다.
| } | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
이 코드 패치의 코드 리뷰를 요청하셨습니다.
showLoadingView메서드는 동일한 작업을 수행하는 중복된 블록을 가지고 있습니다. 이러한 중복성을 제거하여 코드 사용성을 향상시킬 수 있습니다.beforeIndex == 2및beforeIndex == 3인 경우에만selectedIndex == 0여부를 확인하는 조건으로 나누어집니다. 이 조건들은 기능적으로 유사하므로, 하나의if beforeIndex >= 2 && selectedIndex == 0 {}로 합칠 수 있습니다.- 몇몇 중괄호(
{})의 들여쓰기가 일제히 맞지 않습니다. - 에러 핸들링이 없어서 로딩 뷰가 화면에서 강제로 제거될 수 있습니다. 에러 처리를 추가해야 할 필요가 있습니다.
👻 PULL REQUEST
💻 작업한 내용
글쓰기 탭에서 로딩뷰가 뜨는 현상 해결했습니다.
💡 참고사항
📸 스크린샷
📟 관련 이슈