Velikost videa: 1280 X 720853 X 480640 X 360
Zobrazit ovladače přehrávání
Automatické přehrávání
Přehrát
Code Review建議4:16 選擇適合的git hosting,減少踩雷機會5:09 ticket作為branch name,方便追蹤需求與bug6:25 善用template,統一格式有助於加速發PR流程7:38 PR善用GIF,讓reviewer更直觀的了解到這次的改動範圍8:38 善用自動化工具,幫忙程式統一風格或是找出語法錯誤
Nice post,請問有用Python自動化工具的建議嗎?謝謝
@@kyc1109tw 我只用過JS的查了一下Python formatting可以用black。linter 可以用flake8,也可以用pylint,但比較多人推flake8。可以用關鍵字查,找到自己喜歡的工具 例如: python code style check、python formatting tool或是也可以用中文查
最近終於有喘口氣的空閒不報到不行感謝這次也精闢的技術分享外印象中之前有看到低潮文順便加油打氣良性的互動很棒但也不是必要真正支持的人也不會因為留言沒被按讚或回覆就退訂理念不同的人就更不用浪費腦容量了立場上雖然看起來只是無責任的雞湯文但個人覺得在公與私上大家都會遇到相似的問題自己也在努力實踐中希望尼克能肆無忌憚忠於自我的分享沒必要委屈自己回應他人期待、自己開心踏實最重要
Code review 最大的困難就是要放下尊嚴, 接受別人的批評! 因為大多數工程師都對自己寫的程式很自信的.
=.=~幫你code review的通常是你主管不然就是同個專案中比較資深的工程師幫忙看...會覺得要放下尊嚴根本是心態有問題...別人在幫你code review時同時也是在教你=3=~
通常一個月後你就會對著自己的code嘔吐了,這現象在繪圖跟音樂領域都很常見。往往你的自信只是來自於熟悉而不是你的東西真有多好。
每周都在期待 Nic 的影片
感謝 nic 之前來BESG 分享 code review直播效果果然比較讚哈哈
絕對是特別版的
最近參加程式訓練班準備開始分組做專題,感覺這種code review的互動能夠讓組員一起討論成長,收穫良多
請問一下,目前學習的這幾天感受如何
@@user-bg6ym9ks7y 這幾天才剛分完組,討論完題目的架構,身為組長,現在就先分配任務給組員,還在思考下周第一次討論,code review要怎麼實踐才可以比較有效率
新手中的中年人,買DISK 自學但對工程師技能.你講的深但精,不癈話,全面,----訂閱,贊
謝謝提供好的建議~~
客氣惹
有些公司一开始会给一个Style的规范,这样大家写的代码风格不会有太大差异,也就不会在正方面有什么争执而浪费时间 btw。 gif的推荐超实用
應該是說,規範也是慢慢建立起來的只是看說你是一進去就有規範,還是你是要當創建規範的人
感谢 nic 分享,好巧,正好我下周也要进行 code review 相关的分享。
又可以偷师学艺了。🤣
很同意code review 那些好處,每個人的盲點真的不是自己就可以輕易克服。現在回想起來,我以前的工作不是一人打天下,就是同事不願code review(當時我的title 不是senior/team lead,我沒權QQ)
我覺得一人打天下很孤獨 XD所以我都喜歡去找有團隊的工作
同事不願意 code review 遇到我的話就算沒實權我也會硬做其實很多次都是因為自己對這些的堅持跟努力讓其他人看到,慢慢在公司累積 credit 之後反而權力就自己過來找我了 XD有時候我覺得要把自己先放在預期所想的那個位置上,做事自然會有那個模樣 😉
我也有嘗試,但我感覺我上司(Senior)跟boss(PM)也沒太在乎。每次我merge master/tag新version 都經常看見同事做一些之前已經做過的功能/重犯之前已指正的錯誤,我真的受不了,結果通常都是我再處理然後delay😂現在的公司雖然只有上司在review,可是已經好很多。加上疫情WFH不review 我覺得真的很難做事。在個人層面,我有機會學到新知識,也可能找到自己盲點。而且有些changes需要根據一些網站/official doc去說明怎樣比較好,所以很難不review 時看跟理解,同時也能sync knowledge(所以我有偷偷看同事的PR lol)所以現在反而覺得不做code review,在上production 時真的很怕怕。
目前就職的公司,入職正好撞上公司打算重整規則的時期只能說專案真的很需要這些團隊文化努力維護QQ
非常棒的内容!支持!
感謝 Nic 大大分享
超讚分享!!
學習了 感謝 Nic
不客氣
優質影片 感謝分享 不知道Nic大大有沒有分享過自動化測試相關的主題 我現在工作的公司沒有 自己也沒經驗如何寫test。比起code review 我覺自動測試更重要 現在遇到的瓶頸是 每次一開發新功能或維護 就要花大半時間在測試、溝通是以前的bug還是這次改動的問題,我就在想難道花點時間搞定測試的自動化 就算不能一次到位也能慢慢改進開發流程也是好事,只是要說服主管要拿出一套完整的可行作法不能只是提出現在的問題,希望NIc大開示!
感謝分享
需求讨论,出结果以后才开始编码,编码按照公司规定的风格来编码,单测和集成测都没问题以后才开始code review,通过后在合并到master。大部分团队应该是这样的吧 感觉除了耗时没啥缺点 还有就是对于员工的要求。视频中添加修改模板这个流程我觉得很好 !!!
那nic 如何看待 pair programming 以及 mobbing programming 呢?我覺得它本質上會比code review更提前地去發現問題的所在,也可以降低團隊花時間了解別人寫code的痛點~
有些工程師把code review當成權力的展現和炫技的地方,不依照他的方式改就不approve, 甚至在comments上留下污辱性的言論,而老闆又視而不見,請問如何解決?
讚讚讚啦
讚!
謝謝分享產生GIF動圖軟體
gif錄製教學讚唷友立gif編輯器又是免費
單打獨鬥10年以上的人路過...
I think I know that feel bro (請自行腦補meme圖)
我也笑了,躲在別人底下4年,單打4年的路過
請問ticket 5:10 指的是常見的feature嗎?剛剛稍微古狗一下感覺是類似的風格?
讚
謝謝
不客氣 😏
頭香
也太準時!
@@niclin 提示剛好跳出乃~
請問一下 review 過程都是透過系統留 comment 溝通,還是會加入及時對談討論?
comment 優先如果比較長的會直接討論討論完要把重點跟結果寫回 comment不然日子久了大家習慣口頭講不紀錄回到 githosting 翻不出東西對日後維護是一種傷害
Nic 你好,想詢問是否可以分享 code review template 內總共有哪些內容嗎?
我自己在用的如下## Issue Linkxxxxx/(在這填上票號)## Changes(敘述這張票做了什麼)## Screenshots(功能畫面截圖,不需要可刪除)## Notes(填寫注意事項,或者 TODO 內容,不需要可刪除)## Risks(是否需要執行 task 操作?)## CodeReview邏輯正確,避免 concurrency 引起的問題優先處理,再者 Readable 為次要注重其他規範請參照- [基礎 Convention](xxxxx)- [金錢相關 RaceCondition](xxxxx)- [狀態管理 State Machine](xxxxx)
@@niclin 非常感謝分享🙏
我比較喜歡看直播
好 這幾天來開
收下寶物了
W
Code Review建議
4:16 選擇適合的git hosting,減少踩雷機會
5:09 ticket作為branch name,方便追蹤需求與bug
6:25 善用template,統一格式有助於加速發PR流程
7:38 PR善用GIF,讓reviewer更直觀的了解到這次的改動範圍
8:38 善用自動化工具,幫忙程式統一風格或是找出語法錯誤
Nice post,請問有用Python自動化工具的建議嗎?謝謝
@@kyc1109tw 我只用過JS的
查了一下Python formatting可以用black。
linter 可以用flake8,也可以用pylint,但比較多人推flake8。
可以用關鍵字查,找到自己喜歡的工具 例如: python code style check、python formatting tool
或是也可以用中文查
最近終於有喘口氣的空閒不報到不行
感謝這次也精闢的技術分享外
印象中之前有看到低潮文順便加油打氣
良性的互動很棒但也不是必要
真正支持的人也不會因為留言沒被按讚或回覆就退訂
理念不同的人就更不用浪費腦容量了
立場上雖然看起來只是無責任的雞湯文
但個人覺得在公與私上大家都會遇到相似的問題
自己也在努力實踐中
希望尼克能肆無忌憚忠於自我的分享
沒必要委屈自己回應他人期待、自己開心踏實最重要
Code review 最大的困難就是要放下尊嚴, 接受別人的批評! 因為大多數工程師都對自己寫的程式很自信的.
=.=~幫你code review的通常是你主管不然就是同個專案中比較資深的工程師幫忙看...
會覺得要放下尊嚴根本是心態有問題...別人在幫你code review時同時也是在教你=3=~
通常一個月後你就會對著自己的code嘔吐了,這現象在繪圖跟音樂領域都很常見。往往你的自信只是來自於熟悉而不是你的東西真有多好。
每周都在期待 Nic 的影片
感謝 nic 之前來BESG 分享 code review
直播效果果然比較讚哈哈
絕對是特別版的
最近參加程式訓練班準備開始分組做專題,感覺這種code review的互動能夠讓組員一起討論成長,收穫良多
請問一下,目前學習的這幾天感受如何
@@user-bg6ym9ks7y 這幾天才剛分完組,討論完題目的架構,身為組長,現在就先分配任務給組員,還在思考下周第一次討論,code review要怎麼實踐才可以比較有效率
新手中的中年人,買DISK 自學但對工程師技能.你講的深但精,不癈話,全面,----訂閱,贊
謝謝提供好的建議~~
客氣惹
有些公司一开始会给一个Style的规范,这样大家写的代码风格不会有太大差异,也就不会在正方面有什么争执而浪费时间 btw。 gif的推荐超实用
應該是說,規範也是慢慢建立起來的
只是看說你是一進去就有規範,還是你是要當創建規範的人
感谢 nic 分享,好巧,正好我下周也要进行 code review 相关的分享。
又可以偷师学艺了。🤣
很同意code review 那些好處,每個人的盲點真的不是自己就可以輕易克服。
現在回想起來,我以前的工作不是一人打天下,就是同事不願code review(當時我的title 不是senior/team lead,我沒權QQ)
我覺得一人打天下很孤獨 XD
所以我都喜歡去找有團隊的工作
同事不願意 code review 遇到我的話
就算沒實權我也會硬做
其實很多次都是因為自己對這些的堅持跟努力
讓其他人看到,慢慢在公司累積 credit 之後
反而權力就自己過來找我了 XD
有時候我覺得要把自己先放在預期所想的那個位置上,做事自然會有那個模樣 😉
我也有嘗試,但我感覺我上司(Senior)跟boss(PM)也沒太在乎。每次我merge master/tag新version 都經常看見同事做一些之前已經做過的功能/重犯之前已指正的錯誤,我真的受不了,結果通常都是我再處理然後delay😂
現在的公司雖然只有上司在review,可是已經好很多。加上疫情WFH不review 我覺得真的很難做事。
在個人層面,我有機會學到新知識,也可能找到自己盲點。而且有些changes需要根據一些網站/official doc去說明怎樣比較好,所以很難不review 時看跟理解,同時也能sync knowledge(所以我有偷偷看同事的PR lol)
所以現在反而覺得不做code review,在上production 時真的很怕怕。
目前就職的公司,入職正好撞上公司打算重整規則的時期
只能說專案真的很需要這些團隊文化努力維護QQ
非常棒的内容!支持!
感謝 Nic 大大分享
超讚分享!!
學習了 感謝 Nic
不客氣
優質影片 感謝分享 不知道Nic大大有沒有分享過自動化測試相關的主題 我現在工作的公司沒有 自己也沒經驗如何寫test。比起code review 我覺自動測試更重要 現在遇到的瓶頸是 每次一開發新功能或維護 就要花大半時間在測試、溝通是以前的bug還是這次改動的問題,我就在想難道花點時間搞定測試的自動化 就算不能一次到位也能慢慢改進開發流程也是好事,只是要說服主管要拿出一套完整的可行作法不能只是提出現在的問題,希望NIc大開示!
感謝分享
需求讨论,出结果以后才开始编码,编码按照公司规定的风格来编码,单测和集成测都没问题以后才开始code review,通过后在合并到master。大部分团队应该是这样的吧 感觉除了耗时没啥缺点 还有就是对于员工的要求。
视频中添加修改模板这个流程我觉得很好 !!!
那nic 如何看待 pair programming 以及 mobbing programming 呢?
我覺得它本質上會比code review更提前地去發現問題的所在,也可以降低團隊花時間了解別人寫code的痛點~
有些工程師把code review當成權力的展現和炫技的地方,不依照他的方式改就不approve, 甚至在comments上留下污辱性的言論,而老闆又視而不見,請問如何解決?
讚讚讚啦
讚!
謝謝分享產生GIF動圖軟體
gif錄製教學讚唷
友立gif編輯器又是免費
單打獨鬥10年以上的人路過...
I think I know that feel bro (請自行腦補meme圖)
我也笑了,躲在別人底下4年,單打4年的路過
請問ticket 5:10 指的是常見的feature嗎?剛剛稍微古狗一下感覺是類似的風格?
讚
謝謝
不客氣 😏
頭香
也太準時!
@@niclin 提示剛好跳出乃~
請問一下 review 過程都是透過系統留 comment 溝通,還是會加入及時對談討論?
comment 優先
如果比較長的會直接討論
討論完要把重點跟結果寫回 comment
不然日子久了大家習慣口頭講不紀錄
回到 githosting 翻不出東西
對日後維護是一種傷害
Nic 你好,想詢問是否可以分享 code review template 內總共有哪些內容嗎?
我自己在用的如下
## Issue Link
xxxxx/(在這填上票號)
## Changes
(敘述這張票做了什麼)
## Screenshots
(功能畫面截圖,不需要可刪除)
## Notes
(填寫注意事項,或者 TODO 內容,不需要可刪除)
## Risks
(是否需要執行 task 操作?)
## CodeReview
邏輯正確,避免 concurrency 引起的問題優先處理,再者 Readable 為次要注重
其他規範請參照
- [基礎 Convention](xxxxx)
- [金錢相關 RaceCondition](xxxxx)
- [狀態管理 State Machine](xxxxx)
@@niclin 非常感謝分享🙏
我比較喜歡看直播
好 這幾天來開
收下寶物了
W