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

ソース管理リポジトリでレビュー済みのソースコードを管理する最良の方法は何ですか？ソースコードはチェックインされる前にレビュープロセスを経るべきですか、それともコードがコミットされた後にコードレビューが行われるべきですか？コードがリポジトリにチェックインされた後にレビューが発生した場合、どのように追跡する必要がありますか？

12 version-control  code-reviews  svn 

私が働いている会社では、コミットする前にすべてのコードを他の開発者がレビューする必要があります。私のチームのメンバーは、他の開発者がコーディングに忙しすぎてレビューを行うことができないため、特に非常に長い場合、しばしばイライラします。他の開発者にタイムリーなコードレビューを行うためのインセンティブをどのように与えますか？ （私たちはgit-svnを使用しているので、レビューを待っている間もコーディングを続けることができます。しかし、コードをコミットする前に長い時間待たなければならない場合は、いらいらします。）

12 code-reviews 

ソフトウェア開発の一般的なシナリオは、誰かのコードをレビューするコードです。これを行う一般的なツールは、プルリクエストを開くことです。 私の質問は、レビューで問題が見つかったときに、変更すべきかどうかです 個別にコミットする（新しいコミット） または、既存のコミットを変更する必要があります（以前のコミットから誰もブランチしていないと仮定します...共有ブランチからの履歴を書き換えることは悪いニュースです）。 最初のシナリオでは、コミット履歴にノイズを追加しますが、増分の変更を追跡するのは簡単です。2番目のオプションには、長所と短所が逆になっています。

11 git  code-reviews  repository 

閉じた。この質問はより集中する必要があります。現在、回答を受け付けていません。 この質問を改善したいですか？この投稿を編集するだけで1つの問題に焦点を当てるように質問を更新します。 6年前に閉鎖されました。 コミットごとに必須のコードレビューを試行しています。作成者ではなく、少なくとも1人が検証していないマスターは何もありません。開発者と管理者の両方から賛同を得ています（これは驚くべき状況です）。 明らかなバグの削減 プロジェクトの周りで起こっている変化についてのより多くの認識 「誰かがこれを見て、怠けないようにする」/反カウボーイ効果 プロジェクト内/プロジェクト間の一貫性の向上 しかし、速度を低下させることが知られているものを導入しており、間違ってやるとコミットパイプラインに愚かな官僚的なステップができてしまい、時間がかかります。私が心配していること： 単なるピッキングに発展するレビュー （双曲線的に）2行のコミットレビューの一環として、巨大なアーキテクチャの問題を公開する人々。 答えに他のことを偏らせたくありません。 私たちはすべて合理的な人であり、多くの自己分析を行っていますが、レビューセッションで実際にどのようなことを達成しようとしているかについて、戦いで得た洞察を間違いなく使用して、レビューを実際に機能させることができます。動作することがわかっているガイドラインとポリシーは何ですか？

11 teamwork  code-reviews 

つまり、スキーマの変更だけではありません。主キーに対する単純なSELECTでさえ、他の開発者によって（コンテキストで）コードレビューされているにもかかわらず、コードから抽出され、EXPLAIN出力で送信されるすべてのステートメントのDBAレビューがなくても、実稼働に入ることはできません。詳細呼び出される頻度など あなたがそのような環境にいた場合、あなたはそれが純利益であるか、開発の足かせであると思いましたか？リレーショナルデータベースを長年使用してきた人として、「開発者がデータベースを使用することを信頼できない」という態度は不当だと感じており、この状況がどれほど一般的かについては興味があります。価値があるのは、これはWeb開発のためだけであり、重要なことではありません。

11 code-reviews  dba 

時折、最も重い種類の低レベルの最適化を必要とする十分に計算集約的なコードの1％があります。一般的な例としては、ビデオ処理、画像処理、およびあらゆる種類の信号処理があります。 目標は、コードが保守不能になったり、新しい開発者によって削除されたりしないように、最適化手法を文書化し、教えることです。（*） （*）予測できない将来のCPUで特定の最適化が完全に役に立たない可能性があるにもかかわらず、コードは削除されます。 ソフトウェア製品（商用またはオープンソース）が最速のコードを持ち、最新のCPUアーキテクチャを使用することで競争上の優位性を保持していることを考えると、ソフトウェア作成者は特定の同じ出力を取得しながらコードを微調整する必要があることがよくあります少量の丸め誤差を許容するwhlist。 通常、ソフトウェアライターは、実行される各最適化/アルゴリズムの書き換えのドキュメントとして、関数の多くのバージョンを保持できます。これらのバージョンを他の人がどのようにして最適化手法を研究できるようにするのですか？ 関連： 読みやすいコードと読みにくいコード。いつ線を越えるか？

11 documentation  code-reviews  optimization 

まず、コードレビュープロセスを固く信じており、常に誰かにコードをレビューしてもらいたいと思っています。私の質問は、他の誰かのためにコードレビューを実行する上で、どうすればより良い仕事をすることができるのかということに本当に集中していますか？ コードレビューを実行するには、既存のコードの仕組みとローカル標準の知識が必要であり、どちらも非常によくわかっていると感じています。それでも、私は他の人のために十分なコードレビューを決してしないように感じています。また、特定の人は他の人よりも優れたジョブレビューコードを実行しているようだということを知っているので、優れたコードレビュアーである人のために、あなたはどのようなテクニックを使用していますか？

11 code-reviews 

