Не удается найти символ в цикле

Question

Я получаю сообщение об ошибке, что BookCollection.java:67: ошибка: несовместимые типы collection [lastElement ++] = b;

Также не уверен, что мой конструктор настроен правильно? Направления были:

Конструктор: Учитывая параметр, определяющий ограничение на размер коллекции, создается пустая коллекция книг с использованием заданного параметра. Параметр не должен превышать заданный максимальный размер 200.

Я правильно инициализирую свои переменные? Ответ ниже помог мне изменить мой код, но, хотя я не получаю ошибок в моем конструкторе, я чувствую, что это может быть неправильно в соответствии с указаниями.

Я вставлю пару кусков моего код, относящийся к вопросу.

public class BookCollection{ 

    //data fields, need complete 
    private int limit = 200; 
    //Array of type book 
    private int Book[]; 

    //actual size of collection, initialized to zero. Must never exceed limit 
    private int collection[];  

    private int lastElement; 

    //Constructor 
    public BookCollection(int l, int c[], int le,int b[]){ 
     Book = b; 
     collection = c; 
     limit = l; 
     lastElement = le; 
     int lastElement = 0; 
     if(limit <= 200){ 
      Book[] collection = new Book[limit]; 
     } else{ 
      throw new UnsupportedOperationException("CannotExceedLimit"); 
      } 
     } 

ANNDDDD где я получаю сообщение об ошибке:

public void addBook(int b[], int c[]) { 
     Book = b; 
     collection = c; 
     if (lastElement == collection.length) { 
      throw new UnsupportedOperationException("CorrectionFull"); 
     } 
     for (int i = 0 ; i != lastElement ; i++) { 
      if(b.equals(collection[i])) { 
      throw new UnsupportedOperationException("DuplicateBook"); 
      } 
     } 
     collection[lastElement++] = b; 
    } 

Answer 1

Вы не объявляли i как целое число в вашей for цикле. Поэтому добавьте объявление с инициализацией. Заменить этот

for(i=0; i<collection.length; i++){ 

с

for(int i=0; i<collection.length; i++){ 

Answer 2

Вы did't объявили i как переменную в int типа, сделать это как

for(int i=0; i<collection.length; i++){ 
    ^here 
    //... 
} 

Answer 3

Это утверждение

BookCollection[] collection = new BookCollection[limit]; //initialize array of 200  

объявляет местным массивом. Он уничтожается, как только вы покидаете конструктор.

Коллекция, которая остается вокруг этого один:

private int collection[]; 

Он состоит из int с, так что, когда вы пытаетесь сделать это

collection[i].add(b);  

компилятор правильно жалуется, что int не имеет метод называется add.

Хорошие шансы, даже объявляя коллекцию как

private Book[] collection; 

и его инициализации в конструкторе, как

collection = new Book[limit]; 

не собирается помочь, хотя: в отличие от коллекций, Java массивы не позволяют вы динамически меняете свой размер, поэтому вам нужно сохранить индекс последнего элемента массива collection[], который был установлен.

Это приводит к пониманию того, что вам нужно цикл для поиска дубликатов, и отмечая еще: определить элемент int lastElement, установите его равным нулю в конструкторе, и переписать метод addBook следующим образом:

public void addBook(Book b) { 
    if (lastElement == collection.length) { 
     throw new UnsupportedOperationException("CorrectionFull"); 
    } 
    for (int i = 0 ; i != lastElement ; i++) { 
     if(b.equals(collection[i])) { 
      throw new UnsupportedOperationException("DuplicateBook"); 
     } 
    } 
    collection[lastElement++] = b; 
}