<!DOCTYPE HTML>
<html lang="ja">
<head>
<title>課題B2</title>
<meta charset="UTF-8">
<link rel = "stylesheet" type = "text/css" href = "../prettify.css">
</head>
<style>
.image_size{
width: 800px;
height: 550px;
}
</style>
<body>
<h1>課題B2</h1>
<hr size="5">
<h2>課題の概要</h2>
<p>
課題B2では，Algolike の HandsA から LoseA へのデータ転送について見ていきます．
HandsA は，プレイヤーのすべての手札を管理しているクラスで，LoseAは各ターンでプレイヤーがゲームに負けたか否かを表すクラスです．<br>
この課題では，
<blockquote>
<strong>「HandsA の情報が更新されたときにその情報を LoseA にPUSH型で転送しているプログラムを，PULL型で転送するようにリファクタリングすること」</strong>
</blockquote>
を目的とします．<br>
リファクタリングの詳しい作業手順は以下で説明します．
</p>
<hr size="5">
<h2>リファクタリング作業</h2>
<p>
以下では，リファクタリングに要した時間を測っていただきますので，お手元に時計をご用意ください．<br>
時間計測にあたって，急いで作業していただく必要はまったくありません．<br>
最初の課題から最後の課題まで一定のペースを保てるよう, <font color="red">正しくリファクタリングを行うこと</font>を意識してください．<br>
この課題で作業していただく内容は，
<blockquote>
<strong>「HandsA の情報が更新されたときにその情報を LoseA にPUSH型で転送しているプログラムを，PULL型で転送するようにリファクタリングすること」</strong>
</blockquote>
です．<br>
開発環境を立ち上げてソースコードの確認をしていただいて構いませんが，ソースコードの変更は時間計測の準備ができるまで行わないで下さい．<br>
リファクタリングの作業手順を以下に説明します．
</p>
<p>
まず，現時点での HandsA から LoseA へのデータ転送に関するクラス図とシーケンス図を以下に示します．<br>
<img src="../images/algo_class_push.jpg" class = "halfSize">
<img src="../images/algo_seq_push.jpg" class = "halfSize"><br>
現時点ではPUSH型のデータ転送で実装されているため，HandsA クラスの updateResultBy...() メソッドから LoseA クラスの updateHandsA() メソッドを呼び出すために HandsA クラスが LoseA クラスを参照していています．<br>
ここで Algo クラスは，初期化時に HandsA クラスや LoseA クラスのインスタンスを生成し，それらのインスタンスを関連付ける役割を担っています．<br>
また，HandsA クラスや LoseA クラスにアクセスするための入口（Facade）としての役割も担っています．<br>
Algo，HandsA，LoseA クラスのいずれも resources パッケージの中に入っています．<br>
</p>
<p>
これを，以下のクラス図とシーケンス図に示すようなPULL型のデータ転送になるよう，手作業でリファクタリングを行っていただきます．<br>
<img src="../images/algo_class_pull.jpg" class = "halfSize">
<img src="../images/algo_seq_pull.jpg" class = "halfSize"><br>
今度はPULL型のデータ転送であるため，LoseA クラスの getValue() メソッドから HandsA クラスの getValue() メソッドを呼び出すために LoseA クラスが HandsA クラスを参照していることに注意してください．<br>
</p>
<p>
リファクタリングの作業手順の詳細を以下に示します．<br>
<ol>
<li>テストプログラムを実行して正しく動作することを確認する．</li>
<li>LoseA クラスに，handsA クラスのインスタンスを参照するフィールド HandsA handsA を追加する．</li>
<li>LoseA クラスのコンストラクタを，HandsA クラスのインスタンスを引数に取るように書き換える．</li>
<li>HandsA クラスのコンストラクタを，LoseA クラスのインスタンスを引数に取らないように書き換える．</li>
<li>HandsA クラスのフィールド loseA を削除する．</li>
<li>Algo クラスのコンストラクタ内で，LoseA クラスのインスタンスが HandsA クラスのインスタンスを参照するよう，インスタンスの生成順や生成方法を変更する．</li>
<li>LoseA クラスのプレイヤーの勝敗結果を保存するためのフィールド boolean value を削除する．</li>
<li>HandsA クラスの updateResultBy...() メソッド(4つある)の最後の LoseA クラスの updateHandsA() メソッドの呼び出し文を削除する．</li>
<li>LoseA クラスの getValue() メソッドを HandsA クラスの getValue() メソッドを呼び出すように書き換える．</li>
<li>LoseA クラスの updateHandsA() メソッド内で行っていた，プレイヤーの手札の情報をもとに勝敗判定を行う処理を，getValue() メソッドに移動し，判定結果を getValue() メソッドの戻り値として返すようにする．</li>
<li>LoseA クラスの void updateHandsA(List<Card> handsA) メソッドを削除する．</li>
<li>テストプログラムを実行して正しく動作することを確認する．</li>
</ol>
</p>
<p>
作業に着手する前に，メッセージでお伝えした自分専用のブランチ（user??）を切って，一度プッシュを行ってください．<br>
プッシュを行った後，開発環境の準備ができれば，<font color="red">時間計測を開始</font>してください. 時間計測はできる限り，30秒以内の単位での計測をお願いします. <br>
テストプログラムが正しく動作するまでは作業完了とは見なされないので注意してください．
</p>
<p>
リファクタリングの作業が完了したら作業時間を記録し，その後，作業結果を自分専用のブランチにコミット＆プッシュしてください．<br>
ただし，<font color="red">main ブランチには決してマージをしないように，また自分専用のブランチを決して削除しないように</font>注意してください． <br>
プッシュ後に，以下のアンケートにお答えください．
</p>
<blockquote>
<a href="https://docs.google.com/forms/d/e/1FAIpQLScs6FYWlhZb27QHg1ZGKJpBExNn2_6lNJ1ms_2aVU2KbuvUAg/viewform?usp=sf_link" target="_blank">アンケート回答</a>
</blockquote>
<hr size="5">
<p style="margin-bottom:2em;"></p>
<a href="../Closing.html">次へ</a>
</body>
