Не правильно выводит данные о пользователе

@dukesov · Регистрация: 24.08.2015

Author24 — интернет-сервис помощи студентам

PHP

<?php
$query = mysql_query("SELECT * FROM `session` ORDER BY `putdate` DESC");
  $query1 = mysql_query("SELECT * FROM `users` WHERE `user_id`='$_COOKIE[I]'");
  $item1 = mysql_fetch_array($query1);
  while ($item = mysql_fetch_array ($query)){
?>
<tr class="text-center">
  <td><?=$item['id_session']?></td>
  <td><?=$item['link']?></td>
  <td><?php echo date("d.m.Y в H:i:s",strtotime($item["putdate"]));?></td>
  <td><?=$item1['username']?></td>
</tr>
<?php } ?>

Вот тут: <td><?=$item1['username']?></td> у всех пользователей почему то вводит один и тот же логин (мой), хотя в статистике (в бд) в данный момент всего 1 авторизованный пользователь. Помогите исправить.
В $_COOKIE['I'] ид пользователя.

@Пифагор · 26.03.2016, 21:21

Вероятно, из-за того, что в условии $_COOKIE[I] ключ массива без кавычек. Это же не числовой ключ.
Попробуйте так:

PHP

<?php
$I = $_COOKIE['I'];
$query = mysql_query("SELECT * FROM `session` ORDER BY `putdate` DESC");
$query1 = mysql_query("SELECT * FROM `users` WHERE `user_id`='$I'");
$item1 = mysql_fetch_array($query1);
  while ($item = mysql_fetch_array ($query)){
?>
<tr class="text-center">
  <td><?=$item['id_session']?></td>
  <td><?=$item['link']?></td>
  <td><?php echo date("d.m.Y в H:i:s",strtotime($item["putdate"]));?></td>
  <td><?=$item1['username']?></td>
</tr>

@dukesov · 26.03.2016, 21:40 **[ТС]**

Пифагор, все равно так-же, 7 пользователей на сайте, я один авторизован. У всех пишет мой логин.

@Para bellum · 26.03.2016, 21:43

Что содержится в таблице "session"?

@dukesov · 26.03.2016, 21:53 **[ТС]**

Para bellum, ид пользователя, если зарегистрирован, дата, ип адрес, ссылку где находится.

@Para bellum · 26.03.2016, 21:59

Попробуйте так:

PHP/HTML

<?php
$query = mysql_query(
    "SELECT `s`.*, `u`.`username` 
        FROM `session` `s` 
        LEFT JOIN `users` `u` 
    ON `u`.`id`=`s`.`user_id` 
    ORDER BY `putdate` DESC"
);
 
while ($item = mysql_fetch_array ($query)):
?>
<tr class="text-center">
  <td><?=$item['id_session']?></td>
  <td><?=$item['link']?></td>
  <td><?php echo date("d.m.Y в H:i:s",strtotime($item["putdate"]));?></td>
  <td><?=$item['username']?></td>
</tr>
<?php endwhile;?>

Добавлено через 51 секунду
1) Поменяйте имена полей на свои, если они не совпадают с указанными в примере.
2) mysql_* функции устарели.

@dukesov · 26.03.2016, 22:59 **[ТС]**

Para bellum, все, разобрался, добавил условие:

PHP

<td><?=$item['id_session']?></td>
  <td><?=$item['link']?></td>
  <td><?php echo date("d.m.Y в H:i:s",strtotime($item["putdate"]));?></td>
  <td><?php if($item['user'] == $item1['user_id']){ echo $item1['username'];}?></td>

Добавлено через 52 минуты
Para bellum, нет, если второй авторизованный пользователь заходит, у него отображает себя, у меня отображает меня... То-есть мы друг друга не видим...
А Ваш вариант выдает ошибку:
Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in в строке while

Добавлено через 1 минуту
Мне просто не хочется в куках держать логин пользователя... Вот и хочу сделать сравнение, что user в таблице session равен user_id в таблице users. И вывести вместо id, что в куках, логин пользователя...

Добавлено через 4 минуты

PHP

<?php
$query = mysql_query("SELECT * FROM `session` ORDER BY `putdate` DESC");
  while ($item = mysql_fetch_array ($query)){
      $q = mysql_query("SELECT * FROM `users` WHERE `user_id`='$item[user]'");
        $item1 = mysql_fetch_array ($q);
?>
<tr class="text-center">
  <td><?=$item['id_session']?></td>
  <td><?=$item['link']?></td>
  <td><?php echo date("d.m.Y в H:i:s",strtotime($item["putdate"]));?></td>
  <td><?=$item1['username']?></td>
</tr>
<?php } ?>

Вот так нормально, не знаю правильно или нет. Но отображает.

@Para bellum · 27.03.2016, 06:35

Я же Вам вроде показал, как нормально.

@dukesov · 27.03.2016, 11:25 **[ТС]**

Para bellum, ошибку выдает

@Para bellum · 27.03.2016, 12:51

Ну так показывайте, что за ошибка.

