Script-Murks?

[SwKy]

Also, ich weiß nicht. Ich hab da mal was gescriptet. Und ich frage mich, ob das nicht totaler Murks ist und viel einfacher und übersichtlicher ginge.
Das Problem bei mir ist, dass ich meine Scripte immer Schrittweise verfasse: also erst mal muss das funktionieren und dann merke ich, dass ja dann was anderes auch funktionieren muss und dann, dass hier noch was ins Script rein muss, damit...
Ich habe also vorher nicht die Vorstellung, wie das Script (in etwa) aussehen muss. Und ich glaube dadurch werden meine Scripte oft recht komplizier und durcheinander, und bestimmt auch manchmal rechenaufwendiger als nötig.

Also es war folgendes umzusetzten:
Eine Deutschlandkarte in PLZ-Gebiete aufgeteilt. Jedes PLZ-Gebiet ist ein But.
Bei Klick auf den But erscheinen die entsprechenden Daten (PLZ, Ort, Tel., Nr.).
Die Daten sind in txt-Dateien gespeichert, die jeweils "Array + 1.Nr.d.PLZ" heißen, und zwar so: PLZ1, Ort1, Tel.1, Nr.1; PLZ2, Ort2, Tel.2, Nr.2 ...

und hier ist mein Script:

var scrpInhalt = scrpInstanzname.getScrollContent (); //in scrpInhalt wird der Inhalt meiner Scrollpane gespeichert
 
function ticker(ziel) { 
    var tx = ziel.text; 
    tx = tx.substr(1, tx.length) + tx.substr(0, 1); 
    ziel.text = tx; 
} 
 
MovieClip.prototype.myPLZ = function(){
    this.onRollOver = function(){
        this.gotoAndStop(2);
    }
    this.onRollOut = function(){
        this.gotoAndStop(1);
    }
    this.onRelease = function(){
        this.gotoAndPlay(3);
        mystring = "...Bitte haben Sie einen Augenblick Geduld, die Daten werden geladen..."; 
        Geduld.text = mystring; 
        setInterval(ticker, 96, Geduld);
        if(geklickt != undefined){ //fragt ab, ob schon mal ein But geklickt wurde 
            for (tfnr=0; tfnr<=x; tfnr++) {
                scrpInhalt["tf_"+tfnr].removeTextField();//wenn ja, werden die bestehenden txt-felder gelöscht...
                }   
            with(geklickt){
                gotoAndStop(1); //...der zuvor geklickte But wieder in seine Ausgangsposition gebracht...
                enabled = true //... aktiviert...
                _name = wert //...und ihm wieder der ursprüngliche Instanzname gegeben.
            }   
        }
        wert = this._name
        this._name = "geklickt" //der neu geklickte But wird "geklickt" genannt...
        geklickt.enabled = false //...und deaktiviert
        
        
    LadeObj = new LoadVars();
    x=0 // Nummer wird bei jedem onEnterFrame +1
    n=0// Nummer wird bei jedem onEnterFrame +1, wenn durch 4 teilbar, wieder 0
    m=0 // Nummer wenn n % 4 = 0 dann m + 1
    i=0 // für x-Koordinate des neuen Textfeldes
    h=0 // y-Koordinate des neuen Textfeldes
    t=0 // macht die Breite der ersten Textfelder kürzer
 
    LadeObj.onLoad = function(success) { 
        if(success) {
            meinArray = LadeObj.adr.split(";"); //mein geladener Text wird bei ; gesplited...
            meinArray.sort(); // und sortiert
            onEnterFrame = function() {
                neuArray = meinArray[m].split(","); //mein dadurch entstandenes array wird bei , gesplited
                scrpInhalt.createTextField(("tf_"+x),x,5+i,2+h,50+t,22);// in meiner Scrollpane wird ein Textfeld erstellt
                t=120 // bewirkt, das alle jetzt entstehenden txt-Felder breiter werden
                //scrpInhalt["tf_"+x].border = true;
                scrpInhalt["tf_"+x].selectable = true
                scrpInhalt["tf_"+x].embedFonts = true
                mf = new TextFormat();
                mf.font = "arial";
                mf.size = 12;
                scrpInhalt["tf_"+x].text = neuArray[n]// setzt den Inhalt des jeweiligen Textfeldes
                scrpInhalt["tf_"+x].setTextFormat(mf);
                x = x+1
                n = n+1
                i = i+170
                if(n==1){ // bewirkt, dass die txt-felder nach dem ersten länger sind
                    i=i-120
                    }else if((n%5) == 0){
                        i=i+120
                        }
                if((n%4) == 0){// bewirkt, dass nach 4 txt-feldern eine neue Reihe angefangen wird...
                    n=0
                    m = m+1
                    h = h+18 //sorgt dafür, dass das nächste txtfeld weiter unten erstellt wird
                    i=0 //sorgt dafür, dass das erste txtfeld wieder kurz ist
                    t=0
                    }
                if(x==meinArray.length*4){ //beendet das Ganze, wenn keine Elemente mehr im Array sind
                    scrpInstanzname.refreshPane ();
                    Geduld.text = ""
                    delete this.onEnterFrame;
                }
            }
        }
    } 
LadeObj.load("Array"+wert+".txt");}
} 

