Optimieren eines Scriptes

[wangman]

Tag
bin dabei einen slider zu machen inhalte sollen hinteher dyn geladen werden.
wollte fragen ob einer eine idee hat den script zu optimieren da er mir sehr umständlich scheint.
lg
wangman

Nun, deine erste drei if-Abfrage könntest du schon stark verkürzen, die fragen ja immer dasselbe ab:

if (0<_root.mp) {
        _root.pic._x = _root.pic._x+_root.gesch;
        _root.bewegung_r = 1;
        _root.bewegung_l = 0;
    }
        if (0<_root.mp) {
        _root.pic1._x = _root.pic1._x+_root.gesch;
        _root.bewegung_r = 1;
        _root.bewegung_l = 0;
    }
        if (0<_root.mp) {
        _root.pic2._x = _root.pic2._x+_root.gesch;
        _root.bewegung_r = 1;
        _root.bewegung_l = 0;
    } 

Daraus könntest du dann bspw. folgendes machen:

if (0<_root.mp) {
_root.pic._x += _root.gesch;
_root.pic1._x += _root.gesch;
_root.pic2._x += _root.gesch;
        _root.bewegung_r = 1;
        _root.bewegung_l = 0;
} 

Oder sollten das else-Zweige sein?

[rendner[i]]

Zitat von agedoubleju

Oder sollten das else-Zweige sein?

[wangman]

hey
danke für die antwort!!
das problem hab ich jetzt mit funktionen gelöst ich bin eigentlich schon rel weit. aber es gibt nochn paar probleme hier der script :

auf frame 1

bildanzahl=14
for (var i = 1; i<bildanzahl; i++) {
    _root.createEmptyMovieClip("container"+i, i);
    //_root["container"+i].loadMovie("tod"+i+".jpg");
    _root["container"+i].loadMovie("bild.jpg");
    _root["container"+i]._x = 30;
    _root["container"+i]._y = 80*i;
}
function slide(mcmc) {
    // Position maus
    _root.mp = _ymouse;
    // Bewegung rechts
    if (400<_root.mp) {
        mcmc._y = mcmc._y+_root.gesch;
        _root.bewegung_r = 1;
        _root.bewegung_l = 0;
    }
    // Bewegung links
    if (400>_root.mp) {
        mcmc._y = mcmc._y-_root.gesch;
        _root.bewegung_r = 0;
        _root.bewegung_l = 1;
    }
    // Sprung nach hinten
    if (mcmc._y>=820 && _root.bewegung_r == 1) {
        mcmc._y = -20;
    }
    // Sprung nach vorne
    if (mcmc._y<=-20 && _root.bewegung_l == 1) {
        mcmc._y = 820;
    }
    
    //Geschwindigkeits abfrage und festsetzung links
        if (_root.bewegung_l == 1) {
            if (_ymouse>=400) {
                _root.gesch = 2.5;
            } else if (_ymouse>=300) {
                _root.gesch = 5;
            } else if (_ymouse>=200) {
                _root.gesch = 7.5;
            } else if (_ymouse>=100) {
                _root.gesch = 10;
            }
        }
         //Geschwindigkeits abfrage und festsetzung rechts
        if (_root.bewegung_r == 1) {
            if (_ymouse<=500) {
                _root.gesch = 2.5;
            } else if (_ymouse<=600) {
                _root.gesch = 5;
            } else if (_ymouse<=700) {
                _root.gesch = 7.5;
            } else if (_ymouse<=800) {
                _root.gesch = 10;
            }
        }
} 

so ich hab das ganze als funktion gemacht, dass ich das ohne probleme und viel script auf viele objekte umlenken kann.

nämlich so:

onClipEvent (enterFrame) {
        _root.slide(_root.container1);
        _root.slide(_root.container2);
        _root.slide(_root.container3);
        _root.slide(_root.container4);
        _root.slide(_root.container5);
        _root.slide(_root.container6);
        _root.slide(_root.container7);
        _root.slide(_root.container8);
        _root.slide(_root.container9);
        _root.slide(_root.container10);
        _root.slide(_root.container11);
        _root.slide(_root.container12);
        _root.slide(_root.container13);
        _root.slide(_root.container14);
} 

