Hintergrund eines JPanels

[BrooKy]

Moin,

ich hab ein kleines Problem. Das unten stehenden Programm soll einen JFrame erzeugen, in den dann wiederum ein JPanel eingefügt wird, auf dem man Linien malen kann. Das Ganze funktioniert auch einwandfrei, jedoch will es mir nicht gelingen die Hintergrundfarbe des Panels zu ändern. Ich habe das mittels setBackground() an allen möglichen Stellen ausprobiert, aber nie passiert irgedwas. Komischerweise funktioniert es sofort, wenn ich statt eines JPanels ein Panel verwende... Hat jemand ne Idee?

MfG



Frame:

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

public class Paint extends JFrame {
	Container c;
	Linie l = new Linie();


	public Paint(){		

		c = getContentPane();
		c.setLayout(new BorderLayout());

//		l.setOpaque(false);
//		l.setBackground(Color.red);
		System.out.println(l.isBackgroundSet());
		c.add(l);
	}
	
	public static void main(String[] args) {
		Paint fenster = new Paint();
		fenster.setSize(500,300);
		fenster.setTitle("Tolles Java-Paint");
		fenster.setVisible(true);
		fenster.setResizable(false);
		fenster.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);	
	}
}

Panel:

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

public class Linie extends JPanel {

	int xAlt, yAlt, xNeu, yNeu;
	
	public Linie () {

		addMouseListener(new MHandler());         
	    addMouseMotionListener(new MMHandler()); 
	}
	
	
	public void paintComponent (Graphics g) {
		g.setColor(Color.BLACK);                    
		g.drawLine(xAlt, yAlt, xNeu, yNeu);            
		System.out.println("drawing: (" + xAlt + ";" + xNeu + ") - (" + yAlt + ";" + yNeu + ")");
		xAlt = xNeu; yAlt = yNeu; 
	}	
	
	class MHandler extends MouseAdapter {
	    public void mousePressed (MouseEvent e) {
			xAlt = e.getX();                            
			yAlt = e.getY();                            
		}
	}

	class MMHandler extends MouseMotionAdapter {
		public void mouseDragged (MouseEvent e) {
			xNeu = e.getX(); yNeu = e.getY();           
			repaint();                                  
		}
	}
}

[/code]

 

[dieta]

Ich hab's mit setBackground() jetzt auch nicht hin bekommen. Ich hab jetzt in der paintComponent-Methode ein paar Zeilen eingefügt, so dass er am Anfang einmal ein rotes Rechteck über das ganze JPanel zeichnet (ist vielleicht nicht die eleganteste Methode, funktioniert aber):

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

public class Paint extends JFrame {
	Container c;
	Linie l = new Linie();


	public Paint(){		

		c = getContentPane();
		c.setLayout(new BorderLayout());

//		l.setOpaque(false);
//		l.setBackground(Color.red);
		System.out.println(l.isBackgroundSet());
		c.add(l);
	}
	
	public static void main(String[] args) {
		Paint fenster = new Paint();
		fenster.setSize(500,300);
		fenster.setTitle("Tolles Java-Paint");
		fenster.setVisible(true);
		fenster.setResizable(false);
		fenster.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);	
	}
}

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

public class Linie extends JPanel {

	int xAlt, yAlt, xNeu, yNeu;
                boolean first = true;
	
	public Linie () {

		addMouseListener(new MHandler());         
	    addMouseMotionListener(new MMHandler()); 
	}
	
	
	public void paintComponent (Graphics g) {
                                 if(first)
	                {
	                          g.setColor(Color.RED);
	                          g.fillRect(0, 0, (int)getSize().getWidth(), (int)getSize().getHeight());
		          first=false;
	                }
		g.setColor(Color.BLACK);                    
		g.drawLine(xAlt, yAlt, xNeu, yNeu);            
		System.out.println("drawing: (" + xAlt + ";" + xNeu + ") - (" + yAlt + ";" + yNeu + ")");
		xAlt = xNeu; yAlt = yNeu; 
	}	
	
