「オブジェクト指向」 プロフィール変更機能で、fbのように一項目ずつ変更できる場合は。。？？

Question

今していること

オブジェクト指向を勉強中で、自分で簡単にfacebookの旅人バージョンのような物を作っております

コード

class Traveler {
  public $name;
  public $age;
  public $sex;
  public $country;
  
  public function regist($name, $age, $sex, $country) {
    
    $this -> name = $name;
    $this -> age = $age;
    $this -> sex = $sex;
    $this -> country = $country;
    
    echo $name.",".$age.",".$sex.",".$country."をDBに登録しました";
  }
  
  public function editProfile($name, $age, $sex, $country) {
    
    $this -> name = $name;
    $this -> age = $age;
    $this -> sex = $sex;
    $this -> country = $country;
    
    echo $name.",".$age.",".$sex.",".$country."をDBにUPDATEしました";
  }
  
  public function postPublic() {
    echo $this->name.",".
      $this->age.",".
      $this->sex.",".
      $this->country.",".
      "this traveler posted something in public space!!";
  }
}

たぶん見たらお分かりいただけるかと思いますが。
「トラベラー登録機能」
「プロフィール編集機能」
「post機能」
の３つがあります。

ここでプロフィール編集機能を使用する場合は、こちらで可能です。
$traveler -> editProfile("ジョンカビラ", 44, "man", "japan" );

疑問点

これはプロフィール編集画面にformボタンが１個ある場合、と考えられる。
がしかし、facebookのプロフィール編集画面のように、項目を一個ずつ編集できる場合
→一個ずつデータが送られてくる
→クラス内に editName(), editAge(), editSex(), edit()のようなメソッドを作る必要がある？？(本当のfacebookだとすごい数になりそうですが。。)

しかしその機能の場合、どう書いてもメソッド一つでまとめることができないので、何か良い方法はないのかとこちらに質問させていただきました。
よろしくお願いいたします。

Comments

[m.ts10806]

結局どのように解決されたのでしょうか？差し支えなければ回答にコメントいただけたらと。https://teratail.com/help/question-tips#questionTips4-2

Answer 1

任意項目（ラベルも）だと最低限

user_id,label,value,order

のような構成のKVS擬きを使えばいいと思いますよ。

json型の１カラム追加でもいいですね。

Answer 2

Facebook利用は結構長いので登録してすぐのことは覚えてないですが、editのときはともかくaddのときは必須項目ってあったと思います。
※誤解なきよう補足しますとaddのときは必須項目は全て登録するので必須でeditのときは更新したい項目だけ更新できるという意味です
元々実名運用が売りのものだっただけに、名前と性別と誕生日と国と、あと電話番号、メールアドレス辺り。
必須項目が決まっていればaddのときにチェックをしておけば良いので、「DBのカラムにある項目がきているか」を元にチェックをしておけば、引数も1つでいけます。
まとめようと思えばaddとeditをまとめられなくもないですが、用途が違うこともあるので、別のやり方にするとして、参考までにeditをざっくり改修してみます。

※設定ファイルなどに持っておいたほうが楽かもしれませんが、とりあえずの提案ということで
※構文エラーだけ解消して未検証コードです。実装のイメージだけ伝わればと思います。

<?php
class Traveler {
	//DBカラム
	private $dbcolumns = ["name","age","sex","country"];
	//登録用データ
	private $dbinput = [];
	
	// メンバ変数部分省略
	// regist部分省略
	
	/**
	 * @param array $data key:input-name value:input-value
	 */
	public function editProfile($data=[]) {

		//存在するカラムのみ取り出し
		foreach($data as $key=>$value){
			if(in_array($key,$this->dbcolumns)){
				$this->dbinput[$key] = $value;
			}
		}

		if(count($this->dbinput)>0){
			//必須チェックなどのバリデーションを行う（今回は省略）
			
			$editinfo = [];
			foreach($this->dbinput as $column=>$columnvalue){
				$editinfo[] = $column.":".$columnvalue;
				$this->$column = $columnvalue; //元の処理のメンバ変数にも一応値を入れておく
			}
			echo implode(",",$editinfo)."をDBにUPDATEしました";
		}else{
			//更新するカラムがない場合
		}
	}
	// post部分省略
}

これで、実際にこれを使う側との依存性を分離できた、はずです。
使う側は更新したいカラムとその値を渡す（$_POSTそのままでも良い）だけです。
このコード自体は「何が送られてくるか」は知らないので、引数に初期値をつけたり何も送られていない場合の分岐を入れています。
「必須チェックなどのバリデーションを行う（今回は省略）」と書いていますが、もう少しきちんと処理を細分化したいのであればここでバリデーションを行うのではなく、
バリデーションを行った情報を渡しても良いです。
ただ先に書いたように「何が送られてくるか知らない」という前提のもと処理を行うので、DB登録直前でチェックを行っても良いかなと、コメントだけ置いてます。

オブジェクト指向をもう少し進めるならDB共通のinsert,update,select,deleteの処理を行う親クラスをもってそこに共通のプロパティとか持つようにしたほうが汎用的な作りになるのですが、それはまた次の段階ということで。

参考まで。

Comments

[退会済みユーザー]

実名運用というか、出会い系運用（げぶんげふん）

[m.ts10806]

まあ元々創設者が大学時代に作ったコミュニティですしね。