Content-Length: 214161 | pFad | http://b.hatena.ne.jp/paulownia/%E3%82%B3%E3%83%BC%E3%83%89%E3%83%AC%E3%83%93%E3%83%A5%E3%83%BC/
コードレビューのときに見ているところ - 詩と創作・思索のひろば
あるときコードレビューするときにどういうところ見てるんですか？ と訊かれてたしかに自分でもあまり言語化したことはなかったな、と気づいたので簡単に書いておく。 変更意図が要求に沿っているか そもそも実現しようとしていることが、ユーザやプロダクトオーナーの要求に沿っているか。モデリングや実装のコンテキストを自分でも把握しておく。 関連する別の変更やイシューなど、自分が知っていて相手が知らない有意義な情報があったらコメントする。 モデリングが妥当か モデルによって意図が表現できているか。仕事が適切な粒度で明確に切り分けられているか。意図のない共通化がなされていないか。 わかりやすい名前がつけられているか。ここが混乱していると何かがよくないサイン。既存のコードがすでに……ということもある。そういう場合は改善できそうな道筋について議論できるとベター。 仕事にあったインタフェースになっているか。テスト
よいレビューの方法や考え方とは？全エンジニアがレビュー前に知っておきたいこと - 仮想サーファーの日常
Webエンジニアとして働いていると、新規機能の設計レビューや実装レビューをすることがよくありますよね。エンジニアなら当たり前のようにやっている業務です。 ですが、「よくよく考えてみると設計レビューや実装レビューの方法をしっかり時間を使って学んだことってないなぁ...。」という方も意外と多いのでは。 ぼく自身どうやってレビューしてるっけな？と振り返ってみると、「なんとなくコードを読んで、仕様漏れがないか、考慮漏れでバグになるケースがないかをチェックしている」くらい。 普段Webエンジニアとして働いている方の中でも、自分の中でしっかりとレビューの方法を確立できていない方も多いのではと思います。 今回は、Webエンジニアが設計やコードのレビューをするときに意識しておきたいこと、エンジニア初期のころに知っておきたかったレビューの方法と考え方をまとめました。 よりよいレビューの方法と考え方を知ってお
DMMにおけるユーザーレビュー基盤の変革(心理的安全性を高めるワークショップを行いました) - DMM inside
ほぼリモートなのにコミュニケーションの不満はゼロ！DMMポイントクラブグループが実現した「エンジニアの理想の働き方」
コードレビューに費やす時間を短くする - クックパッド開発者ブログ
はじめに こんにちは、広告事業部の芳賀（@func09）です。普段はクックパッドの広告配信周りや純広告・タイアップ広告などの商品開発を行っています。 私が広告事業領域の仕事をするようになって、そろそろ１年になるのですが、初めはエンジニア以外の人（営業、編集、広告入稿、レポート、メール配信、などなど様々な担当者がいます）と業務をすることが多くてコミュニケーションが上手くいかず業務がスムーズに進まないことがありました。 当たり前のことではありますが、エンジニアにしかわからない言葉は使わないとか、できるだけ相手の業務を理解し相手の考え方や視点に立って話すなど、ちょっと工夫することで、長引きがちなMTGや相談がすんなり終わったり、お互い良い気分で終わることが多くなって、費用対効果が高いなと感じています。 一方でエンジニア同士のコミュニケーションでも時間がかかってコストが高いと感じることがあります。
デキるプログラマだけが知っているコードレビュー7つの秘訣
SonicGarden Study #11で放送された資料から一部スライドを抜いたものになります。 http://sonicgarden.doorkeeper.jp/events/13229 ----- 優れたプログラマだけが優れたソースコードを書くことができます。 では優れたプログラマになるにはどうすれば良いでしょうか。 自分の書いたコードを、優れたプログラマに指摘してもらうことが一番の近道です。それがコードレビューです。たった一人でコードレビューも受けずに、ただ書き続けてもクソコードはクソコードのままなのです。 そこで今回は、良いコードが書けるプログラマになるための、コードレビューを上手に実践する秘訣を話します。Read less
1
リリース、障害情報などのサービスのお知らせ
最新の人気エントリーの配信
j次のブックマーク
k前のブックマーク
lあとで読む
eコメント一覧を開く
oページを開く
Fetched URL: http://b.hatena.ne.jp/paulownia/%E3%82%B3%E3%83%BC%E3%83%89%E3%83%AC%E3%83%93%E3%83%A5%E3%83%BC/
Alternative Proxies:
