trouble with mysql_num_rows()

[phence]

I am getting this warning

Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in /home/www/tacit-design/trade/singleSearch.php on line 36

form the following code:

---

....

// query table
LINE 34: $query = "SELECT * FROM listings WHERE MATCH $trading OR wac_1 OR wac_2 AGAINST $searchValue";

LINE 35: $result = mysql_query($query);

LINE 36: $rowsFound = mysql_num_rows($result);

// if the query has results ...

if($rowsFound > 0){  
                   ...

---

and no results are found no matter the $search value

Can u see any problems with the code?

[thecookie]

Not sure, but I think you have to do rows first.. because fetch is emptying the array

[mrmufin]

The proper way to do it is to test for a result handle before doing anything with the result:

// Build a query:
$query = "SELECT columns FROM table WHERE conditions exist";

// Run the query. If the query fails, it won't return a result set identifier:
$result = mysql_query($query);

if(!$result) {
	echo "Query $query failed.";

// If the query failed, MySQL is usually friendly enough to offer a reason why:
echo mysql_error();
} else {
	// Just because a query executed succussfully doesn't necessarily mean there are rows in the result set.
	// So test the number of rows in the result set:
	if(!mysql_num_rows($result)) {
		echo "Query $query returned no results.";
	} else {
		while($data = mysql_fetch_array($result)) {
			// process the data array for each row in the result set here...
		}
	}
}

[Mega]

it may be because no results are being returned.

$result = mysql_query("SELECT * FROM listings WHERE MATCH $trading OR wac_1 OR wac_2 AGAINST $searchValue") or die( mysql_error() ); 

$rowsFound = mysql_num_rows($result) or die( mysql_error() ); 

please see if your query gives you any errors.

and Mr muffin:

// Build a query:
$query = "SELECT columns FROM table WHERE conditions exist";

Commenting EVERYTHIN without leaving a standard number of linebreaks before or after EVERY comment makes your code a lot more difficult to read.

I mean a duck could see your building a query, I don't think it warrants being commented.

🙂