タグ

code-reviewに関するsomathorのブックマーク (23)

  • モデルやメソッドに名前を付けるときは英語の品詞に気をつけよう - Qiita

    はじめに 他の人が書いたコードを読んでいるときに時々気になるのが、英語の間違いです。 特に動詞、名詞、形容詞の使い分けが間違っていたりすると、かなり違和感を感じます。 そこで今回はモデル(=クラス)やメソッドに名前を付けるときの基的な原則をまとめてみます。 また、英文法的に正しい品詞が選べるようになるための習慣についても最後に説明します。 想定する言語/フレームワーク この記事の説明ではRuby/Ruby on Railsを想定しています。 ただし、基的な考え方は他の言語でも同じように使えるはずです。 モデルの名前は名詞にする 例: 「支払い情報」を表すモデルを作りたい場合 × Pay ○ Payment 「支払う = payか。よし。」でモデルを作ってはいけません! payは動詞で、payの名詞形がpaymentです。 Payモデルではなく、Paymentモデルを作りましょう。 例:

    モデルやメソッドに名前を付けるときは英語の品詞に気をつけよう - Qiita
  • 些末なコードレビュー - naoyaのはてなダイアリー

    朝起きて布団から出るのがつらいので、HBFav をつらつらと眺めていた。 あるサービスの JavaScript が重いとか、そのコードが難読化されてないとか、担当者とおぼしき人間が書いたコメントがそのまま残ってるから消しましょうよとか、そんなことが書かれていた。JavaScript が重い、という話は結局そのサービスの JavaScript が重かったのではなく、ユーザーが自分で導入した広告が重いというだけの話だった。 コードが難読化されていない、趣味の製品ではなく会社の製品なのでコメントそのまま残ってるから消しましょう・・・実にくだらない。 ところで話は変わってコードレビューについて。 コードレビューに慣れないチームが、何の考えもナシにコードレビューを始めるととにかく気になったこと大小様々な指摘が行われることになる。一見、いろいろな指摘が出て議論が活発になっているように見えるが、だいたい

    些末なコードレビュー - naoyaのはてなダイアリー
  • レビュアーとレビュイーの関係に関して - 職質アンチパターン

    感じていた違和感の正体がわかった. レビューにおいて,レビュアーとレビュイーの関係には上も下もない. レビューという場では,両者の立場は対等でなければならない.さもなければ,「このレビューおかしい気がするけれど,あの人は立場が上だから指摘しにくい」だとか,「相手は格下だから適当にレビューしても良い」だとか,そういう良くない雰囲気が形成されてしまう. 確かに,レビュアーというポジションはチームの技術力が高い人やそれに伴ってパワーのある人が担うケースが多いと思う.レビュイーはそれに萎縮してしまいがちというか,僕もその1人なんだけれど,そういうのは実際のところ保身でしかなくて,下手に意見言うとレビュアーとの関係悪くなりそう,みたいな卑屈な理屈に基づく駄目な萎縮な感じがする.こういう良くないところはちゃんと直して,健全化していかなければまずい. あとレビューされた内容は素直に受け入れるべきだと思う

    レビュアーとレビュイーの関係に関して - 職質アンチパターン