Aktualisierung eines Buttons

[Ape50]

Hey Leute,
ich hab mal wieder ein Problem:
ich schreib ein Schachprogramm und möchte verschiedene KIs gegeneinander spielen lassen und dabei zugucken können. Das klappt auch alles bis auf das zugucken. Wenn ich auf start drücke wird ein Spiel durchgespielt und dann das ende des spiels angezeigt. und wenn ich dazu noch ein
try{Thread.sleep(200);}catch(InterruptedException e){e.printStackTrace();}
einbaue, passiert gar nichts mehr.
ich denke ich weiß nich genau wie oder wann sich so ein button (oder die oberfläche) repaintet...

kann mir jmd dazu was sagen bzw einen tip geben?

 

[Robat]

Lässt du die Logik zur Berechnung in einem separaten Thread laufen? Wann aktualisierst du die GUI denn?

 

[Ape50]

äh nein, nicht, dass ich wüsste ;P
wie mach ich das?

 

[Ape50]

die gui wird immer nach einem zug aktualisiert in meiner organisationsklasse

 

[Robat]

Lies dich mal zu Threads ein. Deine Berechnungen blockieren wahrscheinlich den EDT weshalb du am Ende erst die aktualisierte GUI siehst

 

[Ape50]

ok super. danke dir!

 

[Ape50]

ok jetzt klappt es meistens... manchmal werden allerdings nicht alle buttons angezeigt. also nicht 8*8 sondern z.b 3*4+5*8(auffällig ist, dass immer genau 8 zeilen da sind, die spalten aber teilweise unterschiedlich und nicht gleich 8 sind)
ich muss sagen, dass ich das mit den threads noch nicht zu 100% durchblickt hab und mir sowas nicht erklären kann...

 

[Robat]

Naja ohne Code kann man da nicht viel mehr sagen als "da läuft wohl was schief"

 

[Ape50]

ok ich versuchs mal...

public class Orga extends Thread{
    private int aus_x;
    private int aus_y;
    private GuiFeld guifeld;
    private PrimFeld primfeld;
    private App app;
    private Menue menue;
    private Schachputer wcomp;
    private Schachputer scomp;
    private int zuganz;
   
    public Orga() {
       
    }
    public Orga(App app) {
       
        this.app=app;
        this.primfeld=new PrimFeld();
        this.menue= new Menue(this);
        this.guifeld=new GuiFeld(this);
        this.aus_x=-1;
        this.aus_y=-1;
        this.zuganz=0;
       
        this.setComp();
    }
   
    public void setComp() {
        int wc=this.menue.Player(true);
        switch(wc) {
        case 0:{
            wcomp=null;
            break;
        }
        case 1:{
            this.wcomp=new Randomchess();
            break;
        }
        default:wcomp=null;
        }
        int sc=this.menue.Player(false);
        switch(sc) {
        case 0:{
            scomp=null;
            break;
        }
        case 1:{
            this.scomp=new Randomchess();
            break;
        }
        default:scomp=null;
        }
       
//        if(bug) {
//            System.out.println(sc);
//        }
       
        if(this.wcomp!=null && this.scomp!=null) {
            this.start();

        }else if(this.wcomp!=null) {
            this.lasscompziehen();
        }
    }
    public void run() {
        while(this.zuganz<100) {
            this.lasscompziehen();
            try
            {
              Thread.sleep(500);     
            }
            catch(InterruptedException ie)
            {
            }
        }
    }
    private void lasscompziehen() {
       
        int[] zug=null;
        if(this.primfeld.weißamzug())zug=this.wcomp.getZug();
        else zug=this.scomp.getZug();
       
        this.aus_x=zug[0];
        this.aus_y=zug[1];
        this.ziehen( zug[2], zug[3] );
    }

    public void ziehen(int x, int y) {
        this.zuganz++;
       
        LinkedList<int[]> zuege= new LinkedList();
        zuege.add(new int[] {this.aus_x,this.aus_y, x, y});
        zuege.add(this.primfeld.ziehen(this.aus_x,this.aus_y, x, y));
       
        Iterator<int[]> i= zuege.iterator();
        while(i.hasNext()) {
            int[] ar=i.next();
            this.guifeld.setFig(ar[0], ar[1]);
            this.guifeld.setFig(ar[2], ar[3]);
        }
        boolean amzug=this.primfeld.weißamzug();
        if(this.wcomp!=null) {
            this.wcomp.ziehen(this.aus_x,this.aus_y, x, y);
        }
        if(this.scomp!=null) {
            this.scomp.ziehen(this.aus_x,this.aus_y, x, y);
        }

        this.guifeld.addButtons();
    }
...
}

