JPanel in JScrollPane neu zeichnen

[redbomber]

Hi zusammen,
mal wieder eine Frage:

Ich habe eine JFrame, in dem sich mehrere JPanels befinden, welche wiederum in eine JScrollane eingebettet sind:

Hier also meine JScrollPane:

JScrollPane listScroller;
listScroller.getHorizontalScrollBar().addAdjustmentListener(new RepaintPlotAction());
listScroller.getVerticalScrollBar().addAdjustmentListener(new RepaintPlotAction());

ich habe also einen Listener hinzugefügt, welche bemerkt wenn der scrollbalken bewegt wird
Wenn dieses Event gefeuert wird, wird die folgende Methode aufgerufen:

// listOfPlots ArrayList<Plot> in der sich alle Plots befinden (vom Typ JPanel)
public void updateAllPlots(){
        //MyDraw (extends JPanel)
	for(MyDraw draw: listOfPlots){
			draw.repaint();
	}
}

Jo, aber was passiert nun:
Wenn ich den Scrollbalken bewege, wird das Panel nicht neu gezeichnet, bzw. doch.
Es wird immer auf die alte Zeichnung neu draufgezeichnet, bis irgendwannd das ganze Panel vollgemalt ist.

Aber ich möchte doch einfach nur auf meinem Panel entlang scrollen können.

 

[Ebenius]

Die Listener sind unnötig, bzw. falsch. Du machst irgendwas beim Zeichnen in MyDraw falsch. Poste doch mal den Source-Code dieser Klasse.

Ebenius

 

[redbomber]

was meinst du mit unnötig?
Sollte ich also von meinem JPanel aus mitbekommen, wenn der Scrollbalker der JScrollPane bewegt wird?

public class MyDraw extends JPanel{
	double miny=Double.MAX_VALUE, minx=Double.MAX_VALUE, maxy=Double.MIN_VALUE,  maxx=Double.MIN_VALUE;   
	protected int height = 200;
	protected int width = 8000;
	protected double yrange, xrange;
	protected ArrayList<Integer> data = new ArrayList<Integer>();
	
	protected double Xscaling=1.0;
	protected double Yscaling=1.0;
	protected double Xshift=0;
	protected double Yshift=0;
	protected Manager manager;
	
	protected AffineTransform orgTf;
	
	protected boolean drawLine = false;
	protected double xLinePos;
	
    /** Creates a new instance of MyDraw */
    public MyDraw(ArrayList<Integer> data, Manager manager) {
    	manager.addPlotToPlotlist(this);
    	this.setBackground(Color.WHITE);
    	this.setSize(width, height);
    	this.setPreferredSize(new Dimension(width, height));
    	this.setOpaque(true);
    	this.data = data;
    	this.setVisible(true);
    	this.manager = manager;
    	
    	this.addMouseListener(new MouseListener(){

			@Override
			public void mouseClicked(MouseEvent arg0) {
				// TODO Auto-generated method stub
				System.out.println("mouseClicked");
			}

			@Override
			public void mouseEntered(MouseEvent event) {
				// TODO Auto-generated method stub
				System.out.println("mouseEntered");
				setLineDrawer(true);
				drawTheLine(event.getLocationOnScreen().getX());
				repaintThePlot();
			}

			@Override
			public void mouseExited(MouseEvent arg0) {
				// TODO Auto-generated method stub
				System.out.println("mouseExited");
				setLineDrawer(false);
				repaintThePlot();				
			}

			@Override
			public void mousePressed(MouseEvent arg0) {
				// TODO Auto-generated method stub
				
			}

			@Override
			public void mouseReleased(MouseEvent arg0) {
				// TODO Auto-generated method stub
				
			}
    		
    	});
    	
    	this.addMouseMotionListener(new MouseMotionListener(){

			@Override
			public void mouseDragged(MouseEvent arg0) {
				// TODO Auto-generated method stub
				
			}

			@Override
			public void mouseMoved(MouseEvent event) {
				// TODO Auto-generated method stub
				System.out.println("mouseMoved");
				setActualTooltip(event.getPoint());
				drawTheLine(event.getLocationOnScreen().getX());
				repaintThePlot();
			}
    		
    	});
    	
    	setMinMaxValues();

    }

// dies hier am besten garnicht betrachten ;)
    public void paintComponent (Graphics g)
    {
		final Graphics2D g2d = (Graphics2D) g;
		orgTf = g2d.getTransform();
		g2d.setBackground(Color.white);
		g2d.setTransform(orgTf);
		orgTf.translate(5., 5.);
		g2d.setColor(Color.RED);
		g2d.drawLine(0, new Double(miny - 1).intValue(), data.size(),
				new Double(miny - 1).intValue());

		for (int i = 0; i < data.size() - 1; i++) {
			// this.setForeground(Color.RED);
			double rx1 = ((i + 1) - minx) / (maxx - minx);
			double ry1 = (data.get(i + 1) - miny) / (maxy - miny);
			double rx2 = ((i + 1) - minx) / (maxx - minx);
			double ry2 = (data.get(i + 1) - miny) / (maxy - miny);

			int fx1 = (int) (rx1 * getWidth());
			int fy1 = (int) (ry1 * getHeight());
			int fx2 = (int) (rx2 * getWidth());
			int fy2 = (int) (ry2 * getHeight());

			g2d.setColor(Color.GREEN);
			g2d.drawLine(fx1, fy1, fx2, fy2);

		}
		
// hiermit zeichne ich eine vertikale Linie an der Position wo ich mich mit der Maus befinde
		if(drawLine){
			g2d.setColor(Color.CYAN);
			g2d.drawLine((int)xLinePos, 0, (int)xLinePos, getWidth()); 
		}

	}
    
