[過去ﾛｸﾞ] プログラマの雑談部屋 ★374 (1002ﾚｽ)

プログラマの雑談部屋 ★374 http://medaka.5ch.net/test/read.cgi/prog/1760328215/

上下前次1-新
通常表示 512ﾊﾞｲﾄ分割 ﾚｽ栞

---

このｽﾚｯﾄﾞは過去ﾛｸﾞ倉庫に格納されています｡
次ｽﾚ検索 歴削→次ｽﾚ 栞削→次ｽﾚ 過去ﾛｸﾞﾒﾆｭｰ

---

798: d(u _ -y) r~ [] 2025/10/27(月) 12:18:43.46 まずオブジェクト指向にこだわってるのか意味のないメソッドを作ってる @cardlist.sort_by辺りの処理 それからこの程度の規模のゲームでGameクラスはいらない。そこで多くを分担するんじゃなく、Playerやストラテジークラスに正しく責務の分担ができる 上で言ってるやつもいるけどGameStateな リファクタリング前に特に気になったのは、場が空だったらプレイヤーが場にカードを出せるっていうplay_selected関数 if 場が空であれば、then play_selectedを呼び出す。　だ 場が空であるかどうかまでplay_selectedの中で判定してるのは責務分離できてない なのでそこは改造した 最も次にやるべきなのはGameクラスを捨てること、そこのinitializeが闇鍋責務になってるから分離すること Game.start();くらいは残してもいいかもしれんが、Gameクラスなんて名前のところにロジックを書くべきじゃない せいぜい許されるのは初期化 config , start くらい カードを配る処理ってのはストラテジー（GameState）に持っていけ case phase ; when: card_init ... when: play中 ... when: ゲーム終了 ... でフェーズ作るのでもなんでもいい。 このソースにはStateの概念がない http://medaka.5ch.net/test/read.cgi/prog/1760328215/798

---

ﾒﾓ帳

(0/65535文字)

---

上下前次1-新書関写板覧索設栞歴

---

あと 204 ﾚｽあります
ｽﾚ情報 赤ﾚｽ抽出 画像ﾚｽ抽出 歴の未読ｽﾚ

---

Google検索

Wikipedia

---

ぬこの手 ぬこTOP 0.013s