Zoek de fouten

Volgende bugs, fouten, minder goede oplossingen komen uit oplossingen van vaardigheidsproeven. Kan jij ontdekken wat er mis? De oplossingen staan achteraan dit document. (de code is hier en daar ingeperkt om de focus op de fout te leggen)

Opgaven

  1.  if (!(e >= 0 && e <= 9))
 {
     Console.WriteLine("Los het dan zelf op he!");
 }
 if (e >= 0 && e <= 9)
 {

  2.  if ( Foutcode < 0 )
     Console.WriteLine("Los het dan zelf op he!");
 if (Foutcode > 9)
     Console.WriteLine("Los het dan zelf op he!");

  3.  if (Foutcodex >= 0 && < 10)

  4.  if ("0" == Answer || "1" == Answer || "2" == Answer || "3" == Answer || "4" == Answer || "5" == Answer || "6" == Answer || "7" == Answer || "8" == Answer || "9" == Answer)
 {
     int Foutcode = Convert.ToInt32(Answer);

  5.  double aantalMin = Convert.ToDouble(Math.Sqrt(foutcode));

  6.  int prijs = nummer.Next(0, 100);
 if (prijs <= 25)
 {
     Console.WriteLine("en u wint ook nog eens 1 jaar gratis IT support");
 }
 else if (prijs >= 26 && prijs <= 100)"

  7.  aantalMin = Math.Round(Math.Sqrt((foutcode * 3 + 0.0)), 1);

  8.  if (fout <0 && fout>=9 )
 {
     Console.WriteLine( " los het zelf op");

  9.  computer = Convert.ToString(Console.ReadLine());

  10. else if (Computer == "nee")
{
    Console.WriteLine("Gaat het power light aan?");
    Console.ReadLine();
    string light;
    light = Convert.ToString(Console.ReadLine());
}
if (light == "j")
{

  11. if (proccount == 1)
{
    Console.WriteLine("50 euro");
}
else if (proccount == 2)
{
    Console.WriteLine("100 euro");
}
else if (proccount == 3)
{
    Console.WriteLine("150 euro");
}
else if (proccount >= 5)
{
    Console.WriteLine("200 euro");
}

  12. if (is64bit)
{
    Console.WriteLine("Hier een bon!");
}
else
{
    //Niets
}

  13. if (kans == 1)
{
    Console.WriteLine("Mooi zo, alles werkt.");
    Console.WriteLine("En u wint ook nog eens 1 jaar gratis IT support!");
}
else
{
    Console.WriteLine("Mooi zo, alles werkt.");
}

  14. if(!! Environment.Is64BitProcess)

Oplossing

  1. De tweede if is onafhankelijk van de eerste. Sowieso is het veiliger om met een if..else if te werken. In dit geval is nog beter om met een if..else te `werken daar de tweede if de coplementaire van de eerste is. Door deze nu alsnog 'voluit' te schrijven verhoog je de kans op bug doordat je bijvoorbeeld een getal aan de grens nu in beide ifs niet opvangt.

  2. Indien beide if'n zelfde code bevatten kan je deze allebei in if zetten. Zo hoef je maar op 1 plek aanpassingen te doen in de toekomst.

  3. Je moet steeds de volledige logica uitschrijven. Wat moet in deze if kleiner zijn dan 10? De goede lezer weet wel dat het om Foutcodex gaat, maar de compiler is zo dom als insert grappig dier/mens/idee.

  4. Ten eerste willen we geen string vergelijken, maar ints. Ten tweede: wat als alle getallen tussen 1 en 100 toegelaten werden?!

  5. De Math.Sqrt geeft altijd een double terug, dus deze moet niet geconverteerd worden naar een double.

  6. De tweede else if vervang je beter door een else.

  7. Waarom +0.0? Ik vermoed om een double aan Sqrt te geven, wat niet hoeft, daar int automatisch reeds naar double zullen worden omgezet indien nodig.

  8. Deze if zal nooit uitgevoerd worden. Geen enkel getal kan zowel kleiner dan 0 zijn en tegelijkertijd groter dan 9.

  9. ReadLine geeft altijd een string terug, en moet dus niet geconverteerd worden naar een string.

  10. Een huzarenstukje, met aardig wat bug.

    • Een ReadLine direct na een vraag (via WriteLine) moet je gebruiken om invoer van de gebruiker in te bewaren, niet om een enter op het scherm te plaatsen. De gebruiker zal een antwoord willen invoeren en niet verwachten dat hij eerst op enter moet duwen én dan pas zijn antwoord invoeren.
    • De ReadLine methode geeft een string terug en moet dus niet geconverteerd worden.
    • De variabele light heeft als scope de acolades binnen de else if. Je kan deze dus niet gebruiken in de volgende if .

  11. De prijs is het resultaat van een berekening. Je kan dus voor proccount tussen 1 en 3 werken met prijs = proccount*50 . (Wat met 4 trouwens?)

  12. Indien je een lege else hebt kan je die ook beter gewoon weglaten.

  13. Iets subtieler: de zin Mooi zo, alles werkt. komt in zowel de if als de else voor. Het is dus veiliger om deze zin VOOR de if..else structuur te zetten (zo hoef je hem maar op 1 plek aan te passen als deze in de toekomst anders moet zijn). De else vervalt op deze manier trouwens.

  14. Een klassieke fout. De ! (not) mag je niet dubbel plaatsen zoals && en ||. Twee maal not is gewoon not not: beide nots neutraliseren mekaar (zelfde als zeggen 'min min vier' wat gewoo 'plus vier' is) waardoor er eigenlijk staat: if(!! Environment.Is64BitProcess)

Oplossing opgave

Oplossing hier met enum , maar dit had je uiteraard ook met een string (die dan de waarden "A","B","C" gaf) of een int (waarden 1,2,3) gewerkt.

enum fases { A, B, C, Onbekend }

static void Main(string[] args)
{
    fases finalFase= fases.Onbekend;

    //Stap 1
    Console.WriteLine("Zet de computer aan");
    Console.WriteLine("Gaat de computer aan?");
    string invoer = Console.ReadLine();
    Console.Clear();
    if (invoer == "ja") //Veronderstelt dat er enkel ja/nee correct wordt ingevoerd
    {

        Console.WriteLine("Zijn er fout boodschappen?");
        invoer = Console.ReadLine();
        Console.Clear();
        if (invoer == "ja")
            finalFase = fases.A;
        else
            finalFase = fases.B;
    }
    else
    {
        Console.WriteLine("Gaat het power light aan?");
        invoer = Console.ReadLine();
        Console.Clear();
        if (invoer == "ja")
        {
            Console.WriteLine("Zet het computer scherm aan");
        }
        else
        {
            Console.WriteLine("Controleer de voedingskabel.");
        }
        Console.WriteLine("Probleem opgelost?");
        invoer = Console.ReadLine();
        Console.Clear();
        if (invoer == "ja")
            finalFase = fases.B;
        else finalFase = fases.C;
    }

    //Stap 2 - fase A

    if (finalFase == fases.A)
    {
        Console.WriteLine("Geef de foutcode tussen 0 en 9");
        int foutcode = Convert.ToInt32(Console.ReadLine());
        if (foutcode >= 0 && foutcode <= 9)
        {
            double X = Math.Sqrt(foutcode * 3);
            Console.ForegroundColor = ConsoleColor.Red;
            Console.WriteLine($"Gelieve je computer gedurende {X:F1} minuten af te zetten");
            Console.ResetColor();
        }
        else
        {
            Console.WriteLine("Loes het dan zelf op he!");
        }
    }

    //Stap 2 - fase B

    else if (finalFase == fases.B)
    {
        Console.WriteLine("Mooi zo, alles werkt.");
        Random dobbel = new Random();
        if (dobbel.Next(0, 4) == 0)
            Console.WriteLine("En u wint ook nog eens 1 jaar gratis IT support!");
    }

    //Stap 2 - fase C

    else if (finalFase == fases.C) //mag ook gewone else zijn
    {
        int proccount = Environment.ProcessorCount;
        bool bit64 = Environment.Is64BitOperatingSystem;

        if (proccount == 1)
        {
            Console.ForegroundColor = ConsoleColor.White;
            Console.BackgroundColor = ConsoleColor.Red;
            Console.WriteLine("1 processor");
            Console.ResetColor();
        }
        else 
        {
            Console.ForegroundColor = ConsoleColor.Blue;
            Console.BackgroundColor = ConsoleColor.Green;
            Console.WriteLine($"{proccount} processor");
            Console.ResetColor();
        }
        if (proccount >= 5)
            Console.WriteLine("200 euro");
        else
            Console.WriteLine($"{proccount * 50} euro");

        if (bit64)
        {
            Console.WriteLine("Hier een bon!");
        }
    }
}

results matching ""

    No results matching ""