Mal Programm

[Franky-2012]

Hi,
ich hab mal versucht ein Malprogramm zu schreiben, doch hab ich ein paar Probleme mit dem zeichnen ;(

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

public class ZeichenPanel extends JPanel{
	
	private int[]x,y;
	private int n,Xsize,Ysize;
    private float stroke;
    private Color color;
    
	public ZeichenPanel(int Xvalue,int Yvalue){  //Konstruktor
	n=0;
	x=new int[100000];
	y=new int [100000];
	color=Color.black;
	stroke=2.0f;
	setSize(Xvalue,Yvalue);
	Xsize=this.getWidth();
	Ysize=this.getHeight();
	this.addMouseMotionListener(new ClickBearbeiter());
	}
	//Überschreibung der methode paintComponent
	public void paintComponent(Graphics g){
	super.paintComponent(g);
	Graphics2D gd=(Graphics2D)g.create();
	gd.setColor(Color.WHITE);
	gd.fillRect(0, 0, Xsize,Ysize);
	
	gd.setRenderingHint(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);
	gd.setStroke(new BasicStroke(stroke));
	gd.setColor(color);

	gd.drawPolyline(x,y,n);
    gd.dispose();

	}
	public void setStroke(int value){
		stroke=(float)value;
		repaint();
	}
	public void setColor(Color c){
		color=c;
		repaint();
	}
	class ClickBearbeiter extends MouseMotionAdapter{
		public void mouseDragged(MouseEvent e){
			
		x[n]=e.getX();
		y[n]=e.getY();
		repaint();
		n++;	
		}
	
		
	}
	}

Das ist mein Zeichenpanel, das Problem ist wenn ich jetzt in meinem "eigentlichen"Editor in einem Colorchooser die Farbe verändere, ändert sich die Farbe von allem vorher gemalten...Logisch
Wie kann ich dies ändern??
Außerdem wenn ich die Maus drücke und male und dann los lasse, woanders hinbewege, wieder drücke, verbindet es zum vorher gemalten ;(
Dies ist natürlich auch unerwünscht.
In meinem Editor verwende ich, in der Mitte, eine JSplitPane, auf der rechten seite habe ich ein JPanel, welches dann das Zeichenpanel added. Oben halt noch ToolBar mit Werkzeugen, Strichstärken,
Farben und MenuBar mit File, Edit...

 

[bERt0r]

Naja, du speicherst alle Punkte, auf die du mit gedrückter Maus fährst in ein Polygon. Ein Polygon is eben eine zusammenhängende Figur. Wenn du mehrere Figuren haben willst, musst du das anders angehen. Guck dir mal das Programm in meiner Sig an.

 

[Franky-2012]

Hi Danke für deine Aw aber das Programm bringt mir ohne Source Code leider nicht soo viel ;(

 

[Michael...]

ändert sich die Farbe von allem vorher gemalten...Logisch
Wie kann ich dies ändern??

Naja, Du musst Dir sämtliche Attribute (Form, Farbe, Position, Größe...) "merken".

Hi Danke für deine Aw aber das Programm bringt mir ohne Source Code leider nicht soo viel

Der ist doch mit dabei.

 

[Franky-2012]

Dankee wie dumm von mir...😉
Ich denke ich werde das Problem lösen!
Nochmal Dankee für eure Aw's!!!
Ich schließ das Thema erstmal nicht......hab bestimmt nochmal Fragen 😀

 

[Franky-2012]

Und da ist auch schon die erste Frage 😀
Habe mein Problem zwar gelöst...doch wenn man länger malt buggt es rum und ist tierisch lam...;(
Wie hast du(bERt0r) das gemacht???
konnte ich im Code nicht finden, oder habs übersehen....
Lern ja auch erst seit ein paar Monaten Java...;(

 

[bERt0r]

In java gibt es das interface Shape. Um das herum hab ich eine Klasse Colored Shape gebastelt, das heist die Klasse speichert ein Shape und eine Farbe.
In der Funktion createShape wird ein ColoredShape abhängig von zwei Punkten erstellt, abhängig welche Einstellungen (Farbe & Form) gerade in der Toolbar oben eingestellt sind. Diese ColoredShapes kommen in eine Liste, welche in der paintComponent einfach durchgegangen und gezeichnet wird.
Das wichtige: keine Arrays verwenden, schließlich weist du nicht wieviele Formen du zeichnen willst, vielleicht brauchst du 100000, vielleicht mehr, vielleicht weniger. Das weist du im vorhinein nicht. Wenn du eine Liste verwendest kann dir das aber auch egal sein. In java gibts die schöne Klasse ArrayList welche die Performance Vorteile von Arrays mit der Flexibilität von Listen ziemlich gut vereint.

 

[Franky-2012]

Sry meine frage war nicht wie du das mit dem Malen und soo geproggt hast, sondern wie du das mit der Performence gemacht hast!
Denn bei mir fängt es nach einer Zeit malen an zu laggen..;(

Ansonsten klappt ja alles 😀
Werden das nicht zu viele obj's???

 

[Michael...]

Denn bei mir fängt es nach einer Zeit malen an zu laggen..

Ohne Code oder Beschreibung was genau Du machst, lässt sich schwer woher das Problem kommt.

 

[Franky-2012]

Soo dies ist meine momentane Klasse ZeichenPanel.

package paint;
import java.awt.*;
import javax.swing.*;
import java.awt.event.*;
import java.awt.geom.Path2D;
import java.util.Vector;

public class ZeichenPanel extends JPanel{
	

	private int Xsize,Ysize;
    private float stroke;
    private Color color;
    coloredShape drawingShape =null;
	Vector<coloredShape> shapes=new Vector<coloredShape>();
	Point d=new Point(0,0);
		
	public ZeichenPanel(int Xvalue,int Yvalue){  //Konstruktor
	color=Color.black;
	stroke=2.0f;
	setSize(Xvalue,Yvalue);
	Xsize=this.getWidth();
	Ysize=this.getHeight();
	this.addMouseMotionListener(new ClickBearbeiter());
	this.addMouseListener(new ClickBearbeiter());
	}
	//Überschreibung der methode paintComponent
	public void paintComponent(Graphics g){
	super.paintComponent(g);
	Graphics2D gd=(Graphics2D)g;
    gd.setColor(Color.WHITE);
    gd.fillRect(0,0,Xsize,Ysize);
    	gd.setRenderingHint(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);
	for(coloredShape cs:shapes)	{

	gd.setStroke(new BasicStroke(cs.getStroke()));	
	gd.setColor(cs.getColor());
	gd.draw(cs.getShape());
	}
	}
	public void setStroke(int value){
		stroke=(float)value;

	}
	public void setColor(Color c){
		color=c;

	}
	public coloredShape createShape(Point a,Point b){
		Path2D.Double part;
		coloredShape colorShape=new coloredShape();
		if(drawingShape==null || !(drawingShape.getShape() instanceof Path2D.Double))
		{
			part=new Path2D.Double();
			part.moveTo(a.x, a.y);
		}
		else
		{
			part=(Path2D.Double)drawingShape.getShape();
		}
		part.lineTo(b.x, b.y);
		colorShape.setShape(part);
		colorShape.setColor(color);
		colorShape.setStroke(stroke);
		return colorShape;
	}
	class ClickBearbeiter extends MouseAdapter{
	
		
		public void mouseDragged(MouseEvent e){
		Point p=new Point(e.getPoint())	;
		drawingShape= createShape(d,p);
		shapes.add(drawingShape);
		repaint();
		}
		public void mousePressed(MouseEvent e) 
		{
			d=e.getPoint();
		}
		
		public void mouseReleased(MouseEvent e)
		{
			Point r=e.getPoint();
			coloredShape cs=createShape(d,r);
			if(cs!=null)
			{
		
				shapes.add(cs);
			}
			drawingShape=null;
			repaint();
		}
		
		
	}
	}

Dann habe ich noch die Klasse coloredShape,mit den Atributen und meine Editor Klasse mit der ganzen Gui.

 

[Michael...]

Ohne vernünftige Einrückung ist der Code schwer zu lesen.
So ganz überblickt habe ich es nicht, aber warum fügst Du beim Mausziehen immer wieder neue Shapes oder bestehende Referenzen neu dazu? Deine shapes Vector wird da ja ewig lang und ich glaube nicht, dass Du das so willst.

		public void mouseDragged(MouseEvent e){
		Point p=new Point(e.getPoint())	;
		drawingShape= createShape(d,p);
		shapes.add(drawingShape);
		repaint();
		}

 

[Franky-2012]

Dankee ist wenigstens schon ein bisschen besser geworden!!
Echt dumm von mir...
Habe mir auch noch ein Test Programm geschrieben, mit nur einem JPanel....laggt aber auch ;(

 

[Franky-2012]

Soo hab den Code nochmal ein bisschen verändert....
Ist aber fast nicht besser geworden ;(
Pls helft mir!

package paint;
import java.awt.*;
import javax.swing.*;
import java.awt.event.*;
import java.awt.geom.Path2D;
import java.util.Vector;

public class ZeichenPanel extends JPanel{
	

	private int Xsize,Ysize;
    private float stroke;
    private Color color;
    coloredShape drawingShape =null;
	Vector<coloredShape> shapes=new Vector<coloredShape>();
	Point a=new Point(0,0);
	Point location;
	
	public ZeichenPanel(int sizex,int sizey){  //Konstruktor
	color=Color.black;
	stroke=2.0f;
	setSize(sizex,sizey);
	Xsize=this.getWidth();
	Ysize=this.getHeight();
	location=new Point(getLocation());
	
	this.addMouseMotionListener(new ClickBearbeiter());
	this.addMouseListener(new ClickBearbeiter());
	}
	
	public void paintComponent(Graphics g){
	  super.paintComponent(g);
	    Graphics2D gd=(Graphics2D)g;
        gd.setColor(Color.WHITE);
        gd.fillRect(location.x,location.y,Xsize,Ysize);
        gd.setClip(location.x,location.y,Xsize,Ysize);
    	gd.setRenderingHint(RenderingHints.KEY_ANTIALIASING,
    			            RenderingHints.VALUE_ANTIALIAS_ON);
    	System.out.println(shapes.size());
	   for(coloredShape cs:shapes)	{

	    gd.setStroke(new BasicStroke(cs.getStroke()));	
	    gd.setColor(cs.getColor());
	    gd.draw(cs.getShape());
	    
	  }
	   
	}
	public void setStroke(int value){
		stroke=(float)value;

	}
	public void setColor(Color c){
		color=c;

	}
	public coloredShape createShape(Point a,Point b){
		Path2D.Double part;
		coloredShape colorShape=new coloredShape();
		if(drawingShape==null){
		
			part=new Path2D.Double();
			part.moveTo(a.x, a.y);
		}
		else {
		
			part=(Path2D.Double)drawingShape.getShape();
		}
		part.lineTo(b.x, b.y);
		colorShape.setShape(part);
		colorShape.setColor(color);
		colorShape.setStroke(stroke);
		return colorShape;
	}
	class ClickBearbeiter extends MouseAdapter{
	
		
		public void mouseDragged(MouseEvent e){
			 Point b=new Point(e.getPoint()) ;
		        drawingShape= createShape(a,b);
		        shapes.add(drawingShape);
		        repaint(location.x,location.y,Xsize,Ysize);
		}
		public void mousePressed(MouseEvent e) {
		
		a=e.getPoint();	
		
		}
		
		public void mouseReleased(MouseEvent e){
		
			
			drawingShape=null;
			
		}
		
		
	}
}

 

[Franky-2012]

Soo entlich funzt es Danke nochmal für eure Antworten ...🙂