查看完整版本 : 邊個負責CODE REVIEW?

gunbuster363 2019-4-19 11:46 AM

邊個負責CODE REVIEW?

最近要睇新個Junior
老細叫佢地寫Program
佢地寫完就隊個Program比我, 好明顯無試清楚
其實我好憎Import人地的Source入我個Eclipse
因為公司無行Central CMS
我一打開見到各種各種問題: Hardcode, 留低垃圾Dead Code, 一個Main Method行晒所有野
到我話有問題, 佢地改完好快又隊個Program去我Email
我又要Download, Unzip再Copy落去
真是好煩
仲要三個一齊來, 個個都話做完, 實情就是我幫佢地Debug

到底是什麼人要負責咁樣做Code Review Debug?
我做左DEV 無8年都7年, 從來無要過老細睇Code, 老細從來不睇, 佢唔睇都情有可原(太忙!)
而且JAVA佢地更加唔識, 一向都是我地班AP自己生性

但呢班Junior, 其實是昆老細話自己識寫Program, 入到來好明顯是生手
無辦法, 公司唔出錢, 淨係請到呢種質素, 我地都只好硬食, 起碼叫聽教聽話
叫佢去東佢去東

無盡外圍的湯圓 2019-4-19 11:52 AM

[quote]原帖由 [i]gunbuster363[/i] 於 2019-4-19 11:46 AM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498029204&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]
最近要睇新個Junior
老細叫佢地寫Program
佢地寫完就隊個Program比我, 好明顯無試清楚
其實我好憎Import人地的Source入我個Eclipse
因為公司無行Central CMS
我一打開見到各種各種問題: Hardcode, 留低垃圾Dead Code, 一個Main Method行晒所有野
到我話有問題, 佢地改完好快又隊個Program去我Email
我 ... [/quote]

Tell to them to Code Review each other before send them to you to review.  Then if you find something wrong, teach all of them in one shot because they all saw the code.  Over 1 month, your team should be healthy.

gunbuster363 2019-4-19 12:19 PM

[quote]原帖由 [i]無盡外圍的湯圓[/i] 於 2019-4-19 11:52 AM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498029367&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]


Tell to them to Code Review each other before send them to you to review.  Then if you find something wrong, teach all of them in one shot because they all saw the code.  Over 1 month, your team s ... [/quote]

Sounds like an excellent idea:loveliness:

PacificIslander 2019-4-19 01:58 PM

[quote]原帖由 [i]無盡外圍的湯圓[/i] 於 2019-4-19 11:52 AM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498029367&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]


Tell to them to Code Review each other before send them to you to review.  Then if you find something wrong, teach all of them in one shot because they all saw the code.  Over 1 month, your team s ... [/quote]

湯圓兄個 group review 的確係個好主意。一則佢哋大家咁高咁大,冇所謂尷尬,二側大家一齊睇一齊聽老大講,慳水慳力。


另外一樣嘢我會加,係糾正觀念,呢個極其重要。佢哋打包畀你,你要做一輪嘢又要 check,呢個模式佢地係幫手 helper,係完全錯誤。佢哋 junior 或者只係寫一個單元,成個成品佢哋冇辦法見到掌握,甘即係冇 ownership,所以變成掉一大堆嘢畀你,你就負責嵌埋一齊,重點變成你嵌得啱唔啱,佢哋角色只係幫你手,噉係完全錯誤。


正確觀念係,佢哋負責嘅單元要係成品甘,佢哋個單元點樣銜接,要佢哋負責搞通佢。你反為係佢哋嘅幫手。即係管理層係 faciliator 一樣,幫忙下面工人有好既工作環境,資訊知識,去完成佢哋嘅工作。佢唔知輸入係點輸出係點你掉畀佢,佢哋要交足功課。


