vector<int> hilfsv::gzeit(int o)
{
    srand(time(NULL));
    vector<int> flugzeug;
    flugzeug.clear();

        if(flip(20) == true){

           flugzeug.push_back(zufall(20000));
           flugzeug.push_back(zufall(99)+1);
           flugzeug.push_back(o);

            }
    return flugzeug;
}

Dabei sieht die Funktion zufall so aus:

int hilfsv::zufall(int z) {
    srand(time(NULL));
    int zufallszahl = 0;
			zufallszahl = rand()%z;
            return zufallszahl;
        }

flip ist einfach nur eine Funktion die einen Münzwurf simuliert. Die Idee dahinter ist dass in der main eine Schleife durchläuft die die Zeit darstellt. Bei jedem Schleifendurchlauf wird mit einer (in diesem Fall 20%igen) Wahrscheinlichkeit ein Flugzeug generiert, welches danach eben übergeben wird.

Das Problem ist, dass bei jedem Schleifendurchlauf nur einmal ein Flugzeug entsteht (oder eben nicht) und wenn es entsteht bei jedem Durchlauf die selben Eigenschaften (also Tank und FlugNr. hat)

Weiß einer von euch weiter?

Allgemein ist rand nicht die beste Methode, um Zufallszahlen zu generieren. Schau dir mal Pseudo-random number generation an.

Deine if-Abfrage ist auch etwas merkwuerdig. Wenn du ganze Zufallszahlen im Bereich \[1,..,N\] erzeugen willst kannst du eifach rand() % N + 1 verwenden (sind dann allerdings nicht wirklich uniform verteilt, ist fuer dein Programm aber wohl kein Problem).

vector<int> hilfsv::gzeit(int o)
{
    int i;
    vector<int> flugzeug;
    flugzeug.clear();

        for(i=0; i<10; i++) {
           flugzeug.push_back(rand()%20000);
           flugzeug.push_back(rand()%100 + 1);
           flugzeug.push_back(o);

     cout << "8:00"  << ": Ein Flug mit der Flugnummer " <<  flugzeug[0] << " und " << flugzeug[1] << "% Tankfuellung kommt an!" << endl;
        }
    return flugzeug;
}

(Die Funktion wird gerade in der main aufgerufen. Das srand(time(NULL)) steht gerade ganz am Anfang in der main.)

Aber danke schonmal für die schnelle Hilfe

Die if Abfrage sorgt einfach nur dafür dass nicht bei jedem Schleifendurchlauf ein Flugzeug generiert wird.

Ach, haette deine Frage genauer lesen sollen.

MrMeikkel schrieb:

Das srand(time(NULL)) sollte also in die main? Wieso kann ich es nicht in der Klasse lassen?

Wie bereits gesagt soll srand nur einmal aufgerufen werden. Gibt keinen Grund das in die Klasse zu stecken (was wenn du mehrere Klassen hast, die rand brauchen, in welcher Klasse rufst du dann srand auf?).

MrMeikkel schrieb:

Und es müsste doch eigentlich auch erstmal mit der rand() Funktion funktionieren (auch wenn die Zufallszahlen dann nicht wirklich zufällig verteilt sind, aber das sei mal dahingestellt).

Ja klar, hier ist das sicher kein Problem. Aber ist immer gut zu wissen, dass es noch im Allgemeinem bessere Moeglichkeiten gibt

MrMeikkel schrieb:

Aber selbst wenn ich innerhalb der Funktion eine Schleife erzeuge sind die Zufallszahlen immer gleich:

vector<int> hilfsv::gzeit(int o)
{
    int i;
    vector<int> flugzeug;
    flugzeug.clear();

        for(i=0; i<10; i++) {
           flugzeug.push_back(rand()%20000);
           flugzeug.push_back(rand()%100 + 1);
           flugzeug.push_back(o);

     cout << "8:00"  << ": Ein Flug mit der Flugnummer " <<  flugzeug[0] << " und " << flugzeug[1] << "% Tankfuellung kommt an!" << endl;
        }
    return flugzeug;
}

(Die Funktion wird gerade in der main aufgerufen. Das srand(time(NULL)) steht gerade ganz am Anfang in der main.)

Der Code sieht eigentlich in Ordnung aus. Kannst du ein minimales, kompilierbares Beispiel posten? (Im Notfall den ganzen Code, wenns nicht zu viel ist).

*Edit
Das flugzeug.clear brauchst du uebringens nicht.

vector<int> hilfsv::gzeit(int o)
{
    int i;
    vector<int> flugzeug;
    flugzeug.clear();

        for(i=0; i<10; i++) {
           flugzeug.push_back(rand()%20000);
           flugzeug.push_back(rand()%100 + 1);
           flugzeug.push_back(o);

     cout << "8:00"  << ": Ein Flug mit der Flugnummer " <<  flugzeug[0] << " und " << flugzeug[1] << "% Tankfuellung kommt an!" << endl;
        }
    return flugzeug;
}

Du schiebst hier 30 Werte in den Vektor (10 mal 3), aber mittels cout gibst Du immer nur dieselben (die ersten beiden) aus ...

#include <iostream>
#include <cstdlib>
#include <time.h>
#include <vector>
#include <queue>

using namespace std;

int main()  {
    srand(time(NULL));
    int i;
    vector<int> flugzeug;
    flugzeug.clear();

        for(i=0; i<10; i++) {
           flugzeug.push_back(rand()%20000);
           flugzeug.push_back(rand()%100 + 1);
           flugzeug.push_back(1); //normalerweise wird hier die globale Zeit übergeben bei der das Flugzeug generiert wurde

     cout << "8:00"  << ": Ein Flug mit der Flugnummer " <<  flugzeug[0] << " und " << flugzeug[1] << "% Tankfuellung kommt an!" << endl;
        }
    return 0;
}

Erzeugt bei mir immer dieselben Tankfüllungen und Flugnummern.

Und gerade wollte ich den Code posten und habe alle Änderungen die ich zum testen gemacht hab rückgängig gemacht, jetzt funktioniert alles wie es soll
Danke für eure Hilfe auf alle Fälle

*Edit
Wups, zu spaet.

Das srand(time(NULL)) sollte also in die main? Wieso kann ich es nicht in der Klasse lassen?

Weil damit der Generator auf einen Startwert gesetzt wird.In diesem Fall ist es eine Anzahl Sekunden seit einem Startpunkt.

Also änderst sich der Startwert nur jede Sekunde. Das Programm ist aber viel schneller und erzeugt somit viele gleiche Werte.

Da es sehr selten ist, dass das Programm mehrmals innerhalb von derselben Sekunde gestartet wird, gibt es bei jedem Programmlauf eine andere Reihenfolge der Zufallswerte (so der Idealzustand).