search

Sizzzz / cnu-team18-custom

0 stars 2 forks source link

[Robin] FULL_HOUSE 테스트 요청 #17

Open napi opened 7 years ago

napi commented 7 years ago

현재 작성된 코드로 아래의 테스트를 통과하지 못할 것 같습니다. Hand : 3, 3, 3, 4, 4, 4, 5

테스트 코드 추가 후 확인 바랍니다. Fail 일 경우, 코드 수정 부탁드립니다.

boncheul92nd commented 7 years ago

image 남겨주신 문제 해결했습니다. 감사합니다.

napi commented 7 years ago

if(result1.size() == 1 && result1 != null) 이렇게 짜여있는 코드가 있는데, 안좋은 습관 입니다.

null 체크가 뒤에 있기 때문에, result1.size() 를 먼저 수행하면서 NullPointerException 이 발생하죠.

그리고 몇가지 리뷰를 드리자면, List result0 = cutCardListIfAvailable(cardList); <-- Generic이 빠져있습니다. List의 size() 만을 사용하려는 것은 이해하지만, cutCardListIfAvailable(cardList) 메서드의 return 값에 대한 확신이 없네요.

둘째로, cutCardListIfAvailable(List cardList) 메서드의 네이밍 문제인데요. 하는 역활은 트리플을 제외한 나머지 카드만 가져오는 것 같은데, 네이밍만 보고 저것을 이해하기는 어려울 것 같네요.

boncheul92nd commented 7 years ago

남겨주신 코멘트에 따라 변경사항 반영하여 보았습니다. 감사합니다. (https://github.com/Sizzzz/cnu-team18-custom/commit/7905ee0901a63e5e96a72bac1766eb8102c8e393)