geht das einfacher? also ein script, statt das immer wieder zu wiederholen?

und die abstände der bilder sollen gleich bleiben d.h. ich muss das mit der gesch irgendwie ändern .. irgendwelche ideen ?
bis dann
danke nochmal für die antwort dachte schon ich bekomm keine mehr
bis später

wangman

p.s.: nein es waren keine else zweige und es stimmt was du sagst

[bloba]

also ich hab gard nur mal kurz drübergschaut und ich denke dass es hier sinnvoll ist mim einem movieclip.prototype zu arbeiten
dh du erstellst eine "funktion" welche du dann auf auf movieclips anwenden kannst.
dann wäre der aufruf nicht mehr

_root.slide(_root.container1);
 _root.slide(_root.container2);
//... 
 

sondern sauber in und kompakt

for (i=1;i<=anzahl;i++){
 _root["container"+i].slide();
}; 

wobei du die funktion "slide" eintprechend umändern musst

[wangman]

hm und wie änder ich die funktion um ?

function slide.prototype(mcmc) { 

so ?

danke für die hilfe schonmal

lg
wangman

[andretti]

aloha!

MovieClip.prototype.slide = function() {
        // Position maus
        _root.mp = _ymouse;
        // Bewegung rechts
        if (400<_root.mp) {
                this._y = this._y+_root.gesch;
                _root.bewegung_r = 1;
                _root.bewegung_l = 0;
        }
        // Bewegung links
        if (400>_root.mp) {
                this._y = this._y-_root.gesch;
                _root.bewegung_r = 0;
                _root.bewegung_l = 1;
        }
        // Sprung nach hinten
        if (this._y>=820 && _root.bewegung_r == 1) {
                this._y = -20;
        }
        // Sprung nach vorne
        if (this._y<=-20 && _root.bewegung_l == 1) {
                this._y = 820;
        }
        
        //Geschwindigkeits abfrage und festsetzung links
        if (_root.bewegung_l == 1) {
                if (_ymouse>=400) {
                        _root.gesch = 2.5;
                } else if (_ymouse>=300) {
                        _root.gesch = 5;
                } else if (_ymouse>=200) {
                        _root.gesch = 7.5;
                } else if (_ymouse>=100) {
                        _root.gesch = 10;
                }
        }
        //Geschwindigkeits abfrage und festsetzung rechts
        if (_root.bewegung_r == 1) {
                if (_ymouse<=500) {
                        _root.gesch = 2.5;
                } else if (_ymouse<=600) {
                        _root.gesch = 5;
                } else if (_ymouse<=700) {
                        _root.gesch = 7.5;
                } else if (_ymouse<=800) {
                        _root.gesch = 10;
                }
        }
}
//++++++++++++++++++++++++++++++
for (i=1;i<=anzahl;i++){
        _root["container"+i].slide();
}; 

[wangman]

danke!!
problem 1 wäre damit gelöst
nun zum zweiten problem
probiert mal den film aus dun fahrt rel lange mit der maus rel zügig hoch und runter und guckt was die bildchen machen.
die bewegen sich nämlich von ihrer position weg also der abstand zwischen den bildern bleibt nicht gleich
schauts euch mal an:

[andretti]

probieren ohne bilder?

[wangman]

ups.
ehm so ists besser:
das ganze soll nur n slider werden mit ner voransicht von bildern und wenn man auf ein icon klickt soll es halt das große bild laden samt überschrift und beschreibung. naja bis dahin is es noch n guter weg zumal das dann alles noch auf xml basieren soll, dass ich ohne großen aufstand dinge hinzufügen kann.

[Yi|]

hi,
bist du schon damit fertig? suche genau sowas, fände es super wenn du es hier rein stellen könntest

gruß

Yi