SQL文(A) or (B)どちらを使うべきでしょうか？

前提・実現したいこと

プログラム(※例 JAVA,PHPなど)で使用するSQL文として、
下記(A)(B)どちらのSQL文を使用する方が望ましいのか、理由・ご意見を頂けたら幸いです。

<MySQL> (A) SELECT IFNULL(name, 'UNKNOWN') FROM user; (B) SELECT CASE WHEN name IS NULL THEN 'UNKNOWN' ELSE name END FROM user;

(A)
<メリット>
・SQL文が見やすい

<デメリット>
・DBが変更になった際に、修正が必要

(B)
<メリット>
・DBが変更になった際に、修正が不要

<デメリット>
・SQL文が(A)より見にくい。

前提

・DB変更する予定はない
・複雑なSQL文の場合、IFNULL関数を使用しても良い

発生している問題

単純なSQL文の場合、標準SQL文で書いた方が良いか？
レビューを受けた際に、どちらが良いか激論になった。

その他

・上記「単純なSQL文の場合」といった条件があると、一貫性がなくなる。＋線引きが必要になる。

疑問

・開発現場毎に、合わせるべきだろう？
・そもそもどちらでも良い？混在していても良い？
・意識として標準SQL文で書く事を心掛けるべきだろうが、今回の場合どっち？う～んとなる。

補足情報

仮にDB変更になった場合、現状主要DBに関しては置換で一発変換できる。

<Oracle> SELECT NVL(name, 'UNKNOWN') FROM user; <PostgreSQL> SELECT COALESCE(name, 'UNKNOWN') FROM user; <SQL Server> SELECT ISNULL(name, 'UNKNOWN') FROM user;

行動規範の内容に同意します

回答7件

ベストアンサー

(A)と(B)を兼ね備えています。

どちらが良いかなんて、目的が決まれば自ずと決まります。

投稿2020/03/04 16:15

sazi

総合スコア25426

kimura-i

2020/03/04 16:56 編集

なるほど～といった感じです。 SQL総合1位の方からのご意見、とても勉強になりました。貴重なご意見ありがとうございます。 teratailに質問してよかったです(^^)♪ ちなみに、現情報で(A) or (B)２択だとどちらがお勧めなんでしょうか。目に留まったらお聞きしたいです(´･ω･｀) 要するに今回はcoalesceという抜け道？をお聞きする事ができましたが、IFNULLを一例と考えて頂いた場合。著しくSQL文がみにくくならないのであればDB変更に備えたSQL文を心掛けるべきか、わかりやすさ重視で行くべきかが今回の議題でもありまして。。。欲しがりな僕、ごめんなさい。

sazi

2020/03/04 18:56

制約なしに、個人的に(A)か(B)のどちらか選べと言われるなら(A)です。理由は記述の簡潔さです。

kimura-i

2020/03/05 00:24

ご回答いただき、誠にありがとうございます。同じ考えで少し自身が付きます(*´ω｀*)

sazi

2020/03/05 06:09 編集

> coalesceという抜け道 coalesce()は標準です。 IFNULL()と違い、引数は幾つでも指定できますので、coalesce()の方が強力です。

kimura-i

2020/03/05 14:26

標準でした。すみませんありがとうございます。

行動規範の内容に同意します

(A)でよいと思います。

DBが変更になった場合、確かに(B)であればこのSQLについては修正が不要かもしれません。しかし、だからと言って動作確認の作業が不要になるわけではありませんし、DB種類に依存しないSQLのみを利用してシステムを作成するのは現実的には難しいです。
であれば「DBが変更になった際に、修正が不要」というのはメリットとしてほぼ無意味でしょう。また「複雑なSQL文の場合、IFNULL関数を使用しても良い」という前提があるのであれば、一貫性の観点からもIFNULL関数に統一すべきだと思います。

投稿2020/03/04 15:42

ku__ra__ge

総合スコア4524

kimura-i

2020/03/04 16:16 編集

>だからと言って動作確認の作業が不要になるわけではありません >、DB種類に依存しないSQLのみを利用してシステムを作成するのは現実的には難しい貴重なご意見ありがとうございます。おっしゃる通りだと思います。 DB変更の際に、完全には難しいですが修正箇所をなるべく減らせるからという理由です。

kimura-i

2020/03/05 14:31

今回の主題に一番沿ったご意見を下さいました。本当にありがとうございます('ω')ノ

行動規範の内容に同意します

SQL
1select coalesce(name,'UNKONWN') as name,case when name is null then 1 else 0 end as name_is_null from tbl;
2

投稿2020/03/05 01:51

yambejp

総合スコア117618

yambejp

2020/03/05 01:53

nameの中にすでに"UNKOWN"と入力されている場合を考慮すると A,Bどちらかではなく、両方併用する必要がある場合もあります

kimura-i

2020/03/05 14:12

貴重なご意見ありがとうございます。すでに"UNKOWN"が入力されているかの考慮、忘れないよう心掛けたいと思います。

行動規範の内容に同意します

みなさま貴重なご意見ありがとうございました。
(A) or (B)であれば、やはり(A)で良いんだと再認識する事ができました。

また、今後はnullチェックにcoalesceを使う事も検討して行きたいなと思いました。
今回の場合、coalesceが最適ですが、現場によりIFNULLの方がわかりやすいなど出てくるのかな～と思いました。

投稿2020/03/05 14:18

kimura-i

総合スコア5

余計ですがnameのNULL、空文字、スペース、全角の時も考えてみました。NULL保障されていれば不要です。

sql
1select if(trim(both from replace(name,'　',' '))>'',name,'UNKNOWN') from test;

投稿2020/03/05 05:45

編集2020/03/05 05:47

amura

総合スコア333

kimura-i

2020/03/05 14:12

ありがとうございます。

行動規範の内容に同意します

(A)が良いと思います。
簡潔で見易い事は、後任に対しても大事な事で、解析し難いPGは後任へ混乱を
招きシステムの不具合を招きかねません。

DB変わった場合のデメリットですが、仮に変わったとしましょう。
現場ではどう対応しますか？
DB変わったけど、動作するから問題なし！でリリースします？
しませんよね、普通は全体的に問題ないか等の調査は確実に行います。
なのでデメリットという感じではなくなると思います。

いままで沢山のSQLを見てきましが、やはり後任には見易さが大事です。
複雑や、やたら長いSQL書くと読み辛いので、解読に時間を要します。
むしろココがデメリットと捉える箇所かと思っております。

投稿2020/03/04 23:23

NEO_PLANETT-777

総合スコア333

kimura-i

2020/03/05 00:35 編集

貴重なご意見ありがとうございます。 >DB変わった場合のデメリットですが、仮に変わったとしましょう。 >現場ではどう対応しますか？私自身３回ほどDB変更対応した経験がありますが、どの現場でもすべてのSQLを見直し、必要な個所を修正していきます。なるべく見やすく同じ記述をしていると、修正が楽だった覚えがあります。 >やはり後任には見易さが大事です。 >やたら長いSQL書くと読み辛いので、解読に時間を要します。やはりDB変更する際の事考えても、見やすさを重視するべきですね。

行動規範の内容に同意します

私の場合、実務でSQL文を書くことも全くありませんし、そうそう個人開発でも使わないことをご承知おきください。あと完全に主観です。

ここは完全に好みなのですが