Anh ấy từ chối lắng nghe nhóm và gần đây anh ấy đã dừng đánh giá mã, kiểm tra đơn vị, chia sẻ chi tiết thực hiện ...
Đánh giá mã không nhất thiết yêu cầu người viết mã phải gửi tác phẩm để xem xét.
Một cách dễ dàng để theo dõi những gì anh ta làm là theo dõi lịch sử của VCS, tìm kiếm các đăng ký của anh ta. Nếu bạn lo lắng về mã của anh ấy, đây là một cách dễ dàng để tìm thấy nó. Nhận một lịch sử khác biệt, nhìn vào những gì anh ấy đưa vào, và xem nếu có bất kỳ lá cờ đỏ nào nhảy ra khỏi bạn. Bắt các đăng ký của anh ta đủ nhanh và nếu bạn tìm thấy một vấn đề, bạn có thể khôi phục lại cam kết và gửi e-mail cho anh ta để có hiệu lực. Bạn được phép gọi ra các thành viên trong nhóm của mình, ngay cả với tư cách là một lập trình viên cơ sở, khi bạn thấy có gì đó không ổn.
Có, anh ta "mã hóa" nhanh, nhưng mã của anh ta chỉ là một trình tạo lỗi. Các thành viên khác trong nhóm và tôi đang trong "giai đoạn sửa lỗi" và 80% lỗi đến từ mã của anh ấy. Tôi không muốn sửa lỗi của anh ấy. Và quản lý là mù quáng, hoặc không muốn thấy điều này, hoặc có thể họ thích "tốc độ" của anh ấy.
Mã xuất phát từ yêu cầu. Yêu cầu dẫn đến các bài kiểm tra có thể chạy được để xác minh các yêu cầu đã được đáp ứng. Các thử nghiệm này có thể được chia nhỏ hơn nữa và có thể được viết trước khi các thay đổi được thực hiện để xác minh rằng các thay đổi đáp ứng các yêu cầu (red-green-refactor; bản chất của TDD).
Thêm số liệu "phạm vi bảo hiểm mã" vào máy chủ xây dựng nhóm của bạn (hy vọng bạn có một; nếu không, đó là vấn đề đầu tiên của bạn). Chỉ cần kiểm tra các bài kiểm tra đơn vị đó sẽ không bắt gặp các vấn đề với mã không TDDed mới của anh ta, được thực hiện ở các khu vực không có bài kiểm tra đơn vị. Sau khi chạy tất cả các bài kiểm tra đơn vị, máy chủ xây dựng nên thực hiện mọi dòng mã một cách lý tưởng, nhưng thực sự có một số thứ bạn không thể kiểm tra đơn vị. Trên thực tế, bạn vẫn có thể mong đợi mức độ bao phủ 95% hoặc tốt hơn (hoặc loại trừ một số thư viện hoặc loại tệp nhất định khỏi phạm vi bảo hiểm). Sớm hay muộn, chàng cao bồi của bạn sẽ kiểm tra thứ gì đó phá vỡ bản dựng vì anh ta đã giảm mức bảo hiểm dưới ngưỡng và bạn gọi anh ta ra.
Và theo như "tốc độ" có liên quan, tốc độ là tốc độ bạn hoàn thành mọi việc "hoàn thành" và nó không "hoàn thành" cho đến khi nó được thực hiện một cách chính xác. Bạn có thể đưa nó cho người quản lý của bạn theo cách này; hãy xem xét một thợ sửa xe, khi người quản lý đưa chiếc BMW của mình đi thay dầu, quên không cắm chảo dầu vào, và kết quả là tất cả dầu mới đổ ra trước khi anh ta lái xe ra khỏi gara. Chắc chắn, việc thay dầu chỉ mất năm phút, nhưng người quản lý sẽ không quan tâm đến điều đó khi động cơ xe của anh ta bị kẹt trên đường về nhà. Anh ta sẽ quan tâm rằng thợ máy đã bỏ lỡ một bước, điều đó sẽ khiến anh ta tốn thêm rất nhiều thời gian và tiền bạc để sửa chữa. Ngay bây giờ, anh ấy trả một chàng cao bồi để thực hiện công việc rất nhanh, và sau đó anh ấy ' s trả cho phần còn lại của đội một khoản tiền lớn hơn nhiều để đến và thực hiện lại công việc một cách chính xác. Điều gì thực sự là lợi thế của việc tiếp tục để chàng cao bồi làm việc của mình?
Có cách nào mà tôi (với tư cách là đồng nghiệp trẻ tuổi của anh ấy, không phải ông chủ của anh ấy) có thể làm gì đó không?
Gọi anh ta ra. Khi bạn tìm thấy thứ gì đó anh ta làm hỏng, hãy cho anh ta thấy mã của anh ta bị lỗi như thế nào, làm thế nào anh ta có thể ngăn chặn vấn đề ngay từ đầu (bao gồm thiết kế phù hợp, TDD, đánh giá mã) và kết quả là bạn sẽ phải làm gì để sửa mã bị hỏng của mình.
Tôi cảm thấy mình là người cuối cùng thực sự quan tâm đến dự án.
klaxons sáng chói, đèn nhấp nháy, còi báo động rền rĩ - nếu bạn thực sự cảm thấy mình là người duy nhất quan tâm đến chất lượng mã do nhóm sản xuất, thì có vấn đề NGHIÊM TRỌNG. Nếu bạn cảm thấy bạn đang cố gắng kéo toàn bộ đội đá và hét vào kỷ nguyên mã hóa tốt, và nó chỉ quá nặng để lôi, hãy bỏ nó đi. Nếu có một đội khác tại công ty đang làm đúng, hãy yêu cầu chuyển nhượng, nếu không thì hãy biến đi.