Methoden Objekte mit Hilfe einer Schleife in einer ArrayList erzeugen

[Chevo]

Hey Leute,
bin noch neu hier um Forum und noch sehr unerfahren mit Java und hab mir deswegen mal gedacht ob ihr mir bei meinem Problem helfen könnt.
Folgendes:
Ich bin gerade dabei ein Projekt zu erstellen in dem man Lotto spielen kann. Nun wollte ich dies mit Hilfe einer ArrayList machen, in dem ich die Objekte mit Hilfe einer Schleife erzeuge. Das Problem ist aber, dass ich keine Ahnung habe, wie ich das machen soll.
Könnt ihr ihr mir vllt helfen wie die Schleife aussehen soll?
Mein Ansatz wäre dieser gewesen, weiter weiß ich aber grade nicht, wäre klasse wenn mir das mal einer erklären könnte

private ArrayList<Kugel> kugelliste = new ArrayList<Kugel>();

public void Kugelliste()
	{
		for (int i = 0; i < kugelliste.size(); i++)
		{


		}
	}

Hier nochmal meine beiden anderen Klassen als weitere Anhaltspunkte:

package paket_Muendlich2;

import java.util.ArrayList;

public class Urne
{
	private String bezeichnung;
	private ArrayList<Kugel>kugelliste;
	
	public Urne()
	{
		kugelliste = new ArrayList<Kugel>();
	}
	
	public void setBezeichnung(String pBezeichnung)
	{
		this.bezeichnung = pBezeichnung;
	}
	
	public String getBezeichnung()
	{
		return bezeichnung;
	}

	public void kugelhinzufuegen(Kugel pZahl)
	{
		kugelliste.add(pZahl);
	}
	
	
	public int getGroesse()
	{
		return kugelliste.size();
	}
	

	
}

package paket_Muendlich2;

public class Kugel
{
	private int zahl;
	private Urne meineUrne;
	
	public Kugel(int pZahl)
	{
		zahl = pZahl;
	}
	
	
	public void setZahl(int pZahl)
	{
		zahl = pZahl;
	}
	
	public int getZahl()
	{
		return zahl;
	}
	
	public void setUrne(Urne pUrne)
	{
		meineUrne = pUrne;
	}
	
	public Urne getUrne()
	{
		return meineUrne; 
	}

	
}

Danke schon mal im vorraus, wäre sehr nett wenn ihr mir helfen könntet

 

[Atze]

hi,

momentan iterierst du in deinem vorschlag für die schleife über die neu erzeugte und somit leere kugelliste.

deinen klassen nach müsstest du doch eigentlich nur eine schleife von 1-49 durchlaufen und mit dem jeweiligen schleifenindex ein Kugel-objekt erzeugen und in diese kugelliste packen.

oder?

 

[Fab1]

private ArrayList<Kugel> kugelliste = new ArrayList<Kugel>();
 
public void Kugelliste()
    {
        for (int i = 0; i < kugelliste.size(); i++)
        {
          kugelliste.add(new Kugel(1));
 
        }
    }

Grundsätzlich würde das gehen. Allerdings musst du schauen, denn kugelliste.size ist natürlich 0 wenn kein Eintrag drin ist und somit würde die Schleife nicht durchlaufen werden.

Und du musst schauen, welchem int Parameter du übergeben möchtest, ich hab einfach mal 1 genommen

 

[Chevo]

hi,

momentan iterierst du in deinem vorschlag für die schleife über die neu erzeugte und somit leere kugelliste.

deinen klassen nach müsstest du doch eigentlich nur eine schleife von 1-49 durchlaufen und mit dem jeweiligen schleifenindex ein Kugel-objekt erzeugen und in diese kugelliste packen.

oder?

Ja genau, aber nur leider komm ich irgendwie nicht drauf wie ich des anstellen soll -.- Bin leider noch unerfahren was Schleifen und ArrayLists angeht.

 

[Atze]

private ArrayList<Kugel> kugelliste = new ArrayList<Kugel>();
 
