ContentPane wird nicht angezeigt!

[T_Wolf]

Liebe Community,
ich habe folgendes Problem:
Ich will ein Graph mit Messwerten Zeichnen und habe dazu ein Programm versucht zu schreiben. Eigentlich müsse der Graph (bisher aus Testwerten) angezeigt werden doch ich erhalte nur ein leeres Fenster.
Ich gehen einfach mal davon aus dass es sich um einen recht trivialen Fehler handelt und wollte fragen ob mir jemand den Fehler aufzeigen könnte.

Gruss Thomas

Code :

import java.awt.Container;

import javax.swing.*;


public class LinienDiagramm extends JFrame{

	Container c;
	Zeichenbrett z;
	int [] xKoordinaten = {0,1,2,3,4,5,6,7,8,9};
	int [] yKoordinaten = {0,1,2,3,4,5,6,7,8,9};
	int Anzahlkoordinaten =10;	
	
	public void LinienDiagramm() {
		c = getContentPane();
		z = new Zeichenbrett(xKoordinaten,yKoordinaten,Anzahlkoordinaten);
		c.add(z);
	}
		public static void main(String[] args) {
		LinienDiagramm fenster = new LinienDiagramm();
		fenster.setTitle("Graph");
		fenster.setSize(400,400);
		fenster.setVisible(true);
		fenster.repaint();
		fenster.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	}
}

import java.awt.Graphics;
import javax.swing.*;

public class Zeichenbrett extends JPanel {
	private int[] x, y; // Koordinaten 
	private int n; // Anzahl Koordinaten
	public Zeichenbrett(int x[], int y[], int n) {
	}
	public void paintComponent(Graphics g) {
		g.drawOval(40, 30, 20, 20);
		g.drawLine(0, 0, 200, 200);
	
	/*	for(int i=1;i<n+1;i++){
		g.drawLine(x[i-1]-200, y[i-1], x[i]-200, y[i]);
		}*/
	}
}

im Unteren Teil habe ich aktuell versucht einfach Irgendwas zu Zeichnen und den eigentlichen Code auskommentiert.

 

[icarus2]

Nimmm mal bei public void LineienDiagramm() das void weg, dann rufst du den Konstruktor der Klasse auf.

Ausserdem braucht es das fenster.repaint() nicht.


Edit:
Hier noch der veränderte Code, sollte so klappen

[Java]
public class LinienDiagramm extends JFrame{

Container c;
Zeichenbrett z;
int [] xKoordinaten = {0,1,2,3,4,5,6,7,8,9};
int [] yKoordinaten = {0,1,2,3,4,5,6,7,8,9};
int Anzahlkoordinaten =10;

public LinienDiagramm() {
c = getContentPane();
z = new Zeichenbrett(xKoordinaten,yKoordinaten,Anzahlkoordinaten);
c.add(z);
}
public static void main(String[] args) {
LinienDiagramm fenster = new LinienDiagramm();
fenster.setTitle("Graph");
fenster.setSize(400,400);
fenster.setVisible(true);
fenster.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
}
}
[/Java]

Edit 2:

Variabeln und Objekte schreibt man im camelCase. Sprich nicht "Anzahlkoordinaten", sonderen "anzahlKoordinaten".

 

[T_Wolf]

Es funktioniert!
Vielen Dank

Variabeln und Objekte schreibt man im camelCase. Sprich nicht "Anzahlkoordinaten", sonderen "anzahlKoordinaten".

Ok das werde ich dann auch mal ändern.
Danke

 

[icarus2]

Noch etwas kleines... in paintComponent(Graphics g) ruft man normalerweise zuerst noch super.paintComponent() auf, das erledigt alle "Aufräumarbeiten" bei einem repaint().

Also so:

[Java]
public void paintComponent(Graphics g) {

super.paintComponent(g);

g.drawOval(40, 30, 20, 20);
g.drawLine(0, 0, 200, 200);

}
[/Java]

 

[Gastredner]

Zuallererst einmal musst du im Konstruktor die übergebenen Werte deinen Variablen zuweisen:

public Zeichenbrett(int x[], int y[], int n) {
		this.x = x;
		this.y = y;
		this.n = n;
}

Das n kannst du übrigens weglassen und stattdessen direkt die Länge der Arrays abfragen:

x.length

Außerdem ist deine for-Schleife sehr verwirrend aufgebaut - besser wäre es, du initialisierst i mit 0 und lässt es laufen bis i<n oder i<=n-1. Dann sparst du dir es auch, immer noch i+1 in der Schleife rechnen zu müssen.

Nichts gezeichnet wird deswegen, weil du keinen Konstruktor in der Klasse LinienDiagramm hast - nur eine Methode ohne Rückgabewert. Entferne das "void", dann hast du einen Konstruktor und es erscheinen auch Linien.

EDIT: Tja, etwas zu lang gebraucht...