array ordnen

[sauerkraut]

ich möchte einen array neu sortieren und zwar so, dass alle leeren felder im array rausgelöscht werden:
array = [bla, bla2, bla3, , , bla7]
array.ordnen();
array = [bla, bla2, bla3, bla7]

gibt es da ne funktion?

[_geo_]

Array.prototype.ordnen = function(){
    for(var i=0; i < this.length; i++){
        if(this[i]== "" or this[i]== undefined or this[i]== null){
            this.splice(i--,1);
        }
    }
}
 
ray = ["bla","bla2","bla3",null,"",undefined,"bla7"];
ray[17]="ha";
ray.ordnen();
trace(ray); 

<edit habs umgeändert damit es jetzt funtkioniert thx @ sal edit>

[salazar]

@_geo_: sehr schöne variante.... hatte grad auch was gescripted und kommentiert, aber jetzt biste mir zuvor gekommen mit deinem post und das auch noch mit einer schöneren lösung! poste trotzdem mal meinen code.... sonst wärja doof

PHP-Code:

//test array bauen
myarr = new Array();
myarr[1] = "eintrag1";
myarr[4] = "eintrag4";
myarr[9] = "eintrag9";

//test array anzeigen
trace(myarr);
trace("\n");

////////////////////////////////////////////////////////////////
//array clean prototype
Array.prototype.cleanmyarr = function() {
    for (var i = 0; i < this.length; i++) {
        if( this[ i ] == undefined ) this.splice( i--, 1 );
    }
};
//---
////////////////////////////////////////////////////////////////

//prototype wird aufgerufen
myarr.cleanmyarr(); 

//test array anzeigen
trace(myarr); 


cya, sal

[sauerkraut]

thx

[salazar]

@_geo_: hab deins grad mal getestet.... funktioniert nicht gaanz so gut wie meins
schau mal.... und mach das mal mit deinem prototype....

PHP-Code:

//test arrays bauen
myarr = new Array();
myarr[1] = "eintrag1";
myarr[4] = "eintrag4";
myarr[9] = "eintrag9";

ray = ["bla","bla2","bla3",null,"",undefined,"bla7"];

//test array anzeigen
trace("myarr: "+myarr);
trace("ray: "+ray);
trace("\n");

////////////////////////////////////////////////////////////////
//array clean prototype.... hier auch mal mit _geo_`s testen... 
Array.prototype.cleanmyarr = function() {
    for (var i = 0; i < this.length; i++) {
        if(this[i] == undefined or this[i] == null or this[i] == "") this.splice(i--, 1);
    }
};
//---
////////////////////////////////////////////////////////////////

//prototype wird aufgerufen
myarr.cleanmyarr(); 
ray.cleanmyarr(); 

//test array anzeigen
trace("myarr: "+myarr);
trace("ray: "+ray); 


cya, sal

[_geo_]

hehe, rate mal was ich gerade austeste

einfach bei deiner if noch or this[i] == "" dazu und deines ist perfekt

auch ja meines funtioniert mit deinem array nicht weil bei de for i in schleife i nicht indixiert wird wenn i undefined ist ! daher kann man for i in hierfür nicht verwenden

mfg geo ( der das natürlich wieder mal nicht gewusst hat und ne super oop lösung hinschmettern wollte )

hier die Veranschaulichung per AS

Array.prototype.ordnen = function(){
        for(i in this){
            trace("name: " +i+ "    value: "+ this[i]+"|");
                if(this[i]== "" or this[i]== undefined or this[i]== null){
                        this.splice(i,1);
                }
        }
}
 
ray = new Array();
ray[1] = "eintrag1";
ray[4] = "eintrag4";
ray[9] = "eintrag9";
ray.ordnen();
trace(ray); 

[salazar]

Zitat von _geo_

einfach bei deiner if noch or this[i] == "" dazu und deines ist perfekt

ist doch drinn...
ja schade mit dem for i in... dann wär deins eleganter

gruß, sal

[_geo_]

jop aber wenns nicht funzt is es egal wie elegant, es ist trotzdem schei**e

so und jetzt auf Teufel komm raus eine Variante mit for i in ;-)

Array.prototype.ordnen = function(){
    var tempary = new Array();
 
    for(i in this){
        if(this[i]!="" and this[i]!=undefined and typeof(this[i])!= "function"){
            tempary.push(this[i]);
            }
        }
 
    tempary.reverse();
    this.splice(tempary.length);
    for(var i=0; i<tempary.length;i++){
        this[i] = tempary[i];
    }
}
 
ray = ["bla","bla2","bla3",null,"",undefined,"bla7"];
ray["ha"] = "nice";
ray[17] = "lol";
 
ray.ordnen();
 
trace(ray); 

und die funktion braucht sogar nur 1 zeile ;-)

Array.prototype.ordnen=function(){var tempary=[];for(i in this)(this[i]!="" and this[i]!=undefined and typeof(this[i])!="function")?tempary.push(this[i]):null;tempary.reverse();this.splice(tempary.length);for(var i=0;i<tempary.length;i++)this[i]=tempary[i];} 

und ich geh jetzt schlafen

n8

[pokepika]

hat es einen besonderen grund, dass ihr nicht die flash interne funktion benutzt??

Array.sort()

[_geo_]

weil mit sort() die Einträge nicht gelöscht werden ;-)

[pokepika]

achso thread komplett lesen hilft wohl wirklich

[_geo_]

Aber zugegeben, der Name "ordnen" für die Funktion ist in dem Zusammenhang etwas verwirrend

[_geo_]

Sieg in der Nachspielzeit durch glückliches getTimer() KO (siehe sonar)

//
// ************** SONAR **************
//
Array.prototype.clean = function () {
        var i = this.length;
        while (i--) {
                if (!this[i]) {
                        this.splice (i, 1);
                }
        }
        return this;
};
then1 = getTimer();
trace(then1)
var a = new Array (1000);
trace (a.clean ());
now1 = getTimer () - then1;
trace(getTimer());
trace ("sonar: " + now1 + " ms");
//
// ************** GEO ****************
//
Array.prototype.ordnen = function(){
        var tempary = new Array();
        
        for(i in this){
                if(this[i]!="" and this[i]!=undefined and typeof(this[i])!= "function"){
                        tempary.push(this[i]);
                }
        }
        return tempary.reverse();
}
then2 = getTimer();
trace(then2);
a = new Array (1000);
trace (a =a.ordnen ());
now2 = getTimer() - then2;
trace(getTimer());
trace ("geo: " + now2 + " ms"); 

[salazar]

geil!
ja servus ihr beiden.
jetzt treibt ihrs aber auf die spitze
aber wo ist jetzt sonar´s post hin? oder war etwa nen fehler drinn?!

meine auch. diesen thread hier sollte man mal anders nennen... könnte noch für andere interessant sein und die finden dann nüscht...

gruß, sal

[sonar]

Oh oh, ich meld mich sofort vom FF ab, lerne den Umgang mit getTimer () und dann komm ich wieder...

Sorry for Schlaubi *schäm*

Aber trotzdem Grüße an geo & sal!

[edith]
ach, ihr habts garnicht gemerkt oder wie...? *pfeif.wegschlender*