2 Objekte, 1 immer Null?

[Chrissy87]

Hallo,

Ich versuche gerade ein Memoryspiel zu programmieren und will nun zwei angeklickte Karten vergleichen.

public class Card extends JPanel implements MouseListener
{
Card card1, card2;
...
public void equalsCard()
	{
		card1= Card.this;
		
		if(card2 == null)
		{
			card2 = card1;
			card1 = null;
		}
	}
...
}

Wenn ich richtig denke sollte so wie ich es programmiert habe card2 nur dann durch card1 ersetzt werden, wenn card2 == null. Jedoch geht das Programm jedesmal in die if-Anwendung und mein card1 ist immer null.

Hat irgendjemand eine Idee?

LG

PS Die Methode equalsCard() wird durch das klicken auf der Karte aufgerufen.

 

[faetzminator]

Klar, wenn [c]card2[/c] [c]null[/c] ist, dann resultiert daraus: [c]card1 == null && card2 == this[/c]. Ansonsten: [c]card1 == this && card2 != null[/c].

 

[Chrissy87]

okay, und wie bekomme ich es hin, dass beide Objekte gefüllt werden. Also beim ersten Klick card2 und beim zweiten Klick card1?

 

[guest7272]

Ungefähr so:

if (card2 == null)
{
    card2 = ...
}
else if (card1 == null)
{
    card1 = ...
}

Da beide Variablen am Anfang null sind springt beim ersten Durchlauf der erste Zweig an, beim zweiten Durchlauf der zweite. Wenn die jeweils andere Karte auf null gesetzt werden soll, musst du das nur entsprechend ergänzen.

Trotzdem sieht das Ganze konzeptionell etwas schräg aus ...

 

[jgh]

Grundsätzlich sollte die eine Card von der anderen Card doch nichts wissen => die Methode [c]equalsCard()[/c] sollte so geändert werden, dass sie sich selbst mit einer anderen Card vergleicht => die Signatur der Methode müsste so aussehen: [c]boolean equalsCard(Card c)[/c]

 

[guest7272]

wobei die Methode in ihrer jetzigen Form überhaupt nichts mit einen equals-Vergleich zu tun hat -> klassischer Fall von falschem / irreführenden Methodennamen

 

[Chrissy87]

Also, ich möchte zwei Karten aus dem Spiel nach einander anklicken und die dann als zwei Objekte speichern. Diese Objekte sollten dann später verglichen werden und wenn es die gleich (in meinen Fall) Fotos sind, dann soll die vom Spielfeld verschwinden.

Mir gehts jetzt erstmal darum, dass ich die beiden Karten als zwei Objekte speichern kann. Leider frisst mein Garbage Collection die immer.

 

[guest7272]

Deine jetzige Beschreibung ist aber was komplett anderes als du oben implementiert hast.

Am einfachsten wäre:

• Karten wirklich als Karten-Objekte modellieren, und nichts als JPanel, also UI und Logik sauber trennen
• Instanzvariablen card1 und card2 löschen. Stattdessen in der Card-Klasse equals überschreiben, und zwar so dass der Vergleich klappt.

 

[Chrissy87]

So ich hab es soweit hinbekommen, dass er zwei Karten speichert und wieder verdeckt.

Jetzt ist mein Problem das Vergleichen. Meine Karten bestehen jetzt nicht nur aus Photos sondern auch aus Dreiecke, Vierecke etc. Sie werden durch die Klasse Card "gemalt".

Ich hab jetzt beim erstellen der Karten durch ein Array eine ID speichern lassen. Das funktioniert auch, aber leider nicht bei den Fotos. Wie könnte ich nun den Photos eine ID zuweisen?
------

hat sich erledigt konnte nicht richtig gucken und hab ne falsche Reihenfolge gehabt