2D-Grafik Tic Tac Toe nach Gewinn rot

The Pi

Mitglied
Hallo Leute,
Ich habe ein TicTacToe Programm mit der GUI erstellt, wo ich gerne hätte, dass die Felder nach dem Gewinn rot angezeigt werden sollen.
Das geht aber nicht. (Ich habe die Funktion "rot"):

Java:
/*
* To change this license header, choose License Headers in Project Properties.
* To change this template file, choose Tools | Templates
* and open the template in the editor.
*/
package tictactoegui;

import java.awt.Color;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.logging.Level;
import java.util.logging.Logger;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;

/**
*
* @author Info_2B
*/
public class Frame extends JFrame implements ActionListener {

    JPanel p = new JPanel();

    JButton b1 = new JButton();
    JButton b2 = new JButton();
    JButton b3 = new JButton();
    JButton b4 = new JButton();
    JButton b5 = new JButton();
    JButton b6 = new JButton();
    JButton b7 = new JButton();
    JButton b8 = new JButton();
    JButton b9 = new JButton();
    JButton button = new JButton();

    JLabel l = new JLabel("Spieler 1 ist dran");

    int status = 1;
    int gewinnt = 0;

    public Frame() {
        add(p);
        setSize(306, 360);
        setVisible(true);
        setResizable(false);
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        p.setLayout(null);

        p.add(b1);
        b1.setBounds(0, 0, 100, 100);
        b1.addActionListener(this);

        p.add(b2);
        b2.setBounds(100, 0, 100, 100);
        b2.addActionListener(this);

        p.add(b3);
        b3.setBounds(200, 0, 100, 100);
        b3.addActionListener(this);

        p.add(b4);
        b4.setBounds(0, 100, 100, 100);
        b4.addActionListener(this);

        p.add(b5);
        b5.setBounds(100, 100, 100, 100);
        b5.addActionListener(this);

        p.add(b6);
        b6.setBounds(200, 100, 100, 100);
        b6.addActionListener(this);

        p.add(b7);
        b7.setBounds(0, 200, 100, 100);
        b7.addActionListener(this);

        p.add(b8);
        b8.setBounds(100, 200, 100, 100);
        b8.addActionListener(this);

        p.add(b9);
        b9.setBounds(200, 200, 100, 100);
        b9.addActionListener(this);

        p.add(button);

        p.add(l);
        l.setBounds(0, 300, 306, 30);
        l.setHorizontalAlignment(JLabel.CENTER);

        Reset();
    }

    @Override
    public void actionPerformed(ActionEvent ae) {

        JButton button = (JButton) ae.getSource();

        button.setEnabled(false);

        if (gewinnt == 1) {
            status = 1;
            gewinnt = 0;
        } else if (gewinnt == 2) {
            status = 2;
            gewinnt = 0;
        }

        if (status == 1) {
            button.setBackground(Color.black);
            try {
                Sieg(status);
            } catch (InterruptedException ex) {
                Logger.getLogger(Frame.class.getName()).log(Level.SEVERE, null, ex);
            }
            status = 2;
        } else {
            button.setBackground(Color.yellow);
            try {
                Sieg(status);
            } catch (InterruptedException ex) {
                Logger.getLogger(Frame.class.getName()).log(Level.SEVERE, null, ex);
            }
            status = 1;
        }
    }

