web-dev-qa-db-ja.com

WP_Queryが異常な量のメモリをリークしている

下記の関数でWP_Query()を呼び出すたびに、Wordpressは8メガバイトのメモリリークを起こします。そして、この関数をよく呼んでいるので、物事はかなり早く毛深いものになります。

function get_post_ids_in_taxonomies($taxonomies, &$terms=array()) {
    $post_ids = array();

    $query = gen_query_get_posts_in_taxonomies($taxonomies, $terms);
    // var_dump($query);

    //Perform the query
    $queryObject = new WP_Query($query); //*****THE 8 MEGABYTES IS LEAKED HERE*****

    //For all posts found...
    if($queryObject->have_posts()) {
        while($queryObject->have_posts()) {
            $queryObject->the_post();

            //Get the $post_id by capturing the output of the_ID()
            ob_start();
            the_ID();
            $post_id = (int) ob_get_contents();
            ob_end_clean();

            // echo $post_id."\n";
            $post_ids[] = $post_id;
        }
    }

    unset($queryObject);

    return $post_ids;
}

gen_query_get_posts_in_taxonomies()は次のとおりです。

function gen_query_get_posts_in_taxonomies($taxonomies, &$terms=array()) {
    //General query params
    $query = array(
        'posts_per_page'    => -1,  //Get all posts (no paging)
        'tax_query'             => array('relation' => 'OR'),
    );

    //Add the specific taxonomies and terms onto $query['tax_query']
    foreach($taxonomies as $tax) {
        //Get terms in the taxonomies if we haven't yet
        if(!array_key_exists($tax, $terms)) {
            $terms[$tax] = array();

            $terms_tmp = get_terms($tax);
            foreach($terms_tmp as $tt)
                $terms[$tax][] = $tt->term_taxonomy_id;
        }

        $query['tax_query'][] = array(
            'taxonomy' => $tax,
            'terms' => $terms[$tax],
            'field' => 'term_taxonomy_id',
        );
    }

    return $query;
}
10
rinogo

WPハッカーに関する優れた回答: http://lists.automattic.com/pipermail/wp-hackers/2012-June/043213.html

あなたがそのクエリでやっていることは、完全な投稿内容を含めて、すべての一致する投稿をメモリにロードすることです。ご想像のとおり、これはおそらくかなりたくさんの項目です。

'fields' => 'ids'をWP_Queryに渡して、代わりに単に一致するpost_idのリストを返すことができます。これにより、メモリ(および処理時間)を大幅に削減できます。

http://codex.wordpress.org/Class_Reference/WP_Query#Post_Field_Parameters

14
rinogo

ここで指摘された記憶問題を研究している間これにつまずいた。

この場合は、バッファリングを使用してIDを取得する代わりにget_the_idを使用できます。また、クエリされたフィールドを絞り込んでIDのみを含めることもできます。

3
Thomas