[✨TIL]JAVA 미니 과제1_REVIEW

오늘은 백엔드 스쿨을 하면서 첫! 미니과제에 대해 코드리뷰 받은 것을 기록하고자 한다.

왜냐면...생각보다 굉장히 실수도 많았고 그 만큼 점수도 낮기 때문에..ㅋㅋㅋㅋㅋ
내가 아무리 알고리즘 공부도 했어도 나한테 아직은 낯선(자바)걸로 개발해서 바보같은 짓을 많이 했다.
다신 그런 실수 안하기 위해 TIL카테고리에 따로 작성할 예정이다.

1. "문자열".equals(변수)의 형태로 하자

나는 계속 문자열을 비교할 때 변수.equals("문자열") 형태로 했다. 이러면 null일 경우에는 어쩔 것인가?
즉, 상수문자열을 기준으로 비교함수를 호출하는 것이 null의 경우에도 오류없이 잘 돌아가기 때문이다.
현업에서는 이런 실수는 치명적일 것 같아 꼭 기억하자

2. 불필요한 for문은 따로 빼자

클린 코드를 지향하면서 가장 기초가 되면서 클린 코드의 첫걸음을 내가 망각하고 있었던 것 같다. 한꺼번에 (for문없이) 값을 받을 수 있는지를 먼저 생각하고,
그게 안된다면 + 계속 for문이 쓰일 것 같다면 메소드로 따로 빼야한다.

총 정리하자면!

그 외에 내가 점수를 깎아 먹은 부분이 주어진 문제를 정확히 이해하고 분석하는 부분이 많이 부족하다는 피드백을 받았다.
먼저 손을 올리고 개발하기 보다는 충분히 손코딩과 문제 의도를 분석하는 시간을 가지자..(코테에서도 이건 불변의 진리.! 인듯)