SortMain2 클래스의 DescComparator 메서드 return 부분이 수정되어야 할 것 같습니다.

Question

영한님 강의 너무 잘 보고 있습니다. 다름이 아니라 Comparator를 공부하며 혼자 이것저것 해보던 중에 SortMain2 클래스의 DescComparator 메서드가 3,2,1 배열을 정렬하면 오름차순 으로 정렬되는 것을 확인하여 문의 드립니다. static class DescComparator implements Comparator<Integer> { @Override public int compare(Integer o1, Integer o2) { System.out.println("o1=" + o1 + " o2=" + o2); return (o1 < o2) ? -1 : ((o1 == o2) ? 0 : 1) * -1; } } 강의에서는 오름차순 정렬된 1,2,3 배열을 내림차순 정렬해서 정렬이 문제가 없었지만 만약 3,2,1 배열을 내림차순 정렬 하게 된다면, return (o1 < o2) ? -1 : ((o1 == o2) ? 0 : 1) * -1; 첫번째 삼항 연산자 부분에서 앞에 o1, o2를 비교하고 -1 이 반환되며 오름차순으로 정렬 되는 것 같습니다. 리턴 부분이 전체 비교 값에 -1이 곱해질 수 있도록 return ((o1 < o2) ? -1 : ((o1 == o2) ? 0 : 1)) * -1; 로 수정 되어야 할 것 같아 문의 드립니다.

김영한 · Answer

오~ 이준영님 그렇네요! 중요한 괄호가 빠졌네요! 다음 버전에 패치할게요^^! 고맙습니다!

인프런 AI 인턴 · Answer

안녕하세요, 인프런 AI 인턴입니다. 이준영님. Comparator를 사용하여 내림차순 정렬을 구현하시다가 궁금한 점이 생기셨군요. 말씀하신 메서드에 있는 (o1 < o2) ? -1 : ((o1 == o2) ? 0 : 1) * -1 부분은 계산 순서 문제로 인해 예상과는 다른 결과를 낼 수 있습니다. 내림차순으로 올바르게 작동하려면, 조건을 반대로 설정하거나 음수 부호를 사용해야 합니다. 다음과 같이 수정할 수 있습니다: static class DescComparator implements Comparator { @Override public int compare(Integer o1, Integer o2) { System.out.println("o1=" + o1 + " o2=" + o2); return (o1 > o2) ? -1 : ((o1 == o2) ? 0 : 1); } } 또는 간단하게: static class DescComparator implements Comparator { @Override public int compare(Integer o1, Integer o2) { System.out.println("o1=" + o1 + " o2=" + o2); return o2.compareTo(o1); } } 유사한 질문이 이미 여러 번 있어서 참고하실 수 있는 URL을 아래에 첨부합니다: 유사 질문: ‘해당 코드는 삽입정렬인가요?’ 유사 질문: ‘Comparator질문입니다’ 이 외에도 다른 분들의 답변이 추가적인 도움을 줄 수 있습니다. 저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다. 현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏 추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.