ソースのダメなところ

Question

以下のような関数を作りました、なんとなくですがもっと賢く書ける気がします
ダメだしなどなど意見、アドバイス頂きたいです

    function search($src)
    {
        if (strpos($src, "A") !== FALSE) {
            return 'AAA';
        }elseif(strpos($src, "B") !== FALSE) {
            return 'BBB';
        }elseif(strpos($src, "C") !== FALSE) {
            return 'CCC';
        }elseif(strpos($src, "D") !== FALSE) {
            return 'DDD';
        }else{
            return '';
        }
    }

Answer 1

っと失礼。特定の文字列が含まれている場合は文字列に応じたキーワードを返すことがやりたいことですかね。

であれば下記でどうでしょう？

function search($src) {
    $arr = array(
        'php' => 'PHP',
        'mysql' => 'MySQL'
    );
    foreach ($arr as $k => $v) {
        if (strpos($src, $k) !== FALSE) {
            return $v;
        }
    }
    return '';
}

Comments

[curekaishine]

特定の文字列が含まれている場合は文字列に応じたキーワードを返す
↑そうです！　これをソースで実現したかったのです！

[KatsumiTanaka]

自分もこれに1票です。
確認したいキー項目が増えていくことが予想されるのであれば、配列なりDBなりでその項目を管理し、コードの冗長を避けるのが良いと思います。

Answer 2

やりたいことと違ってたらすみません。

function search($src) {
    if (preg_match("/\A.*([A-D]).*\Z/", $src, $ret)) {
        return str_repeat($ret[1], 3);
    } else {
        return '';
    }
}

Answer 3

$src = 'Abcstdh';
の時、意図した結果になります？
意図した動きではないと推測するのですが。

Answer 4

phpがそれほど書けないのですが，アルゴリズムで説明させていただきます．
仮の変数領域$tmpに"E"を代入しておきます．
文字列の後方から１文字ずつ検索し，"A"から"D"を見つけたら次の条件分岐をします．
$tmpに格納されている文字よりアルファベット順で早ければ$tmpに代入．
すべての文字を検査し終わったら$tmpに入っている文字列を３回繰り返してreturn．
C言語のようなphpのようなコードで書かせていただきましたが，
なんとなくわかっていただけると思います．

tmp = 'E';
for (i = strlen(str) - 1; i >= 0; i--)
  if ('A' <= str[i] && str[i] <= 'D')
    if (tmp > str[i])
      tmp = str[i];
if (tmp == 'E')
  return '';
else
  return str_repaet(tmp, 3);

Comments

[curekaishine]

投稿ありがとうございます
なるほどこの方式だと処理に無駄がなく高速そうですね。

投稿が言葉足らずでした
"A"というのは特定の文言(等システムの構成はmysql、php...)で
"AAA"というのはMySQL
みたいなものをイメージです