プログラマの雑談部屋 ★374 (929レス)
上
下
前
次
1-
新
797
:
d(u _ ・y) r~ 「日経平均上がっておいちいね!」 Fire済
[d(u _ ・y) r~] 10/27(月)12:02
AA×
>>772
,
776
外部リンク:ideone.com
画像リンク
画像リンク
画像リンク
[240|
320
|
480
|
600
|
100%
|
JPG
|
べ
|
レス栞
|
レス消
]
797: d(u _ ・y) r~ 「日経平均上がっておいちいね!」 Fire済 [d(u _ ・y) r~] 2025/10/27(月) 12:02:50.64 d(u _ ・y) r~ >>772,776 https://ideone.com/7V4mTT まずはGUI化とRuby化 実行環境:ruby 3.0.3 https://i.imgur.com/6UVxvM0.png 主なロジック変更 削除されたメソッド: receive_card, sort_cardlist, find_diamond3, put_diamond3, show_cardlist, put_card1, judge_card, put_free_cards, put_cards2, judge_cards2, show_cardlist, put_card1, put_free_cards https://i.imgur.com/4NjmqG7.png 作成されたメソッド: draw, clicked?, pick_playable, handle_mouse_click, play_selected, play_cards, draw_backside, set_card_positions, update, draw https://i.imgur.com/DGVEmid.png 主な既存メソッドのリファクタリング play_selected play_cards pick_playable AIコメント: 💡 まとめると、作者へのアドバイスはこんな形になります: 継承をやめて Strategy パターンで Human/CPUの行動を委譲 Game#initialize の責務分離(Deck, Player, UIに分ける) Player のカード操作や出すロジックを簡潔に整理 Human/CPU 共通ロジックを抽出して重複を減らす http://medaka.5ch.net/test/read.cgi/prog/1760328215/797
まずは化と化 実行環境 主なロジック変更 削除されたメソッド 作成されたメソッド 主な既存メソッドのリファクタリング コメント まとめると作者へのアドバイスはこんな形になります 継承をやめて パターンで の行動を委譲 の責務分離 に分ける のカード操作や出すロジックを簡潔に整理 共通ロジックを抽出して重複を減らす
上
下
前
次
1-
新
書
関
写
板
覧
索
設
栞
歴
あと 132 レスあります
スレ情報
赤レス抽出
画像レス抽出
歴の未読スレ
ぬこの手
ぬこTOP
0.031s