使用大量引用例如PIXELS_PER_INCH = 72时，如何使用函数减少代码？

问题描述

| 我有一些代码需要减少，在Reddit.com上建议我使用函数来完成。但是问题在于代码主要是

public static final

\“ references \”的集合，例如

PIXELS_PER_INCH = 72

。 如果有人想以这种方式查看我的应用程序，那么我会在GitHub上开源我的应用程序。 这是向常量分配值的示例：

public static final int PICTURES_4 = 0;
public static final int PICTURES_6 = 1;
public static final int PICTURES_8 = 2;
public static final int PICTURES_10 = 3;
public static final int PICTURES_12 = 4;

public static final int PICTURES_OPTION_0 = 4;
public static final int PICTURES_OPTION_1 = 6;
public static final int PICTURES_OPTION_2 = 8;
public static final int PICTURES_OPTION_3 = 10;

这是我使用这些常量引用的示例：

switch (settings.getInt(SettingsActivity.PICTURES_OPTION,SettingsActivity.DEFAULT_PICTURES_OPTION)) {
        case SettingsActivity.PICTURES_4:
            totalImages = SettingsActivity.PICTURES_OPTION_0;
            break;
        case SettingsActivity.PICTURES_6:
            totalImages = SettingsActivity.PICTURES_OPTION_1;
            break;
        case SettingsActivity.PICTURES_8:
            totalImages = SettingsActivity.PICTURES_OPTION_2;
            break;

本质上，我想知道如何使用函数来减少代码，但仍然保持使用可读引用的能力。还是我使用可读引用的想法错误？我在在线编程课程中学习了如何将常量用作可读引用（例如PIXELS_PER_INCH = 72），而不是使用绝对值。 感谢您的时间。 更新： 我最终使用了aioobe和Lloyd Ozymandias Force的建议。我在for循环中使用了

EnumMap

。在for循环中，由于某种原因，我无法使用

myEnumMap.values()

。我用Google搜索了它，发现使用

myEnumMap.class.getEnumConstants()

的另一种方法。     

解决方法

         您可以将选项放在地图上，然后用

get(option)

代替

switch(option)

像这样初始化地图：

Map<Integer,Integer> optionsMap = new HashMap<Integer,Integer>() {{
    put(SettingsActivity.PICTURES_4,SettingsActivity.PICTURES_OPTION_0);
    put(SettingsActivity.PICTURES_6,SettingsActivity.PICTURES_OPTION_1);
    ...
}};

然后像这样更换开关：

totalImages = optionsMap.get(settings.getInt(SettingsActivity.PICTURES_OPTION,SettingsActivity.DEFAULT_PICTURES_OPTION));

或者，（因为您似乎有一个简单的整数到整数映射关系），您可以将值放在查找表中。 或者，您可以将其放在函数中，并用return替换所有中断：

public int getPictureOption() {
    switch (settings.getInt(SettingsActivity.PICTURES_OPTION,SettingsActivity.DEFAULT_PICTURES_OPTION)) {
            case SettingsActivity.PICTURES_4:
                return SettingsActivity.PICTURES_OPTION_0;

            case SettingsActivity.PICTURES_6:
                return SettingsActivity.PICTURES_OPTION_1;

            case SettingsActivity.PICTURES_8:
                return SettingsActivity.PICTURES_OPTION_2;
    }
}

然后做

totalImages = getPictureOption();

如果您认为常量可以自我解释，则可以通过在文件顶部添加静态导入来摆脱所有的

SettingsActivity.

：

import static your.package.SettingsActivity.*;

    ,        不要打开枚举值-这是aiobe提到的用于多态或Map /查找表的工作。 为了说明在Java中使用枚举的一种技术，请考虑以下带有多态getPictureOption（）方法的SettingsActivity枚举。它在枚举主体中声明为抽象，强制每个枚举常量实现该方法。高效Java 2nd Edition（Josh Bloch）中对此技术进行了更详细的描述。

enum SettingsActivity {
    OPTION_1(){
        public int getPicureOption() {
            return  myValue * 6;
        }
    },OPTION_2(){
        public int getPictureOption() {
            return Math.pow(myValue,5);
        }
    },...
    OPTION_N() {
        public int getPictureOption() {
            return myValue-6;
        }
    };

    private final int myValue;

    private SettingsActivity(int myValue) {
        this.myValue = myValue;
    }

    // enums can have abstract methods!
    public abstract int getPictureOption();
}

SettingsActivity foo = getSettingsActivityFromSomewhere();

// polymorphic getPictureOption method - we don\'t
// know which activity was clicked
int something = foo.getPictureOption();

当然，这只是示例代码，用来说明您可以执行的操作。我不确定您的switch语句的意图是什么，或者SettingsActivity枚举来自何处，所以我真的无法提供更多指导。 如果要使用Map，则JDK提供的EnumMap类为枚举提供了一种超高效的Map实现（对于常量少于65个的枚举，它作为单个long值实现，而> 64个常量）。无论哪种方式，查找操作都是O（1）（与哈希映射的O（lg n）相反）。