paint() in anderen Methoden

[Sladda]

Hallo!
Ich habe folgende Klasse:

[HIGHLIGHT="Java"]public class Lampe {

private boolean leuchtet;
private int durchmesser = 50;
private int x;
private int y;


// Konstruktor fuer die Lampe
public Lampe(int x, int y){
leuchtet = false;
x= x;
y = y;
}
public void zeichneAn(Graphics g){
g.setColor(Color.RED);
g.fillOval(x, y, durchmesser, durchmesser);
}

public void zeichneAus(Graphics g){
g.setColor(Color.BLACK);
g.drawOval(x, y, durchmesser, durchmesser);
}

public void paint(Graphics g){
if(leuchtet)
zeichneAn(g);
else
zeichneAus(g);
}
}[/HIGHLIGHT]

Ich will nun in einer anderen Klasse (class Foo) in der main mehrere Lampen erzeugen und dann auf ein Canvas zeichnen (mit lampe.zeichneAus()) aber das klappt nicht (die kreise werden ganz kurz angezeigt und sind dann sofort wieder weg):

[HIGHLIGHT="Java"]public class Foo extends Frame{

class MyCanvas extends Canvas{
public void paint(Graphics g)
{
super.paint(g);
g.drawString("Canvas",10,20);
}
}

Lampe [] lampenArray = new Lampe[7];
Canvas c = new MyCanvas();

public Foo(){
//den super-Konstruktor (von Frame) aufrufen, damit auch ein Frame gezeichnet wird:
super("foo");
//den Frame auf die richtige Groesse setzen:
setSize(700, 700) ;
//und sichtbar machen:
setVisible(true) ;
//Canvas zum Frame hinzufuegen:
add(c);
}

public static void main(String[] args) {

Foo foo= new Foo();

for (int i=0;i<7;i++){
foo.lampenArray = new Lampe(50*i,50*i); //7 Lampen werden erzeugt und in das lampenArray eingefuegt
foo.lampenArray.zeichneAus(foo.c.getGraphics()); //und auf das Canvas gezeichnet
}

}

}[/HIGHLIGHT]
Ich mache bestimmt viel falsch mit paint usw, aber ich weiß einfach nicht wies richtig geht
Ich danke für jede Hilfe!
Gruß
Sladda

(PS: wieso kann man hier den code nicht mehr schön formatieren?)

 

[DarkLoG]

Nur schnell in aller Kürze:

paint() in Lampe wird nie aufgerufen oder? paint() würde von Java nur dann automatisch aufgerufen wenn Lampe von irgendeiner Component abgeleitet wäre und dann müsstest du die "Lampen" natürlich auch dem Frame hinzufügen mit add() etc. - Alternativ könntest du vielleicht die paint()-Methode von Foo überschreiben und dort dann deine Schleife reinpacken, weil so gehst du die Schleife genau einmal durch und beim nächsten automatischen repaint ist natürlich alles wieder weg...

Gruß

DarkLoG

 

[Sladda]

kann ich denn paint() von Lampe in Foo irgendwie aufrufen?

Bin total verwirrt...

 

[Marco13]

Das per Hand aufzurufen funktioniert so nicht. Du brauchst irgendeine Klasse, die von Component erbt. Eine instanz dieser Klasse kannst du dann in einen Frame legen. Und in der paint-Methode dieser Component kannst du dann zeichnen, was du willst. Ähnlich wie es jetzt schon beim Canvas ist. Du könntest (falls das Sinn macht) auch die Lampen in den MyCanvas legen, und dort nach dem
g.drawString("Canvas",10,20);
dann noch
lampe.paint(g);
aufrufen.

 

[Sladda]

Erstmal Danke für eure Antworten.

Allerdings habe ich überhaupt nichts davon hinbekommen.
Ich habe nun versucht alles was mit dem zeichnen zu tun hat in eine klasse zu packen, aber ich kriegs einfach nciht hin.
Das eigentliche Problem: Ich will ja nur ein paar Methoden haben, z.B. zeichneLampe(x,y) o.ä., die ich einfach auf einem Objekt (is ja erstmal egal welches) aufrufen kann, und dann soll aber auch die Lampe gezeichnet werden.
Mein (neuer) Versuch:

[HIGHLIGHT="Java"]class MyCanvas extends Canvas
{
public void paint(Graphics g)
{
super.paint(g);
g.drawString("Canvas",10,20);
}
}


public Canvas c = new MyCanvas();



public void zeichneLampe(int lampe, boolean an, Graphics g){
int x,y;
//if(lampe==1)
g.setColor(Color.BLACK);
g.drawOval(31, 39, 50, 50); ;
}

public void paint(Graphics g){
zeichneLampe(1,true,g);
}[/HIGHLIGHT]
Wie kann ich es denn bloß schaffen, dass ich zB irgendwann (z.B. auch bei einem buttton-klick) meinZeichner.zeichneLampe(1,true,g) aufrufen kann ??

Ich verstehe das mit dem paint einfach nich

 

[Marco13]

Man kann nicht Zeichnen. Punkt. Man kann nicht sagen: Zeichne jetzt dies-und-das da-und-da hin. Es geht nicht. Finde dich damit ab. (Auch wenn es nicht wirklich so ist )

Erfeulicherweise kann man aber zeichnen LASSEN. Man kann eine Klasse erstellen, die von Canvas erbt, und eine Menge Zeichenbarer Objekte enthält. In ihrer paint-Methode zeichnet diese Klasse alle Objekte. Und man kann die Klasse so gestalten, dass man zeichenbare Objekte hinzufügen und entfernen kann.

In deinem Fall: Dein "MyCanvas" kann eine Liste von "Lampen" haben, und Methoden anbieten, um die Lampen hinzuzufügen oder zu entfernen. In seiner paint-Methode zeichnet er alle seine Lampen.

Dass man Lampen noch ein Flag geben kann, ob sie an oder aus sind, oder sonstwas, hat damit nichts zu tun.

Hier nochmal ein (etwas allgemeineres, aber immernoch nur hingehacktes) Beispiel.

import java.awt.*;
import java.awt.event.*;
import java.util.List;
import java.util.*;

interface Paintable
{
    void paintOn(Graphics g);
}


class Lampe implements Paintable
{
    public void paintOn(Graphics g)
    {
        g.drawString("Ich bin eine Schreibtischlampe (mit EINEM 'sch')", 50, 50);
    }

}

class Painter extends Canvas
{
    private List<Paintable> paintables = new ArrayList<Paintable>();

    public void paint(Graphics g)
    {
        super.paint(g);
        for (Paintable paintable : paintables)
        {
            paintable.paintOn(g);
        }
        g.drawString("Canvas",10,20);
    }

    public void addPaintable(Paintable paintable)
    {
        paintables.add(paintable);
        repaint();
    }

    public void clear()
    {
        paintables.clear();
        repaint();
    }

}


class PaintableTest extends Frame
{
    public static void main(String args[])
    {
        new PaintableTest().setVisible(true);
    }

    Painter painter = new Painter();

    public PaintableTest()
    {
        setLayout(new BorderLayout());

        add(painter, BorderLayout.CENTER);

        Button b = new Button("add");
        b.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent e)
            {
                zeichneLampe(1,true);
            }
        });
        add(b, BorderLayout.NORTH);

        b = new Button("clear");
        b.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent e)
            {
                zeichneLampe(1, false);
            }
        });
        add(b, BorderLayout.SOUTH);
        setSize(500,500);
    }


    public void zeichneLampe(int lampe, boolean an)
    {
        if (an)
        {
            painter.addPaintable(new Lampe());
        }
        else
        {
            painter.clear();
        }

    }
}