開発マネージャーは、「目標の傾向」をどのように処理する必要がありますか？

最初に用語を作成します：

コード目標の調整：午前中にコードをチェックアウトし、他の開発者が前日にファイルごとに行ったすべての変更（特に最初に開発したコードファイル）を静かにレビューし、フォーマット、ロジック、変数名の変更、リファクタリングを修正します長いメソッドなど。その後、VCSに変更をコミットします。

このプラクティスには、私が特定したいくつかの長所と短所がある傾向があります。

• Pro：コードの品質/可読性/一貫性が維持されることが多い
• Pro：他の開発者が元のコードにあまり慣れていないため、いくつかのバグが修正されています。
• Con：多くの場合、目標を設定する開発者の時間の無駄です。
• Con：前日、バグのないコードを書いたと考えていた開発者が、髪を引っ張る怒りを引き起こすバグをときどき紹介します。
• コン：他の開発者は、過度つべこべで悪化し、目標入札のコードに貢献嫌いし始めます。

免責事項：公平を期すために、私は実際には開発マネージャーではなく、実際に「目標管理」を行っている開発者です。

私の防御では、これを正当な理由で行っていると思います（非常に大きなコードベースを油を塗ったマシンに保つため）。また、マネージャーがこの問題に対処する必要があることも間違いなく心配しています。

あなたがマネージャーだったら、この問題にどのように対処しますか？

更新：これがあまりにもローカライズされているという意味ではありませんが、一部の人は質問しているので、おそらくいくつかの背景が明らかになるでしょう。私は3年前に巨大なプロジェクト（200K LoC）を割り当てられましたが、最近（1年前）にプロジェクトに追加された開発者が追加されました。通常、製品の全体的な安定性について回答する必要があります。コードベースのコアアーキテクチャ部分に驚くほど変更が加えられると、特に緊張します。この習慣は、最初は他の開発者の貢献について楽観的だったために発生しましたが、数週間後までは発見されない深刻な問題を引き起こす過大なミスを引き起こしました。しばしばこれらの

開発者にフィードバックを提供するのではなく、コードレビューで提案するすべての変更を行っていることを除いて、あなたがやっていることは基本的にコードレビューと同じように思えます。あなた（または他の誰か）がコード品質の問題や明らかなバグについて元の開発者にフィードバックを提供し、それらを修正するように元の開発者に依頼する実際のコードレビューを行うことはほぼ間違いなく良いでしょう。これによりコードの品質が維持されますが、開発者が元のコードとその落とし穴に精通し、将来のコード変更の改善にも役立ちます。さらに、バグが静かに持ち込まれたり、他の開発者が自分の背後で話されていると考えるようになったときに、「髪を引っ張る怒り」を引き起こすという欠点はありません。

率直に言って、私見、これはひどい考えです。

士気がまだ溝に落ちていないのであれば、それを期待しています。

あなたに公平を期すために、あなたはこれを認めます。

ピアコードレビューは問題ありません。「彼ら対私たち」のシナリオではなく、開発者に1つのレベルで責任を負わせます。（管理者/リードの場合）。

一部の開発者は他の開発者よりも注目する必要があるかもしれませんが、すべてのコードを最初に強制的に通過させるのはばかげています。

そして、あなたがどれほど良いと思うにせよ、あなたが間違っている、または「ピッキング」する時があり、これは事態をさらに悪化させます。

私がマネージャーだった場合、この問題に対処するには：

• チームでコード標準とプラクティスを確認し、コーディング標準のコピーを渡してください
• 定期的にコードをピアレビューし、従うべき標準と実践を強化する
• 開発者が標準に従うことを余儀なくされるように、自動化ツールを使用してnitpicking / formattingを実施する
• 標準に従わない悪いチームメイトを排除する
• ゴールテンダーになるのをやめる

あなたの意図は良いですが、実装はひどいものであり、他の人が指摘しているように、チームメンバーの間で士気と狙撃が貧弱になります。

プロジェクトに最初から標準がない場合は、スイッチを切り替えるだけでなく、新しい標準を段階的に緩和してください。

悪いジュジュ。

私は開発マネージャーではありませんが、もし私がいたら、自分の開発者が割り当てられた欠陥や機能の一部ではないコードに触れないようにしたいと思います。限目。あなたが他の誰かのコードに問題が表示された場合、すべての手段によって、責任、開発者（複数可）の注意にそれを持って、しかしないあなたは、他の開発者（と調和していない場合は特に、中だけでダイビングをし、それを自分で解決しますs）最初。

クリーンアップタスクは、チームによる正式なコードレビューの後、およびそれらのタスクが割り当てられている開発者によってのみ実行される必要があります。

イニシアチブは一般的には良いことですが、時にはお尻に噛みつくこともあります。あなたは何にどんなバグを導入した場合されたコードを働いて、あなたは急速にあなたが別のキャリアパスを選択したい希望することができます。