VierGewinnt Überprüfung

[seemsbroken]

Hallo ihr Lieben!
Ich arbeite momentan an einem kleinen Vier Gewinnt, was auch soweit funktioniert, ausser das es nicht stoppt wenn 4 in einer Linie/Spalte/Diagonalen sind.

Ich verzweifele grade buchstäblich & hoffe dass ihr mir irgendwie helfen könnt was hier dran falsch ist.

public class Spielbrett 
{
    Coin[][] spielbrett;; 
     
     public Spielbrett ()
    {
       spielbrett = new Coin[7][6];     
    }
    public boolean istvoll()        
    {
        for (int i = 0; i < spielbrett.length; i++ )
        {
            if (spielbrett[i][0] == null) 
            { 
                return false; 
            }
        }
        return true;
    }
    public void spielbrettausgabe()   
    {                               
        System.out.println();       
        for(int i = 0; i<=5; i++)    
        {
            System.out.print("| ");
            for(int j = 0; j<=6; j++)
            {
               Coin coin = spielbrett[j][i];
                    if(coin == null)
                        System.out.print("_");
                    else
                        coin.coinausgabe();
                    System.out.print(" | ");
            }
            System.out.println();
        }
    System.out.println("-----------------------------");
    System.out.println("  1   2   3   4   5   6   7");
    System.out.println();
    }
     
     
    public boolean coinsetzenfeld(int spalte, Spieler aktuellerspieler)
    { 
        System.out.println(spalte);
        for(int zeile=5;zeile>=0;zeile--)
        {
            if(spielbrett[spalte-1][zeile]==null)
            {
                Coin coin = new Coin(aktuellerspieler);
                spielbrett[spalte-1][zeile]=coin;
                return true;
                     
            }
        }
        return false;
    }   
     
    public Spieler gewinner(int spalte) 
    {                                   
        int zeile = 0;                  
        while (spielbrett[spalte-1][zeile] == null)
        { 
            zeile++; 
        } 
        Coin coin = spielbrett [spalte-1] [zeile];     
        return coin.getspieler();
    }           
     
    public boolean gewinnzug () 
                                 
                                 
    {
        if (gewinnzeile (spielbrett)) return true;
        if (gewinnspalte (spielbrett)) return true;
         
                                 
        return false;
    }
     
    public boolean gewinnzeile(Coin[][] s)  
    {                                       
    Coin[][] spielbrett = new Coin[7][6];
        spielbrett= s;
        for (int zeile=0; zeile<6; zeile++)      
            {int summe=1;
            for (int spalte=0; spalte<4; spalte++)
                {
                if((spielbrett[zeile][spalte])==(spielbrett[zeile][spalte+1])   
                && spielbrett[zeile][spalte] != null                   
                && spielbrett[zeile][spalte+1] != null)                 
                    {summe=summe+1;                                 
                    if(summe>=4)
                    {return true;}}
                    else {summe=1;}
                     
                 
                }   
            }
            return false;
                     
    }
     
 
            static void gewinnzeile(char[][] spielbrett, int spalte, int zeile, char zeichen) {
                // nach links
                int go = zeile - 1; // mit dem Punkt links neber dem gesetzten beginne
                                    // ich
                int i = 1; // der gesetzte Punkt = 1 Treffer
                while (go >= 0) {
                    if (spielbrett[spalte][go] != zeichen)
                        break;
                    go--;
                    i++;
                }
                 
    }
    private boolean gewinnspalte(Coin[][] s)                
    {   
    Coin[][] spielbrett = new Coin[7][6];   
    spielbrett=s;
    for (int spalte=0; spalte<4;spalte++)        
        {int summe=1;
            for (int zeile=0; zeile<6; zeile++)
            {
            if((spielbrett[zeile][spalte])==(spielbrett[zeile+1][spalte])   
                    && spielbrett[zeile][spalte] != null                       
            && spielbrett[zeile+1][spalte] != null)                     
            {summe=summe+1;                                     
            if(summe==4)
                {return true;}
            System.out.println( "gewinnt in der Spalte!");
            }
                else {summe=1;}
                     
            }
        }               
    return false;
    }

danke schonmal im Voraus!



}

 

[seemsbroken]

Zur Sicherheit alle Klassen mal :

public class VierGewinntstart {
	 public static void main(String [] args)
	 {
         Spiel neuesspiel = new Spiel(); 
           
         neuesspiel.start();     
                    
           
                           
} 

}

