[flippin_insane]

Hi everyone. I'd like to ask some help as I am getting this error using my php code.

Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in /Applications/XAMPP/xamppfiles/htdocs/lis 160 database/order.php on line 68

this is the php code im currently working on and I'm trying to print results from a the comment column, in the same row as the query entered in the search bar. Can anyone please help me find my error?

<?php

				$connection = mysql_connect("localhost","root","") or die("Couldn't connect");

				mysql_select_db("saver") or die("Could not open database");

				$xquery = $_REQUEST['query'];
				
				$result = mysql_query("SELECT comment * FROM  shirt WHERE (name LIKE '$xquery') ");

				
				while($row = mysql_fetch_array($result))
				
					{
				 		echo $row['comment'] ;
					}

				?>

[creativecoding]

Try the script from here:
http://www.designpla...?page=1&c_id=25

[Jstall]

You are getting that error because there is something wrong with your query, most likely a syntax error. It is often helpful in these situations to store the query itself as a string so you can echo it out if you get an error :

$sql = "SELECT * from table";
$result = mysql_query($sql);

Your query is pretty simple, maybe remove the * so it is just

 "SELECT comment FROM shirt WHERE name LIKE '$xquery'"

If that doesn't work it could be something like trying to reference a field in the table that doesn't exist, try echoing out the $sql variable and running the query directly and see why kind of error mysql gives you.

[Valek]

That's exactly what it is. SELECT comment * is invalid syntax. The or die() thing suggested is something you will hear parroted on these forums constantly. It is gospel. Never simply assume your query is infallible.

Are you trying to select just the comment field, or all fields? Something worth thinking about when building your query.

[JackOfAllTrades]

There should be some Javascript added to this board (and every board that does PHP) which when detecting "expects parameter 1 to be a resource" in the text pops up a "HEY! DON'T ASSUME YOUR QUERY ALWAYS SUCCEEDS!!!" message with a complete explanation. Add the same functionality for "headers already sent by" messages and like 50% of PHP questions in forums would likely be fixed without wasting DB space on them.

Of course, that's also the function of the traditional FAQ, but who actually READS FAQs anymore?

[flippin_insane]

Thanks a lot guys! I just had to remove the * and it worked!

[JackOfAllTrades]

Cool! Now take all the information you received here regarding debugging SQL queries and use it in the future! There's also another problem here, and that's that you are taking the input and using it directly in your query which likely leaves you wide open for SQL injection attacks.

Here's the way I would do this:

<?php
$connection = mysql_connect("localhost","root","") or die("Couldn't connect");

mysql_select_db("saver") or die("Could not open database");

// $_REQUEST should probably be changed to be specific to
// either $_GET or $_POST, depending on its origin.
$xquery = mysql_real_escape_string($_REQUEST['query']);
               
$query = "SELECT comment from shirt WHERE (name LIKE '$xquery')"; 
$result = mysql_query($query);
if (!$result)
{
    // This should be logged to an error log rather than
    // using die() in a live, non-development environment
    die("Query $query failed: " . mysql_error());
}
               
while($row = mysql_fetch_array($result))
{
    echo $row['comment'] ;
}
?>