Лента новостей на php (проблема с запросом), как реализовать 2 запроса в один?

Question

[Sergey Maley]

Нужна помощь правильно "отфильтровать" вывод данных для ленты новостей. То есть, я выбираю из базы данных те поля где мой id подписан на других пользователей с их id. Проблема в том ЧТО мне даёт только id ПОСЛЕДНЕГО ЮЗЕРА на которого я подписался, а не выдаёт всех (то есть не все поля, а только одно), в цикле ниже просто идёт фильтрация по сообщениях.

// Выбор id залогиненного пользователя (по id буду сортироваться сообщения в ленты новостей от пользоватей на которых он подписан )
$sqlForSubscriberByLogged	   = "SELECT id_request_reciver FROM subscribers WHERE id_request_sender IN('$loggedUser->id') ";
$queryForSubscriberByLogged 	  = $database->query($sqlForSubscriberByLogged);
$fetch_arrayForSubscriberByLogged = $database->fetch_array($queryForSubscriberByLogged);								
								
// Здесь сортировка для ленты новостей от нужных пользователей на которых подписан залогившейся юзер.
// Фильтрация данных будет идти от id_request_reciver
$sqlForWallMessagesBySubscribers   = "SELECT id_mes,id,text,sender,minute,hour,day,month,year FROM wall_messages WHERE
id IN($fetch_arrayForSubscriberByLogged[id_request_reciver]) ORDER BY id_mes DESC LIMIT 15"; //IN()
$queryForWallMessagesBySubscribers = $database->query($sqlForWallMessagesBySubscribers);

Должно получиться вот так


А мне выводит контент только одного пользователя на которого я подписан, а я подписан на 2 юзера...


Задавал этот вопрос здесь, Перейти
Но решения не нашел.

Comments

[Reistlin]

Swift'овские переменные)

[Sergey Maley]

Изнаур Бексултанов: Да да да =) я программист Obj-C, это привычка ;)

[FanatPHP]

ты даже не представляешь, КАК болят глаза. И быть ты хоть тракторист на Objective-C - читателей надо хоть немного уважать. И не сдвигать свой и так растянутый код вообще за экран.

[Sergey Maley]

FanatPHP: Прошу прощения =)

[FanatPHP]

в общем, как правильно написали ниже, тут нужен джойн

Answer 1

[Sergey Maley]

Задачу решил!
Всем спасибо за подсказки та помощь.

Вот что изменил я..

SELECT subscribers.id_request_reciver, wall_messages.* FROM subscribers JOIN wall_messages ON subscribers.id_request_reciver = wall_messages.id WHERE subscribers.id_request_sender = $loggedUser->id ORDER BY wall_messages.id_mes DESC

Результат.


Хотел еще спросить, что дописать в запросе чтобы видеть и свои записи в ленте ?

Comments

[Reistlin]

Извините,но тогда правильней всего было бы выбрать правильным ответом, мой ответ)

Answer 2

[FanatPHP]

Самое первое, что надо сказать:
Не нужно давать временным переменным супер-информативные имена.

• $sqlForSubscriberByLogged - это просто $sql. Эта переменная никогда никому не будет нужна, кроме функции, исполняющей запросы. А ей всё равно, как названа переменная.
  
• Переменной $queryForSubscriberByLogged в принципе быть не должно, это паразитная, ничего не делающая переменная.
  
• $fetch_arrayForSubscriberByLogged - это какой-то адов монстр. Нас не должно интересовать, каким способом получена информация, через fetch_array или fetch_row. Нас должно только интересовать её содержимое. Вот его и надо описать в имени переменной. Но тоже без эксцессов.
  

В итоге должно получиться

$sql = "SELECT id_request_reciver FROM subscribers WHERE id_request_sender = ..."; 
$SubscriberByLogged = $database->query($sql)->fetch_array();

Answer 3

[Reistlin]

Если я Вас правильно понял,прочитайте про объединение запросов,Вам нужно,что-то вроде такого:

$database->query("SELECT (поля которые вам нужны из этих таблиц в виде таблица.поле) FROM subscribers LEFT JOIN wall_messages ON subscribers.id_request_reciver=wall_messages.id WHERE subscribers.id_request_sender = $loggedUser->id")->fetch_array();

Answer 4

[Михаил Тихонин]

Посмотри вардампом, что у тебя в переменной $sqlForWallMessagesBySubscribers и ошибка наверняка найдется.

Comments

[Sergey Maley]

Вывел через var_dump.
В итоге я получил только ОДНО значение, это id последнего пользователя на которого я подписался,. То есть, теперь в ленте новостей я буду видеть посты только юзера с id=4, а юзера с id=3 больше не увижу. Вот в чем проблема у меня. Я вижу записи только последнего юзера на которого я был подписан...

string(137) "SELECT id_mes,id,text,sender,minute,hour,day,month,year FROM wall_messages WHERE id IN(4) ORDER BY id_mes DESC LIMIT 15"

[Reistlin]

spesher: так и будет,пока Вы не свяжите 2 таблицы,так же Вы можете циклом получить всех пользователей на которых Вы подписаны.До того как пытаться написать такую вещь,стоило бы изучить получше PHP(т.к. таких проблем будет слишком много) ,php.net/manual/ru/function.mysql-fetch-assoc.php - вот посмотрите примеры

[Михаил Тихонин]

spesher: ниже Вам уже писали, что нужно использовать implode(',', $fetch_arrayForSubscriberByLogged), либо объединить 2 таблицы в одном запросе. Первый вариант обычно работает быстрее.

Answer 5

[Андрей Мохов]

пипец переменные...
вместо $fetch_arrayForSubscriberByLogged[id_request_reciver] нужно implode(',', $fetch_arrayForSubscriberByLogged)

Comments

[FanatPHP]

не факт, кстати. там, скорее всего, двумерный массив.

[Андрей Мохов]

FanatPHP: да мне все-равно, я подкинул идею в какую сторону копать, тут же ещё проверку нужно делать на непустой массив и т.д.

[FanatPHP]

это плохо, когда все равно

[Александр N++]

Боже спаси и сохрани след. разработчика кто будет поддерживать этот проект.

[Sergey Maley]

Александр N++: Нормально !) Позже переделаю. Пишу как мне удобно ;)

[Андрей Мохов]

spesher: как показывает практика "потом переделаю" значит "никогда"

[Sergey Maley]

Андрей Мохов: -_- Почитай выше чем я занимаюсь основным, я не php программист, а obj-c. Спасибо.