URIヘルパーのAPI https://github.com/iiirc/iiirc/pull/31/files#r3885445

KitaitiMakoto

 tikeda: Atomフィード出すようにしました 11banyan

 tikeda: http://iiirc.org/snippets.atom 0

 banyan: great! 0

 tikeda: コメントの意味がよく取れませんでした 0

 banyan:  0

 banyan: sorry for that... 0

 banyan: 単純にコード読んでて、あの箇所だけ 0

 tikeda: 言いたいことは分かったけど英語が難しくてそっちは分からなかったw 0

 banyan: ちょっとだけ readable じゃなかった気がしました (全然細かいところなんでどうでもいいところですけどw 0

 banyan: user っていう argument は user instance を期待して見てるけど、 0

 banyan: もし、user.respond(:username) じゃなかったら 0

 banyan: username に user を突っ込む 0

 banyan: というのがん?と思いました 0

 banyan: 下見たらなるほどと思いましたけどw 0

 banyan: こういうのどう思いますか? 0

 banyan: という質問でしたw 0

 tikeda: i'll do when this case... how do you think? 0

 tikeda: これがよく分からなかったんですよねw 0

 tikeda: ああ 0

 tikeda: 「もしおれだったらこうするけど」ということですか 0

 tikeda: i'll do 0

 banyan: すんませんw 0

 banyan: 当たりですw 0

 tikeda: 今理解しました 0

 tikeda: 言いたいこと自体はすぐ分かりましたけど。 0

 banyan: なるほどw 0

 tikeda: github_user_urlはsnippet_url(snippet)みたいなのを意識していて 0

 tikeda: だからUserオブジェクトでもユーザー名の名前(String)でもいいようにしたかったです 0

 tikeda: github_user_url自体の定義はリーダブルじゃない(変数名が悪い)のはそうなんですが 0

 tikeda: github_user_urlを使う側のコードは読みやすくなる、というRailsのやり方に効いました 0

 tikeda: 倣い 0

 tikeda: もっとpath_forをまんまぱくったような書き方のほうが意図が伝わったんですかね 0

 tikeda: あれ確か引数の型で判定していたので 0

 tikeda: 或いはgithub_user_url(user_or_username)ならよかったかな・・・ 0

 tikeda: これで納得できなかったら 0

 tikeda: 僕が悪いと思うので 0

 banyan: そうねぇ、僕も user_or_username という案は思いました 0

 tikeda: ハセコさんのあんに変えましょう 0

 banyan: user_or_username.respond_to(:username) 0

 tikeda: github_user_urlの中が汚くなるのは別にいいかなと思ってます・・・短いメソッドなんで・・・ 0

 banyan: というのもなんだかなぁと思ったw 0

 banyan: なるほどw 0

 banyan: まぁそうねぇ 0

 tikeda: これが5行とかになるなら 0

 tikeda: ちょっと考えますけど 0

 banyan: 僕は 0

 banyan: こういう時には 0

 banyan: もし常に User#github_url としてユーザインスタンスから呼び出す、ということになってれば 0

 banyan: そうするし、そうでなくて 0

 banyan: 単純に username だけ渡してどこからでも github_user_url(username) として呼び出すほうが明示的に u.username 0

 banyan: とかにしたほうがシンプルかな、とも思うのですが 0

 banyan: 単純でバカっぽさはあるかもしれません 0

 banyan: 単純に意見が聞いてみたかっただけなので 0

 tikeda: というとじゃあ 0

 tikeda: snippet_urlでSnippetオブジェクトでもID(Integer)でもどっちでもいいってのは 0

 tikeda: 実は好きではない? 0

 tikeda: snippet_url(snippet) 0

 tikeda: snippet_url(snippet.id) 0

 banyan: そうですね。どちらかといえば好きではないですね 11KitaitiMakoto

 tikeda: おおそうなんだw 0

 tikeda: Rails使ってる人みんな好きなんだと思ってました 0

 tikeda: ちなみに僕も好きじゃない 0

 tikeda: ので 0

 tikeda: usernameにしましょうかw 0

 banyan: https://github.com/rails/rails/pull/9419 0

 tikeda: おおー 0

 banyan: https://github.com/rails/rails/pull/9441 0

 tikeda: これすごい 0

 tikeda: <コメントが 0

 banyan: この時、こういうふうな 0

 banyan: root helper だけ引数が string か 0

 banyan: hash か 0

 banyan: みたいな感じの特別扱いされたんですけど 0

 banyan: これも正直思いついたけど 0

 banyan: こんな特別なことするのはそもそもどうなの?そして pullreq あっさり通った。えー 4KitaitiMakoto

 banyan: とか一人で思ってましたw 0

 tikeda: そして pullreq │ 0

 tikeda:  | あっさり通った。えー 0

 tikeda: これすごいそうおもうw 0

 tikeda: weechat・・・ 0

 banyan:  0

 banyan: ちなみに 0

 banyan: snippet_url(snippet) 0

 banyan: の snippet にあたる奴の変数名ってなんなのだろう... 0

 banyan: ちょっと rails みてみよう 0

 banyan: 変数名さえ納得感あれば 0

 banyan: まだ全然いいですねw 0

 tikeda: http://apidock.com/rails/ActionView/Helpers/UrlHelper/url_for 0

 tikeda: options={} 0

 banyan: なるほど 0

 tikeda: あれちがうなこれ 0

 banyan: 引数 hash じゃないすね 0

 tikeda: http://apidock.com/rails/ActionController/PolymorphicRoutes/polymorphic_path 0

 banyan: いいのか 0

 tikeda: さいごのpolymorphic_pathに落ちて 0

 banyan: record_or_hash_or_array 0

 tikeda: こっちのhttp://apidock.com/rails/ActionController/PolymorphicRoutes/polymorphic_path 0

 banyan: おー 0

 tikeda: そうそう 0

 tikeda: なんか貼り付けがうまくいかない・・・ 0

 tikeda: user_or_username・・・ 0

 tikeda: recored_or_string 0

 tikeda: ActiveDecoratorを使えって話になるのかなあ。 0

 tikeda: まあいいや取り敢えずここはusernameにして、String前提でいくことにします 0

 banyan: はいw 0

 banyan: すませんw 0

 tikeda: いえ 0

 tikeda: ハセコさんもsnippet_url(snippet)好きじゃないとしってびっくりしましたw 0

 tikeda: へんに気を回す必要なかった 0

 banyan: これ着回されてたのかw 0

 tikeda: つねに「これ好きじゃないけどRailsっぽいように思えるからこうしよう」みたいな葛藤がありますw 90banyanonk

 banyan: wwwwwwww 0