teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

12

加筆

2021/05/17 01:19

投稿

退会済みユーザー
answer CHANGED
@@ -83,4 +83,22 @@
83
83
  みたいなクエリーも作れます。
84
84
  何個も何十個もプロットせず、ざっくり数字だけ表示しておいて省略表示するべきでしょうね。
85
85
  ネット上に誰かが作ったデータがあるかもしれませんし、
86
- 自分でQGISを使って作ることもできますね。
86
+ 自分でQGISを使って作ることもできますね。
87
+
88
+ ---
89
+
90
+ あー、php上で配列要素から配列要素に、
91
+ ループを回して代入するのは、ほんとに無駄なことをやってますね。
92
+ それを2万件も回しているかと思うと呆れてきます。
93
+
94
+ そういうことは、データベースに任せてください。
95
+ クエリーの仕事です。
96
+ とてもphp上でやるべきことではない。
97
+ つまり、テーブル同士を欲しい情報のカタチにouter joinして
98
+ データの結合をデータベース内にやらせてください。
99
+ 適材適所というか、データの並べ替えや整理が得意なデータベースの仕事を
100
+ わざと不得意なphpにやらせてるのが悪いです。
101
+
102
+ outer joinでテーブルのデータ構造の主従関係に沿って連結して、
103
+ ほしい並び順はorder by句でコントロール。
104
+ それができれば、単にfetchAll()で完結しませんかね。

11

加筆

2021/05/17 01:19

投稿

退会済みユーザー
answer CHANGED
@@ -73,4 +73,14 @@
73
73
  その直前までひとつの文字列変数に文字列連結して文字列連結して文字列連結してってやり方をしていると、
74
74
  パフォーマンスが落ちるかもしれません。
75
75
  配列で保持できるようであれば、
76
- implode()を使って配列を文字列に連結するようにしたらよいかと。
76
+ implode()を使って配列を文字列に連結するようにしたらよいかと。
77
+
78
+ それと余談。
79
+ 狭いところに2万件もプロットする見せ方も、考え直すべきですね。
80
+ 先の空間データ型を使うようになると、
81
+ 地域(例えば市町村、大字、町丁目とか)の形をMULTIPOLYGON型で格納したテーブルを用意すると、
82
+ A市に含まれるPOIが何件、B市に含まれるPOIが何件、
83
+ みたいなクエリーも作れます。
84
+ 何個も何十個もプロットせず、ざっくり数字だけ表示しておいて省略表示するべきでしょうね。
85
+ ネット上に誰かが作ったデータがあるかもしれませんし、
86
+ 自分でQGISを使って作ることもできますね。

10

加筆

2021/05/13 14:45

投稿

退会済みユーザー
answer CHANGED
@@ -33,4 +33,44 @@
33
33
  JOINするときに関数で加工しているなら止めましょうね、
34
34
  全部を一度の連結しなくても、情報を小出しにして何度も取得しに行くよう、一度に取得するデータの粒度を小さくするアルゴリズムを考えるのも大事。
35
35
  2万件が相手だとしても、サーバーのチューニングも含めて徹底的に手を施すと
