Можно ли как-нибудь упростить такой запрос?

Question

[Вячеслав]

Привет всем!
Есть у меня запрос состоящий из 4 UNION ALL и в одном из них есть такие строчки:

' LEFT JOIN (SELECT * FROM (SELECT carid, orderid, link FROM '. $this->db->dbprefix .'cars_photo WHERE crop = 1 ORDER BY orderid DESC) t GROUP BY t.carid) cp ON (cp.carid = fwc.carfollow)' .

если я правильно понимаю то, что я сделал - запрос сначала вытаскивает ВСЕ данные, а потом в этих всех выбирает один по последнему ON( ... )
Можно ли как-то упростить чтобы он напрямую выбирал 1 элемент за 1 select ? У меня никак не получается правильно сформировать
спасибо заранее

---------------------------

public function get_feed($username, $offset, $limit)
	{
		$sql = '(SELECT 1 as type, crs.id as id, carfollow, crb.title, crb.text, crb.date, crb.owner, crb.id as post_id, crs.name as car_name, crm.mark as car_mark, crd.model as car_model, cp.link ' .
		' FROM '. $this->db->dbprefix .'follow_car fwc' .
		' JOIN ' . $this->db->dbprefix . 'cars_blog crb ON (crb.carid = fwc.carfollow) ' .
		' LEFT JOIN ' . $this->db->dbprefix . 'cars crs ON (crs.id = fwc.carfollow) ' .
		' LEFT JOIN ' . $this->db->dbprefix . 'cars_mark crm ON (crm.label = crs.mark) ' .
		' LEFT JOIN ' . $this->db->dbprefix . 'cars_model crd ON (crd.label = crs.model) ' .
		' LEFT JOIN (SELECT * FROM (SELECT carid, orderid, link FROM '. $this->db->dbprefix .'cars_photo WHERE crop = 1 ORDER BY orderid DESC) t GROUP BY t.carid) cp ON (cp.carid = fwc.carfollow)' .
		' WHERE fwc.username = ? AND crs.hide = 0 AND crb.status = 1) ' .
		' UNION ALL ' .
		' (SELECT 2 as type, usr.id as id, userfollow, usb.user_blog_title, usb.user_blog_text, usb.cdate, NULL, usb.post_id, fwu.userfollow, NULL, NULL, usr.avatar ' .
		' FROM '. $this->db->dbprefix .'follow_user fwu ' .
		' JOIN ' . $this->db->dbprefix . 'user_blog usb ON (usb.user_name_blog = fwu.userfollow) ' .
		' LEFT JOIN ' . $this->db->dbprefix . 'users usr ON (usr.username = fwu.userfollow) ' .
		' WHERE fwu.username = ? AND usb.status = 1) ' .
		' UNION ALL ' .
		' (SELECT 3 as type, str.item_id as id, userfollow, str.item_title, str.item_descr, str.date, NULL, str.item_id, fsu.userfollow, NULL, NULL, usr.avatar ' .
		' FROM '. $this->db->dbprefix .'follow_store fsu ' .
		' JOIN ' . $this->db->dbprefix . 'store str ON (str.seller = fsu.userfollow) ' .
		' LEFT JOIN ' . $this->db->dbprefix . 'users usr ON (usr.username = fsu.userfollow) ' .
		' WHERE fsu.username = ?) ' .
		' UNION ALL ' .
		' (SELECT 4 as type, grt.id, groupfollow, grt.title, grt.text, grt.date, grt.author, grt.groupid, gr.title, NULL, NULL, gra.link ' .
		' FROM '. $this->db->dbprefix .'follow_group fgr ' .
		' JOIN ' . $this->db->dbprefix . 'groups_topics grt ON (grt.groupid = fgr.groupfollow) ' .
		' LEFT JOIN ' . $this->db->dbprefix . 'groups gr ON (gr.id = fgr.groupfollow) ' .
		' LEFT JOIN ' . $this->db->dbprefix . 'groups_avatar gra ON (gra.groupid = fgr.groupfollow) ' .
		' WHERE fgr.username = ?) ' .
		' ORDER BY date DESC' . 
		' LIMIT '.$offset.','.$limit;
		
		$query = $this->db->query($sql, array($username, $username, $username, $username));
		return $query->result_array();
	}

Comments

[heartdevil]

