Skip to content

Commit f7dafc2

Browse files
author
Bytekeeper
committed
Moved comparators into their relevant classes.
1 parent d1357a6 commit f7dafc2

File tree

5 files changed

+11
-58
lines changed

5 files changed

+11
-58
lines changed

src/main/java/bwem/Altitude.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,10 @@
1212

1313
package bwem;
1414

15+
import bwapi.Pair;
16+
17+
import java.util.Comparator;
18+
1519
/**
1620
* Immutable wrapper of the integer primitive to satisfy the original C++ definition:
1721
* defs.h:54:typedef int16_t altitude_t;
@@ -21,6 +25,7 @@
2125
public final class Altitude implements Comparable<Altitude> {
2226
public static final Altitude UNINITIALIZED = new Altitude(-1);
2327
public static final Altitude ZERO = new Altitude(0);
28+
public static final Comparator<Pair<?, Altitude>> BY_ALTITUDE_ORDER = Comparator.comparing(p -> p.getRight().intValue());
2429
private final int val;
2530

2631
Altitude(final int val) {

src/main/java/bwem/BWMapInitializer.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,7 @@ private List<Pair<WalkPosition, Altitude>> getSortedDeltasByAscendingAltitude(
149149
}
150150
}
151151

152-
deltasByAscendingAltitude.sort(new PairGenericAltitudeComparator<>());
152+
deltasByAscendingAltitude.sort(Altitude.BY_ALTITUDE_ORDER);
153153

154154
return deltasByAscendingAltitude;
155155
}
@@ -440,7 +440,7 @@ private List<Pair<WalkPosition, MiniTile>> getSortedMiniTilesByDescendingAltitud
440440
}
441441
}
442442

443-
miniTilesByDescendingAltitude.sort(new PairGenericMiniTileAltitudeComparator<>());
443+
miniTilesByDescendingAltitude.sort(MiniTile.BY_ALTITUDE_ORDER);
444444
Collections.reverse(miniTilesByDescendingAltitude);
445445

446446
return miniTilesByDescendingAltitude;

src/main/java/bwem/MiniTile.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,11 @@
1212

1313
package bwem;
1414

15+
import bwapi.Pair;
1516
import bwapi.WalkPosition;
1617

18+
import java.util.Comparator;
19+
1720
import static bwem.AreaId.UNINITIALIZED;
1821

1922
/**
@@ -30,6 +33,7 @@
3033
*/
3134
public final class MiniTile {
3235
private static final AreaId blockingCP = new AreaId(Integer.MIN_VALUE);
36+
public static final Comparator<Pair<?, MiniTile>> BY_ALTITUDE_ORDER = Comparator.comparing(p -> p.getRight().getAltitude().intValue());
3337

3438
private Altitude
3539
altitude; // 0 for seas ; != 0 for terrain and lakes (-1 = not computed yet) ; 1 =

src/main/java/bwem/util/PairGenericAltitudeComparator.java

Lines changed: 0 additions & 28 deletions
This file was deleted.

src/main/java/bwem/util/PairGenericMiniTileAltitudeComparator.java

Lines changed: 0 additions & 28 deletions
This file was deleted.

0 commit comments

Comments
 (0)