タグ付けされた質問 「code-reviews」

このタグは、コードレビューとコードウォークスルーの実践に関する質問に使用します。既存の動作するコードのレビューについては、http://codereview.stackexchange.comを参照してください

5
コードレビューに割り当てられた時間
コードレビューを行っている場合 実装と比較して、コードレビューにどのくらいの時間を費やしますか? どのくらいの変更がコードレビューを受けますか? あなたはそれがはるかに多い/もっとあるべきだと思いますか? 有効性に関する研究はありますか? [編集]回答をありがとう

6
コメントとして図を使用してソースコードに注釈を付ける
私は、計算幾何学やグラフィックス、およびこれらの種類のトピックに触れる多くの(主にc ++およびjavascript)コードを書くので、視覚図は問題を解決するプロセスの不可欠な部分であることがわかりました。 「ああ、手書きの図をなんらかの方法でコメントとしてコードの一部に添付できたら素晴らしいのではないか」と判断しました。これにより、私が取り組んだものに戻ることができます。数日、数週間、数か月前、そしてはるかに迅速にアルゴリズムを再作成します。 視覚的な学習者として、単純な図はあらゆるタイプの非自明なデータ構造に関する理解と推論に役立つため、これはほとんどすべてのタイプのプログラミングで生産性を向上させる可能性があると感じています。たとえばグラフ。大学でのグラフ理論の授業中、私は実際に図式表現を描くことができるグラフ関係を本当に理解することができただけでした。 そう... 私の知る限り、IDEを使用して写真をコードへのコメントとして保存することはできません。 私や他の誰かが、イメージをbase64バイナリ文字列に変換してコードに挿入できる、かなり使いやすいツールを考え出すことができると考えました。 変換/挿入プロセスを十分に合理化できれば、ダイアグラムと実際のコードの接続がはるかに良くなるため、ノートブックを時系列に検索する必要がなくなります。さらに素晴らしい:IDEのプラグインが自動的に画像を解析して表示します。理論的な観点から、これについて絶対に難しいことはありません。 私の推測では、お気に入りのIDEを拡張し、これらのプラグインを維持する方法を実際に把握するには、余分な時間がかかるので、同じ解析を行う一種のコードポストプロセッサに完全に満足し、画像をレンダリングして、コードをブラウザや何かの中に並べて表示します。私は貿易でjavascriptプログラマーだからです。 人々はどう思いますか?誰もこれにお金を払うでしょうか?私は...するだろう。しかし、私は、私またはかなりの数の同僚がそのようなものにお金を払うかどうかに関係なく、そのようなものが成功する可能性が高い唯一の方法は、オープンソースのリリースを通じてであることを指摘するでしょう。

5
トランクにコミットされる前にコードをレビューする最良の方法は何ですか?(SVN)
SVNトランクにコミットされる前にコードをレビューする最良の方法は何ですか?私が考えているアイデアの1つは、開発者にコードをブランチにコミットさせ、ブランチのリビジョンをトランクにマージしながらコードを確認することです。これは良い習慣ですか?そうでない場合、トランクにコミットされる前にコードをレビューするために他に何ができますか?

2
コードレビュープロセスの有効性を判断する方法
組織内にコードレビュープロセスを導入しましたが、うまく機能しているようです。しかし、時間の経過とともにプロセスの有効性を測定できるようにしたいと思います。つまり、コードがクリーンであるためにバグを見つけられないのですか、それとも人々がバグを拾っていないのでしょうか。 現在、効果的な完全自動化されたテストプロセスはありません。私たちは主に手動テストを採用しているため、この段階で見つかった欠陥に頼ってコードレビュープロセスが機能していることを確認することはできません。 誰もこの問題に出くわしたことがありますか、コードレビューの測定で何がうまくいくのかについて考えたことはありますか?

