有没有更有效的方法来写多个如果不是?

Question

我需要将平均分数与字母等级相匹配。这意味着

if (90 < avg && avg < 100) {
return 'A';
} 

依此类推，直到"F"，有 5 个 if-else 语句。

这是很多重复，我匹配的范围是相同的长度。

有没有更有效的方法可以做到这一点？我不想重复 if-else 语句 5 次。

Answer 1

我喜欢使用enum来解决此类问题。

您只需在枚举值中定义范围并使用谓词即可。

import java.util.*;
public class MyClass {
public static void main(String args[]) {
System.out.println(Letter.getValue(13)); // B
System.out.println(Letter.getValue(28)); // C
System.out.println(Letter.getValue(43)); // empty
}
}
enum Letter {
A(1, 10), B(11, 20), C(21, 30);
int min;
int max;
Letter (int min, int max) {
this.min = min;
this.max = max;
}
boolean inRange(int v) {
return v >= min && v <= max;
}
static Optional<Letter> getValue(int i) {
return Arrays.stream(values())
.filter(v -> v.inRange(i))
.findAny();
}
}

Answer 2

您可能会发现这种方法冗长且设计过度。是的，它是。

不过，我喜欢评分系统的定义方式。

class Test {
public static void main(String[] args) {
Map<String, Predicate<Integer>> gradingSystem = new HashMap<>();
gradingSystem.put("A", mark -> mark >= 90 && mark <= 100);
gradingSystem.put("B", mark -> mark >= 80 && mark < 90);
translateScoreIntoGrade(86, gradingSystem); // B
translateScoreIntoGrade(95, gradingSystem); // A
}
public static String translateScoreIntoGrade(int score, Map<String, Predicate<Integer>> gradingSystem) {
return gradingSystem.entrySet().stream()
.filter(gradePredicate -> gradePredicate.getValue().test(score))
.findFirst()
.map(Map.Entry::getKey)
.orElseThrow(() -> new IllegalArgumentException("This grade isn't valid for this system!"));
}
} 

Answer 3

你可以像这个例子一样做：

public static String avgValuetoGrade(int value) {
int index = value/10;
int gradeIndex = Math.max(0, index - 4);
return (char)('F'-gradeIndex );
}

首先将你的平均值除以 10，你得到数字 0-9(整数除法(。接下来，您必须以某种方式将数字 0-4 减少到一个数字-4为您提供数字 0-5 并将 0-4 映射到 0。然后你可以归还字符(感谢@Carlos Heuberger(。

Answer 4

这将是以函数式风格编写的相同内容的示例。没有重复，但很冗长。

除非波段大小不同，否则我不会使用它(您可以在其中利用整数除法(

Map<Range<Integer>, Character> rangeToGrade = Map.of(
Range.between(90, 100), 'A',
Range.between(80, 90), 'B'
//... the rest
);
int mark = 50;
char grade = rangeToGrade.entrySet().stream()
.filter(e -> e.getKey().contains(mark))
.map(Map.Entry::getValue)
.findFirst()
.orElse('?'); // or throw an exception

Map.of来自Java 9。Range来自例如Apache Commons

Answer 5

您可以将其映射到数组并避免分支预测规则，从而使其更有效率并节省if/else内容。

class StackOverflow
{
public static void main(String args[])
{
char grades[]={'A','B','C','D','E','F'};
convert(grades,95);
convert(grades,90);
convert(grades,110);
convert(grades,123);
convert(grades,150);
}
static void convert(char grades[],int marks)
{
if(marks<=90||marks>=150)
{
System.out.println("Invalid marks");
return;
}
int val=marks/10-9;
System.out.println(grades[val]);
}
}

注意：这假设 91-99 是 A，100-109 是B，110-119 是C，依此类推。如果你想避免数字100,110等，只需添加规则||标记%10==0在上面的 if 语句中。

希望这对:)有所帮助

Answer 6

注意：这是使用Java 8进行的，无法访问MapOf，也不使用外部库。 也没有使用流来显示其他选项。

我创建了一个GradeBook类，当它被实例化时，Map的类字段充满了用于查找字母等级的键。 然后，您可以从GradeBook对象调用.getGrade(int)，还可以处理负输入和输入高于 100 的情况，返回N。 否则，它将从Map返回正确的等级。

这更像是一种面向对象的方法，而不是使用static方法调用：

public class GradeBook {
private Map<Integer, Character> map;
public GradeBook() {
constructMap();
}
public char getGrade(int grade) {
if (grade >= 0 && grade <= 100) {
return map.get(Math.max(0, grade/10 - 4));
}
else {
return 'N';
}
}
public void constructMap() {
//You can use MapOf to add all of them at once in Java9+
map = new HashMap<>();
map.put(0, 'F');
map.put(1, 'E');
map.put(2, 'D');
map.put(3, 'C');
map.put(4, 'B');
map.put(5, 'A');
map.put(6, 'A');
}
public static void main(String [] args) {
GradeBook grades = new GradeBook();
//Testing different values
System.out.println(grades.getGrade(100));
System.out.println(grades.getGrade(75));
System.out.println(grades.getGrade(80));
System.out.println(grades.getGrade(91));
System.out.println(grades.getGrade(45));
System.out.println(grades.getGrade(2));
System.out.println(grades.getGrade(-1));
}
}

输出：

A
C
B
A
F
F
N

这种实现的缺点是第一次实现的时间有点长，但好处是它很容易重用，因为你只需要在你需要的任何地方创建一个new GradeBook()。

Answer 7

为了炫耀一下，在 Java 版本 12(启用预览版(中，使用新的 Switch 表达式：

String grade = switch(avg/10) {
case 9,10 -> "A";
case 8    -> "B";
case 7    -> "C";
case 6    -> "D";
case 5    -> "E";
default   -> "F";
};

或者，非常灵活，如果不是懒惰的话：

String grade = switch(avg) {
case 90, 91, 92, 93, 94, 95, 96, 97, 98, 99, 100 -> "A";
case 80, 81, 82, 83, 84, 85, 86, 87, 88, 89      -> "B";
// you got the idea, I AM lazy

实际 (_？( 解决方案：使用NavigableMap(示例TreeMap(及其floorEntry()或lowerEntry()方法：

NavigableMap<Integer, String> grades = new TreeMap<>();
grades.put(90, "A");
grades.put(80, "B"); 
...
// usage
String grade = grades.floorEntry(avg).getValue();

^{地图中的值最终必须调整}