收集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. 是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

通常错误列表的处理不会包含所有这些,它只是试图说明错误列表是一个供人类查看的消息列表,与其显示方式无关,也与处理无关/有用不同的错误不同.

Answer 1

我认为使用字符串列表很好。我倾向于Result至少创建一个专门的类fetchSomthing并使其像这样，特别是如果errorMessagesOut传递的除了一个新的空列表之外什么都不是：

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

然后，我还将处理错误字符串的任何方法也放在此类上，这样您就可以执行以下操作：

String errorMessage = result.getErrorString();

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

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