Programmering C# fastnat på array & string
Det är inte så mycket rätt i din kod tyvärr så det är svårt att hjälpa dig utan att helt göra uppgiften. Tänk på vad du gör med koden helt enkelt. Varför har du en foreach-loop? Är det tänkt att utskriften av menyn ska ligga i ovannämnda loop och alltså göras om för varje string i ryggsäcken? Varför säger inte menyalternativen vad de innebär? Vilken key läser du med ReadKey?
Det är inte så mycket rätt i din kod tyvärr så det är svårt att hjälpa dig utan att helt göra uppgiften. Tänk på vad du gör med koden helt enkelt. Varför har du en foreach-loop? Är det tänkt att utskriften av menyn ska ligga i ovannämnda loop och alltså göras om för varje string i ryggsäcken? Varför säger inte menyalternativen vad de innebär? Vilken key läser du med ReadKey?
Tror jag gått framåt nu iallafall, är detta bättre än föregående?
Jag försöker dock nu fixa en funktion som sammanfattar allting jag stoppat in i ryggsäcken.
Trodde myList + ytterFack skulle fungera men icke, tankar?
edit: För att summera båda listorna i slutet, är jag på rätt väg om jag skriver
foreach (int TotalObjects in ytterFack + myList)
Console.WriteLine (TotalObjects)
Tror jag gått framåt nu iallafall, är detta bättre än föregående?
Jag försöker dock nu fixa en funktion som sammanfattar allting jag stoppat in i ryggsäcken.
Trodde myList + ytterFack skulle fungera men icke, tankar?
edit: För att summera båda listorna i slutet, är jag på rätt väg om jag skriver
foreach (int TotalObjects in ytterFack + myList)
Console.WriteLine (TotalObjects)
Nu har jag aldrig suttit med C# men mer med Java som är ganska likt.
För det första, din indentering är helt knas. Om du försöker få lite struktur, få alla { } på samma rad, allt innanför ska vara intabbat. Det ser OK ut i början men sen blir det helt tokigt.
Jag tror du tänker på fel sätt. Du får försöka bryta ut detta och gå igenom vad allt gör.
t.ex. vill du räkna alla items, gör en metod som heter countItemsInBackpack();
Då kan du t.ex. bara göra så här:
switch (menyVal)
{
case 1:
countItemsInBackpack();
break;
case 2:
görnågotannathär();
break;
}
Det blir mer lättläst och du vet precis vad som händer i vilket case och vill du ändra något i countItemsInBackpack() då går du ner till den metoden och ändrar där nere.
Du skapar en lista som heter ytterFack som har 4 platser i sig.
Sen skapar du en lista till som heter myList. Inga kommentarer här, så vad håller den listan och vart används den?
Med tanke på att du slarvar så mycket med indentering och struktur så glömmer du av att sätta { } vid for each, jag är inte så påläst om C# men en snabb googling så står det med { }
foreach (var item in ytterFack)
{
Console.WriteLine($"Item: {item}");
}
Börja med att bryta ner det du vill göra in i metoder. Ta en metod i taget och lägg bara till ett meny val. När första menyvalet fungerar kanon! Gå till nästa.
När alla 4 fungerar så kan du lägga allt i en while loop och kanske ett till menyval nummer 5 som stänger ner pogrammet.
- Idag Nu stiger hårddiskpriserna med uppemot 10 procent 5
- Idag Analytiker: Apple har överskattat intresset för Vision Pro 40
- Igår AMD, Nvidia och Intel – vad är det för skillnad mellan grafikkortstillverkarna? 21
- Igår Testpilot: MSI MPG 271QRX - Färgsprakande OLED i 360 Hz 13
- 23 / 4 Meta öppnar upp Quests OS – vill bli VR-världens Android 12
- Nu stiger hårddiskpriserna med uppemot 10 procent6
- LegoClockers - Samlingstråd för allt med Lego607
- Webhallen config, kan inte hitta 2 st sata diskar 3,557
- PC crashar oftare och oftare när jag spelar - AKUT hjälp med felsökning7
- Quiz: Vad kan du om Inet?45
- Systembolaget varnar: Dryckesbrist efter hackerattack94
- Ny TV har skada i panel, butik menar att de inte kan återskapa felet22
- En simpel fråga om hur mycket lönehöjning man får varje år16
- Gamingskärm - ultrawide eller inte?32
- Hej, ser denna ok ut? Budget cirka 9k.20
- Säljes 2 set med Corsair 16GB (2x8GB) DDR4 3600MHz CL18 Vengeance LPX AMD
- Köpes 5800x3d / 5700x3d
- Säljes iMac Pro 2017
- Köpes Köpes AM4 moderkort + Minnen.
- Säljes GeForce RTX 2070 ROG STRIX 8GB
- Säljes RETRO: GeForce 8800 GTS
- Säljes Server - 2x Xeon gold 6138 - 256GB RAM - Nvidia P2000
- Säljes Ryzen 9 3950x Mobo 64gb ram!
- Säljes ASUS TUF F17 - i7 | 16GB | 1TB | RTX 4060
- Säljes ASUS 27" ROG Swift PG27AQDM OLED QHD 240 Hz
- Nu stiger hårddiskpriserna med uppemot 10 procent6
- Quiz: Vad kan du om Inet?45
- Analytiker: Apple har överskattat intresset för Vision Pro40
- Microsoft rullar ut Startmenyreklam till alla46
- EU röstar igenom ”rätten att reparera”51
- Viaplay sätter ner foten mot delade konton54
- AMD, Nvidia och Intel – vad är det för skillnad mellan grafikkortstillverkarna?21
- Systembolaget varnar: Dryckesbrist efter hackerattack94
- Veckans fråga: Hur mycket lagringsutrymme har din dator?93
- Valve täpper till kryphål i Steam-återbetalningar19