PHP (konsolen) problem

Hallo zusammen,

ich bin an einem Kartenspiel dran und habe ein Denkfehler.

Das Spiel ist ganz einfach immer die höhere Karte gewinnt und im falle von 2 gleich hohen Karten wird eine Karte von jeden Spieler verdeckt auf den Tisch gelegt und und die nächste Karte entscheidet wer den kompletten stapel gewinnt

PHP-Code:

while ($hasWinner != 10)
{
    $arrGewonneneKarten = array();
    $gleicheKarten =  array();
    $karte = array();
    while ($rundenSieger === false)
    {
        if($stechen == 0)
        {
            #$karte = array();
            foreach($arrSpieler as $objSpieler)
            {
                $karte[] =  $objSpieler->getHand()->spielKarte();
                //Überprüfen ob noch Karten auf dem Stapel leigen
                if ($karte[$k] == null) {
                    echo "Spieler $k hat keine Karten mehr!!!!!!";
                }
                $k++;
            }
            $k = 0;
            for($i = 0; $i < $anzSpieler; $i++)
            {
                echo "\n HHHH Spieler $i: $karte[$i] \n";
            }

            $maxKartenWert = $karte[0]->getSpielwert();
            $maxKarteSpieler = $karte[0]->getSpieler();
            $maxKarte = $karte[0];
            $rundenSieger = true;
            $arrGewonneneKarten[] = $maxKarte;
        }
        for($i = 0; $i < $anzSpieler; $i++)
        {
            if ($maxKartenWert < $karte[$i]->getSpielwert())
            {
                $maxKarte = $karte[$i];
                $arrGewonneneKarten[] = $karte[$i];
                $maxKartenWert = $karte[$i]->getSpielwert();
                $maxKarteSpieler = $karte[$i]->getSpieler();
                $gewinner = $karte[$i]->getSpieler();
                if($i < $anzSpieler)
                {
                    $rundenSieger = true;
                }
                $stechen = 0;
            }
            else if ( $maxKartenWert > $karte[$i]->getSpielwert() )
            {
                $arrGewonneneKarten[] = $karte[$i];

                $gewinner = $maxKarteSpieler;
                if($i < $anzSpieler)
                {
                    $rundenSieger = true;
                }
                $stechen = 0;
            }
            else if ($maxKartenWert === $karte[$i]->getSpielwert())
            {
                if($maxKarte == $karte[$i])
                {

                }
                else
                {
                   
                    $gleicheKarten[] = $maxKarte;
                    $gleicheKarten[] = $karte[$i];
                    //Die Verdeckte Karten die der Gewinner gewinnt
                    $maxKarte = $maxKarte->getSpieler()->getHand()->spielKarte();
                    $karte[$i] = $karte[$i]->getSpieler()->getHand()->spielKarte();
                    $gleicheKarten[] = $maxKarte;
                    $gleicheKarten[] = $karte[$i];
                    $maxKarte = $maxKarte->getSpieler()->getHand()->spielKarte();
                    $karte[$i] = $karte[$i]->getSpieler()->getHand()->spielKarte();
                    $maxKartenWert = $maxKarte->getSpielwert();
                    $gleicheKarten[] = $maxKarte;
                    $gleicheKarten[] = $karte[$i];;
                    $arrGewonneneKarten = $arrGewonneneKarten + $gleicheKarten;
                    $gleicheKarten =  array();
                    $stechen = 1;
                    $rundenSieger = false;
                }
            }
        }
    }
    if($stechen == 0 && $rundenSieger = true)
    {
        echo count($arrGewonneneKarten);
        $gewinner->getHand()->nehmKartenAuf($arrGewonneneKarten);
        echo "\n\n Gewinner {$gewinner->getName()}\n";
        $rundenSieger = false;
        $hasWinner++;
    }

} 


Nun ist mein problem wenn ich 2 gleiche Karten habe dann muss ich ja die 2 gleichen Karten vergleichen welche die Gewinner Karte ist, und da gehts los wie löse ich das am geschicktesten??

Danke im vorraus

Zitat von wono

wenn ich 2 gleiche Karten habe dann muss ich ja die 2 gleichen Karten vergleichen

Wenn du schon weißt, dass es 2 gleiche Karten sind, wozu dann noch vergleichen?

(Den Spaghetticode habe ich mir nicht angesehen.)

Sorry meinte wenn ich zwei gleiche Karten habe und dann 2 neue Karten ziehe muss ich diese vergleichen.

Spaghetticode -> ist mir bekannt allerdings geht es kaum besser......

Hallo,

ich bin dafür, dass du deinen momentanen Algorithmus kurz und knapp (meinetwegen mit stark gekürztem PHP-Code oder etwas Pseudocode) erläuterst und dabei die Stelle kennzeichnest, wo du Hilfe benötigst.

