[Flash MX] Script

[msentinelb]

Ik ben sinds kort begonnen met flash mx actionscripts te maken. gaat steeds beter alleen nu zit ik vast.
http://home.quicknet.nl/qn/prive/m.bays/amd.htm

de code hieronder heb ik geschreven. alleen er zit een fout in en die krijg ik maar niet goed. Ik wil dat als er nix word ingevult, dat er dan ERROR komt te staan. dit doet hij alleen als er al een keer wat ingevult is en dan welgehaald word. Als je de pagina laad en gelijk op berekenen drukt komt er -500 te staan bij Y2 . weet ìemand hoe ik dit kan verhelpen zodat er altijd ERROR bij Y, Y2 komt te staan als een X of X2 leeg gelaten word?

even voor de duidelijkheid:
X = Megahertz Stock
X2 = Megahertz OCed
Y = Rating Stock
Y2 = Rating

code:

on (release) {
    if (X2 == "") {
        Y2 = "ERROR";
    } else {
        Y2= ("Athlon XP ")+(X2*1.5-500)+("+");
    }
    if (X == "") {
        Y = "ERROR";
    } else if (X == "1470") {
        Y = "Athlon XP 1700+";
    } else if (X == "1530") {
        Y = "Athlon XP 1800+";
    } else if (X == "1600") {
        Y = "Athlon XP 1900+";
    } else if (X == "1670") {
        Y = "Athlon XP 2000+";
    } else if (X == "1730") {
        Y = "Athlon XP 2100+";
    } else if (X == "1800") {
        Y = "Athlon XP 2200+";
    } else if (X == "1400") {
        Y = "Athlon XP 1600+";
    }
}

Indien nodig de FLA:
http://home.quicknet.nl/qn/prive/m.bays/amd.fla

[Verwijderd]

code:

if (X2 == undefined or X2 == "") {
 // ERROR
}

[msentinelb]

Mooi ja het werkt Die -500 is weg . alleen die "error" verschijnt helaas niet . als X2 leeg is.

Als X leeg is moet bij Y Error staan
en/of als X2 leeg is moet er bij Y2 Error staan

http://home.quicknet.nl/qn/prive/m.bays/amd.htm

code:

on (release) {
    if (X2== undefined or X2== "") {
        // ERROR
    } else if (Y2= ("Athlon XP ")+(X2*1.5-500)+("+")) {
    }
    if (X == "1400") {
        Y = "Athlon XP 1600+";
    } else if (X == "1470") {
        Y = "Athlon XP 1700+";
    } else if (X == "1530") {
        Y = "Athlon XP 1800+";
    } else if (X == "1600") {
        Y = "Athlon XP 1900+";
    } else if (X == "1670") {
        Y = "Athlon XP 2000+";
    } else if (X == "1730") {
        Y = "Athlon XP 2100+";
    } else if (X == "1800") {
        Y = "Athlon XP 2200+";
    }
}

[ Voor 20% gewijzigd door msentinelb op 13-04-2003 12:48 ]

[Verwijderd]

Flash ActionScript:

on (release) {
    if (X2== undefined or X2== "") {
        // ERROR
    } else if (Y2= ("Athlon XP ")+(X2*1.5-500)+("+")) {
    }
    if(X == undefined | X==''){
        //error
    }else if (X == "1400") {
        Y = "Athlon XP 1600+";
    } else if (X == "1470") {
        Y = "Athlon XP 1700+";
    } else if (X == "1530") {
        Y = "Athlon XP 1800+";
    } else if (X == "1600") {
        Y = "Athlon XP 1900+";
    } else if (X == "1670") {
        Y = "Athlon XP 2000+";
    } else if (X == "1730") {
        Y = "Athlon XP 2100+";
    } else if (X == "1800") {
        Y = "Athlon XP 2200+";
    }
}

[msentinelb]

THX hij doet het nu perfect
Kleine verandering:

code:

on (release) {
    if (X2 == undefined or X2 == "") {
        Y2 = "ERROR";
    } else if (Y2=("Athlon XP ")+(X2*1.5-500)+("+")) {
    }
    if (X == undefined | X == '') {
        Y = "ERROR";
    } else if (X == "1400") {
        Y = "Athlon XP 1600+";
    } else if (X == "1470") {
        Y = "Athlon XP 1700+";
    } else if (X == "1530") {
        Y = "Athlon XP 1800+";
    } else if (X == "1600") {
        Y = "Athlon XP 1900+";
    } else if (X == "1670") {
        Y = "Athlon XP 2000+";
    } else if (X == "1730") {
        Y = "Athlon XP 2100+";
    } else if (X == "1800") {
        Y = "Athlon XP 2200+";
    }
}

Maar dat lag denk omdat ik het wat onduidelijk vertelde toch bedankt allemaal.

[GrimaceODespair]

Mijn brein begint meestal te jeuken als ik zoiets zie. En als mijn brein jeukt, dan moet ik reageren Het is zeker niet slecht geprogrammeerd, maar ik was toch wel blij toen MM met het switch statement kwam:

Flash ActionScript:

on (release) {

    [...]

    switch (X){
    case "1400":
        Y = "Athlon XP 1600+";
        break;
    case "1470":
        Y = "Athlon XP 1700+";
        break;
    case "1530":
        Y = "Athlon XP 1800+";
        break;
    case "1600":
        Y = "Athlon XP 1900+";
        break;
    case "1670":
        Y = "Athlon XP 2000+";
        break;
    case "1730":
        Y = "Athlon XP 2100+";
        break;
    case "1800":
        Y = "Athlon XP 2200+";
        break;
    default:
        //error
    }
}

Het is denk ik niet sneller, maar toch een stuk leesbaarder. Nu ik er zo over nadenk, zou het toch wel sneller kunnen zijn door een verschil in de Flash Bytecodes, maar dat moet je zelf dan maar testen

En omdat ik eigenwijs en ego-patserig ben, doe ik je nog een alternatief van de hand dat, vind ik persoonlijk, onderhoudbaarder is:

Flash ActionScript:

var AthlonList = {_1400 : 1600, _1440: 1700, _1530: 1800, _1600: 1900, _1670: 2000, _1730: 2100, _1800: 2200};
var XPRating = AthlonList["_" + X];
if (XPRating == undefined) {
    // error
} else {
    Y = "AthlonXP " + XPRating + "+";
}

[msentinelb]

hmm ik zal ze is uitproberen. kijken of ik het werkend krijg