kurzes problem zu JPanel Hinergrundfarbe

[Java-Freak]

ich komm mir iwi grad dumm vor...
ich hab grad nen gaanz einfaches zeichenprogramm geschrieben(15 min) aber iwi verstehe ich nicht was ich beim JPanel hintergrundfarbe setzen falsch mache...

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.MouseEvent;
import java.awt.event.MouseMotionListener;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;

public class DrawingBoard extends JPanel implements MouseMotionListener {

	private static final long serialVersionUID = 1L;

	boolean toDraw, clear = false;
	int x, y;
	int oldx, oldy;

	public DrawingBoard() {
		super.setBackground(Color.red);
		this.setBackground(Color.red);
		System.out.println(this.getBackground());
		final JFrame f = new JFrame("Drawing Board");
		f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		f.setLayout(new BorderLayout());
		f.add(this, BorderLayout.CENTER);
		JButton b = new JButton("Clear");
		b.addActionListener(new ActionListener() {
			@Override
			public void actionPerformed(ActionEvent e) {
				clear = true;
				repaint();
			}
		});
		f.add(b, BorderLayout.PAGE_END);
		f.addMouseMotionListener(this);
		f.setSize(500, 500);
		f.setVisible(true);
	}

	public static void main(String[] args) {
		new DrawingBoard();
	}

	@Override
	public void paintComponent(Graphics g) {
		System.out.println(this.getBackground());

		if (toDraw) {
			g.drawLine(oldx, oldy, x, y);
			toDraw = false;
		}
		if (clear) {
			g.clearRect(0, 0, 5000, 5000);
			clear = false;
		}
	}

	@Override
	public void mouseDragged(MouseEvent e) {
		toDraw = true;
		oldx = x;
		oldy = y;
		x = e.getX() - 5;
		y = e.getY() - 25;
		repaint();
	}

	@Override
	public void mouseMoved(MouseEvent e) {
		x = e.getX() - 5;
		y = e.getY() - 25;
	}
}

normalerweise müsste das this/super.setBackground() im konstruktor doch schon ausreichen...
kann mir mal jemand sagen was ich da falsch mache? ich probiere schon die ganz zeit rum und komme mir dabei dumm vor weil ichs nicht schaffe nen JPanel rot zu zeichnen

 

[Gast2]

super.paintComponent(g);

fehlt in deiner paintComponent methode.

 

[Java-Freak]

ja, aber wenn manns einfügt dann macht das malen nicht wirklich mehr spaß
dann kann man gar nichts malen weil des sofort wieder überschrieben wird

 

[Marco13]

Genau wie wenn man mal die Fenstergröße ändert oder so. So geht das eben nicht. Male in der MoueDragged in ein Graphics-Objekt, das du dir von einem BufferedImage holst, und male in der paintComponent NUR noch das BufferedImage. Nach dem super.paintComponent

 

[Kekzii]

f.setBackground(new Color(80,180,255));

vllt so?

Edit: das ist aber kein Rot rot wäre dann

f.setBackground(new Color(255,0,0));

 

[Lexi]

f.setBackground(new Color(80,180,255));

vllt so?

Edit: das ist aber kein Rot rot wäre dann

f.setBackground(new Color(255,0,0));

Wie du im Quelltext des ersten Posts siehst, ruft er die setBackground Methode doch schon auf.

 

[Java-Freak]

danke, ich habs etz mit dem bufferedImage gemacht

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.MouseEvent;
import java.awt.event.MouseMotionListener;
import java.awt.image.BufferedImage;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;

public class DrawingBoard extends JPanel implements MouseMotionListener {

	private static final long serialVersionUID = 1L;
	BufferedImage img;
	Graphics2D g2;
	int x, y;
	int oldx, oldy;
	final JFrame f;