5
すべての開発者にコードレビューを行わせる
私は7-8人の開発者チームのソフトウェア開発者です。私たちは現在かなり長い間コードレビューを行っており、コードの品質は時間の経過とともに向上しています。 しかし、最近、一部の開発者が他の開発者よりも多くのコードレビューを求められていることに気付きました。私は彼らの柔軟な態度のためだと思います。 私の見解では、これはコードレビューの実施方法ではありません。すべてのチームがそれを説明し、コードレビュー担当者が変更を簡単に受け入れる意思があるために選ばれるべきではありません。 チームでこの問題にどのように対処しますか? コードレビュー担当者を選択するためのルールを確立しましたか? コードレビュー担当者は、(良い)コードレビューを行った時間に対して報われるべきだと思いますか?そして、彼らはどのように報われるべきですか? あなたの答え/アイデアをありがとう。

4
ジュニア開発者にコードレビューへの参加を促す方法は?
私は現在、3人の下級者と一緒に上級開発者として働いており、コードレビュープロセスを導入して、生産に入るコードの品質を管理しました。 私たち全員がお互いの作業をレビューすることは非常に有益であると感じていますが、約5週間のプロセスの後、ツール(BitBucket)でコメントをするのは私だけです。 職場には一部文化的な問題があり、コメントが間違っている場合にはおそらく自然な抵抗がありますが、何らかの方法がありますか?

4
インタビュアーがコードを読むように申請者に求めないのはなぜですか?[閉まっている]
閉じた。この質問は意見に基づいています。現在、回答を受け付けていません。 この質問を改善したいですか?この投稿を編集して事実と引用で答えられるように質問を更新してください。 4年前に閉鎖されました。 私は人生で数十回のインタビューを受けました(卒業しようとしています)が、なぜコードを読んで説明するように一度しか頼まれなかったのでしょうか。大体、90%の仕事は既存のシステムの維持に関するものです。他の人のコードを読み取るIMOの能力は重要なスキルです。 インタビュアーがチェックしないのはなぜですか?* * 友達の中では、コードのレビューを依頼されたのは私だけです。

5
コードレビューでコメントする最良の方法は何ですか?
私のチームは、誰かが何かをチェックインするたびに、コードレビューを開始するためにるつぼ/魚眼レンズを使用し始めました。私たちは3人しかいません。 私の質問は、問題が発生したコード行にコメントを残すにはどうすればよいですか?研磨剤のように見えることなく、自分の主張を伝えたい。 私は高い馬に乗っているように見えて、「私はこのようにやっている...」と言い たくありません。また、私は権威を持って、 「これはこの方法で行うべきです...」 明確にする: これは本当に優れたリソースである何、私は上のコメントに見てすべきこと:コードレビューの主観的または客観的(定量化)ですか?、しかし、私はそれにコメントする方法を探しています。