    public void Sieg(int status) throws InterruptedException {
        if (b1.getBackground() == Color.yellow && b2.getBackground() == Color.yellow && b3.getBackground() == Color.yellow
                || b4.getBackground() == Color.yellow && b5.getBackground() == Color.yellow && b6.getBackground() == Color.yellow
                || b7.getBackground() == Color.yellow && b8.getBackground() == Color.yellow && b9.getBackground() == Color.yellow
                || b1.getBackground() == Color.yellow && b4.getBackground() == Color.yellow && b7.getBackground() == Color.yellow
                || b2.getBackground() == Color.yellow && b5.getBackground() == Color.yellow && b8.getBackground() == Color.yellow
                || b3.getBackground() == Color.yellow && b6.getBackground() == Color.yellow && b9.getBackground() == Color.yellow
                || b1.getBackground() == Color.yellow && b5.getBackground() == Color.yellow && b9.getBackground() == Color.yellow
                || b3.getBackground() == Color.yellow && b5.getBackground() == Color.yellow && b7.getBackground() == Color.yellow) {
            l.setText("Spieler 2 gewinnt!");
            gewinnt = 2;
            rot();
            Thread.sleep(1600);
            Reset();
        } else if (b1.getBackground() == Color.black && b2.getBackground() == Color.black && b3.getBackground() == Color.black
                || b4.getBackground() == Color.black && b5.getBackground() == Color.black && b6.getBackground() == Color.black
                || b7.getBackground() == Color.black && b8.getBackground() == Color.black && b9.getBackground() == Color.black
                || b1.getBackground() == Color.black && b4.getBackground() == Color.black && b7.getBackground() == Color.black
                || b2.getBackground() == Color.black && b5.getBackground() == Color.black && b8.getBackground() == Color.black
                || b3.getBackground() == Color.black && b6.getBackground() == Color.black && b9.getBackground() == Color.black
                || b1.getBackground() == Color.black && b5.getBackground() == Color.black && b9.getBackground() == Color.black
                || b3.getBackground() == Color.black && b5.getBackground() == Color.black && b7.getBackground() == Color.black) {
            l.setText("Spieler 1 gewinnt");
            gewinnt = 1;
            rot();
            Thread.sleep(1600);
            Reset();           
        } else if (b1.getBackground() != button.getBackground()
                && b2.getBackground() != button.getBackground()
                && b3.getBackground() != button.getBackground()
                && b4.getBackground() != button.getBackground()
                && b5.getBackground() != button.getBackground()
                && b6.getBackground() != button.getBackground()
                && b7.getBackground() != button.getBackground()
                && b8.getBackground() != button.getBackground()
                && b9.getBackground() != button.getBackground()) {
            l.setText("Keiner gewinnt");
            Thread.sleep(1600);
            Reset();
        } else if (status == 1) {
            l.setText("Spieler 2 ist dran");
        } else {
            l.setText("Spieler 1 ist dran");
        }
    }

    public void Reset() {
        b1.setEnabled(true);
        b1.setBackground(button.getBackground());
        b2.setEnabled(true);
        b2.setBackground(button.getBackground());
        b3.setEnabled(true);
        b3.setBackground(button.getBackground());
        b4.setEnabled(true);
        b4.setBackground(button.getBackground());
        b5.setEnabled(true);
        b5.setBackground(button.getBackground());
        b6.setEnabled(true);
        b6.setBackground(button.getBackground());
        b7.setEnabled(true);
        b7.setBackground(button.getBackground());
        b8.setEnabled(true);
        b8.setBackground(button.getBackground());
        b9.setEnabled(true);
        b9.setBackground(button.getBackground());
    }
   
    public void rot(){
        if(b1.getBackground() == Color.black || b1.getBackground() == Color.yellow && b2.getBackground() == Color.black || b2.getBackground() == Color.yellow && b3.getBackground() == Color.black || b3.getBackground() == Color.yellow){
            b1.setBackground(Color.red);
            b2.setBackground(Color.red);
            b3.setBackground(Color.red);
        } else if(b4.getBackground() == Color.black || b4.getBackground() == Color.yellow && b5.getBackground() == Color.black || b5.getBackground() == Color.yellow && b6.getBackground() == Color.black || b6.getBackground() == Color.yellow){
            b4.setBackground(Color.red);
            b5.setBackground(Color.red);
            b6.setBackground(Color.red);
        } else if(b7.getBackground() == Color.black || b7.getBackground() == Color.yellow && b8.getBackground() == Color.black || b8.getBackground() == Color.yellow && b9.getBackground() == Color.black || b9.getBackground() == Color.yellow){
            b7.setBackground(Color.red);
            b8.setBackground(Color.red);
            b9.setBackground(Color.red);
        } else if(b1.getBackground() == Color.black || b1.getBackground() == Color.yellow && b4.getBackground() == Color.black || b4.getBackground() == Color.yellow && b7.getBackground() == Color.black || b7.getBackground() == Color.yellow){
            b1.setBackground(Color.red);
            b4.setBackground(Color.red);
            b7.setBackground(Color.red);
        } else if(b2.getBackground() == Color.black || b2.getBackground() == Color.yellow && b5.getBackground() == Color.black || b5.getBackground() == Color.yellow && b8.getBackground() == Color.black || b8.getBackground() == Color.yellow){
            b2.setBackground(Color.red);
            b5.setBackground(Color.red);
            b8.setBackground(Color.red);
        } else if(b3.getBackground() == Color.black || b3.getBackground() == Color.yellow && b6.getBackground() == Color.black || b6.getBackground() == Color.yellow && b9.getBackground() == Color.black || b9.getBackground() == Color.yellow){
            b3.setBackground(Color.red);
            b6.setBackground(Color.red);
            b9.setBackground(Color.red);
        } else if(b1.getBackground() == Color.black || b1.getBackground() == Color.yellow && b5.getBackground() == Color.black || b5.getBackground() == Color.yellow && b9.getBackground() == Color.black || b9.getBackground() == Color.yellow){
            b1.setBackground(Color.red);
            b5.setBackground(Color.red);
            b9.setBackground(Color.red);
        } else if(b3.getBackground() == Color.black || b3.getBackground() == Color.yellow && b5.getBackground() == Color.black || b5.getBackground() == Color.yellow && b7.getBackground() == Color.black || b7.getBackground() == Color.yellow){
            b3.setBackground(Color.red);
            b5.setBackground(Color.red);
            b7.setBackground(Color.red);
        }
    }
}

