Array.prototype verursacht Fehler

Question

Ich versuche w2uimulti select in einem der d3-Diagramme zu implementieren, die gerade arbeiten.

Dies ist der Link zu Beispiel jsfiddle mit dem Problem.

Ich habe drei Funktion:

//get a column of an array 
Array.prototype.getColumn = function(name) { 
    return this.map(function(el) { 
    // gets corresponding 'column' 
    if (el.hasOwnProperty(name)) return el[name]; 
    // removes undefined values 
    }).filter(function(el) { 
    return typeof el != 'undefined'; 
    }); 
}; 
//remove duplicates in an array 
Array.prototype.contains = function(v) { 
    for (var i = 0; i < this.length; i++) { 
    if (this[i] === v) return true; 
    } 
    return false; 
}; 
Array.prototype.unique = function() { 
    var arr = []; 
    for (var i = 0; i < this.length; i++) { 
    if (!arr.contains(this[i])) { 
     arr.push(this[i]); 
    } 
    } 
    return arr; 
} 

ich diese drei in einer meiner Funktionen implementieren müssen.

Problem ist, dass, wenn ich versuche, diese Funktionen mit Array.prototype zu implementieren, ich Elemente in der Multiselect als "undefined". Die Nummer "undefined" ist direkt proportional zur Anzahl der Funktionen mit Array.prototype Funktionen.

Wenn ich diese Funktionen zu entfernen, kann ich den Multi-Select bekommt richtig arbeiten (nur der Multi-Select-Teil, nicht das Diagramm als Ganze. Ich verstehe nicht, was den Fehler verursacht.

Jede Hilfe ist willkommen. Vielen Dank.

Answer 1

Im Allgemeinen mit den Kern JavaScript-Objekte zu stören ist eine schlechte Idee, wenn Sie mit den Bibliotheken von Drittanbietern arbeiten.Wenn Sie immer noch so bleiben und dieses spezielle Problem lösen wollen, verwenden Sie das Objekt .defineProperty-Methode, das aufzählbare Bit

deaktivieren So zum Beispiel ändern

Array.prototype.contains = function(v) { 
    for (var i = 0; i < this.length; i++) { 
    if (this[i] === v) return true; 
    } 
    return false; 
}; 

zu

Object.defineProperty(Array.prototype, 'contains', { 
    enumerable: false, 
    value: function(v) { 
     for (var i = 0; i < this.length; i++) { 
      if (this[i] === v) return true; 
     } 
     return false; 
    } 
}); 

und ähnlich für andere Prototyp-Methode, die Sie hinzugefügt haben.