Re: [心得] 如何向別人證明自己寫的是好code?

[richliu (皮皮)]

※ 引述《darkkiwi (....)》之銘言：
: 我總得在離開這間公司的時候講得出自己的價值在哪，我很想要再跨入更高的境界。
: 但老實說，我真的不知道該怎麼做比較好，難道就只能跟著一起墮落嗎?
: 出一張嘴的人很多，做得出實績的人卻很少，我不想當只剩一張嘴的人，
: but what can i do?
: 謝謝將我的牢騷文看完的每一位前輩，小弟再這給您磕頭   <(_._)>

這個是新同學進入 Software 產業最常見的問題.

為什麼我不能改寫程式架構,
前輩寫的 code 這麼 sucks , 怎麼這麼鳥.
為什麼沒有人去整理 code.


這邊為您解答吧.

1. 你改完的 Code 有經過 Test Team(QA) 驗證嗎?
   還是自己改完, hi 完了, 覺得很好, Bug free.
   你有沒有碰過改了這個爛了那個?
   Test Team (QA) 要為你自 hi 的行為再下去測試嗎?

2. 你應該是沒有被操到的新人吧, 所以有非常好的時間可以
   幻想怎麼寫 code 比較好.
   等到有一天碰到那種 ASAP 的 Project , 連續玩個一年
   你就知道什麼叫做有 Code 就好.

3. 整理 Code 的問題, 同上.
   最近還是在做 Code Review. 不要講別的,
   我連看不下去的 Global Variable / Local Variable 混用都不敢改.
   只能改改無關緊要的程式流程

這些問題其實在於
1. Manager 有沒有定時做 Code Review 的習慣, 如果連你的 Manager
   都沒有做 code review, 自然就會出現這些奇奇怪怪的 code.

2. 421 法則.
   合理的 Schedule 是 2
   RD 希望的 Schedule 是 4
   實際上的 Schedule 是 1
   這種狀況下要有好成果是很難的.

3. RD 個人的素養.
   大家學習的起點都不一樣, 對於 Language 的認知也不一樣
   有些人會寫
   strncpy(dst,src,strlen(src));
   有些人只寫到
   strcpy(dst,src);
   如何讓這些人在一起 cowork , 這是很大的學問呢.

4. 你的 code 替公司產生多大產值呢?
   寫得好好的 code 可能不比那些 sucks 的好到那邊去
   說不定 sucks 的 code 還比較得人心.

很現實, 不過實際上就是這樣的呀, 如果要好好寫 Code.
其實就是學生時代要好好練習好, 要不然出去就是這樣.
而大多數時間也只能接受

--
※ 發信站: 批踢踢實業坊(ptt.cc) 
◆ From: 140.113.22.92
→ iincho:這個是管理問題, 其實新人的要求是沒錯的.
→ iincho:長期屈服於現狀是事實,但是不代表沒有改的空間..
推 zaku:新人難免有覺得自己是天才的狀況啦
推 VincentKey:說的好！！！！
→ iincho:4.是我認為最糟糕的狀況, 通常是某些人把成本攤到別人身上
推 jily:大推 我想說的都被說完了
→ krthree:恩．．新鮮人總是比較理想化啊．．過幾年就習慣了
推 yangyr:這是台灣產業現況沒錯，關鍵還是產值及上市時程的問題
→ yangyr:不過呢，當你老是在擦別人的屁股主管or公司又沒感覺..
推 littlebau:對阿 過兩年就會被同化的 ... 我在煽風點火..
→ yangyr:我可不覺得這只能逆來順受 XD
→ littlebau:新人：我才不會被同化！！！ 也不要！
推 tkbible:這就是台灣的產業現況...
推 osforyou:strncpy(dst,src,strlen(src)); strcpy(dst,src); 差在哪
推 pakker:strncpy(dst,src,sizeof(dst)); 這樣吧
推 anr2:沒差別
推 JarnoChen:buffer overflow 的問題啦
→ JarnoChen:好習慣當然是用 strncpy
推 osforyou:這裡是科技工作版吧  pakker 你也錯了
推 anr2:沒錯啊 如果他只要COPY一個字
推 anr2:GNU Coding standard 是本部錯的入門書~~
推 GAMRYA:除了overflow，還有dst裡面有無garbage value也要考慮吧
→ GAMRYA:如果src短於dst，strncpy可以null-padding
→ GAMRYA:pakker好像也沒錯....strncpy也不會做bounds checking...？
推 osforyou:jarnochen 也錯了，這兩個式子的不同不在 overflow
→ osforyou:anr2 也錯了，這兩個式子有差別  差別大了
→ osforyou:gamrya 也錯了  null-padding 跟 src < dst ? 沒關係
推 osforyou:這裡是科技工作版吧…  原 po 講得已經夠好笑了  沒想到
→ osforyou:推文更是錯得亂七八糟
→ iincho:welcome to the real world..?
推 osforyou:real world.. 連最基本的 strxxx 都不懂  也太誇張了吧
推 JarnoChen:osforyou 去看看 msdn的 security note 寫什麼吧
→ JarnoChen:這麼強去跟微軟說他們文件寫錯了呀
→ iincho:翻了一下VC的CRT..strncpy真的只有多做null-padding啊...
→ iincho:那不一樣的到底是...是....是..o_O?
推 osforyou:jarnochen 連自己錯在哪都不知道  大絕放錯方向了
→ osforyou:我轉到 c_and_cpp 版了，那邊總有些高手能為你解答
推 bug001:轉過去是不錯啦，不過只出張嘴說別人錯，你到底有沒有料?
推 UNARYvvv:其實被說錯的該自己回去看看哪邊錯..C板那邊我已經解答了
→ UNARYvvv:推文哦..反而感覺是在討論的語氣上面啦, 直接放一句
→ UNARYvvv:"好笑" "亂七八糟" 的話, 這樣一下子氣氛就僵掉了XD
→ iincho:搞了半天爭執的點根本不在同一個地方..實在是有囧到...
推 osforyou:睡了一個晚上，果然有高手為各位解答了，而且非常詳細
→ osforyou:我本來是想笑 richliu，因為他說原 po 是沒被操作的新人
→ osforyou:所以有非常好的時間可以幻想怎麼寫 code 比較好.
→ osforyou:結果自己的例子似乎顯似說自己花太少時間研究怎麼寫 code
→ osforyou:我沒有好心到像 unaryvvv 那樣，但還是恭喜各位學了一課
→ GAMRYA:推bug01，osforyou該回去檢討檢討，一開始是你先問的
→ GAMRYA:大家以為你不懂所以簡短推文大概提點，又沒要為你上課
→ GAMRYA:結果你只是想藉由否定別人來自hi吧...
推 OnlyRD:其實台彎公司的CODE都寫得蠻糟的..跟大陸人比..