undo funktion in Malprogramm

[anfänger15]

Hallo alle zusammen.

Ich bin gerade dabei mir ein Malprogramm zu programmieren. Ich zeichne in dem Programm zuerst alles in ein BufferedImage und dann das BufferedImage auf einen JPanel. Ich habe versucht mit einem Stack eine undo funktion einzubauen.

Meine Frage: Wie kann ich beim betätigen der undo funktion alles was in dem Stack geschpeichert ist in das BufferdImage malen?

Oder hat jemand eine bessere Idee für eine Rückgängig funktion in einem Malprogramm.

danke

 

[SlaterB]

das ist ja eine logische Frage und was soll man darauf antworten außer 'ja tue es, baue es so dass es funktioniert'?..

bevor du über Undo nachdenkst musst du natürlich allgemein wissen,
wie du ein Bild wieder ganz weiß kriegst und vorgespeicherte Operationen darauf ausführst,

was soll man da sagen?

 

[schalentier]

Also prinzipiell wuerde ich vektororientiert rangehen. Soll heissen, du speicherst keine BufferedImages sondern die Befehle, die der Benutzer durchgefuehrt hat.

Jedes repaint malt dann einfach immer die komplette Liste, quasi eine Art Makro. Vorteil: Du kannst mit der Liste alles moegliche anstellen, zb den letzten Befehl weglassen (=Undo). Oder alles abspeichern.

Wenn das klappt, kannste ueber Performancetuning nachdenken, zb alle paar Zeichenbefehle in der Liste, ein BufferedImage in der Liste ablegen. Repaint zeichnet dann dieses BufferedImage und arbeitet wie gewohnt den Rest ab.

Nur so als Anregungen ;-)

 

[anfänger15]

SlaterB also was mir schalentier gesagt hat hat mir zum Beispiel weitergeholfen
danke schalentier

 

[SlaterB]

> Wie kann ich beim betätigen der undo funktion alles was in dem Stack geschpeichert ist in das BufferdImage malen

> Jedes repaint malt dann einfach immer die komplette Liste

ja, dieser geniale Tipp bringt einen weiter

(sorry, aber wenn du schon so spitz drauf hinweist )

 

[anfänger15]

Ich weiß es ist schon ein weilchen her, hab jetzt aber immer noch ein Problem.
Wenn ich etwas gemalt hab, dann geht die undo funktion, male ich jetzt aber nochmal was und will wieder die undo funktion benutzen geht es nicht mehr, weil sich undo vergrößert hat. Es wird also alles was ich vorher undo gemacht habe wieder gezeichnet.
Hier mal ein bisschen Code:

switch(modus) {
  	case 1:
  	Rcolor.addElement(getForeground());
                undo.addElement(new Rectangle(x, y, w, h));
                break;
}

die undo funktion:

	if(undo.size() > 0){
 	zurück++;
 	 	}

for (int i =0; undo.size()-zurück > i ; i++) { 
 	

    	image.setColor(getForeground());


	    Rectangle p = (Rectangle)undo.elementAt(i);
	    image.setColor((Color)Rcolor.elementAt(i));
	    image.drawRect(p.x, p.y, p.width, p.height);

	
repaint();  
          
         }

Hab mir schon gedacht bei jedem weiteren zeichnen zurück zu erhöhen geht aber nicht.
Hat jemand eine gute Idee???

 

[Marco13]

Ich hab sogar mehrere Ideen: Auf Rechtschreibung achten. Auf Grammatik achten. Auf Zeichensetzung achten.

Aber am wichtigsten: Compilierbaren code posten.

Sonst kann man nur raten und vermuten, WAS du da WANN machst, und was sowas wie "mode" sein soll....

 

[Marco13]

Klicken um Rechtecke zu setzen.

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

