タグ

コードレビューに関するakakitのブックマーク (5)

  • 眼鏡なしのコードレビュー | POSTD

    例えば、あなたが驚くほど聡明な開発チームのメンバーで、コードレビューのみに一日の時間を確保しているとします。しかし作業を開始して2時間後、眼鏡を忘れてきてしまい、午前中はぼんやりとしたカラフルな表示を見つめていただけだったということに気づいたとします。さて、あなたはどうしますか? 家まで歩いて10分もかからないし、天気も良ければ、眼鏡を取りに帰るのが一番です。でも朝家を出るとき、攻撃的なスズメバチの群れが眼鏡の置いてある部屋に巣を作って、邪魔されたくない様子だったらどうしますか? そういう時はもちろん、コンタクトレンズを付けてきたふりをして、恥ずかしい思いをしないようにするのがよいでしょう。実際に読むことなく膨大な量のファイルを見分けることができるということを覚えておいて下さい。 参考コード 1 不安の種は隔離するべきだということに誰も異論はないでしょう。そしてもちろん、あらゆるクラスは一

    眼鏡なしのコードレビュー | POSTD
  • iOSアプリケーション開発のコードレビューで気をつけていること - ninjinkun's diary

    日常的なコードレビューで気をつけていることリストです。GitHub会議(仮)で発表しようと思っていたのですが、日程の都合で参加できないので、書きためておいたメモを公開します。またどこかで発表するかもしれません。 AutoLayoutにできないか AutoLayout化した方がすっきりしそうならAutoLayout化する AutoLayout化できそうなものでやっていないものは、なぜコードで実装したか質問する 例えばUITableViewCell ちゃんと理由があれば別に良い。コードの方が良いことも多い UIAppearanceで解決できないか 各クラスの中にスタイルの指定が入るより、UIAppearanceでスタイル指定を分離して別クラスに書く方がデザイナーも弄りやすくて良い 3.5インチ端末が考慮されているか レイアウトが決め打ちだとここで問題が出ることが多い 着信ステータスバーが考慮さ

    iOSアプリケーション開発のコードレビューで気をつけていること - ninjinkun's diary
  • JenkinsとGerritによるコードレビュー - 川口耕介のブログ

    今月号のWEB+DB PRESSの特集はコードレビューです。(ちなみに、特集3を書いているのはリアルJenkinsさんことikikkoさんです。Cacooは僕の同僚も使っていて「世界で通用する」というのは当です。) WEB+DB PRESS Vol.72 作者: 近藤宇智朗,生井智司,Dr.Kein,tokuhirom,森田創,中島聡,堤智代,A-Listers,はまちや2,竹原,川添貴生,久保達彦,道井俊介,飯田祐基,中村知成,規世やよい,後藤秀宣,天野祐介,奥野幹也,WEB+DB PRESS編集部出版社/メーカー: 技術評論社発売日: 2012/12/22メディア: 大型購入: 11人 クリック: 94回この商品を含むブログ (10件) を見る 特に第二章はまるまるGerritというGit向けのコードレビューツールについて書かれています。Gerritは、こちらでも周りで使っている人

    JenkinsとGerritによるコードレビュー - 川口耕介のブログ
  • MISRA C++ワーキング・グループ 議長に聞くコーディング標準の活用法とコード・レビューのポイント

    MISRA C++ワーキング・グループ 議長に聞くコーディング標準の活用法とコード・レビューのポイント Tech Village編集部 2012年11月8日,全国町村会館(東京都千代田区)にて,MISRA C++ワーキング・グループの議長であり,MISRA Cワーキング・グループのメンバであるChris Tapp氏(英国LDRA社)が「コーディング・スタンダードとの付き合い方」と題した講演を行った(写真1).講演の中で,2013年に策定を予定しているMISRA Cの次期バージョンや,MISRA C以外のコーディング標準の概要,コード・レビューの効率を上げるポイントなどについて解説した. 写真1 講演を行うMISRA C++ワーキング・グループ 議長のChris Tapp氏 ●MISRA Cの次期バージョンはC99に対応  MISRA Cとは,車載ソフトウェア開発に関する業界団体MISRA(

  • [ディスカッション]ソフトウェアインスペクション/コードレビューを成功させる方法(前編)

    「ソフトウェアインスペクション」あるいは「ソフトウェアレビュー」とは、一般にプログラムが動作する以前の段階から問題やバグを発見することをいいます。ソースコードの静的解析やコードレビューは、ソフトウェアインスペクションの中でもよく行われていることの1つといえるでしょう。また、コード以前の要件定義や仕様書などもソフトウェアインスペクションの対象となります。 7月2日に、このソフトウェアインスペクションをテーマにしたイベント「ソフトウェアインスペクション・ワークショップ 2009」が行われました。ソフトウェアレビューやインスペクションの経験者を前提として参加者に集まってもらい、Javaソースコードのインスペクションを実際に実施することを目的としたこのイベントは、地味なテーマ設定のためそれほど参加者が集まらないのではないか、と予想されていました。 しかし実際には約115名もの応募が寄せられ、定員で

    [ディスカッション]ソフトウェアインスペクション/コードレビューを成功させる方法(前編)
  • 1