質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.46%
WordPress

WordPressは、PHPで開発されているオープンソースのブログソフトウェアです。データベース管理システムにはMySQLを用いています。フリーのブログソフトウェアの中では最も人気が高く、PHPとHTMLを使って簡単にテンプレートをカスタマイズすることができます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

ループ

ループとは、プログラミングにおいて、条件に合致している間、複数回繰り返し実行される箇所や、その制御構造を指します

Q&A

解決済

2回答

1791閲覧

WordPressで入れ子にしたループの速度改善策

michir

総合スコア12

WordPress

WordPressは、PHPで開発されているオープンソースのブログソフトウェアです。データベース管理システムにはMySQLを用いています。フリーのブログソフトウェアの中では最も人気が高く、PHPとHTMLを使って簡単にテンプレートをカスタマイズすることができます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

ループ

ループとは、プログラミングにおいて、条件に合致している間、複数回繰り返し実行される箇所や、その制御構造を指します

0グッド

0クリップ

投稿2021/10/04 10:34

編集2021/10/04 10:57

WordPressでループを入れ子にしたら重くなってしまい困っています。
改善策をアドバイスしていただけると嬉しいです。

現行のコードを以下は以下のような感じです。

php

1<?php 2$j=get_query_var('j'); 3$taxonomy_name = 'area'; 4$term_id = get_queried_object_id(); 5$args = array( 6 'post_type' => 'jimusho', 7 'posts_per_page' => -1, 8 'tax_query' => array( 9 array( 10 'taxonomy' => $taxonomy_name, 11 'terms' => $term_id, 12 ), 13 ), 14); 15if($j==="1"){$args['meta_query']=array(array('key'=>'j','value'=>'1'));} 16$custom_query = new WP_Query( $args ); 17if ( $custom_query->have_posts() ) : 18$jimusho="<ul>"; 19 while ( $custom_query->have_posts() ) : 20 $custom_query->the_post(); 21 $address = get_post_meta(get_the_ID(), 'address'); 22 $title=get_the_title(); 23$link=get_the_permalink(); 24$j2=[]; 25$count_b=0; 26$args2 = array( 27 'post_type' => 'b', 28 'meta_key' => 'jimusho', 29 'meta_value' => get_the_ID(), 30 ); 31 $customPosts = get_posts($args2); 32 if ($customPosts) : foreach ($customPosts as $post) : setup_postdata($post); 33$j2[]=get_post_meta(get_the_ID(),'toriatukai'); 34endforeach; 35$count_b=count($customPosts);/ 36else : ?> 37 <p>記事はありません。</p> 38 <?php endif; 39 wp_reset_postdata(); 40$jimusho.="<li><a href='$link'>{$title}{$count_b}名)</a> {$address[0]}</li>\n"; 41$the_term=get_the_terms(get_the_ID(),$taxonomy_name)[0]->term_id; 42if(!isset($count[$the_term])){$count[$the_term]=0;} 43if(isset($j2)){ 44$j2=array_column($j2,0); 45 if(in_array('juu',$j2)){ 46 $jimusho.='この問題を重点的に取り扱っている専門家が所属しています。'; 47 $count[$the_term]++; 48 } 49} 50endwhile; 51$jimusho.="</ul>"; 52wp_reset_postdata(); 53else : 54$jimusho='データがありません。'; 55endif;

「jimusho」は1万6千件ほど、「b」は4万件ほどのレコードがあります。

以下の箇所が処理の時間がかかっているようです。

php

1 $customPosts = get_posts($args2);

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

tabuu

2021/10/04 23:38

>以下の箇所が処理の時間がかかっているようです。 >$customPosts = get_posts($args2); 実際に何秒かかっていて、何回コールされているのでしょうか?
michir

2021/10/05 02:11 編集

返信ありがとうございます。 エリアのよって異なりますが、 最もjimusho数の多い東京都の場合は約5千回、最も少ない鳥取県の場合は35回コールされています。 東京都の処理はタイムアウトしてしまい計測できませんでしたが、鳥取県の場合は約20秒かかっています。 単純計算ですが、東京都の場合は50分近くかかると思います。
tabuu

2021/10/05 02:50

環境にアクセスできないので具体的なアドバイスは難しいですが DBの設計を見直す(postmetaで紐づけるのを止める)、サーバのスペックを上げる、 といった対応が必要と思われます。 コーディングだけでの改善は難しいでしょう。
guest

回答2

0

まずは、Query Monitor プラグイン等を利用してどのクエリが遅いのか確認しましょう。

どこが遅いのかわかれば、

WordPress が生成する SQL をカスタムクエリ を利用して、SQL を書き換えてチューニングする方法

wpdb_Classを用いる方法で自分でチューニングした SQL を実行させて取得する方法もあると思います。

投稿2021/10/05 13:48

CHERRY

総合スコア25175

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

ベストアンサー

meta_queryが重い要因になっていそうなら
まず全件を取得しておいてbフィールドの値でWP_Postオブジェクトを振り分けた
多次元配列を先に作っておいてそれを使うようにする

一つ一つの投稿の表示に伴う処理が重いのが要因そうなら
コストの高そうな処理の結果を予めpost_metaとして各投稿に持たせておく

みたいな感じでしょうか

投稿2021/10/05 05:09

KazuhiroHatano

総合スコア7804

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.46%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問