打個比喻,佢哋係個別嘅裝修師傅,佢哋唔係五金鋪,掉一支二支螺絲俾你由你決定去用唔用得到,水電點銜接,佢哋要夾好。你作為最終收貨人,最多係幫佢哋串連下。希望我解釋得明白。

m.png 2019-4-19 02:23 PM

我地以前都係行 peer code review 的,review 時是 author 與 reviewer side-by-side, 主要由 author 解釋,reviewer 慢慢聽,慢慢睇,有問題就發問。

察覺這個安排的好處係:
- author 知道自己 code 要給人 review, 寫 code 時會小心些,D 核突/懶惰野(如 hard code, 係咁 copy-n-paste, 複雜位睇唔明但又唔加 comments 等等)都能夠忍住手唔做 :-)
- 很多時是 author 自己在 code review 解釋期間發現bug 的。
- 多個人看過D code, 將來做 maintenance 都多個人有 D ideas

我常常跟他們講,一個 bug, 如果在 code review 時發現,可能需要10分鐘 fix 好,但如果要到 deployed to production 後才發現,整個 debug + fix + hot deploy 過程可能要幾個鐘。所以 code review 是有價值的,不要話係浪費時間。

我的無比膏 2019-4-19 03:42 PM

你信機器,可以eclipse 啲plugin scan 下啲code
不過我就用winmerge 逐句睇

smartboy123 2019-4-19 07:20 PM

[quote]原帖由 [i]gunbuster363[/i] 於 2019-4-19 11:46 AM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498029204&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]
最近要睇新個Junior
老細叫佢地寫Program
佢地寫完就隊個Program比我, 好明顯無試清楚
其實我好憎Import人地的Source入我個Eclipse
因為公司無行Central CMS
我一打開見到各種各種問題: Hardcode, 留低垃圾Dead Code, 一個Main Method行晒所有野
到我話有問題, 佢地改完好快又隊個Program去我Email
我 ... [/quote]


download and unpack???? 用 git 啦。 行 CI

java2 2019-4-20 10:50 AM

你個concept 有問題, code review 不等於 unit test 或 debug :smile_34:

[quote]原帖由 [i]gunbuster363[/i] 於 2019-4-19 11:46 AM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498029204&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]
最近要睇新個Junior
老細叫佢地寫Program
佢地寫完就隊個Program比我, 好明顯無試清楚
其實我好憎Import人地的Source入我個Eclipse
因為公司無行Central CMS
我一打開見到各種各種問題: Hardcode, 留低垃圾Dead Code, 一個Main Method行晒所有野
到我話有問題, 佢地改完好快又隊個Program去我Email
我 ... [/quote]

PacificIslander 2019-4-20 01:29 PM

[quote]原帖由 [i]java2[/i] 於 2019-4-20 10:50 AM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498070767&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]
你個concept 有問題, code review 不等於 unit test 或 debug :smile_34:

[/quote]

不過 review 而唔理 bug,又無乜意思。對著甘 junior,review 預左都係幫佢 debug

java2 2019-4-20 02:53 PM

review 當然會嘗試找出一些隱藏的bug 或后門, 未致於連基本自己都未試好的program 就攞去review.

例如 if password.equals(" PacificIslander") return true;  好明顯就要靠review 啦 :smile_30:

[quote]原帖由 [i]PacificIslander[/i] 於 2019-4-20 01:29 PM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498077824&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]


不過 review 而唔理 bug,又無乜意思。對著甘 junior,review 預左都係幫佢 debug [/quote]

PacificIslander 2019-4-20 04:40 PM

[quote]原帖由 [i]java2[/i] 於 2019-4-20 02:53 PM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498081130&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]
review 當然會嘗試找出一些隱藏的bug 或后門, 未致於連基本自己都未試好的program 就攞去review.

例如 if password.equals(" PacificIslander") return true;  好明顯就要靠review 啦 :smile_30:

... [/quote]

初哥根本冇得靠譜。


if ferry_station = "wanchai" return "發達"


