Clear im Grafik Editor

[chain]

Hallo,

ich habe eine einfachen Grafik editor geschrieben. Er funktioniert auch einwandfrei, nur habe ich ein Problem(bitte nicht lachen :roll: :roll: :roll: :roll: )

Also erst einmal: die ganzen Grafikobjekte(wie Kreis, recheck,...) werden in einen Vector gespeichert.
ich habe einen JFrame das die Zeichenoberfläche darstellt.
in der mehode paint werden alle im Vector gespeicherten Grafikobjekte durchlaufen und es wird deren zeichenmethode aufgerufen.

Ich habe eine Funktion Clear eingebaut, die alle Zeichenobjekte von der Zeichenoberfläche löscht(es wird einfach der Vector mit v.clear(); wieder "auf 0 gesetzt". Danach wird die Methode repaint(); aufgerufen, jedoch ändert sich nichts, bis die paint(); methode irgendwie anders aufgerufen wird.

wie mach ich es das sofort nach dem v.clear(); die Zeichenfläche "gerepainted" wird???

Bitte nicht lachen :wink: :wink: :wink: :wink: :wink:


mfg

 

[Leroy42]

Sollte eigentlich funktionieren ???:L

Macht deine clear-Methode noch irgendwas, nachdem sie repaint aufruft?
Wird sie überhaupt in einer z.B. actionPerformed-Methode eines Buttons aufgerufen.

GibUnsMalCode

 

[chain]

public void actionPerformed(ActionEvent event)
{
if(event.getActionCommand().equals("Clear"))
{
v.clear();
this.repaint();
}
}


das ist alles. :bahnhof: :bahnhof:

 

[Leroy42]

Der EventDispatchThread wird also sofort wieder verlassen und das Painting sollte erfolgen.

Dann habe ich auch keine Idee, warum das nicht funzt

 

[chain]

hier mal die ganze methode, vielleicht hilft das!

public void actionPerformed(ActionEvent event)
	{
//das Programm wird beendet
		if(event.getActionCommand().equals("Ende"))
	    {
	    	this.setVisible(false);
	    	this.dispose();
	    	System.exit(0);
	    }
		
//Clear
	    if(event.getActionCommand().equals("Clear"))
	    {
	    	v.clear();
	    	this.repaint();	    	
	    }
//Neue Linie wird am Mauszeiger gezeichnet. Die 2 Koordinaten muss man eingeben
	    if(event.getActionCommand().equals("neue Linie"))
	    {
	    	int x2,y2=0;

                // Hier werden die 2. Koordinaten für Linie ermittelt/eingegeben

	    	x2=this.inXY2(2,'x',m.getMouseX(),m.getMouseY(),"2. X-Koordinate (0=Koordinate des Mauszeigers)","2. Y-Koordinate (0=Koordinate des Mauszeigers)");
	    	y2=this.inXY2(2,'y',m.getMouseX(),m.getMouseY(),"2. X-Koordinate (0=Koordinate des Mauszeigers)","2. Y-Koordinate (0=Koordinate des Mauszeigers)");
	    	
	    	
                //Linie wird zu Vectror hinzugefügt
	    	v.add(new Linie(m.getMouseX(),m.getMouseY(), x2, y2, Color.BLUE, Color.black));
	    	this.repaint(); //hier funktioniert es
	    }
}

alles bis auf das repaint bei clear funzt
BIIIIIIITTTTTTTTTTTTTEEEEEEEEEEE HHHHHHHHHHHHIIIIIIIIIIILLLLLLLLFFFFFFFFFFFFFEEEEEEEEEEE :cry: :cry: :cry: :cry: :cry: :cry:

[Edit by Beni: Codetags]

 

[chain]

das selbe Problem habe ich wenn ich ein Grafikelement lösche!

Bittel Hilfe. Danke :bahnhof: :bahnhof: :bahnhof: :bahnhof: :bahnhof: :bahnhof: :bahnhof:

 

[Leroy42]

Ist es einundderselbe Vector aus dem du zeichnest und indem du löschst?

Zeige doch mal deine repaintComponent-Methode

 

[chain]

hier mal meine paint methode.
ja es is immer der eine Vector v

public void paint(Graphics g)
	{
		this.setBackground(Color.WHITE);
		
//Vector durchlaufen		
for(int i=0;i<v.size();i++)
		{
//Rechtecke zeichnen
			if(v.elementAt(i).getClass().getName().equals(Rechteck.class.getName()))		
				((Rechteck)(v.elementAt(i))).zeichne(g);

//Linien zeichnen
			else if(v.elementAt(i).getClass().getName().equals(Linie.class.getName()))
					((Linie)(v.elementAt(i))).zeichne(g);
				
		}
		
	}

[Edit by Beni: Codetags]

 

[Beni]

Den Hintergrund könntest du noch übermalen:

g.setColor( Color.WHITE );
g.fillRect( 0, 0, getWidth(), getHeight() );

Sonst kann ein altes Bild dableiben, und du zeichnest ja beim löschen nichts mehr.

P.S. und dann befasse dich mal mit Interfaces, deine Casts sind ja übelst kompliziert (und total unnötig wenn die Klassen alle dasselbe Interface implementieren würden) (Beispiel)

 

[chain]

Ich habs jetzt sehr unschön umgangen.
Nach dem nicht funktionierenden repaint, minimiere ich das Fenster einfach und mache es wieder groß!

Aber bitte, falls ihr eine Lösung habt, bitte posten, ich bin am verzweifeln, weil das ist mit minimieren ist echt sch****!

mfg

 

[chain]

danke für Tipp!

währe aber trotzdem an einer "besseren" Lösing interessiert.

 

[Leroy42]

Keine Ahnung. Es sieht alles funktionsfähig aus ???:L

Versuche doch mal einzukreisen wann er überhaupt welchen Code ausführt, indem du
Statusmeldungen via System.out.println (oder Breakpoints falls du eine IDE benutzt) einfügst.

Vielleicht erreicht er aus irgendeinem Grund deine clear-Methode gar nicht:

Aber was ganz anderes:

1) Versuch dir anzugewöhnen Instanzmethoden ohne this. aufzurufen, falls nicht unbedingt nötig.