	public DrawingBoard() {
		f = new JFrame("Drawing Board");
		f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		f.setLayout(new BorderLayout());
		f.add(this, BorderLayout.CENTER);
		JButton b = new JButton("Clear");
		b.addActionListener(new ActionListener() {
			@Override
			public void actionPerformed(ActionEvent e) {
				g2.clearRect(0, 0, img.getWidth(), img.getHeight());
				repaint();
			}
		});
		f.add(b, BorderLayout.PAGE_END);
		f.addMouseMotionListener(this);
		f.setSize(500, 500);
		img = new BufferedImage(1000, 1000,BufferedImage.TYPE_INT_RGB);
		g2 = img.createGraphics();
		g2.setColor(Color.black);
		g2.fillRect(0, 0, img.getHeight(), img.getWidth());
		g2.setColor(Color.white);
		f.setVisible(true);
	}

	public static void main(String[] args) {
		new DrawingBoard();
	}

	@Override
	public void paintComponent(Graphics g) {
		super.paintComponent(g);
		g.drawImage(img, 0, 0,this);
	}

	@Override
	public void mouseDragged(MouseEvent e) {
		oldx = x;
		oldy = y;
		x = e.getX() - 5;
		y = e.getY() - 25;
		g2.drawLine(oldx, oldy, x, y);
		repaint();
	}

	@Override
	public void mouseMoved(MouseEvent e) {
		x = e.getX() - 5;
		y = e.getY() - 25;
	}
}

eigentlich hab ich das auch nur gemacht weil mir langweilig war, aber dann hats mich gewurmt dasses nicht so einfach war wie ich gedacht hatte..
vllt bau ichs noch aus...
edit: meint ihr das ist ein guter aufbau oder eher nicht als grundlage geeignet?

 

[Marco13]

Bei EINER Klasse gibt es nicht so viel, was man als "Aufbau" bezeichnen könnte Bei jeder Form von Erweiterung würde früher oder später nicht viel von dieser Klasse übrig bleiben. (Was nicht heißt, dass daran irgendwas "schlecht" oder "falsch" ist, sondern nur, dass das eben ein Minimalstprogramm zum Malen von ein paar Pixeln ist...)

 

[Java-Freak]

ja, schon klar, aber diese 80 zeilen können schon mal das was ein zeichenprogramm können sollte, also ist das ja der kern ich meinte eig eher das mit dem MouseMotionListener und dem BufferedImage..man kann halt das Image gut weiterverarbeiten..

 

[Marco13]

Ja nun, bei praktisch jedem Malprogramm wird irgendwo ein MouseMotionListener vorkommen. In den mouseDragged-Methoden wird wohl i.a. auch "irgendwie" in "irgendein" BufferedImage reingemalt. Dass bei einem "richtigen" Malprogramm dann noch Stempel, Farben, Pinselformen, Wischeffekte usw dazukommen, bedeutet, dass es dann evtl. auch mehrere MouseMotionListener gibt, von denen immer nur einer aktiv ist, oder einen, der an verschiedene "Tools" die benötigten Informationen weiterleitet - und DAS ist dann etwas, was man als "Aufbau" bezeichnen kann...
Dass es irgendwann auch mehrere BufferedImages geben könnte, die die verschiedenen Zeichenebnen darstellen, sollte auch klar sein...

 

[Java-Freak]

ja, ich werde es warscheinlich so machen das ich nur einen MouseMotionListener nehme und in der MouseDragged die unterschiedlichen programm-zustände unterscheide(if)...allerdings kann das noch ein wenig dauern, da ich vor kurzem ein wichtigeres programm zum schreiben bekommen habe...man hat halt nicht immer freizeit..

 

[Marco13]

ja, ich werde es warscheinlich so machen das ich nur einen MouseMotionListener nehme und in der MouseDragged die unterschiedlichen programm-zustände unterscheide(if)

Das schreit IMHO eher nach Polymorphie, d.h. die MouseEvents werden nur an irgendein "Tool" weitergereicht:

interface Tool { void machWasMit(MouseEvent e); }

class Paint
{
    private Tool currentTool = Tools.normalerPinsel;

    void mouseDragged(MouseEvent e) { currentTool.machWasMit(e); }
}

