tikeda: Atomフィード出すようにしました ★11
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 じゃなかった気がしました (全然細かいところなんでどうでもいいところですけどｗ ★0
banyan: user っていう argument は user instance を期待して見てるけど、 ★0
banyan: もし、user.respond(:username) じゃなかったら ★0
banyan: username に user を突っ込む ★0
banyan: というのがん？と思いました ★0
banyan: 下見たらなるほどと思いましたけどｗ ★0
banyan: こういうのどう思いますか？ ★0
banyan: という質問でしたｗ ★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: すんませんｗ ★0
banyan: 当たりですｗ ★0
tikeda: 今理解しました ★0
tikeda: 言いたいこと自体はすぐ分かりましたけど。 ★0
banyan: なるほどｗ ★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: というのもなんだかなぁと思ったｗ ★0
banyan: なるほどｗ ★0
banyan: まぁそうねぇ ★0
tikeda: これが５行とかになるなら ★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: そうですね。どちらかといえば好きではないですね ★11
tikeda: おおそうなんだｗ ★0
tikeda: Rails使ってる人みんな好きなんだと思ってました ★0
tikeda: ちなみに僕も好きじゃない ★0
tikeda: ので ★0
tikeda: usernameにしましょうかｗ ★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 あっさり通った。えー ★4
banyan: とか一人で思ってましたｗ ★0
tikeda: そして pullreq │ ★0
tikeda: | あっさり通った。えー ★0
tikeda: これすごいそうおもうｗ ★0
tikeda: weechat・・・ ★0
banyan: ｗ ★0
banyan: ちなみに ★0
banyan: snippet_url(snippet) ★0
banyan: の snippet にあたる奴の変数名ってなんなのだろう... ★0
banyan: ちょっと rails みてみよう ★0
banyan: 変数名さえ納得感あれば ★0
banyan: まだ全然いいですねｗ ★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: はいｗ ★0
banyan: すませんｗ ★0
tikeda: いえ ★0
tikeda: ハセコさんもsnippet_url(snippet)好きじゃないとしってびっくりしましたｗ ★0
tikeda: へんに気を回す必要なかった ★0
banyan: これ着回されてたのかｗ ★0
tikeda: つねに「これ好きじゃないけどRailsっぽいように思えるからこうしよう」みたいな葛藤がありますｗ ★90
banyan: ｗｗｗｗｗｗｗｗ ★0