项目中多余的js写法之join和||连用

先看一段代码,框里面和框外面代码有区别吗?
技术分享
也就是说直接给bookHtmlStr和checkHtmlStr赋值和经过一些无意义计算再赋值有什么区别,或者说有什么优劣吗?
首先我们看以上代码执行的结果截图,再分析原因。
技术分享
也就是说,以上代码执行的最终结果为:

var bookHtmlStr = "<tr><td colspan=‘10‘>暂无数据!</td></tr>";
var checkHtmlStr = "<tr><td colspan=‘12‘>暂无数据!</td></tr>";

为什么最终结果是这样的呢?我们先看一下js中join的用法。例如:

var arr = new Array("1","2","3");
document.write(arr.join("."));
打印的结果为: 1.2.3 

在js中||和&&的用法如下:
a() && b() :如果执行a()后返回true,则执行b()并返回b的值;如果执行a()后返回false,则整个表达式返回a()的值,b()不执行;
a() || b() :如果执行a()后返回true,则整个表达式返回a()的值,b()不执行;如果执行a()后返回false,则执行b()并返回b()的值;

在js中,null、undefined、字符串空“”、数字0 转化boolean值时,都是为false。


知道以上的知识后,再看一下最开始的代码:

var bookHtmlArr = [];
var checkHtmlArr = [];
var bookHtmlStr = bookHtmlArr.join("")||"<tr><td colspan=‘10‘>暂无数据!</td></tr>";
var checkHtmlStr = checkHtmlArr.join("")||"<tr><td colspan=‘12‘>暂无数据!</td></tr>";

这样的代码导致bookHtmlArr.join(“”)一直都是false,最后的结果一直都会返回”暂无数据!“。所以估计这段代码应该是当初写的时候从其它地方copy过来改改的。以上代码的原意是:如果bookHtmlArr为空,就把“暂无数据”赋值bookHtmlStr ,如果bookHtmlArr有值,就把bookHtmlArr赋值给bookHtmlStr 。


反思:
知其人还有知其所以然。不管以后看别人代码还是自己写代码,都尽量做到不留多余代码在项目中。

郑重声明:本站内容如果来自互联网及其他传播媒体,其版权均属原媒体及文章作者所有。转载目的在于传递更多信息及用于网络分享,并不代表本站赞同其观点和对其真实性负责,也不构成任何其他建议。