Schleife funktioniert nicht!

  • C#
  • .NET (FX) 4.5–4.8

Es gibt 6 Antworten in diesem Thema. Der letzte Beitrag () ist von RodFromGermany.

    Schleife funktioniert nicht!

    Hallo VB-Paradise Community,

    ich habe diese Schleife erstellt um 5 zufällige Zahlen zu generieren und doppelte Zahlen auszuschließen, jedoch lädt das Programm und lädt und lädt...

    Form1.cs:

    C#-Quellcode

    1. using System;
    2. using System.Collections.Generic;
    3. using System.ComponentModel;
    4. using System.Data;
    5. using System.Drawing;
    6. using System.Linq;
    7. using System.Text;
    8. using System.Threading.Tasks;
    9. using System.Windows.Forms;
    11. namespace WindowsFormsApplication1
    12. {
    14. public partial class Form1 : Form
    15. {
    17. Karte[] AlleKarten = new Karte[11];
    18. Karte[] HandKarten = new Karte[6];
    20. public Form1()
    21. {
    22. InitializeComponent();
    23. }
    25. private void Form1_Load(object sender, EventArgs e)
    26. {
    27. this.KeyDown += Form1_KeyDown;
    28. this.MaximizeBox = false;
    29. LoadCards();
    30. }
    32. void Form1_KeyDown(object sender, KeyEventArgs e)
    33. {
    34. if (e.KeyCode == Keys.Escape)
    35. {
    36. Application.Exit();
    37. }
    38. }
    40. private void LoadCards()
    41. {
    42. Karte TestKarte1 = new Karte("TestKarte1", 100, Image.FromFile("data/cards/1.png"));
    43. Karte TestKarte2 = new Karte("TestKarte2", 200, Image.FromFile("data/cards/2.png"));
    44. Karte TestKarte3 = new Karte("TestKarte3", 300, Image.FromFile("data/cards/3.png"));
    45. Karte TestKarte4 = new Karte("TestKarte4", 400, Image.FromFile("data/cards/4.png"));
    46. Karte TestKarte5 = new Karte("TestKarte5", 500, Image.FromFile("data/cards/5.png"));
    47. Karte TestKarte6 = new Karte("TestKarte6", 600, Image.FromFile("data/cards/6.png"));
    48. Karte TestKarte7 = new Karte("TestKarte7", 700, Image.FromFile("data/cards/7.png"));
    49. Karte TestKarte8 = new Karte("TestKarte8", 800, Image.FromFile("data/cards/8.png"));
    50. Karte TestKarte9 = new Karte("TestKarte9", 900, Image.FromFile("data/cards/9.png"));
    51. Karte TestKarte10 = new Karte("TestKarte10", 1000, Image.FromFile("data/cards/10.png"));
    53. AlleKarten[1] = TestKarte1;
    54. AlleKarten[2] = TestKarte2;
    55. AlleKarten[3] = TestKarte3;
    56. AlleKarten[4] = TestKarte4;
    57. AlleKarten[5] = TestKarte5;
    58. AlleKarten[6] = TestKarte6;
    59. AlleKarten[7] = TestKarte7;
    60. AlleKarten[8] = TestKarte8;
    61. AlleKarten[9] = TestKarte9;
    62. AlleKarten[10] = TestKarte10;
    64. Random myRandom = new Random();
    65. int[] LastNumbers = { 0, 0, 0, 0, 0, 0 };
    67. for (int i = 1; i < 6; i++)
    68. {
    69. bool Check = true;
    70. int Index = 0;
    71. while (Check)
    72. {
    73. Index = myRandom.Next(1, 11);
    74. foreach (int Zahlen in LastNumbers)
    75. {
    76. if (Index == Zahlen)
    77. {
    78. Check = false;
    79. //Überspringen
    80. }
    81. else
    82. {
    83. //Nichts
    84. }
    85. }
    86. }
    87. HandKarten[i] = AlleKarten[Index];
    88. Check = true;
    89. }
    91. Handkarte1.Image = HandKarten[1].GetCardImage();
    92. Handkarte2.Image = HandKarten[2].GetCardImage();
    93. Handkarte3.Image = HandKarten[3].GetCardImage();
    94. Handkarte4.Image = HandKarten[4].GetCardImage();
    95. Handkarte5.Image = HandKarten[5].GetCardImage();
    96. }
    98. }
    100. }


    Karte.cs:

    C#-Quellcode

    1. using System;
    2. using System.Collections.Generic;
    3. using System.ComponentModel;
    4. using System.Data;
    5. using System.Drawing;
    6. using System.Linq;
    7. using System.Text;
    8. using System.Threading.Tasks;
    9. using System.Windows.Forms;
    11. namespace WindowsFormsApplication1
    12. {
    14. class Karte
    15. {
    17. private string Name;
    18. private int AttackPower;
    19. private Image CardImage;
    20. private ToolTip CardToolTip = new ToolTip();
    22. public Karte(string Name, int AttackPower, Image CardImage)
    23. {
    24. this.Name = Name;
    25. this.AttackPower = AttackPower;
    26. this.CardImage = CardImage;
    27. this.CardToolTip.ToolTipTitle = this.Name + Environment.NewLine + "Attacke: " + this.AttackPower.ToString();
    28. }
    30. public string GetName()
    31. {
    32. return this.Name;
    33. }
    35. public int GetAttackPower()
    36. {
    37. return this.AttackPower;
    38. }
    40. public Image GetCardImage()
    41. {
    42. return this.CardImage;
    43. }
    45. }
    47. }


    Ich hoffe ihr könnt mir helfen,

    Gruß DragonSlayerMarc
    There are only 10 types of people in the world: Those who understand binary and those who don't.

    So wie ich das sehe, möchtest Du die Zahlen 0, 1, 2, 3, 4 und 5 (also die Indices für HandKarten) zufällig durcheinanderwürfeln.
    Dafür kannst Du nach zufall sortieren lassen:

    C#-Quellcode

    1. var Rnd = new Random();
    2. var RandomIndices = Enumerable.Range(0, Handkarten.Length).OrderBy(Item => Rnd.NextDouble());

    Aber anhand der Problembeschreibung konnte ich das nicht eindeutig herauslesen.
    "Luckily luh... luckily it wasn't poi-"
    -- Brady in Wonderland, 23. Februar 2015, 1:56
    Desktop Pinner | ApplicationSettings | OnUtils

    Jou.
    @DragonSlayerMarc Arbeite mit Indizes.
    1. eine Zahl aus { 0, 1, 2, 3, 4, 5 } => erste Karte
    2. eine Zahl aus { 0, 1, 2, 3, 4 } => zweite Karte
    3. eine Zahl aus { 0, 1, 2, 3 } => dritte Karte
    4. eine Zahl aus { 0, 1, 2 } => vierte Karte
    5. eine Zahl aus { 0, 1 } => fünfte Karte
    6. die letzte Karte
    Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
    Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch :!:
    Ein guter .NET-Snippetkonverter (der ist verfügbar).
    Programmierfragen über PN / Konversation werden ignoriert!

    @RodFromGermany nur das es eben { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 } im Kartendeck ist (Index 0 wird beim füllen des Arrays sowie bei der Schleife ausgelassen) und ich nur 5 davon brauche. Habe die Schleife auch auf einem Zettel probiert, da hat aber alles funktioniert und ich bekomme 5 zufällige Zahlen.

    EDIT: Ich bin selber doof xD Ich überprüfe ja ob im Array LastNumbers die Nummer schon existiert jedoch fülle ich dieses Array nie und deshalb generiert und generiert es. Trotzdem Danke für eure Hilfe.
    There are only 10 types of people in the world: Those who understand binary and those who don't.

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von „DragonSlayerMarc“ ()

    DragonSlayerMarc schrieb:

    { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 }
    Dann wende dies auf meinen Pseudocode an. :D
    Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
    Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch :!:
    Ein guter .NET-Snippetkonverter (der ist verfügbar).
    Programmierfragen über PN / Konversation werden ignoriert!

    Habe die Schleife ein bisschen erweitert, jz geht wieder nichtsmehr!

    C#-Quellcode

    1. Random myRandom = new Random();
    2. bool FirstNumber = true;
    3. int[] LastNumbers = { 0, 0, 0, 0, 0, 0 };
    5. for (int i = 1; i < 6; i++)
    6. {
    7. bool Check = true;
    8. int Index = 0;
    9. while (Check)
    10. {
    11. Index = myRandom.Next(1, 11);
    12. if (FirstNumber == false)
    13. {
    14. foreach (int Zahlen in LastNumbers)
    15. {
    16. if (Index == Zahlen)
    17. {
    18. Check = false;
    19. //Überspringen
    20. }
    21. else
    22. {
    23. //Nichts
    24. }
    25. }
    26. }
    27. else if (FirstNumber == true)
    28. {
    29. FirstNumber = false;
    30. LastNumbers.SetValue(Index, i);
    31. }
    32. }
    33. HandKarten[i] = AlleKarten[Index];
    34. LastNumbers.SetValue(Index, i);
    35. Check = true;
    36. }

    There are only 10 types of people in the world: Those who understand binary and those who don't.

    DragonSlayerMarc schrieb:

    jz geht wieder nichtsmehr!
    Setz einen Haltepunkt rein und verfolge Deinen Code, überprüfe den Inhalt der Variablen.
    Gugst Du hier.
    Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
    Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch :!:
    Ein guter .NET-Snippetkonverter (der ist verfügbar).
    Programmierfragen über PN / Konversation werden ignoriert!