Java переписывание унаследованных методов

Question

Итак, я наивно написал абстрактный класс (игровая доска), содержащий поле без модификатора доступа для массива, используемого для хранения состояния игры (квадрат []). Этот класс имеет различные методы, реализованные с использованием этого Square []. У меня также есть два подкласса, наследуемых от этого класса, используя эти же методы из своего родительского класса.

Теперь, по той или иной причине, я должен сделать свой квадрат [] закрытым. Я думал, что простое исправление было бы сделать геттер, как обычно,

Class Board { 
... 
    /**An array contain all the Squares for my board. */ 
    private Square[] squares; 

    public Square[] getSquares() { 
     return squares; 
    } 
} 

class MutableBoard extends Board { 

    /** An N x N board in initial configuration. */ 
    MutableBoard(int N) { 
     _N = N; 
     squares = new Square[_N * _N]; 
     for (int i = 0; i < squares.length; i++) { 
      squares[i] = new Square(); 
     } 
    } 

    @Override 
    void copy(Board board) { 
     _N = board.size(); 
     _moves = board.numMoves(); 
     squares = new Square[board.getSquares().length]; //Errors here when used 
     for (int i = 0; i < squares.length; i++) { 
      squares[i] = board.squares[i].clone(); 
     } 
    } 
} 

class ConstantBoard extends Board { 

    /** A new ConstantBoard that allows a read-only view of BOARD. */ 
    ConstantBoard(Board board) { 
     _board = board; 
     squares = board.getSquares(); 
    } 
} 

, но я получаю NULL указателя Исключения каждый раз, когда я пытаюсь получить доступ или изменить мой массив в моих подклассах, используя наследуемые методы. Я думаю, потому что мой квадрат [] определен в моем родительском классе, он пытается получить доступ к копиям массива, а не к подклассам. Кто-нибудь знает, как легко исправить это, что не включает в себя просто поместить все мои методы в оба подкласса (и победить всю точку наследования)?

Например, в моем методе копирования я хочу иметь возможность передавать как MutableBoard, так и Constant Board, но я думаю, что он пытается каким-то образом получить доступ к копии квадратов Board, которые даже не инициализированы.

Answer 1

Вы должны вернуть Square[]

public Square[] getSquares() { 
    return squares; 
} 

squares[] имеет тип Square, поэтому метод должен возвращать Square[] не Board

Вы также должны убедиться, что у вас есть squares[] инициализируется

private Square[] squares = new Square[someSize]; 

for (int i = 0; i < suqares.length; i++){ 
    squares[i] = new Square(); 
} 

Кроме того, вы не указали _N, прежде чем пытаться инициализировать его в вас MutableBoard

_N = N; 

public class MutableBoard extends Board { 
    int _N; 

    public MutablerBoard(int N){ 
     _N = N; 
    } 
} 

-or- 

public abstract class Board { 
    int _N; 
} 

same for _board 

Кроме того, вы должны сделать square[] в вашем Board классе public static. Вы не можете изменить его с MutableBoard, если его private, и вы не можете получить доступ к «текущему состоянию» square от ConstantBoard, если это не static.

Answer 2

Не getSquares() должен возвращать квадраты?

public Square[] getSquares() { 
     return squares; 
} 

Вы заявляете, что хотите вернуть экземпляр класса Board, и вы возвращаете массив квадратов.

Answer 3

Предполагая, что вы имеете в виду:

public Square[] getSquares() { 
return squares; 
} 

Вы не инициализируется массив squares. Вам понадобятся:

Square[] squares = new Square[SIZE]; 

Даже тогда, хотя массив будет инициализирован, каждый Square в массив все равно будет нулевым. Вам нужно initialsie их тоже:

for (int i=0; i<SIZE; i++) { 
    squares[i] = new Square(); //or some other way of getting a `Square` 
} 

Выше SIZE некоторые static final int с указанием количества квадратов.

EDIT:

Ваш MutableBoard не может увидеть squares массив в Board потому, что его личные и подклассы не наследуют частные поля или методы. Код, который у вас есть, не должен компилироваться. Переменная squares в конструкторе MutableBoard составляет неsquares Поле в Board.

Предлагаю вам изменить squares как protected, чтобы подклассы могли наследовать его.