填充可重用方法 Java 中使用的常量映射的最佳实践

Question

我正在使用一种可重用的方法来映射两个代码，当我们收到“a”时，我们想在我们的代码中使用“01”。 这是一个静态方法，所以它只创建一次并且我们重用该方法是正确的，还是这种不好的做法并且每次我们调用这个方法时都不必要地进行处理？ 这是一个很好的做法，还是我应该在其他地方创建这个地图并只使用从它返回值的方法？

    private static String mapStatus(String duckStatus){

       
        Map<String, String> statusMap = new HashMap<String,String>();
        statusMap.put("a","01");
        statusMap.put("b","02");
        statusMap.put("c","03");

        if (!statusMap.containsKey(duckStatus)){
            throw new NullPointerException("Invalid Code does not map to status code");
        }
        return statusMap.get(duckStatus);
    }
    

Answer 1

根据评论， enum可能是一个不错的选择，但正如@kendavidson 已经提到的那样，它是有争议的。 从性能的角度来看，我看不到enum和HashMap之间的太大区别，但在我看来HashMap会更合适，并且会提供更好的可读性。 我在您编写的代码中建议的唯一区别是，使HashMap静态填充，以便在每次调用mapStatus方法时要查找statusMap时避免HashMap建立时间。 修改后的代码如下所示：

public class Test {
    private static final Map<String, String> STATUS_MAP = new HashMap<>();
    static {
        // Mapping the status values in Duck Creek to the Values used here
        STATUS_MAP.put("a", "01");
        STATUS_MAP.put("b", "02");
        STATUS_MAP.put("c", "03");
    }

    private static String mapStatus(String duckStatus) {
        if (!STATUS_MAP.containsKey(duckStatus)) {
            throw new IllegalArgumentException("Invalid code " + duckStatus + " does not map to status code");
        }
        return STATUS_MAP.get(duckStatus);
    }
}

笔记：

1. 不知道，如果你想检查duckStatus在duckCreekToEDMStatusMapper或STATUS_MAP ，我使用STATUS_MAP
2. 使用IllegalArgumentException代替NullPointerException更好，因为它在这里是合适的。