zeichenprogramm ohne saeuberungsfunktion

[chriz_h]

Servus miteinander,
ich sitze seit geraumer Zeit vor meinem Java Buch und habe mich bisher auch strikt an dieses gehalten. Nur leider stosse ich immer wieder auf Probleme, sobald ich eigene Klassen und Methoden in vorgegebene Programme einbauen will. So auch mit folgendem Zeichenprogramm.Okay, die Idee u. der Ansatz sind aus nem Buch, aber ich war schon fleissig und habe eins zwei Klassen selbst implementiert. Wenn ich nun Zeile 125 bis 133 + 141 einbaue bekomm ich ne Fehlermeldung vom Compiler, die mir nicht schluessig ist. Naja, jedenfalls soll diese Klasse den Canvas Inhalt loeschen, so dass man neu zeichnen kann. Kann mir jemand sagen, warum das so nicht funktioniert?
mfG
ChRizZ

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

public class malprogramm extends JFrame{
	meineCanvas m_malflaeche; //zum bemalen
	ButtonGroup m_formauswahl; //aktuelle Form
	int m_Xpos1, m_Ypos1; //aktuelle mausposition
	int m_Xpos2, m_Ypos2;
	
	public static void main(String[]args){
		malprogramm Fenster = new malprogramm("ChRizZ-PAINT");
		Fenster.pack();
		Fenster.setSize(400,350);
		Fenster.setResizable(true);
		Fenster.show();
	}
	
	
	//Konstruktor fuer Malflaeche und radiobuttons
	malprogramm(String titel){
		super(titel);
		
		//einen LayoutManager einrichten
		getContentPane().setLayout(new FlowLayout());
		
		m_malflaeche = new meineCanvas();
		getContentPane().add(m_malflaeche);
		
		
		JPanel panel = new JPanel();
		panel.setLayout(new GridLayout(5,1,20,20));
		//Optionsfelder zur Formauswahl
		m_formauswahl = new ButtonGroup();
		
		//1. Optionsfelder erzeugen
		JRadioButton opt1 = new JRadioButton("Kreis",false);
		JRadioButton opt2 = new JRadioButton("Scheibe",false);
		JRadioButton opt3 = new JRadioButton("Rechteck",false);
		JRadioButton opt4 = new JRadioButton("Freihand",false);
		JButton opt5 = new JButton("clean");

		
		//2. Befehlsnamen fuer Optionfelder
		opt1.setActionCommand("Kreis");
		opt2.setActionCommand("Scheibe");
		opt3.setActionCommand("Rechteck");
		opt4.setActionCommand("Freihand");

		
		//3. Optionfelder in ButtonGroup aufnehmen
		m_formauswahl.add(opt1);
		m_formauswahl.add(opt2);
		m_formauswahl.add(opt3);
		m_formauswahl.add(opt4);
		//m_formauswahl.add(opt5);
		
		//Optionsfelder in Panel aufnehmen
		panel.add(opt1);
		panel.add(opt2);
		panel.add(opt3);
		panel.add(opt4);
		panel.add(opt5);
		
		getContentPane().add(panel);
		
		//Fensterschliesser als innere classe anlegen
		class meinWindowAdapter extends WindowAdapter{
			public void windowClosing(WindowEvent e){
				System.exit(0);
			}
		}
		//schliesslich noch der WindowListener
		addWindowListener(new meinWindowAdapter());

	}
	
	class meineCanvas extends Canvas{
		
		class meinMouseAdapter extends MouseAdapter{
			public void mousePressed(MouseEvent e){
				//die aktuelle Pos. der Mouse merken
				m_Xpos1=e.getX();
				m_Ypos1=e.getY();

			}
			public void mouseReleased(MouseEvent e){
				m_Xpos2=e.getX();
				m_Ypos2=e.getY();
				repaint();
			}
			
		}
		public void update(Graphics g){
			paint(g);
		}
		
		class meinMouseMotionAdapter extends MouseMotionAdapter{
			public void mouseDragged(MouseEvent e){
				ButtonModel aktuell;
				String label;
				//herausfinden welche box gerade aktiviert ist
				aktuell = m_formauswahl.getSelection();
				if (aktuell == null)
					return;
				
				label = aktuell.getActionCommand();
				//wenn freihand gewaehlt, mousepos. merken
				if(label.equals("Freihand")){
					Graphics tmp = m_malflaeche.getGraphics();
					m_Xpos1=e.getX();
					m_Ypos1=e.getY();
					tmp.drawOval(m_Xpos1,m_Ypos1,2,2);
				}
			}
		}
		class meinActionLauscher implements ActionListener{
			public void actionPerformed(ActionEvent e){
				String label;
				label = e.getActionCommand();
				
				if(label.equals("clean"))
					m_malflaeche.repaint();
			}
		}
		
		
				