私たちは、3つのメソッドのように必要なだけの巨大なインターフェースを公開するサードパーティのサービスに依存しています。さらに、インターフェースは頻繁に変更されます... プロジェクトのクラスにインターフェイスをラップし、必要なメソッドのみを公開することにしました。 しかし、私は戻り値をどのように処理するべきかわかりません...インターフェイスは型のオブジェクトを返しますStorage。内部StorageModelには、の内部表現であるタイプがありますStorage。 マッパーで何を返しますか：StorageまたはStorageModel？StorageService挿入されたラッパーの依存関係を取得するDataServiceがあります。 現在、私は基本的に次のようにしています： public class StorageService { private readonly IExternalStorageWrapper externalStorageWrapper; public StorageService(IExternalStorageWrapper externalStorageWrapper) { this.externalStorageWrapper = externalStorageWrapper; } public StorageModel GetStorage(int storageId) { return this.externalStorageWrapper.GetStorage(storageId).ConvertToStorageModel(); } } public class ExternalStorageWrapper : IExternalStorageWrapper { public Storage GetStorage(int storageId) { using(var ext = new ExternalStorage()) { return ext.GetStorage(storageId); } …

11 c#  design-patterns  programming-practices  code-reviews  wrapper 

私は現在、私のチームに参加したばかりのジュニア開発者のコ​​ードのいくつかをレビューしていますが、このレビューの出力をどのように提供すべきか疑問に思っています。 自分でコードを修正する必要がありますか？ レビュープロセスに関するフィードバックを提供し、指示に従って修正を行わせる必要がありますか？もしそうなら、どのようにこのフィードバックをするのですか、特定のテンプレート文書を記入して送信するのですか、それとも後でチェックできるコードファイル内の問題をマークするのに役立つソフトウェアがありますか？（私はVisual Studioを使用しています）。 コードのレビューと修正が完了した後、時間が経ち、過去にレビューしたコードの一部が変更されました。再レビュープロセスを実行するにはどうすればよいですか？すべてのコードをもう一度確認する必要がありますか？または、変更された部分だけをチェックしますか？そして、もしそうなら、コードの二重レビューを避けるために、変更された部分をどのように追跡するのですか？

10 code-reviews  process 

私は、以下の短いサンプルを含めたいくつかのひどいコードを継承しました。 この特定のアンチパターンに名前はありますか？ これをリファクタリングするための推奨事項は何ですか？ // 0=Need to log in / present username and password // 2=Already logged in // 3=Inactive User found // 4=Valid User found-establish their session // 5=Valid User found with password change needed-establish their session // 6=Invalid User based on app login // 7=Invalid User based on network …

10 coding-style  code-quality  coding-standards  code-reviews  code-smell 

私は現在、私が働いている会社でReactの導入に取り組んでいます。トピックに関するコースを行った唯一の人として、コードが「ベストプラクティス」に従っており、正しく実装されていることを確認するために、誰にレビューを依頼できますか？

10 code-reviews 

休業。この質問は意見に基づいています。現在、回答を受け付けていません。 この質問を改善してみませんか？この投稿を編集して、事実と引用で回答できるように質問を更新してください。 4年前休業。 仮説として、私の経験が.NETであるときに誰かに新しいPHP開発者の面接を依頼した場合、彼らが提供したコードサンプルが効率的で質が良いかどうかをどのように判断できますか？ 言い換えれば、言語に慣れていない場合にプログラマーのコードを評価するための最良の方法は何ですか？

10 code-quality  interview  code-reviews 

商用開発で​​はコードレビューが強く求められていることは知っています。しかし、コードレビューはオープンソースソフトウェアで使用されていますか、それとも信頼に基づいていますか？もしそうなら、彼らはどのように実行されますか？[遅延コミット、「プリコミット環境」ですか、パッチを別の開発者に送信できるツールはありますか？] コードレビューを使用するプロジェクトはありますか？ 私の理解では、Linuxカーネルは主にコミッターの信頼に基づいています。MySQLは、主要な著者の承認とパフォーマンスへの影響に基づいていました。

10 open-source  code-reviews  source-code 

ユニットテストの前または後に、コードレビューをいつ実行するかについて同僚と議論しています。ベストプラクティスは何ですか？ 考慮に入れる必要があるいくつかの要因（さらにある場合があります）： コード変更のサイズ-大きな変更は、コードレビューからさらに多くの変更が行われることを意味します。これらの変更が大きい場合、UTがコードレビューの前だった場合は、ほとんどのUTを再度繰り返す必要があります。 単体テストの実行に必要な時間 新しい機能ですか、それともバグ修正ですか

10 language-agnostic  code-reviews 

ここでいくつかのアイデアを探しています。 私は「コードレビューはどのように実行し、コードレビューを実行する必要がありますか」という記事を読みました。これは非常に有益でしたが、以下の質問をさらに明確にする必要があります。 私の質問は、 ターゲットの開発者として、コードをレビューする前に、開発者が組み込むことができるいくつかのベストプラクティスを提案できますか。 現在、私は以下の方法を練習しています 論理フローのPPT 詳細なコメント。 問題：上記のプラクティスを実装しましたが、レビューには役立ちません。私が直面した問題は、特定のロジックが参照されると、実装とフローを探し続け、プロセスに時間がかかりすぎて、人々の神経質になってしまうことです。 多くの開発者も私が経験していることを経験していると思います。

10 c#  code-reviews