ich versuche gerade einen Lootozahlengenerator zu programmieren.
Wie ihr seht, kontrolliert mein Algorihtmus aber nur die Zahl, die ein Index davor ins Datenfeld geschrieben wurde. Wie muss ich das ändern?

#include <iostream>
#include <conio.h>
#include <cstdlib>
#include <time.h>

using namespace std;

int main()
{
    int x, feld[6]={0};
    srand(time(NULL));

    for (int i=0; i<6; i++)
    {
        x=(rand()%49)+1;

        for (int j=0; j<i+1; j++)
        {
            while (x==feld[i])  //<---Hier haperts.  :rage: 
               x=(rand()%49)+1;
        }

        feld[i]=x;
        cout<<feld[i]<<' ';
    }
    getch();
}

Ich verstehe es schon, aber ich wüsste nur die Methode 5 Variablen zu erstellen und dann anschließend diese abzufragen. Das geht doch sicher schicker.

#include <iostream>
#include <ctime>

int main()
{
	int zahlen[6] = { };

	srand(time(NULL));

	for (int i = 0; i < 6; ++i)
	{
		int x = (rand() % 49) + 1;

		for (int j = 0; j < i; ++j)
		{
			if (zahlen[j] == x)
			{
				x = (rand() % 49) + 1;
				j = 0;
			}
		}

		zahlen[i] = x;
	}

	for (int i = 0; i < 6; ++i)
	{
		std::cout << zahlen[i] << std::endl;
	}
}

Ob es so richtig ist - keine Ahnung. Ungetestet!

#include <iostream>
#include <vector>
#include <algorithm>
#include <ctime>

typedef std::vector<int> ZahlenVec;

int main()
{
	ZahlenVec zahlen;

	srand(time(NULL));

	for (int i = 1; i <= 49; ++i)
	{
		zahlen.push_back(i);
	}

	std::random_shuffle(zahlen.begin(), zahlen.end());

	for (int i = 0; i < 6; ++i)
	{
		std::cout << zahlen[i] << std::endl;
	}
}

#include <iostream>
#include <vector>
#include <algorithm>
#include <ctime>

int main()
{
    std::vector<int> ZahlenVec;

    srand(time(0));

    for (int i = 1; i <= 49; ++i)
    {
        ZahlenVec.push_back(i);
    }

    std::random_shuffle(ZahlenVec.begin(), ZahlenVec.end());

    for (int i = 0; i < 6; ++i)
    {
        std::cout << ZahlenVec[i] << std::endl;
    }
    return 0;
}

hmm, das was er gemacht hat hab ich noch nicht in dieser Verwendung gesehen, wird sicher ein Grund haben, wird aber bestimmt so gehen.

korigiert mich wenn hier murks mache

EDIT return Val

ich finde "asdasdasd" vector methode am besten.
Aber er funktioniert nicht ?
Vielleicht so:

#include <iostream>
#include <vector>
#include <algorithm>
#include <ctime>

int main()
{
    std::vector<int> ZahlenVec;

    srand(time(0));

    for (int i = 1; i <= 49; ++i)
    {
        ZahlenVec.push_back(i);
    }

    std::random_shuffle(ZahlenVec.begin(), ZahlenVec.end());

    for (int i = 0; i < 6; ++i)
    {
        std::cout << ZahlenVec[i] << std::endl;
    }
    return 0;
}

hmm, das was er gemacht hat hab ich noch nicht in dieser Verwendung gesehen, wird sicher ein Grund haben, wird aber bestimmt so gehen.

korigiert mich wenn hier murks mache

EDIT return Val

Ich weiß auch gar nicht, was an meinem Algorithmus genau falsch sein soll. Die Vector-Variante müsste jedenfalls zu 100% korrekt sein. Ich sehe dort keine Fehlerquelle mehr...

Was hattest du an meinem Quellcode überhaupt geändert? Du hast ein return eingefügt, okay... Aber bei mir funktioniert das auch ohne, da 0 der Standardrückgabewert ist.

Außerdem hast du das typedef entfernt. Es kann allerdings sehr viel Tipparbeit ersparen!

Beispiel:

std::vector<int> bla;

for (std::vector<int>::iterator...)
{
 std::vector<int> x = ...;
}

vs.

typedef std::vector<int> V;

V bla;

for (V::iterator)
{
 V x = ...;
}

Nur als Beispiel...

gehört das nicht ins c++ forum?

Vermutlich, ja

Phisherman schrieb:

Es ist doch im C++ Forum.

es gibt auch noch das hier:

http://www.c-plusplus.net/forum/viewforum-var-f-is-15.html

MfG,

Probe-Nutzer

Im Zweifelsfall bitte auch folgende Hinweise beachten:
C/C++ Forum :: FAQ - Sonstiges :: Wohin mit meiner Frage?

Dieses Posting wurde automatisch erzeugt.

#include <iostream>
#include <time.h>
#include <cstdlib>
#include <conio.h>

using namespace std;

int main()
{
    srand(time(NULL));
    int x=(rand()%49)+1, feld[6]={0};
    bool checker;

    feld[0]=x;
    cout<<feld[0];

    for (int i=1; i<6; i++)
    {
        cout<<' ';
        x=(rand()%49)+1;
        while (feld[i]==0)
        {
              checker=false;
              for (int j=0; j<i; j++)
              {
                  while (x==feld[j])
                  {
                        x=(rand()%49)+1;
                        checker=true;
                        i--;
                  }
                  if (checker==false && j==i-1)
                  {
                        feld[i]=x;
                        cout<<feld[i];
                  }
              }
        }
    }

    getch();
}

Ich weiß auch gar nicht, was an meinem Algorithmus genau falsch sein soll. Die Vector-Variante müsste jedenfalls zu 100% korrekt sein. Ich sehe dort keine Fehlerquelle mehr...

Bei der nicht-Vector-Variante müsste es "j = -1" statt "j = 0" heißen.

Habs hinbekommen.
Es schaut jetzt so aus:

int x=(rand()%49)+1, feld[6]={0};

Initialisierst du da alle 6 Felder von 'feld' mit 0?

@knivil: Ich kenne "set" und Vektoren nicht. Und ja, ich muss rand jedesmal neu initialisieren, da doch die Zeit benutzt wird. Mit meinem momentan C++ Wissen fällt mir auch keine andere Lösung ein. Ich habe auch nicht nach einer anderen Lösung gefragt, sondern nach einer Korrektur.