Probleme mit Zeichnen

[muemmel_0811]

Hallo zusammen,

ich bin noch Java-Anfängerin und völlig am verzweifeln...
Habe bisher ein kleines Programm, dass doch einfach nur auf Mausklick einen Kreis in ein JPanel zeichnen soll.
Eigentlich tut es dass ja auch schon, nur noch nicht so ganz, wie ich das gerne hätte. Denn direkt beim Programmstart wird ein Kreis in das JPanel gezeichnet, ohne dass irgendwo hingeklickt wurde und außerdem kann ich bisher immer nur einen Kreis in das JPanel zeichnen, obwohl ich doch mehrere zeichnen will.
Folgenden Code hab ich bisher:

public class Chordz extends JFrame {
   FretBoard rahmen;
   int mouse_x, mouse_y;

   public Chordz() {
      super();
      setTitle("ChordZ");
      setLayout(new FlowLayout());
      setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      rahmen = new FretBoard();
      getContentPane().add(rahmen);
   }
   class FretBoard extends JPanel {
   	FretBoard() {
         addMouseListener(new Maus());
         setBackground(Color.WHITE);
         setPreferredSize(new Dimension(160,200));
      }
      class Maus extends MouseAdapter {
      	public void mousePressed(MouseEvent e) {
         	mouse_x = e.getX();
            mouse_y = e.getY();
            repaint();
         }
      }
      public void update(Graphics g) {
         paintComponent(g);
      }
      public void paintComponent(Graphics g) {
      	super.paintComponent(g);
         g.fillOval(mouse_x,mouse_y,20,20);
      }
   }
   public static void main(String[] args) {
   	JFrame fenster = new Chordz();
      fenster.setSize(350,275);
      fenster.setLocation(200,300);
      fenster.setVisible(true);
      fenster.setResizable(false);
   }
}

Könnt Ihr mir bitte verraten, was ich falsch gemacht habe und wie ich es richtig machen muss?

Grüße,
muemmel

 

[Leroy42]

Das Problem ist, das dir die Abfolge der GUI-Zeichnungen noch
nicht in Fleisch und Blut übergegangen ist.

1. Du selbst zeichnest nicht, sondern die GUI läßt
zeichnen, sobald sie meint das es notwendig ist oder
wenn du es via repaint() anordnest. Z.B. auch wenn
das Frame das erste Mal sichtbar geworden ist ohne daß du
bereits geklickt hast.

2. Das Zeichnen findet nicht nacheinander statt, sondern
jedesmal von Neuem ==> Die alten Zeichnungen gehen verloren.

Ansatz:
Du mußt dir alle zu zeichnenden Objekte merken, am besten in
einem Vector oder einer ArrayList. Ein Array kannst du nicht
benutzen, da du ja vor Programmstart nicht weißt, wieviele
Kreise später geklicked werden.

Vector kreise = new Vector();

public void paintComponent(Graphics g) {
  super.paintComponent(g);
  for (int i=0; i < kreise.size(); ++i) {
    Point mittelpunkt = (Point) kreise.get(i);
    g.fillOval(mittelpunkt.x, mittelpunkt.y,20,20);
  }
}

public void mousePressed(MouseEvent e) {
  kreise.add(new Point(e.getX(), e.getY()));
  repaint();
}

 

[muemmel_0811]

Hallo Leroy42,

ich kriegs nicht gebacken...
Deine Erklärung kann ich noch verstehen (Danke, das hat schon etwas geholfen), aber wehe, ich fange an, den Code in meinen zu packen und dann das file zu kompilieren.
Was hab ich hier falsch gemacht bzw., was hab ich vergessen, denn beim Kompilieren bekomm ich die Fehlermeldung, "cannot find symbol
symbol class Vector"
???

import java.util.Vector.*;

public class Chordz extends JFrame {
	FretBoard rahmen;
        Vector kreise = new Vector();

	public Chordz() {
   	   super();
           setTitle("ChordZ");
           setLayout(new FlowLayout());
   	   setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
           rahmen = new FretBoard();
           getContentPane().add(rahmen);
         }
   class FretBoard extends JPanel {
   	 FretBoard() {
         addMouseListener(new Maus());
         setBackground(Color.WHITE);
         setPreferredSize(new Dimension(160,200));
      }
      class Maus extends MouseAdapter {
      	public void mousePressed(MouseEvent e) {
            kreise.add(new Point(e.getX(), e.getY()));
            repaint();
         }
      }
      public void update(Graphics g) {
         paintComponent(g);
      }
      public void paintComponent(Graphics g) {
      	super.paintComponent(g);
         for (int i=0; i < kreise.size(); ++i) {
             Point mittelpunkt = (Point) kreise.get(i);
             g.fillOval(mittelpunkt.x, mittelpunkt.y,20,20);
         }
      }
   }
   public static void main(String[] args) {
      JFrame fenster = new Chordz();
      fenster.setSize(350,275);
      fenster.setLocation(200,300);
      fenster.setVisible(true);
      fenster.setResizable(false);
   }

}

Bitte habt Geduld mit mir, denn ich stehe noch am Anfang meiner Java-Karriere.

Grüße,
muemmel

 

[Leroy42]

import java.util.Vector.*;

Hier liegt der Hund begraben:

Du kannst entweder Klassen gezielt importieren

import java.util.Vector;

oder ganze Pakete, d.h. alle Klassen aus einem package.

import java.util.*;

Was du geschrieben hast war der Versuch alle Klassen aus einem package
java.util.Vector zu importieren. Dieses package gibt es aber nicht.

Kleiner Fehler (Ein Punkt zuviel) und große Wirkung

 

[Leroy42]

Ich hab dein Programm kompiliert und es läuft soweit schon mal
ganz nett.

Ich mußte allerdings noch ein paar imports einfügen

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

 

[muemmel_0811]

Hallo Leroy42,

erstmal vielen, vielen Dank für Deine Geduld und Mühe!!!
Ich hatte ja auch schon

import java.util.*;

verwendet (die anderen imports hatte ich mir hier im Post gespart...), allerdings hatte ich eine Fehlermeldung beim Kompilieren erhalten, die ich nicht wirklich gelesen habe und das war folgende:

Note: Chordz.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details

Nachdem ich dann nach Lesen Deines Posts und entsprechender Anpassung meines Codes das ganze dann auch mit -Xlint kompiliert hab, gings auch. Jetzt frag ich nur, was mir diese Fehlermeldung denn sagen wollte, bzw. was ich vom Kompilieren mit Xlint habe?

Wärst Du noch mal so nett, mir das zu erklären?

Grüße,
muemmel[/code]

 

[Leroy42]

Das sagt mir, daß du das JDK1.5 benutzst.

Da sollte man ab 1.5 nicht mehr.

Besser müßtest du schreiben

Vector<Point> = new Vector<Point>();

und später dann einfach

Point mittelpunkt = kreise.get(i);

ohne (Point)

Ich wollte dir nur die etwas verwirrende neue Syntax ersparen.

Aber wenn du es dir zutraust, benutze ruhig die sicheren Version ohne Warnungen.

 

[muemmel_0811]

Hallo Leroy42,

vielen, vielen Dank - jetzt geht's - Du hast mir wahnsinnig viel geholfen!!! :applaus:

Grüße,
muemmel