たのしくなるコードレビュー - クックパッド開発者ブログ
こんにちは！サービス開発部でAndroidアプリの開発をしているこまたつ（@k0matatsu）です。 みなさんコードレビューしていますか？ 最近ではとりいれているチームも多いと思いますが、良い効果をもたらしてくれる一方で、負荷の高い作業でもあります。 また、コードレビュー自体に馴染みの薄かった人はなにをどうしたらいいのか難しいですよね。 同僚から得たアドバイスと自分なりのノウハウをあわせて、コードレビューの指針を考えていたので公開してみようと思います。 前提として、クックパッドではGitHub Enterpriseとプルリクエストを使った開発プロセスを採用しています。 また、コードレビューの前には自動テストと静的解析ツールによる単純なフォーマット、コードスタイル、頻出バグのチェックは行われているものとします。 静的解析による機械的なチェックはコードレビューよりも低コストで有効な方法ですの
コードレビューのツールについての調査 - Qiita
このドキュメントはコードレビューを支援するツールについての調査を行った結果を記述する。 ツールの紹介 Redmine Code Review プラグイン Redmineのプラグインとして動作する。 http://www.r-labs.org/projects/1/wiki/Code_Review インストール方法 通常のRedmineのプラグインと同様 使用方法 リポジトリ―でコミット済みのリビジョンの差分を取り、レビュー用のコメントを記述する。 作成したレビューはプロジェクトのコードレビュータブで一覧として表示される。 記述したレビューはチケットとして登録される。 そして、レビューで作成したチケットはリポジトリから返信とステータスの遷移が行える。 Trac Peer Review Plugin Tracのプラグインとして動作する。 http://trac-hacks.org/wiki/P
お手軽お気楽コードレビューシステム! codebrag に驚いた
はじめに# Subversion をつかっていて, ソースコードレビューでコードにコメントを書きたいと思った. なにか, お手軽にためせるツールはないか探して見たところ, Codebrag というものを見つけたので試す. Codebrag - your daily code review tool Codebrag とは# オープンソースのコードレビューツール. サーバにインストールして, Web 画面からコードレビューができる. Git と Subversion に対応しているようだ. Subversion は, git-svn をつかって使えるようにしているようだ. インストール# インストール手順は以下にある. https://github.com/softwaremill/codebrag/wiki/Installation 以下から, ダウンロード. ダウンロードでメールアドレス
Gerrit Code Review | Gerrit Code Review
Discuss code Read old and new versions of files with syntax highlighting and colored differences. Discuss specific sections with others to make the right changes. Manage and serve Git repositories Gerrit includes Git-enabled SSH and HTTPS servers compatible with all Git clients. Simplify management by hosting many Git repositories together. Schedule git gc over all managed repositories and replica
メンバー育成にも効果アリ！ コードレビューを充実させ、不具合件数を半減させたJVCケンウッドのテストツール活用術
ソフトウェア開発の現場では、「高品質」「低コスト」「短納期」が要求され、そのための取り組みが行われている。テスト工程においてもこれらの条件を実現するため、ツールを導入が進んでいるが、思ったよりも効果が出ていないという声も多い。ではどのようなツールをいかに運用すれば、高い品質のソフトウェアを低コスト、短期間で開発できるのか。「開発現場の意識が変わる！ 品質向上を実感する『仕組み』作りとは？」と題して、20年以上のテストツール販売実績を誇るテクマトリックス主催のセミナーが開催された。 ツールの導入に加え、メンバーの育成、開発チームの活性化に注力 テスト工程にツールを導入したものの、品質の向上、開発の効率化が図れていない──。こんな課題を持っている開発現場は多い。そんな課題解決のヒントが得られるセミナーが、静的解析、動的解析の両方をサポートしたC／C++言語対応のオールインワンテストツール「C+
コードレビューの高まった言葉 - 職質アンチパターン
ブログ間違った，普段こういう事はこっちに書いてます． http://moznion.hatenadiary.com 最近自分がコードレビューで使いがち，あるいは表立って使ってないんだけど内心評す時に使う言葉が色々とあり，まとめてみることとした．参考にしない方が良いと思う． 左は言葉，右は説明． 屈強 - コードが力強い時に使う．例えば長い一枚スクリプトとか，コメントが一切ないバッチ処理とか．やや批判的な意味合いで使うことが多い． マッチョ - 屈強と同じ文脈で使いがち 屈強だけどしなやか - 屈強だけどしなやかな時に使う．好意的な屈強さと言える． モノリス - 長大なトランザクションスクリプト見た時とかに使う．やや批判的． 言い訳ないですか - 後で直していくぞ！ というメンタルの時に書かれたコードのコメントが案外少ない時に使う言葉．言い訳は無いよりあった方が良い．実際には「もうちょっと言
チームのみんなで成長できるコードレビュー コメントの方法編 - Qiita
mixi Advent Calendar の17日目です。 今までまともにコードレビューをしたことがなかったエンジニアが、半年前からコードレビューを経験し始めて、チームの雰囲気が良くなった（と思っている）レビューの仕方をご紹介します。 私は株式会社ミクシィのグループ会社である株式会社 Diverse へ出向して、そこでデーティングサービス YYC の Android クライアントを開発しております。 現在、 YYC の Android クライアントは大規模な改修を行っていて、それに合わせて開発チームの再編成・開発フローの改善などいろいろな試行錯誤をしています。 その試行錯誤のうちの一つとしてコードレビューでの取り組みをご紹介します。 ここで説明すること、しないこと ここでは以下のことについて説明します。 コードレビューの雰囲気を良くする（良く保つ）方法 自分やチームメンバーの成長を促すため
コードレビューのベストプラクティス | POSTD
Wiredrive では、私たちはかなりの数のコードレビューを行います。しかし、ここで働き始める前には私はコードレビューなどしたことがありませんでした。今回は、私がコードレビューをする時に何に注目するようにしているかや、私の考え出したベストなコードレビューのやり方をお話したいと思います。 コードレビューとは、簡単に言うと2人以上の開発者で問題を引き起こしそうなコードの修正について話し合うことです。コードレビューをすることのメリットについては多くの記事で語られており、知識を共有できること、コードのクオリティが上がること、開発者が成長できることなどが挙げられています。しかし、レビューを行う上で、どのように進めていくかという具体的なことについてはあまり多く語られてないように私は思いました。 レビューで何に注目するか アーキテクチャ/デザイン 単一責任原則 : 1つのクラスは変更する理由が2つ以上
クソコードに対する怒りとコードレビューにおける人格攻撃について | おそらくはそれさえも平凡な日々
デキるプログラマだけが知っているコードレビュー7つの秘訣 7つの秘訣の1〜5は本当にそのとおりだと思います。 「怒り」って言葉を使っているところはなかなか画期的だと感じた。というのも僕は前から「人格攻撃に思われて」しまうような、コードで人を殴るようなことをしてしまう人が出てきてしまうのは何故かということを考えた時に、そこには「コードに対する怒り」があるからだろうなと思っていたからである。怒りがあるからこそ強く指摘しすぎてしまうことが起こりうる。 「怒り」というのはつまり「感情」である。であれば、「その『怒り』はコードに向けられたものであり、書いた人に対してのものではないので、その人に対しての攻撃ではない」というのは、理屈ではかろうじて通るかもしれないが、書いた人の「感情」的には通らないこともあることは理解したほうが良いと思う。 じゃあ怒らなければ良い、という話にはしたくなくて、どうしても怒
iOSアプリケーション開発のコードレビューで気をつけていること - ninjinkun's diary
日常的なコードレビューで気をつけていることリストです。GitHub会議(仮)で発表しようと思っていたのですが、日程の都合で参加できないので、書きためておいたメモを公開します。またどこかで発表するかもしれません。 AutoLayoutにできないか AutoLayout化した方がすっきりしそうならAutoLayout化する AutoLayout化できそうなものでやっていないものは、なぜコードで実装したか質問する 例えばUITableViewCell ちゃんと理由があれば別に良い。コードの方が良いことも多い UIAppearanceで解決できないか 各クラスの中にスタイルの指定が入るより、UIAppearanceでスタイル指定を分離して別クラスに書く方がデザイナーも弄りやすくて良い 3.5インチ端末が考慮されているか レイアウトが決め打ちだとここで問題が出ることが多い 着信ステータスバーが考慮さ
1
リリース、障害情報などのサービスのお知らせ
最新の人気エントリーの配信
j次のブックマーク
k前のブックマーク
lあとで読む
eコメント一覧を開く
oページを開く
先輩から教えてもらったコードレビュー
コードレビューガイドライン #loupestudy