Question

我理解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();

Answer 1

一些小事 - 有点像。

第一个很小。

与this和self事物保持一致。我在这里想到的例子就在你的Canvas构造函数中。

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

不是说它不好做 - 当你做对象组合的事情而不是对象构造的东西时（它经常变成对象）它会更有用完全必要，然后）。

第二件事更多的是问题。如果您正在谈论FireFox / Chrome，__proto__可能会得到相当好的支持，但如果您需要支持大量浏览器，那么您通过执行操作会让您的生活变得更加困难这样做，而不是添加到ConstructorFN.prototype对象。

Javascript类和自定义事件处理程序正确的语法

1 个答案: