重构里面说过不要写magic number(魔法数字),个人觉得应该顺便扩展一下,把magic string也添加上,因为系统总是在变化,有些字符串也是在变更的。
比如
比如
String dsName = System.getProperty("UserDataSource");
if (dsName == null)
dsName = "design";
abc.setUserDataSource(dsName);
dsName = System.getProperty("DefaultDataSource");
if (dsName == null)
dsName = "design";
abc.setDefaultDataSource(dsName);
if (dsName == null)
dsName = "design";
abc.setUserDataSource(dsName);
dsName = System.getProperty("DefaultDataSource");
if (dsName == null)
dsName = "design";
abc.setDefaultDataSource(dsName);
这个里面用到了两次design这个字符串,也就是默认的DataSource,如果这个字符串不是在一个类里面使用,并且今后DataSource,要改名字的时候,就需要全局去查找design这个字符串,到时候的工作量可能比较大,个人觉得是不是应该把这些使用的字符串定义成一个常量的类,或者接口。这样今后改动的时候就比较方便。