说服我，我在使用异常进行用户验证方面是错误的

Question

尽管很多人说你不应该使用异常来处理错误的用户输入。尽管如此，我并不认为在我的具体情况下这样做是坏事。你能试着解释一下为什么我错了吗？

我的代码（请仅关注异常处理方面）如下。我在这里使用异常的理由是，如果我不这样做，假设我希望验证逻辑接近关键字解析（因为解析和验证紧密耦合），我将不得不更改三个方法（submitOnAdd、submitOnUpdate、 getKeywords）让他们处理这种特殊情况。您认为我在这种情况下使用异常肯定是错误的，还是个人风格的问题？

public SubmitResponse internalSubmit(Map<String, String[]> submitParameters) {
  try {
      if (!submitParameters.containsKey("foo")) {
        return submitOnAdd(submitParameters);
      } else {
        return submitOnModify(submitParameters);
      }
  } catch (SubmitErrorException e) {
      return SubmitResponse.fieldError(Arrays.asList(e.getSubmitError()));
  }
}

SubmitResponse submitOnAdd(Map<String, String[]> submitParamters) {
  // do some stuff
  // ...
  if (addKeywordList(createKeywordList(submitParameters.get("concatenated_keywords"))
    return SubmitResponse.OK();
  return SubmitResponse.bad("Failed to add");
}

SubmitResponse submitOnUpdate(Map<String, String[]> submitParamters) {
  // do some other stuff
  // ...
  if (updateKeywordList(createKeywordList(submitParameters.get("concatenated_keywords"))
    return SubmitResponse.OK();
  return SubmitResponse.bad("Failed to update");
}

List<Keyword> getKeywords(String concatenatedKeywords) {
  List<String> rawKeywords = splitKeywords(concatenatedKeywords);
  return Collections.transform(new Function<String, Keyword>() {
      @Override
      public KeywordListProto.Keyword apply(String expression) {
        return buildKeyword(expression);
      }
    });
}

private Keyword buildKeyword(String rawKeyword) {
  // parse the raw keyword
  if (/*parsing failed */)
    throw new SubmitResponseException("Failed to parse keyword " + rawKeyword);

  return parsedKeyword;
}

Answer 1

我不能说我永远不会建议在输入验证中的某个地方使用异常。但在这种情况下，我想说这会增加很多混乱。我会：

• 添加一个单独的方法来处理验证。 （可能必须在多个地方调用此方法，这是一个缺点，但它可能会使代码更容易理解）。
• 在更理想的情况下，我会在更接近用户输入的地方进行验证，并且不允许提交无效数据。 （一个可能的负面影响是验证和解析逻辑的分离，但如果您可以以某种方式使用同一个类来完成这两个任务，则可以避免这种情况）。