#include <cstdlib>
#include <iostream>
#include <string>
#include <ctime>

using namespace std;

string Computer1_Karten ()
{    
     string Karte [51];
     Karte[0] = "Pik Ass";
    Karte[1] = "Pik Koenig";
    Karte[2] = "Pik Dame";
    Karte[3] = "Pik Bube";
...
    Karte[51] = "Herz Ass";
    time_t t;
    srand(time(&t));

        int b = rand() % 51;
       string Computer1_Karte = Karte[b];

        return(Computer1_Karte);     
}
string Karten_Spieler (int a)
{    
    string Karte [51]; // hier werden alle 52Karten aufgezählt
    Karte[0] = "Pik Ass";
    Karte[1] = "Pik Koenig";
...
    Karte[51] = "Herz Ass";
    time_t t;
    srand(time(&t));

    string Karten_Spieler = "";

    do
    {  

        int b = rand() % 51;
        //Überprüfung ob die Karte schon vergeben sind
        if(Karten_Spieler.find(Karte[b])==string::npos)
        {
        a=a-1;
        Karten_Spieler.append(Karte[b]);
        Karten_Spieler.append(", ");
        }

        }while(a > 0); 
        return(Karten_Spieler);      
}

int main(int argc, char *argv[])
{
    int anfang; //Jetzt kommt cout<<Text....(Unwichtig)
cin>>anfang;

    if (anfang == 1)
    {
               string a = Karten_Spieler(8);
          cout<<a<<endl;
          string c = "";

         int d = 8;
          //Überprüfung des Computers
          do
          {

                        string b = Computer1_Karten();
                        if (a.find(b)==string::npos && c.find(b)==string::npos)
                        {
                           c.append(b);
                           c.append(", ");
                           d = d - 1;
                        }
          }while(d>0);
          cout<<c<<endl;

    }
    system("PAUSE");
    return EXIT_SUCCESS;
}

So das ist mein problem-Code. Bei mir wird keine Fehlermeldung in dev rausgegebn (wüsste auch nicht wieso)aber beim straten des Programms wenn ich 1 eingeben sollte er die Kartenverteilung starten und anzeigen Die Karten des Spieler zeigt er auch an. Doch dann kommt eine Fehlermeldung von Windows (Poblem senden ,nicht senden kennt glaub ich jeder) So der Fehler müsste ja eig. im Code sein. Nur ich weis halt nicht wo und weis auch nicht warum mir dev diesen Fehler nicht kennzeichnet.^^ So kann sein das dies ein einfaches Problem ist und ich es einfach übersehenhabe oder es noch nicht nach meinem Wissenstandt aus anderes machen konnte. Also bitte helft mir.

lg

Walter

P.S. Also falls ich diesen Beitrag im falschem Forum gepostet habt sei mir das bitte nachgesehen (wusste nicht recht borland oder hier oder woanders^^) und kann einfach verschoben werden^^

Warum dann 51?

string Arrays fangen doch bei 0 an und deshalb nicht 52 sonder nur 51 oder nicht?

Ja, aber die Anzahl muss schon stimmen. Du musst 52 Einheiten haben. Die fangen dann bei 0 an und gehen bis und mit 51.

Es funktioniert, nur bevor die Karten des Computers anzeigt braucht der so 5sek. ist das normal?

Kaum. Selbst im Debugmodus sollte es schneller sein. Hast du schon das Codestück mit der Zeitverschwendung lokalisieren können? Mit Debugger und Haltepunkten sollte das nicht schwer sein...

Übrigens: In deinem folgenden Code-Stück steht der Bezeichner Karten_Spieler sowohl für die Funktion als auch für die Variable vom Typ std::string . Ich nehm mal an, das ist ein Kopierfehler...

string Karten_Spieler (int a)
{
    string Karten_Spieler = "";
    // ...
        return(Karten_Spieler);
}

Und den Fehler mit 51 statt 52 musst du auch noch an anderen Orten korrigieren, z.B. hier:

int b = rand() % 51;

Und nimm doch C++- anstatt C#-Codetags

string b = Computer1_Karten(); 
                        if (a.find(b)==string::npos && c.find(b)==string::npos) 
                        { 
                           c.append(b); 
                           c.append(", "); 
                           d = d - 1; 
                        }

Habe dadurch das ich die einzelnen Karten von diesem Computer erst in der main funktion überprüft habe sehr viel Zeit verloren. Außerdem habe ich noch folgendes Problem: Habe die Karten nach folgdem Prinzip ausgesucht:
1.Zufaallzahl erzeugen.
2. Das jeweilige string feld suchen
3. Überprüfen ob diese string feld schon vorhanden war. Falls ja das ganze von vorne.

