C# Programmering

@micke_mj *Trådstart redigerat*

Observera att vi inte tillåter direkt uppgiftslösning/läxhjälp i forumet (typ här är uppgiften, lös den åt mig), men det är givetvis helt ok att be om hjälp ifall man har fastnat i ett delmoment.
/moderator

Skriv ett inlägg med den kod du har.

Du är på rätt väg men du behöver jobba på inputen. Du kan göra t ex en for loop som körs 3 gånger eller bara fråga tre gånger i rad om input. En for loop är bra att använda om du vet hur mycket du ska iterera, i det här fallet tre.

While är överlägset bättre än en loop här då man kan sätta andra brytande villkor utan att behöva använda break. Dessutom kan man testa counter mot ett förutbestämt värde för antalet tal man behöver mata in.

do-while skulle jag använt.

Skickades från m.sweclockers.com

@Roffe: om du använder do kan du aldrig skippa loopen, text om antalet inmatade tal ska vara 0.

Men om man inte ska mata in något tal, varför köra funktionen alls?

@Roffe: man kanske ska mata någon kod kopplad till ett kort och vissa kort har 4 siffror, vissa har 6 siffror och vissa kort har ingen kod alls. Då vill man ju inte behöva mata in minst en siffra för det kortet och inte heller skriva något extra villkor för att hantera det.

Stirra dig inte blind på att göra en lösning som bara kan lösa ett problem. Generalisera så mycket du kan så du kan återanvända din kod.

@micke_mj: Jag skulle gjort så här:

string inputString;
int sum = 0;
int counter = 0;

do {

Låt användaren mata in en textsträng. Lägg den i inputString.
Konvertera inputString till heltal. Lägg resultatet i int number.

Om konverteringen lyckades {
Addera number till sum.
Öka värdet på counter.
}

} while (inputString inte är tom eller count mindre än 3, vilket du nu vill använda)

Skriv ut resultatet.

Om du avbryter loopen när inmatade talet är noll så blir det problem om ett eller flera av talen som ska matas in är noll. Bättre att kolla om strängen är tom. Kolla även att konverteringen lyckades annars blir det problem om användaren råkar skriva in en bokstav. Använd int.TryParse i stället för int.Parse.

Edit: Kolla gärna så det verkligen matats in något heltal innan du försöker visa resultatet.

Jag håller absolut med om att man ska försöka skriva sin kod anpassningsbar men det finns gränser för allt. Den här funktionen skulle räkna ut medelvärden, inte ta emot kortkoder. Jag för har lite svårt att se vitsen med att veta medelvärdet av siffrorna i ens kortkoder och dessutom funkar do-while fortfarande utmärkt även om första inmatningen är tom.

+1 med TryParse över Parse i sådana här fall, läraren brukar ge pluspoäng för sånt.

Och ska man då kontrollera inmatning så kör man fördelaktigt programmet i en loop, så man kan backa/inte öka räknaren om inmatningen inte är godkänd.

Tänk på att använda variabelnamn som är beskrivande. Att köra namnet "avg" på en variabel som i praktiken innehåller summan känns lite förvirrande

Rätt ang do while, har aldrig sett det i någon produktionskod, enbart i skolböcker eller kodexempel. I detta fallet tycker jag det är viktigast att ts förstår vad han gör.

Men ja, det är bra att skriva kod som går att generalisera, men om man vet säkert att något inte kommer ändras introducerar det bara ytterliggare komplexitet. Det bästa, imo, är att försöka identifiera områden som är "subject to change" och förbereda dessa för detta. Sedan funkar det ju inte alltid.

Använt Java i många år men blev sedan introducerad till C# och blev frälst ganska snabbt. ;D Det enda jag har emot det är egentligen att det nästan bara fungerar på Windows, men det har ju börjat ändras det också nu iom .Net Core.

Om han gör på det viset så blir funktionen låst till exakt tre tal och om han i framtiden vill att den ska hantera fler så kommer han att bli tvungen att göra om hela jobbet. Nu är detta en väldigt enkel och specifik skoluppgift men att göra som du föreslår och låsa upp sig till att göra exakt det som efterfrågas och inget mer lär knappast ge några pluspoäng heller. Det är i nästan alla lägen en bra idé att skriva kod som lätt kan utökas. Det handlar inte på något sätt om att stila, bara förbereda för framtida behov.

Jag har aldrig sagt att do-while gör jobbet bättre än while, bara att jag personligen skulle använda do-while. Någon skillnad i prestanda lär det inte vara men det känns logiskt i ett fall som detta att göra testet sist även om slutprodukten blir densamma. Huruvida do-while i verkligheten används ofta eller ej känns irrelevant för mig. Man använder de verktyg man har tillgång till.

Det är ett skolexempel. Det handlar om att skaffa sig goda vanor som programmerare och lösningen jag förespråkar är knappast speciellt komplicerad. Med några få rader extra kod har du en funktion som kan hantera hur många tal som helst (nästan), inte bara tre. Jag tycker dock TS ska välja den lösning han tror att läraren uppskattar mest.

Edit: Förresten, med din lösning, vad händer om användaren råkar snubbla på tangenterna och matar in något ogiltigt? Du måste testa att varje inmatning är korrekt och om inte låta användaren göra om den, annars kommer en felaktig inmatning räknas som noll eller avbryta körningen.

Jag vet. Egentligen en rätt meningslös diskussion men du får gärna motivera varför det är mer logiskt att ha testet i början på loopen, innan första inmatningen. Skit samma ur prestandasynpunkt som sagt så det är bara en fråga om personlig preferens.