Chia sẻ, tình huống chưa gặp trong công việc bao giờ

Xin chào mọi người.
Chuyện là hôm nay ở công ty mình có chứng kiến 1 tình huống sau đây.
1 bạn dev cãi nhau với sếp của mình vì code review.
Context là bạn dev có code 1 logic, đâu đó khoảng 100 dòng, rồi đưa vào file index của project, với lý do là đây là logic global.
Sếp yêu cầu tách ra 1 component / hook riêng, không thì sẽ không approve.
Cãi nhau dòng dã 2 ngày thì bạn dev có vẻ tăng xông rồi bắt đầu tag tất cả những anh em chơi trong hội ở công ty để yêu cầu hỗ trợ
Không ai trả lời, bạn dev lại tiếp tục đào mộ code cũ của các dev khác và cho rằng ai cũng có lỗi, sao họ dc approve mà em thì không ?
Vẫn không ai trả lời, bạn dev tiếp tục call cho sếp cao hơn và mách là bị trù dập và đánh giá không khách quan.

Mình là người chứng kiến, shock vì đi làm 4 công ty mà đây là lần đầu tiên thấy vì code review mà tranh luận gay gắt tới mức vậy.
Vì mình cũng đang định hướng lên làm manager, mình muốn hỏi các senior ở đây là tình huống như trên diễn ra thường xuyên không ? Và nếu có thì các bên phải xử lý ra sao mới hiệu quả cho công việc.

Xin cảm ơn mọi người

Mình chưa phải senior nhưng có 1 số nhận xét về phía case này như sau:

• phía bạn dev: "tag tất cả những anh em chơi trong hội ở công ty để yêu cầu hỗ trợ" -> xử lý sai, nếu cần support thì nên call những ae khác ở trong team/squad khác cùng dự án, không nên làm thế này. Không ai trả lời là đúng, đâu phải việc của họ?
• phía anh lead trực tiếp: "Sếp yêu cầu tách ra 1 component / hook riêng, không thì sẽ không approve." ý này chưa được rõ ràng, có thể là do cách nói của ông lead khó nghe, hoặc technical ko đủ cứng để nói cho thằng đệ nghe.
• nếu evident của bạn dev kia chuẩn thì kèo này lead sai rõ, còn evident chày cối thì do ông dev kia tăng xông cãi cùn.

----
mình cũng đã từng gặp 1 số case căng thẳng như này, nếu bạn đi theo hướng managing thì nên tập quen các tình huống như này. lead/manager thì value nằm ở chỗ giải quyết conflic chứ lúc nào cũng yên bình thì cần gì mấy ông này?

Bổ sung thêm cho còm trên:

youknowwho151 said:

Không ai trả lời, bạn dev lại tiếp tục đào mộ code cũ của các dev khác và cho rằng ai cũng có lỗi, sao họ dc approve mà em thì không ?

Click to expand...

Câu này cũng khá cùn.

là do ông dev code ko tách đc hay như nào mà code xong bị reject cũng ko sửa thế này . mấy cái như này thường ae phải họp lại để choý kiến chứ nhỉ

Tách ra là đúng rồi chứ sao. Mẹ bới logic cũ lên rồi so sánh thì cùn vãi nhái.

ngựa này còn non ko mà háu đá v bác

Tách ra thì có sao. Mẹ thằng dev phải tôi tôi cho cút mẹ luôn, cái tính hơn thua không cãi việc này thì cũng cãi việc khác

Thớt cãi cùn vừa thôi, mai lo mà tách code ra đi. Tính post thred tìm đồng minh à.

youknowwho151 said:

Xin chào mọi người.
Chuyện là hôm nay ở công ty mình có chứng kiến 1 tình huống sau đây.
1 bạn dev cãi nhau với sếp của mình vì code review.
Context là bạn dev có code 1 logic, đâu đó khoảng 100 dòng, rồi đưa vào file index của project, với lý do là đây là logic global.
Sếp yêu cầu tách ra 1 component / hook riêng, không thì sẽ không approve.
Cãi nhau dòng dã 2 ngày thì bạn dev có vẻ tăng xông rồi bắt đầu tag tất cả những anh em chơi trong hội ở công ty để yêu cầu hỗ trợ
Không ai trả lời, bạn dev lại tiếp tục đào mộ code cũ của các dev khác và cho rằng ai cũng có lỗi, sao họ dc approve mà em thì không ?
Vẫn không ai trả lời, bạn dev tiếp tục call cho sếp cao hơn và mách là bị trù dập và đánh giá không khách quan.

Mình là người chứng kiến, shock vì đi làm 4 công ty mà đây là lần đầu tiên thấy vì code review mà tranh luận gay gắt tới mức vậy.
Vì mình cũng đang định hướng lên làm manager, mình muốn hỏi các senior ở đây là tình huống như trên diễn ra thường xuyên không ? Và nếu có thì các bên phải xử lý ra sao mới hiệu quả cho công việc.

