phpで重複する記述を省きスマートに記述したい

#　質問内容
phpのファンクションを作っています。
作っていて重複が多く無駄を省けないかと考えたのですが、
力不足でよい方法が思い浮かびませんでした。
ご教示いただけると幸いです。

コード

php
1function example($turn, $tokusyuA, $tokusyuB)
2{
3    if ($tokusyuA == true && $turn == a) {
4        $text = "{$name1}の特殊能力発動！";
5        $koukaon = "exmple.mp3";
6        $turn = change_turn($turn);
7    }
8    if ($tokusyuB == true && $turn == b) {
9        $text = "{$name2}の特殊能力発動！";
10        $koukaon = "exmple.mp3";
11        $turn = change_turn($turn);
12    }
13    return ($turn);
14}

###　コードの説明
プレーヤーAとBがいます。
$turnにはaもしくはbという値が入ります。プレーヤーAのターンのときはaといった具合です。
change_turn($turn)という関数で$turnの値がaだったときはbに切り替え、bだったときはaに切り替えています。
$tokusyuAがtrueでかつ$turnがaのとき、特殊能力を発動させテキストと効果音を関数 kakunou();で格納しています。

kei344

2019/10/13 04:27

$name1が定義されていません。

jinba

2019/10/13 05:14

「$tokusyuA」「$tokusyuB」は true or false の２パターンのみですか？ ($tokusyuA == true && $turn == a) ($tokusyuA == true && $turn == b)←「b]存在する？ ($tokusyuA == true && $turn == c)←「c]存在する？「$turn」は何パターンほど想定されていますか？

nogimareske

2019/10/13 06:04

kei344様すいません、$name1と$name2はグローバル変数で関数の枠外で定義しています。 jinba様 $turnという変数ですがaとbのみが存在し、change_turn($turn)という関数を呼び出してaとbを毎回切り替えています。説明不足だったかもしれないので本文を修正してみます。

tanishi_a

2019/10/13 06:06

質問文のコードだけ見ると特別無駄が多いようには見えないですが、どこかパターンが増えていったときのことを心配してるのですかね。 $text = ... の行と $koukaon = ... の行は使ってなさそうなので削除しても良さそうだけど、きっとこれは疑似コード的なもので、そういう話でも無いんですよね？

nogimareske

2019/10/13 06:15

tanishi_a　様 $text = ... の行と $koukaon = ... グローバル変数でかつ配列でして、元のコードでは配列に値を格納しひとつづつ取り出しています。あまりコードが長くなると読みづらいのではないかと思い重複する部分をある程度削っています。

tanishi_a

2019/10/13 09:19

global $text, $koukaon; みたいな記載も削ったということですね・・。

行動規範の内容に同意します

回答3件

ベストアンサー

PHP
1function example($turn, $tokusyuA, $tokusyuB)
2{
3
4    switch( true ){
5
6        case $tokusyuA == true && $turn == a:
7            $text = "{$name1}の特殊能力発動！";
8            break;
9
10        case $tokusyuB == true && $turn == b:
11            $text = "{$name2}の特殊能力発動！";
12            break;
13
14        default:
15            return ($turn);
16
17    }    
18
19    $koukaon = "exmple.mp3";
20    $turn = change_turn($turn);
21    return ($turn);
22}

これでは無駄を省けていませんが
重複部分を削られているとのことですので
重複部分が増えればスマートになる気がします

投稿2019/10/13 06:55

jinba

総合スコア310

nogimareske

2019/10/13 07:19

回答ありがとうございます。質問で上げたコードは省略されたものなので重複部分は実際にはもっと多いです。現時点で重複する記述を減らせそうなので早速試してみたいと思います。また、switch文で==や&&が使えることを初めて知りました。今まで見たことあるサンプルコードはどれも case '1': break; case '2': break; のようなものしか見たことがなかったので・・・非常に参考になりました。ありがとうございます。

行動規範の内容に同意します