C# skoluppgift

Ja, lägg bara till en ny variabel som sparar alla tal som matats in och skriv ut den när x = 0.

Jag är inte jätteerfaren inom programmering, och har aldrig skrivit något i C#, men det ser ut som att du inte summerar alla talen som matas in? Du skulle kunna göra det med hjälp av ytterligare en int-variabel inne i loopen.

Den andra if-satsen borde inte vara nödvändig, då du vill fortsätta läsa in tal tills x == 0, då kommer programmet gå vidare automatiskt. Varför Console.ReadLine i slutet? Jag kan som sagt inte C#, men det verkar onödigt att läsa in något igen?

När jag började med programmering så tog ett tag för mig att komma över att uttryck som följande faktiskt är giltiga:
int a = 4;
int b = 50;
b = b + a; // Det är dock snyggare att använda uttrcyket +=, och skrivs isf. ut: b += a;

Genom att hålla flera räknare i luften, där en räknares värde beror av en annan räknares, så kommer ni att kunna lösa uppgiften.

Kompilera koden i ditt huvud och gå igenom allting steg för steg. Rad 1 och rad 2 är varsitt statement. Sedan kommer vi till din while-loop (while x != 0) som din 3:e kodrad, och den är ju alltid sann i och med att du redan definerat att int x = 1. Det betyder att koden alltid kommer att gå in i din while-loop och öka y med 1. Därefter skriver konsollen ut att den vill få ett heltal. Det tidigare kända "x=1" skrivs då över och tilldelas ett nytt tal, men ditt y är fortfarande y = 2. Försök skriva upp för dig själv i pseudokod vad du vill att koden ska göra så tror jag du kommer få bättre koll på det här. Spalta upp i ord (och gör det här på papper, jag lovar det ger en annan tankeverksamhet och kreativitetskänsla att scribbla på hederliga pappersark än att doodla i paint/notepad). Få ordningen klar för dig i hur du vill att sakerna ska följa och ske.

Först rekommenderar jag att du byter namn på x och y till tal och summa. Din kod blir då följande:

	int tal=1;
	int summa=tal;

	while (tal != 0 )
	{
		summa += tal;
		Console.WriteLine("Mata in ett tal: ");
		tal = int.Parse(Console.ReadLine());
	}

	Console.WriteLine("Summan av talen: " + summa);

Ser du problemet? Du börjar med att sätta summa till tal och sedan lägger du tal till summa ännu en gång, oavsett om användaren väljer att avsluta programmet direkt genom att mata in 0.

För att lösa uppgiften:

• summan ska börja på 0

• användaren ska mata in ett tal innan det läggs till summa

Fast, egentligen är det "fel" att använda en while-loop här. Fel med extra feta citationstecken, för det är något jag personligen tycker och inte uppfattningen av den breda massan.

Man kan lösa uppgiften utan att använda en while-loop med så kallad rekursion. Rekursioner ger ofta snyggare lösningar men det tar ett tag att lära sig se mönstret i dem.

int SummeraTills0() {
	var tal = int.Parse(Console.ReadLine());
	return tal + (tal != 0
		? SummeraTills0()
		: 0);
}

Console.WriteLine("Summan är: " + SummeraTills0());

Typ? Lite mycket kod dock.

Din kod innehåller småfel som att if(x == 0), behövs inte eftersom while kommer köra tills den är 0 och sen gå vidare.
x kan bara definieras en gång, fylls alltså aldrig på med nya värden, utan blir bara ett nytt värde.

ReadKey anser jag är bättre än ReadLine för att avbryta applikationen.

Int64 result = 0, input = -1; bool keep_going = true; 

            do
            {
                input = Convert.ToInt64(Console.ReadLine());

                switch(input)
                {
                    case 0:
                        keep_going = false;
                        break;
                    default:
                    result += input;
                        break;
                }
            } while (keep_going);

            Console.ReadKey();

Switch-casen och keep_going kan ersättas med en enda rad:

if (input == 0) break;

eller så loopar man tills input är noll..
Listan fyller ingen funktion alls. Det är bra att veta att man kan använda listor naturligtvis

if (input == 0) break;

Stämmer. Även listan kan bytas ut mot en variabel, säg Int64 x som inkrementeras med det nya värdet genom: x += Convert.ToInt64(Console.ReadLine());.

Problem: Fråga om input (heltal) tills användaren matar in 0 
----------------------------------------------------------------

Ett första försök brukar ofta se ut på detta vis:

int input;
Console.WriteLine("Mata in ett tal: ");
input = int.Parse(Console.ReadLine());
while (input != 0) {
  Console.WriteLine("Mata in ett tal: ");
  input = int.Parse(Console.ReadLine());
}

Sedan inser man att man har dubblerat viss kod, och löser det med en falsk input enligt följande:

int input = 1337;  // Skilt från 0, talet är i sig betydelselöst, syftet = "bluffa" villkoret så loopen körs minst en gång.
while (input != 0) {
  Console.WriteLine("Mata in ett tal: ");
  input = int.Parse(Console.ReadLine());
}

Istället kan man då använda en do-while-loop, vilken alltid körs en gång innan villkoret utvärderas:

int input;
do {
  Console.WriteLine("Mata in ett tal: ");
  input = int.Parse(Console.ReadLine());
} while (input != 0);

Alltså, 
Skall loopen köras 0+ gånger, använd while
Skall loopen köras 1+ gånger, använd do-while

Jag personligen skulle mer uppskatta ett sånt här upplägg:

Console.Write("Mata in heltal, när du är nöjd skriv en nolla");
 do 
            { 
                input = Convert.ToInt64(Console.ReadLine()); 

                switch(input) 
                { 
                    case 0: 
                        keep_going = false; 
                        break; 
                    default: 
                    result += input; 
                        break; 
                } 
            } while (keep_going); 

            Console.ReadKey();

Att hela tiden få bli promptad att mata in, lite jobbigt. Switchen gör att man sen kan bygga på och jag vantrivs med if-satser, gå segare att skriva och blir fult. Inte fullt säker på om keep_going krävs men det är ju en escape så.

Du behöver inte ha en if-sats heller

do {
...
} while {input != 0}

Det är bra mycket tydligare än att dölja exit-kriteriet bakom en boolean.
Switch cases i all ära, det ser ganska fult ut i mitten av en loop.

Console.Write("Mata in heltal, när du är nöjd skriv en nolla");
 do 
            { 
                input = Convert.ToInt64(Console.ReadLine()); 

                switch(input) 
                { 
                    case 0: 
                        keep_going = false; 
                        break; 
                    default: 
                    result += input; 
                        break; 
                } 
            } while (keep_going); 

            Console.ReadKey();

Absolut ett alternativ. Om texten skrivs ut en eller flera gånger beror ju bara på om man lägger Console.Write innanför eller utanför loopen. Det jag syftade till att visa med mitt inlägg var skillnaden mellan while (0+ ggr) och do-while (1+ ggr), inte att ge (vad jag tycker är) snyggast möjliga lösning.

TS har precis börjat lära sig de grundläggande funktioner språket erbjuder och du vill redan dra in programdesign!? Var sak har sin tid.