提供一點不一樣的看法
: 我們是新創公司，我進去前已就有一個前端工程師，他從0建構了整個產品A
代表他能力不算差
而且產品A有很多他習慣的practice
: 我是產品B的前端，剛好我們產品線不急，被拉去支援他們 改版而且產品A有很多他習慣的practice
: 但在合作上就覺得跟他相處很不舒服
: 可能是把我當競爭對手吧？
: 喜歡用高姿態/批判的方式codereview,
有些人講話是比較機歪
但review comment只要不是「這邊寫得好爛」這種垃圾話
應該都還是有點價值
然後個人認知中的code review
最主要的目的，是要讓你的code能符合既有的style
細到變數命名，粗到目錄結構、pattern的運用
有沒有重造輪子，還是有既有的utility function可以用但你不知道？
很多工程師（包含小弟我XD
都會有自己的寫法最屌的錯覺
對於沒看過或不熟悉的做法總是看不順眼
code review就是在確保
一個repo在多人維護下也能有一定程度的一致性
而不是每次都要在review過程中討論出一個很猛的寫法或架構
: 就霹靂啪拉回了十句，順帶挑我程式毛病，我覺得更像是用公事來打壓別人
想像一下，你從0開始寫出了整個專案
一個新來的工程師，不了解既有的convention，上來就想照自己的方式寫
你會不會覺得奇怪？
脾氣不好，但又依賴他做事，就更代表他確實能力不錯
文件需求不清楚是小公司滿常見的，而且需求應該是PM負責吧怎麼會是工程師來寫需求？
: 我們做事不得不都要照的他的方式做事，但他又很自我中心，跟他配合心力大概4成是處理情緒問題4成才是程式問題文件需求不清楚是小公司滿常見的，而且需求應該是PM負責吧怎麼會是工程師來寫需求？
由他建立起的專案，按照他的方法做事，應該沒什麼問題
因為你也是短期支援而已，要是被你一通亂改改到他看不懂
後面要收拾的也不是你
: 攻擊性強的同事
: 自以為是的同事
: 他的性格滿符合上面相關搜尋找到的描述
: 不知道各位前輩是怎麼應對的
: 我現在是當練EQ，大概還要半年改版完忍忍
這段都是個人心情的部分，就不多做評論
有好的想法，也要看專案性質跟時程
不一定你在A專案的好做法，到B專案也適用
尤其前端又是生態系很亂的一個環境
光React管理CSS的做法，就有三種以上知名的solution
每過一兩年都會看到「原本那套過時了，現在應該要用這個啦」的說法
如果你想套新的作法，當然就是要跟身為原作者的他討論
很多情況都是一段你看起來很糞的code
但前人已經想過幾種可能的優化方法，只是受限於某些條件無法這樣做
或是大家判斷這邊再改動的可能性很低，乾脆就先放著能動就好
因為沒有實例，所以我也無法給你太具體的建議
不過如果想做refactor的話，最好還是要有條理的整理出
舊做法哪裡不好，新做法哪裡好
是改善了效能、改善可讀性、改善擴充性，還是什麼呢？
如果你真的對自己的做法很有信心
也可以直接做個prototype後PR就丟出來
把重要的人全部拉進去當reviewer
要是多數人認同你的做法，只有他在PR裡面亂噴
就算最後沒辦法merge，那也是幫助你在公司建立credit
他則是在敗自己的人品
那也許有一天專案出事，主管就會想到要找你去救
到時候自然是你想怎麼寫就怎麼寫
--