Syntax vereinfachen

[QULIXX]

Hallo Leute,

ich habe eine Function mit ZWEI if Abfragen gibt es eine Variante dies zu vereinfachen

PHP-Code:

function(){
var xxx
...
...
...

if (zLen>0) {
            i -= steps;
            if (i<-steps) {
                clearInterval(itv);
                
            }
            trace(i);
        } else {
            i += steps;
            if (i>str.length) {
                clearInterval(itv);
                next = next+1;
                
                
                
            }
        }

...
...
...
} 


[Janoscharlipp]

wasn das?

Kannst du genauer sagen, was der Code tun soll, bzw. was die einzelnen Variablen tun?
Welche 2 ifs, das äußere und die zwei inneren, oder nur die zwei inneren?

[QULIXX]

Also hier mal die komplette Function ich dachte es langt der Schnipsel
Dies ist eine umgewandelte TypeWriter Funktion
die einmal den Text reinfliessen lässt und einmal den Text wieder rauslöscht

PHP-Code:

TextField.prototype.typeWriter = function(zLen, str, ms, steps) {
    var me = this, i = zLen;
    var itv = setInterval(function () {
        me.text = str.substring(0, i);
        //*If abfrage 
if (zLen>0) {
            i -= steps;
            if (i<-steps) {
                clearInterval(itv);
            }
//*else zweig
        } else {
            i += steps;
            if (i>str.length) {
                clearInterval(itv);
            }
        }
        updateAfterEvent();
    }, ms);
};

//Funktionsaufrufe// stehen dann später nicht so wie hier nacheinander !!!
//Text wird gefüllt
typeWriter(0, pString, 100, 10);
//Text wird geleert
pStringLen=pString.length
typeWriter(pStringLen, pString, 100, 10); 


*meine Frage war ob ich ich die If Abfrage und den else Zweig vereinfachen kann da ich noch nicht so fit im Proggen bin reine Neugierde um was dazu zulernen der Code funzt ja

[sonar]

Also ich habs jetzt nicht ganz genau analysiert, aber irgendwie siehts bissl umständlich aus... Hab auch mal sowas gebastelt, vielleicht kannst des ja mit einbauen oder so...

TextField.prototype.typeWriter = function(str) {
    var tf = this;
    var fillID = setInterval(function () {
        if (i < str.length) {
            tf.text += str.charAt(i++);
        } else {
            clearInterval(fillID);
        }
    }, Math.random() * 200 + 50);
};
txt.typeWriter("Hello, just a small typewriter..."); 

[QULIXX]

@sonar
Deine Funktion füllt ja nur das TextFeld meine füllt das TextFeld und leert das TextFeld

[sonar]

Ja das hab ich mir schon gedacht, dass da noch was anderes passiert... war ja quasi als "Anregung" gedacht oder so.

[RustyCake]

*gg* der löscht und erstellt auch!!

stop();
c = _root.createEmptyMovieClip("curr", 999);
c.createTextField("currtext", 1000, 0, 80, 160, 10);
c.currtext.border = 0;
c.currtext.autoSize = "center";
c.currtext.variable = "_root.currentanz";
c.currtext.setNewTextFormat(myTextFormat);
currentanz = "choose a song below or not mfg rusty ~~~";
c.onEnterFrame = function() {
    if (currentanz.charAt() != "~") {
        erster = currentanz.charAt();
    } else {
        erster = "~";
    }
    currentanz = currentanz.slice(erster.length, currentanz.length)+erster;
}; 

[QULIXX]

@rusty
danke für den Script der ist sicherlich auch ganz nützlich, aber das macht mein Script nicht, mein Script füllt einen langen text mit vielleicht z.B. 500 Zeichen in großen Steps, dann wenn ein User ein neuses Thema aufruft wird der Text des alten Themas gelöscht und das neue thema eingefüllt! Sozusagen ein TextEffekt.
Wie bereits schon erwähnt der Script funzt einwandfrei meine Frage war nur ob es Programmiertechnisch einfacher geht meinen script zu verkürzen oder zu vereinfachen.

Reine Neugierde um mal wieder was dazu zu lernen

[RustyCake]

AXO!
Sorry dann hab ich was falsch verstanden

[QULIXX]

@rusty
NoProb
aber haste vieleicht ne Antwort auf meine Frage oder ist das korrekt geProggt.

[RustyCake]

Imo fällt mir nix kürzeres ein, des kann sich aber morgen evtl. ändern
Wenn ich wieder nüchtern bin