Как сделать запрос SQl внутри WHILE другого запроса?

Question

[Коля Унгуряну]

Всем привет, есть отрывок кода

<?php
$res = mysql_query($query = "SELECT * FROM `mails` WHERE u_id='1'") or die(mysql_error());
while ($row = mysql_fetch_object($res)) {
	
	$res1 = mysql_query("SELECT * FROM `user` WHERE id='$row->o_id'") or die(mysql_error());
	$data1 = mysql_fetch_row($res1); 
	$row->names = $row->$data1['name'];
	$data_log[] = $row;
};
?>

и выводит он примерно такое содержимое:

<?php foreach ($data_log as $key=>$item) {?>
<tr>
	<td><a href="?mail=<?php  echo $item->id; ?>"><?php  echo $item->tema; ?></a></td>
	<td><?php  echo $item->names; ?></td>
	<td></td>
</tr>
<?php $key+1;}; ?>

Проблема в том что первый запрос все выводит нормально, а второй ничего не выводит, тыкните носом пожалуйста!!

Answer 1

[Талгат Балтасов]

вот так попробуйте $row->names = $data1['name'];

Comments

[Коля Унгуряну]

не сработало

[Талгат Балтасов]

Коля Унгуряну: SELECT * FROM mails as m JOIN user as u on u.id = m.o_id WHERE m.u_id='1'

Answer 2

[Алексей Скобкин]

А почему бы не сделать JOIN вместо того, чтобы делать циклические запросы, которые всё сильно замедлят?

Comments

[Коля Унгуряну]

я так скажем только начинаю изучения PHP и еще не все фишки знаю, про JOIN не знал

[Алексей Скобкин]

Коля Унгуряну: Почитайте про SQL. Это будет очень полезно и не придётся писать бесполезный и медленный код на PHP.

Answer 3

[xmoonlight]

Что мешает сделать один запрос?

Comments

[Коля Унгуряну]

там из первого запроса выводится строка, из данной строки берется ID пользователя и во втором запросе по этому ID выводится имя пользователя

[Алексей Скобкин]

Я уже выше написал про JOIN.

Answer 4

[murrometz]

Ошибка в этой строчке
$row->names = $row->$data1['name'];

P.S. В качестве дополнения, не пойму, зачем вы в цикле получаете user'a, если достаточно получить один раз ..
Насколько я понял по коду

Comments

[Коля Унгуряну]

нет, там юзер может менятся, пару секунд, отредактирую код

Answer 5

[chezzy]

$row->$data1['name'];

Comments

[Коля Унгуряну]

не работает так

Answer 6

[Макс]

а можно по-подробнее про цель действа? Может проще сделать правльный SQL запрос (что-то мне подсказывает, что тут попахивает JOINом)?

И завязывайте с mysql_query. оно скоро умрет.

PS выполнят второй запрос в консоли mysql пробовали? может у вас нет ни одной записи в user с id = 1

Comments

[Коля Унгуряну]

Цель в том что есть к примеру сообщения, они выводятся в список, в списке есть тема сообщения и ID пользователя который написал, по этому ID надо вывести имя пользователя

[Коля Унгуряну]

а mysql_query как умрет? чем его можно заменить?

[Алексей Скобкин]

Коля Унгуряну: Кстати, вам правильно советуют завязывать с mysql_query. Попробуйте лучше PDO. Ну или хотя бы обновлённые возможности от mysqlnd/mysqli.

Answer 7

[Коля Унгуряну]

Сделал как написал Талгат Балтасов

<?php
$res = mysql_query(SELECT * FROM mails as m JOIN user as u on u.id = m.o_id WHERE m.u_id='1');
while ($row = mysql_fetch_object($res)) {
	$data_log[] = $row;
};
?>
<table id="mails">
<tr>
	<th>Тема</th>
	<th>Отправитель</th>
	<th>Действие</th>
</tr>
		<?php foreach ($data_log as $key=>$item) {?>
		<tr>
			<td><a href="?mail=<?php  echo $item->id; ?>"><?php  echo $item->tema; ?></a></td>
			<td><?php  echo $item->name; ?></td>
			<td></td>
		</tr>
		<?php $key+1;}; ?>
</table>

но юзера не выводит(
выводит только из первой таблицы сообщения

Comments

[Талгат Балтасов]

измени JOIN на LEFT JOIN

[Коля Унгуряну]

тоже ничего, просто белый экран, в логах PHP Parse error: syntax error, unexpected T_STRING

[Талгат Балтасов]

Коля Унгуряну: полную ошибку отправь

[Коля Унгуряну]

Талгат Балтасов: [Mon Sep 14 14:41:11 2015] [error] [client 31.216.165.197] PHP Parse error: syntax error, unexpected T_STRING in /home/admin/web/you-cartoon.ru/public_html/encode/main.php on line 21, referer: you-cartoon.ru/encode/login.php

[Талгат Балтасов]

Коля Унгуряну: о боже, ну как я узнаю какая это строка? и потом вот это удали <?php $key+1;}; ?>

[Коля Унгуряну]

21 строка
$res = mysql_query(SELECT * FROM mails as m JOIN user as u on u.id = m.o_id WHERE m.u_id='1');

[Талгат Балтасов]

Коля Унгуряну: а так? $res = mysql_query(SELECT * FROM mails as m JOIN user as u on u.id = m.o_id WHERE m.u_id=1);

[Коля Унгуряну]

Талгат Балтасов: так-же(

[Талгат Балтасов]

Коля Унгуряну: короче выложи код куда нибудь чтобы посмотреть

[Коля Унгуряну]

you-cartoon.ru/main.zip

[Коля Унгуряну]

Талгат Балтасов: сработал ваш вариант! Просто ковычки забыл поставить, спасибо!!!