Prøveeksamen

Tidemann held på med lab5 (Snake), men har gjort noko feil i steget der han skal teikna eit rutenett. Når test-programmet hans view_test.py køyrer, blir programmet til venstre vist, sjølv om han eigentleg skulle ønske at det såg ut som programmet til høgre. Det kjem ingen feilmeldingar i terminalen.

Kva har Tidemann gjort feil? Les koden hans, og forklar:

• kva han har gjort feil,
• kvifor feilen fører til åtferda som blir vist, og
• kva han kan gjera for å retta feila.

Maks 400 ord.

Tidemann har gjort flere feil:

1. I snake_view.py på linje 11 i funksjonen draw_board er det oppgitt at løkken skal gå gjennom elementene i range(rows). Dette fører til at det blir tegnet for få kolonner i rutenettet. For å rette feilen, endre linje 11 slik at løkken går igjennom elementene i range(cols) i stedet.

2. I snake_view.py, på linje 32-33 i funksjonen get_color angis det at fargen skal være oransje dersom den innkommende verdien er større enn eller lik 0. Dette fører til at funksjonen vil returnere fargen oransje selv om verdien er 0, som ikke er riktig oppførsel, og vi ser på bildet at hele brettet tegnes i den oransje fargen, også der det skulle vært grå. Dette problemet kan løses på (minst) tre ulike måter, og ideelt sett burde man gjøre alle disse rettelsene (selv om hvert enkelt grep i seg selv er egentlig er tilstrekkelig for rette funksjonaliteten):

   • I stedet for å sammenligne med >=, burde man sammenlignet med >. Da ville ikke kroppen til denne if-setningen bli utført, og det er istedet den lysegrå fargen som blir angitt til color-variabelen tidligere i funksjonen (linje 30-31) som gjør seg gjeldende.
   • I stedet for å angi verdi til en variabel color som returneres på slutten av funksjonen, kunne man returnert verdier direkte inne i if-blokkene. Da ville funksjonen returnert allerede i den første if-blokken.
   • I stedet for å benytte flere if-setninger, burde man benyttet seg av én if-setning med flere elif-ledd; altså endre if på linje 32 til elif. Da ville color-variabelen blitt angitt kun i det første leddet av if-elif-sekvensen.

3. I snake_view.py, på linje 22-26 i funksjonen draw_board tegnes debug-informasjon for en celle i en if-setning; men denne if-setningen har et innrykk for lite, og utføres derfor kun én gang for hver rad, etter at den innerste løkken er ferdig utført. Variablene x_left, x_right, y_top og y_bottom vil fortsatt ha de verdiene de fikk i siste iterasjon av den innerste løkken, som er ansvarlig for å tegne siste kolonne i raden. Derfor tegnes debug-informasjonen kun for den siste kolonne. For å rette koden, må innrykket på linjene 22-26 økes med ett innrykk (fire mellomrom).

• For få kolonner (2 poeng)
  • Identifiserer at dette er et problem.
  • Foreslår funksjonell løsning for hva som kan gjøres for å rette problemet.

• For mye oransje (2 poeng)
  • Identifiserer at dette er et problem.
  • Foreslår funksjonell løsning for hva som kan gjøres for å rette problemet.

• Debug vises kun for siste kolonne (2 poeng)
  • Identifiserer at dette er et problem.
  • Foreslår funksjonell løsning for hva som kan gjøres for å rette problemet.

• Helhetsvurdering (4 poeng)
  • Meningsfulle forklaringer av sammenheng mellom kode og resultatbilde.
  • Gode bruk av faguttrykk.
  • Finner ikke feil som ikke er feil.

def count_s(string):
    for c in range(len(string)):
        count = 0
        if c == 's': 
            count += 1
        return count 

Forklar:

• Hva er galt?
• Hvordan kan du korrigere programmet slik at det fungerer? Forklar nøyaktig hvor og hvordan du har tenkt å gjøre korrigeringen.

Omtrent to til tre avsnitt, helst ikke mer enn 200 ord.

Koden har flere feil.

• Løkken blander indekser med tegn i strengen. Løkken er over indekser (den bruker range(len(...))), men iteranden c blir brukt som om den er et symbol i strengen (f. eks. c == "a"). For å reparerer dette, må vi endre for c in range(len(s)): til for c in s:.

• count blir satt til 0 på nytt hver gang blokken i løkken kjører. Dette er fordi count blir definert inni løkken. For å reparerer dette, må vi flytte count = 0 utenfor løkken (før løkken starter).

• return count blir kjørt allerede etter første gjennomgang av løkken, og kodeblokken i løkken utføres derfor bare én gang. Når return -setningen utføres inne i en løkke, avbrytes løkken og funksjonskallet med én gang. For å reparerer dette, må vi endre innrykket og flytte return count utenfor løkken (etter at løkken er ferdig).

Reparert versjon (løsning trenger ikke inkludere dette for å få full pott):

def count_a(s):
    count = 0
    for c in s:
        if c == "a":
            count += 1
    return count

print(count_a("banan"))

• 3 poeng: count = 0 må flyttes utenfor løkken.
• 2 poeng: Løkken blander sammen indekser og elementer i strengen.
• 3 poeng: For tidlig return
• 2 poeng: Skjønnsmessig vurdering. Bruker kandidaten gode faguttrykk? Er det sammenheng mellom resonnementer?