내가 업로드한 노래 전체 보기 #225
내가 업로드한 노래 전체 보기 #225
Conversation
HamBP
requested review from
khw3754,
sk000801,
youlalala and
2taezeat
as code owners
| android:layout_height="match_parent"> | ||
|
|
||
| <com.google.android.material.appbar.MaterialToolbar | ||
| android:id="@+id/mt_my_musics" |
There was a problem hiding this comment.
고쳐서 푸시했어!
그런데 해당 부분의 컨벤션이 정리된 문서를 찾지는 못 했어 ㅠㅠ
id 변경은 IDE를 통해 자동 리팩터링이 안 되기도 하고, id 컨벤션에 예외가 너무 많아서 컨벤션을 맞추는 비용이 컨벤션을 맞춰서 얻는 이득보다 커지지 않을까?
There was a problem hiding this comment.
요거는 우리가 페어프로그래밍할때 이야기했었어! 다른 fragment에 toolbar 사용된 id는 다 tb 로 시작하던데..!
There was a problem hiding this comment.
정리가 안되어있는듯하네 내가 추가해놓을게! 여러화면에서 자주 사용되는 컴포넌트들은 맞추는게 좋을 것 같아
| val errorType = | ||
| if (throwable is CtException) throwable.ctError | ||
| else CtErrorType.UN_KNOWN | ||
|
|
||
| _events.emit(MyMusicsEvent.ShowMessage(errorType)) |
server/src/music/music.controller.ts
Outdated
| @Get('ts') | ||
| @HttpCode(HTTP_STATUS_CODE.SUCCESS) | ||
| async getEncodedChunkFiles( | ||
| @Query('music_id') music_id: string, | ||
| @Query('fileName') fileName: string, | ||
| ): Promise<{ file: AWS.S3.Body }> { | ||
| return { | ||
| file: await this.musicService.getEncodedChunkFiles(music_id, fileName), | ||
| }; | ||
| } | ||
| } |
| private val _events = MutableSharedFlow<MyMusicsEvent>() | ||
| val events: SharedFlow<MyMusicsEvent> = _events.asSharedFlow() | ||
|
|
||
| private val exceptionHandler = CoroutineExceptionHandler { _, throwable -> |
There was a problem hiding this comment.
private val exceptionHandler = CoroutineExceptionHandler { _, throwable ->
val errorType =
if (throwable is CtException) throwable.ctError
else CtErrorType.UN_KNOWN
viewModelScope.launch { _events.emit(PlaylistsEvent.ShowMessage(errorType)) }
}emit 할때만, viewModelScope 이용하면 어떨까요?
There was a problem hiding this comment.
val errorType =
if (throwable is CtException) throwable.ctError
else CtErrorType.UN_KNOWNerrorType이 viewModelScope.launch 안에 있는데, 이걸 viewModelScope.launch 바깥으로 빼자는 의미입니다~
There was a problem hiding this comment.
코루틴 안에서 할 필요가 없는 일을, 굳이 코루틴 안에서 하는게 이상하다는 것 이였습니다~
server/src/httpStatusCode.enum.ts
Outdated
| @@ -3,6 +3,7 @@ export enum HTTP_STATUS_CODE { | |||
| 'NOT_DUPLICATED_NICKNAME' = 200, | |||
| 'DUPLICATED_NICKNAME' = 409, | |||
| 'WRONG_TOKEN' = 401, | |||
| 'NOT_FOUND' = 404, | |||
There was a problem hiding this comment.
'NOT_FOUND' = 404'
요거 어느 api 호출하고, 예외 상황일때 발생하나용? (노션 에서 찾을 수 없어서)
| @@ -8,10 +8,18 @@ | |||
| <fragment | |||
| android:id="@+id/my_page_fragment" | |||
| android:name="com.ohdodok.catchytape.feature.mypage.MyPageFragment" | |||
| android:label="mypage" | |||
| android:label="my page" | |||
There was a problem hiding this comment.
마이너 한데,
my_page 는 어떠신가요? (my_musics 도 마찬가지 입니다~)
There was a problem hiding this comment.
label은 사용하는 용도가 아니라 Preview에서 읽는 용도인데, my_page보다 my page가 가독성 좋지 않을까?
There was a problem hiding this comment.
가독성은 비슷할것 같네요, snack_case를 예외 없이 사용하자~ 라는 의미였습니다~
| init { | ||
| fetchMyMusics() | ||
| } |
There was a problem hiding this comment.
왜 init { } 에서 호출하는지, 알 수 있나요?
There was a problem hiding this comment.
MyPage는 onStart 시점에 호출하도록 변경했어
여기는 이 위치가 맞는 거 같은데 코멘트를 단 이유가 궁금해
There was a problem hiding this comment.
제가 MyMusicsViewModel.kt 에서 호출 되는지 착각 했네요,
MyPageViewModel.kt 에서 init { fetchMyMusics() } 하는 줄 착각해서 코멘트 달았습니다~
이 위치 맞습니다~
Issue
Overview
Screenshot (optional)