C＃/ Unity-需要幫助調試我的Chess程序-在同一類的另一個實例中創建類的實例時遇到麻煩

Question

我正在創建一個國際象棋程序。 我決定這樣做，以便在選擇一塊時顯示所有可能的合法動作。 為了使它起作用，我必須有一種機制來預測舉動的結果（例如，因為您不能采取法律行動來阻止自己的國王）。 當我開始與計算機對手打交道時，這也將非常有用，因為AI必須預先能夠看到多個轉彎才能做出明智的決定。

為此，我做到了這一點，以便在每個回合開始時，為玩家的棋子計算所有可能的動作，然后模擬這些可能的動作中的每一個，以查看是否使國王處於控制狀態。 如果是這樣，該選項將被刪除。 我有一個名為BoardManager的主類，它控制視覺效果並包含實際的棋盤布局（在ChessBoardSim中），還有ChessBoardSim類，其對象每個都包含一個可能的棋盤狀態。 ChessBoardSim可以創建ChessBoardSim的其他實例，從而可以預先模擬板的狀態任意數量的匝數。

我有一個已經苦苦掙扎超過5個小時的錯誤，該錯誤是在我嘗試重組許多代碼之后出現的。 我已經快要死了，我只需要一雙新鮮的眼睛。 問題在於，移動時不會將其從原來的位置上移開，並且板上某些部件的位置似乎向上方移動了兩個磁貼。 我相信，基於調試，問題出在ChessBoardSim類的CalculateAllMovementOptions（）中，由於某種原因，當ChessBoardSim具有ChessBoardSim子級，而CalculateAllMovementOptions（）函數在子級中被調用時，它將更改父母。 我在下面提供了完整的代碼，我將盡力描述特定的問題。

問題的流程是：

1：在BoardManager中調用private void Start（）。

2：程序正確繪制並填充了電路板。 創建一個ChessBoardSim（稱為ChessPieces）以包含當前板狀態的數據。

3：在BoardManager中調用StartTurn（）以開始游戲。

4：StartTurn（）調用ChessPieces.CalculateAllMovementOptions（）。 CalculateAllMovementOptions（）的預期功能是獲取列表數組。 數組中的每個列表都包含玩家棋子之一的合法移動。 但是，這實際上似乎正在更改實際的電路板數據，我不知道為什么。

5. 當玩家嘗試移動一個棋子時，游戲會中斷，因為棋子不在應有的位置，軟件也無法應對。 被移動的片段不會將其自身移出其舊位置（盡管我已盡力而為），這導致該對象存在兩個實例，當一段代碼嘗試訪問第17個元素時，導致indexoutofrange異常具有16個元素的數組。

如果有人能幫助我，我將不勝感激，我覺得我已經在此上浪費了整整一天，而且我確定自己缺少一些簡單的東西。

我的代碼的鏈接是https://github.com/FC123321/Chess

Answer 1

Array.Clone不創建一個數組的深層副本，所以當你調用boardLayout.Clone()在SimulateBoard （在又一次ChessBoardSim構造函數），您要復制的引用boardLayout到新陣列。 這意味着新數組中的片段與舊數組中的對象相同。

這意味着，當您在SimulateBoard中的MovePiece中並調用piece.SetPosition等該復制數組的成員時， piece.SetPosition在復制源中設置這些塊的位置。

而不是使用boardLayout.Clone() ，您需要遍歷整個源數組，並執行newBoardLayout[x,y] = new ChessPiece(); 然后將值復制到新的ChessPiece 。 或者，您可以創建一個新的ChessPiece構造函數，該構造函數將使用另一塊棋子並將其值復制到其中：

// ChessPiece copy constructor
public ChessPiece(ChessPiece other) {
    this.Position = new int[2] { other.Position[0], other.Position[1]};
    this.isWhite = other.isWhite;
    this.movementType = other.movementType;
    this.hasMoved = other.hasMoved;
    this.turnDoubleMoved = other.turnDobleMoved;
}

// instead of boardLayout.Clone() in the ChessBoardSim constructor:

for (int x=0 ; x<8;x++){
    for(int y=0; y<8;y++){
        if (boardLayout[x,y] != null)
            this.boardLayout[x,y] = new ChessPiece(boardLayout[x,y]);
    }
}


// In SimulateBoard, take out the redundant Clone call
ChessBoardSim simBoard = new ChessBoardSim(boardLayout, turnNumber);