java - 当委托(delegate)方法具有相同的保护时，如何处理保护子句并引发异常？

Question

这个问题只是想问如何处理调用对象的方法，该方法检查与调用方法相同的条件并抛出相同的异常类型。由于问题的性质，它变成了类设计的问题。

我正在尝试国际象棋程序的类设计。我最初创建了一个 Chessboard 类，其中包含允许您放置棋子的方法。原因是:

1. 它是不可变的，我认为重复创建新的棋盘对象只是为了初始化它似乎没有必要。
2. 理想情况下，棋盘应填充棋子一次，并且应通过棋盘的界面对棋盘进行进一步更改。

我创建了一个可变的 PieceSet 类，该类在 Chessboard 构造函数中传递。

// Showing as interface instead of class, because implementation is not important
public interface PieceSet
{   
    void putBishop(Square square, Color color) throws UndefinedSquareException;

    void putKing(Square square, Color color) throws UndefinedSquareException;

    void putKnight(Square square, Color color) throws UndefinedSquareException;

    void putPawn(Square square, Color color) throws UndefinedSquareException;

    void putQueen(Square square, Color color) throws UndefinedSquareException;

    void putRook(Square square, Color color) throws UndefinedSquareException;

    void removePiece(Square square);

    void relocate(Square fromSquare, Square toSquare) throws UndefinedSquareException;
}

在 PieceSet 的实现中，方法将使用守卫来检查空方 block

if(null == square)
    throw new UndefinedSquareException(...);

Chessboard 类将使用 PieceSet 类，例如，如下所示:

public final class Chessboard
{
    private PieceSet pieces;

    public Chessboard()
    {
        this.pieces = new PieceSet();
    }

    public Chessboard(PieceSet initialPieces)
    {
        this.pieces = new PieceSet(initialPieces);
    }

    public Chessboard removePiece(Square square) throws UndefinedSquareException
    {
        /*
         * Check for null here and throw exception??
         * PieceSet does the same thing.
         */

        PieceSet newPieces = new PieceSet(this.pieces);

        newPieces.removePiece(square);

        return new Chessboard(newPieces);
    }

    public Chessboard movePiece(Square fromSquare, Square toSquare)  throws UndefinedSquareException
    {
        /*
         * Check for null here and throw exception??
         * PieceSet does the same thing.
         */
        PieceSet newPieces = new PieceSet(this.pieces);

        newPieces.relocate(fromSquare, toSquare);

        return new Chessboard(newPieces);
    }
}

那么问题是，由于 Chessboard 也接收 Square 类型参数，并且它们不应该为 null，我是否也将守卫放入Chessboard 的 removePiece 和 movePiece 方法？

if(null == square)
    throw new UndefinedSquareException(...);

另一个想法

我的另一个想法是将 PieceSet 建模为 Chessboard 的构建器(尽管 PieceSet 可能不再是一个好名字了，我不会更改示例的名称)。我想既然 PieceSet 的最初想法是用它来创建一个填充的 Chessboard，那么我会让它成为 PieceSet 的责任来执行此操作，而不是将其传递给Chessboard。

public interface PieceSet
{   
    putBishop(Square square, Color color) throws UndefinedSquareException;

    putKing(Square square, Color color) throws UndefinedSquareException;

    // Same from above, etc. EXCEPT no remove or relocate methods

    // This is new
    Chessboard createBoard();

}

// The Chessboard doesn't have to use PieceSet. It can be implemented using any data structure to store pieces.
public final class Chessboard
{
    private Map<Square, Piece>pieces;

    public Chessboard(Map<Square, Piece> initialPieces)
    {
        this.pieces = new HashMap<Square, Piece>(initialPieces);
    }

    public Chessboard removePiece(Square square)
    {
        // No more duplication of checking for null square, and throwing exception
    }

    public Chessboard movePiece(Square fromSquare, Square toSquare)
    {
        // No more duplication of checking for null square, and throwing exception
    }
}

Answer 1

一般来说，假设我们强烈认为接口(interface)应该抛出已检查的异常。此外，我们想要装饰一个策略对象，提供该接口(interface)的实现。最后，我们认为接口(interface)中的检查异常适用于装饰器的调用者。在这种情况下，如果我们的委托(delegate)抛出的异常需要传递给装饰器的调用者，那么是的，您将在装饰器中抛出相同的异常。

但具体到棋盘问题:我不确定我是否同意 UndefinedSquareException 甚至应该是一个受检查的异常。为什么此 API 的用户会提供对空 Square 或不受此棋盘管理的 Square 的引用，并期望恢复？因此，我倾向于认为 UndefinedSquareException 扩展了 RuntimeException 并且根本不需要出现在您的签名中，或者 Chessboard 从 PieceSet 捕获已检查的异常并在其位置抛出 RuntimeException。

希望这会有所帮助 - 喜欢阅读您的问题并思考您的问题。