Die Farbe wird einfach nicht gesetzt :(

[pro_evo]

Hi all,

bin eigentlich Java Anfänger, aber ich glaub bei AWT is mein Problem am besten aufgehoben.

Es ist ein kleines Malprogramm ... man kann 4 Figuren zeichnen und die Parameter selbst eingeben.

Ich wollte nun einbauen das man die Farbe setzen kann, Zeichenfläche ist das Canvas mit dem Namen "mycan".

Man kann zwar mehrere Haken setzen, aber dabei wird es halt immer die letzte die Farbe sein mit der er zeichnet, weil die Color eben bei jedem Haken machen neu gesetzt wird.
(so war es gedacht) nur leider wird die Farbe ja gar ned gesetzt :cry: :cry:

Also der Code is leider etwas lang, wichtig ist eigentlich die itemStateChanged aus dem Frame .
(der übergebene Grafikkontext ist doch der richtige oder ? bei der MEthode parse_and_paint klappts ja auch) :?


schonmal Danke im Voraus !

import java.awt.event.*;
import java.awt.*;
import java.awt.Polygon;
import java.util.*;
import java.io.*;

interface GeometrieObjekt {
  public void paint (Graphics g);         // Malen
  public boolean contains (int x, int y); // Enthält das Objekt den Punkt (x, y)?
  public String getName ();               // Was bist Du?
}


class Kreis implements GeometrieObjekt {
  int r;      // r = Radius
  int mx, my; // mx = x Mittelpunkt, my = y Mittelpunkt
  public Kreis (int r, int x, int y) {
    this.r = r; mx = x; my = y;
  }
  public void paint (Graphics g) {
  	// Hart rein funzt es , also der Grafikkontext stimmt 
  	/*g = mycan.getGraphics();
  	g.setColor( Color.blue );*/
  	g.drawArc (mx-r, my-r, 2*r, 2*r, 0, 360);
  }
  public boolean contains (int x, int y) {
    return (mx-x)*(mx-x)+(my-y)*(my-y) <= r*r;
  }
  public String getName () { return "Kreis"; }
}

class RechtEck implements GeometrieObjekt {
  int x, y, b, h; // Linke obere Ecke, Breite, Höhe
  public RechtEck (int x, int y, int b, int h) {
    this.x = x; this.y = y;
    this.b = b; this.h = h;
  }
  public void paint (Graphics g) {
    g.drawRect (x, y, b, h);
  }
  public boolean contains (int x, int y) {
     return this.x <= x && x <= this.x+b &&
            this.y <= y && y <= this.y+h;
  }
  public String getName () { return "Rechteck"; }
}

class Ellipse implements GeometrieObjekt {
	int x, y, b, h;
	public Ellipse (int x, int y, int b, int h) {
	    this.x = x; this.y = y;
    	this.b = b; this.h = h;
    }
    public void paint (Graphics g){
    	g.drawOval(x,y,b,h);
    }
    public boolean contains (int x, int y) {
    	return ((x*x/b*b)+(y*y/h*h) != 1 );
	}
  public String getName () { return "Ellipse"; }	
}

class Dreieck implements GeometrieObjekt {
	int x[], y[];
	int n;
	Polygon polygon;
	public Dreieck (int []x, int []y,int n){
		this.x = x;
		this.y = y;
		this.n = n;
		polygon = new Polygon(x,y,n);
	}
	public void paint (Graphics g) {
		g.drawPolygon(x, y, n);
	}
	public boolean contains (int x, int y) {
		return polygon.contains(x, y);
	}
	public String getName () { return "Dreieck"; }
}

class myDialog1 extends Dialog implements ItemListener, ActionListener {
	
	private java.awt.List myList = new java.awt.List(6,false);
	protected String[] geos = {"Kreis","Ellipse","Rechteck","Dreieck"};
	private Button back = new Button ("Cancel");
	private Frame parent;
	
	// Für Dialog 2
	public myDialog1 (Dialog drueber,String titel, boolean modal){
		super(drueber,titel,modal);
	}
	public myDialog1 (Frame parent,String titel, boolean modal){
		super(parent, titel, modal);
		this.parent = parent;
		for (int i=0; i<geos.length; i++)
		myList.add(geos[i]);
		add(myList);
		myList.addItemListener(this);
		add("South", back);
		back.addActionListener(this);
		pack();
		setVisible(true);
	}
	
	public void itemStateChanged (ItemEvent ie){
		dispose();
		new myDialog2(this,myList.getSelectedItem(),parent);
		
	}
	
	public void actionPerformed (ActionEvent e){
  	 if (e.getSource() == back)
  	 	dispose();
  	}
}

class myDialog2 extends myDialog1 implements ActionListener {	
	
	TextField tf;
	String vorbelegung, figur;
	Frame parent;
	private Button back = new Button ("Cancel");
	private Button go = new Button ("Los zeichne");
	
	
	public myDialog2 (Dialog drueber, String figur, Frame parent){
		super(drueber, "Parameter eingeben (Wichtig:Auf die Form achten !)",true); 
		this.parent = parent;
		this.figur = figur;
		setLayout(new BorderLayout());
		if (figur.equals("Kreis"))
			vorbelegung = "radius,X-Koordinate von M,Y-Kooradinate von M";
		if (figur.equals("Rechteck"))
			vorbelegung = ("X-Start,Y-Start;Breite,Höhe");
		if (figur.equals("Ellipse"))
			vorbelegung = ("X-Start,Y-Start,Breite,Höhe");
		if (figur.equals("Dreieck"))
			vorbelegung = ("x1,y1,x2,y2,x3,y3");
		tf = new TextField(vorbelegung);
		add(tf);
		add("West", back);
		add("East", go);
		go.addActionListener(this);
		back.addActionListener(this);
		pack();
		setVisible(true);
	}	
	
	public void actionPerformed (ActionEvent e){
  	 if (e.getSource() == back)
  	 	dispose();
  	 if (e.getSource() == go){
  	 System.out.println("Ihre Eingabe: "+tf.getText());
  	   ((x_mas_nr5)parent).parse_and_paint(tf.getText(), figur);
  	   dispose();
  	  } 
  	}
}

		
public class x_mas_nr5 extends Frame implements ActionListener, MouseMotionListener, ItemListener {
	
  	private Button zeichnen = new Button("Zeichne");
  	CheckboxMenuItem schwarz, blau, rot, gruen, gelb;
  	MenuItem speichern, lade_und_male;
  	private Canvas mycan = new Canvas();  
 	private Vector geo = new Vector();
 	private  int []xf = new int[3];
 	private  int []yf = new int[3];
 	private  int x, y, breite, hoehe, radius;
 	Graphics g = mycan.getGraphics();
 	Vector landschaft = new Vector();
 	String current_to_save; // hat bereits das korrekte Format
 	
 // Main-Frame Konstruktor	
 public x_mas_nr5() {
 		super("Der Wunschzeichner");
 		add("South", zeichnen);
  		add(mycan);
  		zeichnen.addActionListener(this);
  		mycan.addMouseMotionListener (this);
  		initMenu();
  		
  		setBounds(200, 200, 640,480);
 		setVisible(true);
 		
 		addWindowListener (new WindowAdapter() {
      	public void windowClosing (WindowEvent event) {
      		System.err.println("Schon am Ende ?? Na ok. Bye.");System.exit (0);}});
 	}
 	
 public void initMenu(){
  	MenuBar myBar = new MenuBar();
  	Menu myMenu = new Menu("Zeichenfarbe");
  	myBar.add(myMenu);
  	myMenu.add(schwarz = new CheckboxMenuItem("Schwarz"));
  	myMenu.add(blau = new CheckboxMenuItem("Blau"));
  	myMenu.add(rot = new CheckboxMenuItem("Rot"));
  	myMenu.add(gruen = new CheckboxMenuItem("Gruen"));
  	myMenu.add(gelb = new CheckboxMenuItem("Gelb"));
  	myMenu = new Menu ("Landschafts-Optionen");
  	myBar.add(myMenu);
  	myMenu.add(lade_und_male = new MenuItem ("Landschaft laden und malen"));
  	myMenu.add(speichern = new MenuItem ("Landschaft speichern"));

  	
  	schwarz.addItemListener(this);
  	blau.addItemListener(this);
  	rot.addItemListener(this);
  	gruen.addItemListener(this);
  	gelb.addItemListener(this);
  	lade_und_male.addActionListener(this);
  	speichern.addActionListener(this);
  	
  	setMenuBar(myBar);
  } 	 
 	
  // Jede gezeichnete Figur wird dem Vector hinzugefügt	
  public void parse_and_paint(String p, String figur){
  	StringTokenizer st = new StringTokenizer (p,",");
  	try{
  		current_to_save = (figur+":"+p);
  		if (figur.equals("Kreis")){
  		radius = Integer.parseInt(st.nextToken());
  		x = Integer.parseInt(st.nextToken());
  		y = Integer.parseInt(st.nextToken());
  		Kreis k = new Kreis (radius,x,y);
  		k.paint(mycan.getGraphics());		// wieso geht ned "g" ?
  		landschaft.add(current_to_save);
  		}
  		if (figur.equals("Rechteck")) {
  		x = Integer.parseInt(st.nextToken());
  		y = Integer.parseInt(st.nextToken());
  		breite = Integer.parseInt(st.nextToken());
  		hoehe = Integer.parseInt(st.nextToken());
  		RechtEck re = new RechtEck(x,y,breite,hoehe);
  		re.paint(mycan.getGraphics());
  		landschaft.add(current_to_save);
  		}
  		if (figur.equals("Ellipse")) {
  		x = Integer.parseInt(st.nextToken());
  		y = Integer.parseInt(st.nextToken());
  		breite = Integer.parseInt(st.nextToken());
  		hoehe = Integer.parseInt(st.nextToken());
  		Ellipse el = new Ellipse (x,y,breite,hoehe);
  		el.paint(mycan.getGraphics());
  		landschaft.add(current_to_save);
  		}
  		if (figur.equals("Dreieck")){
  		xf[0] = Integer.parseInt(st.nextToken());
  		yf[0] = Integer.parseInt(st.nextToken());
  		xf[1] = Integer.parseInt(st.nextToken());
  		yf[1] = Integer.parseInt(st.nextToken());
  		xf[2] = Integer.parseInt(st.nextToken());
  		yf[2] = Integer.parseInt(st.nextToken());
  		Dreieck dr = new Dreieck (xf,yf,3);
  		dr.paint(mycan.getGraphics());
  		landschaft.add(current_to_save);
  		}
  		System.out.println("Figur wurde korrekt gezeichnet :-)");
  	
  	}
  	catch (NumberFormatException nfe) { System.err.println("Zeichnen nicht moeglich.\nSie haben bei der Eingabe gepatzt.(Falsche Form!)");}
  	catch (NoSuchElementException nse) { System.err.println("Zeichnen nicht moeglich.\nSie haben zuwenig Parameter eingeben.");}
  	
  } 
  
  public void safe_landschaft(){
  	try {
  	FileWriter fw = new FileWriter("landschaft.txt");
  	BufferedWriter bw = new BufferedWriter(fw);
  	PrintWriter pw = new PrintWriter(bw);
  	for (int i=0; i<landschaft.size(); i++){
  		pw.println((String)(landschaft.elementAt(i)));
  	}
  	pw.close();
  	}
  	catch (IOException io) { System.err.println("Mann, Sie haben ein Fehler beim speichern verusacht, grrr !");}
  }
  
  public void load_and_paint(){
  	try {
  	FileReader fr = new FileReader("landschaft.txt");
  	BufferedReader br = new BufferedReader (fr);
  	String zeile;
  	while ((zeile = br.readLine()) != null) {
  	//while (br.ready()) {
  		//zeile = br.readLine();	// ginge auch !!
  	  	StringTokenizer st = new StringTokenizer (zeile,":");
  		String figur = st.nextToken();
  		String parameter = "";
  		parameter = (parameter+st.nextToken()); // Name mit ":" abgetrennt , Parameter mit "," 
  		System.out.println("Gefunden: "+figur+":"+parameter);
  		parse_and_paint(parameter,figur);
  	}
  	br.close();
  	System.out.println("Erfolgreich geladen, Datei wieder geschlossen.");
  	}
  	catch (IOException io) { System.err.println("Mann, Sie haben ein Fehler beim laden verusacht, grrr !");}
  }	
  
  
  // Versprechen an ItemListener
  public void itemStateChanged (ItemEvent ie){
  	if (ie.getStateChange() == ItemEvent.SELECTED){
  		 if (ie.getSource() == schwarz)
  		 mycan.getGraphics().setColor(Color.black);		
  		 else if (ie.getSource() == blau)
  		 mycan.getGraphics().setColor(Color.blue); 	  // Kein Effekt ? 
  		 else if (ie.getSource() == rot)			// getItem wird ja gar ned erfüllt !
  		 System.out.println("ItemEvent: Farbe ROT !!!!! zefix");
  		 //mycan.getGraphics().setColor(Color.red);
  		 else if (ie.getSource() == gruen)
  		 mycan.getGraphics().setColor(Color.green);
  		 else if (ie.getSource() == gelb)
  		 mycan.getGraphics().setColor(Color.yellow);
  		 
  		 System.out.println("ItemEvent: Farbe geaendert.");
  		 // (erfolglose) tests zur Problemfindung
  		 mycan.getGraphics().setColor( Color.blue );
  		 mycan.getGraphics().drawRect(50,50,70,70);
  	}	
  }

  // Versprechen an AcionListener
  public void actionPerformed (ActionEvent e){
  	if (e.getSource()== zeichnen)
  	 	//myDialog1 dia1 = new myDialog1(this,"Figur wählen",true); // SOWAS GEHT NICHT ???????
  	 	new myDialog1(this,"Figur wählen",true);
  	 else if (e.getSource()== speichern){
  	 	safe_landschaft();
  	 	System.out.println("ActionEvent: Datei landschaft.txt erflolgreich erstellt.");
  	 }
  	 else if (e.getSource()== lade_und_male){
  	 	System.out.println("ActionEvent: Lade landschaft.txt");
  	 	load_and_paint();
  	 }
  }
  
  // Versprechen an MouseMotionListener
  public void mouseMoved (MouseEvent e) { }
  long z = 0;
  int x1 = 0;
  int y1 = 0;
  int x2 = 0;
  int y2 = 0;
  public void mouseDragged (MouseEvent e) {  // Spielerei
    g = mycan.getGraphics();
  	z = z+1; 
     //System.out.println ("mouseMoved " + e); 
    if (z%2 == 0){
    x2 = e.getX();
    y2 = e.getY();
    g.drawLine(x1, y1, x2, y2);
    }
    else {
    x1 = e.getX();
    y1 = e.getY();
    g.drawLine(x2, y2, x1, y1);
    }
  }


  public static void main (String []args){
	 x_mas_nr5 mein_programmObjekt = new x_mas_nr5();
  }

}

 

[pro_evo]

hmm niemand ne idee ?

sollte es denn so funzen ? Das würd mir ja schon reichen wenn hier ein java guru sagt ... es "sollte" so eigentlich funzen ... :/

 

[dotlens]

jede klasse hat ihr eigenes Graphics Objekt, du änderst die Farbe im Graphics objekt der Klasse x_mas_nr5

zeichnest aber kreise mit der Klasse Kreis und ellipsen mit der Klasse Ellipse, welche ein anderes Graphics Objekt und damit eine andere Farbe haben. du musst es so einrichten, dass du eine Funktion in den KKlassen kreis und Ellipse schreibst, welche die Farbe setzt, und dann änderst du die Farbe über diese Methode...

 

[pro_evo]

jede klasse hat ihr eigenes Graphics Objekt, du änderst die Farbe im Graphics objekt der Klasse x_mas_nr5

zeichnest aber kreise mit der Klasse Kreis und ellipsen mit der Klasse Ellipse, welche ein anderes Graphics Objekt und damit eine andere Farbe haben. du musst es so einrichten, dass du eine Funktion in den KKlassen kreis und Ellipse schreibst, welche die Farbe setzt, und dann änderst du die Farbe über diese Methode...

Hi,

danke für den Tipp ... das stimmt schon nur kann ich das leider ned so einfach machen .. zum Zeitpunkt wo er die Farben ändern soll hab ich ja noch gar keine Objekte von Kreis, Rechteck etc.....
Und ich muss es nach deiner Aussage ja für jede KLasse KReis, Rechteck extra machen ...

Wenn deine Aussage stimmt, warum geht dann aber folgendes Programm nicht ? Hier habe ich das Problem von oben auf das Problem selbst reduziert und es steht auch alles in einer Klasse :

(Er macht sogar nicht nur NICHTS sondern spuckt noch ne NullPointerException aus, vlt. hilft das ja bei der Fehlersuche ... ich finde ihn nicht
Diese Graphics raubt mir noch den letzten Nerv :cry:

import java.awt.*;
import java.awt.event.*;



public class ColorTester extends Frame implements ItemListener {

	List MyList;

	Graphics g;


	public ColorTester() {

		super("ColorTester-Frame");
		setSize(600, 150);
		setBackground(Color.gray);
		setLayout(null);
		setVisible(true);

		MyList=new List(4, false);

		MyList.add("schwarz");
		MyList.add("weiss");
		MyList.add("blau");
		MyList.add("gelb");

		MyList.setBounds(10,30,100,80);

		add(MyList);

		MyList.addItemListener(this);

	}
	public void paint(Graphics g) {
		g.drawString("Kleine Testausgabe!", 180, 80);
	}


	public void itemStateChanged(ItemEvent e) {
		int MyHit=((Integer)e.getItem()).intValue();

		if	(MyHit==0)
			g.setColor(Color.black);

		else if(MyHit==1)
			g.setColor(Color.white);

		else if(MyHit==2)
			this.getGraphics().setColor(Color.blue);

		else if(MyHit==3)
			getGraphics().setColor(Color.yellow); 

		

		//paint(g);
		repaint();

		
	System.out.println("Item " + e.getItem() + " hat Aktion ausgeloest: " + e.toString());

	}


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

	}
}

 

[Wildcard]

Wenn deine Aussage stimmt, warum geht dann aber folgendes Programm nicht ?

Weil du den gleichen Fehler wieder machst:

Graphics g;

dieses g hat nichts mit dem an die paint-Methode übergebenen Graphics-Objekt zu tun.
Das von dir erstellte g wird mit "null" initialisiert, setColor() führt daher zu einer Exception.

 

[pro_evo]

Wenn deine Aussage stimmt, warum geht dann aber folgendes Programm nicht ?

Weil du den gleichen Fehler wieder machst:

Graphics g;

dieses g hat nichts mit dem an die paint-Methode übergebenen Graphics-Objekt zu tun.
Das von dir erstellte g wird mit "null" initialisiert, setColor() führt daher zu einer Exception.

ok ich glaub jez hab ichs so langsam begriffen thx

So geht's nun ... aber leider nur mit der paint Methode weil ich das Graphics Objekt übergeben kann ... mit repaint leider nicht , aber das ist ja eigentlich nict so schlimm hoff ich mal ...

Das Prinzip aber nun auf mein eigentliches Programm zu übertragen ist glaub ich ne "Mission Impossible",denn paint soll nach dem Color setzen ja gar ned aufgerufen werden, sondern es soll :
"Ganz Allgemein die Farbe gesetzt werden für die ZEICHENFLÄCHE mycan", aber sowas ist wie es aussieht nicht möglich in Java oder ? ... :cry:

import java.awt.*;

import java.awt.event.*;



public class ColorTester extends Frame implements ItemListener {

	List MyList;



	public ColorTester() {

		super("ColorTester-Frame");

		setSize(600, 150);

		setBackground(Color.gray);

		setLayout(null);

		setVisible(true);

		

		MyList=new List(4, false);

		MyList.add("schwarz");

		MyList.add("weiss");

		MyList.add("blau");

		MyList.add("gelb");

		MyList.setBounds(10,30,100,80);

		add(MyList);

		MyList.addItemListener(this);


		addWindowListener( new WindowAdapter() {

			public void windowClosing( WindowEvent Event ) {

				System.exit( 0 );

			}

		});

	}


	public void paint(Graphics g) {

		g.drawString("Kleine Testausgabe!", 180, 80);

	}


	public void itemStateChanged(ItemEvent e) {

		Graphics g = getGraphics();

		int MyHit=((Integer)e.getItem()).intValue();


		if(MyHit==0)

			//MyColor=Color.black;

			g.setColor(Color.black);

		else if(MyHit==1)

			g.setColor(Color.white);

		else if(MyHit==2)

			g.setColor(Color.blue);

		else if(MyHit==3)

			g.setColor(Color.yellow); 

		

		paint(g);

		//repaint();

		

		System.out.println("Item " + e.getItem() + " hat Aktion ausgeloest: " + e.toString());

		System.out.println("Momentane Farbe: " + g.getColor());

	}


	public static void main(String [] args) {

		new ColorTester();

	}

}