本課題では,SimpleTwitterプログラムにアカウント名を変更可能にする機能追加を行っていただきます.
このSSDStoreプログラムが,サイトA以外のサイトからも商品情報を収集して扱えるようにすることを考えます.
以下では,リファクタリングに要した時間を測っていただきますので,お手元に時計をご用意ください.
時間計測にあたって,急いで作業していただく必要はまったくありません.
最初の課題から最後の課題まで一定のペースを保てるよう, 正しくリファクタリングを行うことを意識してください.
開発環境を立ち上げてソースコードの確認をしていただいて構いませんが,ソースコードの変更は時間計測の準備ができるまで行わないで下さい.
まずは、以下の作業手順を一通り読んでから作業を開始してください.
現在の設計では,以下のクラス図ように ItemsByPrice オブジェクトと ItemsByCapacity オブジェクトが,それぞれ SiteA オブジェクトから直接SSDの商品情報をPULL型のデータ転送で取得し,
さらに Price オブジェクトから価格を,Capacity オブジェクトから容量をPULL型のデータ転送で取得して,それらの値を使って別々に検索を行い検索結果を出力しています.

具体的には以下のコードように,ItemsByPrice オブジェクトの getValue() メソッド内部で,サイトAからSSDのリストを取得してその中から Price オブジェクトで指定された価格以下の商品を検索し,
同様に ItemsByCapacity オブジェクトの getValue() メソッド内部で,サイトAからSSDのリストを取得してその中から Capacity オブジェクトで指定された容量以上の商品を検索しています.
public class ItemsByPrice {
private Price price;
private SiteA siteA;
:
public double getValue() {
List<Map<String, Object>> temp_l1 = new ArrayList<>();
{
for (Map<String, Object> item: this.siteA.getValue()) {
if ((Integer) item.get("price") <= this.price.getValue()) {
temp_l1.add(item);
}
}
}
return temp_l1;
}
}
public class ItemsByCapacity {
private Capacity capacity;
private SiteA siteA;
:
public double getValue() {
List<Map<String, Object>> temp_l1 = new ArrayList<>();
{
for (Map<String, Object> item: this.siteA.getValue()) {
if ((Integer) item.get("capacity") >= this.capacity.getValue()) {
temp_l1.add(item);
}
}
}
return temp_l1;
}
}
ここで,このSSDStoreプログラムがサイトBからも商品情報を収集して扱えるように拡張することを考えた場合,ItemsByPrice オブジェクトと ItemsByCapacity オブジェクトの両方の実装を,
SiteA オブジェクトと SiteB オブジェクトを同時に扱えるように書き換えなければならなくなります.
今後,より多くのサイトを扱えるように拡張していくことを考えると,このように ItemsByPrice オブジェクトと ItemsByCapacity オブジェクトが直接,個々のサイトのオブジェクトを参照しているような現在の設計では,拡張作業が煩雑になってしまいます.
そこで,以下のクラス図のように複数のサイトを同時に参照することができる SiteWrapper オブジェクトを新たに導入し,
ItemsByPrice オブジェクトと ItemsByCapacity オブジェクトが,SiteWrapper オブジェクトを通じて,間接的に SiteA オブジェクトや SiteB オブジェクトにアクセスするように設計変更を行います.
そうすることによって,SiteWrapper にいくつサイトを追加しても,ItemsByPrice オブジェクトや ItemsByCapacity オブジェクトの実装に影響が及ばないようにすることができます.

具体的に変更後の設計の中では,以下のように SiteWrapper クラスが SiteA を参照し,
ItemsByPrice オブジェクトと ItemsByCapacity オブジェクトは,この SiteWrapper の getSiteValue() メソッドを呼び出して,最新のSSDの一覧を得るようにします.
そして,新しいサイトを追加する際は,この SiteWrapper クラスを変更するようにします.
public class SiteWrapper {
private SiteA siteA;
public SiteWrapper(SiteA siteA) {
this.siteA = siteA;
}
public List<Map<String, Object>> getSiteValue() {
return this.siteA.getValue();
}
}
本課題では,リファクタリングに要した時間を測っていただきますので,お手元に時計をご用意ください.
時間計測にあたって,急いで作業していただく必要はまったくありません.
最初の課題から最後の課題まで一定のペースを保てるよう, 正しくリファクタリングを行うことを意識してください.
課題に着手する前に開発環境を立ち上げて現状のソースコードの確認をしていただいて構いませんが,実際のソースコードの変更は時間計測の準備ができるまで行わないで下さい.
作業に着手する前に,main ブランチから,メッセージでお伝えした自分専用のブランチ(user??Refactor)を新規作成して,一度 push を行ってください.
push を行った後,開発環境の準備ができれば,時間計測を開始してください. 時間計測はできる限り,30秒以内の単位での計測をお願いします.
ソースコードの変更作業は上記「SSDStore のリファクタリングの概要」にしたがって進めてください.
テストプログラムTestSSDStoreUpdate.javaが正しく動作するまでは作業完了とは見なされないので注意してください.
リファクタリングの作業が完了したら作業時間を記録し,その後,作業結果を自分専用のブランチに commit & push してください.
ただし,main ブランチには決して merge をしないように,また自分専用のブランチを決して削除しないように注意してください.
push 後に,以下のアンケートにお答えください.
アンケートフォーム (別タブが開きます)