1

私は自分のリポジトリに機能を持っています...それらがいくつかのデータである場合はすべてがうまく機能します...しかしそうでない場合:

エラー:未定義の変数$followees_idsが表示されます...

public function foreachFolloweesToFolloweesIds($followees)
{
    foreach ($followees as $followee) {
        $followees_ids[]=$followee['id'];
    } 

    if (empty($followees_ids)) {
        return NULL;
    } else {
        return $followees_ids;
    }
}

私が書いたようにそれを行うことができます(空... bla blaですが、あまりうまく書かれていないようです...そして私はそれをたくさん使うのでいくつかのベストプラクティスが素晴らしいでしょう。変数が未定義にならないようにしますか?

4

2 に答える 2

1

したがって、基本的に、メソッドは(あなたが呼んだ)followees_idsの配列を返します。

このメソッドが何に使用されるのかわからない場合でも、Earlyreturnを使用し$followeesに要素が含まれているかどうかを確認することをお勧めします。

したがって、メソッドの上に追加する必要があります。

if (empty($followees)) {
    return array(); // or null
}

また、

  • 最初に、メソッドが返す配列を初期化する必要があります。
  • 早く戻ったときは、elseステートメントは本当に必要ありません。
于 2013-03-14T00:34:02.490 に答える
1

置くだけ

$followees_ids = array();

foreachの前にあるため、結果がない場合でも$followees_idsは未定義ではありません。

于 2013-03-14T01:48:52.327 に答える