экземпляр объекта, не создающий экземпляр

Question

Я пытаюсь создать копию моего объекта Board, но, похоже, это не экземпляр. Может кто-нибудь сказать мне, почему?

class Board { 
     private Board _copyBoard; 
     private int _size; 
     public Board (int N) { 
      _size = N; 
     } 
     public Board (Board b) { 
      this._copyBoard = b; 
     } 
     int size() { 
      return _size; 
     } 
    } 

Когда я делаю:

Board b = new Board(4); 
    Board x = new Board(b); 
    System.out.println(x.size()); --> 0 instead of 4. 

Answer 1

Board b = new Board(4); 

Это создаст новую доску, где его поле _size = 4.

Board x = new Board(b); 

Это создаст новую доску, где его поле _copyBoard = b.

System.out.println(x.size()); 

Это выведет на поле _size второй доске. Нигде вы не связаны с переменной _size первой платы.

Вы можете решить эту проблему, делая

public Board (Board b) { 
    this._copyBoard = b; 
    } 

    int size() { 
    if(_copyBoard != null) { 
     return _copyBoard.size(); 
    } else { 
     return _size; 
    } 
    } 

все обстоятельства, вы, вероятно, глядя на реальной копии данных, а не эти классы как поля с большим количеством если заявления.

class Board { 
    private int _size; 
    public Board (int N) { 
     _size = N; 
    } 
    public Board (Board b) { 
     _size = b.size(); 
    } 
    int size() { 
     return _size; 
    } 
} 

Answer 2

Copy конструктор должен нормально скопировать переменные экземпляра и при необходимости некоторые глубинное государство - не сохранить ссылку на исходный объект.

Ваши методы получения & должны иметь возможность управлять собственным состоянием объекта, в его собственных переменных. Имея какую-то структуру двойного пути во всем мире, с if утверждениями, которые будут , либо ваши собственные переменные или оригинальный экземпляр, из которого он был скопирован, плохой дизайн и очень сложный/неэффективный.

public class Board { 
    protected int  size; 
    protected Cell[][] grid; 
    public Board (int N) { 
     this.size = N; 
     // for example, create a 2D array. 
     this.grid = new Cell[size][size]; 
    } 
    public Board (Board orig) { 
     this.size = orig.size(); 
     // deep-copy the grid. 
     this.grid = new Cell[size][]; 
     for (int i = 0; i < size; i++) { 
      grid[i] = Arrays.copyOf(orig.grid[i], size); 
     } 
    } 
    public int size() {return size;} 
} 

я сильно предпочитаю protected над частным, для большей универсальности & технического доступа.

Я также не использую префиксы полей, но вместо этого устраняю назначения полей (в сеттерах или других методах) с this.. Этот подход очень хорошо подходит для простых полей.

Для коллекций я суффикс имен полей со списком или картой или что-то еще; параметры называются без суффикса. Это делает множественные операции & метод-вызовы с множественными значениями четкими. Например, add (List<Customer> customers) добавит их в поле customerList.

Answer 3

попробовать

Board b = new Board(4); 
Board x = new Board(b); 
System.out.println(x._size); 

class Board { 
    public Board _copyBoard; 
    public int _size; 
    public Board (int N) { 
     this._size = N; 
    } 
    public Board (Board b) { 
     this._size = b._size; 
    } 
    int size() { 
     return _size; 
    } 
} 

В вы код вы не копируете значение размера х, вместо размера становится копируется в переменную экземпляра внутри X

Если вы переменную общественность, в вашем затем строка ниже будет печатать скопированное значение в вашем коде

System.out.println(x._copyBoard._size);