大佬，有个小建议 #4

mrzqii · 2018-09-18T04:10:00Z

今天在学习插件的封装，所以研究了你的一些插件封装的思路，从你的代码里面获益匪浅，在这例发现一个问题，就是每次调用cookie()的时候都会去执行：

function Cookie() {
        if (!(this instanceof Cookie)) {
            return new Cookie();
        }
}

意味着每次都会创建一个新的实例，这样做在设计和性能方面都不是很好，我建议最好只创建一个实例就可以了。可以做如下改装：

function Cookie() { }
function init() {
        if ( !this.instance ){         
            this.instance = new Cookie();
        }     
        return this.instance; 
}

var cookie = function(name,value,option) {
        var arg = arguments;
        if(arg.length === 0) return init().all();
}

不知道我的思路怎么样？

jaywcjlove · 2018-09-18T06:15:18Z

@mrzqii 开始有思考，有前途有进步

jaywcjlove · 2018-09-18T06:33:34Z

if(!(this instanceof Cookie)) {
  return new Cookie()
};
``

这个就是防止重复 `new` 的判断

mrzqii · 2018-09-18T07:59:18Z

if(!(this instanceof Cookie)) {
  return new Cookie()
};
``

这个就是防止重复 `new` 的判断

没有起到作用哦：

把这里打一个log

if(!(this instanceof Cookie)) {
  console.log(1)
  return new Cookie()
};

然后调用：

<body>
    <script src="./cookie.js"></script>
    <script>
        cookie()
        cookie()
        cookie()
    </script>
</body>

打印的log:
1
1
1

因为每次调用Cookie()的时候this都是指向window

jaywcjlove · 2018-09-18T08:12:44Z

@mrzqii 了解

jaywcjlove closed this as completed in 89226ca Sep 20, 2018

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

大佬，有个小建议 #4

大佬，有个小建议 #4

mrzqii commented Sep 18, 2018 •

edited by jaywcjlove

Loading

jaywcjlove commented Sep 18, 2018 •

edited

Loading

jaywcjlove commented Sep 18, 2018 •

edited

Loading

mrzqii commented Sep 18, 2018

jaywcjlove commented Sep 18, 2018

大佬，有个小建议 #4

大佬，有个小建议 #4

Comments

mrzqii commented Sep 18, 2018 • edited by jaywcjlove Loading

jaywcjlove commented Sep 18, 2018 • edited Loading

jaywcjlove commented Sep 18, 2018 • edited Loading

mrzqii commented Sep 18, 2018

jaywcjlove commented Sep 18, 2018

mrzqii commented Sep 18, 2018 •

edited by jaywcjlove

Loading

jaywcjlove commented Sep 18, 2018 •

edited

Loading

jaywcjlove commented Sep 18, 2018 •

edited

Loading