// Umschalten ganz einfach:
paint.setCurrentTool(Tools.sprühdose);
...
paint.setCurrentTool(Tools.radierer);

Hat den Vorteil dass man neue Tools leichter einfügen und sie flexibler konfigurieren kann....

Aber das ist dann "Aufbau", und da sollte man sich SEHR genau Gedanken drüber machen.

 

[Java-Freak]

wie kann ich denn die klasse Tools implementieren?
es muss ja ein enum sein, aber wie mache ich es dass jedes enum objekt eine andere machWasMit() implementiert?

 

[Marco13]

Ne, das "Tools" sollte nur für irgendwas stehen, wo die ganzen Tools verwaltet werden... Wie auch immer man das dann implementiert... deswegen
Aber das ist dann "Aufbau", und da sollte man sich SEHR genau Gedanken drüber machen.

 

[Java-Freak]

ja, ich mache mir ja zurzeit gedanken über den aufbau...
also ich habs etz ma so gemacht das man dann halt in die dragged schreibt getCurrentTool().dragged(Graphics g, MouseEvent e) und in die moved getCurrentTool().moved(MouseEvent e)
also ich hab dann ne klasse pencil und das umschalten geht halt so: setCurrentTool(new Pencil())

 

[Marco13]

Kann OK sein. Man könnte noch überlegen, ob der MouseEvent und das Graphics in der Signatur auftauchen sollten, oder ob sowas wie
curentTool.handle(previousPosition, currentPosition, modifiers);
nicht allgemeiner wäre, und wenn ja, wie man das genau macht, oder ob es auch mal Ereignisse gibt, die nichts mit der Maus zu tun haben, und auch abgefangen werden, oder wie man die Zustände von Tools verwaltet (oder den Zustand des Zeichenbretts, z.B. die Information dass etwas ausgewählt ist oder so...). Das ganze kann beliebig aufwändig werden.
Oder gibt's die Adobe Creative Suite irgendwo als Shareware zum Download?

 

[Java-Freak]

nicht das ich wüsste
leuchtet mir etz iwi nicht ganz ein wie du es in der singnatur auftauchen lassen willst...

nicht allgemeiner wäre, und wenn ja, wie man das genau macht, oder ob es auch mal Ereignisse gibt,

dieser satz ist etwas ist etwas seltsam, den könntest du bitte noch mal erklären

 

[Marco13]

Zusammengefasst: ja äh :bahnhof: wie ???:L äh :bahnhof: hmf

:joke:

Wenn man definiert

interface Tool
{
    void doSomething(Graphics g, MouseEvent e);
}

dann kann man damit NUR Tools beschreiben, die ein Graphics und einen MouseEvent bekommen. Ein Tool, das einen KeyEvent bekommt, kann man damit nicht beschreiben. Und ein Tool, das z.B. irgendeine Vektorgrafik in eine Liste legen soll, braucht dafür bestimmt kein "Graphics". Abgesehen davon, dass man spätestens bei mehreren Bildebenen mit EINEM Graphics u.U. nicht mehr weit kommt. Und sowas wie ein "Radierer" oder ein "Verwischpinsel" bräuchte mindestens ein BufferedImage... aus einem Graphics kann man keine schon vorhandenen Pixel auslesen...

Oder kurz: Die Methode ist sehr spezifisch (und zwar spezifisch, für das, was du gerade machen willst, eben nämlich mit einer Maus in das Graphics von einem BufferedImage malen).

Und ich meinte, dass sowohl Graphics als auch MouseEvent eben vielleicht(!) NICHT in der Methodensignatur auftauchen sollten. Man kann da aber schon allein in die Planung beliebig viel Arbeit reinstecken, und wenn man sich keine "Grenzen" setzt, besteht die Gefahr, dass man eine "Tiefensuche der Möglichkeiten" macht - also einen Aspekt immer weiter zu verallgemeinern versucht, weil man ja vielleicht, irgendwann, eventuell mal dies-und-das machen will...(Ich hatte mal angefangen, so eine Art ... ja, Polylinien-Editor/Vektorgrafik-Editor zu schreiben, und habe das da gemerkt... das ganze liegt jetzt aber schon eine Weile brach ...)

