0

我知道对于 javascript 的编写方式有很多不同的看法，但我想知道我的编写方式是否合适。我不想进入工作岗位编写没人理解的代码。

这段代码的基本前提是测试对象的自定义事件处理程序，想知道是否有任何明显的“你不应该这样做”类型的东西

    function EventClass() {
        var self = this;
        var events = {};
        var i = 0;
        self.fire = function(evt, args) {
            for (x in events[evt])
                events[evt][x].call(this, args);
        }

        self.on = function(evt, fn) {
            if (events[evt] == null) {
                events[evt] = []
            }
            events[evt].push(fn);
        }
    };

    function Human(x, y) {
        var self = this;
        self.__proto__ = new EventClass();
        var xCoord = 0;
        var yCoord = 0;

        self.events = {
            "MOVEMENT" : "movement"
        };

        self.init = function(x,y) {
            xCoord = x;
            yCoord = y;
        }

        self.draw = function(context) {
            context.beginPath();
            context.arc(xCoord,yCoord,10,0,Math.PI*2,true);
            context.closePath();
            context.fill();
        }

        self.moveLeft = function() {
            xCoord -= 5;
            self.fire(self.events.MOVEMENT, xCoord);
        }
        self.init(x,y);
    };

    function Player(x, y) {
        var self = this;
        self.__proto__ = new Human();
        self.init(x,y);
    };

    function Canvas(c) {
        var self = this;
        var canvas;
        var context;
        var objects = [];

        self.init = function(c) {
            canvas = c;
            context = canvas.getContext("2d");
        };

        this.redraw = function() {
            context.clearRect(0,0,300,300);
            for (x in objects) {
                objects[x].draw(context);
            }
        }

        this.addObject = function(obj) {
            objects.push(obj);
            obj.on(obj.events.MOVEMENT, function(coord) {
                console.log(coord);
                self.redraw();
            });
        };
        self.init(c);
    }


    var canvas = new Canvas(document.getElementById("canvas"));
    var human0 = new Human(75,75);
    canvas.addObject(human0);
    var human1 = new Human(100,100);
    canvas.addObject(human1);
    var player = new Player(200,200);
    canvas.addObject(player);

    canvas.redraw();
4

1 回答 1

0

几件小事——有点像。

第一个很小。

this与和self事物保持一致。我在这里考虑的示例在您的Canvas构造函数中。

事情的真相是，您的特定实现内部没有任何东西需要self. 除非您打算将您的方法提供给其他对象或回调，self否则不需要，直到您开始编写存在于方法内部的函数。即便如此，所有self将有助于访问该特定实例的公共属性/方法，并且不会帮助您访问events或i其他任何内容。

并不是说这样做不好——它只是在你做对象组合的事情时更有用，而不是对象构造的事情（然后经常变得非常必要）。

第二件事是一个更大的问题。 __proto__如果您谈论的是 FireFox/Chrome，可能会得到相当好的支持，但是如果您需要支持广泛的浏览器，那么这样做会让您的生活变得更加困难，而不是增加ConstructorFN.prototype目的。

于 2012-09-19T23:40:31.847 回答