7
コードレビューは本当にアジャイルで機能しますか?
そのため、名前に3文字が含まれる大企業で働き始めました。彼らはアジャイルになろうとしていますが、プロセスがたくさんありますが、アジャイルだとは感じません。 私が一番苦労しているのはコードレビューです。私の最後の仕事は、私が見た、今まで聞いた、そして/または聞いたことがある中で最もアジャイルな開発チームだと言うスタートアップの仕事でした。 とにかく、私の意見では、コードレビューは、UX / UIが極端/強烈な反復開発またはアジャイル開発では時間の無駄であるということです(Apple / Steve Jobsの完璧さを考えてください)。たぶん、ここの誰かが私を解雇する前に理解するのを助けることができますか? これが私の開発プロセスであり、最後のスタートアップのプロセスです...非常にアジャイルです。 開発作業/仕事を分類するために、初期の機能作業を行います。フィードバックを得るために、いくつかのバージョンをモックアップし、ユーザー、チーム、マーケティングに提示します。次に、上記と同じ利害関係者から1つのラウンドを取得するために、別のモックアップの反復を行います。次に、作業を分割して開始します。達成すべきマイルストーンと日付はありますが、プラグインは続けます。この間、コードレビューはありません。開発の数週間に何度か、利害関係者とのセッションを開催し、機能/機能/ UX / UIがまだ適切で目標に合っているかどうかを確認します。 8週間の反復サイクルの終わりに近づくと、QAがテストを開始し、アルファユーザー、そして最終的にベータユーザーに進みます。しかし、アルファ版およびベータ版の開発者は、UXを改善するために、UIを毎日または1時間ごとに繰り返し変更して、新しい機能と古い機能を検討しています。したがって、このリリースで開発されていた機能は、最後の4週間でさらに3回変更されて、改善および完成されるか、いくつかの小さな機能が追加されます(たとえば、コンポーネントを少しスマートまたはスマートにします)。時々、変更は表面的なものであり、CRUD操作は変更または変更されず、すべてのUIが変更されるだけです。 このタイプの開発プロセス、極端なアジャイルでは、コードレビューは時間の無駄ではないでしょうか?別の開発者または2人がコードをレビューしたが、そのコードがさらに3回変更されると、UI / UXのすべての改善により、最初の3回のレビューに時間を無駄にしないという意味です。そのコード/コンポーネント/ UIが廃棄されたコード このプロセスで品質の問題はあまりありませんでした。開発者がすべての知識を手放した場合、そうです。 はい、テスターは3〜4回再テストする必要があるため、多くのテスターがいます。また、すべてのUI / UXが変更される理由を尋ねるのに夢中にならないでください...それがまさにその方法です...それが、アプリがUI / UXに対して多数の賞を受賞し、ユーザーがアプリ。思考プロセスは、1時間余分に時間をとってから何かを2%改善できるかどうかです。ユーザーはより幸せになり、より多くの$またはユーザーを意味します。はい、私たちのユーザーは、アプリが絶えず変化していることで大丈夫です。なぜなら、それが初日から行われている方法であり、悪いまたはネガティブとは思わないからです。 この投稿が華々しいものにならないことを願っていますが、コードレビューが無駄にならないことはわかりません。おそらく、レビューしたコードのすべてのコードの2%にバグがあります。各リリースでは、コードレビューで3つのバグが見つかる可能性があります。それで、リリースごとに開発者ごとに40時間のコードレビュー(4 x 40 = 160時間)になって3〜5個のバグを見つけることになりますか?とにかく3〜5個のバグがQAによって検出される可能性が50%です。開発者が新しい機能を追加したり、既存の機能を改善したりするのに、開発者ごとに40時間を費やす方が良いと思いませんか?

3
コミット後のコードレビューを容易にするためにどのツールを使用できますか?[閉まっている]
閉まっている。この質問はトピック外です。現在、回答を受け付けていません。 この質問を改善したいですか? 質問を更新して、 Software Engineering Stack Exchangeのトピックになるようにします。 5年前に閉鎖されました。 多くのコードレビューツール(ほとんどが無料のもの)を評価していますが、それらはすべて、コミットする前にパッチをレビューすることを目的としているようです。これはSubversionのワークフローに実際には収まらないため、差分ではなくコミットされたリビジョンのレビューをより適切にサポートする代替手段を探していました。推奨事項はありますか?私は無料または安価なツールを好むでしょう。