:smile_o13:

Ultra919 2019-4-20 05:09 PM

有無coding guideline? 有無 Design doc? 無既話怪啲細前先檢討一下。

form5 2019-4-20 08:34 PM

[quote]原帖由 [i]Ultra919[/i] 於 2019-4-20 05:09 PM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498086975&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]
有無coding guideline? 有無 Design doc? 無既話怪啲細前先檢討一下。 [/quote]
Coding guideline 九成九冇
code review 乜鬼,交吾到貨,炒左佢啦,寕缺勿濫

我的無比膏 2019-4-20 10:41 PM

[quote]原帖由 [i]Ultra919[/i] 於 2019-4-20 05:09 PM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498086975&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]
有無coding guideline? 有無 Design doc? 無既話怪啲細前先檢討一下。 [/quote]


公司用單一lang , coding guideline 就非常有用,不過世上有冇公司只有一隻?

Ultra919 2019-4-21 08:37 AM

沒有coding guideline的code review的準則是什麼呢?Code review的目的又是什麼呢?

fx360bx 2019-4-21 09:53 PM

[quote]原帖由 [i]無盡外圍的湯圓[/i] 於 2019-4-19 11:52 AM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498029367&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]
Tell to them to Code Review each other before send them to you to review. [/quote]

未必解決到個問題。
除非三個 developers 裡面,有一個人有齊樓主所提及既 coding 經驗,不過照樓主意思,似乎果個人就係佢。
又或者,如果果三個 developers 可以互補不足,咁都可以閘到部分問題。
但係如果三個都好參次,有可能令成件事 worse,例如 developers A 同 B 覺得 hardcode 好過 constants,有可能會 peer code review 時 ban 左 C 用 constant。
[quote]原帖由 [i]無盡外圍的湯圓[/i] 於 2019-4-19 11:52 AM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498029367&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]
Then if you find something wrong, teach all of them in one shot because they all saw the code[/quote]
呢個就同意,其實樓主已經 point out 左有啲咩 common coding mistakes / 不合乎 convention 既 coding styles,由今日開始寫返個 guideline,唔難姐。

[[i] 本帖最後由 fx360bx 於 2019-4-21 10:00 PM 編輯 [/i]]

fx360bx 2019-4-21 09:56 PM

[quote]原帖由 [i]gunbuster363[/i] 於 2019-4-19 11:46 AM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498029204&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]
最近要睇新個Junior
老細叫佢地寫Program
佢地寫完就隊個Program比我, 好明顯無試清楚
其實我好憎Import人地的Source入我個Eclipse
因為公司無行Central CMS
我一打開見到各種各種問題: Hardcode, 留低垃圾Dead Code, 一個Main Method行晒所有野
到我話有問題, 佢地改完好快又隊個Program去我Email
我 ... [/quote]

SVN / git 今時今日 is a must,你話你次次都要 download & unzip & import,係你地機構冇用 source control,唔關啲細既事。
同埋你地 work on same code base,都唔用 source control,有機會 overwrite 左人地啲 changes 或者 revert 左 code changes 而自己都唔知,亦好難知道大家改左啲乜,要下下 diff,做啲野好文書,完全唔 IT。

如果寫 Java,睇 阿里巴巴Java開發手冊:
[url=https://github.com/alibaba/p3c]https://github.com/alibaba/p3c[/url]

有 PDF,有 IDE plugin。

[[i] 本帖最後由 fx360bx 於 2019-4-21 10:06 PM 編輯 [/i]]

sugar99hk 2019-4-23 11:27 AM

[quote]原帖由 [i]java2[/i] 於 2019-4-20 10:50 AM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498070767&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]
你個concept 有問題, code review 不等於 unit test 或 debug :smile_34:

[/quote]
冇unit test 可以直接彈返轉頭睇都唔睇先係正路

sugar99hk 2019-4-23 11:30 AM