public void Kugelliste()
    {
        for (int i = 1; i <= 49; i++)
        {
          kugelliste.add(new Kugel(i));
        }
    }

ich dachte an sowas

 

[Chevo]

private ArrayList<Kugel> kugelliste = new ArrayList<Kugel>();
 
public void Kugelliste()
    {
        for (int i = 1; i <= 49; i++)
        {
          kugelliste.add(new Kugel(i));
        }
    }

ich dachte an sowas

Ahhh, auf sowas wär ich glaub ich niemals gekommen Besten Dank, das hat mein Problem endlich gelöst

Edit: Wie kann ich mit Hilfe der daraus erstellten Objekte, zufällig davon 6 Objekte entnehmen? Ich habe da an eine weiter ArrayList gedacht, die mit Hilfe von MathRandom() auf die erste ArrayList zugreift. Das Problem ist aber, dass dadurch theoretisch die Zahl doppelt ausgegeben würde.

Mein Ansatz würde so aussehen, aber der wäre ja nicht richtig. Zusätzlich möchte ich das ganze in einer JList ausgeben, aber die Methode list.add() gibt mir ne Fehlermeldung wenn ich list.add(kugelliste); eingeben würde.(The method add(Component) in the type Container is not applicable for the arguments (ArrayList<Kugel>)).
Außerdem glaube ich, das ich die for bedinung ändern sollte.

	public void Lottozahlen()
	{
		int a;
		for (int i = 1; i <= 49; i++) //vllt eher for(int i = i; i <=6;) ?
		{
			a = (int) (Math.random()) * kugelliste.size();
		}
		list.add(kugelliste);

	}

 

[Chevo]

Ahhh, auf sowas wär ich glaub ich niemals gekommen Besten Dank, das hat mein Problem endlich gelöst

Edit: Wie kann ich mit Hilfe der daraus erstellten Objekte, zufällig davon 6 Objekte entnehmen? Ich habe da an eine weiter ArrayList gedacht, die mit Hilfe von MathRandom() auf die erste ArrayList zugreift. Das Problem ist aber, dass dadurch theoretisch die Zahl doppelt ausgegeben würde.

Mein Ansatz würde so aussehen, aber der wäre ja nicht richtig. Zusätzlich möchte ich das ganze in einer JList ausgeben, aber die Methode list.add() gibt mir ne Fehlermeldung wenn ich list.add(kugelliste); eingeben würde.(The method add(Component) in the type Container is not applicable for the arguments (ArrayList<Kugel>)).
Außerdem glaube ich, das ich die for bedinung ändern sollte.

	public void Lottozahlen()
	{
		int a;
		for (int i = 1; i <= 49; i++) //vllt eher for(int i = i; i <=6;) ?
		{
			a = (int) (Math.random()) * kugelliste.size();
		}
		list.add(kugelliste);

	}

Keiner ein Lösungsvorschlag dazu?

 

[Atze]

nicht ungeduldig werden, ist ja kein chat, sondern ein forum

achso, ich dachte du willst erstmal alle 49 kugeln. du willst also nur die 6?

dann erzeuge dir doch so lange mit Random kugelobjekte und füge die in ein Set ein, bis das set die größe von 6 erreicht hat. dann hast du auch keine doppelten einträge

 

[Chevo]

nicht ungeduldig werden, ist ja kein chat, sondern ein forum

achso, ich dachte du willst erstmal alle 49 kugeln. du willst also nur die 6?

dann erzeuge dir doch so lange mit Random kugelobjekte und füge die in ein Set ein, bis das set die größe von 6 erreicht hat. dann hast du auch keine doppelten einträge

Hmm, das verstehe ich jetzt nicht ganz... Wie soll das denn dann am Ende aussehen?

 

[Marcinek]

Implementiere deinen eigenen Vorschlag und Remote die benutze Zahlamts dem Array.

Der neue randomwert muss dann zwischen 0 und 49-1 liegen.

 