2)

//Rechtecke zeichnen
			if(v.elementAt(i).getClass().getName().equals(Rechteck.class.getName()))		
				((Rechteck)(v.elementAt(i))).zeichne(g);

//Linien zeichnen
			else if(v.elementAt(i).getClass().getName().equals(Linie.class.getName()))
					((Linie)(v.elementAt(i))).zeichne(g);

:shock: Gruselig

Genau hierfür gibt es objektorientierte Sprachen um sein ein Gefriemel zu vermeiden.
Außerdem: Was passiert, wenn du oder andere neue Graphik-Elemente hinzufügst?
Die paintComponent-Methode muß jeweils angepaßt werden.
Ansatz:

abstract class Figur {
   abstract public void zeichne(Graphics G);
}

class Rechteck extends Figur {
    private int x;
    private int y;
    ...
    public void zeichne(Graphics G) {
        // hier deine Zeichenroutine für Rechtecke
    }
}

class Linie extends Figur {
    ...
    public void zeichne(Graphics G) {
        // hier deine Zeichenroutine für Linien
    }
}

Dann brauchst du in der paintComponent-Methode keine Typabfrage und keine Herumgecaste sondern
schreibst einfach:

for(int i=0;i<v.size();i++)
    ((Figur) v.elementAt(i)).zeichne(g);

und auch wenn andere neue Figuren hinzufügen, braucht dein Quelltext nicht geändert werden.

 

[Leroy42]

Beni's Idee ein Interface zu verwenden, ist natürlich noch besser als meiner.

Dann müssen deine Klassen nicht unbedingt in einer Hierarchie sein.

 

[Leroy42]

Beni's Idee ... ist natürlich noch besser als meiner.

Das kommt davon, wenn man hinten anders aufhört als man vorne anfängt

 

[chain]

Hey danke das für die hilfe, das ist echt einfacher!
noch ne frage zu abstract:

muss ich da vor dem Konstruktor abstract schreiben???

DANKE @ all[/quote]

 

[Leroy42]

Du mußt es nicht nur nicht, du darfst es nicht

Der Konstruktor hat zwar nichts zu tun aber er kann auch nicht überschrieben werden.

==> Schreib' keinen Konstruktor ==> Es wird der Standard-Konstruktor automatisch eingebunden.

Du brauchst nur die Konstruktoren von Rechteck, Linie, ... zu definieren.

Was anderes ist, wenn alle Figuren noch ein gemeinsames Attribut, z.B. Vornamen, haben.
Dann steht dieses Attribut bereits in der Klasse Figur und ein Konstruktor macht Sinn.

Dennoch: Beni's Vorschlag ist besser. Damit brauchst du überhaupt nichts zu ändern außer
in den 1. Zeilen der Klassendefinitionen.

interface KannMichZeichnen {public void zeichne(Graphics g);}

class Rechteck implements KannMichZeichnen {...}

 

[chain]

Nochmals danke,
habe das jetzt geändert, und es ist um welten kürzer geworden!
aber naja, bin halt noch nicht so gut!

Tolles Forum!