PHP DBからカウントしながら日付で抽出

Question

お世話になります。

引き続きアンケートの機能の作成でてこずっています。

テーブル名 an
an_no 自動で番号が入ります

an_time 回答したタイムスタンプ

an_key 親回答からの番号が入っています

an_data 回答内容が文字で入っています。

下記の内容のSQL文で、同じ回答だった場合（文字が同じ）に、カウントして、後の処理で件数が表示できています。
たとえばA市３件といった感じです。

これに、日付で抽出を加えようと、
$sql='SELECT an_data ,count(an_data) FROM an group by an_data';の文を
$sql='SELECT *,count(an_data) FROM an group by an_data,an_time HAVING (an_time) BETWEEN :before AND :after';のように変えたところ、
抽出内容は一見同じで日付で絞込ができたように見えますが、
A市2件であるべきところがA市1件に減ってしまいます。他の重複して件数が複数でないといけないところも全部1件になってしまいます。

このSQLの使い方はそもそも問題があるのでしょうか？
違う方法でもいいので、カウントと、日付による絞り込みができたらと思います。

分かりにくくて申し訳ないですが、お知恵をお借りしたいと思います。
よろしくお願いします。

require_once( __DIR__.'/common/database.php' ); 
$dbh=new PDO($dsn,$dbUser,$dbPass);
$dbh->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);
$dbh->setAttribute(PDO::ATTR_EMULATE_PREPARES, false);
    
    
//回答の集計をしたい部分です HAVING BETWEENの行を使うと・・・
//$sql='SELECT *,count(an_data) FROM an group by an_data,an_time HAVING (an_time) BETWEEN :before AND :after';   
$sql='SELECT an_data ,count(an_data) FROM an group by an_data';    
$stma=$dbh->prepare($sql);    
$stma->bindValue(':before',$before,PDO::PARAM_STR);
$stma->bindValue(':after',$after,PDO::PARAM_STR);       
$stma->execute();
  
while(true)
{   
        $reca=$stma->fetch(PDO::FETCH_ASSOC);
        if($reca==false)
        {
            break;
        }
        $key=$reca['an_key'];
        $key2=$reca['an_data'];
        $count_kaitou[$key][$key2]=$reca['count(an_data)'];  
} 

//後の処理で、$count_kaitouから内容を表示したりしています。

Answer 1

多分質問者様がしたいことをSQLに表すと、

SELECT
    *
,   count(an_data) 
FROM
    an 
GROUP BY
    an_data
,   an_time 
HAVING
    (an_time) BETWEEN :before AND :after

これは誤りで下記に直す必要があるのかなと推測します。
(※コメントに注釈を書いたので意識してみてください)

SELECT
    an_data
/* 下のようにGROUP BY指定してない項目を単純に指定するのはNG
 * どうしても取りたいなら「MAX(an_no) AS an_no」みたいに集約関数を噛ませましょう。
 */
--    *
,   count(an_data) 
FROM
    an 
WHERE
    an_time BETWEEN :before AND :after
GROUP BY
    an_data
/* 下のように集約単位は変えちゃいけないはず・・・ */
--     , an_time 
/* 集約関数噛ませたものに条件指定するのでなければ、
 * HAVING句は必要ないでありますよ、隊長！
 */
-- HAVING
--     (an_time) BETWEEN :before AND :after

Comments

[mi_]

ご提示いただいた内容で、重複した名前と、その名前毎の件数がとれて、日付で抽出もできました！
コメント頂いた内容しっかり理解して、時々見返したいと思います。この内容が1行にまとまるところ、SQLのすごさにびっくりします。ありがとうございました。

[Panzer_vor]

＞ mi_さん
SQLも他の言語と同じく出来ることが増えてくると楽しいですよ。
覚えるべきことは多いかもしれませんが、
トライアンドエラーで試行錯誤しながら是非力を付けていってください。

どうしても煮詰まった際には、
経緯と問題点、試したことをうまく提示して質問できるようになれば、
より的確な回答も付きやすくなるでしょう。

[mi_]

ありがとうございます！
みなさんに回答いただき、直接助けてもらっていますし、視野も広がって助かっています。

煮詰まった際の、質問の仕方も、冷静になってよく考えるようにしたいと思います。

Answer 2

$stma->fetch(PDO::FETCH_ASSOC) をループしていないからでは？

Comments

[mi_]

いつもありがとうございます。
上記では、あまりいいループにはならないでしょうか？
抽出の仕方をかえてしまったので、後の処理もがらりと変えなければいけないということでしょうか？

[退会済みユーザー]

いったい何を参考に書いて、このソースになったんだろうと思うくらい、謎な仕様です。
SQLについてはノーチェックです。テーブル定義も不明だし、中に入っているでたーも提示されていないので実行不可能。プログラム上のおかしなところは見ればわかるが、それ以外は読んでいません。

[Panzer_vor]

横槍ですが、
修正後のSQL自体本当だと構文エラーとなる書き方なんですよね・・・＾＾；

MySQLだから動くには動きますが、
なるべくGROUP BY句を利用した際に、
SELECT項目にはどういったものが集約関数噛ませずに指定できる・できないを抑えてもらいたい所・・・

[mi_]

Kosuke_Shibuya様　なんとか動きそうとおもって、固執してしまい、知識もないままどんどん複雑にしてしまっておりました。お恥ずかしい限りです。

Panzer_vor様
構文エラーとなるような内容なのですね。おっしゃるところを考えてみたいと思います。