Das ist schlecht. Denn jeweniger karten es zum Schluss gibt desto llänger braucht er um welche zufinden die er nicht hat. Mitlerwiele habe ich alle Karten unter diesen 4Spielern aufgeteilt 52Karten/4 = 13. Bei dem letzden Computer habe ich schon gesagt das er die felder alle durchlaufern soll und die er noch nicht hat ausgeben.

Schneller wäre. Diese String felder. Nach dem jeweiligen Benutzen zu (löschen) oder so kenntlich zumachen das er bei diesem Zufallsprinzip der Karten auswahl nicht mehr auf diese Karte kommen kann. Bsp:
string Karte[9] = "Pik Acht"
so diese Karte würde dann ausgegeben aber dieses String feld soll bei der nächsten Auswahl nicht mehr berücksichtigt werden.

So das mit dem Karten_Spieler ist kein Kopier fehler Ich aknn es schnell ändern wenn das so schlecht ist. Aber vllt kurz sagen warum das so ist damit ich auch was lerne.

So 2Fragen habe ich noch kann ich ein array als rückgabe wert haben (ich glaube schon) und kann ich z.B. einen string und einen int wert zurückgeben und wenn ja wie speichere ich den dann in der Main funktion.

int a;
string b;
return(a,b);

So in der art nur ist halt die Frage was ich dann vor die Funktion schreibe also welchen Rückgabetyp.

Schon mal danke fürs lesen^^ und evtl. antowrten

lg

walter

P.S. Das mit dem C# code ist mir peinlich...

So 2Fragen habe ich noch kann ich ein array als rückgabe wert haben (ich glaube schon) und kann ich z.B. einen string und einen int wert zurückgeben und wenn ja wie speichere ich den dann in der Main funktion.

Ein Array kannst du nicht zurückgeben. Du kannst höchstens einen Zeiger auf das Array zurückgebeb, musst dann aber schauen, dass du das auf Speicher machst, welcher auch reserviert bleibt. (new)

Ansonsten musst du einen Zeiger an die Funktion übergeben.

return(a,b);

Das funktioniert so schon, aber es macht nicht das, was du denkst.
Du wendest den , - Operator an. In wirklichkeit wird aber lediglich a zurückgegeben, was ein int ist.

Wenn du aber einen std::vector nimmst, anstatt eines eines Arrays, kannst du auch einen vector zurückegeben, welcher dann einfach kopiert wird.

So das mit dem Karten_Spieler ist kein Kopier fehler Ich aknn es schnell ändern wenn das so schlecht ist. Aber vllt kurz sagen warum das so ist damit ich auch was lerne.

Naja, mehrdeutige Bezeichner sind nicht gut. Das könnte zu Namenskonflikten führen. Ausserdem solltest du die Funktionen genau nach dem benennen, was sie tun. Zum Beispiel MischeKarten() oder ähnlich, je nach Funktion halt.

Walter S. schrieb:

So 2Fragen habe ich noch kann ich ein array als rückgabe wert haben (ich glaube schon)

Nein. Das heisst, du kannst zwar den Zeiger zurückgeben, aber der ist dann nicht mehr gültig.

Walter S. schrieb:

und kann ich z.B. einen string und einen int wert zurückgeben und wenn ja wie speichere ich den dann in der Main funktion.

Da gibt es mehrere Möglichkeiten:

1. Übergabe per Referenz. Die Argumente werden nicht kopiert, sondern direkt übergeben. Das heisst, sie können auch von der Funktion verändert werden.

void func(int& i, std::string& s) // & bedeutet Referenz
{
   i = 4;
   s = "Hallo";
}

int main()
{
    int IntVar;
    std::string StringVar;
    func(IntVar, StringVar);
    // hier haben die Variablen die Werte 4 bzw. "Hallo"
}

2. Übergabe als Zeiger. Ähnlich wie die Übergabe als Referenz, nur dass du eben mit Zeiger statt Referenzen arbeitest.

void func(int* i, std::string* s) // * bedeutet Zeiger
{
   *i = 4;
   *s = "Hallo";
}

int main()
{
    int IntVar;
    std::string StringVar;
    func(&IntVar, &StringVar); // Adressen übergeben (& ist Adressoperator)
    // hier haben die Variablen die Werte 4 bzw. "Hallo"
}

3. Rückgabe einer Struktur. Du erstellst einen Datentyp, der einen Integer und einen String speichert.

struct Data
{
    int i;
    std::string s;
};

Data func() // func gibt einen Wert vom selbstdefinierten Typ Data zurück
{
    Data MyData = {4, "Hallo"}; // Temporäre Instanz erstellen
    return MyData;              // diese zurückgeben
}

