关于String.prototype

做项目的时候发现一个问题，我们项目里头对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程序的时候

1. 无论什么情况，不要使用 new String 这样的代码（跟我一样无聊测试特性的除外）
2. 无论什么情况，判断是否相等请使用 === 而不要使用 ==

以上