1

php関数に関する簡単な質問です。一部の人にはばかげているように聞こえるかもしれませんが、悪い習慣には入りたくありません。次のことで何か問題がありますか？

function do_something($val)
{
  $a = 1;
  if ($val==$a)
  {
    return true;
  }
  return false;
}

それ以外の;

function do_something($val)
{
  $a = 1;
  if ($val==$a)
  {
    return true;
  }
  else
  {
    return false;
  }
}

申し訳ありませんが、私の例は良くないと思います。基本的に、この関数はデータベースにデータを挿入したり、電子メールを送信したりできます。これらの関数を使用すると、trueまたはfalseを返すことで、成功したかどうかを確認するだけで済みます。if-elseブロックの代わりに短いメソッドを使用できるのが適切かどうかを知りたいと思いました。

それがより明確になることを願っています。

4

4 に答える 4

4

あまり。どちらも同じように機能します。ただし、次のように記述した方がはるかにクリーンです。

function do_something($val)
{
  $a = 1;
  return ($val==$a) ? true : false;
}
于 2012-04-15T10:52:54.163 に答える
2

値を返すとき、関数は残され、その後に何が続くかは問題ではないため、これは非常にクールです。

しかし、これを短くすることができます

function do_something($val)
{
  $a = 1;
  return $val == $a; // this condition will be evaluated to true/false
}
于 2012-04-15T10:53:17.057 に答える
1

The shortest way to do it:

function do_something($val)
{
  return ($val==1) ;
}
于 2012-04-15T10:56:25.687 に答える
0

いいえ、それはまったく問題ありません。実際、複数のケースでアドバイスされています。:)

于 2012-04-15T10:52:40.513 に答える