세 개의 필드 (모든 문자열 유형)가있는 “보고서”개체 목록이 있습니다.
ReportKey
StudentNumber
School
나는 정렬 코드가 있습니다.
Collections.sort(reportList, new Comparator<Report>() {
@Override
public int compare(final Report record1, final Report record2) {
return (record1.getReportKey() + record1.getStudentNumber() + record1.getSchool())
.compareTo(record2.getReportKey() + record2.getStudentNumber() + record2.getSchool());
}
});
어떤 이유로 정렬 된 순서가 없습니다. 하나는 필드 사이에 공백을 두는 것이 좋지만 그 이유는 무엇입니까?
코드에 문제가 있습니까?
답변
코드에 문제가 있습니까?
예. 비교하기 전에 세 필드를 함께 추가하는 이유는 무엇입니까?
나는 아마도 다음과 같이 할 것입니다. (필드가 정렬하려는 순서라고 가정)
@Override public int compare(final Report record1, final Report record2) {
int c;
c = record1.getReportKey().compareTo(record2.getReportKey());
if (c == 0)
c = record1.getStudentNumber().compareTo(record2.getStudentNumber());
if (c == 0)
c = record1.getSchool().compareTo(record2.getSchool());
return c;
}
답변
(원래 여러 필드를 기반으로 Java에서 객체 목록을 정렬 하는 방법에서 )
이 요점의 원래 작업 코드
Java 8 람다 사용 (2019 년 4 월 10 일 추가됨)
Java 8은 Lambda로이 문제를 훌륭하게 해결합니다 (Guava와 Apache Commons는 여전히 더 많은 유연성을 제공 할 수 있음).
Collections.sort(reportList, Comparator.comparing(Report::getReportKey)
.thenComparing(Report::getStudentNumber)
.thenComparing(Report::getSchool));
여기서 한 가지 장점은 게터가 느리게 평가된다는 것입니다 (예 : getSchool()
관련성이있는 경우에만 평가됨).
지저분하고 복잡한 : 손으로 정렬
Collections.sort(pizzas, new Comparator<Pizza>() {
@Override
public int compare(Pizza p1, Pizza p2) {
int sizeCmp = p1.size.compareTo(p2.size);
if (sizeCmp != 0) {
return sizeCmp;
}
int nrOfToppingsCmp = p1.nrOfToppings.compareTo(p2.nrOfToppings);
if (nrOfToppingsCmp != 0) {
return nrOfToppingsCmp;
}
return p1.name.compareTo(p2.name);
}
});
이것은 많은 타이핑, 유지 보수가 필요하며 오류가 발생하기 쉽습니다. 유일한 장점은 getter 가 관련이있을 때만 호출 .
반사적 방식 : BeanComparator로 정렬
ComparatorChain chain = new ComparatorChain(Arrays.asList(
new BeanComparator("size"),
new BeanComparator("nrOfToppings"),
new BeanComparator("name")));
Collections.sort(pizzas, chain);
분명히 이것은 더 간결하지만 대신 문자열을 사용하여 필드에 대한 직접 참조를 잃어 버리기 때문에 오류가 발생하기 쉽습니다 (유형 안전성, 자동 리팩토링 없음). 이제 필드의 이름이 변경되면 컴파일러는 문제를보고하지도 않습니다. 또한이 솔루션은 리플렉션을 사용하기 때문에 정렬이 훨씬 느립니다.
찾아 가기 : Google Guava의 ComparisonChain으로 정렬
Collections.sort(pizzas, new Comparator<Pizza>() {
@Override
public int compare(Pizza p1, Pizza p2) {
return ComparisonChain.start().compare(p1.size, p2.size).compare(p1.nrOfToppings, p2.nrOfToppings).compare(p1.name, p2.name).result();
// or in case the fields can be null:
/*
return ComparisonChain.start()
.compare(p1.size, p2.size, Ordering.natural().nullsLast())
.compare(p1.nrOfToppings, p2.nrOfToppings, Ordering.natural().nullsLast())
.compare(p1.name, p2.name, Ordering.natural().nullsLast())
.result();
*/
}
});
이것은 훨씬 더 좋지만 가장 일반적인 사용 사례에 대해 약간의 상용구 코드가 필요합니다. null 값은 기본적으로 더 작은 값이어야합니다. 널 필드의 경우 Guava에이 경우 수행 할 작업에 대한 추가 지시문을 제공해야합니다. 특정 작업을 수행하려는 경우 유연한 메커니즘이지만 종종 기본 케이스 (예 : 1, a, b, z, null)를 원합니다.
그리고 아래 주석에서 언급했듯이 이러한 getter는 각 비교에 대해 모두 즉시 평가됩니다.
Apache Commons CompareToBuilder로 정렬
Collections.sort(pizzas, new Comparator<Pizza>() {
@Override
public int compare(Pizza p1, Pizza p2) {
return new CompareToBuilder().append(p1.size, p2.size).append(p1.nrOfToppings, p2.nrOfToppings).append(p1.name, p2.name).toComparison();
}
});
Guava의 ComparisonChain과 마찬가지로이 라이브러리 클래스는 여러 필드에서 쉽게 정렬되지만 null 값 (예 : 1, a, b, z, null)에 대한 기본 동작도 정의합니다. 그러나 자신의 Comparator를 제공하지 않는 한 다른 것도 지정할 수 없습니다.
다시 말하지만, 아래 주석에서 언급했듯이 이러한 getter는 각 비교에 대해 모두 즉시 평가됩니다.
그러므로
궁극적으로 그것은 풍미와 유연성 (Guava의 ComparisonChain) 대 간결한 코드 (Apache의 CompareToBuilder)에 대한 필요성으로 귀결됩니다.
보너스 방법
CodeReview 의 우선 순위 에 따라 여러 비교기를 결합하는 멋진 솔루션을 찾았 습니다 MultiComparator
.
class MultiComparator<T> implements Comparator<T> {
private final List<Comparator<T>> comparators;
public MultiComparator(List<Comparator<? super T>> comparators) {
this.comparators = comparators;
}
public MultiComparator(Comparator<? super T>... comparators) {
this(Arrays.asList(comparators));
}
public int compare(T o1, T o2) {
for (Comparator<T> c : comparators) {
int result = c.compare(o1, o2);
if (result != 0) {
return result;
}
}
return 0;
}
public static <T> void sort(List<T> list, Comparator<? super T>... comparators) {
Collections.sort(list, new MultiComparator<T>(comparators));
}
}
Ofcourse Apache Commons Collections에는 이미 다음과 같은 유틸리티가 있습니다.
ComparatorUtils.chainedComparator (comparatorCollection)
Collections.sort(list, ComparatorUtils.chainedComparator(comparators));
답변
내가 사용 비교기 할 것 구아바 의를 ComparisonChain
:
public class ReportComparator implements Comparator<Report> {
public int compare(Report r1, Report r2) {
return ComparisonChain.start()
.compare(r1.getReportKey(), r2.getReportKey())
.compare(r1.getStudentNumber(), r2.getStudentNumber())
.compare(r1.getSchool(), r2.getSchool())
.result();
}
}
답변
이것은 오래된 질문이므로 Java 8에 해당하는 항목이 표시되지 않습니다. 다음은이 특정 사례에 대한 예입니다.
import java.util.ArrayList;
import java.util.Collections;
import java.util.Comparator;
import java.util.List;
/**
* Compares multiple parts of the Report object.
*/
public class SimpleJava8ComparatorClass {
public static void main(String[] args) {
List<Report> reportList = new ArrayList<>();
reportList.add(new Report("reportKey2", "studentNumber2", "school1"));
reportList.add(new Report("reportKey4", "studentNumber4", "school6"));
reportList.add(new Report("reportKey1", "studentNumber1", "school1"));
reportList.add(new Report("reportKey3", "studentNumber2", "school4"));
reportList.add(new Report("reportKey2", "studentNumber2", "school3"));
System.out.println("pre-sorting");
System.out.println(reportList);
System.out.println();
Collections.sort(reportList, Comparator.comparing(Report::getReportKey)
.thenComparing(Report::getStudentNumber)
.thenComparing(Report::getSchool));
System.out.println("post-sorting");
System.out.println(reportList);
}
private static class Report {
private String reportKey;
private String studentNumber;
private String school;
public Report(String reportKey, String studentNumber, String school) {
this.reportKey = reportKey;
this.studentNumber = studentNumber;
this.school = school;
}
public String getReportKey() {
return reportKey;
}
public void setReportKey(String reportKey) {
this.reportKey = reportKey;
}
public String getStudentNumber() {
return studentNumber;
}
public void setStudentNumber(String studentNumber) {
this.studentNumber = studentNumber;
}
public String getSchool() {
return school;
}
public void setSchool(String school) {
this.school = school;
}
@Override
public String toString() {
return "Report{" +
"reportKey='" + reportKey + '\'' +
", studentNumber='" + studentNumber + '\'' +
", school='" + school + '\'' +
'}';
}
}
}
답변
보고서 키, 학생 번호, 학교 순으로 정렬하려면 다음과 같이해야합니다.
public class ReportComparator implements Comparator<Report>
{
public int compare(Report r1, Report r2)
{
int result = r1.getReportKey().compareTo(r2.getReportKey());
if (result != 0)
{
return result;
}
result = r1.getStudentNumber().compareTo(r2.getStudentNumber());
if (result != 0)
{
return result;
}
return r1.getSchool().compareTo(r2.getSchool());
}
}
이것은 물론 어떤 값도 null이 될 수 없다고 가정합니다. 보고서, 보고서 키, 학생 번호 또는 학교에 대해 null 값을 허용해야하는 경우 더 복잡해집니다.
공백을 사용하여 작동하도록 문자열 연결 버전을 얻을 수 있지만 , 자체적으로 공백 등을 포함하는 이상한 데이터가있는 경우 이상한 경우에 여전히 실패합니다. 위 코드는 원하는 논리 코드입니다. 먼저 보고서 키로 비교 한 다음 보고서 키가 동일한 경우에만 학생 번호로 귀찮게합니다.
답변
Java 8 Lambda 접근 방식을 사용하는 것이 좋습니다.
List<Report> reportList = new ArrayList<Report>();
reportList.sort(Comparator.comparing(Report::getRecord1).thenComparing(Report::getRecord2));
답변
Java8에서 여러 필드로 정렬
package com.java8.chapter1;
import java.util.Arrays;
import java.util.Comparator;
import java.util.List;
import static java.util.Comparator.*;
public class Example1 {
public static void main(String[] args) {
List<Employee> empList = getEmpList();
// Before Java 8
empList.sort(new Comparator<Employee>() {
@Override
public int compare(Employee o1, Employee o2) {
int res = o1.getDesignation().compareTo(o2.getDesignation());
if (res == 0) {
return o1.getSalary() > o2.getSalary() ? 1 : o1.getSalary() < o2.getSalary() ? -1 : 0;
} else {
return res;
}
}
});
for (Employee emp : empList) {
System.out.println(emp);
}
System.out.println("---------------------------------------------------------------------------");
// In Java 8
empList.sort(comparing(Employee::getDesignation).thenComparing(Employee::getSalary));
empList.stream().forEach(System.out::println);
}
private static List<Employee> getEmpList() {
return Arrays.asList(new Employee("Lakshman A", "Consultent", 450000),
new Employee("Chaitra S", "Developer", 250000), new Employee("Manoj PVN", "Developer", 250000),
new Employee("Ramesh R", "Developer", 280000), new Employee("Suresh S", "Developer", 270000),
new Employee("Jaishree", "Opearations HR", 350000));
}
}
class Employee {
private String fullName;
private String designation;
private double salary;
public Employee(String fullName, String designation, double salary) {
super();
this.fullName = fullName;
this.designation = designation;
this.salary = salary;
}
public String getFullName() {
return fullName;
}
public String getDesignation() {
return designation;
}
public double getSalary() {
return salary;
}
@Override
public String toString() {
return "Employee [fullName=" + fullName + ", designation=" + designation + ", salary=" + salary + "]";
}
}