看到前任的代碼，我坐不住了

今天剛到公司報道，凳子還沒坐熱就git clone了一把，想着先學習學習。結果這一看沒關係，個人心登時涼了半截，氣得渾身發抖，全身冷汗，手腳冰涼，地獄空蕩蕩魔鬼在人間，這個項目還能不能好了？我到底要怎麼改大家才滿意 ？眼淚不爭氣的流了下來，後邊記不清了。好了，閒言少敘，書歸正傳，跟你們一塊兒討論下我遇到的幾個小問題

使人頭疼的命名

沒錯，這個經典難題。我打開前任代碼的時候，大駝峯、小駝峯、大寫、中劃線、下劃線·····。這裏不是說不能混用各類命名規則，是儘可能不要上一行是className，下一行就忽然class-name。 還有就是好好利用翻譯軟件，我這種英語渣離開百度翻譯估計得整抑鬱了。固然了，機翻也有技巧的，見過很多活動積分整成integral的了。再一個，你整個tempABC、obj123啥的我也就忍了，你上拼音縮寫就過度了，這玩意是真難猜。

註釋

前任對本身的代碼可能過於自信了，各類一把梭，配合不太講究的變量命名，時長看得我喘不過氣來。這裏仍是建議像我同樣的菜鳥，儘可能帶上註釋，大白話的那種，把看你代碼的人當傻子的那種。別怕行數多，打包的時候會去掉的。

函數不要太長

前任還有一個問題，就是不太注意方法的分割、提取。一不當心就能搞出接近屏幕高度的方法，尤爲是在vue裏，不少時候三大件都是寫在一塊兒的，跟蹤起來更是苦不堪言。這裏建議像我同樣的菜鳥多向大佬學習，這真沒別的路子。

if(state == 1){}

這類判斷在我看來也是值得優化的。首先就是==判斷（僅針對javascript哦），建議想盡辦法用===其次，1234這種狀態給我老老實實轉成字符串來表達啊，不是說1轉成'1'這樣，是const STATE = 1這種啊。還有就是if···else if···else if···else if···這種寫法仍是想辦法優化下，要不跟蹤問題得累死。

res.result.datas.list[0].....

喜訊喜訊，特大喜訊，ES新特性提供了可選鏈運算符廣大切圖仔使用，不限經驗，不限學歷哦。無論是培訓班剛出來菜鳥，仍是p100級的大佬，都可無償使用。配合babel效果更好哦。不想用新特性？太張揚？用&&也行啊，加個try catch也行啊。總之，確保你的代碼穩如哈士奇。

console.log過量

做爲一個前端，我都是先按f12在num run dev的，我比較怕看到控制檯一堆不清不楚的輸出，不禁得想定位到那行趕忙註釋掉。以致於我有個習慣，就是給log包上一層，加個952七、1024這樣的前綴便於在控制檯過濾。log雖好，不要貪杯哦。

---

其實這些習慣或者規範改起來其實很容易的，由於不是邏輯上的錯誤，不容易致使bug，但容易招罵。這些錯誤我也不是沒犯過，好在我知道本身菜，願意改過來。代碼這玩意，總要不斷覆盤的，最好是別人一看到你的代碼，就全身冒熱氣，手腳發燙，嘴裏叫着這個社會真的好了，天堂滿蕩蕩，天使在人間，咱們碼農怎麼敲都讓大家滿意，忍不住笑嘴角不由上揚，這個社會處處充斥着對碼農的讚賞，碼農如今真正的站起來了。

就到這裏把，以上有說的不對的地方，你們隨便噴，我沒關係的。剛來公司，感受跳進到了新坑，有的優化了。