php mysql AND operator

Question

I get a error message saying:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'key='12345' AND id='98765' LIMIT 1' at line 1

My code is:

$key = '12345'; 
$id = '98765';  
include realpath('./inc/config.php');

            $query = mysql_query("SELECT * FROM users WHERE key='{$key}' AND id='{$id}' LIMIT 1", $config) or die(mysql_error());
            $result = mysql_fetch_assoc($query);

Now can anyone tell me whats wrong in this?

I am not certain but are you sure you have used proper quotes for key (it appears to me as non-digit data)??? — Umer, Nov 24 '11 at 05:39
Damn sorry guys key is a predefined function in mysql or something like that you know primary key and all so it gives and error i changed it from key to keys and it worked and its not necessary to put braces?? EDIT: No sorry anything except key even keys are not working — Keshav Nair, Nov 24 '11 at 05:45

score 9 · Accepted Answer · answered Nov 24 '11 at 05:44

9

key is a reserved word, you need to properly quote it with backticks if you want to use it as a field name.

SELECT * FROM users WHERE `key`='{$key}' AND id='{$id}' LIMIT 1

answered Nov 24 '11 at 05:44

Maerlyn

33,687
18
94
85

+1 You beat me to it by a few minutes. Oh, how I hate my net connection :P – Pranav Hosangadi Nov 24 '11 at 05:49

Ghostman · Answer 2 · 2011-11-24T08:37:37.680

1

SELECT * FROM users WHERE `key`='{$key}' AND id='{$id}' LIMIT 1

key is a reserved word

edited Nov 24 '11 at 08:37

answered Nov 24 '11 at 05:37

Ghostman

6,042
9
34
53

try running that query in ur mysql database directly and try by taking off ' ' too and run it – Ghostman Nov 24 '11 at 05:42

score -2 · Answer 3 · answered Nov 24 '11 at 05:48

Your key and id are obviously numeric. Although adding quotes wouldn't hurt, you definitely don't need them. You edon't need brackets in any query period. Try this:

        $query = mysql_query("SELECT * FROM users WHERE key=$key AND id=$id LIMIT 1", $config) or die(mysql_error());
        $result = mysql_fetch_assoc($query);

If that doesn't work just run this using PHPMyAdmin or whatever you use to run queries on your db.

         SELECT * FROM users WHERE key=12345 AND id=98765

I also don't see why you would need LIMIT clause. It wouldn't break anything but if your id is actually row id it should give you a unique record.

php mysql AND operator

3 Answers3