Ampelsteuerung mit Timer

[Kanissell]

hi,

ich habe hier meine Ampelsteuerung implementiert:

public class Ampelsteuerung implements ActionListener{


    protected Ampel[] ampeln;
    
    public void actionPerformed(ActionEvent e){
        if (e.getActionCommand().equals("NEXT_PHASE")){
        for (Ampel ampel : this.ampeln) {
            ampel.toNextPhase();
            }
        }
    }




}

und hier meine public class Ampel:

public class Ampel implements Drawable {
    
    protected int x;
    protected int y;
    protected String phase;
    
    public Ampel(int x, int y, String phase){
        this.x = x;
        this.y = y;
        this.phase = phase;
    }
    
    public Ampel(int x, int y){
        this.phase = Ampelphase.rot;
        this.x = x;
        this.y = y;
    }
    
    public Ampel(Ampel a){
        this.x = a.x;
        this.y = a.y;
        this.phase = a.phase;
    }
    
    public String toString(){
        return String.format("[X = %03d Y = %03d Phase = %03s]", x, y, phase);
    }


    public void toNextPhase(){
        if (phase == "rot"){
            phase = Ampelphase.rotGelb;
        }
        if (phase == "rotGelb"){
            phase = Ampelphase.gruen;
        }
        if (phase == "gelb"){
            phase = Ampelphase.rot;
        }
        if (phase == "grün"){
            phase = Ampelphase.gelb;
        }
    }
    
    public void draw(Graphics g){
        g.setColor(Color.GRAY);
        g.fillRect(x, y, 100, 200);
        
        if (phase == "rot"){
            g.setColor(Color.RED);
            g.fillOval(x + 25 , y + 15, 50, 50);
            g.setColor(Color.LIGHT_GRAY);
            g.fillOval(x + 25, y + 70, 50, 50);
            g.setColor(Color.LIGHT_GRAY);
            g.fillOval(x + 25, y + 125, 50, 50);
        }
        if (phase == "gelb"){
            g.setColor(Color.LIGHT_GRAY);
            g.fillOval(x + 25 , y + 15, 50, 50);
            g.setColor(Color.YELLOW);
            g.fillOval(x + 25, y + 70, 50, 50);
            g.setColor(Color.LIGHT_GRAY);
            g.fillOval(x + 25, y + 125, 50, 50);
        }
        if (phase == "grün"){
            g.setColor(Color.LIGHT_GRAY);
            g.fillOval(x + 25 , y + 15, 50, 50);
            g.setColor(Color.LIGHT_GRAY);
            g.fillOval(x + 25, y + 70, 50, 50);
            g.setColor(Color.GREEN);
            g.fillOval(x + 25, y + 125, 50, 50);
        }
        
        
    }


}

in der Klasse Ampeltest sollen wir nun die Ampeln ausgeben lassen mit dem timer:

public class Ampeltest {


    public static void main(String[] args) {
        DirtyPainter painter = new DirtyPainter();
        Ampel a = new Ampel(3, 3);
        Ampelsteuerung steuerung = new Ampelsteuerung();
        Timer timer = new Timer(1500, steuerung);
        timer.setActionCommand("NEXT_PHASE");
        timer.addActionListener(steuerung);
        timer.start();
        painter.add(a);
        painter.showDrawing();
        


    }


}

meine Frage ist jetzt, wie ich der Ampelsteuerung bescheid geben kann, das der Timer gestartet ist und er alle Ampeln im Array umschalten soll ?

Ampelsteuerung, Ampel und Ampeltest sind eigene public classes.

Danke für eure Antworten

 

[InfectedBytes]

Deine Ampelsteuerung wird automatisch alle 1,5 Sekunden benachrichtigt. Das Problem ist vermutlich dass du die Änderung nur nicht siehst, da du keine repaint() Aufruf machst.
Füge in deiner Ampel einfach mal einen repaint()-Aufruf am Ende der toNextPhase() Methode ein.
(bzw. wie auch immer Drawable zum neuzeichnen angeregt wird^^)

Ansonsten fügst du deine Steuerung zweimal als listener hinzu.
Dem Timer Konstruktor übergibst du den Listener und außerdem fügst du ihn nochmal mit addListener hinzu.

 

[Kanissell]

hi,

das mit dem repaint hab ich dann nach der toNextPhase eingefügt im Aufruf

public class Ampelsteuerung implements ActionListener{


	protected Ampel[] ampeln;
	
	public Ampelsteuerung(Ampel[] ampel){
		this.ampeln = ampel;
	}
	
	public void actionPerformed(ActionEvent e){
		DirtyPainter painter = new DirtyPainter();
		if (e.getActionCommand().equals("Next_Phase")){
		for (Ampel traffic : this.ampeln) {
			traffic.toNextPhase();
			painter.add(traffic);
			}
		painter.showDrawing();
		}
		
	}

aber irgendwie ist der Phasenübergang falsch, er springt von Rot nach Grün und umgekehrt und bei Gelb bleibt er auf Gelb..

 

[InfectedBytes]

Wie gesagt, du fügst den Listener zweimal hinzu, einmal durch den Konstruktor und einmal durch addListener

Timer timer = new Timer(1500, steuerung);
timer.addListener(steuerung)

Daher wird der Code auch zweimal ausgeführt.
Entferne einfach das addListener

 

[Kanissell]

Aso, sorry hab mich da wohl bisschen verlesen

Es klappt jetzt zu 90%, manchmal springt er immernoch von grün nach rot, aber nur alle 5 schritte

 

[Kanissell]

War mein Fehler, hab da was doppelt ausgeführt

Vielen Dank für deine Hilfe, hast mir sehr geholfen

 

[egos]

@Kanissell
Als blutiger Anfänger hat mich dein Programm sehr interessiert. Was ich nicht herausgefunden habe: wo findet man den DirtyPainter? das gibt bei mir einen Fehler. Ebenso die Anweisung ...implements Drawable. Gibt es dazu eine Importanweisung?
Gruss Oski

 

[Kanissell]

@egos

hi,

der DirtyPainter hat unser Prof selber geschrieben, er wollte uns damit das Leben leichter machen ( was ich aber nicht finde )

du kannst anstatt den Dirtypainter einfach den aufruf ampel.repaint machen, macht genau das gleiche.

und Drawable hat uns auch gegeben, es gibt aber sicherlich ein äquivalent dazu.

Falls du die Bibliotheken haben willst, schreib mir und ich schicke sie dir

 

[egos]

@Karnissell

Danke für die Antwort und dein Angebot. Ich möchte nur mit der SE arbeiten, ohne "Fremdbibliotheken". Es gibt bei mir noch einige Fehler mehr. Zum Beispiel Ampelphase.rot usw. kann Java nicht verarbeiten. Auch ampel.repaint wird nicht akzeptiert. Bei der Instanzierung von Ampelsteuerung wird ein undefinierter Konstruktor gemeldet.
Ich versuche es auf einem anderen Weg.
Gruss Oski