해결하지 못한 것

[WraithKim]

• gui 최소 높이가 제한이 안되는 문제
• Hamcrest를 활용하지 못함
• 테스트 내용 부족
• 알고리즘 관련 클래스에서 템플릿 메서드 패턴전략 패턴을 활용할 수도 있었는데 그러지 못함
• 클래스를 좀 더 잘게 쪼개야 했음.
• 생성자에 의존성이 생기는 문제를 해결 못함
• 처음 설계 단계가 미흡해서(설계 패턴에 대한 이해 부족, GUI 라이브러리 이해 부족) 개발 단계에서 낭비가 발생함
• 커밋을 짧게 끊어야 하는데 그러지 않음.
• 문자 인코딩 인식 라이브러리가 좋은 라이브러리가 아님. 다른 걸로 교체해야 함.
• 그래들 설정에서 개발 환경에 따라 프로젝트 설정을 해줄 수 있는 코드가 없음.
• UI 디자인이 구림
• css 코드가 리스트셀에서 동적으로 적용되는데 이게 좋은 구현 방식인지 모르겠음
• 클래스, 메서드, 멤버 변수명 등, 이름이 직관적이지 못함.
• 안 쓰는 변수나 import 문 지워야 함.
• 복붙한 코드를 메서드로 바꿔야 함.
• 인터페이스로 public 메서드를 죄다 묶어뒀더니, 인터페이스를 구현한 클래스가 OCP를 따르지 않는 것 같음
• 그 외 더 나은 설계로 극복할 수 있는 점이 있으면 개선해야 함.

[WraithKim]

• jre 타겟 버전이 너무 높은지 확인해봐야함

[WraithKim]

• fileReader가 인코딩 감지와 파일 읽기를 둘 다 하고 있음

[WraithKim]

• FXML이라는 어노테이션이 붙은 필드들은 왠만하면 건드리지 말 것. GUI 구조를 정의해주는 다른 파일이랑 1:1 대응임.
• EditorPaneFXController에서 ViewMode를 State Pattern처럼 적용했는데 inner class라서 코드 스멜이 좋지 않기 때문에 개선 가능하면 개선해야함.
• interface 좀 정리해야함. 기준은 모르겠다. 쓸데 없는 것도 있을 수도 있고 분리를 더 해야하는데 안한 것도 있을 수도 있음. 그냥 인터페이스 이름을 ~~Interface라고 박아놔서 뭘 위한 건지도 파악하기 어려운것도 있음.
• MainWindowFXController에 55,56번 줄과 62,63번 줄이 겹침
• @OverRide나 @SupressWarning()같은 어노테이션이 빠져있는 경우 넣어놔야함.
• Controller가 fxml을 불러올때 객체가 생성되기 때문에 컨트롤러에 대응되는 모델 객체를 직접 set하도록 되어있음. 이러면 컨트롤러의 멤버 필드인 모델 객체가 null일 경우도 존재함. 이 부분 해결해야함. null 처리를 하든지 null이 안 생기게 하든지.
• EditorPane에서 편집 모드가 비교 모드로 바뀌면 MainWindow에도 행동을 요구하게 되어잇음. (EditorPaneFXController:136) 이거 옵저버든 뭐든 사용해서 편집창이 메인윈도우를 직접 명령 내리는 하극상 좀 막아야함.
• 내부 구현이 긴 함수좀 잘라야 함. ex) controller 클래스의 initialize()
• view 팩토리에 추상 팩토리 패턴 비스무리하게 생긴 녀석들이 있음. 이거 객체화해서 컨트롤러가 멤버필드로 갖게 할건지 아니면 그냥 static으로 할건지 결정해야함.
• 복잡한 생성자에 factory method pattern 적용 가능
  모델 부분은 난 모른다. 읽고 생각나면 말함.

[tjdgus3537]

• Diff class에 makePairBlocks 함수 내용이 너무 긺. 좀 잘라야할 것 같고, ArrayList에 너무 의존적인 듯(private이라서 내부에서만 사용하지만 일부는 List만 사용해도 충분할 듯 싶음). 이 클래스 내부에 다른 함수들도 대체로 긴 것 같음, 더 잘게 나누면 좋을 듯(로직이 오래 전에 짠거라 당장은 정확히 기억이 안 남).
• 굳이 String만 사용할 것이 아니라 템플릿 T를 사용해서 범용성을 키우면 좋을 듯 --> 인터페이스도 String이 아니라 T로 변경. (내가 과거에 짠 코드들 많이 더럽고 부끄럽다..)
• 전략 패턴을 사용할 수 있을 것 같음. 그 외에 다른 패턴들을 어떻게 적용시켜야 할지 감이 잘 안 옴

[tjdgus3537]

• Block의 List를 직접 외부와 주고 받는데 이러지 말고 이터레이터 패턴을 적용하여 읽기만 가능하게 해주는 것도 괜찮아 보임. 근데 Copier class에서 복사할 block의 번호를 함께 받기 때문에 좀 난감할 수도 있음.(사용하는 쪽 내부를 잘 몰라서 얘기를 해봐야겠다)
• PairBlock이라는 class를 사용하는 것 대신 Pair으로 사용하는 것도 고려해봐야 함
• Diff와 Copier에 싱글톤 패턴을 적용해서 MainModel에서 필드로 갖는 것 말고 함수 호출 때마다 get해서 사용하는 것 고려

[WraithKim]

자바에는 pair 없음. 내가 잘못 기억함

[tjdgus3537]

ㅇㅇ.. 나도 착각함