PHPの引数で、代入される内容の違いについてのバージョンを作る方法を知りたいです。

Question

今このようなPHPがあるのですが、ほぼ同じコードで一部を変えて、function sitsumon(){}の関数にバージョンを増やしたいです。

$fluits1と$fluits2に代入される内容だけが違うバージョンです。

function sitsumon(){
								
	function example(){
		$xxx = 'xxx';
		$yyy = 'yyy';
		if($xxx == $yyy){
			return false;
		}
	}

	$fluits1 = get_post_meta($post->ID , '形' ,true);
	$fluits2 = get_post_meta($post->ID , '色' ,true);

	$itemArr = example($fluits1, $fluits2);
	
	foreach ($itemArr as $product => $price) {
		echo "<p>{$product}</p><p>{$price}</p>";
	}
	
}

代入される内容の具体的な違いは、
$fluits1 = get_post_meta($post->ID , '形' ,true);
$fluits2 = get_post_meta($post->ID , '色' ,true);
の部分だけが
$fluits1 = get_the_terms( $post->ID, '味' );
$fluits2 = get_the_terms( $post->ID, '趣' );
となるようなバージョンを作りたいと思っています。

現状は<?php sitsumon(); ?>とすれば<p>{$product}</p><p>{$price}</p>がechoされるわけですけれど、

質問は<?php sitsumon(meta); ?>と<?php sitsumon(terms); ?>のようなバージョンを作りたいイメージです。

そのようなケースでは、上のPHPはどう書き直せばいいのかご教示頂ければ幸いです。
よろしくお願い致します。

Comments

[退会済みユーザー]

matsuzakaq もとい dadanda もとい kujiran もとい ajinomoty　祖国にプログラミングの質問サイトは無いのか？

[EggBenedict]

もう許してほしいです。あとさすがに生まれた国は馬鹿にしないでくださいませ。

Answer 1

class Situmon{
  private $target_object,$param;
  function __construct($target_object,$param){
    $this-> target_object =$target_object;
    $this->param=$param;
  }
  function exec(){
    ....
    $fluits1 = call_user_func('get_'.$this-> target_object,$post->ID, $this->param[0]);
    $fluits2 = call_user_func('get_'.$this-> target_object,$post->ID, $this->param[1]);
    ....
  }
}

$situmon1=new Situmon('post_meta',['形','色']);
$situmon1->exec();

$situmon2=new Situmon('the_terms',['味','趣']);
$situmon2->exec();

そのためのオブジェクト指向

Comments

[EggBenedict]

おはようございます。どうもありがとうございます。
オブジェクト指向との新しいキーワードに触れて早速検索してみましたが、これまた難しいですね

ご提示のコードを見た感じ、

situmon1を実行すると、
param[0]
param[1]
に
Situmon(['形','色']
が入り、

situmon2を実行すると、
param[0]
param[1]
に
Situmon(['味','趣']
が入る。

というイメージでしょうか。

これは役に立ちそうですね。

まぁ下記はサッパリわかりませんが…笑
private $param;
function __construct($param){
$this->param=$param;
}

ところでですね、get_the_terms だけでなくて、get_post_meta のような違いがあるのですが、その場合ですとどうなりますでしょう？

[KazuhiroHatano]

オブジェクト指向を基本から色々調べてみましょう
そこからいろんなものが見えてきます

処理を共通化するときのケースごとの最適なやり方
というのもオブジェクト指向を調べる中で学ぶことになるかと思います

[EggBenedict]

少し検索したところ、KazuhiroHatanoさんがかなり使いこなしてるなぁという印象を受けましたが、私は __construct あたりをちょっと読んで挫折しました…。

とはいえ、良い検索の機会をどうもありがとうございます。
またどうぞ宜しくお願い致します。

結局、次の方法でなんとかこなしました。ご報告まで。

<?php
function sitsumon( $type ){

if ( $type == 'meta' ){
$fluits1 = get_post_meta($post->ID , '形' ,true);
$fluits2 = get_post_meta($post->ID , '色' ,true);
}
if ( $type == 'terms' ){
$fluits1 = get_the_terms( $post->ID, '味' );
$fluits2 = get_the_terms( $post->ID, '趣' );
}

}
?>

<?php sitsumon('meta'); ?>
<?php sitsumon('terms'); ?>

[KazuhiroHatano]

回答のコードはあんまり参考にしちゃダメですよ
クラスを使ってるっていうだけで何も考えてないコードです

質問のコードについて推測の範囲で提案するなら

質問クラスは値の取得方法と比較方法ごとに用意された
共通のインターフェースを持つ値取得＋比較クラスのインスタンスを
コンストラクタの引数として取る
質問クラスは受け取った値取得＋比較クラスのインスタンスの
値取得メソッドと比較メソッドを用いて出力を行う
みたいな設計が分担がはっきりしてキレイなんじゃないでしょうか

質問クラスの抽象クラスからの拡張で
バリエーションを展開するというのもシンプルでいいかもしれないです

Answer 2

例えば、

function sitsumon($type){

のようにsitsumon 関数に引数を渡すように実装すれば良いのでは？

Comments

[EggBenedict]

はい、まさにそういうイメージです！