[Chevo]

Implementiere deinen eigenen Vorschlag und Remote die benutze Zahlamts dem Array.

Der neue randomwert muss dann zwischen 0 und 49-1 liegen.

was bitteschön bedeutet Remote? Der Begriff ist mir jetzt neu


Habs mal bissle abgeändert, scheint für mich jetzt auf den ersten Blick richtig zu sein, dummerweise habe ich das Problem das ich es iwie nicht ausgeben kann um es zu überprüfen :/ Wäre nicht schlecht wenn ihr mal ein Blick drauf werfen könntet und mir gegebenfalls mal meinen Fehler beleuchtet Vorallem dankbar wäre ich dann auch noch für das Problem mit der Ausgabe Noch besser wäre es dann, mir bei dem JList Problem zu helfen, aber erst mal Schritt für Schritt.

		  for (int i = 0; i <= 6; i++) 
		  { 
			  int lottozahl =(int)(1+Math.random()*49); 
		  }

Ach ja, ich mach es deswegen doppelt, weil ich die Methode mit Hilfe eines Buttons in meiner GUI durchführen lassen möchte, deswegen scheint es auch auf den ersten Blick so, dass ich es unnötiger Weise doppelt mache, aber das war nur so mein Gedanke, ihr könnt mich auch eines besseren belehren

 

[HimBromBeere]

Deine Idee mit der Ziehung der sechs ist schonmal nicht völlig daneben. Die for-Schleife geht demnach 6-mal durch, auch OK (abgesehene von dem Sonderfall, dass du zufälligerweise zweimal die gleiche Zahl gezigen hast, was ja eigtl. nicht gehen sollte, weshalb du dann natürlich einen weiteren Schleifendurchlauf machen müsstest). Aber du kannst in deine zweite Liste doch gleich innerhalb der Schleife das gerade berechnte Element einfügen und nicht am Schluss die gesamte Liste Kugeln:

public void Lottozahlen()
    {
        int a;
        for (int i = 1; i <= 6; i++) //vllt eher for(int i = i; i <=6;) ?
        {
            a = (int) (Math.random()) * kugelliste.size();
            list.add(kugelliste.get(a));
        }
        
    }

Zu deinem Problem mit der JList: deine Kugelliste ist, wie dir die Fehlermeldung wahrscheinlich bereits gesagt hat, keine JComponent. Schau dir mal die API für JList an, vlcht. findest du dort einen Konstruktor, der eine variable Anzahl versch. Objekte zulässt...

 

[faetzminator]

Eigentlich kannst du einfach [c]Collections.shuffle(kugelliste)[/c] aufrufen und danach die Elemente 0 bis n ausgeben, sofern die Liste nicht sortiert bleiben muss. Ansonsten kannst du immer noch eine Kopie erstellen ([c]List kopie = new ArrayList(kugelliste);[/c]).

 

[Chevo]

Deine Idee mit der Ziehung der sechs ist schonmal nicht völlig daneben. Die for-Schleife geht demnach 6-mal durch, auch OK (abgesehene von dem Sonderfall, dass du zufälligerweise zweimal die gleiche Zahl gezigen hast, was ja eigtl. nicht gehen sollte, weshalb du dann natürlich einen weiteren Schleifendurchlauf machen müsstest). Aber du kannst in deine zweite Liste doch gleich innerhalb der Schleife das gerade berechnte Element einfügen und nicht am Schluss die gesamte Liste Kugeln:

public void Lottozahlen()
    {
        int a;
        for (int i = 1; i <= 6; i++) //vllt eher for(int i = i; i <=6;) ?
        {
            a = (int) (Math.random()) * kugelliste.size();
            list.add(kugelliste.get(a));
        }
        
    }

Zu deinem Problem mit der JList: deine Kugelliste ist, wie dir die Fehlermeldung wahrscheinlich bereits gesagt hat, keine JComponent. Schau dir mal die API für JList an, vlcht. findest du dort einen Konstruktor, der eine variable Anzahl versch. Objekte zulässt...

