Как встроить в запрос еще одну проверку?

Question

[Евгений Шевцов]

Я уже совсем хочу спать... Час назад все работало, а сейчас просто в отказ идет.=)
Написал вот такой запрос

$join .= 'INNER JOIN ' . $this->wpdb->postmeta .' price
ON price.post_id = posts.ID AND
price.meta_key LIKE \'%_to_price_%\' ';

Как мне сюда добавить проверку по post_id ?
Что то вроде AND price.post_id = '1050'

Comments

[Евгений Шевцов]

Весь запрос.
foreach ( $taxitoz as $postoz ) {
$postidz = $postoz->post_id;
if (isset( $params['price_from'] ) && $params['price_from'] != '')
$where[] = 'price.meta_value >= ' . $params['price_from'];
if (isset( $params['price_to'] ) && $params['price_to'] != '')
$where[] = 'price.meta_value <= ' . $params['price_to'];
$join .= 'INNER JOIN ' . $this->wpdb->postmeta .' price
ON price.post_id = posts.ID AND
price.meta_key LIKE \'%_to_price_%\' ';
if ($sorted_field == '_price') $sorted = 'cast(price.meta_value AS signed)';
if(empty($sorted_direction)) $sorted_direction = 'ASC';
}

Answer 1

[Эдвард]

Мне всегда казалось что на тостере сидят разработчики, а не участники шоу интуиция или битва экстрасенсов и я бы хотел вам помочь но понятия не имею что находится в
$this->wpdb->postmeta
и не имею особого желания угадывать

Comments

[Эдвард]

Я вас возможно удивлю но не все программисты работают с WP, поэтому ваш стандартный код WP мне не о чем не говорит и с каких пор не знать WP === не знать SQL?

Answer 2

[ugodrus]

1) Для начала хотелось бы увидеть весь запрос.
2) использование LIKE и прочей бурды в JOIN ON не рекомендуется.
3) Возможно вам стоит переложить запрос без Join по такому принципу
SELECT something FROM price t1, price t2 WHERE и здесь указать правила соединения таблиц и прочие условия
но не зная конечный запрос сказать тяжело.

Comments

[Евгений Шевцов]

Тут без like я не обойдусь так как meta_key-и там все с префиксом _to_price_ и их мне нужно получить.

[Евгений Шевцов]

Запрос добавил в комментарии.

[ugodrus]

А кто мешает переместить это в WHERE? Так будет более корректно да и шустрее.

[Евгений Шевцов]

@ugodrus: Да я пробовал его переписать, но ни в какую не хочет...
Мне даже не переписать нужно, а дополнить, чтобы по определенным post_id искал.

[Евгений Шевцов]

@ugodrus: Ну может вы подскажете, как его правильно переписать=)

[ugodrus]

Начнём с того что это не запрос. Это конструктор части тела запроса. Неужели сложно сделать дамп запроса перед отправкой в дб. А так, если этот джоин единственный, то будет проще переложить по моему варианту. И все ваши проверки плавно переместятся в WHERE.

... WHERE price.post_id = posts.ID AND price.meta_key LIKE \'%_to_price_%\' AND price.post_id = '1050' и т.д.

И вообще топик не в той теме. Это не про php. И не понятно с какой БД мы имеем дело. Если мы имеем дело с MySQL, то:
In MySQL, JOIN, CROSS JOIN, and INNER JOIN are syntactic equivalents (they can replace each other).
Таким образом можно сделать как я указал, и это будет правильно, нежели пихать всё подряд в параметры соединения таблиц.

[ugodrus]

Советую поискать обзоры по join`ам вообще и по join`ам в mysql в частности. Join - это соединение наборов данных. В ON и USING указывается точка соединения этих наборов (по какому принципу и полям мы соединяем и расширяем эти наборы данных). А фильтрация результатов идёт в WHERE и HAVING(в последнюю очередь). В данном случае вы добавили фильтрацию не туда.