MySQLでの売上集計

質問

お世話になっております。MySQLのSQLに関する質問ですが、変数で処理しているため、PHPカテゴリで質問させて下さい。

テーブル：slip
slip_no　　member_no　　sales_date
1　　　　　　2　　　　　　2013-06-06
2　　　　　　1　　　　　　2013-06-07
3　　　　　　4　　　　　　2013-06-08
4　　　　　　3　　　　　　2013-06-08

テーブル：sales
sales_no　　sale_slip_no　　goods_no　　goods_name　　request_money　　receipt_money
1　　　　　　　1　　　　　　　　2　　　　　　キュウリ　　　　5250　　　　　　　　NULL
2　　　　　　　1　　　　　　　　3　　　　　　イチゴ　　　　　5250　　　　　　　　NULL
3　　　　　　　2　　　　　　　　6　　　　　　バナナ　　　　　5250　　　　　　　　NULL
4　　　　　　　2　　　　　　　　5　　　　　　りんご　　　　　5250　　　　　　　　NULL
5　　　　　　　2　　　　　　　　point　　　　NULL　　　　　　NULL　　　　　　　　3000
6　　　　　　　3　　　　　　　　1　　　　　　ほうれん草　　　9450　　　　　　　　NULL
7　　　　　　　3　　　　　　　　4　　　　　　しいたけ　　　　2100　　　　　　　　NULL
8　　　　　　　4　　　　　　　　7　　　　　　マンゴー　　　　6300　　　　　　　　NULL

このようなテーブルがあります。(MySQL5)

slip_noとsale_slip_noは紐付くデータで、slipテーブルにあるmember_no毎に、
salesテーブルのgoods_noが整数を条件に、request_moneyの1ヵ月前の期間内の
request_moneyの合計を算出したいと考えております。

member_no 1を呼出したら、
total 10500
とこのように算出。

以前、http://okwave.jp/qa/q8117105.html
こちらで、ご親切に回答頂き、解決したのですが、このテーブルに関連する質問を
http://okwave.jp/qa/q8126398.html
こちらで伺ったところ、ご親切に正規化を指摘され、テーブルを上記のように分割したという経緯が御座います。

以上のことから、
http://okwave.jp/qa/q8117105.html
でアドバイス頂いたSQLを元に、inner join を用いて以下のようなSQLを試していますが、得たい結果を得ることが出来ません。

直接関係ないかもしれませんが、FROMの前にmember_noやsale_slip_no、slip_noなどのカラムを記述するとエラーになったりで、かれこれ3時間もこのSQLで格闘しています。

$sql = "SELECT SUM(request_money) FROM sales INNER JOIN slip ON sales.sale_slip_no = slip.slip_no WHERE slip.member_no = '1' AND sales.goods_no REGEXP '^[0-9]+$' AND slip.sales_date between curdate() - interval 1 month + interval 1 day AND curdate()";
$result = mysql_query($sql);
$sum_point = mysql_fetch_assoc($result);
$one_month_buy_money = $sum_point['SUM(request_money)'];

最近、質問ばかりで恐縮ですが、アドバイスのほど頂戴出来ればと質問させて頂きました。
お忙しい中恐縮ですが、宜しくお願い申し上げます。

.

回答No.1

agunuz · ベストアンサー

>MySQLのSQLに関する質問ですが、変数で処理して
>いるため、PHPカテゴリで質問させて下さい。

発想が逆です。まずMySQLでキチンと動くSQL文を組み立ててから、それの一部を（プレースホルダなどを使って）変数に置換する順番でやるべきです。

MySQLのマニュアルを読んだりphpMyAdminなどでSQL文を実行させて試行錯誤せずに、どうやってSQL文を覚えるのでしょう・・・

>FROMの前にmember_noやsale_slip_no、slip_noなどのカラムを記述するとエラーになったりで、

selectに記述した列は（sumの項目があるので）group by に記述していないといけません（その項目毎に集計すると明示する）。

select slip.member_no, sum(request_money) as total from sales join slip on sales.sales_slip_no = slip.slip_no group by slip.member_no where (ここにwhere条件を列記)

where条件にmember_noがあるなら（特定のメンバーの合計だけでいいなら）selectにslip.member_noは不要ですし、griup by もしなくていいです。

select sum(request_money) as total from sales join slip on sales.sales_slip_no = slip.slip_no where (ここにwhere条件を列記)

とりえあずphpMyAdminでSQL実行して「どういうエラーになるか」（ここがおかしいという部分が表示されるハズです）を確認してください。

なお「php」という話で言えばネイティブなMySQL関数を使っている時点で論外です。今から書くのであればMySQLiもしくはPDOで書きましょう。

お礼

agunuzさま

ご親切なアドバイスをありがとうございます。
また、ご挨拶が遅れて申し訳ありません。

> なお「php」という話で言えばネイティブなMySQL関数を使っている時点で論外です。
> 今から書くのであればMySQLiもしくはPDOで書きましょう。

やはりそうなんですね。自身、PHPとMySQLをサイト構築のために同時に勉強を始めた（当初購入した本が同時に話を進めていた）ので、うまく言えませんが同じ延長線上で物事を考えていました。

まだ、一部WEHER句においてMySQLでの関数が残っているので、この処理をどうしようか？など課題はありますが、SQLを見直し期待している結果を得ることが出来ました。

貴重なアドバイスをありがとうございます。

回答No.2

yambejp

途中経過をすっとばすとして

$one_month_buy_money = $sum_point['SUM(request_money)'];
でSUMした集計値を変数に取り込めないというだけの話であれば

$sql = "SELECT SUM(request_money) AS total ・・・";
というように別名を付けておき

$one_month_buy_money = $sum_point['total'];
のように別名でアクセスします

お礼

yambejpさま

お世話になっております。毎度ご親切なアドバイスをありがとうございます。
また、ご挨拶が遅れて申し訳ありません。

まだまだ理解が乏しい身、分かっているようで分からないことだらけで、色々なアドバイスはとても勉強になります。

今回のASでの指定の仕方、勉強になりました。
まだまだ見直すべきところがありますが、とりあえず正常に稼働するまでに至りました。
ありがとうございました。