Problem mit datenkonsistenz

[biker126]

hallöchen...

hab ein mir schleierhaftes problem (sonst hätt ich's wohl schon gelöst ). hier erstmal den code dazu:

private PlayerCard[] createPlayerCardDeck(Card[] cards, Player p)
	{
		PlayerCard[] pc = new PlayerCard[14];
		int[] nachbarn = new int[3];
		sort(cards);
		int[] dummy = {0,0,0};
		pc[0] = new PlayerCard(0,0,0,dummy,p);
		for (int i=1; i<14; i++)
		{
			if (i < 11)
			{
				nachbarn[0] = i+1;
				nachbarn[1] = i+2;
				nachbarn[2] = i+3;
				pc[i] = new PlayerCard(cards[i].getFarbe(), cards[i].getWert(), i, nachbarn, p);
			}
			else if (i == 11)
			{
				nachbarn[0] = i+1;
				nachbarn[1] = i+2;
				nachbarn[2] = 0;
				pc[i] = new PlayerCard(cards[i].getFarbe(), cards[i].getWert(), i, nachbarn, p);
			}	
			else if (i == 12)
			{
				nachbarn[0] = i+1;
				nachbarn[1] = 0;
				nachbarn[2] = 0;
				pc[i] = new PlayerCard(cards[i].getFarbe(), cards[i].getWert(), i, nachbarn, p);
			}	
			else if (i == 13)
			{
				nachbarn[0] = 0;
				nachbarn[1] = 0;
				nachbarn[2] = 0;
				pc[i] = new PlayerCard(cards[i].getFarbe(), cards[i].getWert(), i, nachbarn, p);
			}
			
		}
		return pc;
	}

jetzt das problem:

die pc's sind innerhalb der for-loop "sichtbar". das heisst

pc[i].getNachbarPos()[0]

(funktion, die mir das nachbarpos-array zurücktgibt) enthält den richtigen wert. aber ausserhalb der loop, dh. direkt vor dem "return pc" ergibt mir ein

pc[i].getNachbarPos()[0]

immer 0 zurück...

hab ich da was falsch gemacht mir variablen anlegen (speicher) und initialisieren usw. oder so? was anderes fällt mir als ursache nicht ein... aber kA...

wär cool wenn man jmd auf die sprünge helfen könnte (oder einfach tips worans liegen könnte..)

gruss biker

 

[kaie]

Du übergibst jeder PlayerCard das selbe nachbarn-Array, d.h. wenn Du für eine PlayerCard den Inhalt von nachbarn änderst, ändert sich dieser Wert für alle. Da Du zuletzt, für die Karte Nr. 13, das gesamte Array auf 0 setzt, ist es für alle Karten gleich 0. Einfache Lösung: erzeuge jedesmal ein neues int-Array:

private PlayerCard[] createPlayerCardDeck(Card[] cards, Player p) 
   { 
      PlayerCard[] pc = new PlayerCard[14]; 
      sort(cards); 
      int[] dummy = {0,0,0}; 
      pc[0] = new PlayerCard(0,0,0,dummy,p); 
      for (int i=1; i<14; i++) 
      { 
         if (i < 11) 
         { 
            int[] nachbarn = new int[] { i+1, i+2, i+3 };
            pc[i] = new PlayerCard(cards[i].getFarbe(), cards[i].getWert(), i, nachbarn, p); 
         } 
         else if (i == 11) 
         { 
            int[] nachbarn = new int[] { i+1, i+2, 0 };
            pc[i] = new PlayerCard(cards[i].getFarbe(), cards[i].getWert(), i, nachbarn, p); 
         }    
         else if (i == 12) 
         { 
            int[] nachbarn = new int[] { i+1, 0, 0 };
            pc[i] = new PlayerCard(cards[i].getFarbe(), cards[i].getWert(), i, nachbarn, p); 
         }    
         else if (i == 13) 
         { 
            int[] nachbarn = new int[] { 0, 0, 0 };
            pc[i] = new PlayerCard(cards[i].getFarbe(), cards[i].getWert(), i, nachbarn, p); 
         } 
          
      } 
      return pc; 
   }

kaie

 

[biker126]

hm, ehrlich gesagt seh ich ned ganz wieso die pc's (bzw. deren attribute) überschrieben werden...

klar hab ich nur ein nachbarn[] array und überschreib die werte darin immer... aber das sollte ja eigentlich mit den if's kontrolliert ablaufen... ??

naja, werd mal deinen vorschlag probieren
danke schonmal!