Probleme bei der Gui Anzeige

[SwingAwt]

Hallo, ich habe ein Programm in Java das einen Binärbaum darstellen soll. Leider wird die Gui nicht richtig angezeigt. (Buttons und Textfelder werden erst angezeigt wenn man an ihre eigentliche Position klickt. Wenn ich das ganze auf AWT umstelle klappt alles. Ich kann mir aber nicht erklären warum es in Swing nicht klappt, weil ich das doch eigentlich nicht mische oder ? Gut ich habe zwar Objekte der Klasse Graphics und Image aber ich dachte bis dato das das kein Problem wäre und ich nur nicht z.B Button und JButton mischen dürfte ....

Wie kann ich das ganze denn dann auf Swing umschreiben ?

Hier mal der Code :
[Java] import java.awt.*; import javax.swing.*; import java.awt.event.*; public cl - Pastebin.com

(Habe ihn der Lesbarkeit halber auf pastebin hochgeladen, da sonst hier der Beitrag so lang geworden wäre)

 

[SlaterB]

soll das denn eine normale Gui sein, ein Frame wie die Klasse auch heißt, oder tatsächlich ein Applet?

jedenfalls malst du in paint() ein Image über die gesamte Fläche, kein Wunder dass darunter Komponenten leiden,
sie werden übermalt

 

[Michael...]

Hab's mir nicht genau angeschaut, aber was mir aufgefallen ist:
- Wenn man einem BorderLayout bzw. dessen Komponente andere Komponenten übergibt, sollte man den Bereich (z.B. NORTH, SOUTH) mit angeben, sonst werden alle Komponenten in den CENTER Bereich gesetzt und die letzte gewinnt.
- Ich würde niemals (zumindest nicht für solche Zwecke) die paint eines JApplets (gilt auf für JFrame) überschreiben. Besser man nimmt ein JPanel überschreibt dessen paintComponent(Graphics g) und platziert es als einzige Komponente in den CENTER Bereich des JApplets
- Nach dem Zeichnen auf des BufferedImage sollte man die Graphics Resource wieder mit dispose() freigeben. D.h. nicht am Anfang einmal mit getGraphics() (empfohlen wird createGraphics()), sondern jedes mal vor dem Neuzeichnen das Graphics Objekt besorgen.

 

[irgendjemand2]

in swing sollte man auch kein paint() sondern paintComponent() verwenden und darin auch super.paintComponent() callen ... sonst würdest du wieder AWT und swing mischen ... was man wie du shcon sagtest vermeiden sollte

das problem : JApplet erbt dierekt von Applet .. und nicht von JComponent ... wesshalb du dort kein paintComponent() hast ...

sinnvoll wäre es jetzt mit einem JLayeredPane und JPanel zu arbeiten ...
dann zeichnest du auf das untere JPanel mit paintComponent() dein bild ... und legst dann das andere JPanel auf dem deine JComponent's sind eine eben höher ...

so habe ich es zumindest gemacht als ich mal mit sowas gearbeitet habe

 

[SwingAwt]

Hm also das ich das Bild über das ganze Fenster zeichne kann doch eigentlich nicht sein, weil es mit AWT Komponenten funktioniert, also diese nicht übermalt werden.

Ich weiß das ganze ist ein bisschen quick&dirty und das beste wäre es wohl das ganze so wie vorgeschlagen mit einem Panel und der paintComponent zu lösen. Trotzdem wäre mir für den Augenblick eine schnelle Lösung lieber.... Also kann ich das ganze hier relativ schnell lösen oder muss ich dazu mehr Zeit aufwenden ?

PS: Ja eigentlich sollte es ein Frame werden, wollte es nachdem alles richtig angezeigt wird zu einem Frame umschreiben.

 

[SlaterB]

wenn am Ende ein AWT-Frame draus wird, dann mag es mit paint ja hinkommen, warum wechselst du nicht gleich dorthin?

ansonsten gibts nichts schnelleres als das korrekte
> mit einem [J]Panel und der paintComponent zu lösen

sind ja auch nur 10 Zeilen neuer Code, Rest vielleicht kopieren,
für AWT musst du das dann allerdings später wieder umändern, falls da überhaupt Panel + paint geht,
habe irgendwie in Erinnerung dass das nicht so schön wird,

gibt dort ja noch java.awt.Canvas..

oder eben wie aktuell paint im Frame, nicht zu verachten

 

[SwingAwt]

Hm hatte gehofft es auch mit Swing lösen zu können. Also Fakt ist das wenn ich es mit Swing lösen will, kein Weg über den Panel geht ?

 

[L-ectron-X]

Nicht mit Panel aber mit JPanel.

 

[SlaterB]

ja, das kann man als einen Fakt unterschreiben

 

[SwingAwt]

Ok habe das ganze jetzt mit dem Panel versucht aber Scheitere bisher immer noch. Habe das Image jetzt mit einem BufferedImage ersetzt und eine neue Klasse die von JPanel erbt erstellt. Diese zeichnet dann das Bild und eine Instanz dieser Klasse wird im Center des JFrames angezeigt. So weit so gut. Doch wie kriege ich jetzt die paint Methode des JFrames weg und zeichne alles auf dem center Panel ?

 

[SwingAwt]

Achso sorry beim uploaden des Codes ist was schief gegangen irgendwie kann man ihn nicht mehr einsehen also hier nochmal:
Frame:

import java.awt.*;
import javax.swing.*;
import java.awt.event.*;
import java.awt.image.*;


public class BaumFrame extends JFrame implements ActionListener
{
	static final int breite = 128;
	JTextField einF, delF, suchF;
	JButton einB, randB, del1B, del1randB, suchB;
	Element root;
	int tiefe, anzahl, mtief;
	Font font = new Font("Times", Font.PLAIN, 9);
    Font banner = new Font("Times", Font.BOLD, 36);
	Graphics offG;
	BufferedImage offImage;
	String aktion = "";
	JPanel nord, sued;
	
	public BaumFrame() {
	    

        setLocation(200,200);

        
		einF = new JTextField(3);
		delF = new JTextField(3);
		suchF = new JTextField(4);
		einB = new JButton("Einfügen");
		randB = new JButton("10x Zufall einfügen");
		del1B = new JButton("Löschen");
		del1randB = new JButton("Zufalllöschen");
		suchB = new JButton("Suchen");
		
		

		einB.addActionListener(this);
		randB.addActionListener(this);
		del1B.addActionListener(this);
		del1randB.addActionListener(this);
		suchB.addActionListener(this);
		
		nord = new JPanel();
		sued = new JPanel();

		offImage = new BufferedImage(800,800,BufferedImage.TYPE_INT_ARGB);
		offG = offImage.createGraphics();

		offG.setColor(new Color(230, 230, 255));
		offG.fillRect(0,60,800,800);
		offG.setColor(Color.black);
		
		ImagePanel imp = new ImagePanel(offImage);
		nord.add(einF);
		nord.add(einB);
		nord.add(randB);

		sued.add(del1B);
		sued.add(delF);

		sued.add(del1randB);

		nord.add(suchF);
		nord.add(suchB);
		setLayout(new BorderLayout());
		add(nord, BorderLayout.NORTH);
		add(imp,BorderLayout.CENTER);
		add(sued,BorderLayout.SOUTH);
		
		root = null;
		setSize(800,800);
		setVisible(true);
		
	}
	
	public void paint(Graphics g) 
	{
		tiefe = 0; anzahl = 0; mtief = 0;
		parse(root, 1);
		if (offG != null)
		{
			offG.drawString("Anzahl: " + anzahl, 310, 20);
			offG.drawString("Tiefe: " + tiefe, 310, 40);
			if (anzahl > 0) 
				offG.drawString("Mittl.Tiefe: " + Double.toString(1.0*mtief/anzahl), 420, 30);
			offG.setColor(Color.red);
			offG.drawString("Vorher", 110, 20);
			offG.drawString("Nachher:  " + aktion, 420, 20);
			offG.setColor(Color.black);
			g.drawImage(offImage, 0, 60, null);
		}
	}
	
	public void paintTree(Graphics g, Element zeiger, int tiefe, int start, int px, int py, boolean linkerZeiger) 
	{
		int x = start, y = 30 * tiefe;
		int diff = breite;
		for (int i = 1; i <= tiefe; i++) diff /= 2;
		if (zeiger != null)
		{
			if (linkerZeiger)
			{
				x = (px > x ? x : px);
			} else {
				x = (px < x ? x : px);
			}
			g.drawLine(x, y, px, py);
			
			g.drawRect(x-7, y, 14, 20);
			g.drawRect(x-7, y+12, 7, 8);
			g.drawRect(x, y+12, 7, 8);
			g.setFont(font);
			g.drawString((zeiger.value<10?"0":"") + zeiger.value, x-4, y+10);
			g.fillOval( x - 5, y + 15, 4, 4);
			g.fillOval( x + 3, y + 15, 4, 4);
			
			if (zeiger.left != null)
			{
				paintTree(g, zeiger.left, tiefe + 1, start - diff, x - 3, y + 17, true);
			}
			if (zeiger.right != null)
			{
				paintTree(g, zeiger.right, tiefe + 1, start + diff, x + 5, y + 17, false);
			}
		}
	}

	public Element delete1(int wert, Element zeiger)
	{
		if (zeiger.value == wert && anzahl > 0)
		{
			if (zeiger.left == null) return zeiger.right;
			if (zeiger.right == null) return zeiger.left;
			Element templeft = zeiger.left;
			Element tempright = zeiger.right;
			while (tempright.left != null) tempright = tempright.left;
			tempright.left = templeft;
			return zeiger.right;
		}
		else
		{
			if (wert < zeiger.value) zeiger.left = delete1(wert, zeiger.left);
			if (wert > zeiger.value) zeiger.right = delete1(wert, zeiger.right);
		}
		return zeiger;
	}
	
	
	public Element sortIn(int wert, Element zeiger)
	{
		if (zeiger == null)
		{
			zeiger = new Element(wert);
		}
		else
		{
			if (wert < zeiger.value) 
			{
				zeiger.left = sortIn(wert, zeiger.left);
			}
			if (wert > zeiger.value)
			{
				zeiger.right = sortIn(wert, zeiger.right);
			}
		}
		return zeiger;
	}
	
	@Override
	public void actionPerformed(ActionEvent e) 
	{
		try
		{
			if (e.getSource() == einB || e.getSource()  == randB || e.getSource()  == del1B 
			||  e.getSource()  == del1randB)
			{
				offG.setColor(new Color(230, 230, 255));
				offG.fillRect(0,0,1000,1000);
				offG.setColor(Color.black);
				paintTree(offG, root, 1, breite + 10, breite + 10, 0, true);
				
				if (e.getSource() == einB)
				{
					if (!einF.getText().equals(""))
					{
						int zahl = Integer.parseInt(einF.getText());
						root = sortIn(zahl, root);
						einF.setText("");
						aktion = new String(zahl + " einsortiert");	
					}
				}
				
				if (e.getSource() == randB)
				{
					for (int i = 0; i < 10; i++) root = sortIn(zufall(1, 99), root);
					aktion = new String("");
				}
				
				
				
				if (e.getSource() == del1B)
				{
					if (!delF.getText().equals("")  && such(Integer.parseInt(delF.getText()), root))
					{
						int zahl = Integer.parseInt(delF.getText());
						
							root = delete1(zahl, root);
						
						delF.setText("");
						aktion = new String(zahl + " gelöscht");	
					}
				}
				
				if ((e.getSource() == del1randB) && anzahl > 0)
				{
					int zz = zufall(1, 99);
					while (!such(zz, root)) zz = zufall(1, 99);
					{

							root = delete1(zz, root);
						
						aktion = new String(zz + " gelöscht");	
					}
				}
								
				paintTree(offG, root, 1, breite + 270, breite + 270, 0, true);
				repaint();	

			}		
			if (e.getSource() == suchB)
			{
				if (!suchF.getText().equals(""))
				{
					int s = Integer.parseInt(suchF.getText());
					suchF.setText(s + (such(s, root) ? " :-)" : " :-("));
				}

			}
		}
		catch (java.lang.NumberFormatException nfe)
		{
			offG.setColor(Color.red);
			offG.setFont(banner);
			offG.drawString("Bitte natürliche Zahlen eingeben!", 40, 120);
			offG.setFont(font);
			offG.setColor(Color.black);
			repaint();
		}

	}
	

	public boolean such(int x, Element z)
	{
		if (z == null) return false;
		if (z.value == x) return true;
		if (x < z.value) return such(x, z.left); else return such(x, z.right);
	}
	
	public void parse(Element zeiger, int tief)
	{
		if (zeiger != null)
		{
			if (tief > tiefe) tiefe = tief;
			parse(zeiger.left, tief + 1);
			anzahl++;
			mtief += tief;
			parse(zeiger.right, tief + 1);
		}
	}
	
	public void ausgabe(Element zeiger)
	{
		if (zeiger != null)
		{
			ausgabe(zeiger.left);
			System.out.println(zeiger.value);
			ausgabe(zeiger.right);
		}
	}
	
	public int zufall(int von, int bis)
	{
		return (int)(von + Math.random() * (bis - von));
	}
}

Panel:

import javax.swing.JPanel;
import java.awt.image.BufferedImage;
import java.awt.Graphics;

class ImagePanel extends JPanel {
    BufferedImage image;
 
    ImagePanel(BufferedImage image) {
        this.image = image;
    }
 
    protected void paintComponent(Graphics g) {
        super.paintComponent(g);
        g.drawImage(image, 0, 70, this);
    }
}

 

[SlaterB]

damit die Buttons über dem Bild erscheinen musst du nord + sued in imp einfügen, auch bzw. vor allem dort dann das BorderLayout,
setze sie auch auf durchsichtig (setOpaque(false); ),

male ab Höhe 0 statt 70 falls nicht oben ein freier Streifen bleiben soll

was immer sonst zu malen ist muss entweder auch in imp-paintComponent
oder in ein anderen Panel welches in imp als Center eingefügt werden könnte