if/else problems

[The Little Guy]

It goes into the else on where it says if($cat != 0) and $cat is equal to Music when I echo it out.

 

I can't see what I am doing wrong (it is the 3rd if statement)....

 

$title = strip_tags($_POST['title']);
$description = strip_tags($_POST['description'],'<a>');
$cat = $_POST['category'];
$vidExt = $_POST['videoType'];
$good = FALSE;
if(strlen($title) > 3){
    if(strlen($description) > 19){
        if($cat != 0){
            if(in_array($vidExt,$fileTypes)){
                $good = TRUE;
            }else{
                $good = FALSE;
                $r = 'fType';
            }
        }else{
            $good = FALSE;
            $r = $cat;
        }
    }else{
        $good = FALSE;
        $r = 'descript';
    }
}else{
    $good = FALSE;
    $r = 'title';
}
echo $r;

[n3ightjay]

You 3rd if is comparing incomparable types (I don't know if thats the actuall definition of the solution but this works i tested it) ??? if you change it to

if($cat != ""){

or

if($cat != "0"){

 

it will work

[n3ightjay]

oh oh oh ... or

 

if(strlen($cat) != 0){

[The Little Guy]

0 is one of the possible values passed through...

 

But... I am giving $cat != "0" a try

[DarkWater]

How about:

if (!empty($cat)) { }

[n3ightjay]

damn ... im sure theres a million more ....

 

technically this qualifies as well

 

if(isset($cat)){

[DarkWater]

isset() is not the same as !empty().  Since he has $cat = $_POST['category';, it'll be set no matter what.  Example:

 

<?php
$foo = '';
var_dump(isset($foo));

 

Outputs:

bool(true)

[The Little Guy]

putting quotes around the zero worked

 

I now can't get the last if statement to work

 

I realized that the variable for the second value in the in_array function was spelled wrong, so I fixed it.

 

ill see what value is being passed to it.

[n3ightjay]

isset() is not the same as !empty().  Since he has $cat = $_POST['category';, it'll be set no matter what.  Example:

 

<?php
$foo = '';
var_dump(isset($foo));

 

Outputs:

bool(true)

 

Good point got excited in trying to find multiple ways of answer the same question... assumed checking against zero was checking if it was empty ... therefore leading me down the path to isset()

[redarrow]

use a switch ...........