Kann man das nicht auch kürzer schreiben

[Artworx3D]

Hallo, Ich habe hier folgendes AS, was tadellos funktioniert, aber sehr lang und warscheinlich umständlich geschrieben ist:

on (release) {
    _root.menue_bg.gotoAndStop (23);
    _root.kreis.rechteck.gotoAndStop (1);
    if (_root.rechteck == 1) // Quadrat, also ein abstand von 6 Farben von Ecke zu Ecke
    {
    for(z=1; z<=4; ++z){
        _root.kreis.auswahl["a" + z + "_col"] = new Color (_root.kreis.auswahl["a" + (z)]);
        if (z==1){
        _root.kreis.auswahl["a" + z + "_col"].setRGB(_root["farbkreis_f"][21]); } // es giebt 24 Farben, da aber Farbkreis, geht es nach der 24. Farbe wieder bei der ersten los
        if (z==2){
        _root.kreis.auswahl["a" + z + "_col"].setRGB(_root["farbkreis_f"][3]); }
        if (z==3){
        _root.kreis.auswahl["a" + z + "_col"].setRGB(_root["farbkreis_f"][9]); }
        if (z==4){
        _root.kreis.auswahl["a" + z + "_col"].setRGB(_root["farbkreis_f"][15]); }
    }
    }
    else if (_root.rechteck == 2) // Rechteck mit den Abständen 4 und 8 von Ecke zu Ecke
    {
        for(z=1; z<=4; ++z){
        _root.kreis.auswahl["a" + z + "_col"] = new Color (_root.kreis.auswahl["a" + (z)]);
        if (z==1){
        _root.kreis.auswahl["a" + z + "_col"].setRGB(_root["farbkreis_f"][22]); }
        if (z==2){
        _root.kreis.auswahl["a" + z + "_col"].setRGB(_root["farbkreis_f"][2]); }
        if (z==3){
        _root.kreis.auswahl["a" + z + "_col"].setRGB(_root["farbkreis_f"][10]); }
        if (z==4){
        _root.kreis.auswahl["a" + z + "_col"].setRGB(_root["farbkreis_f"][14]); }
    }
    }
    else if (_root.rechteck == 3) //Rechteck mit den Abständen 2 und 12 von Ecke zu Ecke
    {
    for(z=1; z<=4; ++z){
        _root.kreis.auswahl["a" + z + "_col"] = new Color (_root.kreis.auswahl["a" + (z)]);
        if (z==1){
        _root.kreis.auswahl["a" + z + "_col"].setRGB(_root["farbkreis_f"][23]); }
        if (z==2){
        _root.kreis.auswahl["a" + z + "_col"].setRGB(_root["farbkreis_f"][1]); }
        if (z==3){
        _root.kreis.auswahl["a" + z + "_col"].setRGB(_root["farbkreis_f"][11]); }
        if (z==4){
        _root.kreis.auswahl["a" + z + "_col"].setRGB(_root["farbkreis_f"][13]); }
    }
    }
        } 

kann man das nicht Besser machen?


AWX

[merlinvs]

moin!
ich habe nicht so viel ahnung von AS...

... aber wenn ich mir deine datei mal angucke, ist das aber alles ziemlich konfus aufgebaut, finde ich, undzwar, weil du da ziemlich viele objekte hast, und so ziemlich alles hat ein eigenes actionscript. so oft wie ich da irgendwo stop(); gesehen habe, wozu machst du das?
bei einem gotoAndStop(); befehl, braucht man das doch nicht...

naja, es ist auch irgendwie ziemlich schwer, da die zusammenhänge drin zu finden...
joa, und hm, wo ist in dem flashfilm denn das script, was du gepostet hast?
zu dem script, man könnte aus den for schleifen eine machen, und das if in die schleife setzen, aber ob das sinvoll ist, weis ich nicht...

ich ahb mal die schleifen als eine gemacht, und 2 functionen gemacht... so ist es etwas einfacher, aber mal gucken, ich denke, es geht noch viel einfacher...

on (release) {
    _root.menue_bg.gotoAndStop(23);
    _root.kreis.rechteck.gotoAndStop(1);
    for (z=0; z<4; ++z) {
        if (_root.rechteck == 1) {
            // Quadrat, also ein abstand von 6 Farben von Ecke zu Ecke
            // setCOLOR
            setColor(z)
            // setRGB
            if (z == 1) {
                funcSetRgb(z,"21");
            }
            if (z == 2) {
                funcSetRgb(z,"3");
            }
            if (z == 3) {
                funcSetRgb(z,"9");
            }
            if (z == 4) {
                funcSetRgb(z,"15");
            }
        } else if (_root.rechteck == 2) {
            // Rechteck mit den Abständen 4 und 8 von Ecke zu Ecke
            // setCOLOR
            setColor(z)
            // setRGB
            if (z == 1) {
                funcSetRgb(z,"22");
            }
            if (z == 2) {
                funcSetRgb(z,"2");
            }
            if (z == 3) {
                funcSetRgb(z,"10");
            }
            if (z == 4) {
                funcSetRgb(z,"14");
            }
        } else if (_root.rechteck == 3) {
            //Rechteck mit den Abständen 2 und 12 von Ecke zu Ecke
            // setCOLOR
            setColor(z)
            // setRGB
            if (z == 1) {
                funcSetRgb(z,"23");
            }
            if (z == 2) {
                funcSetRgb(z,"1");
            }
            if (z == 3) {
                funcSetRgb(z,"11");
            }
            if (z == 4) {
                funcSetRgb(z,"13");
            }
        }
    }
}
function setColor(col_z){
    _root.kreis.auswahl["a"+col_z+"_col"] = new Color(_root.kreis.auswahl["a"+(col_z)]);
}
function setRgb(rgb_z, nummer){
    _root.kreis.auswahl["a"+rgb_z+"_col"].setRGB(_root["farbkreis_f"][nummer]);
} 

