做项目的时候发现一个问题，我们项目里头对js的String的原型对象进行了扩展
比如
String.prototype.formatStringRN = function ()
{
var cd = this;
if (cd != "")
{
cd = cd.replaceAll('\r\n', '<br//>').replaceAll('\n', '<br//>').replaceAll('\r', '<br//>');
}
return cd;
};
这是原来项目里的代码，作用是将字符串中的 "\r\n" 全部替换成 "<br//>"
这里有一个平时都没仔细想过的问题
就是方法内部对this进行了判断，如果这玩意儿等于空串的时候直接返回 this
但是js内部的处理比较困扰，这里的this，其实已经不是一个简单的 "str" 了
看看我刚在谷歌的控制台里做写的一段代码就明白了
这里的this，其实已经变成了一个类似new String 得到的对象，至于这个对象是怎么得到的，我就不瞎说了（应该说这个东西比较复杂，如果真展开了，又是另外一篇文了），反正需要明白的是，这个对象其实跟普通的直接写的字面量是不一样的，了解javascript中 == 与 === 区别的应该明白
图片，实验结果
然后，这样有什么问题呢？
其实如果你拿它当字符串用，也没什么大问题，当然区别在于
"aa" === "aa" => true
new String("aa") === "aa" => false
但是
如果你这样做
if(""){
//do
}
do是不会被执行到的
if(new String("")){
//do
}
这个do是会被执行到的
如下
new String("")
所以， 不要 直接返回this
最后说一下怎么解决这个问题
其实我本来想的是返回一个类似 this.value 这样的东西，但是貌似没有这个属性或者类似的方法
所以这个方法就变成了
String.prototype.formatStringRN = function ()
{
var cd = this;
if (cd != "")
{
cd = cd.replaceAll('\r\n', '<br//>').replaceAll('\n', '<br//>').replaceAll('\r', '<br//>');
}
return String(cd);
};
注意返回值，用"String"处理了一下，这样得到的就是一个普通的字面量值了
当然，其实也可以在一进来的时候直接处理 this
变成这个样子
String.prototype.formatStringRN = function ()
{
var cd = String( this);
if (cd != "")
{
cd = cd.replaceAll('\r\n', '<br//>').replaceAll('\n', '<br//>').replaceAll('\r', '<br//>');
}
return (cd);
};
最后说点题外话
写js程序的时候
- 无论什么情况，不要使用 new String 这样的代码（跟我一样无聊测试特性的除外）
- 无论什么情况，判断是否相等请使用 === 而不要使用 ==
以上
