某一線前端小組長的 Code Review 技術(shù)總結(jié)
作者:悄悄哥 原文:https://juejin.cn/post/7052570403029385253
?? 前言
大家平時網(wǎng)上搜索能搜出一堆Code Review 實踐、Code Review 流程、如何如何 Code Review 等等。好像實際的 review 記錄并不多,恰好本人有幸擔任前端小組長一職,平時摸魚就去偷偷看看大家又寫了什么優(yōu)秀代碼。所以今天就由小弟我,給大家分享一下,小廠一線前端平時寫的代碼??
作者能力有限,里面的評論不一定十分正確,僅做記錄,拋磚引玉,以供學(xué)習(xí)參考~
項目背景:
react 16.8+ antd@4
?? Talk is cheap. Show me the code!
location.replace 和 location.href的使用區(qū)別

正則判斷沒有對用戶可能輸入的特殊符號進行轉(zhuǎn)義

解析:該組件本意是想實現(xiàn)對一串字符中的關(guān)鍵字進行高亮展示;而正則表達式中有些字符具有特殊的含義,如果在匹配中要用到它本來的含義,需要進行轉(zhuǎn)義(在其前面加一個\)。如:
* . ? + $ ^ [ ] ( ) { } | \ /
Table行點擊事件的處理
這里示例代碼想實現(xiàn)點擊表格行跳轉(zhuǎn)頁面效果,但是會導(dǎo)致一些鼠標操作被覆蓋

在一個較大的組件內(nèi),需要注意輸入組件導(dǎo)致的重渲染問題

避免使用反直覺的編碼

做好路由字符串拼接的缺省處理

沒有還原本地測試代碼

解析:一些本地運行時的mock代碼,發(fā)到線上時忘記還原代碼
維護好項目內(nèi)的公共組件

?? 寫在最后
本篇文章整體內(nèi)容稍微零散,對于沒有基礎(chǔ)的同學(xué)可能不太友好,不過如果大家看完覺得有所收獲,可以留言點贊!
最后
如果你覺得這篇內(nèi)容對你挺有啟發(fā),我想邀請你幫我個小忙:
點個「喜歡」或「在看」,讓更多的人也能看到這篇內(nèi)容
我組建了個氛圍非常好的前端群,里面有很多前端小伙伴,歡迎加我微信「sherlocked_93」拉你加群,一起交流和學(xué)習(xí)
關(guān)注公眾號「前端下午茶」,持續(xù)為你推送精選好文,也可以加我為好友,隨時聊騷。

評論
圖片
表情