Ja, bei deinem Ansatz taucht eben das Problem mit der add() Methode auf, da Eclipse mir ja das oben bereits erwähnte Problem anzeigt. Hab mir des mal durchgelesen, aber iwie nichts gefunden das zur Lösung meines Problems beitragen könnte. -.-

 

[ARadauer]

oben bereits erwähnte Problem anzeigt

welche?

Also dein problem ist, dass du nicht zwei gleiche Kugeln ziehen willst?
1. Entweder shuffle und dann immer die erste nehmen
2. oder so lange ziehen bis du eine hast die du noch nicht hast

 

[Chevo]

Hab mir jetzt mal gedacht das es so funktionieren sollte:

	public void Lottozahlen()
	{

		int a;
		for (int i = 0; i <= 6; i++)
		{
			a = (int) (1 + Math.random() * kugelliste.size());
			kugelliste.remove(a);
			list.add(); //hier komm ich nicht weiter -.-
		}

	}

Die Methode Kugelliste() hab ich mit kugelliste.removeAll(kugelliste); ergänzt, so das ich es vermeide doppelte Objekte in meiner ArrayListe zu erzeugen. Aber das war jetzt nur mal so ein Gedanke, ich weiß nicht ob es auch in der Praxis so klappt...
Mein Problem ist nur noch die Zeile mit list.add(); , wo ich die Objekte in eine Liste einfügen wollte um sie auszugeben, was aber nicht geht
Es wär nicht schlecht wenn man mir überhaupt erst mal zeigen könnte wie ich des ausgebe, damit ich überprüfen kann ob es funktioniert(also ein beispiel mit system.out.println() wär nicht schlecht ^^)

 

[HimBromBeere]

Deine ArrayLists kennen die Methoden

add()

und

remove()

. Letztere gibt es laut API in der Version

remove(int index)

und

remove(Object o)

. Im Moment verwendest du die erste, was aber ziemlich kontraproduktiv sein sollte, da die Indizes mit jedem remove oder add ja neu berechnet werden und deshalb nicht jede Kugel als Index den Wert hat, der auf ihr drauf steht.
Lange Rede, kurzer Sinn:
In

a

steckt nicht zwangsläufig deine tatsächliche Zahl drin, sondern die Stelle, an der die Zahl in deiner Liste steht. Also musst du in deine Liste auch nicht die Zahl a reicnstecken, sondern eben das Element deiner Kugelliste an dieser Stelle. Darum hab ich ja in meinem letzten Beitrag auch

list.add(kugelliste.get(a))

geschrieben...

 

[Chevo]

Deine ArrayLists kennen die Methoden

add()

und

remove()

. Letztere gibt es laut API in der Version

remove(int index)

und

remove(Object o)

. Im Moment verwendest du die erste, was aber ziemlich kontraproduktiv sein sollte, da die Indizes mit jedem remove oder add ja neu berechnet werden und deshalb nicht jede Kugel als Index den Wert hat, der auf ihr drauf steht.
Lange Rede, kurzer Sinn:
In

a

steckt nicht zwangsläufig deine tatsächliche Zahl drin, sondern die Stelle, an der die Zahl in deiner Liste steht. Also musst du in deine Liste auch nicht die Zahl a reicnstecken, sondern eben das Element deiner Kugelliste an dieser Stelle. Darum hab ich ja in meinem letzten Beitrag auch

list.add(kugelliste.get(a))

geschrieben...

Ahh jetzt hab ichs verstanden und weiß warum ich am Anfang eine Fehlermeldung bekommen habe. Meine ArrayList hatte den falschen Typ anfangs (String), aber damit

list.add(kugelliste.get(a))

funktioniert hätte ich

ArrayList<Kugel> lotto = new ArrayList<Kugel>();

schreiben müssen. Das Problem ist aber, wenn ich es mit dann mit

System.out.println(lotto);

