[RESOLVED] Question about an If....

bwoogie

is there a way to set a variable in an if statement if the result of a function is not true?

if(!myfunction($a, $b, $c, $d))
{
    echo myfunction($a, $b, $c, $d);
} 
else
{ 
    //do something else
}

What I'm asking is, is it possible to shorten this so I dont have to call myfunction twice.

intenz

if( !( $var = myFunc( $a, $b, $c, $d ) ) ) { }

bwoogie

Ah! Thank you very much!

intenz

Remember to mark thread as resolved in "Thread Tools" 🙂

bwoogie

Ok, so i hit resolve before it truly was... o_O
Ran into a problem.

That works... How do I check if $var is returning true?
What exactly is happening there in that code? Is it checking the value of $var or that $var was set to myFunc's result successfully?
myFunc returns true on success or a string $err if it fails. But that statement is always returning true no matter what myFunc returns. It never triggers else{}

Weedpacket

bwoogie wrote:
Is it checking the value of $var or that $var was set to myFunc's result successfully?

An assignment operation [font=monospace]=[/font] evaluates to the value of whatever is on its right-hand side. At the same time, it sets the variable on the left hand side to the same value. So the if() test is checking the value of whatever was put into $var.

bwoogie wrote:
myFunc returns true on success or a string $err if it fails. But that statement is always returning true no matter what myFunc returns.

Only a few values are considered equivalent to false in an if() test; everything else is true.

If the function returns values of different types depending on what happened, you can tell what happened by looking at the type of the returned value.

$var = myFunc($a, $b, $c, $d);
if(is_string($var))
{
    // $var contains an error message
}
else
{
    // $var contains something else (a boolean, presumably)
}

bwoogie

K, I got it all figured out now I guess.... I change true to 1 and it works. I thought they were equivalent? But oh well.

if(($var=myFunc($a, $b, $c, $d))!=1){//handle false} else {//handle true }