마이페이지 이름 수정 불가 및 ui 수정 #34
Conversation
| modifier = Modifier.padding(bottom = 12.dp).weight(1f).height(40.dp) | ||
| .clip(RoundedCornerShape(12.dp)).background(WalkieColor.GrayDefault), |
There was a problem hiding this comment.
체이닝 했을 때 하나하나 개행을 해주지 않았던 것 같아요
There was a problem hiding this comment.
ktlint가 그래요???
체이닝했을 때 개행하는게 가독성이 훨씬 좋지 않나요?
There was a problem hiding this comment.
넵 줄 길이를 넘지 않으면 따로 개행해주지는 않아서 직접 개행해야 하네요
| ) | ||
| } | ||
| } | ||
| } | ||
| } | ||
| } | ||
|
|
||
| enum class TEXTVISIBLESTATE { |
There was a problem hiding this comment.
enum class 클래스명 일반 클래스명과 컨벤션이 동일하지 않나요??
| val name = viewModel.name.collectAsStateWithLifecycle(initialValue = "") | ||
| val nick = viewModel.nick.collectAsStateWithLifecycle(initialValue = "") |
There was a problem hiding this comment.
요거 String 확장 프로퍼티로 만들어놔도 좋을 것 같아요!
지금 도메인에 혹시 없나요?
There was a problem hiding this comment.
// before
val name = viewModel.name.collectAsStateWithLifecycle(initialValue = "")
// after
val name = viewModel.name.확장함수()위 형태가 될 텐데 collectAsStateWithLifecycle 함수 이름이 좀 길긴해도
함수 하나를 굳이 다른 함수로 만들 필요는 없지 않을까 생각했어요
There was a problem hiding this comment.
아하 좋습니다! 도메인에 만들어서 사용도록 변경했습니다
| val name = viewModel.name.collectAsStateWithLifecycle(initialValue = "") | ||
| val nick = viewModel.nick.collectAsStateWithLifecycle(initialValue = "") | ||
|
|
||
| EditProfileContent(name = name.value ?: "null", nick = nick.value ?: "null") { |
There was a problem hiding this comment.
공백확장 프로퍼티 사용이 좋겠네요
| @@ -156,9 +162,12 @@ fun EditProfileContent(name: String, nick: String, onCloseClicked: () -> Unit) { | |||
|
|
|||
| Spacer(modifier = Modifier.height(16.dp)) | |||
|
|
|||
| // TODO 임시로 만든 변수, viewModel, uiState로 관리하도록 변경 필요(139, 171 line) | |||
| var nickCheckState: Boolean? by rememberSaveable { mutableStateOf(null) } | |||
There was a problem hiding this comment.
기본 상태가 3가지 상태를 가져야하는데 enum 클래스로 정의하지 않고 그냥 널러블 boolean 을 사용했는데 좋지 않아보이네요
기본(체크 이전) -> null
체크 성공 -> true
체크 실패 -> false
이렇게 3가지 상태를 표현합니다.
There was a problem hiding this comment.
그 예전에 말씀하셨던 nullable Boolean 이네요 ㅋㅋㅋㅋㅋㅋㅋ
간단한 enum class 하나 만들까요?
There was a problem hiding this comment.
맞아요 세상에서 제일 쓸모없는게 nullable Boolean 같았지만 야매코드 쓰다보니깐 생각없이 쓰게 되네요..ㅋㅋㅋ
There was a problem hiding this comment.
ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ 하지만 그 맘 이해갑니다
| @@ -136,7 +136,7 @@ fun SignInUserNameScreen(onSuccess: () -> Unit) { | |||
| viewModel.setUserNameState( | |||
| userNameState.copy( | |||
| year = year, | |||
| month = month, | |||
| month = month + 1, | |||
There was a problem hiding this comment.
월일 표현 방식이 1부터 시작하는 라이브러리들도 있고 0부터 시작하는 라이브러리들도 있는데 타임피커에서 월을 0부터 시작하네요
|
@soopeach 오랜만에 코드 종합적으로 건설적인 리뷰를 받으니 너무 좋았습니다 감사합니다~ |
아좌좌 화이팅!! 용한님 덕분에 저도 힘이나요 |
😎 작업 내용
🧐 변경된 내용
🥳 동작 화면
🤯 이슈 번호
🥲 비고