class UndoTest extends JPanel implements MouseListener
{
    public static void main(String args[])
    {
        JFrame f = new JFrame();
        f.getContentPane().setLayout(new BorderLayout());

        final UndoTest ut = new UndoTest();
        ut.addMouseListener(ut);
        f.getContentPane().add(ut, BorderLayout.CENTER);


        JButton b = new JButton("Undo");
        b.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent e)
            {
                ut.undo();
            }
        });
        f.getContentPane().add(b, BorderLayout.NORTH);

        b = new JButton("Redo");
        b.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent e)
            {
                ut.redo();
            }
        });
        f.getContentPane().add(b, BorderLayout.SOUTH);
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        f.setSize(400,400);
        f.setVisible(true);
    }

    private ArrayList<Shape> drawn = new ArrayList<Shape>();
    private ArrayList<Shape> undone = new ArrayList<Shape>();

    public void mouseEntered(MouseEvent e) {}
    public void mouseExited(MouseEvent e) {}
    public void mousePressed(MouseEvent e) {}
    public void mouseReleased(MouseEvent e) {}
    public void mouseClicked(MouseEvent e)
    {
        Rectangle r = new Rectangle(e.getX(), e.getY(), 10, 10);
        drawn.add(r);
        repaint();
    }

	public void undo()
	{
		if (drawn.size() == 0) return;
		Shape s = drawn.get(drawn.size()-1);
		drawn.remove(drawn.size()-1);
		undone.add(s);
		repaint();
	}

	public void redo()
	{
		if (undone.size() == 0) return;
		Shape s = undone.get(undone.size()-1);
		undone.remove(undone.size()-1);
		drawn.add(s);
		repaint();
	}


    public void paintComponent(Graphics g)
    {
        super.paintComponent(g);
        g.setColor(Color.black);
        for (Shape s : drawn)
        {
            ((Graphics2D)g).draw(s);
        }
    }
}

 

[anfänger15]

Marco13:
Ich hab sogar mehrere Ideen: Auf Rechtschreibung achten. Auf Grammatik achten. Auf Zeichensetzung achten.

dazu kann ich nur sagen, deutsche Sprache schwere Sprache.

Marco13:
Aber am wichtigsten: Compilierbaren code posten.

Danke für den Code oben, aber dein Code ist bei mir auch nicht compilierbar![/b]

 

[Chris_1980]

OMG, da bekommst du n komplettes KSKB was kompilier- und auch ausführbar ist und dann sowas... :shock:

 

[anfänger15]

Entschuldigung aber wenn ich das bei mir compilier kommt folgendes:

C:\Dokumente und Einstellungen\Luki\Desktop\Bildbetrachter\UndoTest.java:42: <identifier> expected
    private ArrayList<Shape> drawn = new ArrayList<Shape>(); 
                     ^
C:\Dokumente und Einstellungen\Luki\Desktop\Bildbetrachter\UndoTest.java:43: <identifier> expected
    private ArrayList<Shape> undone = new ArrayList<Shape>(); 
                     ^
C:\Dokumente und Einstellungen\Luki\Desktop\Bildbetrachter\UndoTest.java:79: ';' expected
        for (Shape s : drawn) 
                     ^
C:\Dokumente und Einstellungen\Luki\Desktop\Bildbetrachter\UndoTest.java:83: illegal start of expression
    } 
    ^
C:\Dokumente und Einstellungen\Luki\Desktop\Bildbetrachter\UndoTest.java:82: ';' expected
        } 
         ^
C:\Dokumente und Einstellungen\Luki\Desktop\Bildbetrachter\UndoTest.java:52: cannot resolve symbol
symbol  : variable drawn  
location: class UndoTest
        drawn.add(r); 
        ^
C:\Dokumente und Einstellungen\Luki\Desktop\Bildbetrachter\UndoTest.java:58: cannot resolve symbol
symbol  : variable drawn  
location: class UndoTest
      if (drawn.size() == 0) return; 
          ^
