数组的setter方法

Question

我有一个复习课，用于在另一个名为游戏信息的课中创建一个数组。

我创建了一个 setter 方法，用于向 gameInfo 的评论数组添加评论。我有 addReview 函数来搜索这个游戏评论，如果有一个空的评论槽，然后将评论输入添加到评论数组中。我只是想确保 addReview 方法的逻辑正确。

class Review {
    //review class variables
    public String reviewText;
    public int numberOfStars;

    //review class constructor
    public Review(String reviewText, int numberOfStars) {
        this.reviewText=reviewText;
        this.numberOfStars=numberOfStars;
    }
}

class GameInfo {
    //game info class variables
    private String title;
    private Review[] reviews = new Review[10];

    //game info class constructor
    public GameInfo(String title, Review[] reviews) {
        this.title=title;
        this.reviews = reviews;
    }

    //setter to add single review to reviews[]
    public void addReview(Review r) {
        int i;
        for(i = 0; i < this.reviews.length; i++)    {
            if(this.reviews[i] == null) {
                this.reviews[i].reviewText = r.reviewText;
                this.reviews[i].numberOfStars = r.numberOfStars;
                break;
            }
        }
    }
}

Answer 1

我只是想确保 addReview 方法的逻辑正确。

不，两个问题：

1. 您让它寻找null，但随后尝试分配给null 上的字段：

   if(this.reviews[i] == null) {
       this.reviews[i].reviewText = r.reviewText; 
       this.reviews[i].numberOfStars = r.numberOfStars;
       break;
   }
   

   ...这将引发 NPE，因为您试图在 null 上设置属性 reviewText 的值。您需要创建一个Review（或使用传入的那个，具体取决于 API 设计及其是否可变）。

   如果您的 Review 是不可变的（一旦创建就无法更改），但事实并非如此，我们可以安全地使用传入的实例：

   if(this.reviews[i] == null) {
       this.reviews[i] = r;
       break;
   }
   

   但由于它是可变的（可以更改），我们需要创建自己的（大概）：

   if(this.reviews[i] == null) {
       this.reviews[i] = new Review(r.reviewText, r.numberOfStars);
       break;
   }
   

   在那里，我将离散参数传递给 Review 构造函数；我宁愿向Review 添加一个复制构造函数，然后这样做：

   if(this.reviews[i] == null) {
       this.reviews[i] = new Review(r);
       break;
   }
   

2. addReview 如果阵列已满，则会静默失败。您会想要大声失败:-) 或者创建一个更大的新数组，将旧数组复制到其中，然后添加到末尾。

   数组在这里可能不是理想的选择，您可能希望使用List<Review> 代替（您可以使用ArrayList<Review> 或LinkedList<Review> 进行初始化，任何适合的）。它会简单得多。但我不知道设计限制。

---

旁注：您的Review 具有公共字段。通常在 Java 中被认为不是最佳实践；考虑改用私有字段和访问器方法。