		//meineCanvas Konstruktor setzt Farben & implementiert das Mousehandling
		meineCanvas(){
			addMouseListener(new meinMouseAdapter());
			addMouseMotionListener(new meinMouseMotionAdapter());
			addActionListener(new meinActionLauscher());
			setBackground(Color.black);
			setForeground(Color.yellow);
		}
		
		//in dieser Methode wird gezeichnet
		public void paint(Graphics g){
			String label;
			ButtonModel aktuell = null;
			
			//Stellt die angeklickte Form fest
			aktuell = m_formauswahl.getSelection();
			
			//entsprechend handeln
			if(aktuell == null)
				return;
			
			//int w = (int) (Math.random()*300);
			//int h = (int) (Math.random()*300);
			label = aktuell.getActionCommand();
			
			if(label.equals("Kreis")){
				Graphics tmp = m_malflaeche.getGraphics();
				g.drawOval(m_Xpos1,m_Ypos1,(m_Xpos2-m_Xpos1)*2,(m_Xpos2-m_Xpos1)*2);
			}
			
			if(label.equals("Scheibe"))
				g.fillOval(m_Xpos1,m_Ypos1,(m_Xpos2-m_Xpos1)*2,(m_Ypos2-m_Ypos1)*2);
			
			if(label.equals("Rechteck"))
				g.drawRect(m_Xpos1,m_Ypos1,(m_Xpos2-m_Xpos1)*2,(m_Ypos2-m_Ypos1)*2);
		}
		
		
		
		//Diese Methode liefert die minimale Groesse der Canvas
		
		public Dimension getMinimumSize(){
			return new Dimension(300,300);
		}
		
		//Lieblingsgroesse auf Minimalgroesse setzen
		
		public Dimension getPreferredSize(){
			return getMinimumSize();
		}
	}
}

 

[Karl]

Hallo,

1) Der ActionListener muss an dem Button angemeldet werden, nicht an dem Canvas. Du hast den Clean-Button opt5 genannt, was aber nicht sinnvoll ist, da er ja nicht zu den Radiobuttons in der Gruppe gehört, sondern eine Schaltfläche darstellt. Vorläufig nenne ihn btnClean und deklariere ihn oben in der Klasse (da wo Du auch die ButtonGroup deklariert hast).
Dann änderst Du die Anmeldung des ActionListeners:

     // addActionListener(new meinActionLauscher());   // geht nicht!
     btnClean.addActionListener(new meinActionLauscher()); //am Button angemeldet

2) Du wirst als nächstes feststellen, dass nun zwar Dein Listener angesprochen wird, sich aber nichts tut.
Zum einen löscht Du nicht! Zum Löschen der Zeichenfläche ist innerhalb Deiner paint-Implementierung der Aufruf super.paint(g); am Einfachsten. Damit wird der Hintergrund gemalt. Das reicht aber nicht, weil Du beim Durchlauf Deiner paint-Methode das letzte Malereignis wiederholst. Du musst Dir also was überlegen, wie die Du den clean-Aufruf dort unterscheidest von den Malaufrufen. Du kannst ja vorerst ein flag setzen oder die Koordinaten ausnullen und Dir später eine vernünftige Lösung überlegen - soll ja erstmal laufen

3) Generelle Anmerkungen:
- Klassennamen beginnen mit einem Großbuchstaben und verwenden CamelCase (MeinSuperMalprogramm) - Variablennamen sollten keine Unterstriche enthalten (nur bei Konstanten), das "m_" (VisualBasic, oder?) hat hier keinen Sinn.
- Die Sichtbarkeit von Klassen und Konstruktoren (z.B. Malprogramm(String titel)) sollte (hier public) angegeben werden. Das Weglassen ist nämlich nicht etwa ohne Bedeutung, sondern hat eine festgelegte Semantik (package-protected). Nur Klassen desselben Package sehen diese so deklarierten Klassen bzw. solche Konstruktoren. Als Beginner stellt man sich leicht ein Bein, weil man schwer versteht, warum man z.B. bei Vererbung in anderen Packages die eigenen Klassen bzw. deren Konstruktoren um's Verrecken nicht sieht.
- Sowas sollte man nicht mit awt-Canvas machen, nimm ein JPanel oder einfach JComponent als Malfläche.
Swing und AWT sollten nach Möglichkeit nicht vermischt eingesetzt werden.

Möglicherweise ist Dein Buch zu alt und nicht besonders?

Gruß,
Karl

 

[chriz_h]

Danke erstmal fuer die ausfuehrlichen Tipps. Werd ich gleich mal ausprobieren, wenn ich @home bin. Das Buch ist in der Tat schon etwas aelter. (von Data Becker) Aber ich hab mich jetzt sowieso entschlossen, erstmal nen paar Tutorials von dieser Seite zu studieren und das Buch beiseite zu legen.
Ist wohl nicht ueberall Java rin wo Java draufsteht...
mfG
ChRizZ