配列のStateにconcatしたい

Question

Reactのチュートリアル（三目並べ）を終わらせて追加課題の１つ目をやっています。

1.履歴内のそれぞれの着手の位置を (col, row) というフォーマットで表示する。

座標を表示させるために２次元配列にして座標の表示機能はなんとか実装できました。

しかし、チュートリアル終了時点ではできていたはずの過去の盤面に戻る機能が壊れてしまいました。

propのhistory(履歴)の中身を見てみると１手目〜最新手まですべて最新のsquares(盤面)になっていました。

handleClick内のsetStateがうまく出来ていないのだと思います。(74行目あたり)

sliceで複製できるのは１次元配列だけという記事も見ました。

それでもsliceしないと戻った後に別の手を実行した際の矛盾が解決できないと思います。

２次元配列を使わないようにすればできるかもしれませんが、２次元配列を使って出来るようにしたいです。

import React from 'react';
import ReactDOM from 'react-dom';
import './index.css';

function Square(props) {
    return (
        <button className="square" onClick={props.onClick}>
            {props.value}
        </button>
    );
}

class Board extends React.Component {
    renderSquare(col, row) {
        return (
            <Square
                value={this.props.squares[col][row]}
                onClick={() => this.props.onClick(col, row)}
            />
        )
    }
    render() {
        return (
            <div>
                <div className="board-row">
                    {this.renderSquare(0, 0)}
                    {this.renderSquare(0, 1)}
                    {this.renderSquare(0, 2)}
                </div>
                <div className="board-row">
                    {this.renderSquare(1, 0)}
                    {this.renderSquare(1, 1)}
                    {this.renderSquare(1, 2)}
                </div>
                <div className="board-row">
                    {this.renderSquare(2, 0)}
                    {this.renderSquare(2, 1)}
                    {this.renderSquare(2, 2)}
                </div>
            </div>
        );
    }
}

class Game extends React.Component {
    constructor(props) {
        super(props);
        var squares = new Array(3);
        for (let y = 0; y < 3; y++) {
            squares[y] = new Array(3).fill(null);
        }
        this.state = {
            history: [{
                squares: squares,
                coordinate: {
                    col: null,
                    row: null,
                }
            }],
            stepNumber: 0,
            xIsNext: true,
        }
    }
    
    handleClick(col, row) {
        const history = this.state.history.slice(0, this.state.stepNumber + 1);
        const current = history[history.length - 1];
        const squares = current.squares.slice();
        if (calculateWinner(squares) || squares[col][row]) {
            return;
        }
        squares[col][row] = this.state.xIsNext ? 'X' : 'O';

        this.setState({
            history: history.concat({
                squares: squares,
                coordinate: {
                    col: col,
                    row: row,
                }
            }),
            stepNumber: history.length,
            xIsNext: !this.state.xIsNext,
        });
    }

    jumpTo(step) {
        this.setState({
            stepNumber: step,
            xIsNext: (step % 2) === 0,
        })
    }

    render() {
        const history = this.state.history;
        const current = history[this.state.stepNumber];
        const winner = calculateWinner(current.squares);

        const moves = history.map((step, move) => {
            const desc = move ?
                'Go to move #' + move + ' (' + step.coordinate.col + ',' + step.coordinate.row + ')' :
                'Go to game start';
            return (
                <li key={move}>
                    <button onClick={() => this.jumpTo(move)}>{desc}</button>
                </li>
            );
        });

        let status;
        if (winner) {
            status = "Winner: " + winner;
        } else {
            status = "Next player: " + (this.state.xIsNext ? "X" : "O");
        }

        return (
            <div className="game">
                stepNumber:{this.state.stepNumber}
                <div className="game-board">
                    <Board
                        squares={current.squares}
                        onClick={(col, row) => this.handleClick(col, row)}
                    />
                </div>
                <div className="game-info">
                    <div>{status}</div>
                    <ol>{moves}</ol>
                </div>
            </div>
        );
    }
}
ReactDOM.render(
    <Game />,
    document.getElementById('root')
);
function calculateWinner(squares) {
    for (let x = 0; x < 3; x++) {
        if (squares[x][0] && squares[x][0] === squares[x][1] && squares[x][0] === squares[x][2]) {
            return squares[x][0];
        }
    }
    for (let y = 0; y < 3; y++) {
        if (squares[0][y] && squares[0][y] === squares[1][y] && squares[0][y] === squares[2][y]) {
            return squares[0][y];
        }
    }
    if (squares[0][0] && squares[0][0] === squares[1][1] && squares[0][0] === squares[2][2]) {
        return squares[0][0];
    }
    if (squares[0][2] && squares[0][2] === squares[1][1] && squares[0][2] === squares[2][0]) {
        return squares[0][2];
    }
    return null;
}

Answer 1

ざっとソースを見た限りで、「ここに問題あり」と思われる箇所を回答します。

squares は 3×3の二次元配列なので、ご質問にある、以下のコード

修正前：

const squares = current.squares.slice();

だと、完全な（深い）コピーを作ることになっていないです。上記を以下のように修正すれば、外側の配列の各要素である、内側の配列についてもコピーに置き換えられた squares が得られます。

修正後：

const squares = current.squares.map(ary => ary.slice());

以下は、上記2つのコードの違いを確認するサンプルです。

• 動作確認用サンプル: https://codepen.io/jun68ykt/pen/PoqxZXq?editors=0012

上記の修正のみで、アプリ全体として意図通り動作するかは分かりませんが、上記の修正により何らかの進展はあると思います。

参考までに、プログラミング一般の用語でいえば、修正前のコードで作っているコピーは浅いコピー(Shallow Copy)、修正後のコードによるコピーは深いコピー(Deep Copy)と呼ばれます。"Shallow CopyとDeep Copyの違い" などで検索すれば多数の記事がヒットします。

補足

初期状態のsquares を得るための

var squares = new Array(3);
for (let y = 0; y < 3; y++) {
  squares[y] = new Array(3).fill(null);
}

は、以下のように1行で書くこともできます。

var squares = [...Array(3)].map(_ => Array(3).fill(null));

Comments

[dyekv]

ご回答ありがとうございます。
サンプルまで用意して頂き、とても丁寧な解説で分かりやすかったです。

ご回答していただいた内容で履歴に戻れない件について無事解決できました。
Shallow CopyとDeep Copyの違いについて知らなかったので勉強になりました。

アプリ全体としては「履歴から過去の盤面に戻って別の手を実行したときに、履歴が残ったままになる」という課題があります。これも同様にhistoryをsliceでコピーしているところを工夫すれば解決できるのかなと思いますので時間が空いたらやってみます。

ありがとうございました！

[jun68ykt]

どういたしまして。
> ･･･ 無事解決できました。
とのことでよかったです????