共有
  • 記事へのコメント62

    • 注目コメント
    • 新着コメント
    その他
    bake0937
    今思ったけど,回によってコードレビューする時上手くできた・できなかったのバラツキあるな...もっとレニューの質を上げていかないとな...

    その他
    mtomitomi
    “気”

    その他
    Kil
    いいところは褒める、に尽きる。個人的には、先に不足している部分・直すべき部分を指摘→良い点に触れ、褒める、の順が良いと思っている。「直さないとダメ」と言わず「直したらもっとよくなるね」と言う感じ。

    その他
    capytan_el34
    “の”

    その他
    telegib
    “モックの数: モックは素晴らしいものですが、全てをモックで代用するのはよくありません。私は自身の経験から、1つのテストに3つ以上モックがある場合は再考が必要だと判断しています。こういう時はテストの範囲が

    その他
    rjge
    “開発者として、私たちはコードの動作とメンテナンス性の両方に責任があります” 後者は忘れられがちだったりするけどほんと重要だよね…

    その他
    hiwa4
    “コードレビューにおいて最も重要なことの1つは開発者の成長や努力に報いることです。同僚からの賞賛以上に嬉しいことなど、あまりないでしょう” それな

    その他
    shiopon01
    これみんなやってるんやな“目を細めてコードをチェック”

    その他
    lalupin4
    確かに俺もディスプレイから離れてコードチェックしてるな。画面が黒すぎたらダメ、白すぎても何か問題がある。> 目を細めてコードをチェック

    その他
    hr-tachikawa
    うお、ちょうどこういうの必要だった

    その他
    junglejungle
    こんなコードレビューは嫌だ:結合テストが終わってからコードレビュー

    その他
    otihateten3510
    これ自分で直すのはいいけど指摘されるのは辛い

    その他
    suzukiMY
    http://kevinlondon.com/2015/05/05/code-review-best-practices.html

    その他
    sisopt
    sisopt “1つのクラスは変更する理由が2つ以上あってはいけないという考え方です。”ん、何でこの訳になった?原文では1つのクラスは唯一の責任を持つようにと書いてあるけど…

    2019/06/13 リンク

    その他
    tofu-kun
    セルフレビューをしたかどうかが大きい要素なんじゃないかと思ってる。自分のコストをチームメンバーに転嫁するのをなるべく避ける。ただ抱え込みすぎても仕方がないので、そこは気楽に。

    その他
    tatsu_tora
    "人から指摘されるより自分で気づいた方が気持ち的にも楽"、確かに

    その他
    kbysta1
    同感 "コードのメンテナンス性を高めることは、バグを引き起こしているコードを修正するのと同じぐらい重要です。"

    その他
    d_animal141
    コードレビューのベストプラクティス | 開発手法・プロジェクト管理

    その他
    takc923
    コードレビューする度にこれ読んでちゃんと出来てるか確認したい

    その他
    bricklife
    「1つのテストに3つ以上モックがある場合は再考が必要」「褒める、グッドプラクティスを強化する」

    その他
    igrep
    褒めるのね、忘れないように。もちろんほかも大事だけど。

    その他
    mas-higa
    レビューに限らない話

    その他
    fukajun
    fukajun だいたい、やってる気がする。けど一覧化されてるのはいいな。

    2015/06/24 リンク

    その他
    nilab
    コードレビューのベストプラクティス | 開発手法・プロジェクト管理 | POSTD

    その他
    katzchang
    katzchang “レビューした人がコードの一部について質問してきた場合、それは今後、他の人にも混乱を引き起こす可能性があるでしょう”

    2015/06/17 リンク

    その他
    shiba_yu36
    途中のコードレビューのやり方が参考になる

    その他
    ashitakaeruu
    品質確保

    その他
    yasuharu519
    コードレビューについて

    その他
    bleu-bleut
    単一責任原則:1つのクラスは変更(?)する理由が2つ以上あってはいけない。開放/閉鎖原則:拡張に対して開いていて修正に対して閉じているか。

    その他
    d4-1977
    まだまだコードレビューの勘所がわかってないので助かる

    その他

    注目コメント算出アルゴリズムの一部にLINEヤフー株式会社の「建設的コメント順位付けモデルAPI」を使用しています

    アプリのスクリーンショット
    いまの話題をアプリでチェック!
    • バナー広告なし
    • ミュート機能あり
    • ダークモード搭載
    アプリをダウンロード

    関連記事

    コードレビューのベストプラクティス | POSTD

    Wiredrive では、私たちはかなりの数のコードレビューを行います。しかし、ここで働き始める前には私は...

    ブックマークしたユーザー

    すべてのユーザーの
    詳細を表示します

    同じサイトの新着

    同じサイトの新着をもっと読む

    いま人気の記事

    いま人気の記事をもっと読む

    いま人気の記事 - テクノロジー

    いま人気の記事 - テクノロジーをもっと読む

    新着記事 - テクノロジー

    新着記事 - テクノロジーをもっと読む

    同時期にブックマークされた記事

    いま人気の記事 - 企業メディア

    企業メディアをもっと読む