Coding Problem!

viti

I am trying to query the username of a particular user who has posted at a particular time.

for ($i = 1; $i <= 10; $i++) {


$result = mysql_query("SELECT poster_id FROM posts WHERE post_time = 10:45-example");

if (!$result) {
    echo 'Could not run query: ' . mysql_error();
    exit;
}
$poster_id = mysql_fetch_row($result);


[B]$user_id = "$poster_id[0]";[/B]


$result = mysql_query("SELECT username FROM users WHERE id = $user_id");
if (!$result) {
    echo 'Could not run query: ' . mysql_error();
    exit;
}

$user = mysql_fetch_row($result);//username

echo "<tr><td>$user[0]</td></tr>";


}

The line which seems to be wrong is:

$user_id = "$poster_id[0]";

As I expect it "$user_id" should be an integer to use to compare this:

$result = mysql_query("SELECT username FROM users WHERE id [B]= $user_id[/B]");

I would be grateful if somebody would give me advice about that, since this code does not work.

viti

Installer

You probably need to enquote your values if they're date or string types.

$result = mysql_query("SELECT poster_id FROM posts WHERE post_time = '10:45-example'");

viti

That is not the problem, since it is just an example and on its place is in reality a variable. When I echo $poster_id[0] everyting is fine but when I try to use $poster_id[0] as a variable (it should give an integer?) later on it does not work.

thorpe

Probably wont help with your problem but this line...

$user_id = "$poster_id[0]";

Should be...

$user_id = $poster_id[0];

At least now your code makes better sense.

Now.... you need to explain what doesn't work meens. While your at it, could you explain why this is all wrapped in a for() loop?

viti

Actually that is only part of the code, so you can ignore the for loop.

One question: $poster_id[0] should return an integer, right? If I use for example $user_id = 4; , it works like it is supposed to work.

It is probably a syntax error, which I have in the code. The problem is that I cannot find it.

Thanks for the replies so far! 🙂

Installer

Well, if you post your actual code, instead of an "example" or "part of", we could probably help you better, without chasing red herrings.

Weedpacket

viti wrote:
One question: $poster_id[0] should return an integer, right?

Well, we didn't design the database. But if it's not an integer, what is it? Have you looked?

And you still haven't explained what "doesn't work" means.

I would have pointed out that both queries could have been combined into one, and then there wouldn't have been any need for these intermediate variables, but that's probably one of the red herrings Installer mentioned.

We need to see genuine code that exhibits the problem. If the code you post doesn't have the exact same problem that the real code has, then there is no point in posting it. (Note that we won't be able to run it ourselves either way, because we don't have your database. So you have to tell us what happens when you run it.)

viti

Actually I have solved the problem. The code syntax is correct. 🙂 The problem was that the first query did not return anything, so the following queries could not use the poster_id variable.

This is the only the "most part" of the code because I thought it was unnecessary to include the code for connecting to the database and final code for outputting the results on the webpage.

Thanks that you tried to help! 🙂