	class MHandler extends MouseAdapter {
	    public void mousePressed (MouseEvent e) {
			xAlt = e.getX();                            
			yAlt = e.getY();                            
		}
	}

	class MMHandler extends MouseMotionAdapter {
		public void mouseDragged (MouseEvent e) {
			xNeu = e.getX(); yNeu = e.getY();           
			repaint();                                  
		}
	}
}

 

[Guest]

Das ist schonmal was, danke!

Aber ich würde gern noch eine Auswahl für die Hintergrundfarbe hinzufügen und dann wird's mit dieser Methode schwierig, da er dann das bereits gezeichnete ja immer wieder übermalt.

Kann mir jemand sagen, wieso das mit setBackground() nicht geht? Grmpf...

 

[BrooKy]

Ich nochmal... Habe jetzt wenigstens rausfinden können, dass es an der paintComponent()-Methode liegt. Wenn ich die wegmache, funktioniert setBackground() wie gewohnt. Jemand ne Idee, was ich an der Methode ändern könnte?

 

[lhein]

Ich vermute mal, dass es damit zu tun hat, dass Du die paintComponent() überschrieben hast.
Diese Methode ist ja für das Zeichnen der Komponente verantwortlich. Da ich dort nichts sehe, dass den Hintergrund mit der gesetzten Farbe füllt, wird es auch nicht funktionieren.

lr

 

[AlArenal]

Ich vermute mal, dass es damit zu tun hat, dass Du die paintComponent() überschrieben hast.
Diese Methode ist ja für das Zeichnen der Komponente verantwortlich. Da ich dort nichts sehe, dass den Hintergrund mit der gesetzten Farbe füllt, wird es auch nicht funktionieren.

Genauso siehts aus. Man kann nicht einfach den Motor im Auto durch ein nasses Taschentuch ersetzen und erwarten, dass er funktioniert wie vorher... Bevor man wie wild Methoden überschreibt, sollte man sich in Ruhe die Doku durchlesen und ggf. einen Blick auf die Original-Implementierung werfen.

 

[muemmel_0811]

Hi,

versuch doch mal die setBackground in den Konstruktor des JPanels zu schreiben, also so:

public class Linie extends JPanel {

   int xAlt, yAlt, xNeu, yNeu;
   
   public Linie () {

      addMouseListener(new MHandler());         
       addMouseMotionListener(new MMHandler());
       setBackground(Color.RED);
   }
...

und dann sollte es eigentlich funktionieren

Grüße,
muemmel

 

[lhein]

Das glaube ich eher weniger.

 

[Azrahel]

LR hat recht, so gehts nicht.

Die paintComponent überschreiben ist nicht so sinnvoll, ausser du rufst noch die super.paintComponent auf.
Hat aber den Nachteil das bei jedem aufruf das bisher gezeichente wieder überzeichnet wird.

Ich hab neugierigerweise ein wenig dran rum gebaut. Aber schön ists nicht.

package classes.programme.diallp.openprog;

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

public class Linie extends JPanel { 

   int xAlt, yAlt, xNeu, yNeu;
   boolean paintBackGround=true;
    
   public Linie () { 
	      
      addMouseListener(new MHandler());          
       addMouseMotionListener(new MMHandler()); 
   } 
    
    
   public void paintComponent (Graphics g) { 
	   if(paintBackGround){
		   super.paintComponent(g);
		   paintBackGround=false;
	   }
      g.setColor(Color.BLACK);                    
      g.drawLine(xAlt, yAlt, xNeu, yNeu);            
      System.out.println("drawing: (" + xAlt + ";" + xNeu + ") - (" + yAlt + ";" + yNeu + ")"); 
      xAlt = xNeu; yAlt = yNeu;
   }    
    
   class MHandler extends MouseAdapter { 
       public void mousePressed (MouseEvent e) { 
         xAlt = e.getX();                            
         yAlt = e.getY();                            
      } 
   } 

