I have a question about the order of operations in a condition of an if statement.

If I have a line like this:


Is it reasonably safe to assume that the two methods will be called in the order that they are listed in the condition?

--- OR ---

Would it be safer to do something like this?


Thanks for any help!

__________________
-- Bill
"Why is it drug addicts and computer aficionados are both called users?" -Clifford Stoll

I have never tried this before, but you can do a quick test to see.
See what that does for you.

__________________
There are No Stupid Questions. But there a LOT of Inquisitive Idiots.

When it comes to conditional statements, you should always use order of precedence - if the first criteria in an AND or && operation is not met, it will not procede to the second.
-m

This runs funcOne then funcTwo correctly.

So it does follow an order of precedence... Good. Thanks folks!

The code just looks a little better to me as I do not have to nest layers and layers of if's. The thing with this section is that it needs to go in this order as the code calling the ftp wrapper class I wrote changes to the correct directory and then deletes a file. I wanted to make sure that the directory change always happens first as the delete will fail if it does not.

Thanks again!

__________________
-- Bill
"Why is it drug addicts and computer aficionados are both called users?" -Clifford Stoll

Hmm, in the case that you describe, it may actually be better to do a nested situation for proper error handling.

ie

if (changeDirectory())
{
deleteFile();
}
else
{
// an error occured changing to the correct directory.
}

I am not sure what the error rate is on changing directories (if there is one at all), but this is just good coding practices :D

__________________
There are No Stupid Questions. But there a LOT of Inquisitive Idiots.

The error handling is one thing that I had considered, but due to the application's user base, there is really not an issue with them needing to know what the error actually was.

It is a group of mostly computer-illiterate salesmen who, despite the presence of any kind error message, will continue to try and delete the file/report they are looking at. They will never call regarding the error. I know them well!

What I may do is nest them anyway. It is good practice, and I may get one user to call. It will be easier to find a problem if it occurs.

Thanks!

__________________
-- Bill
"Why is it drug addicts and computer aficionados are both called users?" -Clifford Stoll

Don't worry about the order in which the conditions are evaluated. But if you asked, I'll answer: the order in which the methods return types is checked is the same for both cases. The difference is that in the first case, both conditions are evaluated, while in the second case (nested), if the first condition fails, the parser will never hit the second condition. How you check the conditions is entirely up to you, depending on what you need to accomplish.

__________________

@xenon:

What you say seems to contradict what delayedinsanity says.

The thing is - I do care what order the methods are called. The script needs to change to the proper directory THEN try to delete the file. If it does not, then the delete will fail. Regardless, I nested them to provide better error handling/reporting and render the possible precedence problem irrelevant.

I guess the real question is: does PHP do short-circuit evaluation? I did a little Googling and found that it does. My original statement will work, but the error handling left a bit to be desired...

__________________
-- Bill
"Why is it drug addicts and computer aficionados are both called users?" -Clifford Stoll

I think what he's trying to say is that;

The only time it would make a real difference is if you need to run an operation after the first condition is met and before the second is checked.
-m

Yeah, that seems pretty logical. I was using the principle 'false && false === true', but it seems that the above statement evals to false, so...sorry.

__________________

Exactly. I really do not need to run that operation in the middle, but I am going to. It does make for better error handling.

There is no need to apologize xenon... I wish I could have conveyed it better.

__________________
-- Bill
"Why is it drug addicts and computer aficionados are both called users?" -Clifford Stoll