Xin cảm ơn mọi người

Click to expand...

đôi khi cái mạnh của manager là dập tắt được binh biến chứ ko phải là họ quá giỏi về tech. Đọc câu chuyện của bạn chưa thấy lý do gì bạn dev kia phải đưa vào project index cho bằng được, milestone?

Việc tách ra 1 component / hook riêng thì có khó khăn gì không nhỉ, sao mà dev kia không chịu?

"bắt đầu tag tất cả những anh em chơi trong hội ở công ty để yêu cầu hỗ trợ" => ngớ ngẩn, hỏi những người cùng dự án thì còn có lý, chứ người ở dự án khác thì họ biết cái gì mà lên tiếng

"Không ai trả lời, bạn dev lại tiếp tục đào mộ code cũ của các dev khác và cho rằng ai cũng có lỗi, sao họ dc approve mà em thì không" => cái này tùy thuộc thời điểm, nếu là code lâu rồi thì có thể quy trình, yêu cầu lúc đó khác. Còn nếu là mới thì cái này có lý

Khi sếp trực tiếp không giải quyết được, việc escalate lên cấp cao hơn thì cũng đúng thôi. Nhưng phải 3 mặt 1 lời, tranh luận lịch sự. Ông sếp cần giải thích được tại sao cần tách component riêng, và chuyện approve code của dev khác nhưng dev này thì không

dandelion.voz said:

Việc tách ra 1 component / hook riêng thì có khó khăn gì không nhỉ, sao mà dev kia không chịu?

"bắt đầu tag tất cả những anh em chơi trong hội ở công ty để yêu cầu hỗ trợ" => ngớ ngẩn, hỏi những người cùng dự án thì còn có lý, chứ người ở dự án khác thì họ biết cái gì mà lên tiếng

"Không ai trả lời, bạn dev lại tiếp tục đào mộ code cũ của các dev khác và cho rằng ai cũng có lỗi, sao họ dc approve mà em thì không" => cái này tùy thuộc thời điểm, nếu là code lâu rồi thì có thể quy trình, yêu cầu lúc đó khác. Còn nếu là mới thì cái này có lý

Khi sếp trực tiếp không giải quyết được, việc escalate lên cấp cao hơn thì cũng đúng thôi. Nhưng phải 3 mặt 1 lời, tranh luận lịch sự. Ông sếp cần giải thích được tại sao cần tách component riêng, và chuyện approve code của dev khác nhưng dev này thì không

Click to expand...

đọc cmt thấy có thím này phân tích khá chuẩn này. Trong những công ty âu mỹ thì khi giải trình với cấp cao hơn thì ông sếp có thể sẽ là người sai (về lí). Nhưng khi giải quyết xong rồi thì người thiệt nhất vẫn là bạn dev kia. Chả có ai muốn làm việc chung team với một người tính tình nhạy cảm cả. Và cậu dev kia cũng có vẻ không coi trọng sếp của mình lắm (có thể bạn đấy là senior mới join cty và có suy nghĩ sếp trình gà hoặc là junior ngựa non háu đá)

supernija said:

đọc cmt thấy có thím này phân tích khá chuẩn này. Trong những công ty âu mỹ thì khi giải trình với cấp cao hơn thì ông sếp có thể sẽ là người sai (về lí). Nhưng khi giải quyết xong rồi thì người thiệt nhất vẫn là bạn dev kia. Chả có ai muốn làm việc chung team với một người tính tình nhạy cảm cả. Và cậu dev kia cũng có vẻ không coi trọng sếp của mình lắm (có thể bạn đấy là senior mới join cty và có suy nghĩ sếp trình gà hoặc là junior ngựa non háu đá)

Click to expand...

còn công ty việt thì sao bác

Dị văn hóa rồi, ông dev có vẻ cũng không giỏi mấy, chắc sẽ nhanh văng thôi. Gần thì sang bộ phận khác, xa thì bay qua cty khác.

Đi làm hướng tới mục tiêu chung, có kết quả sao cho có kq tốt nhất thôi tranh cãi ai đúng ai sai làm gì ko biết

youknowwho151 said:

bạn dev lại tiếp tục đào mộ code cũ của các dev khác và cho rằng ai cũng có lỗi, sao họ dc approve mà em thì không ?

Click to expand...

Là sao ta, ý là biết code mình lỗi vẫn đòi merge ak

via theNEXTvoz for iPhone

Gặp leader @tuanbeohdvn thì chắc dev này bay rồi, nhưng mấy thằng dev chỗ tôi nịnh giỏi lắm chứ ko cãi như thằng dev này.

