Skip to content

Commit a816c5c

Browse files
committed
Use Integer.signum instead of explicit comparison
Turns out Integer.signum already produces what I wanted with that use of the ternary operator.
1 parent 7aebf5f commit a816c5c

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

src/main/java/com/packt/datastructuresandalg/lesson4/activityselection/ActivitySelection.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ public Activity(int i, int s, int f) {
2020
public List<Activity> select(List<Activity> activities) {
2121
List<Activity> selected = new ArrayList<>();
2222
List<Activity> sortedActivities = new ArrayList<>(activities);
23-
Collections.sort(sortedActivities, (o1, o2) -> o1.finish < o2.finish ? -1 : o1.finish > o2.finish ? 1 : 0);
23+
Collections.sort(sortedActivities, (o1, o2) -> Integer.signum(o1.finish - o2.finish));
2424
if (sortedActivities.size() > 0)
2525
selected.add(sortedActivities.get(0));
2626
for (int i = 1; i < sortedActivities.size(); i++)

0 commit comments

Comments
 (0)