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

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

新規登録して質問してみよう
ただいま回答率
85.50%
SQL Server

SQL Serverはマイクロソフトのリレーショナルデータベース管理システムです。データマイニングや多次元解析など、ビジネスインテリジェンスのための機能が備わっています。

SQL

SQL(Structured Query Language)は、リレーショナルデータベース管理システム (RDBMS)のデータベース言語です。大きく分けて、データ定義言語(DDL)、データ操作言語(DML)、データ制御言語(DCL)の3つで構成されており、プログラム上でSQL文を生成して、RDBMSに命令を出し、RDBに必要なデータを格納できます。また、格納したデータを引き出すことも可能です。

Q&A

解決済

2回答

497閲覧

SQL_該当するidに紐づく評価点を取得する

ryu41

総合スコア5

SQL Server

SQL Serverはマイクロソフトのリレーショナルデータベース管理システムです。データマイニングや多次元解析など、ビジネスインテリジェンスのための機能が備わっています。

SQL

SQL(Structured Query Language)は、リレーショナルデータベース管理システム (RDBMS)のデータベース言語です。大きく分けて、データ定義言語(DDL)、データ操作言語(DML)、データ制御言語(DCL)の3つで構成されており、プログラム上でSQL文を生成して、RDBMSに命令を出し、RDBに必要なデータを格納できます。また、格納したデータを引き出すことも可能です。

0グッド

0クリップ

投稿2019/10/19 04:58

前提・実現したいこと

・各商品(記事)かつ特定のユーザーが投稿した平均評価(5段階)を算出するSQLを作成したいです。

■投稿した口コミの属性情報が格納されているDB(postmeta)
・投稿するたびに、post_id:500,501…とレコードが追加されていきます。
・meta_idは単なる一意のキーです
・meta_key, meta_value
meta_keyの内容に応じて、meta_valueに値が表示されます。
wpcr3_review_post:レビューを投稿した記事(商品)のID
wpcr3_review_name:投稿者の名前
wpcr3_review_rating:投稿された評価(5段階)

meta_idpost_idmeta_keymeta_value
1500wpcr3_review_post10
2500wpcr3_review_nameeditor
3500wpcr3_review_rating5
4501wpcr3_review_post11
5501wpcr3_review_nameyuki
6501wpcr3_review_rating3

発生している問題・エラーメッセージ

各商品毎の平均評価算出は①のSQL分で正しく動くのですが、各商品かつ特定のユーザー(meta_value='editor'のユーザー)という条件を指定したく、 WHERE句の中に、AND (meta_value='editor' and meta_key='wpcr3_review_name'))
というのを追記すると、上手く値が取れません。

該当のソースコード

①各商品毎の平均評価算出

$review_query="SELECT round(AVG(meta_value),1) AS review_avg,count(meta_value) AS review_cnt FROM $wpdb->postmeta WHERE post_id IN (SELECT post_id FROM $wpdb->postmeta WHERE (meta_value=". get_the_ID() ." and meta_key='wpcr3_review_post') AND meta_key='wpcr3_review_rating'"

②各商品毎かつ特定のユーザーの平均評価算出

$review_query="SELECT round(AVG(meta_value),1) AS review_avg,count(meta_value) AS review_cnt FROM $wpdb->postmeta WHERE post_id IN (SELECT post_id FROM $wpdb->postmeta WHERE (meta_value=". get_the_ID() ." and meta_key='wpcr3_review_post') AND (meta_value='editor' and meta_key='wpcr3_review_name')) AND meta_key='wpcr3_review_rating'"

試したこと

上記に記載済

補足情報(FW/ツールのバージョンなど)

None

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

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

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

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

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

guest

回答2

0

典型的なDB設計のアンチパターンエンティティ・アトリビュート・バリュー(汎用的な属性テーブル)になってます。
RDBでこれをすると、今ハマっているように複雑で判りづらく遅くて間違いやすいクエリになってしまうのです。

ざっくりとですが、
reviewerテーブル
reviewer_id, name

itemテーブル
item_id, name

postテーブル
post_id, item_id, reviewer_id, rating

こんな感じのテーブルにすると良いでしょう。

投稿2019/10/19 08:33

hihijiji

総合スコア4150

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

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

ryu41

2019/10/20 06:02

ご回答いただきありがとうございます。 エンティティ・アトリビュート・バリューという考え方は知らなかったのですが、頂いたリンクから見てみました。 処理が複雑化する原因になるのですね。 Wordpressのプラグインを使っているので、なかなかDB構成を変更するまでの知識は無いのですが、自分で作るときは、気を付けるようにします!
guest

0

ベストアンサー

()は正しく閉じられていますか?
SQL的には以下のはずです。

SQL

1SELECT round(AVG(meta_value),1) AS review_avg, count(meta_value) AS review_cnt 2FROM postmeta 3WHERE meta_key='wpcr3_review_rating' 4 and post_id IN ( 5 SELECT post_id FROM postmeta 6 WHERE meta_key='wpcr3_review_post' and meta_value='10' 7 ) 8 and post_id IN ( 9 SELECT post_id FROM postmeta 10 WHERE meta_key='wpcr3_review_name' and meta_value='editor' 11 )

コスト的には、existsで判断した方が効率的かもしれません。

SQL

1SELECT round(AVG(meta_value),1) AS review_avg, count(meta_value) AS review_cnt 2FROM postmeta p 3WHERE meta_key='wpcr3_review_rating' 4 and exists( 5 SELECT 1 FROM postmeta 6 WHERE post_id=p.post_id and meta_key='wpcr3_review_post' and meta_value='10' 7 ) 8 and exists( 9 SELECT 1 FROM postmeta 10 WHERE post_id=p.post_id and meta_key='wpcr3_review_name' and meta_value='editor' 11 )

こっちの方が早いかも

SQL

1SELECT round(AVG(rating.meta_value),1) AS review_avg, count(rating.post_id) AS review_cnt 2FROM postmeta rating 3 inner join postmeta post 4 on rating.post_id=post.post_id 5 and post.meta_key='wpcr3_review_post' and post.meta_value='10' 6 inner join postmeta name 7 on rating.post_id=name.post_id 8 and name.meta_key='wpcr3_review_name' and name.meta_value='editor' 9WHERE rating.meta_key='wpcr3_review_rating'

構造的に三つのメタ属性がセットなら、結合条件と抽出条件に明確に分けた方が、可読性は良くなります。

SQL

1SELECT round(AVG(rating.meta_value),1) AS review_avg, count(rating.post_id) AS review_cnt 2FROM postmeta post 3 inner join postmeta name 4 on post.post_id=name.post_id 5 and post.meta_key='wpcr3_review_post' and name.meta_key='wpcr3_review_name' 6 inner join postmeta rating 7 on post.post_id=rating.post_id 8 and rating.meta_key='wpcr3_review_rating' 9WHERE post.meta_value='10' and name.meta_value='editor'

投稿2019/10/19 08:00

編集2019/10/19 09:08
sazi

総合スコア25138

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

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

ryu41

2019/10/20 06:03

様々なオプションまでご教示頂きありがとうございました。 無事動作するとともに、とても勉強になり感動しました!!!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問