làm sếp trong trường hợp này phải giải thích đc cho cậu dev kia hiểu
1. tại sao phải tách, là do tách để quản lý cho dễ hay sao
2. tại sao cái cũ đc approve mà cái mới lại ko, do thay đổi quy trình hay bây h yêu cầu sản phẩm cao hơn
Cho dù bây h hay sau này cũng sẽ có người mới vào, cũng sẽ xảy ra trường hợp tranh cãi như vậy, nếu ông ko giải thích được yêu cầu của mình cho công việc mà cứ bắt làm rập khuôn theo như vậy thì khó làm sếp lắm

thôi fen lần sau manager yêu cầu tách thì tách nhé, đừng cãi cùn làm gì cho khổ, kiếp thợ gõ chịu khó vậy, không thì kiếm cty khác

youknowwho151 said:

Xin chào mọi người.
Chuyện là hôm nay ở công ty mình có chứng kiến 1 tình huống sau đây.
1 bạn dev cãi nhau với sếp của mình vì code review.
Context là bạn dev có code 1 logic, đâu đó khoảng 100 dòng, rồi đưa vào file index của project, với lý do là đây là logic global.
Sếp yêu cầu tách ra 1 component / hook riêng, không thì sẽ không approve.
Cãi nhau dòng dã 2 ngày thì bạn dev có vẻ tăng xông rồi bắt đầu tag tất cả những anh em chơi trong hội ở công ty để yêu cầu hỗ trợ
Không ai trả lời, bạn dev lại tiếp tục đào mộ code cũ của các dev khác và cho rằng ai cũng có lỗi, sao họ dc approve mà em thì không ?
Vẫn không ai trả lời, bạn dev tiếp tục call cho sếp cao hơn và mách là bị trù dập và đánh giá không khách quan.

Mình là người chứng kiến, shock vì đi làm 4 công ty mà đây là lần đầu tiên thấy vì code review mà tranh luận gay gắt tới mức vậy.
Vì mình cũng đang định hướng lên làm manager, mình muốn hỏi các senior ở đây là tình huống như trên diễn ra thường xuyên không ? Và nếu có thì các bên phải xử lý ra sao mới hiệu quả cho công việc.

Xin cảm ơn mọi người

Click to expand...

mếu liên quan tới ngành code củng này, tuy nhiên thấy có một số vấn đề:
1. Ngồi lại thảo luận cùng sếp tại sao cần tách ra như vậy. Nếu hợp lý thì làm thôi. Nếu không hợp lý, nhưng đây là yêu cầu công việc -> thì cứ làm thôi, hoặc không muốn làm thì đề nghị chuyển dự án, chuyển leader, thậm chí là chuyển việc.
2. Việc tag hội ae vào y hệt bọn trẻ trâu kéo đàn kéo đội vào. Nhưng lớn rồi, đi làm rồi, hầu hết mọi người không đủ ngu để đi đối đầu với lãnh đạo đâu, lãnh đạo là người trả miếng cơm manh áo, còn bạn thì không.
3. Việc đào mộ code của dev khác thật vô nghĩa, giống ở cty mình, end user lấy ví dụ một trường hợp vi phạm bản quyền trong quá khứ để ngụy biện cho vi phạm của họ. Vớ vẩn, quá khứ có thể đúng hoặc sai, nhưng tương lai thì là bất định. Nó không phải là máy dập trong dây chuyền sản xuất mà phải yêu cầu giống nhau. Thằng trước dẫm vào cứt thì thằng sau không có nghĩa vụ phải dẫm lại bãi cứt đó để đảm bảo sự đồng đều.
4. Call đến sếp cao hơn cũng được, nhưng phải dựa trên sự hợp lý, ở đây là phân tích kỹ thuật, tính hợp lệ, phù hợp.... còn call kiểu mách lẻo chung chung thì thôi.
...
n. Cho bạn viết bài có định hướng làm manager, thì nên coi chuyện này là bình thường, shock làm gì, nó như drama các bạn nữ hay xem thôi mà. Gặp càng nhiều càng tốt, nó cho mình càng nhiều cái nhìn và cách xử lý vấn đề.

voz.tahn said:

làm sếp trong trường hợp này phải giải thích đc cho cậu dev kia hiểu
1. tại sao phải tách, là do tách để quản lý cho dễ hay sao
2. tại sao cái cũ đc approve mà cái mới lại ko, do thay đổi quy trình hay bây h yêu cầu sản phẩm cao hơn
Cho dù bây h hay sau này cũng sẽ có người mới vào, cũng sẽ xảy ra trường hợp tranh cãi như vậy, nếu ông ko giải thích được yêu cầu của mình cho công việc mà cứ bắt làm rập khuôn theo như vậy thì khó làm sếp lắm

Click to expand...

trường hợp cháu kia ko hiểu thì làm thế nào, chả nhẽ chạy cả ngày theo nó để giải thích