JSリファクタ+細かな改善

[tomii9273]

WHY

一旦コードを整理したい。(#9 の続き) それによって容易に追加可能になった機能を追加したい。

WHAT

• リファクタリング
  • [x] 変数名の変更
  • 定数は大文字に
  • 1,2文字変数の変更
  • [x] クリック時の処理 (ずらっと書いてあるもの) それぞれを関数に切り出す
• 改善
  • [x] 番号を指定して盤面を作成した際も、番号と最大得点を (更新して) 表示
  • [x] 自分が石を置いた (石を取る処理もした) 直後の盤面を一瞬表示 (相手がコウで置けない地点も赤で示す)
  • [x] ログにインデントを入れて見やすくする
  • [x] 「0～最大値」以外の無効な値をフォームに入力して作成ボタンを押した場合はランダムに選択
  • 詳細は下記
  • SQLインジェクション対策にもなるかも
  • [x] 終局時のログの得点表記を変える・最大得点達成時にそのことを表示
  • 引き分け → 0点、n点負け → -n点、のように、最大得点表示と揃えた
  • 最大得点超え (現在、2手打ちなどのバグでしかならないはず) も表示

ToDo

PRが大きくなってきたので以下は別に行う

• TypeScript導入
• 盤面などを含む状態はクラスで持つ (オブジェクト指向)
  • 文字列にsubstrを使わないようにする

[tomii9273]

盤面番号フォームのテスト

• 0, 12, 5025 → そのまま有効
• 空欄, 1-2, 1+2, 9を910個並べたもの → 空文字列扱いになり、0番が選ばれた
• -12, 5026, 1.2, 9を91個並べたもの → 不正となりランダム選択

問題なさそう → 追加: ddac65d

参考

• https://chatgpt.com/c/943a73fc-c629-40fe-bf12-76f625ecc498
• https://webukatu.com/wordpress/blog/5856/
  • 「テーブルで見る、JavaScript数値チェック」