4
プルリクエストが大きいときにコードレビューを改善するにはどうすればよいですか?
免責事項:同様の質問がいくつかありますが、大規模なプルリクエストのレビュー中に直面する問題に特に関係する質問は見つかりませんでした。 問題 私のコードレビューはもっと良い方法でできると思います。特に、20以上のファイルに多くの変更が加えられた大きなコードレビューについて話している。 明らかなローカルコードの問題をキャッチするのは非常に簡単です。ただし、コードがビジネス基準を満たしているかどうかを理解することは別の話です。 コード作成者の思考プロセスに従うのに苦労しています。変更が多数あり、複数のファイルに分散している場合は非常に困難です。特定の変更に関連するファイルのグループに焦点を当てようとしています。次に、グループを1つずつ確認します。残念ながら、私が使用しているツール(Atlassian Bitbucket)はあまり役に立ちません。私がファイルにアクセスするたびに、現在確認されている変更の一部に関連していないことがよくありますが、ファイルは表示済みとしてマークされます。言うまでもなく、一部のファイルは複数回アクセスする必要があり、それらの変更は個別にレビューされます。また、悪いパスをたどったときに関連するファイルに戻るのは簡単ではありません。 可能な解決策、およびなぜそれらが私にとってうまくいかないのか コミットによってプルリクエストを確認することでサイズの問題を解決できることがよくありますが、古い変更を頻繁に見ているので、私はそれが好きではありません。 もちろん、より小さなプルリクエストを作成することは改善策のように見えますが、それが何であるか、時には大きなプルリクエストを取得し、それをレビューする必要があります。 また、コード全体の論理的な側面を無視することもできますが、特にコードが経験の浅いプログラマーのものである場合は、かなりリスクが高いようです。 より良いツールを使用することは役立つかもしれませんが、私はそれを見つけませんでした。 ご質問 コードレビューで同様の問題がありますか?どのように彼らに直面しますか? たぶんあなたはより良いツールを持っていますか?

1
GitHubプルリクエストでピアレビューを行う方法
BitbucketからGitHubに移行していますが、私たちが苦労しているのは、次のようにBitbucketで非常にスムーズに機能するピアコードレビューです。 著者がプルリクエストを開きました(GitHub:同じ) 著者は同僚をレビュアーとして追加しました(GitHub:?? 複数の譲受人とここで苦労しています) レビュアー: 緑のチェックマークが付いたPRを承認しました(GitHub:??) コメントを追加(GitHub:同じ) 軽量タスクを作成しました(GitHub:- [ ]PRの説明で構文が使用されている場合に似ていますが、タスクでは機能しないことが残念です) 一目で確認できるPRのリストがあり、レビューされていてマージしても問題なく、さらに注意が必要なものがあります(GitHub:??) 可能な限りサードパーティのコードレビューツールを避け、何らかの回避策を備えたバニラGitHubを使い続けたいと思います。

5
コードレビューは誰が行うべきですか?
私の会社では、主にアーキテクトがコードレビューを行っています。彼は非常に経験豊富で賢いソフトウェアの男なので、非常に得意です。開発者がコードレビューを行うとき、彼らは半分も行いません。開発者にさらにコードレビューを行わせるようにしましたが、コードレビューの品質は良くありませんでした。開発方法としてScrumを使用します。 ただし、現在のシステムには2つの問題があります。 建築家がボトルネックになる 開発者は、コードの品質とアーキテクチャ(すべての種類の問題につながる)に対して責任を負いません。 これらの問題にどのように対処できますか?コードレビューの担当者を変更する必要がありますか?

6
Try-Catch内ですべてのステートメントを記述する必要があるのはなぜですか?
私の会社長は、すべてのコード、つまりTry-catchステートメント内のすべてのコードを記述する必要があると言います。ここで、「申し訳ありませんが安全です」というアプローチはここで理解できますが、ラベルが作成され、フォームの位置が設定されたときに例外があると考えるのはあまりにも簡単ではありません。このような単純な操作で例外が発生する場合があります。

9
コードレビュー、利点は何ですか?[閉まっている]
現在のところ、この質問はQ&A形式には適していません。回答は事実、参考文献、または専門知識によってサポートされると予想されますが、この質問は議論、議論、世論調査、または広範な議論を求める可能性があります。この質問を改善し、場合によっては再開できると思われる場合は、ヘルプセンターをご覧ください。 7年前に閉鎖されました。 私のチームでは、正式なコードレビューは行いません。ペアプログラミングと回転ペアで十分であると考える傾向があります。 正式なコードレビューを検討する必要がありますか?利点は何でしょうか?

弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.