我似乎偶尔会创建这两种方法:
// return null on errors, and append errors to 2nd param, otherwise return result
String fetchSomething(String parameter, List<String> errorMessagesOut);
// return empty list or null on no errors, otherwise list of errors
List<String> verifySomething(String parameter);
然后调用这些的代码将使用适当的分隔符(例如简单的逗号、换行符、HTML 标记...)加入错误列表,通常使用 Apache CommonsStringutils.join
方法。并且在正常情况下没有错误,并且列表将为空。
于是,我开始思考这两个问题:
您是否看到将错误消息字符串作为列表返回的问题?如果是这样,有什么更好的选择?(不是异常,调用这些方法的代码会在需要时抛出异常。)
对于预计保持为空的列表,并且当它不为空时通常应该只有顺序迭代器访问,是或
new LinkedList()
更好?new ArrayList(0)
new ArrayList()
编辑:示例用例:
List<String> verifyParameters(JSONObject params) {
List<String> ret = new ArrayList<String>(0);
if (!verifyKey(params.get("key")))
ret.add("Invalid key: " + key);
if (!verifyAccess(params.get("user"), params.get("pass")))
ret.add("Authentication error");
return ret;
}
...
List<String> errors = verifyParameters(params);
if (!errors.isEmpty()) {
connection.sendErrorListMessage(errors);
logger.warn(StringUtils.join(errors, ", "));
controlPanel.show("Errors: \n- " + StringUtils.join(errors, "\n- ") + '\n');
throw new AbortException("invalid params); // or maybe return false/null;
}
// proceed with valid params
通常处理错误列表不会包含所有这些,它只是试图说明错误列表是供人类查看的消息列表,与它的显示方式无关,也与处理无关/无用不同的错误不同。