Hat jemand eine Idee, was fehlt?
 

Jardcore

Top Contributor
Es fehlt vor Allem die Einhaltung des DRY Prinzips. Don't Repeat Yourself.
Du Schreibst z.B x-Mal:
Java:
if(b3.getBackground() == Color.black || b3.getBackground() == Color.yellow &&
b5.getBackground() == Color.black || b5.getBackground() == Color.yellow &&
b7.getBackground() == Color.black || b7.getBackground() == Color.yellow){}

Du kannst das aber auch in mehreren Methode auslagern, dann musst du dich nicht immer wiederholen.
Java:
private boolean isBlack(Button b) {
    return b.getBackground() == Color.BLACK;
}
usw.

Ich glaube auch, das du diese ganzen hässlichen Abfragen auch alle gut hinter Methoden und Klasse verstecken kannst, bzw. vielleicht sogar gar nicht brauchst.

Wenn dein Code gut strukturiert ist, findest du auch leichter deine Fehler.

Hier vielleicht eine kleine Anregung, wie du erstmal komplett ohne GUI das Spiel entwickelst. Und später dann abhängig vom Status des Spiels die GUI einfach nur updatest.

Java:
public class VierGewinnt {

    private static final int PLAYER_1 = 1;
    private static final int PLAYER_2 = 2;
    private static final int EMPTY = 0;
  
    private static final int MAX_ROWS = 9;
    private static final int MAX_COLUMNS = 9;
  
    private int[][] field = new int[MAX_ROWS][MAX_COLUMNS];

    public void setField(int player, int row) {
        for(int column = 0; column < MAX_COLUMNS; column++) {
            if(field[row][column] == EMPTY) {
                field[row][column] = player;
                break;
            }
        }
    }
  
    public boolean winCondition() {
        for(int row = 0; row < MAX_ROWS; row++) {
            for(int column = 0; column < MAX_COLUMNS; column++) {
                if(horizonalVictory(row, column) ||
                verticalVictory(row, column) ||
                diagonalVictory(row, column)) {
                    return true;
                }
            }
        }
      
        return false;
    }
}
 
Zuletzt bearbeitet:
Ähnliche Java Themen
  Titel Forum Antworten Datum