und nu sacht ma was dazu und wenn der eine oder andere das/was aus dem Script gebrauchen kann, würds mich auch freun

[soloFlash]

also:
mir geht es oftmals genau so. Etas, hat man bei der Überlegung vergessen, wird dann noch schnell reingebaut... und so wächst der Code!
Du könntest so etwas vielleicht in funktionen auslagern, dann werden deine Blöcke nicht zu gross. Aber da Du viel kommentiert hast (weit mehr, als der durchschnitt, würde ich sagen -> *lob*) ist eigentlich alles recht gut lesbar.
Einzig: warum machst Du ein onEnterFrame? Um deine textfelder zu erstellen währe Doch eine for-Schleife sinnvoler? oder willst DU das die Felder sichtbar nacheinander erstellt werden?
Funktionalität habe ich übrigends nicht geprüft, sondern nur generelle Verständlichkeit.
meinArray ist vielleicht nicht die geglückteste Bezeichnung für eine Variable

grüße,
solo

[SwKy]

funktionieren tuts

@soloFlash: ach so, eine for-schleife läd erst alles und zeigt es dann? Ist eine For-schleife weniger rechenintensiv? Dass man es sieht, wenn sich das ganze aufbaut, find ich ok, dann sieht man wenigstens, das was passiert, aber es dauert schon recht lang.

[soloFlash]

Hi,

onEnterFrame: ist eine Art-Schleife, die bei jedem Frame (abhängig von Framezahl pro Sekunde, und von der Belastung des Rechners) durchlaufen wird. Solange bis Du es mit delete löschst, oder überschreibst. Damit kannst Du zeitliche Abläufe gut darstellen.

for-schleifen: laufen innerhalb eines Frames ab, und zwar komplett. Der Anwender sieht nur das Endergebnis.

