PHP from xxx where code error plz help !

I'm no php person, but at a glance it looks like you have an extra " in there

Try this line instead:

$a = mysql_query('select * from users where oauth_token <> "" order by rand() limit ".$screen_name."');

or mask the "

Don't tell him the query's quotations are incorrect when they're not. They are fine.

I'd like to know what the $screen_name variable is, though. Your query is trying to limit by the screen name ("ORDER BY rand() LIMIT $screen_name") but you can only limit the query by numbers, i.e. only return a certain number of rows: ("ORDER BY rand() LIMIT 5") would be a proper example of using limit.

I imagine this is probably calling the Twitter API? From my experience, the OAuth info does not change, so I imagine your query is trying to retrieve the OAuth codes for a user?

If so, your query would simply be:
("SELECT * from `users` where `oauth_token`<>'' AND `screen_name`='" . $screen_name . "'")

Assuming the field for $screen_name is called screen_name

That's only if $screen_name is holding a username (like my "screen name" would be artizhay). Your for-loop implies that $screen_name is a number, although the variable name implies text data.

Let us know what these variables are first. Also if the script is throwing an error you should post that as well. Given the current code snippet, there is no established MySQL connection, so I don't know if this is omitted or if you forgot it. Please provide any errors you've been given.

zelma143 said:

try this
$a = mysql_query("select * from users where oauth_token <> order by rand() limit '$screen_name'");

Click to expand...

This would cause an error because the <> operator now has no data to compare oauth_token to, and the limiting offset is now in single quotes, which is improper. You people need to stop giving improper responses if you don't know what you're talking about.

Please, don't use mysql_* functions for new code. They are no longer maintained and the php community has begun the deprecation process. See the red box? Instead you should learn about prepared statements and use either PDO or MySQLi. If you can't decide, this article will help to choose. If you care to learn, here is good PDO tutorial.

Like art mentioned, you are also having extra " at your query:


Should be:

Keep in mind that limit is a number either single or comma separated as in 1,10 that means start and amount and does not work under single quotes and if you do not sanitize it prior to using it you may suffer from sql injections.

For instance let's say you reuse the limits on your url like:

Then some one could go for something like:

By using mysqli or PDO with prepared statments you will avoid this cases at easy.