Ernesto95 HTTP Mit JavaScript erzeugte dynamische Webseite auslesen und nach einem Schlüsselwort durchsuchen Allgemeine Java-Themen 6
D Image bewegt sich nicht nach Klicken auf Button Allgemeine Java-Themen 15
I 2D-Grafik Vektor-Grafik über die Zwischenablage nach Adobe Illustrator transferieren Allgemeine Java-Themen 8
M Suche nach String mit unbekannten characters Allgemeine Java-Themen 53
L 2 Dimensionale ListArray Abfrage nach einem Wert suchen Allgemeine Java-Themen 5
torresbig Url nach Webseiten-Login auslesen & Daten an Webseite senden Allgemeine Java-Themen 9
R Best Practice Erfahrungswerte für eine Migration von JSF nach Angular (oder anderes JS-Framework) Allgemeine Java-Themen 1
OnDemand Ram Freigabe erfolgt nicht nach Prozessende Allgemeine Java-Themen 18
G Geotools Probleme nach PC-Wechsel Allgemeine Java-Themen 6
K Verbesserung der Laufzeit beim Sortieren von Einwohnern nach ihrem Geburtsjahr Allgemeine Java-Themen 0
E Variablen Nach Übergabe einer Variable den Constructor aufrufen Allgemeine Java-Themen 16
I In Java geschriebene Software nach Mac OS portieren Allgemeine Java-Themen 7
M TicTacToe Sound nach jedem Zug Allgemeine Java-Themen 21
I HTML / XHTML Seite nach Excel exportieren. Suche Lib Allgemeine Java-Themen 12
J4n5chmiddi Methoden Website-URL im Browser öffnen nach erfolgreicher Basisauthentifizierung in Java Allgemeine Java-Themen 12
pkm Frage nach eventuellem syntaktischen Zucker bei der Konkatenation von ArrayLists Allgemeine Java-Themen 4
Monokuma String List nach Zahlen und Worten sortieren Allgemeine Java-Themen 9
H Collections Aktuellen Index generell und nach Sortierung ausgeben Allgemeine Java-Themen 6
Kirby.exe Filename nach bestimmtem Pattern durchsuchen Allgemeine Java-Themen 5
S Wörterliste nach Wörtern mit u durchsuchen und diese auf der Konsole ausgeben lassen Allgemeine Java-Themen 33
W Pdf verwerfen, weil Checkbox nach Unterschrift geaendert wurde Allgemeine Java-Themen 5
G File not found - nach dem Kompilieren Allgemeine Java-Themen 6
S Swing Speichern nach Button-Klick Allgemeine Java-Themen 5
Meeresgott Effizientester Weg um nach der Value einer verschachtelten Map aufzulösen Allgemeine Java-Themen 5
M Java 8 nach Java 6 konvertieren Allgemeine Java-Themen 7
N Neustarten des Codes nach der Fehlermeldung Allgemeine Java-Themen 17
L Nach dem Login // Java Desktop Software Allgemeine Java-Themen 7
N Programm nach Abschluss neustarten lassen Allgemeine Java-Themen 6
F Java Code ausführen direkt nach Anmelden in Windows Allgemeine Java-Themen 2
J Jasper Reports - Compilerproblem nach Umstellung von Groovy auf Java Allgemeine Java-Themen 7
looparda Liste filtern nach Prädikaten verschiedener Typen Allgemeine Java-Themen 3
S Apache POI Filtern nach bestimmten Kriterium Allgemeine Java-Themen 1
L Korrektur nach der Berechnung vornehmen, aber wie? Allgemeine Java-Themen 11
C Config nach bestimmten Wertdurchsuchen. Allgemeine Java-Themen 2
B Suche nach einem Testprogramm für meine BA Allgemeine Java-Themen 0
B Maven Keycloak library wirft exceptions nach maven package Allgemeine Java-Themen 1
D BufferedReader bricht nach 1248 Iterationen ab Allgemeine Java-Themen 14
G Eclipse Java findet MySQL Driver nach export nicht mehr Allgemeine Java-Themen 2
H IDEA IntelliJ Java Mail funktioniert nach Export nicht mehr! Allgemeine Java-Themen 1
F Zurnung nach Buchstaben und deren Prüfung Allgemeine Java-Themen 9
M Dateien nach kopieren vergleichen Allgemeine Java-Themen 9
MiMa Sortieren nach Stellenangaben Allgemeine Java-Themen 7
L Erste Schritte Liste von Datums filter nach Monate Allgemeine Java-Themen 4
GreenTeaYT Elemente eines 2Dim LinkedList von links nach rechts ausgeben? Allgemeine Java-Themen 0
J Ausgabe von Links nach Rechts ausgeben? Allgemeine Java-Themen 2
K JAR Datei Corrupt nach Kopieren Allgemeine Java-Themen 4
G Programm, das nach abgearbeiteter main Methode weiterläuft Allgemeine Java-Themen 72
C PDFBox: Nach RegEx ganze Zeile Allgemeine Java-Themen 4
R javax.comm --> Programm funktioniert nach Export nicht mehr Allgemeine Java-Themen 0
L Suche nach CalDav Server API Allgemeine Java-Themen 0
K Java ruft Methoden nicht der Reihe nach auf Allgemeine Java-Themen 14
T Textarea nach nur 1 wort durchsuchen Allgemeine Java-Themen 3
D Methoden Buttons erscheinen doppelt nach Wiederholung in Schleife Allgemeine Java-Themen 1
I nach Image Load in ListView, kann Ordner nicht mehr gelöscht werden Allgemeine Java-Themen 1
K Auf einer Website nach einem String suchen Allgemeine Java-Themen 5
C Eclipse OutOfMemory nach dem exportieren Allgemeine Java-Themen 4
D Erste Schritte Array von einer forschleife nach ausserhalb trasferieren Allgemeine Java-Themen 3
VfL_Freak Große und seltsame Probleme nach Java-Update auf V1.8.0_91 Allgemeine Java-Themen 3
heyluigi Random Integer Array Ausgabe nach Größe sortieren Allgemeine Java-Themen 6
D Java Datei nach Eclipse Export funktioniert nicht Allgemeine Java-Themen 0
B Bild aus Jar kann nach Export nicht mehr gefunden werden Allgemeine Java-Themen 13
B Umgebungsvariable Anpassen der Umgebungsvariablen nach Java-Update ? Allgemeine Java-Themen 14
H jid3lib nach schreiben keine Tags im Folder angezeigt Allgemeine Java-Themen 1
F Methoden Arraylist weiterverwenden nach methoden Aufruf Allgemeine Java-Themen 2
KilledByCheese Dezimal nach Hexadezimal rechner wirft seltsame exception Allgemeine Java-Themen 4
J Programm meldet "Keine Rückmeldung" nach Verbindung zum Server Allgemeine Java-Themen 4
E Java wird beendet nach paar Sekunden Allgemeine Java-Themen 14
H Best Practice setHeader in jsp nach RequestDispatcher.include Allgemeine Java-Themen 0
L Nach Button drücken den Text festspeichern Allgemeine Java-Themen 9
M .jar nach Datei prüfen Allgemeine Java-Themen 2
F String nach Schlüsselwörtern durchsuchen und ganze Zeile ausgeben Allgemeine Java-Themen 4
HarleyDavidson Input/Output Heruntergeladene Datei direkt nach dem Download öffnen ohne zu speichern Allgemeine Java-Themen 1
J Swing Cursor.WAIT funktioniert nicht nach JFileChooser Allgemeine Java-Themen 1
VfL_Freak JDK installieren Problem mit Erstellungspfad nach Wechsel von Java7 auf Java8 Allgemeine Java-Themen 1
B Eclipse Nach Export einer .jar Fehler: Hauptklasse konnte nicht gefunden oder geladen werden Allgemeine Java-Themen 5
thet1983 nach teilen eines Dateinamens suchen Allgemeine Java-Themen 6
F JLabel nach 5 Sekunden wieder leeren Allgemeine Java-Themen 7
M Fasta nach Mustern durchsuchen dauert zu lange Allgemeine Java-Themen 2
J Bilder halb in falscher Farbe nach kopieren aus Web Allgemeine Java-Themen 3
Thallius Neuen Prozess starten, der auch nach Beedingung des Starter-Prozesses weiterläuft? Allgemeine Java-Themen 5
T Nach Java Update: Jar Datein öffnen sich nicht mehr mit doppelklick Allgemeine Java-Themen 3
S Start des zweiten Threads erst nach Beenden des ersten Threads Allgemeine Java-Themen 13
A Funktionen aufrufen nach Schema x Allgemeine Java-Themen 2
G JavaFX Problem nach Update auf Java 8 Allgemeine Java-Themen 0
AssELAss String jeweils nach x Zeichen Zeilenumbruch Allgemeine Java-Themen 1
F E-Mail aus JAVA senden nach Umstellung auf Netbean 7.4 mit Java 7U45 nicht mehr möglich Allgemeine Java-Themen 4
J Ausgabe nach Excel Allgemeine Java-Themen 1
K PCM_UNSIGNED nach PCM_SIGNED Allgemeine Java-Themen 0
D Object nach Vererbung mit Class Object überprüfen Allgemeine Java-Themen 4
AssELAss Zeilenumbruch immer nach bestimmtem Zeichen Allgemeine Java-Themen 1
L Strings nach sortiertem String zurück ordnen Allgemeine Java-Themen 0
A Java - Suche nach Datensatz mit DateChooser Allgemeine Java-Themen 0
L Strings nach gleichem Muster ordnen Allgemeine Java-Themen 4
F Nach Export wird PDF Datei nicht mehr gefunden Allgemeine Java-Themen 0
K Sortieren nach Vorgabe Allgemeine Java-Themen 6
G nervendes Problem mit unterschieden zwischen Javax64 und x86 | je nach Programmbedarf beides nötig Allgemeine Java-Themen 2
L nach form submit textfeld an java übergeben? Allgemeine Java-Themen 2
L iText PDF Form-Felder werden nach Bearbeitung mit iText nicht mehr richtig erkannt. Allgemeine Java-Themen 2
C Objekt Datenverlust nach Methodenaufruf Allgemeine Java-Themen 9
P Datentypen String-Daten zu Byte-Zahlen konvertieren - Komme nicht weiter nach vielem versuchen :-/ Allgemeine Java-Themen 7

Ähnliche Java Themen

Neue Themen


Oben