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.
- Datorhallar åker på miljardstor skattesmäll – ljög om att utvinna krypto68
- Vad lyssnar du på just nu?13826
- Vilken film såg du senast?13368
- Asus lanserar 8K-skärm med Mini LED44
- [LEK] Gissa spelet15472
- Ikea släpper nytt skärmstativ för 399 kronor35
- Bästa spelmusik/soundtracks136
- Hifi – bilder på era anläggningar här1521
- Rabbel.se - Ett dagligt ordspel693
- Amazons Fallout-serie regisseras av Jonathan Nolan (Inte den Nolan, men hans brorsa)310
- Säljes Custom speldator
- Säljes Beyerdynamic 770 dt pro 80ohm
- Säljes Gamingdator med Ryzen 7700X och Gigabyte Radeon RX 7900 Gre
- Säljes ASUS TUF GeForce RTX 4070 TI Super 16GB Gaming OC
- Säljes 4090 RTX TUF ASUS OG OC 24GB
- Säljes Lian li Strimer Plus V2
- Säljes Ny/Oöppnad WD BLACK SN850X 1TB NVMe SSD Heatsink
- Säljes LianLi O11 XL, bequiet! 1200W PSU, 32GB G.Skill 6000Mhz CL30, EK Coolstream 360mm radiatorer
- Säljes Vårstäd/Garderobsrens. Datorer, Datordelar, Streaming tillbehör, Vattenkylningsprylar, tillbehör, etc.
- Säljes G.Skill Flare X 16GB (2x8GB) Ryzen / 3200MHz / DDR4 / CL14 / F4-3200C14D-16GFX
- Ikea släpper nytt skärmstativ för 399 kronor35
- Snabbtest: Bli mer Pro med mindre tangentbord17
- Stockholm får en coworking-hubb för spelutvecklare6
- Max spikar priser inför lanseringen104
- Bedragare låtsades vara Lastpass VD med AI27
- SFW! Läckra ROG Zephyrus G14 med ROG Nebula OLED Display8
- Quest 2 får prissänkning för andra gången i år25
- Elgato lanserar tillbehörsserie för ”vanligt folk”12
- Enhance! Edge kan få klassisk sci-fi-funktion16
- Efter konkursryktena – Louqe är tillbaka20