[SOLVED] coders block (and issue)

[Q695]

There is no data, so shouldn't it be saying that you don't have anything in a way instead of death?

output:SELECT * FROM `map` WHERE `location_x`='-3' AND `location_y`='3' AND `level`='1';

<?php
function loc_img ($mx, $my, $location_x, $location_y){
global $location_x, $location_y, $level;
	$location2_x=$location_x+$mx;
	$location2_y=$location_y+$my;
	$sql="SELECT * FROM `map` WHERE `location_x`='$location2_x' AND `location_y`='$location2_y' AND `level`='$level';";
	$result=@mysql_query($sql,$con) or die(death($sql));
	$m_row=mysql_fetch_array($result);
}
?>

[Mchl]

Why don't you display mysql_error?

[corbin]

Just a guess, but I don't see $con set anywhere.

[Q695]

$con=//universal connection to database

 

death doesn't output an error, even though it has a mysql_error() built in.

[corbin]

Errr....  You're not declaring it global.

[Mchl]

At least remove the error suppresion from mysql_query

[PFMaBiSmAd]

A query that executes but has zero rows in the result set is a successful query. Only queries that fail to execute return a FALSE value that would trigger an or die() statement.

 

To test if there were any rows returned you would need to use mysql_num_rows() or put the $m_row=mysql_fetch_array($result) inside of a conditional statement -

 

if($m_row=mysql_fetch_array($result)){
    // fetch worked, process $m_row here
} else {
   // no row to fetch
   echo "zero matching rows<br />";
}

[Q695]

Thanks, that worked PFM .