ArrayList

[rap]

Hallo, mein kleines Programm funktioniert nicht mehr seit ich eine Arraylis eingefügt habe, ich weiß aber nicht wieso... Es geht um eine Schlange, die sich bewegt und ihre Position auf dem Feld in eine ArrayList speichern soll. Vorher hat sie die Position nach jeder Bewegung ausgegeben, das hat geklappt...Hier ist der Ausschnitt:

	public ArrayList<position> test(){
		ArrayList<position> path = new ArrayList<position>();
		int xx = position.x;
		int yy = position.y;
		int richtung1 = position.richtung;
		position p = new position(xx,yy,richtung1);
		path.add(0,p);

		return path;
	}

	public String [] getpath(){
		ArrayList<position> zw = new ArrayList<position>();
		zw = this.test();

		String [] ans = new String [zw.size()];
		System.out.println(zw.size());
		for (int i = 0; i <= zw.size(); i++){
			ans[i] = toString(zw.get(i));
		}

		return ans;
	}
	


		
	public String toString(position position) {
		String Richt = "";
		if(position.richtung == 3){
			Richt = "East";
		}
		else if (position.richtung == 0){
			Richt = "North";
		}
		else if (position.richtung == 1){
			Richt = "West";
		}
		else if (position.richtung == 2){
			Richt = "South";
		}
		
		String r = "X-Koordinate: " + position.x + "   Y-Koordinate " + position.y + "   Richtung: " + Richt;
		return r;
	}

Ich habe inen Test eingefgüt (Z.17), komischerweise gibt der 1 zurück...
Das sind die Methoden in denen der das macht und hier ist das Testprogramm:

import java.util.Arrays;

import *******snake.Snake;
import *******snake.Territory;


public class test {

	/**
	 * @param args
	 */
	public static void main(String[] args) {
		Territory kaefig = new Territory ("14.txt");
		kaefig.setScale(30);
		position pos = new position(2,3,Snake.DIRECTION_SOUTH);
		superschlange schlange = new superschlange(pos.x, pos.y, kaefig,Snake.DIRECTION_SOUTH,superschlange.eggs);

		schlange.move();
		schlange.move();
		schlange.move();
		schlange.turnLeft();
		schlange.turnLeft();
		System.out.println(Arrays.toString(schlange.getpath()));
	
		
	}

}

Ich hoffe auf schnelle Hilfe...
Viele Grüße

 

[Der Müde Joe]

In deinem Code sind so viele unbekannten. Blick da grad nicht durch.

Wenn du das Snake vom Natel baust, wäre wohl eine Deque (im speziellen LinkedList (Java Platform SE 6)) besser geeignet.

 

[rap]

Ja, sorry ich hab vergessen die Klasse Position einzufügen... Das kann ich morgen machen ( bin gerade nicht am pc... Schon mal vorab: die Klasse Position ha 3 attribute int x int y int Richtung und noch ein paar Methoden aber das ist ja nicht so wichtig... Viele grüße

 

[EdelStoff]

War deine Frage wieso er 1 ausgibt?

public ArrayList<position> test(){
        ArrayList<position> path = new ArrayList<position>(); // hier erstellst du eine leere ArrayList
        int xx = position.x;
        int yy = position.y;
        int richtung1 = position.richtung;
        position p = new position(xx,yy,richtung1);
        path.add(0,p); // hier fügst du ihr eine postion hinzu
 
        return path; // diese ArrayList gibst du zurück
    }
 
    public String [] getpath(){
        ArrayList<position> zw = new ArrayList<position>(); // diese Zeile ist unnötig ArrayList<position> zw = test(); reicht
        zw = this.test(); // nun weist du zw der ArrayList mit dem einen position Element aus test() zu
 
        String [] ans = new String [zw.size()]; // => zw.size() == 1
        System.out.println(zw.size());
        for (int i = 0; i <= zw.size(); i++){
            ans[i] = toString(zw.get(i));
        }
 
        return ans;
    }

 

[rap]

Ok, ich glaube ich habe mich nicht wirklich eindeutig ausgedrückt...Die Methode test() wird bei jeder Bewgung der Schlange ausgelöst...Da sie vorher schon mindestens 5 Bewegungen gemacht hat, müssten schon mehrere Elemente in der ArrayList sein...

	public void turnLeft(){
		super.turnLeft();
		position.richtung = this.getDirection();
		this.test(); //Hier wird die Methode ausgeführt...
	}

Und hier beim Schritt:

	public void move(){
		super.move();
		if (superschlange.DIRECTION_NORTH == this.getDirection()) {
		position.y -= 1;
		}
		else if (superschlange.DIRECTION_SOUTH == this.getDirection()){
		position.y += 1;

		}
		else if (superschlange.DIRECTION_WEST == this.getDirection()){
			position.x -= 1;
			}
		else if (superschlange.DIRECTION_EAST == this.getDirection()){
			position.x += 1;
			}
		this.test(); //Hier wird die Methode ausgeführt...
	}

Viele Grüße,
Rap

 

[Geeeee]

Wie EdelStoff schon geschrieben hat:
Du legst immer eine neue ArrayList an und packst 1 Element rein. Auch wenn du das 5 mal machst, wird sich nur ein Element in der Liste befinden.

 

[rap]

Ah, etzt habe ich verstanden...Ich lege immer eine neue ArrayList an...
Danke

 

[rap]

Thx