nabokov7; rehash : O/Rマッパーはなぜ悪か

December 07, 201208:49 カテゴリプログラミングmysql O/Rマッパーはなぜ悪か タイムラインで「SQL上級者こそ知って欲しい、なぜO/Rマッパーが重要か？」ってのを見かけて居ても立ってもいられなくなったので、既出を承知で反論しておきたい。 スライドだけから話の内容を推測すると、 -- 販売成績上位10個を抽出 select * from sales where deleted = false order by amount desc limit 10 といったSQLを Sales.active().top(10).all() のように、細かく分解した部品を組み合わせて表現できた方が便利だし構造的でしょ？という話のようだ。 これは確かに一見美しいのだが、これこそが「敷居を下げすぎて、dbの性質を分かってない人まで気軽にSQLをいじるようになった結

[tripleshot]

いまどきのORMはそんなクソSQL吐かないし、ORM 作ってる人たちが SQL のシロウトって可能性は、新人がいきなり非常に効率の良い SQL を書ける可能性より低いんじゃないですかね。

[rgfx]

ORMはSQL吐いてるうちはクソ、というかDBMSがSQL読んで立てる実行計画よりも効率良く、DBMSのAPIを直接叩く様になれば話を聞いてやらんでもない感。

[moro]

『たぶん、(略)テーブルから全行をスキャンし、クライアントアプリケーションへネットワーク越しに転送する』(ｷﾘｯ)とか言っちゃうレベルで絡まれたらkwatch気の毒すぎるというか、さすがにひどいだろこれ。

[nakag0711]

大体真のsqlerはビジネスロジックを全てストプロで書きクライアントにテーブルへのアクセス権なんて付与しない。スケールはしにくいが

[ka-ka_xyz]

せめてN+1問題とか・・・・。てか業務に使えるレベルで成熟したORMで、件数制限をかけてるにもかかわらず全行取得するようなSQLを生成する奴って居るんだろうか。

[ledsun]

全行取ってロジックで絞るでいいじゃない。遅かったらチューニングすればいいじゃない。

[NOV1975]

論旨には同意できるけど、そこまでアホなコードは生成されないだろうし、最後の話は、クソSQLを書く奴とクソO/Rマッパーのどっちがましか程度の話でしかない

[katin]

元のスライドで「等価である」と明示されているにも関わらず、脳内でアホなロジックに置き換えちゃうのはちょっと

[lombriz]

少なくとも、元ネタに書かれてる例ではほぼ等価なSQLが生成されたはず。全行スキャンなどというあほな事にはならない。言いたいことはわかるし、O/RマッパーのはくSQLは微妙なものも多いけど、ちょっと甘く見すぎ。

[houyhnhm]

関数の動作のタイミング勘違いしてると思う。ただ、結局SQLにして飛ばしてるなら、SQL生成プログラム書いている程度になっちゃいますよね、みたいな所で。O/R Mapperが効いて来るのは持って来てからでしょーね。

[nacika_inscatolare]

こういう形式が怖いな・・

[As_hsp]

こういう形式が怖いな・・

[morygonzalez]

アホでもいじれるようにしてしまうと、案の定アホがいじるようになって、結果アホでない人が迷惑する

[estragon]

O/Rマッパーはなぜ悪か

[k-holy]

この話題、定期的にあがりますね。O/Rマッパー使ってないけど、複合インデックス考慮したSQL吐いてくれるのかな

[ham007]

O/Rマッパーに限らず。 ＞アホでもいじれるようにしてしまうと、案の定アホがいじるようになって、結果アホでない人が迷惑する

[iaskell]

最近SQL（とrails）触り始めてまだあんまり深く理解できてないんだけど、何故かこの気持ちは逆に分かる

[lzfx]

むかし日本の岡崎市に・・・＞これもDBコネクションをセッションに格納していた結果、コネクションが確保できなくなる不具合でO/Rマッパーは関係ない。

[airj12]

最近のO/Rマッパーはもうちっと賢いだろうとは思いつつも気持ちはわかる / SQLServerがLINQを直解釈し始めたら呼んで下さい

[hatz48]

"結局、出力ログを見ながら、望みどおりのSQLが発行されるように O/Rマッパーの記述を調整する、みたいな本末転倒の作業が不可欠になる"

[SUM]

「SQLと取得される結果が同じとしても内部のやり方まで同じとは限らず、ひどいときはSQLよりとことんひどい」というのはまったくその通りだろう。

[zetamatta]

あぁ、やっぱり糞なO/Rマッパーだと、安易に全行スキャンの SQL を発行してしまうのね

[yayugu]

予期せぬクエリが飛ぶ可能性があるってだけでも動的SQL生成はこわいわー

[mikage014]

CakePHPだとORMとPaginatorが不可分なのでORMから望みのSQLが出てるかはログをチェックする。ORMで作りづらい複雑なJOINはDB側でviewを作るけど管理が宙ぶらりんになるのが悩みどころ。

[hiroyuki1983]

うーむ。

[mumincacao]

jQuery もだけど敷居下げすぎたってのはあるかもかなぁ？ ただ某図書館のは O/R まっぱなんて高級品使わず SQL 直でせっしょん毎に接続創りまくる残念仕様が原因だったはずなのです（・ｘ【みかん

[kiyo_hiko]

読んだ

[akasata]

DB知らない人間がO/Rマッパーを使うべきでないというのは、現状正しいと言わざるを得ない。ただし、本記事のサンプルは多くの事例に当てはまらないので適切ではないってとこか

[tsucchi1022]

もうちょいORM が賢くならないといけないし、それまでは ORM による抽象化は諸刃の剣ですよねー

[Dolpen]

(；´Д｀)

[koyancya]

言ってる事は良く分かる。しかし、だからこそ「SQL 上級者こそ知って欲しい」と前置きしているのではないだろうか。

[todesking]

ひどい記事だ(；´Д｀)

[Naruhodius]

(；´Д｀)