アプリの開発等に携わっています。上で行われており、書いたコードをプロダクトに取り込む前には基本的に第三者のコードレビューが必須です。コードレビューはプロダクトの品質向上に貢献していますが、往々にして結構な時間と労力がかかるものです。出してレビューをしてもらい指摘の修正を繰り返していると、場合によってはマージに数日〜1週間ほどかかってしまうこともあります。ために心がけていることを紹介したいと思います。見出しや箇条書きのスタイルを利用することでかなり読み易くすることができます。次のような形式で書くことが多いです。説明にスクリーンショットを用いることが多いと思いますが、このとき変更後のものだけでなく変更前の画像も比較できるようにしておくと非常にレビューがし易いです。ようにするとシュッとした感じになります。width="320"/>長い画面のスクリーンショット余談ですが、モバイルアプリではスクロールビューのように画面に収まらない長いビューのスクリーンショットを撮りたい場合があります。いうアプリでスクロールしながら撮った複数のスクリーンショットを結合したりしています。開発中はシミュレータを使用していることも多く、PC内に保存した画像を同様に結合できる手段が欲しかったためそのためのスクリプトを書いて使ったりもしています(雑ですが)。重なり部分を判定していい感じに結合してくれます。積極的に貼っています。社内用にアレンジしたものが使われています。映しそれを先述のツールで撮影しています。変更は数百行までにやはり大きな変更のレビューには時間がかかる上に問題点の発見も困難になります。行を超えるとかなりレビューがしんどく感じます。ブランチに向けて出すのが良いでしょう。敬遠され中々レビューされなかったり(よくある)、問題点があっても発見されづらくなるリスクがあるため、適度に分割することが開発効率の向上に繋がります。コメントという形で指摘してくれます。いったツールもコードレビューの負担を減らしています。ついての詳細は過去の紹介記事をご覧ください。クックパッド開発者ブログまとめサービス開発では品質の保持と同時に開発・検証のサイクルを速く回すことも求められます。前者を保証するためにコードレビューという作業は必要ですが、開発速度のボトルネックになることもあります。つくるよう心がけましょう。会員事業部の三吉です。...2017-06-28Android開発のコードレビューbotを乗り換えた話モバイル開発で利用しているコードレビューbotを最近乗り換えた...2016-10-04iOSアプリの継続的デリバリーへの取り組みについての勉強会を開催しました技術部モバイル基盤グループの三木(@giginet)です。使ったセキュリティログ検索基盤の構築サービス特性にあった検索システムの設計戦略クックパッド採用説明会「クックパッドはサービスの作り手を採用したいんです。/*}*/スターをつけました引用をストックしましたストック一覧を見る閉じる引用するにはまずログインしてくださいログイン閉じる引用をストックできませんでした。再度お試しください閉じる限定公開記事のため引用できません。