Lottobollar, uppgift med C#

[noparse] är din vän! Det går till exempel att få [code]...[/code] genom att skriva så:

[cmd][noparse][code]...[/code][/noparse][/cmd]

Och med Better SweClockers slipper du skriva BB-koden själv.

Om du menar Better SweClockers så är det "nya" Better SweClockers från slutet av december, plus några uppdateringar sedan dess.

            for (int i = 0; i < vektor.Length; ++i)
            {
                int element = 0;  
                if (int.TryParse(Console.ReadLine(), out element))  
                {
                    vektor[i] = element; 
                }
                else
                {
                    Console.WriteLine("Du måste mata in ett tal mellan 1-25");  
                    --i;  
                }
            }

Du gör ingen kontroll att heltalet (element) är vad du vill att den ska vara, mellan 1 (minstaInmatning) och 25 (högstaInmatning).

Kolla på exemplet från @zoomster2, int.tryParse ser till att det enbart är heltal som matas in, då kan du i else visa ett meddelande om att inmatningen var felaktig.

Sen tycker jag du borde lägga in inmatningen i vektorn EFTER du gjort kontrollen. Annars kommer du få problem om det inte är ett heltal.

for (int i = 0; i < vektor.Length; ++i)
            {
                int element = Convert.ToInt32(Console.ReadLine());

                
                vektor[i] = element;

                if (vektor[i] > 25 || vektor[i] < 1 )
                {
                    Console.WriteLine("Du får bara ange tal mellan 0-25");
                    --i;
                }
            }

Du behöver inte hårdkoda 1 och 25, du har ju redan variabler för det.
Du tog bort TryParse-koden vilket gjorde att du blev av med koden som kontrollerar att det enbart är heltal som gäller.

Skriv en kommentar innan varje rad med kod som du tycker är besvärlig eller inte kommer minnas vad den gör.

int element;
int högstaSlumptal = 25;
int minstaSlumptal = 1;

//Om inmatad text går att tolkas som ett heltal
if (int.TryParse(Console.ReadLine(), out element))
{
   //Ja, ett heltal
}
else
{
   //Nej, inte ett heltal
}

//Om inmatat heltal är större än 25 eller mindre än 1
if (element > högstaSlumptal  || element < minstaSlumptal )
{
   //Ja
}
else
{
   //Nej
}

Ni får inte använda while loop? Känns mer logiskt om jag tolkar det rätt, d.v.s. användaren ska mata in 10 tal mellan 1-25 sen då 10 tal är inmatad så ska programmet "köras" och du ska få ut resultat om man vunnit?
Eller ska den kontrollera varje tal man matat in om det matchar det slumpade talet?

Alla saker man behöver iterera går att göra med vilken form av loop som helst. Valet handlar om omkringliggande saker, vad som behövs och hur logiken följer. Olika loopar är lämpliga för olika saker men det är inget direkt tvång att använda en specifik typ av loop. Lite enkelt skulle jag säga att i vanliga fall använder man for-loopar för iterationer där man vet antalet varv som ska göras. En while-loop använder man för iterationer där man inte vet antalet varv som ska göras. do-while använder man när man inte vet antalet varv utom att minst ett varv ska göras.

Av utbildningsskäl så rekommenderar jag att försöka lösa den med både while och do-while så snart som möjligt.

Lite mer överkurs är att utelämna delarna i for-loopen. Exempelvis att ta bort eller flytta "iterator"-delen (i++). Det är väldigt enkelt att man blir låst i utseendet av en for-loop och att man glömmer bort att den består av tre delar där ingen av delarna är ett krav för att for-loopen ska exekveras.

@mattin123: Vad menar du med att indatasträngen har felaktigt format? Om du ger Convert.ToInt32 en sträng som inte kan tolkas som ett tal så kastas ett FormatException, som du bör fånga med en try-catch (eller alternativt använda Int32.TryParse istället). Se dokumentationen för Convert.ToInt32 för exempel.

När det gäller att lagra de inmatade talen i en vektor så är det ju exakt det din kod gör, så vad är det du inte förstår gällande det? Syntaxen är vektor[index] = värde; för att tilldela ett element i vektorn ett värde. Om värdet kommer från användaren via t.ex. Console.ReadLine() eller någon annanstans ifrån spelar ingen som helst roll.

1,2,3,4,5,6,7,8,9,10 är en sträng och den kommer inte parsa det, vilket gör att det crashar med ett FormatException. Som du skriver koden nu måste du ange ett tal i taget 10 gånger.
T.ex.
1 (tryck enter)
2 (tryck enter)
3 (tryck enter)
4 (tryck enter)
5 (tryck enter)
etc..
Då du når 10 tal (längden på arrayen och loopen) kommer den hoppa ur loopen och fortsätta din kod efter blocket för loopen.

@mattin123: Använd debuggern för att se vad som faktiskt sker när programmet körs och vad din vektor innehåller. Om du inte använt debuggern tidigare så gör dig själv en tjänst och lägg några minuter på att lära dig, det är inte alls komplicerat och är det mest användbara verktyg du har för att felsöka din kod. Här är en kort genomgång av hur det fungerar.

Vad händer om jag skriver in samma nummer fler gånger? Är det dubbla oddset då om jag vinner?

Om du vill skriva in flera nummer kan du först använda "string.Split(','). sedan får du parsa alla värdena i spliten och se om de funkar.

All felhantering är inte med i min kod, den är bara ett exempel Helge..

int slump_tal= randomerare.Next(1, 26);
string[] tiotal = Console.ReadLine().Split(',');
int tal = -1;
int counter = 0;

foreach (var strTal in tiotal )
{
    
    if (int.tryParse(strTal, out tal)
   {
      if (tal == slump_tal)
     {
         Console.WriteLine("Grattis, du har vunnit, rätt tal var: " + slump_tal);
         break;
     }
     if (++counter >= 10) break;
   }
 }

if (tal != slump_tal) Console.WriteLine("Tyvärr du har förlorat, rätt tal var: " + slump_tal);