   class MMHandler extends MouseMotionAdapter { 
      public void mouseDragged (MouseEvent e) { 
         xNeu = e.getX(); yNeu = e.getY();            
         repaint();                                  
      } 
   } 
}

Ansonsten müsstest du dir merken wo du überall schon eine Linie gezeichnet hast und die bei jedem paintComponent nachzeichnen. Aber vielleicht hat jemand ne bessere Lösung...

 

[muemmel_0811]

Haut mich bitte nicht, aber ich kann ja nur schreiben, was bei mir geht:

public class Fenster extends JFrame {
   Hintergrund rahmen;

   public Fenster() {
      super();
      setTitle("Test");
      setLayout(new FlowLayout());
      setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      rahmen = new Hintergrund();
      getContentPane().add(rahmen);
   }
   class Hintergrund extends JPanel {
      Hintergrund() {
         setBackground(Color.WHITE);
         setPreferredSize(new Dimension(160,200));
      }
   }
   public static void main(String[] args) {
      JFrame fenster = new Fenster();
      fenster.setSize(350,275);
      fenster.setLocation(200,300);
      fenster.setVisible(true);
      fenster.setResizable(false);
   }
}

Grüße,
muemmel

 

[Azrahel]

Ne Muemmel, ich hau dich nicht, aber ich hab was anderes

so richtig schön nah am Wahnsinn. Wir bauen uns eine verknusperte Koordinatenklasse aus 2 Vectoren ungefähr so:

import java.util.Vector;

public class LineMemory {
	private Vector xCord;
	private Vector yCord;
	private Vector breakUp;
	private int count;
	
    public LineMemory() {
    	clean();
	}
    
    public void add(int x, int y, int b){
    	xCord.add(""+x);
    	yCord.add(""+y);
    	breakUp.add(""+b);
    	count++;
	}

    public void clean() {
    	xCord=new Vector();
    	yCord=new Vector();
    	breakUp=new Vector();
    	count=0;
	}
    
    public int size(){
    	return count; 
    }
    
    public int get(int row, int colum){
    	if(colum==1)
    		return Integer.parseInt(xCord.elementAt(row).toString());
    	if(colum==2)
    		return Integer.parseInt(yCord.elementAt(row).toString());
    	if(colum==3)
    		return Integer.parseInt(breakUp.elementAt(row).toString());
    	return 0;
    }
}

Packagedeclaration nicht vergessen

Dann bauen wir uns die Linie-Klasse ein wenig um:

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

public class Linie extends JPanel { 

   int xAlt, yAlt, xNeu, yNeu;
   LineMemory lm;
    
   public Linie () { 
	      lm=new LineMemory();
      addMouseListener(new MHandler());          
       addMouseMotionListener(new MMHandler()); 
   } 
    
    
   public void paintComponent (Graphics g) { 
	   super.paintComponent(g);
	   g.setColor(Color.BLACK);
	   if(lm.size()>0){
		   xAlt=lm.get(0,1);
		   yAlt=lm.get(0,2);
		   for(int i=1;i<lm.size();i++){
			   xNeu=lm.get(i,1);
			   yNeu=lm.get(i,2);
			   if(lm.get(i-1,3)==0){
				    g.drawLine(xAlt, yAlt, xNeu, yNeu);            
					System.out.println("drawingNew: (" + xAlt + ";" + xNeu + ") - (" + yAlt + ";" + yNeu + ")");
				}
			   xAlt = xNeu; yAlt = yNeu;
				  
		   }
	   }
   }    
    
   class MHandler extends MouseAdapter { 
	   public void mousePressed (MouseEvent e) { 
	     	  lm.add(e.getX(),e.getY(), 0);
	      } 
	   public void mouseReleased (MouseEvent e) { 
	     	  lm.add(e.getX(),e.getY(), 1); 
	      } 
   } 

   class MMHandler extends MouseMotionAdapter { 
      public void mouseDragged (MouseEvent e) { 
    	  lm.add(e.getX(),e.getY(),0);
            
         repaint();                                  
      } 
   } 
}

Dann schneiden wir noch aus der Paint-Klasse das