public class GuiFeld extends JPanel{

    private boolean bug=false;
    private JButton[][] button;
    private Orga org;
   
    public GuiFeld(Orga org) {

        this.setLayout(new GridLayout(8,8));
       
        //his.button= new SchachButton[8][8];
        this.button=new JButton[8][8];
        this.org=org;
       
       
        for(int i =0; i<8;i++) {
            for (int k=0;k<8;k++) {
                this.button[i][k]=new JButton();
                this.button[i][k].addActionListener(new FrameBut(i,k,this.org));
            }
        }
        this.setFiguren();
        this.addButtons();
    }
 
    public void addButtons() {
        this.removeAll();
        for(int i =7; i>=0;i--) {
            for (int k=0;k<8;k++) {
                this.add(button[i][k]);
            }
        }
        this.repaint();
    }
...
}

reicht der code? die anderen methoden und klassen sollten eigentlich nich viel ändern, denk ich

 

[mihe7]

reicht der code?

Gut gemeinter Rat: schreib Dein Programm nochmal neu und zwar ohne GUI (dafür mit vernünftigen Bezeichnern ). Konzentriere Dich dabei auf den Kern; die Darstellung am Bildschirm ist völlig nebensächlich. Wenn Du das hast, kannst Du später eine grafische Oberfläche vernünftig anbinden. Bis dahin kannst Du Dein Spielfeld auch auf der Konsole ausgeben und Züge über Texteingaben ausführen.

 

[Robat]

Da muss ich @mihe7 Recht geben.
Dein Code ist momentan sehr unstrukturiert, sodass es sehr schwer ist dort den Überblick zu bekommen.

Abgesehen davon noch eine Frage zu deinem Problem: kannst du den Fehler irgendwie reproduzieren? War dieser vor den Threads schon da?

 

[mihe7]

Ich denke, da fehlen zwei Dinge: 1. invokeLater und 2. revalidate()

 

[Ape50]

es klappt alles außer die grafische ausgabe. und die klappt auch nur nicht, wenn der pc gegen den pc spielt...

 

[Ape50]

ja sry... ich hab lange nicht programmiert und wollte das mal wieder etwas auffrischen. und weil mir schach spaß macht dieses projekt.
ich denke ich verstehe nicht wirklich warum und wo der ablauf nicht nacheinander abgefertigt wird. also warum aktualisiert sich der button nicht sofort, wenn man ihm sagt akualisier dich, sonder scheinbar erst, wenn die funktion, die das möchte, beendet ist.
ist das überhaupt richtig?
das bedeutet ja, dass die oberfläche ein thread ist, oder? also muss ich irgendwie meinem denkenden programm an der stelle sagen, dass es warten soll bis die oberfläche ihr zeugs erledigt hat. einfach schlafen zu legen scheint nicht genug zu sein.
ist das so erstmal richtig?
Kann ich mein JFrame irgendwie als Thread betrachten?

 

[Ape50]

ok, also revalidate() scheint für das problem die lösung zu sein.... vielen dank

nächstes problem: threads kann man ja nicht noch einmal starten, wenn sie einmal angehalten haben, richtig?
das bedeutet ich muss, um ein spiel neu zu starten, mein denkendes programm(weil es ja ein thread ist) wegwerfen und mir ein neues erzeugen. oder gibt es da andere alternativen?

 

[Robat]

threads kann man ja nicht noch einmal starten, wenn sie einmal angehalten haben, richtig?

Das ist richtig

Ein Thread bekommt einen Runnable übergeben. Der Runnable enthält die Logik, welche in dem Thread ausgeführt werden soll.
Was du also machen kannst, ist dir die Logik (also das Runnable-Objekt) zu speichern und immer einen neuen Thread zu starten

// Deine aktuelle Lösung
Thread t1 = new Thread(() -> System.out.println("Hello World"));
t1.start();
Thread t2 = new Thread(() -> System.out.println("Hello World"));
t2.start();
...

// Mein Vorschlag:
Runnable logic = () -> System.out.println("Hello World");
Thread t1 = new Thread(logic);
t1.start();
Thread t2 = new Thread(logic);
t2.start();
...

 

[Robat]

das bedeutet ja, dass die oberfläche ein thread ist, oder?

Dazu schaust du dir am Besten mal den Event Dispatch Thread an

 

[Ape50]

danke dir!!!

in 99.9 % der fälle läuft jetzt alles rund...

 

[Robat]

in 99.9 % der fälle läuft jetzt alles rund...

Also klappt ja immer noch irgendwas nicht