Spel Lottobollar c#

Jag skulle rekommendera att du använder int.TryParse istället för att använda int.Parse då det medför att du måste hantera eventuella fel med try catch. Exceptions ska inte användas som ett normalt programflöde, utan när oväntade fel uppstår, t.ex. om man plötsligt tappar nätverksanslutning.

Det verkar som om du faktiskt vill lära dig, inte bara kopiera en färdig lösning, så här kommer en lösning du kanske vill kika på

        static void Main(string[] args)
        {
            var lottoBoll = new int[10];
            int användarensSvar;

            Console.WriteLine($"Hej och välkommen till Lotto, låt oss spela!\nVälj {lottoBoll.Length}st nummer mellan 1-25\n");

            for (int i = 0; i < lottoBoll.Length; i++)
            {
                Console.Write($"Spela på nummer {i + 1}: ");
                int.TryParse(Console.ReadLine(), out användarensSvar);

                if (användarensSvar < 1 || användarensSvar > 25)
                {
                    Console.WriteLine("Vänligen ange ett heltal mellan 1 och 25");
                    i--;
                }
                else
                {
                    lottoBoll[i] = användarensSvar;
                }
            }

            int slumptal = new Random().Next(1, 26);
            bool spelarenVann = false;

            for (int i = 0; i < lottoBoll.Length; i++)
            {
                if (lottoBoll[i] == slumptal)
                {
                    spelarenVann = true;
                    break;
                }
            }

            if (spelarenVann)
            {
                Console.WriteLine($"\nBINGO! Det rätta talet var {slumptal}");
            }
            else
            {
                Console.WriteLine($"\nIngen bingo denna gång, det slumpade talet var {slumptal}!");
            }

            Console.WriteLine("\nSpelet är slut, välkommen åter!");
            Console.ReadKey();            
        }

Här använder jag int.TryParse för att få tag på värdet användaren skriver in och break för att sluta leta efter träffar om man gissat rätt siffra. Det är kanske inte exakt så här jag skulle löst det själv, men jag försökte hålla mig så nära din kod som möjligt, samma variabelnamn och struktur.

Nej, du måste deklarera en variabel om du vill använda TryParse. Om du kikar på raden

int.TryParse(Console.ReadLine(), out användarensSvar);

så ser du att det står "out" innan variabelnamnet. Det innebär att funktionen placerar ett värde i variabeln du skickar in, du måste alltså "ta emot" det värdet med en tidigare deklarerad int-variabel. Detta är rätt ovanligt sätt att skicka runt värden på, i princip är detta det enda stället jag generellt använder den metoden. Anledningen till att det görs på det sättet här är att funktionen i sig, int.TryParse, returnerar en bool som indikerar om den lyckades konvertera indatasträngen, här console.ReadLine(), till en int eller inte. Jag ignorerar dock det värdet då returvärdet i användarensSvar ändå sätts till 0 om den misslyckas med att parsa strängen, vilket kontrolleras efteråt. Vill man att användaren ska kunna skriva värdet 0 måste man dock se om funktionen lyckades eller inte, typ

if (int.TryParse(console.ReadLine(), out användarensSvar))
...

Ungefär så. Man slipper try-catch för att man använder TryParse istället för Parse, sedan utnyttjar jag det faktum att TryParse returnerar 0 om den misslyckas, alltså ett värde utanför det godkända intervallet 1-25, för att kontrollera om man skrivit ett giltigt tal. På det sättet behöver man inte både verifiera att det är ett tal och att talet är mellan 1 och 25, utan man kan göra det i ett steg.

Det är alltid bra att veta vad funktioner returnerar, inte både när den lyckas/misslyckas, men även om den kan kasta ett exception eller inte. Parse kastar exception medan TryParse inte gör det.

Hehe... ja... i princip... Men det är inte så att Parse är en meningslös funktion. Det är betydligt bättre prestanda i Parse-funktionen, så om jag t.ex. skulle läsa in en stor textfil där jag behövde konvertera massa värden till int och jag var relativt säker på att den var i rätt format så skulle jag använda Parse med en try-catch. Ska man bara ta emot data inskriven av en användare är TryParse rätt val.

Nejdå, du kan använda lottoBoll[i] istället för variabeln användarensSvar i TryParse, det var nog bara jag som misstolkade frågan. Då kan man helt plocka bort både den variabeln och else-satsen i if:en, men jag skulle personligen inte gjort så. Det blir bara krångligare att läsa koden då när man använder arrayen för att både spara korrekta värden plus det senast inmatade värdet som behöver kontrolleras. När man kommer tillbaka till sin kod 10 år senare upptäcker man att lättläst kod är en miljon gånger bättre än kod där man sparat ett par rader bara för att det gick.

Man kan plocka bort användarSvar-variabeln och göra så här istället

            for (int i = 0; i < lottoBoll.Length; i++)
            {
                Console.Write($"Spela på nummer {i + 1}: ");
                int.TryParse(Console.ReadLine(), out lottoBoll[i]);

                if (lottoBoll[i] < 1 || lottoBoll[i] > 25)
                {
                    Console.WriteLine("Vänligen ange ett heltal mellan 1 och 25");
                    i--;
                }
            }

Men som jag skrev i ett annat svar är inte detta en lösning jag personligen skulle föredra då man i det här fallet använder arrayen för flera ändamål vilket gör det stökigt att felsöka och svårare för någon annan att följa. Läraren borde inte ha föreslagit det som en primär lösning. Visst, man ska inte deklarera massa variabler i onödan, man kan ofta komprimera ihop flera kodrader till en istället för att mellanlagra värden i variabler, men om någon annan (eller du själv om några år) ska läsa koden blir det då mycket svårare att förstå vad som händer. Det är en avvägning man lär sig med tiden, efter att man sett sin egen, bortglömda, kod ett antal gånger Men att använda samma variabel (eller array) för flera ändamål, som läraren föreslår här, är definitivt en dålig idé. Undantaget skulle vara om man kodar något extremt CPU-intensivt, t.ex. en 3D-motor, där varje operation man kan undvika resulterar i bättre prestanda, men sådant kodar man knappast i C#. För vanliga affärsapplikationer / websidor är läsbar kod a och o.

int.TryParse(Console.ReadLine(), var out användarensSvar);

Du råkade visst byta plats på out och var där
Men, ja, så kan man göra. Jag vet dock inte hur detta hanteras i en loop, om man då skapar en ny variabel i varje iteration som GCn sedan behöver ta hand om.

Efter lite googlande; nej, det skapas tydligen inga onödiga, kortlivade variabler på det sättet, det blir samma resultat i IL. Där lärde man sig något nytt

Edit: Råkade använda code istället för cmd...
Edit2: Man kan väl säga att det är snäppet bättre att deklarera variabeln på det sätt som Thomas visade då man i så fall indikerar i vilket scope den används.