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.
- Igår Efter konkursryktena – Louqe är tillbaka 17
- Igår Kunskapsquiz: IT och det moderna försvaret 41
- 17 / 4 Datorhallar åker på miljardstor skattesmäll – ljög om att utvinna krypto 62
- 17 / 4 Veckans fråga: Möss eller ljud – Vad lägger du mest pengar på? 73
- 16 / 4 X kan råda bot på bottar med betallösning 35
- Vilken serie såg du senast?1832
- Efter konkursryktena – Louqe är tillbaka18
- Snart ber Microsoft dig överge ditt lokala konto113
- Dags för robotgräsklippare432
- Quest 2 får prissänkning för andra gången i år19
- Köpa dator i finland9
- moderbord eller moderkort?32
- "Dumma" strömbrytare som folk inte använder av misstag?2
- Vad vill ni se i nästa Battlefield?86
- Experter. Vilket 4070 ti 12gb är detta?7
- Säljes Säljer GTX 1080 Ti
- Säljes Moderkort ASUS prime B660-plus d4 och ram
- Köpes Köper Samsung Galaxy Buds 2 Pro / Buds 2
- Säljes LC Power 39 tum 165 hz Bildskärm
- Säljes 12700k | 980 PRO 1tb | Contact Frame TG
- Köpes Söker USB-C transmitter till Steelseries Arctis 7X PLUS
- Säljes Diverse PS5-spel, Ritplatta och Motorola Moto G 5g plus!
- Säljes RTX 3050 8 GB ROG STRIX GAMING OC
- Säljes Sennheiser HD560S och RÖDE NT-USB
- Säljes Mp600 Pro 2TB, hel dator gtx 1080,6700k, ryzen 3600 samt gtx 1070
- SFW! Läckra ROG Zephyrus G14 med ROG Nebula OLED Display4
- Quest 2 får prissänkning för andra gången i år19
- Elgato lanserar tillbehörsserie för ”vanligt folk”10
- Enhance! Edge kan få klassisk sci-fi-funktion16
- Efter konkursryktena – Louqe är tillbaka18
- Snart ber Microsoft dig överge ditt lokala konto113
- Kunskapsquiz: IT och det moderna försvaret41
- Här är priserna på LG:s nya OLED-arsenal52
- 3dfx grafikkort återuppstår i hobbyprojekt19
- Logitech släpper ”AI-knapp” – snabbgenväg till Chat GPT12