if statement ... problems.

Fritz

I am desparate... and what should be simple isn't...

when I try to upload a .jpg file

this works... and ONLY works for .jpg (good so far)

<?php

$ext = strtolower(strrchr($photo_name,'.'));

if ( $ext != '.jpg' )  {error stuff} else{good stuff}
?>

IF I change it to .gif it works and ONLY works for a .gif... (again GOOD)

YET... If I check that it is a .gif OR a .jpg IT Returns FALSE or either type of file

<?php
//ask that it IS a .jpg  or a .gif 
if ( $ext != '.jpg' || $ext != '.gif' )  {error stuff} else {good stuff}
?>

I have even tried using 'or' instead of || and doesn't accept either a .gif or .jpg...

Help please.

ahundiak

if (($ext == '.jpg') || ($ext == '.gif'))
{
/ Do good stuff /
}
else
{
/ Error stuff /
}

Fritz

ahundiak? thanks for the reply...

i DID try nesting it and still had the same problems.

Originally posted by Fritz
I am desparate... and what should be simple isn't...

when I try to upload a .jpg file

this works... and ONLY works for .jpg (good so far)
<?php

$ext = strtolower(strrchr($photo_name,'.'));

if ( $ext != '.jpg' )  {error stuff} else{good stuff}
?>
IF I change it to .gif it works and ONLY works for a .gif... (again GOOD)

YET... If I check that it is a .gif OR a .jpg IT Returns FALSE or either type of file
<?php
//ask that it IS a .jpg  or a .gif 
if ( $ext != '.jpg' || $ext != '.gif' )  {error stuff} else {good stuff}
?>
I have even tried using 'or' instead of || and doesn't accept either a .gif or .jpg...

Help please. [/B]

ahundiak

if ( $ext != '.jpg' || $ext != '.gif' ) {error stuff} else {good stuff}

What the above is saying is that if $ext does not equal .jpg or $ext does not equal .gif then its an error. Well, $ext only has one value so it's always going to be not equal to one of them and hence the statement is always true.

What I posted was
if (($ext == '.jpg') || ($ext == '.gif'))
Which says to do something if $ext is equal to either jpg or gif. See the difference?

Fritz

Originally posted by ahundiak
if ( $ext != '.jpg' || $ext != '.gif' ) {error stuff} else {good stuff}

What the above is saying is that if $ext does not equal .jpg or $ext does not equal .gif then its an error. Well, $ext only has one value so it's always going to be not equal to one of them and hence the statement is always true.

What I posted was
if (($ext == '.jpg') || ($ext == '.gif'))
Which says to do something if $ext is equal to either jpg or gif. See the difference?

Agreed... When I reverse the if/then logic it DID work (nested or not) -
if .jpg OR .gif (good) else (error)

What I find curious is to why it won't work as I stated...

if not .jpg or not .gif (error) else (good)

thoughts?

ahundiak

Like I said, it's always not going to be one of them.