Uncaught SyntaxError: Unerwartetes Token)

Question

* Hinweis - Dieser Code stammt von einer Drittanbietererweiterung. Ich hatte keinen Teil davon und vor einigen Jahren war es die einzige Erweiterung, die zu dieser Zeit verfügbar war. So, während ich Ihre Meinungen schätze, hoffe ich, dass alle Kommentare nur für Vorschläge auf einer Entschlossenheit sein können. Vielen Dank!

Wir haben viele Websites, die eine Google Maps-Komponente für ein CMS ausführen, mit der Kunden Markierungen und Umrisse (Polygone) zu ihren Google Maps hinzufügen können.

Dies funktioniert seit Jahren. Zu beachten ist, dass Google Maps JS API 2 verwendet wird, das nicht API 3, sondern Google eingestellt wurde. Allerdings hat Google festgestellt, dass API 2 auch 2013 noch gut funktionieren wird. Das ist nicht das Problem. Allerdings müssen sie etwas geändert haben, denn auf den anderen Seiten erscheinen die Karten und Polygone auf allen unseren Seiten. Sie sind auf verschiedenen Servern.

Bevor es keine Fehler war, aber jetzt in Chrome heißt es:

"Uncaught Syntaxerror: unerwartetes Token)" für die Linie 1669 in einer JS-Datei. Sie können die Datei im folgenden Gist sehen:

https://gist.github.com/2238148

Wie Sie sehen nicht es fehlt „)“ und der Code ist seit Jahren auf fast 100 Standorten unmodifizierten arbeiten, also davon ausgehen, etwas auf Googles Ende Muss habe sich geändert. Aber können wir uns an diesen Code anpassen, um dem Wandel entgegenzuwirken? -Update am 25. März, wenn alle diese brach Google ein Update für ihre Google Maps API 2.

das Web durchsucht gemacht und hier ist ein Beispiel-Website die gleiche Komponente mit dem gleichen Fehler mit: http://goo.gl/GMgOs

Dieses Problem erscheint in der Nähe sein:

 
// extract current digraph from overlay function 
    var digraph = GMap.addOverlay.toString().replace(/\s/g,'').replace(/.push\([^{]+\);a.initialize\([^{]+\);a.redraw\([^{]+\).+$/,'').replace(/^.+\./,''); 
    // add multiple overlays at once (api hack to improve loading speed) 
    GMap2.prototype.addOverlays = function(a) { 
     var b = eval('this.' + digraph); 
     var i = a.length; 
     while (i--) { 
      b.push(a[i]); 
      a[i].initialize(this); 
      a[i].redraw(true); 
     } 
    } 

Wenn dieser Code die geänderte Version auf einem Kern oder pastie jemand veröffentlichen werden muss alterted könnte?

Ein weiteres Update - Dieser Code in der oben genannten ich auskommentiert, da es nicht benötigt wird, war ein Hack, um die Dinge zu beschleunigen. Immer noch ein Fehler wird jedoch in den Kommentaren vermerkt. Ich habe jedoch hier bemerkt: https://developers.google.com/maps/documentation/javascript/v2/reference#GMap2.Methods.Overlays, dass es den Code "addOverlay" anstatt "addOverlays" nennt, also frage mich, ob vielleicht das s im letzten API Google Update entfernt wurde. Das Entfernen der s an allen drei Stellen schießt einfach einen neuen Fehler hervor, der [Objekt] viele Male wiederholt.

Answer 1

Der Code ist ein Greuel, in der Tat habe ich nicht gesehen ein schlechteres Add-on in fünf Jahren der Arbeit mit und hilft Benutzern der Version 2. Es überschreibt GMap (Teil von die Version 2 API, um die Kompatibilität mit Version 1) ohne erneute Deklaration zu gewährleisten. Der Fehler, den Sie bekommen, ist ein direktes Ergebnis eines Hacks zum minimierten Code: Dieser Fehler musste irgendwann fehlschlagen und sollte niemals implementiert worden sein.

Das Beste, was Sie tun können, ist die var digraph Zeile zu entfernen und dann die neue Methode GMap2.prototype.addOverlays, die darauf folgt, neu zu definieren. Dadurch kann der Code die addOverlay()-Funktion der API verwenden und das Problem beheben.

GMap2.prototype.addOverlays = function(a) { 
     var i = a.length; 
     while (i--) { 
      this.addOverlay(a[i]); 
     } 

Es scheint, dass addOverlays() eine Reihe von Overlays nimmt. Die vorhandene Methode versucht, sie direkt zu dem internen Array von Überlagerungen hinzuzufügen, das verschoben wurde. Die vorgeschlagene Methode verwendet einfach die eigene addOverlay() Methode von GMap2, um jedes Mitglied des Array von Objekten hinzuzufügen. Daher verwenden wir eine exponierte Methode und versuchen nicht, den minimierten Code der API zu entführen, wenn wir das wieder tun, würde es höchstwahrscheinlich wieder zerbrechen.

---

GMarker.prototype.openInfoWindowFX und GMarker.prototype.updateInfoWindow sind Ergänzungen GMarker und unwahrscheinlich zu Problemen führen (vor allem, wenn sie zur Zeit arbeiten), obwohl auch sie verwenden Eigenschaften von GMarker - was nicht wirklich recommnded ist.

Answer 2

Dieser Code ist hässlich. Sie tun GMap.addOverlay.toString(), ersetzen Sie etwas mit regulären Ausdrücken und führen Sie es dann mit Eval.

Warum müssen Sie etwas in einem Funktionscode ersetzen? Was ist der Code von GMap.addOverlay? Wenn Sie diese Fragen beantworten können, sollten Sie sehen können, warum Ihre Regexp nicht mehr funktioniert und ungültigen Code zurückgibt.

Answer 3

Offensichtlich wurde die GMap.addOverlay Funktionsdefinition geändert und Ihr RegExp-Hacken am Quellcode funktioniert nicht mehr.

alert(GMap.addOverlay.toString()) 

und

alert(digraph) 

Um zu sehen, was die neue Funktion ist und was Ihre reguläre Ausdrücke, um es tun.

Warum genau müssen Sie ihren Code trotzdem ändern? Warum haben Sie nicht einfach die gewünschte Funktionsdefinition kopiert, die Änderungen vorgenommen und dann diese Version verwendet? (Ohne eine eval, entweder.)

Answer 4

Der von Ihnen angegebene Code basiert auf internen Implementierungsdetails für GMap2.prototype.addOverlay, die jederzeit geändert werden können.

Es wäre besser, sich nur auf die externe Schnittstelle für GMap2 zu verlassen. Sie können eine addOverlays Methode als solche implementieren:

GMap2.prototype.addOverlays = function(overlays) { 
    for (var i = 0, I = overlays.length; i < I; ++i) { 
    this.addOverlay(overlays[i]); 
    } 
}; 

Können Sie den Code-Schnipsel aus Ihrem Post mit diesem ersetzen und sehen, ob es funktioniert?