有没有更有效的方法来编写多个 if else？ [复制]

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

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

我不会使用它，除非乐队的大小不同（你可以exploit integer division）

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 等，只需在上面的 if 语句中添加规则 ||marks%10==0。

希望这有帮助:)

Answer 6

注意：这是使用Java 8 制作的，无权访问MapOf，也没有使用外部库。也没有使用 Streams 来显示其他选项。

我创建了一个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();

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