[Backend : fix] 리펙토링 - 클린코드

[Dayon-Hong]

• ResponseEntity 굳이 필요없는곳에는 사용 X

• spring security url - Token 넣어도 권한설정 안되는 오류 해결하기

• 큰 메소드들 분리하기

• Repository에서 Optional으로 null Check 대신 따로 예외처리 조건문 사용하기
  https://www.baeldung.com/java-optional

• api url 중복 -> yml prefix 설정

• 서비스와 서비스 impl 분리
  Service와 ServiceImpl 분리 이유 : 참고글 1 , 참고글 2

---

• 테스트 코드가 아예 없음
• DB n + 1 이슈 #82
• 8. s3 구형 방식 설정 -> 신형 방식으로 변경하기
• 9. S3 로직 아쉽네용
  • transactional 왜 씀?
  • throws 왜 던짐?
  • null 체킹 왜 함?

---

[Dayon-Hong]

Twitter 공식 API 문서

• https://www.postman.com/twitter/workspace/twitter-s-public-workspace/collection/9956214-784efcda-ed4c-4491-a4c0-a26470a67400

[Dayon-Hong]

1. flightService 내에 있는 findAirportCode의 접근 제어자 - 외부에서 사용 안하면 private로 변경

2. == 연산자를 통한 null 처리

   • 옵셔널로 묶어서 ifPresent, isPresent 처리

3. 배열 대신 리스트 사용

4. 불필요한 주석 삭제

1. 항상 존재하는 city만 검색할 예정이지만, 그래도 옵셔널로 묶어서 null 처리