Conversation

Notices

1. のえる :cava_red: DTP鯖管 (noellabo@dtp-mstdn.jp)'s status on Tuesday, 25-Dec-2018 22:52:05 JST のえる :cava_red: DTP鯖管

   • tateisu​ :force:​:r_9a:
   • Maya Minatsuki :neko_smiley:

   @tateisu @mayaeh こちらの件、
   https://mastodon.juggler.jp/@tateisu/101301391429326074

   とりあえずそのままプルリク投げました。
   https://github.com/tootsuite/mastodon/pull/9625

   • Maya Minatsuki :neko_smiley: (mayaeh@taruntarun.net@taruntarun.net)'s status on Wednesday, 26-Dec-2018 00:02:11 JST Maya Minatsuki :neko_smiley:

     in reply to

     • tateisu​ :force:​:r_9a:

     @noellabo @tateisu ありがとうございます！
     この記法でいいのか私には判断できないためすぐには行動できませんが、疑問などコメントあれば協力したいと思います。

   • のえる :cava_red: DTP鯖管 (noellabo@dtp-mstdn.jp)'s status on Wednesday, 26-Dec-2018 00:18:12 JST のえる :cava_red: DTP鯖管

     in reply to

     • tateisu​ :force:​:r_9a:
     • Maya Minatsuki :neko_smiley:

     @mayaeh @tateisu
     この記法もアリかな。

     def require_user!
     if current_user && !current_user.disabled? && current_user.confirmed?
     set_user_activity
     elsif current_user && current_user.disabled?
     render json: { error: 'Your login is currently disabled' }, status: 403
     elsif current_user && !current_user.confirmed?
     render json: { error: 'Email confirmation is not completed' }, status: 403
     else
     render json: { error: 'This method requires an authenticated user' }, status: 422
     end
     end

   • Maya Minatsuki :neko_smiley: (mayaeh@taruntarun.net@taruntarun.net)'s status on Wednesday, 26-Dec-2018 00:58:11 JST Maya Minatsuki :neko_smiley:

     in reply to

     • tateisu​ :force:​:r_9a:

     @noellabo @tateisu 私はこちらの方が好きです…が、独学の私にはどちらの記法がより良いのか私には判断できません。

   • のえる :cava_red: DTP鯖管 (noellabo@dtp-mstdn.jp)'s status on Wednesday, 26-Dec-2018 07:30:46 JST のえる :cava_red: DTP鯖管

     in reply to

     • tateisu​ :force:​:r_9a:
     • Maya Minatsuki :neko_smiley:

     @mayaeh @tateisu あっさりマージしてくれましたね。

     私は、最初に提案した書き方が、シンプルで重複がなく、ミスが混入しにくい点で優位と考えています。判定を追加するのもシンプルに対応できるし。正常判定のパスは先頭か末尾のいずれかであればどちらでもいいんじゃないかな。

     ただ、独学というか、rubyに至ってはまともに文法勉強会してないので(!)ガイドラインなどがあるのかどうか知らなかったり……。

     ちゃんと目を通したのはcrystalのリファレンスだけw

   • のえる :cava_red: DTP鯖管 (noellabo@dtp-mstdn.jp)'s status on Wednesday, 26-Dec-2018 08:07:40 JST のえる :cava_red: DTP鯖管

     in reply to

     • tateisu​ :force:​:r_9a:
     • Maya Minatsuki :neko_smiley:

     @mayaeh @tateisu 正常系のパスをif〜endの外に出した方がわかりやすい、とかはあるかもしれないなぁ。ifで判定するブロックは全部エラー判定（before_action的な）にするとか。

     などという、特に結果に寄与しないコード談義も楽しいのであります。戦争が起きる系ですがw

   • Maya Minatsuki :neko_smiley: (mayaeh@taruntarun.net@taruntarun.net)'s status on Wednesday, 26-Dec-2018 08:49:59 JST Maya Minatsuki :neko_smiley:

     in reply to

     @noellabo ?
     ひとつ勉強になりました、ありがとうございます！
     Crystal 、リレーのカスタマイズですね。

   • のえる :cava_red: DTP鯖管 (noellabo@dtp-mstdn.jp)'s status on Wednesday, 26-Dec-2018 10:17:48 JST のえる :cava_red: DTP鯖管

     in reply to

     • Maya Minatsuki :neko_smiley:

     @mayaeh 私はアセンブリ言語とかCの時代が長かったので、同じチェックが何度も書かれているの見ると避けたくなるんですよねぇ。

     最近の言語は、タブかスペースか、などの不毛な議論が起きないようにフォーマッタがついてたり（crystal tool format とか go fmt）するので、良い時代だなと思います。

     ruby、railsの流儀は、すごーくなんとなくやってるので、もう少しマジメに勉強します……。

     sumiyaki likes this.