#33 都道府県表示のクラス化

[fr-itaya]

TODO:dizzy:

• [x] 都道府県クラスのインスタンス化(form.php, confirm.php)
• [x] メソッド呼び出しを記述
  • [x] フォーム画面　→　セレクトボックス生成
  • [x] 確認画面　　　→　都道府県表示
• [x] 都道府県クラス作成(pref.php)
  • [x] プロパティ設定
  • [x] コンストラクタ作成
  • [x] メソッド作り込み
  • [x] セレクトボックス作成(フォーム画面)
  • [x] 都道府県表示(確認画面)

---

• [x] メソッド呼び出し修正

レビュー対応

---

• [x] セレクトボックス作成メソッドをglobal使わないように修正(pref.php)
• [x] 都道府県リストの取得をform.phpへ
• [x] プロパティのアクセス修飾子をprotectedへ(pref.php)
• [x] コンストラクタ引数の順序変更(pref.php)

---

• [x] セレクトボックス作成メソッドの引数削除(pref.php)
• [x] メソッドのreturnを値返すだけに修正(pref.php)
• [x] コンストラクタ処理整理(pref.php)
• [x] pref_arrayの宣言部分確認→修正(db_fetch_pref.php, confirm.php)

[fr-sasaki]

インスタンス化部分は特に問題ないと思います。 実装を進める前に、クラスの設計をして下記項目をコメントに書いてみてください。

• プロパティの変数名
• コンストラクタの引数
• 実装するメソッド名、引数

上手く設計すると、DB接続部分が Prefecture クラスの外に追いやられるので今回はこれを意識してみましょう。

[fr-itaya]

拙いですがこのような感じに。 レビューよろしくお願い致します。

---

• プロパティの変数名
  • $pref_array(都道府県名をDBから取得し--だけの配列とmerge)
  • $pref_id(元の関数GetSelectBoxTagでいう$prefecture_sel_value==sel_valueをまとめたもの)
  • $menu_tag(上記2つの変数の中身をselectタグのoptionと連結する)
• コンストラクタの引数
  • $menu_tag? (初期化が必要といえば...)
  • prefecture? ($_POST[prefecture]の値を取得して$pref_idに代入?)
• 実装するメソッド名、引数
  • createSelectBoxTag($pref_array, $pref_id, $menu_tag)
  • getPrefById($pref_id)
  • (DB接続部分をクラスの外に出すにはまだメソッドが要りそう?)

[fr-sasaki]

• プロパティの変数
  • $pref_arrayと$pref_idはOKです！
  • $menu_tagは、メソッドの引数として渡されてくるものなので、プロパティからは削除しておきましょう
• コンストラクタの引数
  • ここでは$pref_arrayと$pref_idを初期化するための値を渡しましょう！
• 実装するメソッド名、引数
  • $pref_array,$pref_idはプロパティにあるので引数で渡す必要は特にないです
  • getPrefByIdは、$pref_array,$pref_idを用いて実装してみましょう、このメソッド内でのDB接続部分は削除してしまって大丈夫です

[fr-sasaki]

マージします！:+1: