GUI wird zu spät aufgebaut

[markus.s]

Hallo zusammen

Ich habe ein bisschen ein seltsames Problem. In einer Klasse wird ein Fenster mit diversem Inhalt aufgebaut und ein Midi-Ton produziert. Nun soweit so gut, alles funktioniert prima. Rufe ich nun diese Klasse durch Drücken eines Buttons von einer anderen Klasse aus auf, wird zuerst das leere JFrame dargestellt, danach der Ton abgespielt und erst am Schluss die restlichen GUI-Elemente aufgebaut.

import javax.sound.midi.InvalidMidiDataException;
import javax.sound.midi.MidiSystem;
import javax.sound.midi.MidiUnavailableException;
import javax.sound.midi.Receiver;
import javax.sound.midi.ShortMessage;
import javax.sound.midi.Synthesizer;
import javax.swing.*;


class testen extends JPanel {
	public testen(){//Konstruktor mit GUI Elementen
		JLabel blabla = new JLabel("Guten Morgen!!");
		add(blabla);
	}
	
public static void GUI (){
		JFrame fenster=new JFrame ();
		fenster.setSize (400,240);
		fenster.setTitle ("testen");
		fenster.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		JComponent newContentPane = new testen();
		newContentPane.setOpaque(true);
		fenster.setContentPane(newContentPane);
		fenster.setVisible(true);
	}

public static void ton() throws MidiUnavailableException, InvalidMidiDataException, InterruptedException {
	Synthesizer synth = MidiSystem.getSynthesizer();
    synth.open();
    Receiver rcvr = synth.getReceiver(); 
    ShortMessage msg = new ShortMessage(); 
    msg.setMessage(ShortMessage.NOTE_ON, 0, 68, 90); 
    rcvr.send(msg, -1); 
    Thread.sleep(2000);
    msg.setMessage(ShortMessage.NOTE_OFF, 0, 0, 0);
    rcvr.send(msg, -1); 
    synth.close(); 
   
}
	public static void main(String[] args) {
		testen.GUI();
		try {
			ton();
		} catch (Exception e) {
			e.printStackTrace();
		}
		}
	}

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JComponent;
import javax.swing.JFrame;
import javax.swing.JPanel;


public class testen2 extends JPanel implements ActionListener {
	public testen2(){
	JButton but = new JButton();
	but.addActionListener(this);
	
	add(but);
	}
	public static void GUI (){
		JFrame fenster=new JFrame (); 
		fenster.setSize (400,240);
		fenster.setTitle ("testen2");
		fenster.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		JComponent newContentPane = new testen2();
		newContentPane.setOpaque(true);
		fenster.setContentPane(newContentPane);
		fenster.setVisible(true);
	}
	public static void main(String[] args) {
		GUI();
		}

	public void actionPerformed(ActionEvent e) {
			testen.main(null);
}
}

Ich beschäftige mich noch nicht so lange mit Java. Hat jemand eine Idee, an was das liegen kann?

Vielen Dank

markus

 

[AlArenal]

Was erwartest du, wenn du den EDT 2 Sekunden anhältst?

 

[Guest]

Dass der Ton zwei Sekunden lang gespielt wird und dies das GUI, das bereits vorher aufgebaut sein müsste, nicht tangiert. Es funktioniert ja auch richtig, wenn ich die Klasse direkt starte, sprich nicht über eine andere Klasse aufrufe.

 

[Marco13]

 public void actionPerformed(ActionEvent e) 
{
    Thread t = new Thread(new Runnable()
    {
        public void run()
        {
             testen.main(null);
        }
    });
    t.start();
}

Aber schön ist das nicht. Eher im Gegenteil.

 

[markus.s]

Vielen Dank. So funktioniert es.

Allerdings begreife ich immer noch nicht ganz, warum der zusätzliche Thread notwendig ist. Könnt ihr mich da aufklären?

Aber schön ist das nicht. Eher im Gegenteil.[/code]

Wie macht denn ihr das? Muss ich etwas komplett anderst machen beim Code? Sry für die Fagen, ich kenne mich noch nicht so gut aus in Java.

 

[Marco13]

Hm. Eigentlich kenne ich mich da mit Java Audio zu wenig aus (d.h. ich weiß nicht, was dort wie in eigenen Threads gemacht wird, was direkt in Hardware gemacht wird (Midi!?) und wie die sonstigen Abläufe sind), aber grundsätzlich, und vielleicht (ums deutlicher zu machen) nicht übertrieben präzise:

Wenn du die Applikation startest (d.h. OHNE den Button-Klick sondern direkt), dann wird das in einem initialen Thread gemacht. Dieser initiale Thread "startet" den Event-Dispatch-Thread, der für ALLES verantwortlich ist, was mit dem GUI zu tun hat. D.h. wenn du die testen.main aufrufst (indem du 'testen' startest) dann führt der initiale Thread das ganze aus. Unter anderem wird 'ton' aufgerufen und darin der initiale thread für 2 Sekunden angehalten. Das stört den Event Dispatch Thread aber nicht, d.h. der kann dererweil das GUI aufbauen. (Wie gesagt, etwas unpräzise - so ganz stimmt das afaik nicht - eigentlich müßte das Aufbauen des GUI explizit in den EventDispatchThread ausgelagert werden)

Wenn du aber in der 'testen2' auf den Button Klickst, dann wird
public void actionPerformed(ActionEvent e) { testen.main(null); }
ausgeführt, und DAS passiert dann (wie alle Event-Handling Sachen) im Event-Dispatch-Thread. Der Event-Dispatch-Thread ruft dann also auch 'ton' auf, und wird für 2 Sekunden angehalten. So lange kann er nicht das GUI aufbauen (wofür er eigentlich da ist!)

Schau vielleicht auch nochmal hier:
http://java.sun.com/docs/books/tutorial/uiswing/concurrency/index.html


Die Kritik am Ende bezog sich darauf, dass der Ablauf ziemlich Unklar ist. Die statischen Methoden sollten vmlt. alle weg. Und die "main" von einer anderen Klasse aus per Hand aufzurufen ist schlecht. Dass man Klassennamen groß und Methodennamen klein schreibt ist im Vergleich dazu noch das geringste Übel.

 

[markus.s]

Vielen Dank. Du hast mir sehr geholfen.