import java.util.Scanner;
public class Spieler 
{
     
     
    public String name;
    public int nr;
    public Spieler (int nummer) 
    {
          Scanner s=new Scanner(System.in);
     
            System.out.println("Spieler " + nummer + ", bitte Namen eingeben");
            name=s.nextLine();
            this.nr = nummer;
    }
 
    public String getname() 
    {
        return this.name;
    }
    public int getspielernr() 
    {
        return this.nr;
    }
 
    public int coinsetzenspieler() 
    {
     
        int spalte;
        System.out.println(this.getname()+", geben Sie bitte die Spaltennummer ein:");
        spalte=Tools.intEingabe();
        return spalte;
    }
 
}

public class Coin
{
 
    public int nr;          
    public Spieler spieler;
    public String name;
                                 
 
 
    public Coin (Spieler aktuellerspieler)  
                                             
    {
        nr = aktuellerspieler.getspielernr(); 
        spieler = aktuellerspieler; 
    }
 
    public int getcoinnr() 
    {
        return this.nr;
    }
 
    public Spieler getspieler() 
    {
        return this.spieler;
    }
    public void coinausgabe()   
                                 
    {
        if(this.getcoinnr() == 1)
            System.out.print("Z");
        else
            System.out.print("L");
    }
}

import java.util.Scanner;



public class Spiel 
{
    public Spielbrett spielbrett;   
    public Spieler spieler1;        
    public Spieler spieler2;        
    public Spieler aktuellerspieler;
 
 
    public Spiel() 
                    
    {
        spielbrett = new Spielbrett(); 
        spieler1 = new Spieler(1);  
        spieler2 = new Spieler(2);  
        aktuellerspieler = spieler2;
    }
 

 
 
    public void start()  
    {	
        spielausgabe();  
        int spalte;      
        do
        {
            spielerwechsel();
            spalte = aktuellerspieler.coinsetzenspieler();  
                                                             
            spielbrett.coinsetzenfeld(spalte, aktuellerspieler);
                                                                        spielausgabe();                                     
        }while(!gewinncheck(spalte));   
    }
     
    private void spielerwechsel() 
    {
        if (aktuellerspieler == spieler1) 
            aktuellerspieler = spieler2;      
        else
            aktuellerspieler = spieler1;
    }   
    private boolean gewinncheck(int spalte)        
    {
        if(spielbrett.istvoll())  
                                 
        {
            System.out.println("Das Spielbrett ist voll! Unentschieden!");
            return true;    
        }
        if (spielbrett.gewinnzug()) 
        {                           
         
            Spieler spieler = spielbrett.gewinner(spalte); 
                                                           
                                                            
            System.out.println("Der Spieler "+ spieler.getname()  +" hat gewonnen");
            return true;
        }
        return false;
    }
    public void spielausgabe()      
    {
        spielbrett.spielbrettausgabe(); 
    }
}

import java.util.Scanner;
 
public class Tools {
    static int intEingabe() 
    { 
        Scanner scanner = new Scanner(System.in); 
        return scanner.nextInt(); } 
    }

 

[Tiding]

Ich könnte dir einen Algorithmus anbieten, der auf alle Fälle funktioniert !
opcorn:

 

[seemsbroken]

Und der passt zu meinem Rest
Ne Quatsch, bin fuer jede Hilfe dankbar !!

 

[Tiding]

Ich kann dir aber auch den Tipp geben, dass du in deinen Überprüfungsschleifen niemals dazu kommst, das der Summe 1 addiert wird!

Sprich der "Coin" den du da überprüfen willst, ist immer null.

 

[Tiding]

Deine Algorithmen sind aber auch echt merkwürdig :autsch:

 

[seemsbroken]

Ja also glaube die sind komplett falsch ..
Die Summe muss doch immer um 1 erhöht werden, falls gleichfarbige Coins nebeneinander liegen oder?

 

[Tiding]

Jaja, natürlich, meine Vorgehendsweise wäre auch nur geringfügig verschieden zu deiner!

Aber: gewinnzeile, du überprüfst die Zeilen 0-5 hat es nicht 7 Zeilen?
und die Spalten 0-4, für 0-5 würds ja noch Sinn machen, da du immer die Nachfolger überprüfst, aber 0-4? Evtl. überseh ich ja auch was

Selbes bei den Spalten.