ausgebe um es zu überprüfen wird mir folgendes ausgegben: [paket_Muendlich2.Kugel@193385d, paket_Muendlich2.Kugel@5973ea, paket_Muendlich2.Kugel@c9131c, paket_Muendlich2.Kugel@105b99f, paket_Muendlich2.Kugel@dda25b, paket_Muendlich2.Kugel@ce2187, paket_Muendlich2.Kugel@fb56b1]

Das heißt also, dass ich in der ArrayList wohl die falschen Objekte erzeugt habe oder?

 

[jgh]

probiere es mal mit

System.out.println(Arrays.toString(lotto));

PS:
falls noch nicht, musst du [c]import java.util.*;[/c] importieren

 

[HimBromBeere]

Meine ArrayList hatte den falschen Typ anfangs (String), aber damit list.add(kugelliste.get(a)) funktioniert hätte ich ArrayList<Kugel> lotto = new ArrayList<Kugel>(); schreiben müssen.

Richtig...

Damit dein Sysout auch funktioniert, musst du in der Klasse Kugel eine Methode toString implementieren. Sie sollte etwa so aussehen:

public String toString() {
     return zahl;
}

Dann wird die Zahl quasi als "Standardwert" verwendet, wann immer irgendwo eine String-Entsprechung deiner Instanz benötigt wird (also z.B. in deinem Sysout).

[EDIT]Achja, mit jgh´s Vorschlag dürfte´s aber einfacher gehen, da die Klasse Arrays ja bereits eine solche Methode besitzt...[/EDIT]

 

[faetzminator]

public String toString() {
     return String.valueOf(zahl);
}

Denn [c]zahl[/c] ist kein String

 

[Chevo]

Dankeeeeeeeee es hat endlich geklappt )))) Auch das Problem mit der List hab ich dank euch gelöst!

Meine einzigen Fragen wären nur noch wie ich die gezogenen Zahlen nach ihrer Größe anordne und wieso meine Methode in der Klasse Kugel

	public void kugelhinzufuegen(Kugel pZahl)
	{
		kugelliste.add(pZahl);
	}

in meiner Schleife nicht funktioniert?
Weile wenn ich es dann so mache:

private Urne urne;
private Zahl zahl;
public void Kugelliste()
	{
		kugelliste.clear();
		for (int i = 1; i <= 49; i++)
		{
			zahl = new Kugel(i);
			kugelliste.add(zahl); // hier klappts mit urne.kugelhinzufuegen(zahl) nicht -.-
		}

	}

Ihr seid echt klasse! Danke schon mal für alles wo ihr mir bisher geholfen habt

 

[HimBromBeere]

[...]wie ich die gezogenen Zahlen nach ihrer Größe anordne [...]

Zu Sortieralgorithmen findest du beim großen Freund mit G genug, andernfalls mal hier im Forum suchen, das findest du schon...

[...]und wieso meine Methode in der Klasse Kugel public void kugelhinzufuegen(Kugel pZahl) { kugelliste.add(pZahl); }
in meiner Schleife nicht funktioniert?

Ich tippe mal darauf, dass die Instanzvariable

urne

zu der Zeit noch gar nicht belegt (also

null

) ist.
Aber schreib doch mal bitte zu der Klasse, in der du diese Variable verwendest, den ganzen Code (Urne und Kugel sehe ich, aber da fehlt ja irgendwo noch eine dritte Klasse...). Warum gibt´s in dieser Klasse eine Variable mit Namen kugelliste? Ist das nicht eine Eigenschaft, die zur Urne gehört?

 

[Chevo]

Zu Sortieralgorithmen findest du beim großen Freund mit G genug, andernfalls mal hier im Forum suchen, das findest du schon...


Ich tippe mal darauf, dass die Instanzvariable

urne

zu der Zeit noch gar nicht belegt (also

null

) ist.
Aber schreib doch mal bitte zu der Klasse, in der du diese Variable verwendest, den ganzen Code (Urne und Kugel sehe ich, aber da fehlt ja irgendwo noch eine dritte Klasse...). Warum gibt´s in dieser Klasse eine Variable mit Namen kugelliste? Ist das nicht eine Eigenschaft, die zur Urne gehört?

