java - 收集 Java 方法中的错误，空 ArrayList 与 LinkedList

Question

我似乎偶尔会创建这两种方法:

// 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 Commons Stringutils.join 方法。在正常情况下没有错误，列表将是空的。

所以，我开始思考这两个问题:

1. 您是否看到将错误消息字符串作为列表返回的问题？如果是这样，什么是更好的选择？ (不是异常，调用这些方法的代码会在需要时抛出异常。)

2. 对于 预期保持为空，并且当它不为空时通常应该只有顺序迭代器访问？

---

编辑:示例用例:

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

通常错误列表的处理不会包含所有这些，它只是试图说明错误列表是供人类查看的消息列表，与显示方式无关，也与/无关对于以不同方式处理不同的错误很有用。

Answer 1

我认为对字符串使用列表很好。我倾向于至少为 fetchSomthing 创建一个专用的 Result 类，并像这样，特别是如果传递的 errorMessagesOut 只是一个新的空列表:

Result result = fetchSomething(String parameter);
if (result.hasErrors()) {
    List<String> errors = result.getErrors();
} else {
    String fetched = result.getValue();
}

然后我还会将任何处理错误字符串的方法也放在这个类上，这样你就可以做类似的事情:

String errorMessage = result.getErrorString();

这将错误数据及其格式封装在一个类中，这意味着在没有错误的情况下，您不需要在 Result 实例内部创建任何列表。

我上面的观点主要是关于代码设计的。在没有分析和没有基准来与任何更改的结果进行比较的情况下尝试微优化是没有意义的。