我继承了一个应用程序,它使用java属性文件来定义配置参数,例如数据库名称.
有一个名为MyAppProps的类看起来像这样:
public class MyAppProps {
protected static final String PROP_FILENAME = "myapp.properties";
protected static Properties myAppProps = null;
public static final String DATABASE_NAME = "database_name";
public static final String DATABASE_USER = "database_user";
// etc...
protected static void init() throws MyAppException {
try {
Classloader loader = MyAppException.class.getClassLoader();
InputStream is = loader.getResourceAsStream(PROP_FILENAME);
myAppProps = new Properties();
myAppProps.load(is);
} catch (Exception e) {
threw new MyAppException(e.getMessage());
}
}
protected static String getProperty(String name) throws MyAppException {
if (props==null) {
throw new MyAppException("Properties was not initialized properly.");
}
return props.getProperty(name);
}
}
Run Code Online (Sandbox Code Playgroud)
需要获取属性值的其他类包含以下代码:
String dbname = MyAppProps.getProperty(MyAppProps.DATABASE_NAME);
Run Code Online (Sandbox Code Playgroud)
当然,在第一次调用MyAppProps.getProperty之前,MyAppProps需要像这样初始化:
MyAppProps.init();
Run Code Online (Sandbox Code Playgroud)
我不喜欢init()需要调用的事实.初始化不应该发生在静态初始化块或私有构造函数中吗?
除此之外,代码似乎有些其他问题,我无法完全理解它.属性实例通常包含在自定义类中吗?这里有什么别的错吗?
如果我像这样制作自己的包装类; 我总是喜欢为值创建强类型的getter,而不是通过静态final变量公开所有内部工作.
private static final String DATABASE_NAME = "database_name"
private static final String DATABASE_USER = "database_user"
public String getDatabaseName(){
return getProperty(MyAppProps.DATABASE_NAME);
}
public String getDatabaseUser(){
return getProperty(MyAppProps.DATABASE_USER);
}
Run Code Online (Sandbox Code Playgroud)
静态初始化器看起来像这样;
static {
init();
}
Run Code Online (Sandbox Code Playgroud)
话虽如此,我很乐意说我不是静态初始化器的忠实粉丝.
您可以考虑查看依赖注入(DI)框架,如spring或guice,这些将允许您将适当的值直接注入需要使用它们的位置,而不是通过附加类的间接方式.很多人发现使用这些框架可以减少对这种管道代码的关注 - 但只有在你完成了框架的学习曲线之后.(DI框架很快学会,但需要花费很长时间才能掌握,所以这可能比你真正想要的更重要)
| 归档时间: |
|
| 查看次数: |
10554 次 |
| 最近记录: |