@dukesov · 27.03.2016, 13:53 **[ТС]**

Para bellum,

Сообщение от dukesov

А Ваш вариант выдает ошибку:
Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in в строке while

@Para bellum · 27.03.2016, 14:15

Сделайте так:

PHP

$query = mysql_query(
    "SELECT `s`.*, `u`.`username` 
        FROM `session` `s` 
        LEFT JOIN `users` `u` 
    ON `u`.`id`=`s`.`user_id` 
    ORDER BY `putdate` DESC"
) or die(mysql_error());

И скажите, что выведет.

@dukesov · 27.03.2016, 14:35 **[ТС]**

Para bellum, все, разобрал ошибку. Спасибо!

Добавлено через 5 минут
Para bellum, а в чем отличие моего варианта от вашего?

@Para bellum · 27.03.2016, 14:44

В том, что мой вариант выбирает для каждого пользователя своё имя, а не одно и то же для всех.

@dukesov · 27.03.2016, 15:13 **[ТС]**

Para bellum,

Сообщение от dukesov

$query = mysql_query("SELECT * FROM `session` ORDER BY `putdate` DESC");
* while ($item = mysql_fetch_array ($query)){
* * * $q = mysql_query("SELECT * FROM `users` WHERE `user_id`='$item[user]'");
* * * * $item1 = mysql_fetch_array ($q);

PHP

[quote="Para bellum;8941204"]$query = mysql_query(
* * "SELECT `s`.*, `u`.`username` 
 * * * *FROM `session` `s` 
 * * * *LEFT JOIN `users` `u` 
 * *ON `u`.`id`=`s`.`user_id` 
 * *ORDER BY `putdate` DESC"
)[/quote]

Имею ввиду вот эти 2.

@Para bellum · 28.03.2016, 06:30

Ааа. Тут различие в том, что мой запрос будет работать гораздо быстрее, так как делает всё разом. А в Вашем случае идёт множество запросов.
Например, если сейчас онлайн 100 пользователей — вариант с JOIN вытащит их из базы за 1 запрос.
А вариант с циклом вытащит данные за 101 запрос (первый — это записи из "session", а остальная сотня — запросы на выборку имени).

Одним словом, нужно всегда подыскивать менее ресурсоёмкий вариант.

@dukesov 19 / 18 / 11 Регистрация: 24.08.2015 Сообщений: 471
	26.03.2016, 21:40 [ТС]	3
	Пифагор, все равно так-же, 7 пользователей на сайте, я один авторизован. У всех пишет мой логин. 0

@Para bellum 5755 / 4134 / 1508 Регистрация: 06.01.2011 Сообщений: 11,276
	26.03.2016, 21:43	4
	Что содержится в таблице "session"? 0

@dukesov 19 / 18 / 11 Регистрация: 24.08.2015 Сообщений: 471
	26.03.2016, 21:53 [ТС]	5
	Para bellum, ид пользователя, если зарегистрирован, дата, ип адрес, ссылку где находится. 0

@Para bellum 5755 / 4134 / 1508 Регистрация: 06.01.2011 Сообщений: 11,276
	27.03.2016, 06:35	8
	Я же Вам вроде показал, как нормально. 0

@dukesov 19 / 18 / 11 Регистрация: 24.08.2015 Сообщений: 471
	27.03.2016, 11:25 [ТС]	9
	Para bellum, ошибку выдает 0

@Para bellum 5755 / 4134 / 1508 Регистрация: 06.01.2011 Сообщений: 11,276
	27.03.2016, 12:51	10
	Ну так показывайте, что за ошибка. 0

@dukesov 19 / 18 / 11 Регистрация: 24.08.2015 Сообщений: 471
	27.03.2016, 13:53 [ТС]	11
	Para bellum, Сообщение от dukesov А Ваш вариант выдает ошибку: Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in в строке while 0

@dukesov 19 / 18 / 11 Регистрация: 24.08.2015 Сообщений: 471
	27.03.2016, 14:35 [ТС]	13
	Para bellum, все, разобрал ошибку. Спасибо! Добавлено через 5 минут Para bellum, а в чем отличие моего варианта от вашего? 0

@Para bellum 5755 / 4134 / 1508 Регистрация: 06.01.2011 Сообщений: 11,276
	27.03.2016, 14:44	14
	В том, что мой вариант выбирает для каждого пользователя своё имя, а не одно и то же для всех. 0

@Para bellum 5755 / 4134 / 1508 Регистрация: 06.01.2011 Сообщений: 11,276
	28.03.2016, 06:30	16
	Ааа. Тут различие в том, что мой запрос будет работать гораздо быстрее, так как делает всё разом. А в Вашем случае идёт множество запросов. Например, если сейчас онлайн 100 пользователей — вариант с JOIN вытащит их из базы за 1 запрос. А вариант с циклом вытащит данные за 101 запрос (первый — это записи из "session", а остальная сотня — запросы на выборку имени). Одним словом, нужно всегда подыскивать менее ресурсоёмкий вариант. 0