C# Programmering

Jag har skrivit så tydligt jag kan hur jag tycker att uppgiften ska lösas. Du får själv översätta till C#. Jag skulle kunna ge dig koden direkt men då gör jag bara läxan åt dig och du lär dig ingenting av det. Säg dock gärna till om det är något specifikt du inte reder ut.

static void Main(string[] args)
        {
            int number= 0;    

            for(int i = 0; i < 3; i++)
            {
                Console.WriteLine("Skriv in nummer " + (i+1));
                number  = int.Parse(Console.ReadLine());              
            }
            Console.WriteLine("Summan blev " + number);
            Console.WriteLine("Medelvärdet blev " + number/ 3);
            Console.ReadKey();
        }

Såhär hade jag löst det.

1. "summa" kommer alltid att vara sista inmatade talet gånger 3.

2. Som sagts tidigare, kolla upp Int.TryParse() för nu har du ingen möjlighet att kontrollera att inmatningen är korrekt.

3. "counter" gör ingenting och har ingen funktion i den här lösningen.

4. Personlig åsikt men din första lösning var vettigare.

static void Main(string[] args)
        {
            int totalNumber = 0;
            int totalInput = 0;

            for(totalNumber = 0; totalInput < 3; totalInput++)
            {
                Console.WriteLine("Skriv in ett nummer");
                int userInputNumber = int.Parse(Console.ReadLine());
                totalNumber += userInputNumber;
            }
            Console.WriteLine("Summan blev " + totalNumber);
            Console.WriteLine("Medelvärdet blev " + totalNumber / totalInput);
            Console.ReadKey();
        }

Funkar men spontant skulle jag gissa att hans lärare uppskattar om man lägger in en felkontroll i en funktion som tar inmatning från användaren.

Du behöver inte ha en variabel för varje inmatat tal. Använd summa += Convert.ToInt32(str) efter inmatningarna och ta bort raden

int summa = number1 + number2 + number3;

och lägg till

int summa = 0;

överst.

Yupp det stämmer. += ska det vara

Håller med här angående KISS och lägger till YAGNI; You Aint Gonna Need It.
Det är väldigt lätt att vilja rädda världen och skriva super generell och dynamisk kod. Och det är ju ok sålänge man utbildar sig, för man lär sig mycket av att joxa och experimentera med koden, utforska alla vägar typ.
Men när man väl börjar få betalt för sittjobb inser man snabbt att man inte kan ta höjd för alla de miljoner hypotetiska scenarion som kan förekomma i ett projekt. Framför allt vill inte kunden betala för det...
Vill TS maxa sitt lärande kan man, när koden funkar, fråga sig 'kan jag göra på ett annat sätt' och labba vidare med andra lösningar. Då får man också reda på vad som inte funkar, och det är minst lika viktigt som att veta vad som funkar 😄

Skickades från m.sweclockers.com

Viktigaste är att TS fått rätt på sin uppgift, och förhoppningsvis förstått den. Vilken typ av loop som passar bäst var kan vi starta en ny tråd om, verkar vara ett hett ämne

Det är väl sant men samtidigt kan ju hans betyg till viss del bero vad läraren tycker om hans lösningar. Om jag var lärare skulle jag absolut se det som ett plus om hans funktion lätt kunde utökas, samt om den hade någon form av kontroll för felinmatningar.

Men visst, huvudsaken är väl att det fungerar.

@Roffe: Jo, men det är hans tankesätt som ska betygsättas, inte andras. Jag håller fullt med om att man ska försöka göra en skalbar och elegant lösning i de fall man kan, det är bra praxis för framtiden, men om han inte kommer fram till det själv så kommer läraren märka rätt snabbt om det är saxat från nätet

Det låter som att du tror att jag argumenterar emot enkelhet. Det gör jag absolut inte. Jag argumenterar bara emot inställningen att man bara ska göra det absolut minsta som begärts hela tiden. I det här fallet är det också så enkelt och smidigt att lägga till en rudimentär felkontroll samt förbereda för att kunna ta emot ett nästan obegränsat antal värden att jag inte kan komma på några som helst argument emot det.

Det där är dock helt irrelevant i det här fallet. Det handlar om att ta emot data från användaren och det krävs så lite för att gardera sig mot de vanligaste felen att jag inte kan se varför man inte skulle vilja göra det.

for(i=0, i<3, i++)
{
    parsea input till summeringsvariabel...
}

For-loopar funkar alldeles utmärkt när man vet i förväg hur många gånger de ska köras, vilket man förvisso vet i det här fallet. Jag tänker mest på att TS lärare nästa vecka kan säga att han ska modifiera sin funktion för att ta emot tio tal, eller helt enkelt ta emot tal tills användaren slutar. Då krävs det någon slags loop. Jag tror inte det ses som positivt att han lämnat in en lösning som inte går att utöka på ett elegant sätt.

Ja det är sant. Det var därför jag skrev mitt förslag i klartext i stället för att ge honom koden direkt så att han skulle få klura något själv.

@micke_mj: Jag tycker det där är ganska läsbart men en aning tillkrånglat. Vill du snygga till det lite så kan du göra så här:

static void Main () {

	Console.WriteLine ("Skriv ett decimaltal: ");

	string str = Console.ReadLine ();
	decimal decimaltal = Convert.ToDecimal (str);

	Console.WriteLine ("Decimaltalet" + " " + decimaltal + " " + "avrudans till heltalet" + " " + Math.Round (decimaltal));
	Console.ReadKey ();

}

Du behöver i detta fallet inte deklarera variabeln "decimaltal" i förväg utan kan göra det på samma rad som konverteringen sker.

Vill du korta ner ytterligare kan du strunta i att lägga inmatningen i en egen strängvariabel. Kan dock vara lite svårare att läsa för en nybörjare.

decimal decimaltal = Convert.ToDecimal (Console.ReadLine ());

Du kan även göra så här så slipper du att det kraschar om användaren skriver in ogiltiga tecken:

static void Main () {

	Console.WriteLine ("Skriv ett decimaltal: ");

	bool success = float.TryParse (Console.ReadLine (), out float decimaltal);

	if (success == true)
		Console.WriteLine ("Decimaltalet" + " " + decimaltal + " " + "avrudans till heltalet" + " " + Math.Round (decimaltal));
	else
		Console.WriteLine ("Felaktig inmatning.");

	Console.ReadKey ();

}

Hoppas det inte snurrar för mycket i skallen nu.