Rechteck zeichnen mit Canvas

[JAVAsk]

Ich habe ein kleines Programm geschrieben, welches durch Eingabe der x- & y-Koordinate sowie der Breite und Höhe, ein entsprechendes Rechteck zeichnet.
Dazu die 2 Klassen:

import javax.swing.*;
import java.awt.*;
 
public class MyGUI {
public static void main(String[] args){
	JFrame myFrame = new JFrame("My GUI");
	JPanel keyDisplayPanel = new JPanel();
	MyCanvas myCanvas = new MyCanvas();
  
    myFrame.setLayout(new BorderLayout(20,20));
    keyDisplayPanel.setLayout(new BorderLayout(20,20));
     
    keyDisplayPanel.add(myCanvas, BorderLayout.CENTER);
    myFrame.add(keyDisplayPanel, BorderLayout.CENTER);
  
	myFrame.setSize(500,500);
    myFrame.setVisible(true);
  }
 
}

import java.awt.*;
 
public class MyCanvas extends Canvas {
  public void paint(final Graphics g){
   
	System.out.println("Geben Sie bitte die x-Koordinate ein:");
    int x= Integer.parseInt(System.console().readLine());
	
	System.out.println("Geben Sie bitte die y-Koordinate ein:");
    int y= Integer.parseInt(System.console().readLine());
	
	System.out.println("Geben Sie bitte die Breite ein:");
    int width= Integer.parseInt(System.console().readLine());
	
	System.out.println("Geben Sie bitte die Höhe ein:");
    int height= Integer.parseInt(System.console().readLine());
	
	if(x>=0 && y>=0 && width>=0 && height>=0){	
	super.paint(g);
	g.drawRect(x, y, width, height);
	}
	else{ 
	System.out.println("Bitte keine negativen Werte eingeben!");
	}
	
  }
}

Das Programm zeichnet zwar das Rechteck, aber er fragt mich jeweils 2-Mal nach den Werten(s. oben). D.h., er zeichnet das kurz und löscht es gleichzeitig, beim zweiten Zeichnen bleibt es jedoch da. Was muss ich jetzt ändern?

Vielen Dank!???:L

 

[qwert]

Probier's mal damit, einen Konstruktor in der Klasse "MyCanvas" zu erstellen, und die Ausgaben und Einlesevorgänge dahin zu verschieben.

 

[JAVAsk]

So habe ich es auch ausprobiert. Funktioniert leider nicht. Bitte um Hilfe! Vielen Dank!

 

[Harry Kane]

"Funktioniert nicht" ist keine Fehlermeldung, schon gar nicht ohne Code. Du kannst übrigens nicht bestimmen, wie oft die paint-Methode aufgerufen wird. Unter Umständen kann dies sehr häufig der Fall sein. Aus diesem grund sollte man dort auch nur zeichnen, und keinesfalls Benutzereingaben anfordern.

 

[Flown]

Dein Ansatz muss anders laufen. Du fügst von außen ein neues Rechteck hinzu und lässt die pane repainten.

Spoiler: code

import java.awt.Canvas;
import java.awt.Graphics;
import java.util.ArrayList;
import java.util.List;
import java.util.Scanner;

import javax.swing.JFrame;

public class Test {
  public static void main(String... args) {
    new Test();
  }
  
  public Test() {
    JFrame frame = new JFrame("MyCanvas");
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setLocationByPlatform(true);
    MyCanvas pane = new MyCanvas();
    frame.add(pane);
    frame.setSize(800, 600);
    frame.setVisible(true);
    
    try (Scanner scanner = new Scanner(System.in)) {
      while (true) {
        int x, y, width, height;
        System.out.print("x: ");
        x = scanner.nextInt();
        System.out.print("y: ");
        y = scanner.nextInt();
        System.out.print("width: ");
        width = scanner.nextInt();
        System.out.print("height: ");
        height = scanner.nextInt();
        pane.addRectangle(new MyRectangle(x, y, width, height));
      }
    }
  }
  
  class MyRectangle {
    private int x, y, width, height;
    
    public MyRectangle(int x, int y, int width, int height) {
      this.x = x;
      this.y = y;
      this.width = width;
      this.height = height;
    }
    
    public int getX() {
      return x;
    }
    
    public void setX(int x) {
      this.x = x;
    }
    
    public int getY() {
      return y;
    }
    
    public void setY(int y) {
      this.y = y;
    }
    
    public int getWidth() {
      return width;
    }
    
    public void setWidth(int width) {
      this.width = width;
    }
    
    public int getHeight() {
      return height;
    }
    
    public void setHeight(int height) {
      this.height = height;
    }
  }
  
  class MyCanvas extends Canvas {
    private static final long serialVersionUID = 1L;
    
