[ #14 ] TIL

✏️ 0502      

---

[ 발제 ] 팀과제

3시 심화반 염탐

개인과제 피드백 반영

Java 문법 종합반 강의_5주차

---

 

심화반

 

나는 보충반이지만 심화반 수업이 궁금해서 염탐하러 다녀왔다

네트워크 기초 내용을 진행하고 계셨는데

예전에 네트워크 보안 관련 수업을 들어본 적이 있던 것과 유사한 내용이었다

그때 복습하는 느낌이 들어서 재밌었다

 

 

개인과제 피드백

 

해설 영상을 다 보고 피드백이 왔나 싶어서 확인했는데 와있더라

근데 튜터님의 등록날짜? 로 추정되는 것이 내가 제출한 시간보다 과거의 시간이라

과거의 튜터님이 써주신건가(?) 생각했다 😝

 

 

커밋 기록에 자세한 내용을 적어주면 협업에 좋다고 말씀해주셔서 바로 반영했다!

 

 

확실히 팀원들 코드를 구경하면서 느낀건데

내가 쓴 코드는 내가 알기 때문에 무엇을 했는지 단번에 보이지만

남이 쓴 코드를 볼 때는 무얼 했는지 적어두지 않으면 어디를 수정하고 추가했는지 쉽게 보기는 어려웠다

 

이 부분에서 확실히 튜터님 말씀처럼 잘 적어야 다른 사람들이 코드 리뷰를 더 잘해줄 수 있지 않을까!

이것보다 더.. 자세히 적어야하나?

 

// 사칙연산 계산하는 메서드
    public int calculate(char operator, int firstNumber, int secondNumber) throws Exception {
        // 오류 검사 (사칙연산, 두번째 숫자 0)
		....
        int answer = 0;
        switch(operator) {//연산자별 계산 코드}
        numberList.add(answer);
        return answer;
    }

 

두번째 피드백은 calculate 메서드는 말그대로 계산만 하는 메서드이니 단일책임원칙을 위해서

연산결과를 리스트에 추가하는 코드를 분리하라고 말씀해주셨다

 

사실 이 코드 해설영상 보면서 코드 수정하다가 어느 순간 삭제되어서 사라진지도 몰랐다

아마 코드를 분리하고 삭제하면서 잘못 지워진 것 같다

피드백 받은 후에야 확인했더니 add 메서드가 사라진 걸 확인했다 ㅋㅋㅋ

 

오히려 오류가 안나서 더 무서웠던 것 같다..😅

 

클래스로 만들까 메서드로 만들까 고민하다가

자식클래스의 계산기과 원넓이 둘 다 리스트에 추가해주니

부모 클래스에 추가해서 상속받게 만들었다

 

// 저장 메서드
public void addResult(double result) {
    numberList.add(result);
}

 

간단하게 바로 수정!

했는데 의도하신 피드백이 맞는지는 자신감이 없긴 하다.. ㅎㅎ

 

 

4번째 피드백은

부모 클래스에 존재하던 PI 상수값은 계산기에는 필요없으니 자식 클래스로 옮기라는 말씀인 것 같아서 바로 수정했다

옮기는 게 코드 정리에 좋을 것 같다

 

이러한 부분에서 세심하고 꼼꼼하게 코드를 봐야하는 구나 느꼈다

 

 

5번째 피드백

2단계까지 밖에 못해서 3단계는 바로 해설 영상으로 봤지만 5주차 강의를 듣지 못한 상태에서 봤기 때문인지

반이상은 이해하지 못한 것 같다 약간 원리는 이해했지만 그냥 머리속에 박아넣은 느낌이랄까

 

주입식 교육의 폐해 아닐까...

 

그래서 급하게 5주차 강의를 듣게 되었다

고생해서 들었던 3주차 강의와는 또 다른 느낌의 머리아픔이었다

 

코드를 고쳐보긴 했는데 아직 해결은 못하고 있다

성공하게 되면 해당 내용을 올려보겠다!

 

 

회고

 

자바는...악명높은 만큼 어렵구나 싶었다

어려운데 재밌기도 하고 그치만 어려운데

놓치고 싶지는 않고 다시 생각나는 그런 갈대같은 마음이 들게 한다