36
- クエリーだけなら6秒以上待たせる場面は減らせそうな気がする。(独断と偏見。)
36
+ クエリーだけなら6秒以上待たせる場面は減らせそうな気がする。(独断と偏見。)
37
+
38
+ ---
39
+
40
+ クエリー3種類、それをphpコードでどう処理しているのか、
41
+ っていうところも見たかったですが。
42
+
43
+ 緯度経度を伴うテーブルの、緯度経度を文字列のまま保持しているんでしょうか?
44
+ mysqlでもpostgresqlでも空間情報を扱う機能があるので、
45
+ 然るべき測地系の空間データとして格納し、
46
+ 空間インデックスを仕込むべきです。
47
+
48
+ [MySQL :: MySQL 5.6 リファレンスマニュアル :: 11.5 空間データの拡張](https://dev.mysql.com/doc/refman/5.6/ja/spatial-extensions.html)
49
+ この近辺をよく読んで、空間データとして扱えるよう変換したテーブルを作って変換してください。
50
+ 参考になる記事:[【MySQL】Geometry型で位置情報を扱う - Qiita](https://qiita.com/nkojima/items/bb21c72858606dedd817)
51
+
52
+ それと。
53
+ shop_table.idに対してshopimg_table.idは複数、
54
+ 同じく
55
+ shop_table.idに対してmenu_table.idも複数、
56
+ っていう感じでしょうか。
57
+ クエリー結果をもとにループしている中で、
58
+ 別のクエリーを実行するのは、あまり良くないような。
59
+ プリペアドステートメントの仕組みを使って、
60
+ パラメータのみ変えて実行するならまだSQL文の解釈にかかる処理時間を省くことができますが、
61
+ もしも、一番外側のループの値を都度、SQL文に文字列連結してクエリーを作っているなら、
62
+ 即刻やめて
63
+ [PDO::prepare](https://www.php.net/manual/ja/pdo.prepare.php)
64
+ [PDOStatement::bindValue](https://www.php.net/manual/ja/pdostatement.bindvalue.php)
65
+ [PDOStatement::execute](https://www.php.net/manual/ja/pdostatement.execute.php)
66
+ を組み合わせてプリペアドステートメントによるクエリー実行に直します。
67
+
68
+ [PHPでデータベースに接続するときのまとめ - Qiita](https://qiita.com/mpyw/items/b00b72c5c95aac573b71)
69
+ をなぞって、prepare→bindValue→executeの流れを丁寧にやるといいです。
70
+
71
+ 最後に、
72
+ 出力用のjson文字列を作るのに、最後にjson_encode()にかけるとして
73
+ その直前までひとつの文字列変数に文字列連結して文字列連結して文字列連結してってやり方をしていると、
74
+ パフォーマンスが落ちるかもしれません。
75
+ 配列で保持できるようであれば、
76
+ implode()を使って配列を文字列に連結するようにしたらよいかと。

9

見直し

2021/05/13 14:39

投稿

退会済みユーザー
answer CHANGED
@@ -23,4 +23,14 @@
23
23
  サーバー側も然るべき仕組み(MapServerとか)を組み合わせることで、
24
24
  表示範囲に限定した情報のみ非同期に取得して地図情報反映する仕組みが実現しやすいです。
25
25
  自前で処理を起こさず、仕組み建てを変えてみるのも良いかと。
26
- サーバーサイドが明るくないのであればなおさら。
26
+ サーバーサイドが明るくないのであればなおさら。
27
+
28
+ とはいえ、仕組みをいじるのは腰が重いでしょうから、
29
+ データベース周りをしっかりやってほしいかな、
30
+ ヘボいクエリーを叩き直すことで応答までの時間がケタ1つ2つ変わるなんて話はよくあるし。
31
+ 主キー設定してますか、
32
+ テーブルをJOINするお互いのキーにインデックスはありますか、
33
+ JOINするときに関数で加工しているなら止めましょうね、
34
+ 全部を一度の連結しなくても、情報を小出しにして何度も取得しに行くよう、一度に取得するデータの粒度を小さくするアルゴリズムを考えるのも大事。
35
+ 2万件が相手だとしても、サーバーのチューニングも含めて徹底的に手を施すと
36
+ クエリーだけなら6秒以上待たせる場面は減らせそうな気がする。(独断と偏見。)

8

見直し

2021/05/12 06:13

投稿

退会済みユーザー
answer CHANGED
@@ -19,7 +19,7 @@
19
19
 
20
20
  それと、
21
21
  Googleマップを表示したいからGoogle Map APIを使うというのはある意味正しいですが、
22
- OpenLayersとかLeafletなどの地図を扱えるjsライブラリを組み合わせつつ、
22
+ Mapbox GL JS(有料)やOpenLayersとかLeafletなどの地図を扱えるjsライブラリを組み合わせつつ、
23
23
  サーバー側も然るべき仕組み(MapServerとか)を組み合わせることで、
24
24
  表示範囲に限定した情報のみ非同期に取得して地図情報反映する仕組みが実現しやすいです。
25
25
  自前で処理を起こさず、仕組み建てを変えてみるのも良いかと。

7

見直し

2021/05/12 06:06

投稿

退会済みユーザー
answer CHANGED
@@ -12,7 +12,15 @@
12
12
  それと、
13
13
  [MySQL :: MySQL 5.6 リファレンスマニュアル :: 11.5.3.5 空間分析の最適化](https://dev.mysql.com/doc/refman/5.6/ja/optimizing-spatial-analysis.html)
14
14
  のように空間情報のためのインデックスもあるので、
15
- 例えばマップ表示中の中心の緯度経度情報を元に、
15
+ 例えばマップ表示中の中心の緯度経度情報を元に、
16
16
  表示範囲に相当する範囲のPOIだけ抽出して送信する、
17
17
  みたいなクエリーだってできるはず。
18
- いきなり2万件送信するんじゃなくて。
18
+ いきなり2万件送信するんじゃなくて。
19
+
20
+ それと、
21
+ Googleマップを表示したいからGoogle Map APIを使うというのはある意味正しいですが、
22
+ OpenLayersとかLeafletなどの地図を扱えるjsライブラリを組み合わせつつ、
23
+ サーバー側も然るべき仕組み(MapServerとか)を組み合わせることで、
24
+ 表示範囲に限定した情報のみ非同期に取得して地図情報反映する仕組みが実現しやすいです。
25
+ 自前で処理を起こさず、仕組み建てを変えてみるのも良いかと。
26
+ サーバーサイドが明るくないのであればなおさら。

6

加筆

2021/05/12 06:01

投稿

退会済みユーザー
answer CHANGED
@@ -7,4 +7,12 @@
7
7
  2万件をどうやってjsonに加工しているのか。
8
8
  文字列の連結を都度繰り返すのか、implode()で一発で連結するのかとか、
9
9
  処理の区切りごとにmicrotime(true)で測定して比較するとか。
10
- [PHPの処理速度を計測 - Qiita](https://qiita.com/ao_love/items/a2d8b8566d0426195002)
10
+ [PHPの処理速度を計測 - Qiita](https://qiita.com/ao_love/items/a2d8b8566d0426195002)
11
+
12
+ それと、
13
+ [MySQL :: MySQL 5.6 リファレンスマニュアル :: 11.5.3.5 空間分析の最適化](https://dev.mysql.com/doc/refman/5.6/ja/optimizing-spatial-analysis.html)
14
+ のように空間情報のためのインデックスもあるので、
15
+ 例えばマップ表示中の中心店の緯度経度情報を元に、
16
+ 表示範囲に相当する範囲のPOIだけ抽出して送信する、
17
+ みたいなクエリーだってできるはず。
18
+ いきなり2万件送信するんじゃなくて。

5

見直し

2021/05/12 04:28

投稿

退会済みユーザー
answer CHANGED
@@ -1,4 +1,10 @@
1
1
  SQLの実行計画のチェック。
2
2
  [MySQL :: MySQL 5.6 リファレンスマニュアル :: 8.8.1 EXPLAIN によるクエリーの最適化](https://dev.mysql.com/doc/refman/5.6/ja/using-explain.html)
3
3
 
4
- インデックスが適切に施されていない、あるいは施されていても効いていないって話かと。
4
+ インデックスが適切に施されていない、あるいは施されていても効いていないって話かと。
5
+
6
+ それと、コードを見ていないので当てずっぽう感ありますが、
7
+ 2万件をどうやってjsonに加工しているのか。
8
+ 文字列の連結を都度繰り返すのか、implode()で一発で連結するのかとか、
9
+ 処理の区切りごとにmicrotime(true)で測定して比較するとか。
10
+ [PHPの処理速度を計測 - Qiita](https://qiita.com/ao_love/items/a2d8b8566d0426195002)

4

見直し

2021/05/12 02:26

投稿

退会済みユーザー
answer CHANGED
File without changes

3

加筆

2021/05/12 02:26

投稿

退会済みユーザー
answer CHANGED
File without changes

2

加筆

2021/05/12 02:23

投稿

退会済みユーザー
answer CHANGED
@@ -1,2 +1,4 @@
1
1
  SQLの実行計画のチェック。
2
- [MySQL :: MySQL 5.6 リファレンスマニュアル :: 8.8.1 EXPLAIN によるクエリーの最適化](https://dev.mysql.com/doc/refman/5.6/ja/using-explain.html)
2
+ [MySQL :: MySQL 5.6 リファレンスマニュアル :: 8.8.1 EXPLAIN によるクエリーの最適化](https://dev.mysql.com/doc/refman/5.6/ja/using-explain.html)
3
+
4
+ インデックスが適切に施されていない、あるいは施されていても効いていないって話かと。

1

加筆

2021/05/12 02:23

投稿

退会済みユーザー
answer CHANGED
File without changes