ワンランク上のSQLを書くためのポイント3つ - Qiita

この記事はNuco Advent Calendar 2022の17日目の記事です。 ワンランク上のSQLとは？ 1年近く、データ分析に関わる業務に携わっってきた中で、良いSQL、いまいちなSQLをいろいろ見てきました。 自分が書くSQLも、最初の頃は目も当てられないSQLも書いてきました。そんな中で、こんなことを意識していくと、より良いSQLになるのでは？というポイントをまとめていきます。 とりあえずSQLの文法は一通り勉強して、取得したいデータをとってくるSQLをかけるようになったぞ。という人に向けたものなので、本当に基礎的な文法は解説していません。 ワンランク上のSQLを書くためのポイントは、 ・読みやすい ・再利用しやすい ・処理が早い の３つを押さえられているかどうかだと感じています。 可読性が高いメリット 間違いにくくなる/デバックが容易になる エラーが出てくれれば間違っているこ

[diveintounlimit]

“予約語/関数は大文字にする ”で何かもういいやってなった。その直後の例文でもうcount小文字だし

[chiroruxx]

命名についてはあまり共感できず。僕は省略しないほうが好き。

[mak_in]

ワンランクでなく基本だった。基本は大事。さらにいうとコレがちゃんと適用できるように、ちゃんとテーブル設計をせよ、だよな。実はそれのほうが難しかったり。ドメインの複雑度をどこで吸収するのか問題に帰結する

[kwsktr]

土日に頑張って投稿したのに、めちゃくちゃ言われてかわいそうな気持ちになった。タイトルに「ワンランク上のSQL」と入れてしまうところは、若さもあると思いますが勇気があって素敵だと思いました。頑張ってください

[miquniqu]

中身がわかる変数名（カラム）でuuになっちゃったのはマジ？と思った。

[mayoshino]

"デバック"が気になる

[quabbin]

COUNT(column_name)とCOUNT(*)の区別がついてない時点で、やり直して来たほうがいい感が結構する。COUNT内のCASE文は逆に、COUNT(column_name)で十分に見えるしなぁ。

[JackP]

例のSQL文の品質が良くない（ある名前が場所によって大文字、小文字バラバラで著者当人がLv.1未満というのは中々…）変数名の短縮もやりすぎでは？そもそもカラム名を変数って。採用担当者ですか……株式会社Nucoさんね

[shag]

俺も昔予約語は大文字で書いていたけど、今時シンタックスハイライトなしで SQL 書かないし、小文字で書いた方が書くの楽なのよね。

[nakamura-kenichi]

クエリの書き様とかテーブル設計次第やからなｗ。そものテーブル設計があかんかったら何をどうやってもしゃーないやで。

[itotto]

記事は複数人で書いたのかな？

[ya--mada]

ワンランク上。とりあえず先ずはランキングの概観書こうぜ。

[hdampty7]

ケースバイケースってところまでたどり着くのが目標じゃね？

[zkq]

これ多分イメージしていることを言語化できないタイプの人かも

[shinobue679fbea]

ワンランク上でなくても良い。せめて発行回数とジョインの規模だけは考えて書いて欲しい

[ragi256]

click_logsでuser_idがnull考慮しないとか、JOINの説明で常にLEFT JOINを例に出すところとか、色々もうちょっと説明を追加して欲しいなって。あと最後のSQL、10/31の分をとれてますか？ 2022-10-31 00:00:00 までになってない？

[Kil]

click_uuなんて略すくらいならclick_unique_userと書いて欲しい。むしろ、明記しなくてもuniqueは自明だろって扱いでclick_userでいい気がする、略したいなら。

[Shinwiki]

処理系に任すよ。つーかjoinからあとわけわからん

[saiid]

予約語小文字派

[kabochatori]

コードの書式が揃ってないのはどっかからコピペしてきたんだろうなとググってみたら案の定‥。

[mysql8]

マテビューもいいよ

[GARAPON]

頑張って欲しい

[delphinus35]

distinct case when is not などなど……これら全部予約語じゃないのか？？

[kagehiens]

インデックスとGROUP BYの絡め方とかの話かと思ったら違ってた。予約語は先頭大文字で書くのが個人的には見やすい。

[yarumato]

“Lv.2 再利用性が高いSQLにする　Lv.3 処理時間が短いSQLにする”

[asamaru]

確かにワンランクだった。些事徹底。

[kaerucircus]

“mial_id”が気になってその先進めない

[civicpg]

最後のはJOINする前に共通テーブル式WITHで男女のみのレコードに絞ってからJOINした方が良いと解釈したのだけど、JOINのONで男女の条件もANDでつける方が簡潔じゃないかな。ON click.user_id = info.user_id AND info.sex IN (1,2)

[hasiduki]

where句の位置でパフォーマンス変わるってマジかよ！？！？？？？うそくせー！！！！！知らんけど！！！！！！

[eighth8]

とてもわかりやすい