|
|
Posted by Colin McKinnon on 05/27/05 00:22
Brian spilled the following:
> Hi all
>
> whats wrong with this?
>
> if ( (strtolower($myrow[status]) != 'n') || (strtolower($myrow[status]) !=
> 'u') ) {
> .......
> }
>
> I'm trying to say if $myrow[status]) if not 'n' or 'u' then do X but it's
> just not
> playing ball here? I have tried putting in brackets, taking them out,
> moving them about and all sorts, just can't get it to work
>
For starters, the line is too long and unnecessarily re-evaluating the
function.
It's doing exactly what you tell it - you have a semantic error in your
program. Try making it a bit simpler and it might be clearer:
$ch = strtolower($myrow[status]);
if ( ($ch!='n') || ($ch!='u') ) { ...
At least one of the two expressions will always be true (since $ch cannot be
'u' and 'n' at the same time)
I suspect you really mean not 'n' *AND* not 'u'
HTH
C.
Navigation:
[Reply to this message]
|