mein erster prototype

[bboy]

Hallo zusammen,

ich probiere gerade meinen ersten Array-Prototype zu schreiben. Der Prototype soll das Array säubern, damit mein ich alle nicht besetzten Postionen entfernen so das das Array danach nur auf den Positionen 0-n ohne Lücken belegt ist. Momentan passiert aber leider gar nichts.

PHP-Code:

a = new Array();
a[1] = "value1";
a[4] = "value4";
//
Array.prototype.clean = function() {
    var tmpArr = new Array();
    for (var i in this) {
        tmpArr.push(this[i]);
    }
    tmpArr.reverse();
    return tmpArr;
};
a.clean(); 


Kann wer erkennen was ich da falsch mache? Bis jetzt hatte ich das ganze als normale Funktion, und da hat es hervorragen funktioniert.

[bboy]

Vielleicht zum besseren Verständnis hier noch meine ursprüngliche Funktion.

PHP-Code:

a = new Array(); 
a[1] = "value1"; 
a[4] = "value4"; 

function cleanArray(Arr) {
    var tmpArr = new Array();
    for (var i in Arr) {
        tmpArr.push(a[i]);
    }
    tmpArr.reverse();
    return tmpArr;
}

a = cleanArray(a); 


Hier ist mein Array a nach dem funktionsaufruf nur noch an den ersten beiden positionen [0] und [1] belegt. Die gleiche Funktionalität hätte ich aber gerne als Array.prototype

[rendner[i]]

EDIT:
Habs nochmal geändert.

PHP-Code:

a = new Array(); 
a[1] = "value1"; 
a[4] = "value4"; 
 
Array.prototype.clean = function() { 
    for (var i = 0; i < this.length; i++) 
    { 
        if( this[ i ] == undefined ) this.splice( i--, 1 ); 
    } 
}; 
a.clean(); 


uuups der warrantmaster war schneller...

[warrantmaster]

PHP-Code:

a = new Array();
a[1] = "value1";
a[4] = "value4";
Array.prototype.clean = function() {
    var tmpArr = new Array();
    for (var i=0;i<this.length;i++) 
    {
        this[i]!=undefined ? tmpArr.push(this[i]) :0;;
    }
    return tmpArr;
};
trace(a)
trace(a.clean()); 


[sonar]

So, und weil's so schön is, gleich noch ne Möglichkeit...

a = new Array(); 
a[1] = "value1"; 
a[4] = "value4";
//
Array.prototype.clean = function () {
    var i = this.length;
    while (i--) {
        if (this[i] == undefined) {
            this.splice (i, 1);
        }
    }
    return this;
}; 

Hi Freund warrant

[warrantmaster]

tach freund sonar

btw, hast die schickste lösung! edit->oder doch nicht ? nochmal edit-> doch!
grz

[rendner[i]]

@ warrantmaster:
...da wahr ich mal wieder zu voreilich mit den beschuldigungen...habs dann auch gesehen...sonar's Versi fängt ja hinten an ...aber das return könnte man sicher weglassen...

[sonar]

@warrant: Zumindest die kelor'sche Lösung, hehe...
("...while is des schnellste...")

@rendner: ja, kann man weglassen, aber mit return kannst mit dem neuen Array gleich weiterarbeiten:

trace (a.clean ()); 

[bboy]

Vielen Dank für die gleich mehrfache Lösung meines Problems. Nichts desto trotz versteh ich nicht wieso mein eigener Ansatz nicht funktioniert.

[sonar]

Stimmt...
Aber er funktioniert doch, hat nur n paar Unschönheiten:
1. das tmpArr brauchts nicht
2. du kriegst beim tracen ein [type Function] mit raus (das is die prototype-Methode selber) - lässt sich aber durch die erste Zeile hier im prototype gleich in den Griff kriegen...

var a = new Array ();
a[1] = "value1";
a[4] = "value4";
// 
Array.prototype.clean = function () {
    ASSetPropFlags (Array.prototype, null, 1, 1);
    var tmpArr = new Array ();
    for (var i in this) {
        tmpArr.push (this[i]);
    }
    tmpArr.reverse ();
    return tmpArr;
};
trace (a.clean ()); 

[bboy]

Mein prototype hat funktioniert und ich habs gar nicht bemerkt!

Der Unterschied ist aber das ich bei meinem (von sonar verbesserten) prototype a = a.clean() schreiben muss um mein Array zu säubern, bei sonars eigener Lösung aber nur a.clean().
So hatte ich das davor auch immer probiert, und da passiert bei meinem eigenen nichts. Versteh nur nicht wieso, es wird doch bei beiden am ende nur ein Array returned...

[sonar]

Zitat von bboy

Der Unterschied ist aber das ich bei meinem (von sonar verbesserten) prototype a = a.clean() schreiben muss um mein Array zu säubern, bei sonars eigener Lösung aber nur a.clean().

Ja, das liegt eben an dem tmpArr - da wird dein Original-Array nicht überschrieben. Bei meiner Version schon, das is ja eigentlich auch der Sinn des ganzen, oder..?

[bboy]

Ahh, so langsam check ichs. Vielen Dank noch mal.