字符串拼接引发的BUG

时间: 2017-11-20阅读: 304标签: js知识

译者按: bug虽小,却是个磨人的小妖精!

原文: Fixing a bug: when concatenated strings turn into numbers in JavaScript 
译者: Fundebug

为了保证可读性,本文采用意译而非直译。另外,本文版权归原作者所有,翻译仅用于学习。

这是一篇很简短的博客,记录了我今天早上花了一个小时才解掉的一个bug。


准备工作

在已有的网站页面,我们已经有一段JavaScript代码用于构建字符串并把它插入到DOM中,如下所示:

function GetTemplate(url, html)
{
// 省掉部分细节代码
// ...
var template = '<div><a href="'
+ url
+ '" target="_blank"><strong>Details: </strong><span>'
+ html
+ '</span></a></div>';
return template;
}

请忽略这段代码的粗糙。接下来,我们的需求很简单:如果summary存在,那么在<strong>标签前面添加一个额外的<span>标签将该值显示出来。是不是很简单?我们来试一试。


首次尝试

我快速实现了如下代码:

function GetTemplate(url, html, summary) {
// other details removed
var template = '<div><a href="'
+ url
+ '" target="_blank">';
if(summary) {
template += '<span>'
+ summary
+ '</span>';
}
template +=
+'<strong>Details: </strong><span>'
+ html
+ '</span></a></div>';
return template;
}

看上去一切OK,没有问题。F5刷新页面,看起来不大对:

The summary NaNThis is the inner message


你知道哪里出问题了吗?

由上面的代码生成的HTML长这样:

<div class="something"><a href="https://thewebsite.com" target="blank">
<span class="summary">The summary</span>NaNThis is the inner message</span></a>
</div>

发现问题了吗?如果没发现,我们接着往下看。

你的线上代码真的没有BUG吗?欢迎免费使用Fundebug!我们可以帮助您第一时间发现BUG!


字符串拼接 vs 加法

仔细查看生成的HTML代码,你会发现NaN出现在</span>标记的后面,然而<strong>标签不见了。NaN是一个很好的线索,表明这里有类型转换发生,并且是转换为Number类型,但是我当时一直没有找到发生转换的原因!

接下来,我们先温习一下JavaScript基础知识。在JavaScript中,根据+左右两边变量的类型的不同,+符号可以用于数字相加或则字符串拼接。

console.log('value:' + 3); // 'value:3'
console.log(3 + 1); // 4
console.log('value:' + 3 + '+' + 1); // 'value:3+1'
console.log('value:' + 3 + 1); // 'value:31'
console.log('value:' + (3 + 1)); // 'value:4'
console.log(3 + ' is the value'); // '3 is the value'

在上面的这些例子中,如果+的任何一边是字符串,那么另一边一定会转换为字符串。否则,将看做是数字相加。

因此,NaN预示着一定是字符串被误用为数字了。但我并没有使用parseInt()函数做类型转换,所以逻辑上说不通啊!


问题原因

最终,我逐步缩小出错区域,发现是如下代码出错:

template +=
+'<strong>Details: </strong><span>'
+ html
+ '</span></a></div>';

如果你还是没看出来,那么我们换个写法:

template += +'<strong>Details: </strong><span>' + html + '</span></a></div>';

我用了string += +string这样的写法,也就是说:由于写代码的时候拷贝黏贴,不小心整了一个多余的+号?所以,相当于使用了一元运算+。根据一元运算符(+)的官方解释:+c会显示地将c转换为Number类型。

这就是我的代码出现bug的根源:一元运算符+号尝试将<strong>Details: </strong><span>转换为数字,但是失败了返回NaN。然后NaN又转换为字符串拼接起来。当我把这个额外的+删掉后,代码就正确运行了。


额外建议

另外值得一提的是,我使用了gulp-uglify来压缩我的JavaScript代码。在构建过程中,一元运算(+'<strong>Details: </strong><span>')已经在压缩后的代码中存储为NaN了。Gulp已经识别出代码错误。

从这一次Debug的经历吸取了一个教训:不要马马虎虎的拷贝黏贴代码!而且我立即想到如果有一个小的gulp插件可以识别并提醒压缩代码中有莫名其妙的NaN的话,也可以适当避免问题。


补充

> parseInt('<strong>Details: </strong><span>')
NaN
> +'<strong>Details: </strong><span>'
NaN

版权声明:
转载时请注明作者Fundebug以及本文地址:
https://blog.fundebug.com/2017/11/20/bug-fix-concatenated-string-trun-into-numbers/

js中offset、scroll、event、client的区别和使用

用一句话概述:offset用于获取元素的实际显示尺寸 , scroll用于获取滚动后全部尺寸 , client用于获取不包括滚动条的实际显示尺寸,event用于获取鼠标的坐标位置。下面就详细介绍它们之间的使用和区别

js判断日期是否为今天

需求如下:后端返回字符串数据,需要前端判断该日期是否为今天。比如返回日期格式为:2018-08-14,那么需要如何来实现呢,这篇文章整理实现的几种方式供大家参考。

javascript中的伪线程,使用setTimeout模拟一个多线程

浏览器的内核是多线程的,一个浏览器一般至少实现三个常驻线程:javascript引擎线程,GUI渲染线程,浏览器事件触发线程。当我们要循环过百万级的数据甚至亿的时候怎么办?那就用setTimeout模拟一个多线程。

web worker是什么?理解并使用web worker

Web Worker 是为了解决 JavaScript 在浏览器环境中没有多线程的问题。正常形况下,浏览器执行某段程序的时候会阻塞直到运行结束后在恢复到正常状态,而HTML5的Web Worker就是为了解决这个问题,提升程序的执行效率。

JS方法整理_js常用函数大全

都是日常工作中使用的一些js方法,整理出来以便大家学习使用。主要包括:Js获取页面地址参数 、千分位 、判断是否数字 、图片按比例压缩、截取指定字节数的字符串、判断是否微信 、获取时间格式的几个举例 、获取字符串字节长度 、对象克隆、深拷贝 ...

Performance_js中计算网站性能监控利器

Performance提供的方法可以灵活使用,获取到页面加载等标记的耗时情况。Performance.timing属性对象提供了浏览器从打开网页到加载完成之间各个节点的耗时数据,包括重定向开始、DNS查询、浏览器响应数据、DOM解析等相关节点

js检测页面上一个元素是否已经滚动到了屏幕的可视区域内

只要页面加载了,其中在页面中出现的li就向控制台输出第几个发送请求;在本次加载的页面中,再将滚动条滚回前边的li,不再向控制台输出东西,也就是说已经显示过的li,不再向控制台输出东西。

浅谈js自记忆函数

最近阅读《JavaScript忍者秘籍》看到了一种有趣的函数:自记忆函数。记忆化(memoization)是一种构建函数的处理过程,能够记住上次计算结果,当函数计算得到结果时,就将该结果按照参数存储起来。

柯里化与反柯里化

柯里化,是一个逐步接收参数的过程。在接下来的剖析中,你会深刻体会到这一点。 反柯里化,是一个泛型化的过程。它使得被反柯里化的函数,可以接收更多参数。目的是创建一个更普适性的函数,可以被不同的对象使用。

js中减少使用不必要的if-else或switch_利用数组/对象代替if-else,switch

无论使用if-else,还是switch。当条件多的时候代码显得非常冗长,而且每次添加条件时需要修改主流程的代码,这样就破坏了类的开闭原则。为解决日后的维护可能存在问题,我们可以采用另一种比较优雅的实现方式来替换if-else,switch吗?