    protected void setMinMaxValues(){	
		minx = 0;
		maxx = data.size()+100;
		
    	for(int i = 1; i < data.size(); i++){
    		miny = miny<data.get(i)?miny:data.get(i);
    		maxy = maxy>data.get(i)?maxy:data.get(i);
    	}
    }
    

    
    protected void setRange(){
    	yrange = maxy - miny; 
    	xrange = maxx - minx;
    	
    	System.out.println("yrange: " +yrange);
    	System.out.println("xrange: "+ xrange);
    }
	
	protected void setActualTooltip(Point point){
		this.setToolTipText("<html><body>xPos "
				+ point.getX() + "
" 
				+ "yPos " + point.getY() + "</body></htlm>");
		
	}
	
	protected void drawTheLine(double xPos){
		xLinePos = xPos;
	}
	
	protected void repaintThePlot(){
		this.repaint();
	}
	
	protected void setLineDrawer(boolean val){
		drawLine = val;
	}
}

 

[Ebenius]

paintComponent muss so aussehen:

public void paintComponent (Graphics g) {
  super.paintComponent(g);
  final Graphics2D g2d = (Graphics2D) g; 
  final AffineTransform orgTf = g2d.getTransform();
  try {
    g2d.translate(5., 5.);
    // YOUR PAINTING HERE!
  } finally {
    // reset old transform
    g2d.setTransform(orgTf); 
  }
}

 

[redbomber]

jo, das hattest du mir auch schon gezeigt.
aber da kommt trotzdem nur murks dabei raus...

 

[SlaterB]

Beispielprogramm posten..

 

[redbomber]

Also:

1. der Plot wird jetzt korrekt repainted()
Also wenn ich den Scrollbalken bewege wird der Plot jedes mal neu gezeichnet. Also so wie es sein soll.
Lag also alles meiner falschen paintComponent Methode.


Habe leider aber noch zwei weitere Probleme:
1. Da mein JPanel über die Bildschirmgröße hinaus geht, gibt es ein Problem mit dem Einzeichnen eines senkrechten Strichs an der Mausposition.
Sobald ich den Scrollbalken bewege, zeigt es den strich nicht mehr an.
Vermutlich wird der Strich dann in dem jetzt unsichtbaren Bereich des JPanels gezeichnet.
Siehe hier: hier

2. Und meine Daten werden noch immer falsch herum gezeichnet, obwohl ich die x,y - Werte umgerechnet habe.
Siehe hier: hier

 

[Ebenius]

1. der Plot wird jetzt korrekt repainted()
Also wenn ich den Scrollbalken bewege wird der Plot jedes mal neu gezeichnet. Also so wie es sein soll.
Lag also alles meiner falschen paintComponent Methode.

:wink:

Habe leider aber noch zwei weitere Probleme:
1. Da mein JPanel über die Bildschirmgröße hinaus geht, gibt es ein Problem mit dem Einzeichnen eines senkrechten Strichs an der Mausposition.
Sobald ich den Scrollbalken bewege, zeigt es den strich nicht mehr an.
Vermutlich wird der Strich dann in dem jetzt unsichtbaren Bereich des JPanels gezeichnet.
Siehe hier: hier

Wahrscheinlich hängt (wie im anderen Thread beschrieben) der MouseMotionListener am falschen Panel. Das erkennt man aber am Code-Ausschnitt nicht.

2. Und meine Daten werden noch immer falsch herum gezeichnet, obwohl ich die x,y - Werte umgerechnet habe.
Siehe hier: hier

Das erreichst Du mit der scale(double, double)-Methode auf dem Graphics2D. Also ungefähr:

g2d.scale(1,-1);

Ebenius