Über rechnerleistung kann ich nichts sagen ;(

Unter "Aufbau" verstehe ich eher, dass die felder irgendwie nett animiert auf der bühne erscheinen, nicht dass sie einzeln kreiert werden.

lies Dir mal was zu onEnterFrame durch.

grüße,
solo

[SwKy]

bin ich grad zu blöd???

wollte das mit der for-Schleife mal ausprobieren. Hab das jetzt so gemacht:

var scrpInhalt = scrpInstanzname.getScrollContent (); //in scrpInhalt wird der Inhalt meiner Scrollpane gespeichert
 
function ticker(ziel) { 
    var tx = ziel.text; 
    tx = tx.substr(1, tx.length) + tx.substr(0, 1); 
    ziel.text = tx; 
} 
 
MovieClip.prototype.myPLZ = function(){
    this.onRollOver = function(){
        this.gotoAndStop(2);
    }
    this.onRollOut = function(){
        this.gotoAndStop(1);
    }
    this.onRelease = function(){
        this.gotoAndPlay(3);
        mystring = "...Bitte haben Sie einen Augenblick Geduld, die Daten werden geladen..."; 
        Geduld.text = mystring; 
        setInterval(ticker, 72, Geduld);
        if(geklickt != undefined){ //fragt ab, ob schon mal ein But geklickt wurde 
            for (tfnr=0; tfnr<=x; tfnr++) {
                scrpInhalt["tf_"+tfnr].removeTextField();//wenn ja, werden die bestehenden txt-felder gelöscht...
                }   
            with(geklickt){
                gotoAndStop(1); //...der zuvor geklickte But wieder in seine Ausgangsposition gebracht...
                enabled = true //... aktiviert...
                _name = wert //...und ihm wieder der ursprüngliche Instanzname gegeben.
            }   
        }
        wert = this._name
        this._name = "geklickt" //der neu geklickte But wird "geklickt" genannt...
        geklickt.enabled = false //...und deaktiviert
            
    LadeObj = new LoadVars();
    x=0 // Nummer wird bei jedem onEnterFrame +1
    n=0// Nummer wird bei jedem onEnterFrame +1, wenn durch 4 teilbar, wieder 0
    m=0 // Nummer wenn n % 4 = 0 dann m + 1
    i=0 // für x-Koordinate des neuen Textfeldes
    h=0 // y-Koordinate des neuen Textfeldes
    t=0 // macht die Breite der ersten Textfelder kürzer
 
    LadeObj.onLoad = function(success) { 
        if(success) {
            meinArray = LadeObj.adr.split(";"); //mein geladener Text wird bei ; gesplited...
            meinArray.sort(); // und sortiert
            for(f=1;f==1;f=1) {
                neuArray = meinArray[m].split(","); //mein dadurch entstandenes array wird bei , gesplited
                scrpInhalt.createTextField(("tf_"+x),x,5+i,2+h,50+t,22);// in meiner Scrollpane wird ein Textfeld erstellt
                t=120 // bewirkt, das alle jetzt entstehenden txt-Felder breiter werden
                //scrpInhalt["tf_"+x].border = true;
                scrpInhalt["tf_"+x].selectable = true
                scrpInhalt["tf_"+x].embedFonts = true
                mf = new TextFormat();
                mf.font = "arial";
                mf.size = 12;
                scrpInhalt["tf_"+x].text = neuArray[n]// setzt den Inhalt des jeweiligen Textfeldes
                scrpInhalt["tf_"+x].setTextFormat(mf);
                x = x+1
                n = n+1
                i = i+170
                if(n==1){ // bewirkt, dass die txt-felder nach dem ersten länger sind
                    i=i-120
                    }else if((n%5) == 0){
                        i=i+120
                        }
                if((n%4) == 0){// bewirkt, dass nach 4 txt-feldern eine neue Reihe angefangen wird...
                    trace("nächste Reihe");
                    n=0 
                    m = m+1
                    h = h+18 //sorgt dafür, dass das nächste txtfeld weiter unten erstellt wird
                    i=0 //sorgt dafür, dass das erste txtfeld wieder kurz ist
                    t=0
                    }
                if(x==meinArray.length*4){ //beendet das Ganze, wenn keine Elemente mehr im Array sind
                    scrpInstanzname.refreshPane ();
                    Geduld.text = ""
                    trace("eigentlich zu Ende"),
                    f=2;
                }
            }
        }
    } 
LadeObj.load("texte/Array"+wert+".txt");}
} 

nicht grad elegant, aber funktionieren müßte es doch, oder?
mir wird "eigentlich zu Ende" getraced, also wird f doch auf 2 gesetzt, das müsste die Schleife doch beenden, oder? tut es aber nicht. Flash fragt mich dann irgendwann, ob ich abbrechen will, oder lieber warten, bis der Rechner abstürzt.

[soloFlash]

wie nett, das Flash frägt

for(<startwert setzen>;<Abbruchbedingung>;<Wertveränderung>)

in Deinem fall so (ohne gewähr, schnell hingeklatscht von mir):
for(x=1;x<=meinArray.length*4;x++)

ach ja, die Zeile mit "x = x+1" muss dan raus, ebenso Deine Abbruchabfrage unten im Coding.

Vielleicht mal mit Schleifen beschäftigen (DO, WHILE, FOR)?!?!

wenn probleme, dann fragen.
solo

[SwKy]

Ja. Mein Flash redet mit mir. Deins nicht??? (manchmal reden wir auch aneinander vorbei )


... probier ich mal. danke.

[SwKy]

Vielleicht mal mit Schleifen beschäftigen (DO, WHILE, FOR)?!?!

Ich hatte mich natürlich mit Schleifen beschäftigt, jedenfalls mit for, denn das wollte ich ja benutzen. Wollte es auch so machen, wie du, dachte aber das geht nicht, weil ich x ja das erste mal als 0 brauch. Ich wußte nämlich nicht, dass erst die Schleife durchläuft und dann der nächste Ausdruck ausgewertet wird.
Aber wo der Unterschied zwischen den drein (do,while, for) ist, außer in der Art wie sie angewendet werden, hab ich nicht verstanden.

Hier nun mein Schleifenbericht:

for: geht bei nicht so vielen Textfeldern sehr schnell (schneller als EnterFrame); bei vielen Textfeldern dauert es länger als EnterFrame und will auch dauernd abbrechen.

mit while ist es genauso.

onEnterFrame braucht zwar bei wenigen Textfeldern etwas länger, ackert das ganze aber brav bis zum Schluß durch.

Ergo: ich muss onEnterFrame lassen, oder hat jemand eine andere Idee? Oder sollte man so was gar nicht mir Flash lösen?

[soloFlash]

DO wird mindestens einmal durchlaufen, weil die Prüfung immer am Ende der Schleife ist, WHILE ist, analog FOR, eine Schleife, die am ASnfang immer prüft. Alles klar? wenn nicht auchegal. FOR reicht i.d.R. volkommen aus.

so, oef ist schneller? hmmm, warum, was bremst dich? wieviele durchläufe hast du denn?

solo

[SwKy]

... ich glaube, beim meißten sind es um die 450 Textfelder (also auch Durchläufe)
oef ist ja nur schneller, wenn es so viele sind, dass for lahmgelegt wird. Wenn es nur 80 oder so sind, ist for schneller.

[soloFlash]

450 durchläufe??? Was machst Du da???

Wenn dein Film mit 30fps läuft, dann benötigst Du aber 450/30=15 Sekunden.
For benötogt länger als 15 Sekunden? Kann ich kaum glauben!
Oder meinst Du, dass Dein Film weiterläuft (wird ja pro Frame nur ein Textfeld und Co erstellt, und for Deinen Film zum stehen bringt, bis die 450 Textfelder und Co fertig sind?

Oder solltest Du an der Konzeption was ändern?!?!

Grüße,
solo

[SwKy]

wohl ein bisschen viel, was?
Ich wusste aber nicht, wie ichs anders lösen soll. Ich hab für jedes Element (plz, Ort, tel., Nr.) ein Textfeld genommen, obwohl sie in einer Reihe stehen sollen, damit jede plz unter der anderen steht. Also wie eine Tabelle. Und wenn es dann 120 Adressen gibt, gibts 480 Textfelder.

Ich denke, dass der film mit for irgendwann langsamer wird, weil flash meint, dass es in einer Endlosschleife hängt:

"Macr. Flash Player wurde durch ein Script in diesem Film verlangsamt. Wenn das Script weiterhin ausgeführt wird, reagiert der computer möglicherweise nicht mehr. Möchten sie die Ausführung des Scriptes jetzt unterbrechen?"