Code optimieren

[schrankwand]

Ich würde gerne folgendes ActionScript noch ein wenig zusammenfassen um die Flexiblität zu optimieren. Und ich würde gerne _root durch relative Adressierungen ersetzen. Hat vielleicht irgendjemand eine Idee wie ich das angehen könnte?

var menus = new Object();
menus.products = 0;
menus.company = 0;
menus.contact = 0;
//invisible Button
this.inv_btn.onRollOver = function() {
    for (var proberty in menus) {
        if (menus[proberty] == 1) {
            _root[proberty+"menu_mc"].gotoAndPlay('slideup');
            menus[proberty] = 0;
        }
    }
};
//slide function
function slide(slidedown:String, slideup:Array) {
    for (var button in slideup) {
        if (menus[slideup[button]] == 1) {
            _root[slideup[button]+"menu_mc"].gotoAndPlay("slideup");
            menus[slideup[button]] = 0;
        }
    }
    if (menus[slidedown] == 0) {
        _root[slidedown+"menu_mc"].gotoAndPlay("slidedown");
        menus[slidedown] = 1;
    }
}
//products menu
this.products_btn.onRollOver = function() {
    var down = "products";
    var up = new Array("contact", "company");
    slide(down, up);
};
//company menu
this.company_btn.onRollOver = function() {
    var down = "company";
    var up = new Array("contact", "products");
    slide(down, up);
};
//contact menu
this.contact_btn.onRollOver = function() {
    var down = "contact";
    var up = new Array("products", "company");
    slide(down, up);
}; 

[tolleiv]

hallo

also warum du dir so viel streß machst du hast hier meiner ansicht nach ein statisches menu das du dynamisch aussehen lassen willst

statisch darum weil du jedesmal wenn ein neuer hinzukommt in allen anderen punkten auch etwas ändern mußt und besonders flexibel ist das nicht

wie ich es aufbauen würde :

PHP-Code:

var menu = new Array("punkt1","punkt2",...,"punktn");
for(var btn in menu)
{
  mc = this.createEmptyMovieClip("btn_" + btn,btn);
  mc.target = menu[btn];
  mc.active = 0;
  mc.onRollOver = function()
  {
    for( var prob in this._parent.menu)
    {
       if(this._parent["btn_" + prob].active == 1) 
       {
          this._parent["btn_" + prob].gotoAndPlay("whatever");
          this._parent["btn_" + prob].active = 0;
       }
    }
  }
  mc.onRollOut = function()
  {
      ... und hier genauso ...
  }
} 


hat zwar auch seine nachteile aber ist zumindest das was ich unter dynamisch verstehe - da kannst du später dann auch locker ne XML-Datei o.ä. ranhängen ums noch dynamischer zu machen

und um (quell)code nicht irgendwo doppelt zu haben schreibst du alles was von mehreren verschiedenartigen elementen gebraucht wird in prototype's

mfg

[schrankwand]

Das würde nicht klappen, da sich das menü nicht beim rollOut des Buttons schon sondern erst beim RollOver eines der anderen beiden Button oder der unsichtbaren Schaltfläsche deaktivieren soll.

Außerdem bezweifel ich das das mit this._parent funktioniert.

[pape]

haste des mal ausprobiert @ schrankwand?

[schrankwand]

was meinste? das mit this_parent anstatt _root? also das hab ich ausprobiert sowie nur _parent und nur _root alleine und beides funktionierte nicht als ich es in meinem skript eingefügt habe.

[pape]

ne, dein Menü nach der Vorlage von tolleiv aufzubauen!
wie gesagt ist das Stichwort : "dynamisch"!
und die RollOvers kannste ja durch Das ersetzen, was du brauchst/willst.

Robert

[tolleiv]

ihr seit ja süß

@schrankwand - das war nur ein vorschlag den ich so oder ähnlich schon öfter genutzt habe was du in die funktionen reinpackst ist dein ding aber du solltest nicht einfach sagen das etwas nicht funktioniert wenn du es nicht versuchst
und das ich dir hier kein fertig gescriptetes menu hinpack wenn du nach optimierungsvorschlägen fragst sollte auch klar sein

mfg