Привет. Мало что понятно из вашего куска запроса. Вставьте весь запрос и отметьте тот кусок, который вызывает сомнения. Далее, приведите скрин выборки того, что есть сейчас, и опишите детально то, что хотите получить. Так будет проще понять, о чем речь.

[Макс]

1 элемент за 1 селект - LIMIT 1. Но это может быть далеко не то, что Вам реально надо. Так что присоединюсь к heartdevil - выкладывайте полный запрос, всем миром посмотрим.

[ComodoHacker]

И отформатируйте SQL, пожалуйста. Заставлять это делать нас невежливо.

[Вячеслав]

ComodoHacker: что есть "отформатируйте" ? расставить табы и ентеры? я пишу запросы в таком виде как выложил :) по остальному сейчас обновлю топик

[Вячеслав]

heartdevil: Выложил весь запрос. он абсолютно рабочий, меня смущает только тот момент, что я выложил. 9 строчка сверху. Смущает он меня не потому что не работает, а просто мне кажется что он может притормаживать остальную часть, т.к. делает лишнюю операцию.

Answer 1

[romy4]

одноразовый запрос. в общем запросе он не нужен и может быть выполнен до общего

SELECT carid, orderid, link FROM '. $this->db->dbprefix .'cars_photo WHERE crop = 1 ORDER BY orderid DESC) t GROUP BY t.carid

Упростить можно. Сохраняйте запрос в отдельную таблицу, а далее уже вытаскивайте по одному значению.

Comments

[Вячеслав]

Он не одноразовый, он напрямую непосредственно относится к 1 запросу общего UNION ALL. Конкретно эта строчка вытаскивает фотографию конкретной машины

[romy4]

Вячеслав: я не вижу, чтобы этот запрос был как-то завязан на другие. вот другие на него завязаны — это да. чтобы на каждую строчку этот запрос не выполнялся, данные необходимо подготовить и сохранить ДО основного

[Вячеслав]

romy4: честно говоря не понимаю вас вообще. на этот запрос не завязано абсолютно ничего, этот запрос конечный в своем роде и он вытаскивает фотографию для машины, которая выбирается запросом выше - ... ' FROM '. $this->db->dbprefix .'follow_car fwc' .

[romy4]

он выполняется для каждой вложенной строки родительского для него запроса. лишняя трата 100500 вызовов.
тут даже можно весь

SELECT * FROM (SELECT carid, orderid, link FROM '. $this->db->dbprefix .'cars_photo WHERE crop = 1 ORDER BY orderid DESC) t GROUP BY t.carid)

выполнить до основного.

[Вячеслав]

romy4: Стоп. Так я и хочу избавиться от этого запроса вообще. Зачем мне выдергивать фотографии абсолютно всех машин, а потом еще раз по нему пробегать сравнением? получается что вы предлагаете тот же способ, только в другой руке. Машин в последствии будут миллион, два, три. Не, это не годится.

[romy4]

Вячеслав: запрос получает N строк во временку без каких-либо индексов, потом на каждый чих тупым сравнением делается cp.carid = fwc.carfollow — это крайне нерационально, учитывая что "Машин в последствии будут миллион, два, три"

[romy4]

Вячеслав: "тупым сравнением" - имел в виду банальным перебором по всей временнной таблице

[Вячеслав]

romy4: упростил сам) '

LEFT JOIN ' . $this->db->dbprefix . 'cars_photo cp ON cp.id = (SELECT link FROM '. $this->db->dbprefix .'cars_photo WHERE carid = fwc.carfollow AND crop = 1 ORDER BY orderid DESC LIMIT 1)' .

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

Answer 2

[ComodoHacker]

Попробуйте

SELECT TOP 1 carid, link 
  FROM cars_photo 
 WHERE crop = 1
   AND cars_photo.carid = fwc.carfollow
 ORDER BY orderid DESC

и без GROUP BY.

Comments

[Вячеслав]

я уже сделал такой вариант сам:
' LEFT JOIN ' . $this->db->dbprefix . 'cars_photo cp ON cp.link = (SELECT link FROM '. $this->db->dbprefix .'cars_photo WHERE carid = fwc.carfollow AND crop = 1 ORDER BY orderid DESC LIMIT 1)' .

PS проверил ваш - не хочет почему-то работать, выдает Error Number: 1064 You have an error in your SQL syntax; при TOP 1
попробовал TOP(1) говорит что TOP not found =)