javascript - 具有多个返回值的代码是否违反了 DRY 原则？

Question

这段代码是否违反了DRY原则？

if (notAuthorized) {
    return sendErrorCode(new ForbiddenException())
} else if (notValid) {
    return sendErrorCode(new InvalidArgumentException())
} else if (outDated) {
    return sendErrorCode(new Error())
} else {
    return sendResult(result)
}

我的意思是所有这些带有 sendErrorCode 的行都是错误处理代码。
现在，我只是在发生错误时发送错误代码，但是如果我需要记录错误或向分析或其他内容发送请求，我需要编辑三行代码怎么办。
也许我应该将 sendErrorCode 包装在更通用的函数中，例如handleError

Answer 1

是的，这违反了不要重复自己原则，因为在多个地方都包含对错误处理程序的调用。虽然它仍然是一个需要修改的地方，但最好只修改一个地方，并让每个 block 都存在单个关注点。因此 sendErrorCode 始终就是这样。它也不是一个隐藏的错误记录器。

为了更接近 DRY，您实际上应该将决策处理逻辑包装到新函数或 block 中，然后决定是否要稍后发送错误代码。

这里我选择了一个函数，它接受一个可能的错误库和一个可能的结果。如果发现错误，则将其返回。否则返回结果。

const isThereAProblem = ({ notAuthorized, notValid, outDated }, result) => {
  if (notAuthorized) return new ForbiddenException();
  if (notValid) return new InvalidArgumentException();
  if (outDated) return new Error();
  return result;
}

您可以使用分支逻辑轻松地做到这一点，但您明白了。设置一次，处理一次。

稍后在代码中

const result = isThereAProblem(possibleErrors, possibleResult);

if (result instanceof Error) return sendErrorCode(result);

return sendResult(result)

现在您只需修改一个区域。只需确保您的异常是 Error 的扩展，您就可以使用 instanceof 运算符对其进行检查。要DRY，您必须使用该语言提供给您的工具。

您应该将在多个位置调用的任何代码或函数提取到它们自己的区域中，以便在需求发生变化时可以轻松对其进行修改。通过将错误检查移至单个位置，从而删除对其自身区域的任何特殊处理，然后仅检查通用情况，您将获得更具适应性的代码。现在，如果您想在发回错误之前执行某些操作，只需在处理程序中展开 if block 即可:

if (result instanceof Error) {
  console.error(result);
  return sendErrorCode(result);
}

一处需要修改。需要针对不同类型的错误使用更专门的逻辑吗？将其放入 isThereAProblem 函数

const isThereAProblem = ({ notAuthorized, notValid, outDated }, result) => {
  if (notAuthorized) {...}
  if (notValid) {...}
  if (outDated) {...}
  return result;
}

无论哪种方式，现在您都可以轻松地将任何可能的错误的处理放到更有意义的地方。要么在哪里识别错误，要么在哪里处理错误。