Vielleicht ist es nur eine Frage des Namens: Wenn das Ding

interface [b]MouseGraphics[/b]Tool
{
    void doSomething(Graphics g, MouseEvent e);
}

heißen würde, wäre das vielleicht sogar OK : Wenn man antizipiert, dass man mit einem solchen Tool grundsätzlich nur mitt der Maus Veränderungen an einem Graphics-Objekt machen will, passt das ja....

 

[Java-Freak]

hallo
ich habe mal wieder ein problem bei dem zeichen-programm: wie kann ich ein durchsichtiges BufferedImage über ein anderes malen?oder überhaupt ein durchstichtiges erzeugen?
also, ich möchte jetzt eine klasse Brush implements MouseGraphicsTool schreiben, bei der ich nicht nur ein strich zeichnen kann sondern einen ganzen kreis voll. dann muss ich iwi den kreis kennzeichnen, deshalb male ich in der moved einen kreis und übermale den von letztden mal mit weiß...jetzt bleiben aber wenn ich z.b. was schwarzes habe die weißen lösch-striche übrig...
da hab ich mir gedacht ich könnte ja ein 2 bufferedImage nehmen, das über das erste drüberzeichnen und hätte das problem das sachen vom 1. gelöscht werden aber da komm ich jetzt nicht mehr weiter...
geht das auch anders und wenn wie?
ich poste mal den code, das sagt mehr als wenn ich ewig viel schreib^^...
anmerkung: ich hab die versuche mit dem BufferedImage etz mal auskommentiert also nich wundern
DrawingBoard.java

package drawingboard;

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.InputEvent;
import java.awt.event.KeyEvent;
import java.awt.event.MouseEvent;
import java.awt.event.MouseMotionListener;
import java.awt.image.BufferedImage;
import javax.swing.AbstractAction;
import javax.swing.Action;
import javax.swing.ButtonGroup;
import javax.swing.JButton;
import javax.swing.JColorChooser;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JPanel;
import javax.swing.JRadioButtonMenuItem;
import javax.swing.KeyStroke;
import javax.swing.WindowConstants;

public class DrawingBoard extends JPanel implements MouseMotionListener {

	private static final long serialVersionUID = 1L;
	BufferedImage img, showImg;
	Graphics2D g2, showGraphics;
	MouseGraphicsTool currentTool;
	final JFrame f;
	JMenuBar menuBar;
	Color currentColor = new Color(0, 0, 0);

	private void setColor(Color v) {
		getCurrentTool().setCurrentColor(v);
	}

	void setCurrentTool(MouseGraphicsTool t) {
		currentTool = t;
		getCurrentTool().setCurrentColor(getCurrentColor());
	}

	private Color getCurrentColor() {
		return currentColor;
	}

	MouseGraphicsTool getCurrentTool() {
		return currentTool;
	}

	public static void main(String[] args) {
		new DrawingBoard();
	}

	public DrawingBoard() {
		f = new JFrame("Drawing Board");
		f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		f.setLayout(new BorderLayout());
		f.add(this, BorderLayout.CENTER);
		JButton b = new JButton("Clear");
		b.addActionListener(new ActionListener() {
			@Override
			public void actionPerformed(ActionEvent e) {
				g2.setColor(Color.white);
				g2.fillRect(0, 0, img.getWidth(), img.getHeight());
				setColor(Color.black);
				repaint();
			}
		});
		f.add(b, BorderLayout.PAGE_END);
		f.addMouseMotionListener(this);
		f.setSize(500, 500);
		// // <menue>
		JMenuBar menuBar = new JMenuBar();
		// points
		JMenu fileMenu = new JMenu("File");
		JMenu helpMenu = new JMenu("Help");
		menuBar.add(fileMenu);
		menuBar.add(helpMenu);
		// end points; Actions
		Action aboutAction = new AbstractAction("About") {
			private static final long serialVersionUID = 1L;

			public void actionPerformed(ActionEvent e) {
				JFrame frame = new JFrame("Drawing Board - About");
				frame.setDefaultCloseOperation(WindowConstants.HIDE_ON_CLOSE);
				frame.setVisible(true);
				frame.setSize(500, 100);
				frame.setLayout(new GridLayout(2, 0));
				frame.setBackground(Color.white);
				frame.add(new JLabel("©: Jonas Kleineisel."));
				frame
						.add(new JLabel(
								"Bei Fragen wenden sie sich bitte an [email]jonas@kleineisel.de[/email](grad ka lust zu übersetzen)"));
			}
		};
		Action exitAction = new AbstractAction("Quit") {
			private static final long serialVersionUID = 1L;

			public void actionPerformed(ActionEvent e) {
				System.exit(0);
			}
		};

		Action pencilToolAction = new AbstractAction("Pencil") {
			private static final long serialVersionUID = 1L;

			public void actionPerformed(ActionEvent e) {
				setCurrentTool(new Pencil());
			}
		};
		Action brushToolAction = new AbstractAction("Brush") {
			private static final long serialVersionUID = 1L;

			public void actionPerformed(ActionEvent e) {
				setCurrentTool(new Brush());
			}
		};
		Action colorAction = new AbstractAction("Change Color") {
			private static final long serialVersionUID = 1L;

			public void actionPerformed(ActionEvent e) {
				setColor(JColorChooser.showDialog(DrawingBoard.this,
						"Choose Background Color", getCurrentColor()));
			}
		};
		// end actions
		// tool - menue

		JMenu toolSubmenu = new JMenu("Tools");

		ButtonGroup group = new ButtonGroup();

		JRadioButtonMenuItem pencilMenuItem = new JRadioButtonMenuItem(
				pencilToolAction);
		pencilMenuItem.setMnemonic(KeyEvent.VK_P);

		JRadioButtonMenuItem brushMenuItem = new JRadioButtonMenuItem(
				brushToolAction);
		brushMenuItem.setMnemonic(KeyEvent.VK_B);
		brushMenuItem.setSelected(true);

		group.add(pencilMenuItem);
		group.add(brushMenuItem);

		toolSubmenu.add(pencilMenuItem);
		toolSubmenu.add(brushMenuItem);
		// end tool-menue
		fileMenu.add(toolSubmenu);
		fileMenu.add(colorAction);
		JMenuItem exitItem = fileMenu.add(exitAction);
		helpMenu.add(aboutAction);
		exitItem.setAccelerator(KeyStroke.getKeyStroke('Q',
				InputEvent.CTRL_DOWN_MASK));
		f.setJMenuBar(menuBar);
		// JToolBar toolBar = new JToolBar("Werkzeugleiste");
		// toolBar.setFloatable(false);
		// f.add(toolBar, BorderLayout.PAGE_START);

		// // </menue>

		/*showImg = new BufferedImage(500, 500, BufferedImage.TYPE_USHORT_GRAY );
		showGraphics = showImg.createGraphics();*/
		img = new BufferedImage(500, 500, BufferedImage.TYPE_INT_ARGB);
		g2 = img.createGraphics();
		clearImg();
		setCurrentTool(new Brush());
		f.setVisible(true);
	}

	@Override
	public void paintComponent(Graphics g) {
		super.paintComponent(g);
		g.drawImage(img, 0, 0, this);
		//g.drawImage(showImg, 0, 0, this);
	}

	@Override
	public void mouseDragged(MouseEvent e) {
		getCurrentTool().dragged(g2, e);
		repaint();
	}

	@Override
	public void mouseMoved(MouseEvent e) {
		getCurrentTool().moved(g2, e);
		repaint();
	}

	void clearImg() {
		g2.setColor(Color.white);
		g2.fillRect(0, 0, img.getHeight(), img.getWidth());
	}
}

MouseGraphicsTool.java

package drawingboard;

import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.MouseEvent;

interface MouseGraphicsTool {
	void dragged(Graphics g,MouseEvent e);
	void moved(Graphics g, MouseEvent e);
	void setCurrentColor(Color c);
}

Pencil.java

package drawingboard;

import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.MouseEvent;

public class Pencil implements MouseGraphicsTool {
	int oldx;
	int oldy;
	int x;
	int y;
	Color c= Color.black;
	@Override
	public void moved(Graphics g, MouseEvent e) {
		x = e.getX() - 5;
		y = e.getY() - 25;
		y-=25;
	}

	@Override
	public void dragged(Graphics g, MouseEvent e) {
		oldx = x;
		oldy = y;
		x = e.getX() - 5;
		y = e.getY() - 25;
		y-=25;
		g.setColor(getCurrentColor());
		g.drawLine(oldx, oldy, x, y);
	}


	private Color getCurrentColor() {
		return c;
	}

	@Override
	public void setCurrentColor(Color v) {
		c = v;
	}
}

Brush.java

package drawingboard;

import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.MouseEvent;

public class Brush implements MouseGraphicsTool {
	int prevx;
	int prevy;
	int x;
	int y;
	Color c= Color.black;
	@Override
	public void moved(Graphics g, MouseEvent e) {
		//
		g.setColor(Color.black);
		prevx = x;
		prevy = y;
		x=e.getX()-20;
		y=e.getY()-60;
		g.drawArc(x,y, 30, 30, 0 ,360);
		g.setColor(Color.white);
		g.drawArc(prevx,prevy, 30, 30, 0 ,360);
	}


	@Override
	public void dragged(Graphics g, MouseEvent e) {
		g.setColor(getCurrentColor());
		g.fillArc(e.getX()-20,e.getY()-60, 30, 30, 0 ,360);
	}


	private Color getCurrentColor() {
		return c;
	}

	@Override
	public void setCurrentColor(Color v) {
		c = v;
		System.out.println(v);
	}

}

also meiner frage ist eigentlich wie ich für den Brush den vorschaukreis zeichnen kann und ob das mit nen 2 BufferedImage funktioniert
ach ja, wenn sich eh schon mal jemand den code anschaut kann er gleich noch posten ob das ganze nur müll ist und eh nix draus wird oder ob das eine oder andere verwertbar is^^
in der jar is das ganze startbar damit ihr nicht alles kompilieren müsst...

 

[Marco13]

Ich will dich ja nicht desillusionieren, aber ... "verwertbar" ist das erstmal für dich, als Übung. Oder was meintest du? Irgendwas höher-ambitioniertes? ???:L


Falls ich das richtig überflogen habe, gibt es dafür mindestens zwei relativ einfache Möglichkeiten: Entweder, du setzt einfach den passenden Cursor, den du mit dieser Methode aus einem Bild erzeugen kannst, oder das Zeichnen der Zeichentool-Vorschau (d.h. des Kreises) darf nicht auf das BufferedImage gemacht werde, sondern nur auf das Graphics aus der paintComponent - grob im Sinne von

   @Override
    public void paintComponent(Graphics g) {
        super.paintComponent(g);
        g.drawImage(img, 0, 0, this);
        [b]getCurrentTool().drawPreviewThing(g);[/b]
    }
 
    @Override
    public void mouseDragged(MouseEvent e) {
        getCurrentTool().dragged(g2, e); // Hier drin nicht mehr das preview-ding malen!
        repaint();
    }

 

[Java-Freak]

nene, das ist ja alles nur nen übungs-programm.
ich wollte nur wissen ob der aufbau so sinnvoll ist..evtl aweng falsch ausgedrückt
was ich etz gesehen habe kann man dann ja bei dieser methode nen bild als cursur nehmen...ich möchte dann halt noch verschiedene radien des pinsels einbauen, evtl braucht man dann ne funktion Image getCursur(int Radius)
Ich werd morgen mal schaun...
Aber danke schon mal...