函数返回代码块查询

Quick question to do with php functions, it may sound silly to some of you but I dont want to get in to bad habits. Is there anything wrong with doing the following?

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

Instead of;

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

Sorry guys I think my example isn't great. Basically the function could insert data into a database or send an email etc. With these functions I may only need to now whether it was successful or not by returning true or false. I wanted to know whether its suitable that I can use the shorter method instead of the if-else block.

I hope that makes it clearer.

No, that is perfectly fine, and in fact advised in multiple cases. :)

Not really. Both works the same. However, it would be much cleaner to write it like this:

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

That's totally cool, because when returning a value, the function is left and it doesn't matter what follows.

But you could shorten this with

function do_something($val)
{
  $a = 1;
  return $val == $a; // this condition will be evaluated to true/false
}

The shortest way to do it:

function do_something($val)
{
  return ($val==1) ;
}