흡연장 메소드 분리

gnoesnooj·2022년 5월 12일
0

문제점

기존 코드는 한 메소드 내에서 여러가지 역할을 했다.
1. api key 를 통해서 URl connection 해주기
2. BufferedReader 로 JSON 받아오기
3. JSON 파싱해서 원하는 데이터 뽑아내기 (위도, 경도)
4. 파싱한 데이터 DB에 저장해주기.
5. 각종 exception 상황에 대한 검증 로직 또한 포함되어있음.

이렇다 보니 기존의 코드가 객체지향스럽지 않은 코드로 보였고, 더 좋은 코드를 작성하기 위해 메소드 분리를 하기로 했다.

해결

기존의 코드를
API 연결 > JSON 파싱 > DB 저장 이렇게 3가지 역할로 나누어서 메소드를 분리를 하였다.

profile
누구나 믿을 수 있는 개발자가 되자 !

0개의 댓글