Làm thế nào để trung hòa với những người viết mã xấu trong nhóm?

Tôi luôn thích này bài viết về JoelOnSoftware được gọi là "Getting Things Done Khi Bạn Chỉ một Grunt". Tôi đặc biệt có thể liên quan khi tôi là người mới (và vẫn cảm thấy như tôi sẽ LUÔN là một).

Khoảng # 4, vô hiệu hóa bozo. Bạn có lời khuyên nào cho việc thực hiện điều này trong các tình huống thực tế tại nơi làm việc? Nó dường như không dễ dàng (ít nhất là trong nhóm của chúng tôi) chỉ đơn giản là đăng nhập một lỗi chống lại mã xấu của ai đó. Điều gì làm việc cho tất cả những người còn lại ngoài kia?

team bug

— C Johnson
nguồn

Súng. Rất nhiều trong số họ.

— CodeInChaos

Câu trả lời:

Đánh giá thường trực.

Vào cuối mỗi ngày, hãy dành 30 phút để xem lại những gì họ đã viết. Nếu họ đã làm điều gì sai, hãy khiến họ viết lại.

Trừ khi bạn làm điều này, một ngày nào đó bạn sẽ nhận ra rằng một phần trong ứng dụng của bạn, trong khi dường như có khả năng thực hiện công việc, hoàn toàn không thể nhận ra, được thiết kế không đúng cách và sẽ gây ra nhiều vấn đề trong tương lai hay thậm chí là vào ngày mai.

Ngay cả khi điều đó sẽ làm cho chúng kém năng suất hơn, nó vẫn sẽ tốt hơn nhiều nếu chúng tạo ra một số mã tốt chống lại kích thước gấp đôi nhưng không thể vượt qua được lỗi.

— P Shved
nguồn

Câu trả lời chính xác. Nếu tôi chỉ có thể nối vào đó, nếu người này là đồng đẳng, thì tốt nhất là được lãnh đạo nhóm thực hiện. Bằng cách đó, phản ứng không tốt của nó sẽ hiệu quả hơn nhiều nếu được chỉ huy bởi hệ thống phân cấp của cửa hàng.

@Surfer, nó hoàn toàn ngược lại. Bạn trở thành trưởng nhóm bằng cách làm những việc như thế này, bằng cách đề xuất các giải pháp tốt hơn, bằng cách quan tâm đến những gì nhóm làm. Không phải ngược lại. (Nhưng, tất nhiên, nhận viện trợ từ các cấp bậc cao hơn sẽ giúp).

— P Shved

Vì vậy, câu hỏi trở thành, ai có thẩm quyền để làm cho họ viết lại nó? Tôi đoán câu trả lời là, thẩm quyền đạo đức của toàn đội, nếu các vấn đề được phát ra cho toàn đội.

— C Johnson

Trong trường hợp không có cơ chế xem xét mã mạnh mẽ cho mọi thành viên trong nhóm thì điều này là phù hợp. Nó cũng thích hợp để đảm bảo bozo không đào một cái hố thật sâu (thậm chí không đáng với chi phí sửa chữa) trước khi được yêu cầu làm lại nó.

— mattnz

Nếu người đó không biết gì hơn, nhưng muốn tìm hiểu, hãy cung cấp một số hướng dẫn và đánh giá mã. Hãy chắc chắn rằng chúng được tiếp xúc với mã tốt.

Các nhà phát triển thực sự nghèo là những người bị đặt theo những cách xấu và chống lại việc học bất cứ điều gì mới. Hy vọng duy nhất của bạn là để cho họ chịu đựng thông qua việc duy trì mớ hỗn độn của riêng họ hoặc một số thứ đơn giản hơn. Lý tưởng nhất, một người có thẩm quyền đứng lên và nói phù hợp hoặc rời đi.

— Jeff
nguồn

Thái độ thực sự quan trọng. Tôi thường thấy người mới khiêm tốn hơn, và cởi mở để đánh giá và phê bình mã. Những người này rất dễ nói chuyện. Và bạn có thể dễ dàng nói chuyện với họ về điểm yếu của họ. Đó là những cựu chiến binh kiêu ngạo, người sẽ vỡ tan như một triệu mảnh thủy tinh khi họ bị chỉ trích về công việc của họ.

— C Johnson