作为前端小白的我，试着用面向对象的方式写了一个拖拽，麻烦大神指正下哪里写得不好。

Question

Drag.prototype = {    mousedown: function(dom) {        var that = this;        dom.onmousedown=function(e) {            var ev=e || event;            that.disX=ev.clientX-dom.offsetLeft;            that.disY=ev.clientY-dom.offsetTop;            document.onmousemove=function(e){                var ev=e||event;                dom.style.left=ev.clientX-that.disX+'px';                dom.style.top=ev.clientY-that.disY+'px';            };            that.mouseup();        };    },    mouseup: function () {        document.onmouseup=function () {            document.onmousemove=null;            document.onmouseup=null;        }    },    init:function (dom){        this.mousedown(dom);    }};var dd = new Drag();var oDiv= document.getElementById('div1');dd.init(oDiv);

Answer 1

有点觉得呢，你的代码写的似乎不怎么整洁，像函数里的函数，或者闭包这种东西也不要乱用，起码也要学懂了，不然会有问题的