Clipping funktioniert mit eigenem Polygon nicht

[CroweGirl]

Also wie Clipping funktioniert ist mir klar. Wenn ich z.B.

Rectangle k=new Rectangle(20,20,200,200);
         g2.clip(k);
        g2.draw(k);

schreibe, dann wird genau dass gemacht, was ich will. Nun habe ich aber eine eigene Klasse MyRectangle in der ich Rechtecke per MausClick auf meinem Panel erzeuge und die dann in einem Vektor(allrec) verwaltet werden.

public class MyRectangle implements Shape{
	 //..........................

	public MyRectangle(int x, int y, int width, int height, Color color,
			boolean fill) {
//.....................

	}

	/**
	 * 
	 * Zeichnet ein Rechteck
	 */
	public void paintComponent(Graphics g) {

		Graphics2D g2 = (Graphics2D) g;
		g2.setColor(color);

		if (fill)
			g2.fillRoundRect(x, y, width, height, arcWidth, arcHeight);
		else
			g2.drawRect(x, y, width, height);
		

	}

und in meiner Klasse MyPanel hab ich dann in der paintComponent(Graphics g) u.a. folgendes stehen

// zeichnet alle Rechtecke
		for (int i = 0; i < allrec.size(); i++) {
			g2.clip(((MyRectangle) allrec.get(i)));  //das funktioniert aber nicht, bekomme eine NullPointerException in der EventQueue
((MyRectangle) allrec.get(i)).paintComponent(g2);

Da es ja mit einem "normalen Rectangle" funktioniert, kann der Fehler ja nur bei meiner Klasse MyRectangle liegen. Ich habe MyRectangle von Shape erben lasse, damit ich überhaupt g2.clip(shape) machen konnte, dass ist vermutlich falsch...aber wie könnte ich es denn sonst machen? Oder mache ich einen grundsätzlichen Fehler?

 

[Quaxli]

1. Welche Java-Version benutzt Du?
2. Wieso implementiert MyRectangle nicht von Rectangle, sondern implementiert Shape?
3. Bist Du sicher, daß Dein Vector allrec auch instanziiert und gefüllt ist?

Meines Erachtens weist die NullPointerException darauf hin, daß mit Deinem Vector etwas nicht stimmt. Poste doch mal den Code, wo Du den Vector instanziierst und füllst.

 

[Guest]

Hi,

1. 5.0
2. Davor wollte ich einfach nur per linker Mausklick aufs Panel ein Rechteck erzeugen....die ich dann in einem Vektor verwaltet habe...nun da ich per Click das Clipping machen möchte, habe ich aufgrund der Meldung, dass clip nur mit Shape funktioniert, halt implements Shape geschrieben....wenn ich extends Polygon schreibe funktioniert es aber auch nicht
3. Bei meiner ersten Version habe ich einfach per Mausclick ein Rechteck gezeichnet, Teil aus MauseListener:

public void mouseClicked(MouseEvent e) {
 if (e.getButton() == MouseEvent.BUTTON1) 
MyRectangle r = new MyRectangle(cx - b - (rbreite / 2), cy - h- (rhoehe / 2), rbreite, rhoehe, Color.BLUE, true);
myFrame.getMyPanel().addRec(r); // Rechteck wird in Vector gespeichert
myFrame.repaint(); 
System.out.println("linke Maustaste gedrückt----fertig");  //das kann ich noch lesen, danach kommt die Exception...sie muss also irgendwo in der paintComponent(Graphics g) von meinem Panel entstehen

und in meiner Klasse Panel habe ich den Vektor verwaltet und in paintComponent einfach mit

for (int i = 0; i < allrec.size(); i++) {
			((MyRectangle) allrec.get(i)).paintComponent(g2);//hier wird paintComponent() aus meiner Klasse MyRectangle aufgerufen

gezeichnet, was auch super funktioniert hat....nun möchte ich halt clippen und hab einfach nur die clip Zeile dazugefügt.
Der Vektor wird 100% gefüllt, da ich an allen möglichen Stellen prints habe und sehen kann, dass die Linke Maustaste gedrückt wurde und auch der Vektor um 1 Element erhöht wurde, also nicht mehr leer ist.

 

[Quaxli]

Ich habe nicht unbedingt eine Idee, woran es liegen könnte. Probiere doch mal Folgendes: Anstelle Deiner for-Schleife folgenden Code:

  for (MyRectangle mr: allrec){

     System.out.println(mr);

  }

Wenn Dein Vector korrekte Werte enthält, gibt er die auch aus, wenn nicht, hast Du schon mal keine Elemente im Vector und weißt, wo Du evtl. ansetzen mußt.

Und noch eine Frage am Rande: Wozu der Aufwand mit dem Clipping? Kannst Du nicht einfach alles noch mal zeichnen? In der Regel macht sich das nicht bemerkbar.

 

[CroweGirl]

Also wenn ich das so mache,

for (MyRectangle mr: allrec){

     System.out.println(mr);

  }

dann bekomme ich die Fehlermeldung "cannot Convert from element type Object to MyRectangle"...aber mit der meiner For-schleife hat das "normale" Zeichnen ja auch funktioniert, entsprechen sollte das Clipping ja auch funktionieren.
Da ich mir allrec().size printen lasse, weiß ich 100% das in meinem Vector auch was drin ist.

Der Sinn der ganzen Übung ist, dass ich ein Foto habe, auf dem diese Rechtecke gezeichnet werden und ich dann bestimmte Veränderungen nur an dem geclippten Ausschnitt vornehme (z.B. s/w, Helligkeit usw.). Wenn ich das wie in meiner ersten Nachricht mit einem "einfachen Rechteck" mache, funktioniert ja alles auch bestens....nur mit meinem eigenen Rechteck halt nicht.

 

[Marco13]

In dem Vector ist vielleicht was drin. Aber dort können auch 'null'en drin sein. (Und das SIND sie auch, der Exception nach). Wenn du nicht

for (MyRectangle mr: allrec){
     System.out.println(mr);
}

sondern

for (Object object: allrec){
     System.out.println(object);
}

machst, wird es keine Fehlermeldung (convert type blah blah) mehr geben, und mit an Sicherheit grenzender Wahrscheinlichkeit wird irgendwo 'null' ausgegeben.

 

[CroweGirl]

Hi,
danke für den Tipp, aber in meinem Vektor ist was drin und ich bekomme an keiner Stelle null ausgegeben, wenn ich die Zeile mit dem clip entferne.
Sobald ich die clip-Zeile aber hinzufüge bekomme ich dann folgende Fehlermeldung, würde also schon sagen, dass in meinem Vektor was drin ist:

linke Maustaste gedrückt
Vektor Groesse davor= 0
Vektor Groesse danach= 1
linke Maustaste gedrückt----fertig
MyRectangle@1551d7f
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException

Vielleicht mache ich bei clip ja auch was falsch...

 

[Quaxli]

Evtl. liegt es auch am Methodenaufruf. Ich kann es aus dem Code oben nicht recht erkennen, aber evtl. liegt es am Aufruf des clip(...).
Schau Dir mal in der API, die Langtexte zu clip(..) bzw. setClip(..) an, evtl. bekommst Du hier einen Hinweis, woran es liegen könnte.
Ich habe den Eindruck, daß du evtl. setClip(...) solltest, bzw. an der Ecke noch ein bißchen Logik einbauen mußt.

 

[Marco13]

Poste im zweifelsfall mal ein compilierbares Programm, wo der Fehler auftritt. Dann ist die Fehlersuche i.a. nur eine Sache von Minuten.