Upstream/Plan
1
0
Fork 0
mirror of https://github.com/plan-player-analytics/Plan.git synced 2024-09-10 05:48:13 +02:00
Code Issues Projects Releases Wiki Activity

Modify the method getting the biggest int / long of a collection to using streams instead of doing it manually.

Browse Source 
The reason for it is to adjust it to the other methods in the same class
This commit is contained in:
Fuzzlemann 2017-07-23 20:54:14 +02:00
parent 4399f8aecc
commit 31b432d01e
1 changed files with 14 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
Plan/src/main/java/com/djrapitops/plan/utilities/analysis/MathUtils.java
View File

@ -3,6 +3,8 @@ package main.java.com.djrapitops.plan.utilities.analysis;
import java.io.Serializable;
import java.util.Collection;
import java.util.OptionalDouble;
import java.util.OptionalInt;
import java.util.OptionalLong;
import java.util.stream.Stream;
/**
@ -124,13 +126,13 @@ public class MathUtils {
* @return
*/
public static int getBiggest(Collection<Integer> values) {
int biggest = 1;
for (Integer value : values) {
if (value > biggest) {
biggest = value;
}
OptionalInt biggest = values.stream().mapToInt(i -> i).max();
if (biggest.isPresent()) {
return biggest.getAsInt();
} else {
return 1;
}
return biggest;
}
/**
@ -139,12 +141,12 @@ public class MathUtils {
* @return
*/
public static long getBiggestLong(Collection<Long> values) {
long biggest = 1;
for (Long value : values) {
if (value > biggest) {
biggest = value;
}
OptionalLong biggest = values.stream().mapToLong(i -> i).max();
if (biggest.isPresent()) {
return biggest.getAsLong();
} else {
return 1;
}
return biggest;
}
}