Danke, google hat mir hier mal wirklich geholfen

Ne habe nur die beiden Klassen oder meinst du noch meine Fensterklasse wo ich meine GUI und alles habe?

 

[HimBromBeere]

Ich meine die Klasse, in der die Variablen

kugelliste

und

urne

verwendet werden (wird wohl deine GUI sein).

kugelliste.add(zahl); // hier klappts mit urne.kugelhinzufuegen(zahl) nicht -.-

Was "klappt" hier denn nicht? Meine Vermutung war, dass du eine

NullPointerException

bekommst, weil die Variable

urne

zu der Zeit dieses Aufrufes nicht belegt ist, ist aber reine Spekulation, könnte auch an der verringerten Dichte von Cookie-Partikeln im Universum sein...

 

[Chevo]

Ich meine die Klasse, in der die Variablen

kugelliste

und

urne

verwendet werden (wird wohl deine GUI sein).


Was "klappt" hier denn nicht? Meine Vermutung war, dass du eine

NullPointerException

bekommst, weil die Variable

urne

zu der Zeit dieses Aufrufes nicht belegt ist, ist aber reine Spekulation, könnte auch an der verringerten Dichte von Cookie-Partikeln im Universum sein...

Mit der

NullPointerException

hast du recht, habs überprüft und es war tatsächlich diese Fehlermeldung die mir in der Konsole ausgegeben wird.
Hier mal meine GUI, ist noch nicht ganz fertig, hab noch das Problem mit meiner Methode kugelhinzufuegen und wie ich meine Tippzahlen mit den Zahlen in der Arrayliste vergleichen kann.

package paket_Muendlich2;

import java.awt.EventQueue;

public class Hauptfenster extends JFrame
{
	private JPanel jContentPane;
	public final JLabel lbWillkommen = new JLabel(
			"Herzlich Wilkommen! Was möchten Sie tun?");
	public final JButton btnNeuesSpiel = new JButton("Neues Spiel");
	public final JButton btnBeenden = new JButton("Beenden");
	public final JLabel lbZahl1 = new JLabel("Zahl 1: ");
	public final JLabel lbZahl2 = new JLabel("Zahl 2: ");
	public final JLabel lbZahl3 = new JLabel("Zahl 3: ");
	public final JLabel lbZahl4 = new JLabel("Zahl 4: ");
	public final JLabel lbZahl5 = new JLabel("Zahl 5: ");
	public final JLabel lbZahl6 = new JLabel("Zahl 6: ");
	public final JTextField tfZahl1 = new JTextField();
	public final JTextField tfZahl2 = new JTextField();
	public final JTextField tfZahl3 = new JTextField();
	public final JTextField tfZahl4 = new JTextField();
	public final JTextField tfZahl5 = new JTextField();
	public final JTextField tfZahl6 = new JTextField();
	private Urne urne;
	private Kugel zahl;
	private ArrayList<Kugel> kugelliste = new ArrayList<Kugel>();
	public final JButton btnSpielen = new JButton("Spielen");
	public final List list = new List();

	/**
	 * Launch the application.
	 */
	public static void main(String[] args)
	{
		EventQueue.invokeLater(new Runnable()
		{
			public void run()
			{
				try
				{
					Hauptfenster frame = new Hauptfenster();
					frame.setVisible(true);
				}
				catch(Exception e)
				{
					e.printStackTrace();
				}
			}
		});

	}

