Java Puzzlers : 1. Oddity

아래 코드가 제대로 작동 할 것인가?

-- CODE start --
   public static boolean isOdd(int i){
       return i % 2 == 1;  
   }
-- CODE end --

조금 주의 깊게 생각해 보면 쉽게 알 수 있을지 모르겠지만 --''

뭐... 당연히 잘 작동할 만한 코드를 보여주면서 잘 작동할꺼냐고 물었으니... 눈치 빠른 사람이라면 아니라고 대답했겠죠?

당근 정답도 아니오 --;;
인자로 받아드리는 int의 1/4에 대해서는 오답을 return하게 된다.

이유는... 음수에 대한 고려가 안되어 있네요.
음수를 2로 나머지 연산을 하면 결과값이 -1이 나옵니다.
주의해야 할 부분이죠?

이런 문제를 사전에 방지하고자 하는 경우, 꼭 음수, 0, 양수에 대해 테스트를 해 봐야 합니다.

위 문제의 코드를 바르게 고치는 여러 가지 방법 중... 쉽게 생각할 수 있는 것은
-- CODE start --
   public static boolean isOdd(int i){
      return i % 2 != 0;
   }
-- CODE end --

간단하죠??

성능을 고려해서 바꿔 본다면??
-- CODE start --
   public static boolean isOdd(int i){
       return (i & 1) != 0;  
   }
-- CODE end --

전산을 전공한 사람이라면 쉽게 눈치 챘을만한 코드죠..??

저도 이 책 보고 알았지만.. 참...

책에서도 강조합니다. 성급한 optimize는 나쁜 생각이다. (It's a bad idea to optimize prematurely)
optimize는 코딩 하면서 하는게 아니고 프로그램을 다 작성한 후 병목인 부분을 찾아 고치는 것이 맞는 절차라고 합니다.
물론 이 경우에는 처음 수정된 코드와 비교해도 최적화 코드 자체가 명확하기 때문에 별 문제는 없다라고 하네요.

이상 오늘 공부한 내용 끝~




출처 : Java Puzzlers