java - 你觉得返回日志语句怎么样

Question

我有这个 java 代码:

if(isNull(office) || isNull(pricelist)) {
    log.warn("The document {0}-{1} is not valid.",codDoc,numDoc);
    return null;
}

你觉得我改写成这样可以吗

if(isNull(office) || isNull(pricelist)) 
    return log.warn("The document {0}-{1} is not valid.",codDoc,numDoc);

这样

public void warn(String logLine, Object... args)
{...}

会变成:

public Object warn(String logLine, Object... args)
{...;return null;}

考虑到在这个系统中日志记录是强制性的。感谢您的评论。

Answer 1

我永远不会这样做，因为它会创建一个令人惊讶的 API。特别是在 Java 中，重点不在于简洁，而在于清晰。有一个额外的返回行很有值(value)，因为它告诉您返回与日志记录无关。

而且，如果您使用的方法本身有一个 void 返回值，您无论如何都必须将返回值放在单独的一行上。

无论如何，如果您声明它返回 Object，则该方法必须返回 Object，否则您必须将其强制转换，这会变得更糟。您可以使用泛型来解决这个问题:

  public <T> T warn(String message, Object... params) { return null; }

但在一般情况下，这似乎都不是一个好主意。