	/**
	 * Create the frame.
	 */
	public Hauptfenster()
	{
		ausblenden();

		tfZahl6.setBounds(115, 268, 64, 20);
		tfZahl6.setColumns(10);
		tfZahl5.setBounds(115, 234, 64, 20);
		tfZahl5.setColumns(10);
		tfZahl4.setBounds(115, 200, 64, 20);
		tfZahl4.setColumns(10);
		tfZahl3.setBounds(115, 166, 64, 20);
		tfZahl3.setColumns(10);
		tfZahl2.setBounds(115, 132, 64, 20);
		tfZahl2.setColumns(10);
		tfZahl1.setBounds(115, 98, 64, 20);
		tfZahl1.setColumns(10);
		setTitle("LottoMaster v1.0.0");
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBounds(100, 100, 450, 400);
		jContentPane = new JPanel();
		jContentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
		setContentPane(jContentPane);
		jContentPane.setLayout(null);
		lbWillkommen.setBounds(74, 11, 255, 23);

		jContentPane.add(lbWillkommen);
		btnNeuesSpiel.addMouseListener(new MouseAdapter()
		{
			@Override
			public void mouseClicked(MouseEvent e)
			{
				einblenden();
			}
		});
		btnNeuesSpiel.setBounds(56, 55, 105, 31);

		jContentPane.add(btnNeuesSpiel);
		btnBeenden.addMouseListener(new MouseAdapter()
		{
			@Override
			public void mouseClicked(MouseEvent e)
			{
				System.exit(0);
			}
		});
		btnBeenden.setBounds(235, 55, 105, 31);

		jContentPane.add(btnBeenden);
		lbZahl1.setBounds(39, 97, 52, 23);

		jContentPane.add(lbZahl1);
		lbZahl2.setBounds(39, 131, 52, 23);

		jContentPane.add(lbZahl2);
		lbZahl3.setBounds(39, 165, 52, 23);

		jContentPane.add(lbZahl3);
		lbZahl4.setBounds(39, 199, 52, 23);

		jContentPane.add(lbZahl4);
		lbZahl5.setBounds(39, 233, 52, 23);

		jContentPane.add(lbZahl5);
		lbZahl6.setBounds(39, 267, 52, 23);

		jContentPane.add(lbZahl6);

		jContentPane.add(tfZahl1);

		jContentPane.add(tfZahl2);

		jContentPane.add(tfZahl3);

		jContentPane.add(tfZahl4);

		jContentPane.add(tfZahl5);

		jContentPane.add(tfZahl6);
		btnSpielen.addMouseListener(new MouseAdapter()
		{
			@Override
			public void mouseClicked(MouseEvent e)
			{
				// list.setVisible(true);
				// ziehen();
				pruefeFelder();
			}
		});
		btnSpielen.setBounds(56, 301, 105, 31);

		jContentPane.add(btnSpielen);
		list.setBounds(230, 139, 156, 162);

		jContentPane.add(list);
	}

	public void ausblenden()
	{
		tfZahl1.setVisible(false);
		tfZahl2.setVisible(false);
		tfZahl3.setVisible(false);
		tfZahl4.setVisible(false);
		tfZahl5.setVisible(false);
		tfZahl6.setVisible(false);
		lbZahl1.setVisible(false);
		lbZahl2.setVisible(false);
		lbZahl3.setVisible(false);
		lbZahl4.setVisible(false);
		lbZahl5.setVisible(false);
		lbZahl6.setVisible(false);
		btnSpielen.setVisible(false);
		list.setVisible(false);
	}

	public void einblenden()
	{
		btnNeuesSpiel.setVisible(false);
		tfZahl1.setVisible(true);
		tfZahl2.setVisible(true);
		tfZahl3.setVisible(true);
		tfZahl4.setVisible(true);
		tfZahl5.setVisible(true);
		tfZahl6.setVisible(true);
		lbZahl1.setVisible(true);
		lbZahl2.setVisible(true);
		lbZahl3.setVisible(true);
		lbZahl4.setVisible(true);
		lbZahl5.setVisible(true);
		lbZahl6.setVisible(true);
		btnSpielen.setVisible(true);
	}

	public void Kugelliste()
	{
		kugelliste.clear();
		for (int i = 1; i <= 49; i++)
		{
			zahl = new Kugel(i);
			kugelliste.add(zahl);
			// urne.kugelhinzufuegen(zahl);
		}

	}