[quote]原帖由 [i]fx360bx[/i] 於 2019-4-21 09:56 PM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498144592&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]


SVN / git 今時今日 is a must,你話你次次都要 download & unzip & import,係你地機構冇用 source control,唔關啲細既事。
同埋你地 work on same code base,都唔用 source control,有機會 overwrite 左人地啲 changes 或者 revert 左 code changes 而自己都唔知 ... [/quote]
全中,睇到個post 最嚇人既就係完全冇用source control ... 或者唔識用?

我的無比膏 2019-4-24 09:08 AM

[quote]原帖由 [i]sugar99hk[/i] 於 2019-4-23 11:30 AM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498215759&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]

全中,睇到個post 最嚇人既就係完全冇用source control ... 或者唔識用? [/quote]


我反而覺得唔係無用,做法可能係樓主review 完,先放上去sources control ,即係佢下屬無權改source control

sugar99hk 2019-4-24 10:38 AM

[quote]原帖由 [i]我的無比膏[/i] 於 2019-4-24 09:08 AM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498259008&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]



我反而覺得唔係無用,做法可能係樓主review 完,先放上去sources control ,即係佢下屬無權改source control [/quote]
你咪玩la
Download, unzip, copy
用開source control d terms 應該係
Checkout, patch, merge lor

stephen 2019-4-24 11:33 AM

Yes, peer review first provided that i......

我的無比膏 2019-4-24 11:36 AM

[quote]原帖由 [i]sugar99hk[/i] 於 2019-4-24 10:38 AM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498262958&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]

你咪玩la
Download, unzip, copy
用開source control d terms 應該係
Checkout, patch, merge lor [/quote]


唔係玩
我跟vendor 關係,dl, unzip, copy 去 my source control , 我唔俾vendor direct access

sugar99hk 2019-4-24 11:39 AM

[quote]原帖由 [i]我的無比膏[/i] 於 2019-4-24 11:36 AM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498265815&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]



唔係玩
我跟vendor 關係,dl, unzip, copy 去 my source control , 我唔俾vendor direct access [/quote]
咁會唔會應該分開兩個repository
你control 哂兩邊,vendor control 一邊
咁你就可以做merge le?
仲唔係玩?
:fst_002:

我的無比膏 2019-4-24 11:45 AM

[quote]原帖由 [i]sugar99hk[/i] 於 2019-4-24 11:39 AM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498265929&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]

咁會唔會應該分開兩個repository
你control 哂兩邊,vendor control 一邊
咁你就可以做merge le?
仲唔係玩?
:fst_002: [/quote]


各自一個,我唔會理佢用咩,我都唔會管佢個 svn,我每次都要dl and unzip 架

sugar99hk 2019-4-24 03:10 PM

[quote]原帖由 [i]我的無比膏[/i] 於 2019-4-24 11:45 AM 發表 [url=https://www.discuss.com.hk/redirect.php?goto=findpost&pid=498266289&ptid=28167108][img]https://www.discuss.com.hk/images/common/back.gif[/img][/url]



各自一個,我唔會理佢用咩,我都唔會管佢個 svn,我每次都要dl and unzip 架 [/quote]
我唔知你實際上有咩原因or做d 咩
不過code integration 可以做branch merge,可以做svn external link,可以做nuget 或者更多方法
但係我好肯定既係,dl and unzip 就一定有問題

But, ultimately, who cares, not my problem
:fst_002:

utbabya 2019-4-25 03:46 PM

Good luck with peer review, even bigger wiggle room.

kinchesskin 2019-5-7 04:04 PM

沒有SVN嗎
沒有app scanning tools嗎

csmwlee 2019-5-8 03:25 PM

Shouldnt code review be compulsory in dev cycle? Just like unit , integration, qa test...
U should have implemented an efficient mechanism to perform it as daily task
頁: [1]
查看完整版本: 邊個負責CODE REVIEW?