int main()
{
   Data D = func();
   int IntVar = D.i;             // Zugriff auf Strukturelemente
   std::string StringVar = D.s;
}

Walter S. schrieb:

P.S. Das mit dem C# code ist mir peinlich...

Oh, ist doch kein Problem. Hier gibts viel Schlimmeres, z.B. Leute, die immer noch ohne irgendeine Formatierung ihren Code aufschreiben, obwohl es ihnen schon mehrfach gesagt wurde

Ich würde dir zum Einsatz der STL raten, das vereinfacht einiges. Da gibt es Container, die sich wie dynamische Arrays verhalten, aber noch zusätzliche Funktionen und Vereinfachungen bieten. Ein Beispiel ist der std::vector . So ein Vector kann im Gegensatz zu Arrays auch Rückgabetyp einer Funktion sein.
Es gibt auch einen STL-Algorithmus std::random_shuffle() , der einen solchen Container (oder auch ein Array) zufällig durchmischt. Damit wär auch dein Performanceproblem gelöst. Aber ich weiss nicht, wie weit du schon in C++ gekommen bist, vielleicht ist es auch noch besser, damit zu warten.

Walter S. schrieb:

Habe die Karten nach folgdem Prinzip ausgesucht:
1.Zufaallzahl erzeugen.
2. Das jeweilige string feld suchen
3. Überprüfen ob diese string feld schon vorhanden war. Falls ja das ganze von vorne.

Was meinst du mit "das ganze"? Den gesamten Algorithmus neu starten? Wenn ja, wunderts mich nicht, dass das so lange geht. Du solltest die vorherigen Elemente so belassen, und wenn eines schon vorgekommen ist, halt nur genau das noch einmal suchen. Dann sollte es bei nur 52 Elementen eigentlich nicht so lange dauern.

Aber du hast natürlich Recht, die Zeitkomplexität dieses Algorithmus ist sehr schlecht (exponentiell). Eine Funktion, die zuverlässig zufällige Permutationen erzeugt, ist auch nicht gerade einfach. Falls dich das Thema interessiert, kannst du dich hier genauer informieren, ansonsten solltest du entweder halt den bisherigen Algorithmus in Kauf nehmen oder std::random_shuffle() benutzen.

Ok mit dem Nameskonflikt ist mir einleuchtend^^ Habe es soweit mal geändert. Mit den arrays ist i-wie schade^^.Ok werde die 3 Möglichkeiten noch ausprobieren. Und mich dann für eine Entscheiden. Das STl hört sich interesaant an nur ich habe diesen Sommer einen workshop für c++ gemacht da wurden halt Hauptfunktion/Kommentare/Ein-und ausgabe, Varablen,Operatorn(Grundrechenarten,Modulo), Verzweigungen( Vergleichs-logische Operatoren), Schleifen, Zufallszahlen, arrays ,klasse string besprochen nun habe ich im herbst halt noch mal ein workshop und ich will halt vorher noch nicht so viel vorweg nehmen sonst brauch ich den kurs ja net zu bezahlen wenn ich da von schon was kann^^. Deshalb möchte ich noch nichtso viel neues machen Aber ich sehe hier kommen auf jeden fall sehr viele ideen von leuten die sich gedanken machen^^. Find ich toll... habe schon andere Foren erlebt.

Mit dem ganzen meine ich das er wenn er beim 3 Punkt bin und er die zahl schon hat wieder bei 1 anfängt. D.h. es könnte passieren das er wieder diese Zahl bekommt^^ und wieder von vorne anfangen muss, weil diese Zahl ja noch im string array mit drin ist und nicht außen vorgelassen wird. Von der Zeit her muss ich sagen das es momentan bei 52 elementen sofort klappt. Ohne das ein merkbarer zeitverlust besteht. (fürs auge erkennbar)

ansonsten schon danke für die tipps^^ kann sein das ich im laufe dieses Projekts mich nochmal melde^^. Weeril habe bis jetzt ja nur die Kartenverteilung jetzt mussnoch ein bischen KI ins Spiel kommen. Hat da jemand einen Tipp für mich ich könnte zwar lauter if-Sätze machen aber meine Frage ist geht das auch schneller oder soll ich lieber mit meinen kenntnissen if sätze schreiben. Kurz zum Sinn des Spiels 52Kartenspiel, Pik ist Trumpffarbe, Man muss immer bedienen, duie hörere Karte gewinnt oder halt trumpf wer am ende die meisten stiche hat gewinnt. (Alles andere wäre ein bischen zu hoch gegriffen glaube ich mal)^^

lg

walter