	public void ziehen()
	{
		Kugelliste();
		ArrayList<Kugel> lotto = new ArrayList<Kugel>();
		int a;
		for (int i = 1; i <= 6; i++)
		{
			a = (int) (1 + Math.random() * kugelliste.size());
			lotto.add(kugelliste.get(a));
			kugelliste.remove(a);
		}
		Collections.sort(lotto);
		list.add(String.valueOf(lotto));
		int zaehler = 0;
		if(lotto.contains(Integer.parseInt(tfZahl1.getText()))) //Der Befehl wird iwie nie durchgeführt auch wenn eine der Zahlen übereinstimmt
		{
			zaehler = zaehler + 1;
		}
		else
		{
			if(lotto.contains(Integer.parseInt(tfZahl2.getText())))
			{
				zaehler = zaehler + 1;

			}
			else
			{
				if(lotto.contains(Integer.parseInt(tfZahl3.getText())))
				{
					zaehler = zaehler + 1;

				}
				else
				{
					if(lotto.contains(Integer.parseInt(tfZahl4.getText())))
					{
						zaehler = zaehler + 1;

					}
					else
					{
						if(lotto.contains(Integer.parseInt(tfZahl5.getText())))
						{
							zaehler = zaehler + 1;

						}
						else
						{
							if(lotto.contains(Integer.parseInt(tfZahl6
									.getText())))
							{
								zaehler = zaehler + 1;

							}
							else
							{
								if(zaehler == 0)
								{
									JOptionPane.showMessageDialog(null,
											"Keine Zahlen stimmen überein");
								}
								else
								{
									JOptionPane.showMessageDialog(null,
											"Sie haben" + zaehler
													+ " Zahlen richtig");

								}
							}
						}
					}
				}
			}
		}
		System.out.println(zaehler);

	}

	public void pruefeFelder()
	{

		if(tfZahl1.getText().equals(""))
		{
			JOptionPane.showMessageDialog(null, "Bitte Zahl eingeben");
			leeren1(tfZahl1);
		}
		else
		{
			if(tfZahl2.getText().equals(""))
			{
				JOptionPane.showMessageDialog(null, "Bitte Zahl eingeben");
				leeren1(tfZahl2);
			}
			else
			{
				if(tfZahl3.getText().equals(""))
				{
					JOptionPane.showMessageDialog(null, "Bitte Zahl eingeben");
					leeren1(tfZahl3);
				}
				else
				{
					if(tfZahl4.getText().equals(""))
					{
						JOptionPane.showMessageDialog(null,
								"Bitte Zahl eingeben");
						leeren1(tfZahl4);
					}
					else
					{
						if(tfZahl5.getText().equals(""))
						{
							JOptionPane.showMessageDialog(null,
									"Bitte Zahl eingeben");
							leeren1(tfZahl5);
						}
						else
						{
							if(tfZahl6.getText().equals(""))
							{
								JOptionPane.showMessageDialog(null,
										"Bitte Zahl eingeben");
								leeren1(tfZahl6);
							}
							else
							{
								list.setVisible(true);
								ziehen();
							}
						}
					}
				}
			}

		}

	}

	public void leeren1(JTextField pTextfeld)
	{
		pTextfeld.setText("");
		pTextfeld.requestFocus();
	}

}

 

[HimBromBeere]

Initialisier doch deine Urne gleich bei der Deklaration mit:

private Urne urne = new Urne();

Dann sollte die NPE verschwinden und du kannst

urne.kugelhinzufuegen(zahl)

verwenden.

Anmerkung: wenn deine Klasse schon Kugel heißt, dann nenn doch bitte auch die Variable dazu Kugel. Ich kam hier andauernd durcheiandner, weil ich dachte, du übergibst Zahken, keine Kugeln...

Kugel kugel = new Kugel(3);

Oder einfach:

urne.kugelhunzufuegen(new Kugel(3));