Frame mit Rechteck!

[Zitrus]

Hallo ich möchte simpel einen Frame erstellen und ein Rechteck draufmalen.
Habe nun zwei Klassen erstellt.

Klasse 1 Fenster

package snakegame;

import java.awt.Color;
import java.awt.Graphics;

import javax.swing.*;

public class Fenster extends JFrame {
	int x = 50;
    int y = 50;
    int width = 20;
    int height = 10;
	
	public static void main(String[] args) {
	    Fenster fenster = new Fenster();
	    Schlange s1 = new Schlange();
	  }
	
	public Fenster() {
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        getContentPane().setBackground(Color.WHITE);
        setTitle("SnakeGamev0.1");
        setSize(400,300);
        setVisible(true);
	}
}

Klasse 2 Schlange

package snakegame;

import java.awt.Color;
import java.awt.Graphics;

public class Schlange {
	int x = 50;
    int y = 50;
    int width = 20;
    int height = 10;
    
	public Schlange() {
		  
	}
	public void paint(Graphics g) {
	    g.setColor(Color.RED);
        g.fillRect(x,y,width,height);
	}
}

So nun zu meiner Frage.. Wieso wird das Rechteck nicht angezeigt?
Mfg, Zitrus

 

[Harry Kane]

Weil du zwar eine Instanz der Klasse Schlange erstellst, aber ihre paint-methode nicht aufrufst.

 

[Zitrus]

Hallo Harry Kane,
Vielen Dank für die Antwort.. Ich weiß aber nicht unter welchen Parametern ich die paint-methode aufrufen muss.
Es heißt ja public void paint(Graphics g). Was muss ich dann in die Klammern reinschreiben?

Bzw. wie rufe ich sie auf?

 

[Java20134]

Du musst folgendes hinzufügen:

s1.paint(g);
Name des Objekts von Schlange.paint(g);//allgemein

Und davor musst du noch ein Graphics namens g erstellen.

Du kannst zum Beispiel auch noch den Konstruktor einbinden:

int x, y, width, height; 

public Schlange(int x, int y, int width, int height){
           this.x = x; 
           this.y = y;
           this.width = width;
           this.height = height; 
}

 

[Harry Kane]

Leite Schlange von JComponent ab, und nenne deine paint-Methode in paintComponent um. Dann kümmert sich nämlich Java darum, die Methode zu gegebener Zeit aufzurufen, und auch einen passenden Graphics zur Verfügung zu stellen.

 

[Zitrus]

package snakegame;
 
import java.awt.Color;
import java.awt.Component;
import java.awt.Graphics;
 

import javax.swing.*;
 
public class Fenster extends JFrame {
	int x = 50;
    int y = 50;
    int width = 20;
    int height = 10;
 
	public static void main(String[] args) {
	    Fenster fenster = new Fenster();
	    Schlange s1 = new Schlange();
	  }
 
	public Fenster() {
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        getContentPane().setBackground(Color.WHITE);
        setTitle("SnakeGamev0.1");
        setSize(400,300);
        setVisible(true);
	}
}

package snakegame;
 
import java.awt.Color;
import java.awt.Graphics;

import javax.swing.JComponent;
 
public class Schlange {
	int x;
    int y;
    int width;
    int height;
    Graphics g;
 
	public Schlange() {
		x = 50;
		y = 50;
		width = 50;
		height = 50;
	}
	
	public void paint(Graphics g) {
	    g.setColor(Color.RED);
        g.fillRect(x,y,width,height);
        paint(g);
	}
}

Wo ist der Fehler?

 

[Harry Kane]

Wo ist der Fehler?

Du hast beide Ratschläge von mir nicht umgesetzt.
Und so, wie du die paint(Graphics) Methode umgesetzt hast, ruft sie sich selber auf, was zu einer Endlosschleife führt.
Eins habe ich noch übersehen: Wenn Schlange JComponent erweitert (und damit zu einer JComponent wird), muss sie dem JFrame bzw. dessen content pane noch hinzugefügt werden.

 

[VfL_Freak]

Moin,

Wo ist der Fehler?

WELCHER Fehler ??? :noe:

Isses denn so schwer ???
Gruß
Klaus

 

[Zitrus]

Irendwie schon^^ Warum klappts nicht?

package snakegame;
 
import java.awt.Color;
import java.awt.Component;
import java.awt.Graphics;
 

import javax.swing.*;
 
public class Fenster extends JFrame {
	int x = 50;
    int y = 50;
    int width = 20;
    int height = 10;
 
	public static void main(String[] args) {
	    Fenster fenster = new Fenster();
	    Schlange s1 = new Schlange();
	  }
 
	public Fenster() {
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        getContentPane().setBackground(Color.WHITE);
        setTitle("SnakeGamev0.1");
        setSize(400,300);
        setVisible(true);
	}
}

package snakegame;
 
import java.awt.Color;
import java.awt.Graphics;

import javax.swing.JComponent;
 
public class Schlange extends JComponent {
	int x,y,width,height;
    Graphics g;
 
	public Schlange() {
		x = 50;
		y = 50;
		width = 20;
		height = 10;
		paintComponent(g);
	}
	
	public void paintComponent(Graphics g) {
	    g.setColor(Color.RED);
        g.fillRect(x,y,width,height);
	}
}

 

[Harry Kane]

Ist dir die NullPointerException aufgefallen?

 

[VfL_Freak]

Irendwie schon^^ Warum klappts nicht?

ich meinte eigentlich, ob es wirklich so schwer ist zu beschreiben, was nicht klappt und ggf. auftretende Fehlermeldungen zu posten ?????

Wir machen hier keine Ratespiele !!

Gruß
Klaus

 

[Zitrus]

@ Harry Kane ja ich bekomme die NullPointerException

Exception in thread "main" java.lang.NullPointerException
	at snakegame.Schlange.paintComponent(Schlange.java:21)
	at snakegame.Schlange.<init>(Schlange.java:17)
	at snakegame.Fenster.main(Fenster.java:18)

@Vfl_Freak entschuldige mein Verhalten. Ich werde versuchen nächstes Mal sofort die Fehlermeldung bzw. den Fehler an sich zu beschreiben.

Fehler:

Beim Ausführen erscheint die NullPointerException und naja es lässt sich nicht ausführen (Fehlermeldung oben).

 

[Harry Kane]

Kannst du dir vorstellen, wo die Exception herkommt?
Sorry, wenn du dich etwas in die Mangel genommem fühlst, aber glaube mir, du wirst davon mehr profitieren als von einem fertigen Code.

 

[Zitrus]

Vllt. wegen falschen Parametern?
Vielen Dank für deine Hilfe ich weiß das wirklich zu schätzen!

 

[Harry Kane]

Vllt. wegen falschen Parametern?

Geht es etwas genauer?

 

[Zitrus]

Ich denke paintComponent(g) klappt nicht. Bin mir aber nicht ganz sicher^^

 

[Harry Kane]

Ok, du hast gewonnen. Schluss mit der Salamitaktik.
Du rufst im Konstruktor von Schlange die paintComponent-Methode auf und verwendest dabei die Instanzvariable g als Parameter. Diese ist aber null. Deshalb fliegt beim Aufruf von g.setColor() die NPE.
Führe bitte folgendes aus:
Streiche den o.g. Methodenaufruf aus dem Konstruktor von Schlange
Initialisiere "Schlange" nicht in der main-Methode von deiner Fenster-Klasse, sondern in deren Konstruktor, in dem du folgende Zeile vor die Zeile mit dem setVisible() schreibst:
getContentPane().add(new Schlange());