thread nimmt veränderte boolean nicht

[Pascale07]

Ich lasse aktuell durchgehend eine while schleife in einem anderen Thread laufen. Dort werden public booleans aus dieser Klasse geprüft. Ist eine boolean true, soll in einer anderen Klasse ein JPannel erstellt werden und einem JFrame hinzugefügt werden. Jetzt ist in einem JPannel ein button, der eine der Variablen auf true setzt. Trotz dessen passiert nichts. Wenn ich die Variable aber manuel auf true setzte macht er es. Kann mir jemand sagen, wie ich das Problem lösen kann?

Der Code:

public class Start implements Runnable{

public boolean running = true;
public boolean multi = false;
public boolean single = false;
public boolean Startscreens = true;
public static JFrame window = new JFrame();

public static void main(String[] args){

window.setLocationRelativeTo(null);
window.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
window.setTitle("Monopoly");
window.setResizable(false);
window.setVisible(true);
window.setLocationRelativeTo(null);


Start start = new Start();
start.startGameThread();


}

public void startGameThread(){
Thread gameThread;
gameThread = new Thread(this);
gameThread.start();
}


@Override
public void run(){

gamePanel panel = new gamePanel();
startscreen Startscreen = new startscreen();

while(running){

if(Startscreens){

window.add(Startscreen);
window.pack();
window.setLocationRelativeTo(null);
start.Startscreens = false;

}

if(multi){ //dieses boolean wird mit dem button verändert, trotz dessen wir der Code in der if nicht ausgeführd

window.add(panel);
window.pack();
window.setLocationRelativeTo(null);
start.multi = false;

}

}

}


}



public startscreen(){

gamePanel Gamepannel = new gamePanel();
Start start = new Start();

this.setPreferredSize(new Dimension(Gamepannel.width, Gamepannel.height));
this.setDoubleBuffered(true);
this.setFocusable(true);
this.setVisible(true);


JButton multiplayer = new JButton("MULTIPLAYER");
multiplayer.setVisible(true);
multiplayer.setPreferredSize(new Dimension(300, 100));

multiplayer.addActionListener(new ActionListener(){

public void actionPerformed(ActionEvent e){

start.multi = true;

}

});

this.add(multiplayer);


}

}

 

[KonradN]

Erst einmal: Code bitte immer in Code Tags posten! Also einfach den Knopf </> links oberhalb des Eingabebereiches nutzen!

Zu Deinem Problem: Java "cached" Variablen zwischen Threads. Wenn Du auf eine Variable mit mehreren Threads zugreifen willst, dann sollte die Variable volatile sein.

Dazu kannst Du im Internet sehr viel finden. Ein Baeldung Beitrag erläutert die Hintergrund ebenfalls:

Guide to the Volatile Keyword in Java | Baeldung

Learn about the Java volatile keyword and its capabilities.

www.baeldung.com

 

[Pascale07]

Erst einmal: Code bitte immer in Code Tags posten! Also einfach den Knopf </> links oberhalb des Eingabebereiches nutzen!

Zu Deinem Problem: Java "cached" Variablen zwischen Threads. Wenn Du auf eine Variable mit mehreren Threads zugreifen willst, dann sollte die Variable volatile sein.

Dazu kannst Du im Internet sehr viel finden. Ein Baeldung Beitrag erläutert die Hintergrund ebenfalls:

Guide to the Volatile Keyword in Java | Baeldung

Learn about the Java volatile keyword and its capabilities.

www.baeldung.com

Irgendwas scheine ich nicht verstanden zu haben. Wenn ich die Variable als volatile deklariere und diese dann verändere, wird die if-Anweisung trotzdem nicht ausgeführt. Zudem hatte ich gelesen, dass es reichen würde, dass eine variable public ist um von allen Threads gelesen werden zu können. Wenn ich jetzt was falsch verstanden habe, lassen sie es mich bitte wissen.
Trotzdem danke für ihre Hilfe

 

[KonradN]

Du hast auch noch ein weiteres Problem:

Du hast zwei Instanzen von Start. Das ist wie, wenn Du zwei Autos hast: Wenn Du in den Kofferraum des einen Autos etwas legst, dann wirst Du es im Kofferraum des zweiten Autos nicht finden!

Das ist aber blöd zu sehen, da der Code so nicht formatiert ist. Wäre super, wenn Du den Code noch einmal in Code Tags posten würdest (über den </> Knopf!).

 

[KonradN]