bis dann - Merlin

[hgseib]

on (release) {
    _root.menue_bg.gotoAndStop(23);
    _root.kreis.rechteck.gotoAndStop(1);
    var vCAry = [[21, 3, 9, 15], [22, 2, 10, 14], [23, 1, 11, 13]];
    for (z=1; z<=4; ++z) {
        ausw = "a"+z+"_col";
        _root.kreis.auswahl[ausw] = new Color(_root.kreis.auswahl["a"+(z)]);
        pos = vCAry[_root.rechteck-1][z-1];
        _root.kreis.auswahl[ausw].setRGB(_root["farbkreis_f"][pos]);
    }
} 

[merlinvs]

wow, ok, an arrays hab ich gedacht, und auch schon angefangen, aber das das so sehr geht...

[Artworx3D]

Uii, das nenn ich mal komprimiert. vielen Dank. genau an sowas hab ich gedacht. ich werd das gleich mal ausprobieren. bin grad dabei mein fla etwas aufzuräumen.
Aber das sieht alles sehr gut aus!

AWX

[Artworx3D]

beim ausprobieren is mir jez was aufgefallen. da der "code" noch nich ganz fertig war, als ich ihn gepostet habe*sehr boese* mit dem neuen "Supa"code was nicht funktioniert.
Und zwar: wenn _root.rechteck == 2 bzw 3 ist, im _root.kreis.rechteck.gotoAndStop (18 bzw. 30) angesteuert werden soll. - bei _root.rechteck==1 war das ja (1) - dass kann ich in den kurzen code leider nicht selber reinbauen, da ich noch versuche ihn vollständig zu verstehen.

zur sicherheit habe ich die aktuelle Version von meinem noch "schrott" FLA mal hochgeladen. (inzwischen funktionieren alle Rechtecke mit allen Farben)

F.Y.I. das is übrigens ein Grundlagen Hochschul Projekt, in dem die Technik von den Prof`s her erstmal gar keine Rolle spielt. Da ich so überhaupt keine ahnung habe freue ich mich sehr, dass ihr helft mein Flasch-Stümpertum zu reduzieren!

Danke aber für die äusserst lehrreichen Tips!

[hgseib]

der zusammenhang ist doch offensichtlich. dein code wiederholt sich, mit ausnahme der zahlen. also brauchst du doch nur die zahlenwerte zu indezieren.

on (release) {
    _root.menue_bg.gotoAndStop(2);
    // array für = [_root.rechteck == 1 , _root.rechteck == 2 , _root.rechteck == 3]
    // je inneres array = [gotoAndStop, z == 1,z == 2,z == 3,z == 4]
    var vCAry = [[4, 0, 6, 12, 18], [9, 0, 4, 12, 16], [21, 0, 2, 12, 14]];
    for (z=1; z<=4; ++z) {
        pos = vCAry[_root.rechteck-1];
        _root.kreis.rechteck.gotoAndStop(pos[0]);
        ausw = "a"+z+"_col";
        _root.kreis.auswahl[ausw] = new Color(_root.kreis.auswahl["a"+z]);
        _root.kreis.auswahl[ausw].setRGB(_root["farbkreis_f"][pos[z]]);
    }
} 

[Artworx3D]

Hmm, Irgendwie funktioniert das nicht.
Die Farben kommen nicht richtig in die Auswahl (die erste Fläche wird mit der 2.Farbe gefüllt und dadurch erhält die 4. gar keine?!) und ich kann nicht das keyframe angeben, in dem die richtige position für das rechteck mit den "Pointern" ist.
Für Rechteck 1 is das auch frame 1 bei rechteck 2 ist das Frame 14 und bei Rechteck 3 Frame 30. (bei dieser Farbe).
Auf reder der Farben liegt so ein script und die rechtecke haben eine bestimte position im _root.kreis.rechteck...

Da ich noch ein beginner im scripten bin fällt es mit noch nicht so leicht durch die verschachtelungen der arrays zu steigen.
Mit deinem Code auf Lila sieht dass dann so aus:

[Artworx3D]

also so klappts jez:

on (press) {
    _root.menue_bg.gotoAndStop (23);
    var vCAry = [[1, 21, 3, 9, 15], [18, 21, 1, 9, 13], [30, 21, 23, 9, 11]];
    actual = vCAry[_root.rechteck-1];
    _root.kreis.rechteck.gotoAndStop(actual[0]);    
    for(z=1; z<=4; ++z){
        _root.kreis.auswahl["a" + z + "_col"] = new Color (_root.kreis.auswahl["a" + (z)]);
        _root.kreis.auswahl["a" + z + "_col"].setRGB(_root["farbkreis_f"][actual[z]]);
    }
} 

Danke alle