Как можно упростить данный код?

Question

[nezzard]

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

// Начало выборка айди постов
$post_ids = $wpdb->get_results("SELECT ID FROM wp_posts
		WHERE post_author = $user_ID AND post_status = 'publish'", ARRAY_A);

$ids = array();
foreach ($post_ids as $k) {	
	$ids[] = $k['ID'];
}
// Конец выборка айди постов


// Начало выборка значений рейтинга
$posts_ratings = $wpdb->get_results('SELECT meta_value FROM wp_postmeta WHERE meta_key = "votes" AND post_id IN (' . implode(",", $ids) . ')', ARRAY_A);

$summ_rating = array();
foreach ($posts_ratings as $k) {
	$summ_rating[] = $k['meta_value'];
};
// Конец выборка значений рейтинга


//Вывод суммы рейтинга
echo array_sum($summ_rating);
//Конец суммы рейтинга

Comments

[Елена]

вопрос: для чего его нужно упрощать ?

Answer 1

[Sekira]

SELECT SUM(meta_value) FROM wp_postmeta WHERE meta_key = "votes" AND post_id IN (SELECT ID FROM wp_posts WHERE post_author = $user_ID AND post_status = 'publish')

Comments

[nezzard]

Спасибо, как его вывести Array ( [0] => Array ( [SUM(meta_value)] => 33 ) )
Пробую [0] опускается ниже, но все ровно не выводит 33

[Sekira]

Возможно в wordpress есть другой вариант запроса, кроме "get_results" для возврата единственного значения в запросе, но если делать через него, то можно:

Заменить SELECT SUM(meta_value) FROM ... на SELECT SUM(meta_value) as `sum_rating` FROM ...
А потом выводить через echo $sum_rating[0]['sum_rating'];

[Sekira]

Точнее переменную, конечно, можно назвать по-другому, а не одинаково, как нравится, и алиас в базе тоже. А не исправляя запрос можно вывести так: $sum_rating[0]['SUM(meta_value)']; , но я считаю, это не так красиво, как через алиас =)

[nezzard]

@Sekira ой я забыл что там есть get_row

Answer 2

[Артур Борденюк]

Можно использовать WP_Query и array_map, например.

Но пока кода работает как надо...

Comments

[nezzard]

WP_query для комментариев будет работать?

[Артур Борденюк]

если верно его приготовить, разве что

Answer 3

[Melkij]

$post_ids потом нафиг не нужен?

SELECT sum(meta_value) FROM wp_postmeta join wp_posts on wp_posts.id=post_id and post_author = $user_ID AND post_status = 'publish' WHERE meta_key = "votes"

fetch и echo сразу.