Thread und JColorChooser?

[gastfrager]

hallo ich versuche hier gerade, einen jcolorchooser in nem thread laufen zu lassen, geht aber nicht. hier die exception:



Exception in thread "Thread-8" java.lang.NullPointerException
at ColorThread.run(ColorThread.java:12)



so, ich hab den thread also schon ausgelagert, aber es geht einfach nicht. hier der code von colorthread.java:

import javax.swing.JColorChooser;
import javax.swing.JFrame;

class ColorThread extends Thread {
    
	public void run() {
		Color chatTextColor = JColorChooser.showDialog(new JFrame("colortest"), "Farbe des Chattextes", Color.ORANGE);
			
		System.out.println("color: " + chatTextColor);
		int red = chatTextColor.getRed(); //HIER KOMMT DIE EXCEPTION, da chatTextColor=NULL
		int green = chatTextColor.getGreen();
		int blue = chatTextColor.getBlue();

		String redHex = Integer.toHexString(0x100 | red).substring(1).toUpperCase();
		String greenHex = Integer.toHexString(0x100 | green).substring(1).toUpperCase();
		String blueHex = Integer.toHexString(0x100 | blue).substring(1).toUpperCase();

		String colorString = "#" + redHex + greenHex + blueHex;
		System.out.println(colorString);
		
	}
}

der colorchooser wird angezeigt, aber jedesmal, wenn ich mir die farbe ausgesucht hab und OK drücke, kommt die obige exception. ohne thread gehts ohne eprobleme. ist das ein bug der VM? ich hab das java 1.5 jdk 1.5.0_06 installiert, aber mit dem 1.5.0_04 gings auch nicht. mit dem 1.5.0 gehts...seltsam. ne idee?

danke für jeden hinweis!!

 

[Mag1c]

Hi,

unter 1.4.2 gehts auch

darf man mal fragen, wozu das ganze in einem Thread laufen soll ?

Gruß
Mag1c

 

[gastfrager]

weil das sonst den chat lahmlegt, da es kein normaler colorchooser ist (ist etwas komplexer). toll, dann ists ein fehler in der JVM...wie könnte man das denn noch machen? habs auch schon mit prioritäten probiert...

gruß

 

[Roar]

lol, ich würd den fehler erstmal bei dir suchen anstatt in java: wenn der benutzer abbrechen geklickt oder den dialog einfach geshclossen hat wird null zurückgeliefert, also vorher auf null überprüfen bevor du weitermachst.

 

[Mag1c]

Hi,

der JColorChooser ist ein JComponent. Du kannst also auch einfach ein JFrame öffnen und dort eine Instanz von JColorChooser reinpacken. Musst dann nur noch die Buttons selbst drunterbaumeln, da die nur beim Dialog generiert werden. Das ganze sollte dann auch ohne Thread funktionieren und wäre auf jeden Fall sauberer.

Gruß
Mag1c

 

[gastfrager]

hi, ich will aber keine komponente selber bauen. wenn ich abbrechen klicke, dann kommt null, aber doch nicht, wenn ich ne farbe wähle und ok drücke...da es unter 1.5.0_0 läuft, nehm ich an, es liegt an der jvm...toll

gruß

 

[Roar]

ah, habs reproduzieren können. das liegt aber keineswegs an einem bug in der jre swing ist nicht threadsicher, dein showDialog() muss im awt edt ausgeführt werden: thread weglassen oder mit SwingUtilities.invokeAndWait() ausführen.

 

[gastfrager]

wie sieht das jetzt genau aus?

gruß und danke

 

[Roar]

SwingUtilities.invokeAndWait(irgendeinRunnableInDessenRunMethodeDuShowDialogAufrufst);
} catch (...) ...

 

[gastfrager]

also jetzt hab ichs so:

import java.awt.Color;
import java.lang.reflect.InvocationTargetException;

import javax.swing.JColorChooser;
import javax.swing.JFileChooser;
import javax.swing.JFrame;
import javax.swing.SwingUtilities;

class ColorThread extends Thread {
    
	public void run() {

		try {
			SwingUtilities.invokeAndWait(new Runnable() {
				public void run() {
					Color chatTextColor = JColorChooser.showDialog(new JFrame(), "Farbe des Chattextes", Color.ORANGE);

					System.out.println("color: " + chatTextColor);
					int red = chatTextColor.getRed();
					int green = chatTextColor.getGreen();
					int blue = chatTextColor.getBlue();

					String redHex = Integer.toHexString(0x100 | red).substring(1).toUpperCase();
					String greenHex = Integer.toHexString(0x100 | green).substring(1).toUpperCase();
					String blueHex = Integer.toHexString(0x100 | blue).substring(1).toUpperCase();

					String colorString = "#" + redHex + greenHex + blueHex;
					System.out.println(colorString);
				}
			});
		} catch (InterruptedException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		} catch (InvocationTargetException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}

	}
}

das geht...aber dann ist das aufrufende programm ja wieder lahmgelegt, oder wie macht der scheduler das jetzt?

gruß und danke

 

[Mag1c]

hehe, ich wollt grad sagen

wären wir bei Mathe, würde ich jetzt sagen, daß du den Thread gegen das invokeAndWait wegkürzen kannst :shock:

Bei dem Aufwand, den du bis hier schon getrieben hast, hättest du locker den JColorChooser in ein Frame packen können:

	public class ColorChooserFrame extends JFrame {
		public ColorChooserFrame(String title) {
			super(title);
			chooser = new JColorChooser();
			add(chooser, BorderLayout.CENTER);
			JPanel btnPanel = new JPanel();
			JButton okBtn = new JButton("Ok");
			okBtn.addActionListener(new ActionListener() {
				public void actionPerformed(java.awt.event.ActionEvent e) {
					Color color = chooser.getColor();
					// hier jetzt was mit der Farbe machen
					dispose();
				};
			});
			btnPanel.add(okBtn);
			JButton cancelBtn = new JButton("Cancel");
			cancelBtn.addActionListener(new ActionListener() {
				public void actionPerformed(java.awt.event.ActionEvent e) {
					dispose();
				};
			});
			btnPanel.add(cancelBtn);
			add(btnPanel, BorderLayout.SOUTH);
			pack();
		}

		private JColorChooser chooser;
	}

Gruß
Mag1c

 

[Roar]

hehe so ist der trhead ja ziemlich unnötig
wenn ich richtig lese wolltest du den colorchooser in nen trehad legen weil sonst dein chat hängt? du musst das IO zeugs in einen anderen thread auslagern, nicht das gui zeugs, das muss nämlich im gui thread stattfinden.