      System.out.println(l.isBackgroundSet());

aus und freuen us über das Ergebniss :-D

Ich erwarte demütig meinen Rausschmiss aus dem Forum *grins*

 

[muemmel_0811]

Ne Muemmel, ich hau dich nicht,

Na dann bin ich ja beruhigt.

 

[Azrahel]

Ich bin ja ein völlig gewaltfreier Mensch ... fast schon mit budhistischer Ruhe ... Abgesehen von der Zeit zwischen dem gewaltsamen weckens durch meine Funkuhr und der Einnahme der 3 Tasse Kaffee


Aber ich hab nochwas, man kann ja die LineMemory erweitern

public class LineMemory { 
   private Vector xCord; 
   private Vector yCord; 
   private Vector breakUp; 
   private Vector color; 
   private int count; 
    
    public LineMemory() { 
       clean(); 
   } 
    
    public void add(int x, int y, int b, Color c){ 
       xCord.add(""+x); 
       yCord.add(""+y); 
       breakUp.add(""+b); 
        color.add(c)
       count++; 
   } 

    public void clean() { 
       xCord=new Vector(); 
       yCord=new Vector(); 
       breakUp=new Vector(); 
        color=new Vector();
       count=0; 
   } 
    
    public int size(){ 
       return count; 
    } 
    
    public int get(int row, int colum){ 
       if(colum==1) 
          return Integer.parseInt(xCord.elementAt(row).toString()); 
       if(colum==2) 
          return Integer.parseInt(yCord.elementAt(row).toString()); 
       if(colum==3) 
          return Integer.parseInt(breakUp.elementAt(row).toString()); 
       return 0; 
    } 
    
    public Color getColor(int row){ 
       return (Color)color.elementAt(row);
    } 
}

dann müsste man nur noch in der Linie ein paar Änderungen machen :

[/code]public void paintComponent (Graphics g) {
super.paintComponent(g); if(lm.size()>0){
xAlt=lm.get(0,1);
yAlt=lm.get(0,2);
for(int i=1;i<lm.size();i++){
xNeu=lm.get(i,1);
yNeu=lm.get(i,2);
if(lm.get(i-1,3)==0){
g.setColor(lm.getColor(i));
g.drawLine(xAlt, yAlt, xNeu, yNeu);
System.out.println("drawingNew: (" + xAlt + ";" + xNeu + ") - (" + yAlt + ";" + yNeu + ")");
}
xAlt = xNeu; yAlt = yNeu;

}
}
}

class MHandler extends MouseAdapter {
public void mousePressed (MouseEvent e) {
lm.add(e.getX(),e.getY(), 0, Color.Red);
}
public void mouseReleased (MouseEvent e) {
lm.add(e.getX(),e.getY(), 1, Color.Red);
}
}

class MMHandler extends MouseMotionAdapter {
public void mouseDragged (MouseEvent e) {
lm.add(e.getX(),e.getY(),0, Color.Red);

repaint();
}
}

wenn man Color dann noch dynamisch wählbar macht (über ne Toolbar zum Beispiel) hat man schon fast PhotoShop

 

[Azrahel]

sorry da hab ich wohl blöd getagt

 

[dieta]

Dass "setBackground(Color)" geht, musst du am Anfang deiner "paintComponent(Graphics)"-Methode den Aufruf

super.paintComponent(g);

einfügen.

 

[Azrahel]

Das mit am Anfang das

super.paintComponent(g)

hab ich auch versucht, Ergebniss war aber das dann immer alles schon gezeichnete wieder überzeichnet wurde und ich nur den letzten gezeichneten Strich gesehen hab. Oder hat ich da was falsch gemacht?[/code]

 

[dieta]

Nee, du hast schon alles richtig gemacht, aber du musst wenn du es mit super.paintComponent() lösen willst alles neu Zeichnen, was so wie so die beste Lösung ist (es reicht ja, wenn du dir die letzten 2048 Pixel merkst), da bei der anderen Methode nach demMinimieren des Fensters auch alles weg ist.