C:\Dokumente und Einstellungen\Luki\Desktop\Bildbetrachter\UndoTest.java:59: cannot resolve symbol
symbol  : variable drawn  
location: class UndoTest
      Shape s = drawn.get(drawn.size()-1); 
                          ^
C:\Dokumente und Einstellungen\Luki\Desktop\Bildbetrachter\UndoTest.java:59: cannot resolve symbol
symbol  : variable drawn  
location: class UndoTest
      Shape s = drawn.get(drawn.size()-1); 
                ^
C:\Dokumente und Einstellungen\Luki\Desktop\Bildbetrachter\UndoTest.java:60: cannot resolve symbol
symbol  : variable drawn  
location: class UndoTest
      drawn.remove(drawn.size()-1); 
                   ^
C:\Dokumente und Einstellungen\Luki\Desktop\Bildbetrachter\UndoTest.java:60: cannot resolve symbol
symbol  : variable drawn  
location: class UndoTest
      drawn.remove(drawn.size()-1); 
      ^
C:\Dokumente und Einstellungen\Luki\Desktop\Bildbetrachter\UndoTest.java:61: cannot resolve symbol
symbol  : variable undone  
location: class UndoTest
      undone.add(s); 
      ^
C:\Dokumente und Einstellungen\Luki\Desktop\Bildbetrachter\UndoTest.java:67: cannot resolve symbol
symbol  : variable undone  
location: class UndoTest
      if (undone.size() == 0) return; 
          ^
C:\Dokumente und Einstellungen\Luki\Desktop\Bildbetrachter\UndoTest.java:68: cannot resolve symbol
symbol  : variable undone  
location: class UndoTest
      Shape s = undone.get(undone.size()-1); 
                           ^
C:\Dokumente und Einstellungen\Luki\Desktop\Bildbetrachter\UndoTest.java:68: cannot resolve symbol
symbol  : variable undone  
location: class UndoTest
      Shape s = undone.get(undone.size()-1); 
                ^
C:\Dokumente und Einstellungen\Luki\Desktop\Bildbetrachter\UndoTest.java:69: cannot resolve symbol
symbol  : variable undone  
location: class UndoTest
      undone.remove(undone.size()-1); 
                    ^
C:\Dokumente und Einstellungen\Luki\Desktop\Bildbetrachter\UndoTest.java:69: cannot resolve symbol
symbol  : variable undone  
location: class UndoTest
      undone.remove(undone.size()-1); 
      ^
C:\Dokumente und Einstellungen\Luki\Desktop\Bildbetrachter\UndoTest.java:70: cannot resolve symbol
symbol  : variable drawn  
location: class UndoTest
      drawn.add(s); 
      ^
18 errors

Process completed.

Und ich kann auch nichts dafür das ich ne Rechtschreibschwäche hab, aber falls ich was falsch gemacht hab Entschuldigung.
Danke für deine Hilfe, die undo-Funktion geht jetzt, hab es aber mit einem Stack gemacht.

Sorry sorry sorry!!!!!!

 

[Wildcard]

Sieht aus als hättest du Java < 1.5

 

[anfänger15]

Ja ich hab Version 1.6 Konnte ja nicht wissen, dass es mit den älteren Versionen geht.

 

[Wildcard]

nein, mit allem >1.4 müsste es gehen.

 

[Marco13]

Das mit der Rechtschreibung bezog sich primär(!) darauf, dass ich deinen ersten Post 3 oder 4 mal lesen musste, bis ich gemerkt habe, dass an einer Stelle, wo du das Wort "und" geschrieben hast, eigentlich "undo" gemeint war das kann schon verwirrend sein :wink:

Und ... ja, es muss mit Java 1.5 compiliert werden. Die Alternative ist: Alles, was mit <spitzen Klammern> zu tun hat, wegmachen, die for-Schleife entsprechend als for(int i...) schreiben, und ggf. casten.

 

[anfänger15]

OK, danke für alle die mir gehlofen haben. Die undo Funktion in meinem Programm geht jetzt ja.