5

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

// 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方法。并且在正常情况下没有错误，并且列表将为空。

于是，我开始思考这两个问题：

  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

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

4

4 回答 4

3

我认为对字符串使用列表很好。我倾向于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 实例内部创建任何列表。

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

于 2013-03-04T16:37:55.113 回答
1

这些错误是为了什么？您是否要根据方法返回的错误做出一些业务决策？否则，如果错误只是用于记录，一个简单的记录解决方案会建议在错误发生时立即记录。我说的是那些logger.debug("Error message");。

无论如何，你能给我们一个例子，说明这些错误返回后如何处理？

不过，我要指出一件事：如果您对处理结果和错误使用相同的数组/列表/集合，这往往会令人困惑。如果您的方法返回错误列表（或在没有错误的情况下为空/空列表）也会令人困惑，因为看起来错误是执行您的方法的结果。

于 2013-03-04T16:37:08.317 回答
1

尝试使用列表收集所有错误信息并没有错。使用 anArrayList可以正常工作，无需担心...

如果你知道没有错误，你可以return Collections.emptyList();。

一般来说，集合避免返回两者null或空列表。我总是使用空列表，因为不会破坏循环。

于 2013-03-04T16:37:28.510 回答
1

您是否看到将错误消息字符串作为列表返回的问题？如果是这样，有什么更好的选择？

不。如果列表可以解决您的问题，那么使用列表没有任何问题。

对于预计保持为空的列表，并且当它不为空时通常应该只有顺序迭代器访问，是或更好new LinkedList()？new ArrayList(0)new ArrayList()

知道列表将存储多少元素并使用该容量实例化列表不会对性能有任何显着改善：

每个 ArrayList 实例都有一个容量。容量是用于存储列表中元素的数组的大小。它总是至少与列表大小一样大。随着元素被添加到 ArrayList，它的容量会自动增长。除了添加一个元素具有恒定的摊销时间成本这一事实之外，没有指定增长策略的细节。

new ArrayList(0)但是，如果列表通常为空，则您将节省一些内存，因为列表的大小10在未指定初始容量时被初始化。

于 2013-03-04T16:39:15.623 回答