[Review] ATDD 1주차 1차 PR

NextStep에서의 첫 리뷰다! 

송용주 리뷰어 님께 감사의 말을 전하고 싶다!!

 

1. 리뷰 정리

1-1) Private 메서드는 public 밑으로

원래 내 코드들을 보면 Private 메서드들은 public 아래 뒀었다. 근런데 이번에 왠지 모르게 위에 두고 싶었다?

ㅎㅎ 바로 리뷰에서 말씀을 주셔서 다시 원상복귀 시켜야 겠다 ㅎㅎ!!

 

1-2) 상태코드 검증하기

다만 의문이 있는데...

1. 상태코드 확인은 부차적인것이 아닐까?

사실 인수조건(시나리오)을 봤을때

When 그 지하철역을 삭제하면
Then 그 지하철역 목록 조회 시 생성한 역을 찾을 수 없다

 

then 절에 해당되는 생성한 역을 찾을수 있는지가 핵심이라 생각되는데, 이때 When에서 수행한 행동의 상태코드를 꼭 검증해야 하나? 라는 생각이 들었습니다 ㅎㅎ

어떠 한 사건이 일어났을때(when) 결과가 어떠해야 한다(then)에서, when에서 상태코드를 확인하는 것은 부차적인 일 아닐까? 란 생각을 했는데, 검증하는 쪽이 더 좋은 방향인것 이겠죠?

 

2. 상태 코드 확인은 when? then?
말씀하신 지하철 역 삭제같은 경우 when에서 바로 검증을 해야 할거 같기도 하고... then 에서 삭제에 대한 "결과"를 검증해야 할것 같기도 하고... 보통 어느쪽이 맞을까요?

 

답변 : 

 

1-3) 의미가 명확한 변수이름

이건 항상 느끼지만 그당시에는 나는 명확하다 생각해서 저렇게 사용한 것 인데....

돌이켜 보면 별로인 뭐 그런?? ㅠ,ㅠ

 

항상 변수명을 신경쓰자 생각하면서도, 자주 생각 못하는 부분 같기도 하다....