lørdag, januar 23, 2021

Hjælp, kode fejl?

Daily Rush Debat Programmering Hjælp, kode fejl?

  • Forfatter
    Emne
  • #0

    Zacker
    Bruger
    586 indlæg
    Offline

    Alt kode skulle ellers være færdigt og gennemtestet nu. Men der er 2 store bugs i øjeblikket. Variabler win2 og tab bliver altid De radikale af en eller anden underlig grund. Kan i se nogen fejl i disse kode linjer?

    // Hvilket parti har flest point og
    // vil udfylde “win” variablen
    if (K>=V && K>=A && K>=Ø && K>=O && K>=Q && K>=R) {
    win = “Konservativt Folkeparti”;
    } else if (V>=A && V>=Ø && V>=O && V>=Q && V>=R) {
    win = “Venstre”;
    } else if (A>=Ø && A>=O && A>=Q && A>=R) {
    win = “Socialdemokratiet”;
    } else if (Ø>=O && Ø>=Q && Ø>=R) {
    win = “Enhedslisten”;
    } else if (O>=Q && O>=R) {
    win = “Dansk Folkeparti”;
    } else if (Q>=R) {
    win = “Kristeligt Folkeparti”;
    } else {
    win = “De Radikale”;
    }
    // Hvilket parti har fået mindst point og
    // vil udfylde “tab” variablen
    if (KV && K>A && K>Ø && K>O && K>Q && K>R)) {
    win2 = “Konservativt Folkeparti”;
    }
    else if ((VA && V>Ø && V>O && V>Q && V>R)) {
    win2 = “Venstre”;
    }
    else if ((A
    Ø && A>O && A>Q && A>R)) {
    win2 = “Socialdemokratiet”;
    }
    else if ((Ø
    O && Ø>Q && Ø>R)) {
    win2 = “Enhedslisten”;
    }
    else if ((O
    Q && O>R)) {
    win2 = “Dansk Folkeparti”;
    }
    else if ((Q
    R)) {
    win2 = “Kristeligt Folkeparti”;
    } else {
    win2 = “De Radikale”;
    }

    Michael 'Zacker' Schmidt | www.thefuture.dk - Level Designer at IO Interactive | www.ioi.dk

Viser 5 kommentarer - 1 til 5 (af 5 i alt)
  • Forfatter
    Kommentarer
  • #1

    pred
    Bruger
    1.520 indlæg
    Offline

    tja “tab” delen giver ingen videre mening, med en masse startede paranteser and so.

    #2

    Zacker
    Bruger
    586 indlæg
    Offline

    // Hvilket parti har flest point og
    // vil udfylde “win” variablen
    if (K>=V && K>=A && K>=Ø && K>=O && K>=Q && K>=R) {
    win = “Konservativt Folkeparti”;
    } else if (V>=A && V>=Ø && V>=O && V>=Q && V>=R) {
    win = “Venstre”;
    } else if (A>=Ø && A>=O && A>=Q && A>=R) {
    win = “Socialdemokratiet”;
    } else if (Ø>=O && Ø>=Q && Ø>=R) {
    win = “Enhedslisten”;
    } else if (O>=Q && O>=R) {
    win = “Dansk Folkeparti”;
    } else if (Q>=R) {
    win = “Kristeligt Folkeparti”;
    } else {
    win = “De Radikale”;
    }
    // Hvilket parti har fået mindst point og
    // vil udfylde “tab” variablen
    if (KV && K>A && K>Ø && K>O && K>Q && K>R)) {
    win2 = “Konservativt Folkeparti”;
    }
    else if ((VA && V>Ø && V>O && V>Q && V>R)) {
    win2 = “Venstre”;
    }
    else if ((A
    Ø && A>O && A>Q && A>R)) {
    win2 = “Socialdemokratiet”;
    }
    else if ((Ø
    O && Ø>Q && Ø>R)) {
    win2 = “Enhedslisten”;
    }
    else if ((O
    Q && O>R)) {
    win2 = “Dansk Folkeparti”;
    }
    else if ((Q
    R)) {
    win2 = “Kristeligt Folkeparti”;
    } else {
    win2 = “De Radikale”;
    }

    Michael 'Zacker' Schmidt | www.thefuture.dk - Level Designer at IO Interactive | www.ioi.dk

    #3

    Zacker
    Bruger
    586 indlæg
    Offline

    øhhh af underlige grunde er de ikke kommet med i kopieringen. De er der i virkeligheden, men ellers mange tak!

    Michael 'Zacker' Schmidt | www.thefuture.dk - Level Designer at IO Interactive | www.ioi.dk

    #4

    PBT
    Admin
    3.900 indlæg
    Offline

    Zacker,

    hvilket sprog er i din kode skrevet i? Javascript?

    Generelt gælder det om at gøre så nemt for sig selv som muligt, når man programmerer. Som du kan se, bliver koden hurtigt uoverskuelig, hvis du bruger en masse if/else if sætninger. Derudover bliver det besværligt at tilføje f.eks. et nyt parti, og du vil nemt kunne lave små fejl, når du retter i koden.

    Afhængigt af hvilket sprog du bruger, vil det være nemmere at lave enten en klasse, der indeholder parti og antal stemmer, og derefter fylde et array med objekter af den klasse og sortere det, eller lave f.eks. et associativt array/hash array/hashmap eller lignende, og sortere efter værdien (antallet af stemmer).

    Skriv tilbage hvilket sprog du bruger, og jeg skal give dig et eksempel.

    PBT

    Startede Daily Rush og Quake3.dk sammen med Webster i et andet årtusind

    #5

    PBT
    Admin
    3.900 indlæg
    Offline

    Jeg kan se på et andet indlæg, at det er ActionScript/Flash du arbejder i; det kan jeg desværre ikke lige hjælpe dig med. Du skal dog stadig prøve at komme af med de mange if-sætninger, ellers bliver din kode ekstremt rodet og svær at gennemskue. Prøv at undersøge hvad sproget har af muligheder for at sortere arrays eller helst associative arrays (som er arrays hvor du kan sige arr[“K”] = 35).

    Startede Daily Rush og Quake3.dk sammen med Webster i et andet årtusind

Viser 5 kommentarer - 1 til 5 (af 5 i alt)
  • Du skal være logget ind for at kommentere på dette indlæg.