JSLint-Fehler: Unerwarteter 'dieser'

Also in anderen Worten, JSLint nicht automatisch erwarten, dass mich ein Konstruktor-pattern?

Wissen Sie, ich glaube, du hast Recht. Ihre Frage nervte mich, und ich habe mich für Crockford ' s JSLint Diskussion Gruppe und gefragt. Er antwortete, aber ignoriert die Lösung, die ich vorstelle, weiter unten, die ich denke bedeutet, dass es okay ist, auf die gleiche Weise JSLint nicht beschweren, wenn etwas vergeht aufbringen.

_{(ich warte immer noch auf eine aktualisierte Gute Teile, obwohl.)}

Diesem VORBEHALT beiseite, hier ist was ich vorschlagen würde zu tun für OO-JavaScript, dass geht Beta JSLint (bis heute jedenfalls).

Werde ich umschreiben, ein Beispiel von MDN ' s Seite, "Einführung in die objektorientierte Programmierung," die selbst verwendet this liberal.

Mit this

Hier ist das original, unlinted MDN-Beispiel aus dem Abschnitt " verknüpfte, oben:

var Person = function (firstName) {
  this.firstName = firstName;
};

Person.prototype.sayHello = function() {
  console.log("Hello, I'm " + this.firstName);
};

var person1 = new Person("Alice");
var person2 = new Person("Bob");

//call the Person sayHello method.
person1.sayHello(); //logs "Hello, I'm Alice"
person2.sayHello(); //logs "Hello, I'm Bob"

Folgt, dass die Konventionen, die wir kennen und lieben.

Ohne this

Ist es ziemlich einfach, herauszufinden, wie man "Konstrukteure", die nicht Folgen, dass die Muster, aber wir verlieren die Verwendung von prototype, wenn ich mich nicht etwas fehlt, und haben alle mit den Methoden eines Objekts in unser Konstruktor, wollen wir alle unserer Peeps zu teilen.

/*jslint white:true, devel:true */
var Peep = function(firstName) {
    "use strict";
    var peep = {};
    peep.firstName = firstName;

    peep.innerSayHello = function() {
        console.log("Hello, I'm " + peep.firstName + ".");
    };

    return peep;
};

var peep1 = new Peep("Bob");
var peep2 = new Peep("Doug");

peep1.innerSayHello();
peep2.innerSayHello();

So gibt es eine lintable alternative. Das bedeutet, andere als die return peep; und die innere definition von Methoden, machen JavaScript zu handeln, wie viele OO-Sprachen, die Sie stoßen könnten. Es ist nicht falsch, mindestens.

Keinen Zugang zu prototype ist nicht schrecklich; es ist wirklich eine schlechte Nachricht zu ändern prototype irgendwo, dass ist nicht direkt neben dem Konstruktor, als würde der code gehen Sie zu spaghetti. "Einige Persons haben sayGoodbye() und einige nicht, je nachdem, ob wir würden geändert, das Vorbild an dem Punkt Ihrer Konstruktion." Das ist schrecklich. Also diese alternative Konvent hat seine Vorteile.

Können Sie immer noch, natürlich, hinzufügen von Funktionen zu einer einzigen Instantiierung Peep später, aber ich bin mir nicht sicher, wie Sie würde den Zugang firstName ohne Verwendung this, also vielleicht will er uns zu stoppen munging Objekte nach dem Bau.

person1.sayGoodbye = function (other) { 
    console.log("Goodbye, " + other + "."); 
};

(Ich meine, wir könnten auch noch monkey-patch Peep zu ändern Mitte-Prozess, aber es ist schrecklich, dumm Programmierung. In der Regel.)

Vererbung (ohne this)

- Und Erbrecht ist einfach genug, denke ich.

var PeepWithGoodbye = function (firstName) {
    "use strict";
    var peepWithGoodbye = new Peep(firstName);

    peepWithGoodbye.innerSayGoodbye = function (otherPeep) {
        if (undefined === otherPeep) {
            otherPeep = { firstName: "you" };
        }
        console.log("This is " + firstName 
            + " saying goodbye to " + otherPeep.firstName + ".");
    };

    return peepWithGoodbye;
};

var pwg1 = new PeepWithGoodbye("Fred");
pwg1.innerSayHello();           //Hello, I'm Fred.
pwg1.innerSayGoodbye(peep1);    //This is Fred saying goodbye to Bob.
pwg1.innerSayGoodbye();         //This is Fred saying goodbye to you.

EDIT: Siehe auch diese Antwort, wo der Fragesteller später gefunden Crockford vorgeschlagenen Mittel zur Erstellung von OO-javascript. Ich bin versucht, Sie zu überzeugen, dass guy zu löschen, die Q&A und bewegen Sie den Ein hier. Wenn er das nicht tut, werde ich wohl hinzufügen, seine Sachen und community wiki hier.

EDIT: Sehen diese von MDN, warum es funktioniert:

(Normalerweise Konstruktoren keinen Wert zurückgeben, aber Sie können wählen, zu tun
also, wenn Sie wollen, um das überschreiben der normalen Objekt-Erstellung.)

In einem strikten Modus this Referenz festgelegt ist undefined.

So dass beide Ihre Aussagen verursachen, Lesen der Eigenschaften der undefined Objekt, die zu einer Ausnahme führen.

Wie kann ich richtige mein code?

Entfernen Sie die beiden Linien.

UPD: was ich Staat oben ist, wie JSLint behandelt dein code, nicht, wie ich das tun.

Eine alte Frage, ich weiß, aber falls es hilft jemand, ich war gerade ein sprechen von Douglas Crockford, in dem er sagt (ungefähr 23mins), dass er nahm es heraus, weil einem Angreifer das ausführen einer Methode eine Funktion und erhalten Zugang zu den globalen Bereich mit der "this" - Schlüsselwort.

Er sagt, es bedeutete auch nicht mit dem Objekt.erstellen Sie mehr - eine Funktion, die er half bei der Einführung in die Sprache!