チームで悪いコードを書いている人を「中和」する方法は?


9

私はJoelOnSoftwareに関するこの記事をいつも気に入っていました。私は特に初心者だったときに関係することができました(それでも、私は常に1つになるつもりです)。

#4については、bozoを中和します。これを実際の職場で実際に実装するためのアドバイスはありますか?(少なくとも私たちのチームでは)誰かの悪いコードに対するバグを記録するほど簡単ではないようです。残りのすべての人のために何がうまくいきますか?


1
銃。それらの多く。
CodesInChaos 2014年

回答:


9

永続的な評価。

ただの終わりにそれぞれの日、彼らが書いたものを確認するために30分を費やしています。彼らが何か間違ったことをした場合、彼らにそれを書き直させます。

これを行わないと、ある日、アプリケーションの一部が、その仕事をする能力があるように見えますが、完全に保守不可能であり、不適切に設計されており、将来、または明日にも多くの問題を引き起こすことがわかります。

たとえそれが彼らの生産性を低下させたとしても、彼らが2倍以上のサイズであるが維持不可能なバグの膨らみに対していくつかの良いコードを生成した場合、それはそれでもずっと良いでしょう。


2
すばらしい答えです。私がそれに追記することができるなら、この人が仲間であるなら、それはチームリーダーによって行われるのが最善です。そのようにすると、店の階層によって命令された場合、それがうまくいかないという応答がはるかに効果的になります。

1
@サーファー、それは正反対です。あなたはなるチームが何を気遣うことで、よりよい解決策を提案することで、このようなことをすることによって、チームのリーダー。その逆ではありません。(しかし、もちろん、より高い階層レベルから援助を得ることは役立ちます)。
Pが

1
それで、問題は、彼らにそれを書き換えさせる権限を誰が持っているのですか?問題がチーム全体にブロードキャストされる場合、答えは、チーム全体の道徳的権威だと思います。
Cジョンソン

すべてのチームメンバーに対して堅牢なコードレビューメカニズムがない場合、これは適切です。再加工するように言われる前に、ボゾが本当に深い穴を掘らないようにすることも適切です。
mattnz

5

その人が何もよく知らないが、学びたい場合は、メンターシップとコードレビューを提供してください。それらが適切なコードに公開されていることを確認してください。

本当に貧しい開発者とは、悪い方法で設定され、何か新しいことを学ぶことと戦う人々です。あなたの唯一の希望は、彼ら自身の混乱やより単純なもののいくつかを維持することによって彼らを苦しめることです。理想的には、権限のある人物が立ち上がって、「同意する」または「去る」と言います。


1
態度は本当に重要です。私は通常、初心者の方が謙虚で、コードのレビューや批評を受け入れます。これらの人々は話しやすいです。そして、あなたは彼らの弱点について彼らと簡単に話すことができます。彼らが仕事に対して批判されるとき、それは百万枚のガラスのように粉々になるであろう傲慢な退役軍人です。
Cジョンソン
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.