Ich habe jetzt versucht durch den Code zu schauen, aber da scheint etwas nicht zu stimmen - public startscreen(){ muss doch ein Konstruktor sein, damit es valide ist, aber ich habe nur die Klasse Start gesehen? Oder übersehe ich etwas?

Wenn Du zwei Klassen hast, dann sollte der Konstruktor von startscreen eine Instanz von Start nehmen. Dann wird die Initialisierung zu
startscreen Startscreen = new startscreen(this);

Die Instanz von Start speichert Su dann in einer Instanzvariable, d.h. Du erzeugst da keine neue Instanz mehr sondern nutzt die übergebene.

Und schon klappt es mit den Zugriffen.

 

[Pascale07]

Also ich lade jetzt nochmal den Code richtig hoch. Allerdings verstehe ich die Erklärung nicht. Wenn ich dort this eintrage, dann bekomme ich einen Error, dass der konstruktor startscreen in der Klasse startscreen nicht auf den angegebenen Typen angewendet werden kann. Tut mir leid, dass ich so begriffsstutzig bin. Ich bin recht neu in java und dachte mir ich lerne am Besten, wenn ich einfach ein Projekt starte.

import javax.swing.*;

public class Start implements Runnable{
    
  public volatile boolean running = true;
  public volatile boolean multi = false;
  public volatile boolean single = false;
  public volatile boolean Startscreens = true;
  public static JFrame window = new JFrame();
 
  public static void main(String[] args){
      
    window.setLocationRelativeTo(null);
    window.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    window.setTitle("Monopoly");
    window.setResizable(false);
    window.setVisible(true);
    window.setLocationRelativeTo(null);
    
    
    Start start = new Start();
    start.startGameThread();
    
    
  }
 
  public void startGameThread(){
    Thread gameThread;
    gameThread = new Thread(this);
    gameThread.start();
  }
 
 
  @Override
  public void run(){
    
    gamePanel panel = new gamePanel();
    startscreen Startscreen = new startscreen();
        
    while(running){
      
      System.out.println(multi);
      
      if(Startscreens){
        System.out.println("ki");   
        window.add(Startscreen);
        window.pack();
        window.setLocationRelativeTo(null);
        Startscreens = false;
        
      }
      
      if(multi == true){
        System.out.print("hi"); 
        window.add(panel);
        window.pack();
        window.setLocationRelativeTo(null);
        multi = false;
          
      }
      
      try{
         Thread.sleep(1000);
      }catch(Exception e){}
      
    }
    
  }
 
 
}

public class startscreen extends JPanel{
 
  public startscreen(){
      
    gamePanel Gamepannel = new gamePanel();
    Start start = new Start();
    
    this.setPreferredSize(new Dimension(Gamepannel.width, Gamepannel.height));
    this.setDoubleBuffered(true);
    this.setFocusable(true);
    this.setVisible(true);
 
 
    JButton multiplayer = new JButton("MULTIPLAYER");
    multiplayer.setVisible(true);
    multiplayer.setPreferredSize(new Dimension(300, 100));
 
    multiplayer.addActionListener(new ActionListener(){
      
      public void actionPerformed(ActionEvent e){
        
        Start start = new Start();
        start.multi = true;
            
      } 
    
    });
 
 
    JButton singleplayer = new JButton();
 
    this.add(multiplayer);
 
    
  }
 
}

 

[KonradN]

Du musst die KLasse startscreen natürlich auch anpassen, so dass der Konstruktor eine Instanz von Start entgegen nimmt. Also das, was ich hier geschrieben habe:

Wenn Du zwei Klassen hast, dann sollte der Konstruktor von startscreen eine Instanz von Start nehmen.

Die Instanz von Start speichert Su dann in einer Instanzvariable

Das würde dann so aussehen:

public class startscreen extends JPanel{
  private Start start;  // Du hast jetzt eine Instanzvariable.
    
  public startscreen(Start start){ // Hier hast Du jetzt einen Parameter
      
    gamePanel Gamepannel = new gamePanel();
    this.start = start;  // Und hier speicherst Du, was Du im Parameter bekommen hast
    
    this.setPreferredSize(new Dimension(Gamepannel.width, Gamepannel.height));
    this.setDoubleBuffered(true);
    this.setFocusable(true);
    this.setVisible(true);
 
 
    JButton multiplayer = new JButton("MULTIPLAYER");
    multiplayer.setVisible(true);
    multiplayer.setPreferredSize(new Dimension(300, 100));
 
    multiplayer.addActionListener(new ActionListener(){
      
      public void actionPerformed(ActionEvent e){
        
// Diese Zeile muss natürlich auch weg!
//        Start start = new Start();
        start.multi = true;
            
      }
    
    });
 
 
    JButton singleplayer = new JButton();
 
    this.add(multiplayer);
 
    
  }
 
}

Und es gab noch eine weitere Änderung - Du darfst in dem ActionListener natürlich keine neue Instanz schaffen.

 

[Pascale07]

Es klapp jetzt. Viele dank für ihre Hilfe.