Der Code ist einfach zu viel und schlecht gekapselt, daher hat man einfach keine Lust, sich da durch zu wühlen.

Edit:

Zitat von wono

Spaghetticode -> ist mir bekannt allerdings geht es kaum besser......

Bist du da ganz sicher? Ich behaupte, es geht sehr wohl noch besser. Da du sowieso schon tendenziell objektorientiert arbeitest, könnte man die ganzen Vergleiche, Runden, Ergebnisse, ... auch oo realisieren.

Gruß,

Amica

Zitat von wono

Spaghetticode -> ist mir bekannt allerdings geht es kaum besser......

Oh doch, das geht! Und hätte den Vorteil, dass es dann sicherlich eine Funktion oder Methode gäbe, die man im Falle gleicher Karten aufrufen kann.
Bei deinem Stil bleibt dir nichts anderes übrig, als in dem leeren If-Block wieder einen ganzen haufen Spaghetticode zu schreiben.

OK ich versuch nochmal xD, also ich habe z.B 3 Spieler und gehe einfach davon aus das meine 1 Spieler die Höchste Karte besitzt dies ist dann meine

PHP-Code:

 $maxKarte 


Danach nehm ich einfach jede weiter Karte und schaue ob der SpielWert der karte höher oder niedriger ist in diesen beiden fällen funktioniert dies auch soweit.

PHP-Code:

 
       
         for($i = 0; $i < $anzSpieler; $i++)
         {
             if ($maxKartenWert < $karte[$i]->getSpielwert())
             {
                 $maxKarte = $karte[$i];
                 $arrGewonneneKarten[] = $karte[$i];
                 $maxKartenWert = $karte[$i]->getSpielwert();
                 $maxKarteSpieler = $karte[$i]->getSpieler();
                 $gewinner = $karte[$i]->getSpieler();
                 if($i < $anzSpieler)
                 {
                     $rundenSieger = true;
                 }
                 $stechen = 0;
             }
             else if ( $maxKartenWert > $karte[$i]->getSpielwert() )
             {
                 $arrGewonneneKarten[] = $karte[$i];
 
                 $gewinner = $maxKarteSpieler;
                 if($i < $anzSpieler)
                 {
                     $rundenSieger = true;
                 }
                 $stechen = 0;
             } 


Meine problem ist wenn die zwei höchsten Karten gleich sind. Denn in diesem fall muss ich zwei Karten verdeckt auf den $gleicheKarten[] "Stapel" legen und anschließend zwei neue Karten von Stapel der jeweiligen Spieler nehmen. Das Funktioniert auch.

PHP-Code:


            else if ($maxKartenWert === $karte[$i]->getSpielwert())
            {
                if($maxKarte != $karte[$i])
                {                   
                    $gleicheKarten[] = $maxKarte;
                    $gleicheKarten[] = $karte[$i];
                    //Die Verdeckte Karten die der Gewinner gewinnt
                    $maxKarte = $maxKarte->getSpieler()->getHand()->spielKarte();
                    $karte[$i] = $karte[$i]->getSpieler()->getHand()->spielKarte();
                    $gleicheKarten[] = $maxKarte;
                    $gleicheKarten[] = $karte[$i];
                    $maxKarte = $maxKarte->getSpieler()->getHand()->spielKarte();
                    $karte[$i] = $karte[$i]->getSpieler()->getHand()->spielKarte();
                    $maxKartenWert = $maxKarte->getSpielwert();
                    $gleicheKarten[] = $maxKarte;
                    $gleicheKarten[] = $karte[$i];;
                    $arrGewonneneKarten = $arrGewonneneKarten + $gleicheKarten;
                    $gleicheKarten =  array();
                    $stechen = 1;
                    $rundenSieger = false;
                } 


Ich weis jetzt nicht wie ich die 2 "neuen" Karten vergleichen soll.......

ich hoffe es ist jetzt etwas verständlicher

Vielleicht solltest du dann (wenn die gleich sind) eine while-Schleife starten, die solange zieht, bis die Karten nicht mehr gleich sind, denn es kann ja sein, dass es mehrmals hintereinander passiert, dass die Karten gleichwertig sind.

Ansonsten: viel verständlicher ist es nicht. Von einem "erklärten Algorithmus" ist gar nichts zu sehen und die Bedeutung der einzelnen Variablen kennst nur du. Da wäre auch eine Erklärung hilfreich, welche Variable jetzt für welche Karte steht und so weiter. Da du ohnehin die ganze Zeit irgendwelche Karten vergleichst, ist nicht ersichtlich, wo jetzt das Problem liegt bzw. sieht es so aus, als würdest du das ja schon behandeln. Jedenfalls sehe ich so nicht ganz durch.