Re: [討論] 對岸的軟體工程師

看板Tech_Job作者 (竹科管理處網軍研發人員)時間3年前 (2020/11/06 23:04), 3年前編輯推噓95(96推 1噓 86→)
留言183則, 93人參與, 3年前最新討論串2/2 (看更多)
※ 引述《Ferrara (紅燒冰淇淋)》之銘言： : 本ID在台北一家陸商待過一個月 : 發現對岸SW RD的整code習慣是這樣 : 覺得自己寫好了，就commit了 : commit之前不做驗證，不初步抓一下bug : 連local build pass都沒有 : 負責管respitory 的人得一條條幫他們檢查 : 最近被一家台陸合資的公司找去面試 : 主管說他在管理gerrit的時候很難整合daily build : 因為對岸的工程師丟上來的東西常常build不過 : 請問各位鄉民 : 你們共事過的對岸工程師也會這樣嗎 分享一下現在中國公司工作的狀況好了， 程式碼 build 都沒過，是絕對不能回家的，你會害很多人被扣錢。 首先程式碼 commit到分支前，都要設定好jenkins 使用 git push 程式碼到 repository 的分支時， 會觸發CICD流程，大致會執行以下流程: 編譯 build --> 弱點掃描 --> 程式碼取名規則檢查 --> UT Test 除了程式沒有語法上的bug 可以build 如果程式碼的變數，函數名稱不符合規範， 或程式碼有常見弱點，或缺陷defect CICD是不會通過的。 UT Test 除了測功能外，還要滿足測試程式碼的覆蓋率 Coverage。 如果 UT 的 Test Case 亂寫，或是Test Case 沒有覆蓋到75%的程式碼，CICD也不會過。 功能正常還是不夠的， 現在幾千萬人使用，7x24 的系統，非常追求程式碼的穩定，效率，可維護，透明。 我們公司一出現停機的Bug，一堆人都要扣幾千塊人民幣的。 我們也在意機器查不到的業務邏輯 Bug。 還要在意可維護性，也要避免有寫作弊程式碼，不可維護的黑箱，神秘的隱藏程式碼。 所以程式碼能跑，還不夠，要靠人去提高程式碼的品質。 接下來就是透過 gerrit，去找人 review程式碼。 review的人，有不同的權限，至少要有權限高的人+2 才能將程式碼merge到分支。 這時候，問題就來了，那麼review的人偷懶不就好了? 大家省事。 當初我也是這樣想的。 你的同事或資深工程師Review完的Comment， 每周會有更資深的工程師或部長，再檢查這些人的review是否合理。 程式碼出了事情，停機了，這些資深工程師都是要扣薪水的。幾千幾千人民幣的扣。 所以除非你是老闆兒子，不然你討好同層，或討好主管根本沒用。 另外，一次交大量的程式碼，減少review次數，也是不行的。 超過300行程式碼的commit 都會要有資深工程師或主管審核，才能夠merge程式碼， 而且每日自動檢查會通報一次性review超過300行程式碼的行為。 沒有Bug，CICD過了，review過了，程式碼merge到分支，總沒事了吧。 我剛來公司時也是這樣天真的。 結果合併到分支的程式碼，每天定期都會自動跑CICD， 而且UT會因為網路不穩連接時間太長失敗。 網路不穩，怪軟體工程師囉? 對，就是那麼坑。 晚上12點CICD沒過，不管是不是程式碼本身的問題， 你和你的主管都要扣本月績效分數，影響本月薪水。 所以沒人敢讓每日定期檢查的CICD不過。 所以程式碼的Test Case又要很聰明的，知道何時使用mock測試， 避免各種網路連線不穩定的UT測試失敗。 哇靠，那麼麻煩，我乾脆不寫程式碼或少寫好了，寫越多事情越多。 不行喔，每周/每月，都會統計程式碼行數， 然後大家比較一番，然後落後的人給點壓力。 以上只是每日的開發工作， 每周還有Coverity的靜態程式碼掃描，不過就通報。 軟體發新版本時，測試工程師從分支拉取程式碼， Coverity的靜態程式碼掃描不過，也進不了發版本的測試階段。 當然Coverity的掃瞄常常誤報，即使誤報也要走流程，然後安全專家同意去取消。 其他懶得說了，反正一堆為了提高軟體品質的規定，走火入魔的規定比較常見。 有空再交流。 -------------- 當然不是每家中國軟體公司都這樣搞人的。 但是，真的沒聽過 build 不過，還可以正常混到薪水的。 有的這種公司話，請站內信推薦，真的。一定一堆人搶著去爽。 ※ 編輯: DrTech (116.77.73.243 中國), 11/06/2020 23:08:12
推
11/06 23:07, 3年前 , 1F
親!信你了!
11/06 23:07, 1F
推
11/06 23:17, 3年前 , 2F
人力成本超高...，方便問sprint和release週期嗎？
11/06 23:17, 2F
→
11/06 23:19, 3年前 , 3F
發版的周期是，每個月發兩個release版本。長假前後凍結。
11/06 23:19, 3F
推
11/06 23:21, 3年前 , 4F
也太硬...突然覺得現在過好爽
11/06 23:21, 4F
→
11/06 23:21, 3年前 , 5F
人力成本其實不高，風氣就是晚上8點算早下班。
11/06 23:21, 5F
→
11/06 23:23, 3年前 , 6F
加班根本不給薪水，是福報阿。真心羨慕原文那種隨意公司。
11/06 23:23, 6F
推
11/06 23:27, 3年前 , 7F
好猛，除了很硬，扣薪也太可怕
11/06 23:27, 7F
推
11/06 23:30, 3年前 , 8F
親 你好猛
11/06 23:30, 8F
※ 編輯: DrTech (116.77.73.243 中國), 11/06/2020 23:36:57
推
11/06 23:37, 3年前 , 9F
還好我一行都拆好幾行寫0..0
11/06 23:37, 9F
→
11/06 23:45, 3年前 , 10F
扎心了 老鐵
11/06 23:45, 10F
推
11/06 23:52, 3年前 , 11F
這篇可信度高
11/06 23:52, 11F
推
11/06 23:57, 3年前 , 12F
規模大了就變法治，有些地方就踩很硬
11/06 23:57, 12F
推
11/07 00:00, 3年前 , 13F
沒想到寫程式也跟學音樂ㄧ樣，練習不夠，隨便亂寫，都有
11/07 00:00, 13F
→
11/07 00:00, 3年前 , 14F
人知道
11/07 00:00, 14F
推
11/07 00:14, 3年前 , 15F
我決定開始點炸雞排的技能樹了...
11/07 00:14, 15F
推
11/07 00:20, 3年前 , 16F
原po大神
11/07 00:20, 16F
推
11/07 00:22, 3年前 , 17F
好恐怖
11/07 00:22, 17F
推
11/07 00:22, 3年前 , 18F
這麼肝是給多少啊...
11/07 00:22, 18F
推
11/07 00:24, 3年前 , 19F
我以為只有 google 會注意軟體品質
11/07 00:24, 19F
推
11/07 00:29, 3年前 , 20F
哥 您年薪？
11/07 00:29, 20F
→
11/07 00:31, 3年前 , 21F
這跟年薪與能力無關吧。剛畢業的大學生也是這樣管理。
11/07 00:31, 21F
推
11/07 00:32, 3年前 , 22F
一個code review這麼多遍人力成本當然高。有沒有加班費是
11/07 00:32, 22F
→
11/07 00:32, 3年前 , 23F
另一件事情。
11/07 00:32, 23F
→
11/07 00:32, 3年前 , 24F
只能說有經驗的公司，早就把人性看得很透了，所以訂一堆
11/07 00:32, 24F
→
11/07 00:33, 3年前 , 25F
規則防止人偷懶或失誤。
11/07 00:33, 25F
推
11/07 00:33, 3年前 , 26F
沒想到大陸也有公司走這套了,直接扣幾千人民幣真的會怕XD
11/07 00:33, 26F
→
11/07 00:40, 3年前 , 27F
之前請上面code review還要一直寄信跪求QQ
11/07 00:40, 27F
推
11/07 00:48, 3年前 , 28F
原po這間都是local pay再加一點點 XDDD
11/07 00:48, 28F
→
11/07 00:48, 3年前 , 29F
所以一個月就閃了
11/07 00:48, 29F
推
11/07 00:54, 3年前 , 30F
一個月兩次 release 有點少,通常我們都一個禮拜四次
11/07 00:54, 30F
→
11/07 00:55, 3年前 , 31F
盡量發布小的 commit, 有問題都可以很快 trace
11/07 00:55, 31F
→
11/07 00:56, 3年前 , 32F
原 po 公司可能是金融相關的？才會用這麼嚴謹的方式上版？
11/07 00:56, 32F
→
11/07 00:56, 3年前 , 33F
突然想到 每個月生這麼多程式碼 改這麼多碼 修這麼多bug
11/07 00:56, 33F
推
11/07 00:57, 3年前 , 34F
H
11/07 00:57, 34F
→
11/07 00:57, 3年前 , 35F
最後這系統到底會變成什麼 這麼多人才生出來的東西 肯定
11/07 00:57, 35F
→
11/07 00:58, 3年前 , 36F
很強很潮吧 若不是這樣就很無奈了...釘不完的釘子阿....
11/07 00:58, 36F
→
11/07 01:00, 3年前 , 37F
一個禮拜四次release? 這好像有點屌.....
11/07 01:00, 37F
→
11/07 01:02, 3年前 , 38F
我現在是做多媒體串流平台的後台，現在用戶多的軟體出問題
11/07 01:02, 38F
還有 105 則推文
推
11/07 19:46, 3年前 , 144F
台灣二線公司, 花錢買coverity授權都...
11/07 19:46, 144F
→
11/07 19:46, 3年前 , 145F
小公司就更別談了
11/07 19:46, 145F
推
11/07 19:50, 3年前 , 146F
如果這樣搞法,公司還不願意花錢請頂級人才
11/07 19:50, 146F
→
11/07 19:50, 3年前 , 147F
一堆測試問題就把錢都給扣光光了
11/07 19:50, 147F
推
11/07 20:04, 3年前 , 148F
用過國際第一流大廠(非台商)的SDK, 問題都一卡車了...
11/07 20:04, 148F
→
11/07 21:41, 3年前 , 149F
其實扣薪水都不是什麼大錢，有些公司就是 fail build 請
11/07 21:41, 149F
→
11/07 21:42, 3年前 , 150F
全公司飲料咩，群暉以前也有這樣做
11/07 21:42, 150F
→
11/07 21:42, 3年前 , 151F
一般 "有制度" 的 "大" 公司流程都是這樣
11/07 21:42, 151F
推
11/07 22:18, 3年前 , 152F
還有在比行數??
11/07 22:18, 152F
推
11/07 23:08, 3年前 , 153F
聽完我想去賣雞排了 有夠硬
11/07 23:08, 153F
推
11/07 23:13, 3年前 , 154F
gerrit code review、Jenkins、coverity都算有意義，可
11/07 23:13, 154F
→
11/07 23:13, 3年前 , 155F
以增加軟體品質。不過算行數真的很好笑，依LinkedList
11/07 23:13, 155F
→
11/07 23:13, 3年前 , 156F
為例，Datapath寫的漂亮的和爛的，總行數可能差到三倍以
11/07 23:13, 156F
→
11/07 23:13, 3年前 , 157F
上。
11/07 23:13, 157F
推
11/08 01:18, 3年前 , 158F
推
11/08 01:18, 158F
推
11/08 03:02, 3年前 , 159F
看起來就是正經在做事情的公司，而且有一定規模
11/08 03:02, 159F
推
11/08 05:45, 3年前 , 160F
行數 只是為了比較 讓你感到羞恥而已
11/08 05:45, 160F
→
11/08 05:46, 3年前 , 161F
這點不是很人性就是了 會因爲這點而做不下去
11/08 05:46, 161F
推
11/08 09:22, 3年前 , 162F
這麼多毛的公司我pass
11/08 09:22, 162F
推
11/08 09:41, 3年前 , 163F
扣除行數要求，這些要求都還好吧？
11/08 09:41, 163F
推
11/08 12:51, 3年前 , 164F
好想去開眼界啊...
11/08 12:51, 164F
→
11/08 13:06, 3年前 , 165F
比行數真的不知道能幹嘛..程式重點不是演算法嗎？好的演
11/08 13:06, 165F
→
11/08 13:06, 3年前 , 166F
算法比寫一堆垃圾code去拼出功能好多了
11/08 13:06, 166F
推
11/08 16:58, 3年前 , 167F
這看起來很正常吧...
11/08 16:58, 167F
推
11/08 18:34, 3年前 , 168F
嗯,的確很正常,很多大公司軟體程度就是這麼低落,這樣的
11/08 18:34, 168F
→
11/08 18:34, 3年前 , 169F
CICD若對提高品質有效的話,馬上就不需要commit程式碼了,
11/08 18:34, 169F
→
11/08 18:35, 3年前 , 170F
還一直要求commit程式碼,邏輯錯亂的一堆
11/08 18:35, 170F
→
11/08 18:37, 3年前 , 171F
跟產線的良率當KPI 87%像 有的血汗公司產線良率還會採連坐
11/08 18:37, 171F
→
11/08 18:37, 3年前 , 172F
法 科科
11/08 18:37, 172F
推
11/09 11:04, 3年前 , 173F
除了扣錢 其它都蠻合理的
11/09 11:04, 173F
推
11/09 12:20, 3年前 , 174F
這種公司能永續發展才奇怪 動不動就罰錢
11/09 12:20, 174F
→
11/09 12:29, 3年前 , 175F
小公司的爛才是突破天際
11/09 12:29, 175F
→
11/09 13:21, 3年前 , 176F
行數當kpi到底是什麼爛管理腦= =
11/09 13:21, 176F
→
11/09 13:25, 3年前 , 177F
除了行數kpi還有扣錢以外，以軟體開發來說剛剛好而已
11/09 13:25, 177F
推
11/09 15:14, 3年前 , 178F
推
11/09 15:14, 178F
推
11/09 21:16, 3年前 , 179F
扣錢XD
11/09 21:16, 179F
推
11/10 00:15, 3年前 , 180F
UT是啥 unittest？？？這也要簡稱？
11/10 00:15, 180F
→
11/10 00:15, 3年前 , 181F
以軟體開發來說這就是大公司做法啊 除了扣錢
11/10 00:15, 181F
推
11/10 01:53, 3年前 , 182F
仔細看敘述，行數不是KPI
11/10 01:53, 182F
推
11/10 10:02, 3年前 , 183F
比行數，真的笑了
11/10 10:02, 183F
‣ 更多 DrTech 的文章
文章代碼(AID): #1VfMOR2u (Tech_Job)
‣ 更多 DrTech 的文章
文章代碼(AID): #1VfMOR2u (Tech_Job)