SortMain2 클래스의 DescComparator 메서드 re... - 인프런

김영한의 실전 자바 - 중급 2편

정렬1 - Comparable, Comparator

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

작성

수정됨

영한님 강의 너무 잘 보고 있습니다.
다름이 아니라 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;
로 수정 되어야 할 것 같아 문의 드립니다.

java 객체지향 코딩-테스트 알고리즘

답변 2

김영한

지식공유자

오~ 이준영님 그렇네요! 중요한 괄호가 빠졌네요!

다음 버전에 패치할게요^^! 고맙습니다!

인프런 AI 인턴

안녕하세요, 인프런 AI 인턴입니다. 이준영님.

Comparator를 사용하여 내림차순 정렬을 구현하시다가 궁금한 점이 생기셨군요. 말씀하신 메서드에 있는 (o1 < o2) ? -1 : ((o1 == o2) ? 0 : 1) * -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);
    }
}

또는 간단하게:

static class DescComparator implements Comparator<Integer> {
    @Override
    public int compare(Integer o1, Integer o2) {
        System.out.println("o1=" + o1 + " o2=" + o2);
        return o2.compareTo(o1);
    }
}

유사한 질문이 이미 여러 번 있어서 참고하실 수 있는 URL을 아래에 첨부합니다:

유사 질문: ‘해당 코드는 삽입정렬인가요?’
유사 질문: ‘Comparator질문입니다’

이 외에도 다른 분들의 답변이 추가적인 도움을 줄 수 있습니다.

저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다.
현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏
추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.

인프런 커뮤니티 질문&답변

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