Gennemse løsningen for at forbedre udfordringen med kodelæsbarhed

Fuldført

Følgende kode er en mulig løsning på udfordringen fra det forrige undermodul.

/*
   This code reverses a message, counts the number of times 
   a particular character appears, then prints the results
   to the console window.
 */

string originalMessage = "The quick brown fox jumps over the lazy dog.";

char[] message = originalMessage.ToCharArray();
Array.Reverse(message);

int letterCount = 0;

foreach (char letter in message)
{
    if (letter == 'o')
    {
        letterCount++;
    }
}

string newMessage = new String(message);

Console.WriteLine(newMessage);
Console.WriteLine($"'o' appears {letterCount} times.");

Denne kode er blot "én mulig løsning". Du har muligvis fundet forskellige variabelnavne og forskellig lodret afstand og indrykning af tabulatorer. Her er en liste over de ændringer, der er foretaget.

  • Koden indeholder en beskrivelse på højere niveau af, hvad hele kodelisten forsøger at opnå i en kommentar med flere linjer øverst. Du kan hævde, at dette er en lille forbedring i forhold til de oprindelige kodekommentarer, men i betragtning af udfordringens beskrivelse af koden var der ikke meget mere kontekst tilgængelig.
  • De enkelte kommentarer blev fjernet, fordi de ikke gav nogen reel indsigt i kodens formål eller funktion.
  • Der blev tilføjet flere tomme linjer for at forbedre kodelistens udtryk. Hold kodelinjer sammen, når de ligner hinanden, eller når de arbejder sammen for at udføre en lille opgave.
  • Der blev tilføjet stregfeeds og faner for at forbedre sætningens foreach og sætningens if udseende.
  • Navngivningskonventioner for lokale variabler blev anvendt for bedre at formidle formålet med hver værdi.

Hvis du har identificeret de samme problemer og løst dem på samme måde, tillykke! Fortsæt til videnstjekket i det næste undermodul.

Vigtige oplysninger

Hvis du havde problemer med at fuldføre denne udfordring, skulle du måske gennemse de tidligere enheder, før du fortsætter.