Order By

[poreko]

Hi

I am trying to order the following query by price when the varaible $concern=="price"; But it is not working. Here is my code.

 
 
    for($key=0 ;$key<count($first);$key++)
  {
   if(($first[$key])>0.1){
            $blank++;
            $set=$key+1;
        if($concern=="price"){
            $sql2 = "SELECT * FROM laptop2 WHERE CODE=$set ORDER BY PRICE";
            $query2 = $DB->Query($sql2);
           }
 
 echo'<p> <hr noshade="noshade" /></p>';
   }
 
               if($array = $DB->FetchArray($query2)){
 
    echo "<table  bgcolor=LightGrey border=1>\n";
    echo"<tr><td>ID</td><td>Brand</td><td>Model</td><td>Processor (GHz)</td>
    <td>Hard Drive (GB)</td><td> Graphic card (MB)</td>
    <td>Battery life(Hours)</td><td>Multimedia</td><td>Price (£)</td></tr>\n";
  // do {
 
 printf("<tr><td>%s</td><td>%s</td><td>%s</td><td>%s</td><td>%s</td><td>%s</d><td>%s</d><td>%s</d><td>%s</d> </tr>",$array['CODE'],$array['BRAND'],$array['MODEL'],$array['PROCESSOR (Ghz)'],
            $array["STORAGE (GB)"],$array["GRAPHIC CARD (MB)"],$array["BATTERY"],$array["MULTIMEDIA"],$array["PRICE"]);
     // print_r($array);
 
    }while($array=$DB->FetchArray($query2));
   echo"</table>\n";
 

Can you spot something?

Thanks

[Markus]

I doubt your column 'price' is uppercase (PRICE - like you wrote in your query). Make it lowercase. Also, for the future's sake, you should *really* be using back-ticks around your column names and table names.

 
SELECT
    *
FROM
    `laptop2`
WHERE
    `code` = '{$set}'
ORDER BY
    `price`
 
 

However, your columns may be uppercase and, if so, disregard my suggestion.

[poreko]

Thank you for your suggestions but in my columns price is uppercase.

[Markus]

Does your $DB->query() return any errors? Try echo()ing mysql_error() after you have performed the query.

[poreko]

$DB->query() does not return errors. What do you mean by echo()ing mysql_error()?

[Markus]

You can get the last error generated by mysql by mysql_error(). So, after the query, do this:

 
die(mysql_error());
 
 

[Atli]

That would be your standard:

$result = mysql_query($sql) or die(mysql_error());

If you have created yourself a class to handle the database interactions, you should add something to capture the error in case the query fails.

Edit: Seconds to late :]

[poreko]

I already have that added it my class

[Markus]

OK. What exactly is not working?

[poreko]

It is not sorting it by price, the results are still random.