说服我使用异常进行用户验证我错了

Question

虽然many people say you should not use exceptions to handle bad user input，但很难找到共识。不过，我不相信在我的具体情况下这样做是坏事。你能解释一下我为什么错了吗？

我的代码（请只关注异常处理方面）如下。我在这里使用异常的理由是，如果我不这样做，假设我想让验证逻辑接近关键字解析（因为解析和验证是紧密耦合的），我将不得不更改三个方法（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

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

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