    List<MyRectangle> rectangles;
    
    public MyCanvas() {
      rectangles = new ArrayList<>();
    }
    
    public void addRectangle(MyRectangle rect) {
      rectangles.add(rect);
      repaint();
    }
    
    @Override
    public void paint(Graphics g) {
      super.paint(g);
      for (MyRectangle rect : rectangles) {
        g.drawRect(rect.getX(), rect.getY(), rect.getWidth(), rect.getHeight());
      }
    }
  }
}

 

[JAVAsk]

Ok..Das Meiste habe ich verstanden.
Könntest du mir bitte die Zeilen erklären, bei denen ich "???" als Kommentar geschrieben habe und überprüfen, ob die restlichen Kommentare, die ich geschrieben habe, stimmen.

    import java.util.Scanner;
    import javax.swing.JFrame;
     
public class Test{
    public static void main(String... args) {
    new Test();
    }
     
    public Test() {
    JFrame frame = new JFrame("Zeichne dein ");
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); // sorgt dafür, dass beim Schließen des Fensters keine Exceptions ausgegeben werden
    frame.setLocationByPlatform(true); // ???
    MeinCanvas pane = new MeinCanvas();
    frame.add(pane);
    frame.setSize(700, 700);
    frame.setVisible(true);
     
    try (Scanner scanner = new Scanner(System.in)) {
    while (true) {
    int x, y, width, height;
    System.out.print("x-Koordinate: ");
    x = scanner.nextInt(); // macht das Selbe wie --> x = Integer.parseInt(System.console().readLine());
    System.out.print("y-Koordinate: ");
    y = scanner.nextInt();
    System.out.print("Breite: ");
    width = scanner.nextInt();
    System.out.print("Höhe: ");
    height = scanner.nextInt();
    pane.addRectangle(new MyRectangle(x, y, width, height));
    }
    }
    }
    
    }

	import java.awt.Canvas;
    import java.awt.Graphics;
    import java.util.ArrayList;
    import java.util.List;

class MeinCanvas extends Canvas {
    private static final long serialVersionUID = 1L; // ???
     
    List<MyRectangle> rectangles;
     
    public MeinCanvas() {
    rectangles = new ArrayList<>();
    }
     
    public void addRectangle(MyRectangle rect) { // Die Methode fügt rekursiv weitere Rechtecke hinzu
    rectangles.add(rect);
    repaint();
    }
     
    @Override // ???
    public void paint(Graphics g) {
    super.paint(g);
    for (MyRectangle rect : rectangles) { // ??? (Bedingung)
    g.drawRect(rect.getX(), rect.getY(), rect.getWidth(), rect.getHeight());
    }
    }
    
    }

class MyRectangle{
    private int x, y, width, height;
     
    public MyRectangle(int x, int y, int width, int height) {
    this.x = x;
    this.y = y;
    this.width = width;
    this.height = height;
    }
     
    public int getX() {
    return x;
    }
     
    public void setX(int x) {
    this.x = x;
    }
     
    public int getY() {
    return y;
    }
     
    public void setY(int y) {
    this.y = y;
    }
     
    public int getWidth() {
    return width;
    }
     
    public void setWidth(int width) {
    this.width = width;
    }
     
    public int getHeight() {
    return height;
    }
     
    public void setHeight(int height) {
    this.height = height;
    }
    }

Vielen Dank!

 

[Flown]

Achja falls du Fragen zu den einzelnen Sachen hast, dann sieh dir bitte mal die Dokumentation der Java API an und Google ist dein Freund.

frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); //Beendet das Javaprogramm beim drücken auf das x des Fensters
frame.setLocationByPlatform(true); //Das Betriebssystem setzt dir das Fenster hin wo es das haben will (sagt eigentlich schon der Name!)
x = scanner.nextInt(); // macht das Selbe wie --> x = Integer.parseInt(System.console().readLine()); -- nein (aber in diesem Kontext ja)
private static final long serialVersionUID = 1L; //Benötigt man zum De-/Serialisieren -> lies nach wenn du näheres wissen willst!
public void addRectangle(MyRectangle rect) { ... }// Die Methode fügt rekursiv weitere Rechtecke hinzu -> Nein! sie fügt nur ein rect in das private Listenfeld der Instanz hinzu
@Override // Zeigt an das eine Methode der Superklasse überschrieben wird (ist optional aber IDEs fügen es zur besseren Lesbarkeit hinzu)
for (MyRectangle rect : rectangles) { //Das nennt sich eine for-each Schleife und kannst genügend im Internet dazu lesen

 

[JAVAsk]

Vielen Dank für deine Mühe! Jetzt habe ich alles verstanden!