PHP: 'or' statement on instruction fail: h

2020-01-31 23:40发布

Everyone here should know the 'or' statemens, usually glued to an die() command:

$foo = bar() or die('Error: bar function return false.');

The most of the times we see something like:

mysql_query('SELECT ...') or die('Error in during the query');

However, i cant understand how exactly that 'or' statement works.

I would like to throw a new exception instead of die(), but:

try{
    $foo = bar() or throw new Exception('We have a problem here');

Doesnt work, and neither

$foo = bar() or function(){ throw new Exception('We have a problem here'); }

The only way i found to do that is this horrible thought:

function ThrowMe($mess, $code){
    throw new Exception($mess, $code);
}
try{
    $foo = bar() or ThrowMe('We have a problem in here', 666);
}catch(Exception $e){
    echo $e->getMessage();
}

But there is a way to throw a new exception directly after the 'or' statement?

Or this kind of structure is mandatory (i dont liek the ThrowMe function at all):

try{
    $foo = bar();
    if(!$foo){
        throw new Exception('We have a problem in here');
    }
}catch(Exception $e){
    echo $e->getMessage();
}

Edit: what i want is really to avoid the use of an if() check every potential dangerous operation i do, for example:

#The echo $e->getMessage(); is just an example, in real life this have no sense!
try{
    $foo = bar();
    if(!$foo){
        throw new Exception('Problems with bar()');
    }
    $aa = bb($foo);
    if(!$aa){
        throw new Exception('Problems with bb()');
    }
    //...and so on!
}catch(Exception $e){
    echo $e->getMessage();
}

#But i relly prefer to use something like:

try{
    $foo = bar() or throw new Exception('Problems with bar()');
    $aa = bb($foo) or throw new Exception('Problems with bb()');
    //...and so on!
}catch(Exception $e){
    echo $e->getMessage();
}

#Actually, the only way i figured out is:

try{
    $foo = bar() or throw new ThrowMe('Problems with bar()', 1);
    $aa = bb($foo) or throw new ThrowMe('Problems with bb()', 2);
    //...and so on!
}catch(Exception $e){
    echo $e->getMessage();
}

#But i'll love to thro the exception directly instead of trick it with ThrowMe function.

7条回答
\"骚年 ilove
2楼-- · 2020-02-01 00:02

I think you want to use something like the last structure, although there's really no point in using exceptions for that:

$foo = bar();
if(!$foo){
    echo 'We have a problem in here';
}

Per comment - I don't think you can do that in a single line (i.e. without the if(!$foo) check), and I agree that the exception throwing method is pretty horrible. Personally, I prefer the explicitness of:

$foo = bar();
if(!$foo){
    throw new Exception('We have a problem in here');
}

but that's a personal preference. If you want the single-line thing, I think you'll have to go with your exception-throwing function option.

I guess this limitation is probably down to PHP's dynamic-typing stuff, it can cast the results of a function call to a conditional, but not the results of a throw.

查看更多
仙女界的扛把子
3楼-- · 2020-02-01 00:06

A solution I found that can replace "or die()" everywhere is to wrap the throw with a anonymous function that gets called immediately by call_user_func:

call_user_func(function(){
    throw new Exception("ERROR");
});

You can see that it works by executing this on a dummy script:

false or call_user_func(function(){throw new Exception("ERROR");});
查看更多
【Aperson】
4楼-- · 2020-02-01 00:07

Here's a single-line solution without the extra function:

if (!($foo = bar())) throw new Exception('We have a problem here');
查看更多
成全新的幸福
5楼-- · 2020-02-01 00:10

I've simply defined the function toss for this.

function toss(Exception $exception): void
{
    throw $exception;
}

Because the file/line/stack information is captured when the exception is constructed (new) not thrown (throw) this doesn't interfere with the call stack.

So you can just do this.

something() or toss(new Exception('something failed'));
查看更多
再贱就再见
6楼-- · 2020-02-01 00:12

You can also create a custom exception class and use it's static constructor method instead of throw new Exception() construction.

Exception class:

class CustomException extends Exception {
  static public function doThrow($message = "", $code = 0, Exception $previous = null) {
    throw new Exception($message, $code, $previous);
  }
}

Usage:

try {

  $foo = bar() or CustomException::doThrow('Problems with bar()');
  $aa = bb($foo) or CustomException::doThrow('Problems with bb()');

} catch(Exception $e){
  echo $e->getMessage();
}

Note

If you are using PHP 7 and higher - you can rename static method doThrow() to simply throw(), since in PHP 7 and higher it's allowed to use reserved keywords as method names.

查看更多
淡お忘
7楼-- · 2020-02-01 00:21

or is just a logical operator, and it's analogous to ||.

The common trick of

mysql_query() or die();

could just as well be written

mysql_query() || die();

What happens here is the "logical or" operator (whichever you choose) is trying to determine if either operand evaluates to TRUE. This means the operands must be expressions that can be cast as a boolean.

So, the reason

bar() or throw new Exception();

is illegal, is because

(boolean)throw new Exception();

is also illegal. In essence, the process of throwing an exception doesn't generate a return value for the operator to check.

But calling a function does generate a return value for the operator to check (no explicit return value will result int the function returning NULL which casts as FALSE) which is why it works for you when you wrap exception throwing in a function.

查看更多
登录 后发表回答