絞り込み条件によって、WP_Queryが冗長になってしまう問題について

Question

teratailさんのように、「マイタグで絞り込み」などの条件を設置したいのですが、その際にWP_Queryが冗長になってしまうのを避けたいです。

###実現したいこと
Cookieに保存された絞り込み条件「$_COOKIE["refine"] )」から、表示記事の条件「WP_Query( $args )」を生成したいです。

しかし絞り込み条件に「マイタグ」「執筆者」などがあると、PHPが冗長になってしまいそうで困っています。

具体的には、該当のソースコードにあるように
// ➀マイタグで絞り込む場合
// ➁執筆者で絞り込む場合
// ➂マイタグと執筆者で絞り込む場合
という３つのパターンそれぞれのWP_Query( $args )を作らなければならないのか？と思っておりまして、この点の冗長さを避けたいです。

###該当のソースコード

<!-- 新着記事一覧 -->
<section class="RelatedPost">
	
	<?php
	
	/*
	* 絞り込みの条件を$refineへ
	*/
	$refine = $_COOKIE["refine"] );

	/*
	* 表示記事の条件を$argsへ
	*/
	
	// ➀マイタグで絞り込む場合
	if(preg_match('/マイタグ/',$refine)){	
	$args = array(	
		// 新着記事の条件
		'orderby'        => 'date',	
		// マイタグの条件
		'tax_query' => array(
			array(
				'taxonomy' => 'tag',
				'field'    => 'slug', // IDのときはterm_id
				'terms'    => explode(",", $mytag),
			),
		),
	);							
	}
	
	// ➁執筆者で絞り込む場合
	if(preg_match('/執筆者/',$refine)){	
	$args = array(
		// 新着記事の条件
		'orderby'        => 'date',		
		// 執筆者の条件		
		'author' => $authoe,
	);							
	}				

	// ➂マイタグと執筆者で絞り込む場合
	if(preg_match('/マイタグ,執筆者/',$refine)){	
	$args = array(	
		// 新着記事の条件
		'orderby'        => 'date',		
		// マイタグの条件	
		'tax_query' => array(
			array(
				'taxonomy' => 'tag',
				'field'    => 'slug', // IDのときはterm_id
				'terms'    => explode(",", $mytag),
			),
		),
		// 執筆者の条件		
		'author' => $authoe,					
	);							
	}

	$the_query = new WP_Query( $args );
	?> 
	
	<!-- ループあれこれ書く -->

</section>

<!--コメント数記事一覧 -->
<section>
ここには上の
// 新着記事の条件
'orderby'        => 'date',
を
// コメント数の条件
'orderby'        => 'comment_count',
にしただけで他が同じソースコードを書く？
</section>

さらにソースコードの最後にあるように、<!-- 新着記事一覧 -->だけでなく<!--コメント数記事一覧 -->なども追加する予定なのですが、そうなるとその都度➀➁➂の条件を書かないといけないのか？と思っておりまして、どうしたものかと途方に暮れています。

この辺りをなんとかスッキリとさせるためにはどのようなコードがベストでしょうか？

Answer 1

配列に値を追加していきゃいいんじゃないでしょうか

$refine_arr=explode(',',$_COOKIE['refine']);

$args=[];

if(新着順の条件){$args['orderby']='date';}
elseif(コメント数順の条件){$args['orderby']='comment_count';}

if(in_array('マイタグ', $refine_arr,true)){
  $args['tax_query']=[
    ['taxonomy' => 'tag','field'=>'slug','terms'=>explode(',',$mytag)]
  ];
}
if(in_array('執筆者', $refine_arr,true)){
  $args['author']=$author;
}
$the_query = new WP_Query( $args );

ついでにpreg_match('/マイタグ,執筆者/',$refine)という判定条件は危うく感じます
ここはたとえ値の順番が入れ替わっても引っかかるようにしたいはずですし
将来的に条件が増えて間に別の値が入ってきたとしても同じく引っかかるようにしたいはずです

Comments

[makirons]

どうもありがとうございます。追加すれば良さそうかも、とは思い至ったのですが具体的な方法がわからず、それについて別の質問をしたりしていました。（あちらにもご回答感謝です。）

判定条件についてもご指導誠にありがうございます。
なるほど「preg_match」でなく「in_array」を使うために「explode」をかけたのですね。

[makirons]

すみません、大変恐縮ですが、簡略化した下記でも何も表示されないのですが、何か…、、お間違いではございませんでしょうか？すみません。

<?php
$args=[];
$args['orderby']='date';
);
$the_query = new WP_Query( $args );
?>

[papinianus]

ほんとに↑書いてるなら
);
が構文エラー