sql_modeのonly_full_group_byを守っていないためにエラーになった件


# 問題
![スクリーンショット 2020-11-04 午後1.44.09.png](https://qiita-image-store.s3.ap-northeast-1.amazonaws.com/0/547640/cba28b48-c13e-79a6-ba4e-2d47e5f3e3c2.png)

グラフを表示しようとしたら下記のエラーが表示された。

```
Mysql2::Error: Expression #1 of SELECT list is not in GROUP BY clause and contains nonaggregated column 'pfc-master_production.posts.created_at' which is not functionally dependent on columns in GROUP BY clause; this is incompatible with sql_mode=only_full_group_by: SELECT `posts`.`created_at` FROM `posts` WHERE `posts`.`user_id` = 8 GROUP BY date(created_at)
```

```ruby:charts_controller.rb
@dates = dates_calorie.map { |dates| dates.created_at }
```

# エラーログを読み解く

```
Mysql2::Error: Expression #1 of SELECT list is not in GROUP BY clause and 
Mysql2 :: Error：SELECTリストの式＃1がGROUP BY句になく、

contains nonaggregated column 'pfc-master_production.posts.created_at' 
which is not functionally dependent on columns in GROUP BY clause;
GROUPBY句の列に機能的に依存していない非集計列 
'pfc-master_production.posts.created_at'が含まれています

this is incompatible with sql_mode=only_full_group_by: 
これはsql_mode = only_full_group_byと互換性がありません

SELECT SUM(`posts`.`calorie`) AS sum_calorie, date(created_at) AS date_created_at FROM `posts` WHERE `posts`.`user_id` = 8 GROUP BY date(created_at) ORDER BY created_at DESC
```

**mapメソッドとは？**
>・mapメソッドを一言で表すと「各要素へ順に処理を実行してくれるメソッド」です。
・要素それぞれにアクセスし、指示した処理を行ってくれる。
[【Rails入門】mapメソッドを完全攻略!配列操作の基礎を学ぼう](https://www.sejuku.net/blog/58958)

```
配列変数.map {|変数名| 具体的な処理 }

今回の場合
@dates = dates_calorie.map(配列変数) { |dates(変数名)| dates.created_at(処理) }
```

**`dates_calorie`という配列には何が入っているのか？**

```ruby
dates_calorie = current_user.posts.group("date(created_at)").select(:created_at)
```
ログイン中ユーザーの投稿が1日に複数ある場合、その日ごとでグループ化し`created_at`カラムを取得したものが入っている。
すなわち、1日につき1つの`created_at`カラムが取得され、1日ごとの`created_at`カラム情報たちが配列の中に入っている。

このような情報を取得する理由は、1日ごとの体重を表示するグラフを実装するためだ。
1日に複数の投稿があった時に、グラフに表示するのは1日につき1つだけにしたいためだ。

✖️ [2020-11-09 00:00:00, 2020-11-09 00:11:00, 2020-11-10 00:12:00]
○ [2020-11-09 00:00:00, 2020-11-10 00:12:00, 2020-11-11 00:12:00]

**mapメソッドを使ってどのような処理をしているのか？**

```ruby
@dates = dates_calorie.map(配列変数) { |dates(変数名)| dates.created_at(処理) }
```

上記で説明したとおり、配列`dates_calorie`には1日ごとの`created_at`たちが配列の中に入っていて、それをmapメソッドを使って配列の中の各要素をdatesという変数で1つずつ取り出し、created_atを取得している。

それらをgonでグラフを表示させるchart.jsに渡してグラフで使うために、インスタンス変数に代入している。

# なぜ今回エラーになってしまったのか？

only_full_group_byがわからなかったため、リファレンスでonly_full_group_byについて調べた。

**sql_mode = only_full_group_byとは？**
>GROUP BY 句で名前が指定されていない非集約カラムを、選択リスト、HAVING 条件、または (MySQL 5.6.5 以降で) ORDER リストが参照するクエリーを拒否します。

>ONLY_FULL_GROUP_BY が有効な場合、次のクエリーは無効です。1 番目は、選択リスト内の非集約の address カラムが GROUP BY 句で名前を指定されておらず、2 番目は、HAVING 句の max_age が GROUP BY 句で名前を指定されていないため、ともに無効になります。

>[MySQLリファレンス ONLY_FULL_GROUP_BYとは？](https://dev.mysql.com/doc/refman/5.6/ja/sql-mode.html)

sql_modeとは、MySQLの公式ルールのようなもので、only_full_group_byはMySQL5.6.5以降で新たに設定されたルールらしい。

・only_full_group_byというルールを守っていないためにエラーになったと考えられる。
・GROUP BY句で名前をしていないためエラーになってしまったと考えられる。

**GROUP BY句とは？**
>・「GROUP BY」は、グループ化を行うために使用される命令。
・主に「種類ごとに集計関数を使用する」などといった形で使用するケースが多い。
・例えば「チームごとの人数を調べる」という場合など。
[【SQL】GROUP BYで自在に集計!集計関数やHAVINGと合わせて使おう](https://www.sejuku.net/blog/72923)

```
select [表示する要素名] from [テーブル名] GROUP BY [グループ化する要素名];
SELECT team, COUNT(team) FROM user GROUP BY team;
```

# 対策
上記のことから、考えられる対策は2通りだ。

① SQLモードのonly_full_group_byを遵守するため、GROUP BY句を使ったコードに変更する。
② my.cnf(MySQLの設定ファイル)を変更し、only_full_group_byという制約を外す。

今回は②の方法を選択した。
理由は、時間がないためだ。

現在webエンジニアへの転職活動を始めるところで、とにかく一刻も早く企業の採用担当者の方にアプリを見せられる状態にしなくてはならない。
本当は①が望ましいが、今回は時間優先のため②を選択した。

### 今回の解決方法

```
my.confの場所を探す
[naota@ip-10-0-0-32 ~]$ mysql --help | grep my.cnf
                      order of preference, my.cnf, $MYSQL_TCP_PORT,
/etc/my.cnf /etc/mysql/my.cnf /usr/etc/my.cnf ~/.my.cnf 

my.confを開く
[naota@ip-10-0-0-32 ~]$ vi /etc/my.cnf

読み込み専用ファイルを上書き保存する時に使う
:w !sudo tee %
```
`my.conf`に下記の記述を追加した。

```:my.conf
[mysqld]
sql_mode=STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION
```
`NO_ZERO_IN_DATE` 、`NO_ZERO_DATE`といったsql_modeを調べてみたが、これら自体に`only_full_group_by`をOFFにする意味はないとわかった。
おそらく上記の設定で`only_full_group_by`にしたのOFFにしたのではなく、`only_full_group_by`を含んで書いていないから設定されていないのだと考えられる。

→ `only_full_group_by`追加したら冒頭のエラーが発生したので、この仮説は正しかった。

上記の設定を反映させるためにMySQLを再起動した。

[MySQL5.7にアップデートしたらonly_full_group_byでエラーになった](https://www.p-nt.com/technicblog/archives/204)

```
MySQLを停止
[naota@ip-10-0-0-32 ~]$ sudo systemctl stop mysqld.service

MySQLを起動
[naota@ip-10-0-0-32 ~]$ sudo systemctl start mysqld.service
```

グラフが表示されました！
![スクリーンショット 2020-11-04 午後1.45.51.png](https://qiita-image-store.s3.ap-northeast-1.amazonaws.com/0/547640/4421e7fa-90ea-b9b2-2ea4-59ffd0d1857e.png)


# ソースコード
charts_controller.rbはグラフを表示させるためのコントローラー。

```ruby:charts_controller.rb
def index
    # カロリー
    @sampleuser = User.find_by(id: 3)
    if user_signed_in?
      # 日付ごとで分けてカロリー合計を算出
      sum_calorie = current_user.posts.group("date(created_at)").sum(:calorie)
      # 日付ごとのカロリー合計がハッシュの形なので値を取得して配列に入れて変数に代入
      array_calorie = sum_calorie.values
    else
      sum_calorie = @sampleuser.posts.group("date(created_at)").sum(:calorie)
      array_calorie = sum_calorie.values
    end
    # gonを使ってデータをjs側に渡す
    gon.data = []
    # mapメソッドで日付ごとのカロリー合計を1つずつ取り出す
    # mapメソッドの使い方 → 配列変数.map {|変数名| 具体的な処理 }
    gon.data = array_calorie.map { |calorie| calorie }

    if user_signed_in?
      # 日付ごとにまとめてそのうちcreated_atカラムだけ取得。配列の形で格納されている
      dates_calorie = current_user.posts.group("date(created_at)").select(:created_at)
    else
      dates_calorie = @sampleuser.posts.group("date(created_at)").select(:created_at)
    end

    gon.date = []
    @dates = dates_calorie.map { |dates| dates.created_at }
    # each文で日付の表記を1つずつ取り出して変える
    @dates.each do |a|
      gon.date << a.strftime("%Y年%m月%d日")
    end


    # 体重
    if user_signed_in?
      gon.weight = current_user.posts.group("date(created_at)").select(:weight).map { |weight| weight[:weight] }
    else
      gon.weight = @sampleuser.posts.group("date(created_at)").select(:weight).map { |weight| weight[:weight] }
    end
  end
```