这些被同事喷的JS代码风格你写过多少？

NO：对于求值获取的变量，没有兜底。

const MIN_NAME_LENGTH = 8;  
let lastName = fullName[1];  
if(lastName.length > MIN_NAME_LENGTH) { // 这样你就给你的代码成功的埋了一个坑，你有考虑过如果fullName = ['jackie']这样的情况吗？这样程序一跑起来就爆炸。要不你试试。  
    ....  
} 

YES：对于求值变量，做好兜底。

const MIN_NAME_LENGTH = 8;  
let lastName = fullName[1] || ''; // 做好兜底，fullName[1]中取不到的时候，不至于赋值个undefined,至少还有个空字符，从根本上讲，lastName的变量类型还是String，String原型链上的特性都能使用，不会报错。不会变成undefined。 
 if(lastName.length > MIN_NAME_LENGTH) {  
    ....  
}  
其实在项目中有很多求值变量，对于每个求值变量都需要做好兜底。  
let propertyValue = Object.attr || 0; // 因为Object.attr有可能为空，所以需要兜底。  
但是，赋值变量就不需要兜底了。  
let a = 2; // 因为有底了，所以不要兜着。  
let myName = 'Tiny'; // 因为有底了，所以不要兜着。 

二、函数相关

（1）函数命名

NO：从命名无法知道返回值类型

function showFriendsList() {....} // 现在问，你知道这个返回的是一个数组，还是一个对象，还是true or false。你能答的上来否？你能答得上来我请你吃松鹤楼的满汉全席还请你不要当真。 

Yes：对于返回true or false的函数，最好以should/is/can/has开头

function shouldShowFriendsList() {...}  
function isEmpty() {...}  
function canCreateDocuments() {...}  
function hasLicense() {...} 

（2）功能函数最好为纯函数

NO: 不要让功能函数的输出变化无常。

function plusAbc(a, b, c) {  // 这个函数的输出将变化无常，因为api返回的值一旦改变，同样输入函数的a，b,c的值，但函数返回的结果却不一定相同。  
        var c = fetch('../api');  
        return a+b+c;  
} 

（编辑：宿州站长网）

【声明】本站内容均来自网络，其相关言论仅代表作者个人观点，不代表本站立场。若无意侵犯到您的权利，请及时与联系站长删除相关内容!