안녕하세요. 개발하는 정주입니다.
오늘은 Sign Up 코드 리뷰를 받은 것에 대해 포스팅하려고 합니다.
재제출 결과 Pass를 받을 수 있었습니다. 이번 리뷰어는 두 가지 코멘트를 달아주셨는데요.
이에 대해 자세히 알아보겠습니다.
Fail 이유에 대해서는 4편을 참고해 주세요!
목차
종합 의견
리뷰어가 남긴 종합 의견은 "깔끔하다."입니다. 이 멘트를 보고 참 기분이 좋았습니다.
Clean Code에 신경을 쓰자는 다짐을 하고 iOS에 임한 만큼
코드가 깔끔하다는 평가는 잘 동작한다보다 훨씬 크게 다가왔습니다.
보람이 있네요.
칭찬과 아쉬움
칭찬을 받은 점은 화면 밖을 터치할 때 키보드가 내려가게 처리한 점이었습니다.
이에 대해서는 자세히 포스팅을 했기에 지난 포스팅으로 대신하겠습니다.
하지만 아쉽다는 평을 받은 코드도 있었습니다.
if id.trimmingCharacters(in: .whitespacesAndNewlines) == "" ||
password.trimmingCharacters(in: .whitespacesAndNewlines) == "" ||
introduction.trimmingCharacters(in: .whitespacesAndNewlines) == "" {
return false
}공백을 체크하기 위해 넣은 코드입니다.
Text Field 값을 trim 하여 공백 여부를 확인했는데요.
Swift의 함수형 프로그래밍 언어의 특성을 살려 isEmpty()를 사용했다면 더 깔끔했을 거라는 리뷰를 해주셨습니다.
추가로 조건을 ||로 묶어 체크하지 말고 guard를 이용했다면 더욱 Swift 스러운 코드가 되었을 거라고 하네요.
해당 부분들을 수정하여 마지막 커밋을 해야겠습니다.
마무리 잡담
두 번째 프로젝트는 1 Fail 후 Pass를 받았는데요.
세심하게 체크하지 못한 실수라 더욱더 아쉽습니다.
세 번째 프로젝트부터는 꼼꼼하게 체크해야겠습니다.
감사합니다!
아직은 초보 개발자입니다.
더 효율적인 코드 훈수 환영합니다!
공감과 댓글 부탁드립니다.
