smartnut.ruでコードレビューを行う理由

こんにちは、Habr! 私は、SmartNutの開発者の1人です。SmartNutは、それをサポートおよび外部委託する小規模企業向けのWebサービスです。 私たちは自分たちがやっていることを誇りに思っています。 トピックコードレビューに関する実生活のストーリーを共有したいと思います。



内部のすべての人々は完璧主義者です。 あなたが人にこれを見ないなら、これはこれが人にないということを全く意味しない。 おそらく彼は自宅で完璧な注文をしているかもしれませんが、コードをあまり上手く書けません。 反対に、もう一方はコードまたはマシン内で秩序を保つことができますが、どこかで自分自身を黙らせることができます。

ただし、他の人との関係で完璧主義者を見つける最も簡単な方法です。 もちろん、自分でうまくやる方法を知っていれば、理想的な仕事を要求するのは非常に簡単です。



以前のプロセスの様子


プログラマーVasyaはコード行を作成し、チェックしました。 おそらく彼はいくつかのテストを書いたかもしれませんし、書いていないかもしれません。 次に、落ち着いて、共通のリポジトリにコミットします。 しばらくして、他のプログラマーが新しい改訂コードを研究し、Vasyaが書いたものを見て、Vasyaを開いてhara-kiriがVasyaに来て、彼らの静脈を開いて、hara-kiriに何が間違っていたのか、どうすれば改善できるかを伝えました。 Vasyaは謙虚に同意し、誰もが満足を得ましたが、コードはそのままでした。 既に伝達され、おそらくテストされています。 そして、それは修正されたエラーでした。



どうでしたか


私たちの多くは、eXtremeプログラミングの実践に精通しています。 それらの1つはペアプログラミングです。 経験から、このプラクティスを完全に使用することは非常に困難です-「パーソナルスペース」を放棄することは困難であり、このプラクティスを上司に「売り込む」ことは困難です(「プログラマーは2倍遅く動作します!」)など。

しかし、象を部分的に食べ始めることができます。 最初の部分はコードレビューです!



プログラマーVasyaがタスクを完了しました。 タスクが完了したと見なされるには、スクラムボードのコードレビュー列を通過する必要があります。



したがって、私たちは:



そして、「コードレビューを誰が行うべきか」という質問がないように、各反復にはこの手順を実行する義務があります。



コードレビューに追加のツールは使用しませんが、常に座って変更を確認します。

この場合にいくつかのツールを使用すると、処理が遅くなるだけです。 同じ部屋にいる場合は、電子メールでピンポンをする必要はありません。すべてを言葉で説明する方がはるかに簡単でわかりやすいですが、表示する方が良いでしょう。 また、個人的なコミュニケーションは、各従業員の開発およびチーム全体にとってはるかに役立ちます。



したがって、 smartnut.ruを実行し、同じことを望みます。



柔軟に、友達になって、すぐに会いましょう!



All Articles