Laravel4のstaticコールについてのやりとり

Hirohisa Kawase @HiroKws

頑固プログラマーとLaravel開発者のTaylor氏が、討論しています。というより、頑固だから、Facadeでスタティックメソッドも実はオブジェクト生成しているのを受け入れがたいよう。 要は、staticに見えてしまうからなんでしょう。

Masao Maeda @brtriver

マジックメソッドはできるだけ使いたく無い派 / “Laravel 4 Uses Lots of Static, Not True.” http://t.co/MocYqyb4Uh

Masashi Shinbara @shin1x1

まあ面白いこと考えるよね / “Laravel 4 Uses Lots of Static, Not True.” http://t.co/XZTHB5CMKE

Masashi Shinbara @shin1x1

@tanakahisateru つ RT @shin1x1: まあ面白いこと考えるよね / “Laravel 4 Uses Lots of Static, Not True.” http://t.co/XZTHB5CMKE

田中ひさてる @tanakahisateru

@shin1x1 読んだ。やはり魔法少女ララベル。ところでどうやってモック差し込むのかわからなかったけど、ようするにインスタンス見つけてそっち使えということなのかなー

cakephper ichikawa @cakephper

Static callを窓口にしてるだけだから、laravel4がstatic地獄みたいな印象を持つのは間違いという記事。 マジックメソッド__callStatic()を使ってそれっぽく見せてるだけ。 / “Laravel 4 U…” http://t.co/CELEUeaHuW

Masashi Shinbara @shin1x1

@tanakahisateru モック差し込むなら、テスト時はこれ書き換えることになるのかな。 https://t.co/xMPD5XIEcf

Hidenori Goto | COO and CTO at KabuK Style @hidenorigoto

@cakephper 「それっぽく見せてるだけ」ではないと思います。内部ではstaticではないオブジェクトで構成されている部分もありますが、アプリケーションではstaticに呼び出す以外に機能させる方法が無いと認識しています。

田中ひさてる @tanakahisateru

@shin1x1 これ指してるのがクラス名だから、テストケースごとに別々なクラスファイルがいっぱい要りそうな...

Masashi Shinbara @shin1x1

@tanakahisateru 実はモック用クラスがフレームワークに用意されているとか。で、テストケースで生成したモックのインスタンスをモック用クラスに差し込むとそっちを呼んでくれるとか。（ソース見てないけど）

Hidenori Goto | COO and CTO at KabuK Style @hidenorigoto

@shin1x1 @tanakahisateru プロダクション用Facadeがモック機能持ってたはず。。。 http://t.co/SlGAZkfky1 (コード https://t.co/2cA2hr9ugH)

Masashi Shinbara @shin1x1

@hidenorigoto @tanakahisateru @kenji_s なるほどー。この辺ですね。 https://t.co/OL81ObwAWN

Hirohisa Kawase @HiroKws

Laravel4のコアのstaticが裏でインスタンス化されているということが、日英で今日は良くツイートされています。 staticメソッドのほうが、読み書きしやすが、テストはできないという点を解決するための方策です。 ですから、読みやすいと思わない人には、役に立ちません。

Hirohisa Kawase @HiroKws

Laravelのファサードの仕組みは、読み書きしやすさとテスタビリティーの両立を目指しています。 コアクラスに対し、依存注入を行わなくても、CoreClass::shouldReceive('メソッド名')で、Mockeryのモックが用意されます。

Hirohisa Kawase @HiroKws

ドキュメントにも軽く書かれ、日本語のブログ記事も既にいくつか存在し、テスト本と構造本でも取り上げられていますが、英語のブログ記事をきっかけに、今日は話題として取り上げられているようです。

Hidenori Goto | COO and CTO at KabuK Style @hidenorigoto

@HiroKws 目指す意図は理解します。ですが、モックライブラリにMockeryしか使えない点、そのテストを動作させるためにLaravelコアに依存してしまう点など、標準で汎用性のある方法にはなっていないと私は認識しています。

Hirohisa Kawase @HiroKws

@hidenorigoto その通りです。:D フレームワークですから。結局は好みなのです。 全てのフレームワークが、汎用性を目指しているわけでないでしょうし、少なくともLaravelの人気は、お手軽さがあってのことですしね。 依存注入でコーディングしてもいいわけですし。

田中ひさてる @tanakahisateru

@shin1x1 @hidenorigoto @kenji_s みなさんありがとうございますー。それにしてもインスタンスまでが遠いというか、モックごときにコンテナ全体を巻き込む感じというか...Laravel難しい子

Hidenori Goto | COO and CTO at KabuK Style @hidenorigoto

@tanakahisateru コンテナで管理されているオブジェクトで、プロダクションコード内でstaticコールしているやつをモックする場合の機能ですよね。

田中ひさてる @tanakahisateru

よく考えたらこれすごい状況なんじゃないかと… https://t.co/v3djM1zcDf

Hirohisa Kawase @HiroKws

２０年くらい前でも、UIに関しては、難しさはユーザーに隠しなさい、というのはよく知られていました。 便利なライブラリー、いくつかのプログラムパターンは、内部構造を利用者から隠す目的です。 そうしたアイデアをフレームワークであるLaravelは取り入れているだけです。

田中ひさてる @tanakahisateru

@hidenorigoto コンテナでFacadeと対応してるDB::とかの単位でのみ、コールを騙せるという感じなのは見えました。

Hirohisa Kawase @HiroKws

開発者のTaylorは、Symfonyのミートアップでもレクチャーしてますし、そもそもSymfonyとは方向性が違います。ですから、SymfonyからLaravelへの釣り記事は少ないです。 同じ方向性ではありませんが、お手軽さが同程度なのかCI=>Laravelは多いです。

田中ひさてる @tanakahisateru

@hidenorigoto ここだけの話(になってないけど)、スタティックコールが気になる点は、教育的にどうかというところだったりします。Laravel本体は中でいろいろ工夫してても、そこまで見てないユーザは、スタティックで設計するのはアリアリなんだって思ってしまわないかなーと。

Hirohisa Kawase @HiroKws

ちなみに、Laravelに関してずらずら、ツイートしているのは、Laravelで検索した時に、別の見方もあることを示すためにやっていることで、特定の意見の反論を目的としていません。 フレームワーク選定中の人の、頭を一方に向いてしまうのを中立へリセットするためです。