如何在公共方法中使用私有方法来检查有效性？

Question

    private boolean isValid(int aRating)
{                     

  return aRating >= 1 && aRating <= 10;

}

                or

    private boolean isValid(int aRating)
{                     

  if (aRating >=1 && aRating <=100
      return true;
  else
      return false

}

我现在需要编写一个方法 setRating(aRating)，如果评级有效，则将评级设置为 aRating。所以我假设我需要在公共方法中使用上面的方法来检查它是否有效。如果它有效，我需要将 rating = 设置为 aRating。到目前为止，我的想法是这样的：

    public void setRating(int aRating)
{
   if (isValid() == true)
       rating = aRating;

}

但我不能使用 isValid 作为 true 的 == 因为它是一种方法。我也尝试使用 isValid();为了尝试使用该方法，但由于顶部的 (int aRating) ，它不允许。如果我没有标识符，那么它根本不允许我使用 aRating...

现在

public void setRating(int aRating)
     {
       if (isValid(aRating))
          rating = aRating;
     }

我需要创建一个 setRating() 方法，允许用户从键盘输入某些内容，并且它必须是有效的，我不断收到错误，因为它说我不能重载 SetRating(int aRating) 两次，我明白。但是如果我尝试取出 int 部分，它是无效的，因为它需要一个参数。这就是我所拥有的

    public void setRating()
{
    Scanner keyboard = new Scanner(System.in);
  if (isValid(aRating))
      rating = keyboard.nextInt();
}

Answer 1

public void setRating(int aRating)
{
   if (isValidRating(aRating))
       this.rating = aRating;
}

您的作业可能不需要这样做，但在现实世界中，您可能还需要考虑在验证失败时抛出 IllegalArgumentException。

例子：

public void setRating(int aRating)
{
    if (isValidRating(aRating))
        this.rating = aRating;
    else 
        throw new IllegalArgumentException("Invalid rating.");
}

Answer 2

您还必须将 aRating 参数传递给 isValid 方法。与其跳过错误值的赋值，通常最好抛出异常，如下所示：

public void setRating(int rating) {
  checkRating(rating);
  this.rating = rating;
}

private void checkRating(int rating) {
  if (!(1 <= rating && rating <= 10)) {
    throw new IllegalArgumentException("Invalid rating: " + rating);
  }
}

这种类型的参数检查由 java.util.ArrayList 等流行类使用，例如get(int) 方法。