4

我正在尝试Version为我的应用程序创建一个类,该类将在加载时从清单中读取版本号,然后仅Version.MAJOR在其他地方需要它时引用例如等。但是,我在这样做时遇到了问题。这是我当前的代码:

 public class Version {

    public static final int APPCODE;
    public static final int MAJOR;
    public static final int MINOR;
    public static final char RELEASE;
    public static final int BUILD;

    static {

        try {
            Class clazz = Version.class;
            String className = clazz.getSimpleName() + ".class";
            String classPath = clazz.getResource(className).toString();
            if (classPath.startsWith("jar")) {
                String manifestPath = classPath.substring(0, classPath.lastIndexOf("!") + 1) + "/META-INF/MANIFEST.MF";
                Manifest manifest = new Manifest(new URL(manifestPath).openStream());
                Attributes attr = manifest.getMainAttributes();
                APPCODE = Integer.parseInt(attr.getValue("APPCODE"));
                MAJOR = Integer.parseInt(attr.getValue("MAJOR"));
                MINOR = Integer.parseInt(attr.getValue("MINOR"));
                RELEASE = attr.getValue("RELEASE").charAt(0);
                BUILD = Integer.parseInt(attr.getValue("BUILD"));
            }
        } catch (IOException e) {
            System.exit(9001);
        }
    }
}

它不会编译,因为static final变量可能没有被初始化(例如,如果加载了错误的清单或加载它时出现异常),我无法弄清楚执行此操作的正确过程是什么。

阅读这个问题给了我一些不使用的见解public static final。我应该使用public staticgetter 方法吗?

4

4 回答 4

8

如果您确保始终只为final字段分配一次,编译器会很高兴:

public class Version {

    public static final int APPCODE;
    public static final int MAJOR;
    public static final int MINOR;
    public static final char RELEASE;
    public static final int BUILD;

    static {
        int appcode = 0;
        int major = 0;
        int minor = 0;
        char release = 0;
        int build = 0;
        try {
            Class clazz = Version.class;
            String className = clazz.getSimpleName() + ".class";
            String classPath = clazz.getResource(className).toString();
            if (classPath.startsWith("jar")) {
                String manifestPath = classPath.substring(0,
                        classPath.lastIndexOf("!") + 1)
                        + "/META-INF/MANIFEST.MF";
                Manifest manifest = new Manifest(
                        new URL(manifestPath).openStream());
                Attributes attr = manifest.getMainAttributes();
                appcode = Integer.parseInt(attr.getValue("APPCODE"));
                major = Integer.parseInt(attr.getValue("MAJOR"));
                minor = Integer.parseInt(attr.getValue("MINOR"));
                release = attr.getValue("RELEASE").charAt(0);
                build = Integer.parseInt(attr.getValue("BUILD"));
            }
        } catch (IOException e) {
            System.exit(9001);
        }
        APPCODE = appcode;
        MAJOR = major;
        MINOR = minor;
        RELEASE = release;
        BUILD = build;
    }
}
于 2013-01-20T11:59:11.407 回答
3

我会:

  • 删除最终修饰符
  • 降低从公共到私人的可见性。
  • 为所需字段提供(静态)getter
于 2013-01-20T11:56:17.457 回答
0

如果您使用public final字段,则必须分配默认值,因为它们是常量。将可见性更改为私有并删除 final 修饰符并提供 getter/setter。这应该是解决您的问题的最佳方法。

于 2013-01-20T11:59:14.863 回答
0

您可以从 Version 类中删除处理读取清单的代码块并将其放入单独的类中 - 例如 (ManifestReader) - 并使用构造函数中的实际值直接初始化版本实例。

我会将“public static final”更改为“private final”(不是静态的),因为如果您有多个版本类的实例,则所有实例都必须有自己的应用程序代码、主要次要等!

除了提供 getter() 来访问私有 final 字段!

于 2013-01-20T12:21:52.813 回答