Function checking

[lilRachie]

I'm getting

Warning: array_key_exists() expects parameter 2 to be array, null given in /home/blay/blah/blah/blah/functions.php on line 5

function check_exist($key,$arr){
	if(array_key_exists($key, $arr)){ //line5
	return mysql_real_escape_string(mb_strtolower($arr[$key]));  

	}else{
	return NULL;
	}
}

Anyone see what's up there? All it does is checks the existence of a key and returns its value scrubbed to be SQL safe.

it appears as many times as I have rows in my script.

lilRachie

[jeepin81]

what do you get if you

print_r($arr); 

?

[NogDog]

Whatever is being passed as the 2nd parameter to the function is not an array. You could add some defensive coding to the function such as with [man]is_array/man, and perhaps use [man]debug_backtrace/man in the error branch of that check to capture where the offending call to the function is being made.

[lilRachie]

ok i put

function check_exist($key,$arr){
	if(array_key_exists($key, $arr) AND is_array($arr[$key])){  //line5
	return mysql_real_escape_string(mb_strtolower($arr[$key]));  

	}else{
	return NULL;
	}
}

Warning: array_key_exists() expects parameter 2 to be array, null given in /home/blah/blah/blah/blah/functions.php on line 5

This in an included function file on a db update table

lilRachie

[NogDog]

You need to do the is_array() test before the array_key_exists() test.Also, the arg for is_array(should just be the array variable name -- no array index. You can just swap the order in your code, or nest them in order to better indicate what happens on each call.

function check_exist($key,$arr){
    if(is_array($arr)) {
        if(array_key_exists($key, $arr)
            return mysql_real_escape_string(mb_strtolower($arr[$key]));  

        }else{
            return NULL;
        }
    } else {
        $err = debug_backtrace();
        user_error("<pre>".print_r($err, 1)."</pre>");
        return false;
    }
}

Alternatively, if using PHP 5.3 or later and if you do not need to support older versions, you can use type-hinting (which will throw an exception if the type is not valid):

function check_exist($key, [B][COLOR="Red"]array[/COLOR][/B] $arr) {

[lilRachie]

Nog I tried both

Using the

function check_exist($key,$arr){
    if(is_array($arr)) {
        if(array_key_exists($key, $arr)
            return mysql_real_escape_string(mb_strtolower($arr[$key]));
        }else{
            return NULL;
        }
    } else {
        $err = debug_backtrace();
        user_error("<pre>".print_r($err, 1)."</pre>");
        return false;
    }
} 

I got this
Parse error: syntax error, unexpected T_RETURN in /home/blah/blah/blah/blah/functions.php on line 7

[NogDog]

Missing closing paren on the inner IF.

[lilRachie]

oh dear that's interesting

mailing you result

[lilRachie]

also upon investigating the db it removed values instead of adding the added 1