Probleme beim adden von Elementen in einer ArrayList

[icarus2]

Huhu

Ich möchte im moment gerade gerne ein Spiel programmieren. Um die Koordinaten der Schife zu speichern möchte ich eine ArrayList nutzen.

Doch irgendwie funktioniert das adden von Elementen in der ArrayList. Ich weiss aber nicht warum, die Länge bleibt einfach immmer 0.

Die ArrayList:
[Java]
private ArrayList<Integer> xCoordinates = new ArrayList<Integer>();
private ArrayList<Integer> yCoordinates = new ArrayList<Integer>();

[/Java]

Anschliessend habe habe ich eine Methode, in der neue Elemente geaddet werden sollen:

[Java]
@Override
public void createNewObject(){

xCoordinates.add(calculateXCoordinatesNewShip());
yCoordinates.add(calculateYCoordinatesNewShip());
currentStructure.add(currentStructure.size(),initialStructure);

}

[/Java]

Doch wie gesagt, es funktioniert nicht. Die Länge bleibt anschliessend immer 0.

Btw... die Methoden, die als Parameter übergeben werden bei add() sehen beide so aus:

[Java]
protected int calculateYCoordinatesNewShip(){
return (int)(Math.random()*500-50);
}
[/Java]

Ich habe überhaupt keine Ahnung woran das liegt... kann mir da jmd helfen. Oder soll ich mal etwas mehr Code reinstellen?

 

[Ark]

Ich denke, das Problem ist nur aus Sicht eines größeren Kontextes erkennbar. Wir brauchen also mehr Code drumherum, vor allem die Stellen, wo die Objekte zu leben beginnen und wo die Größe der Listen abgefragt wird.

Ark

 

[Schandro]

Poste mal ein KSKB oder bau bei dir ein paar sysout's zum debuggen ein (oder benutz nem Debugger).

Ansonsten, wenn ich dir nen Tipp geben darf: Du solltest die x- und y-Koordinate niemals getrennt behandeln. Mach eine "Container"-Klasse, wo du die x-und y-Koordinate gleichzeitig drin speichern kannst. Dann brauchst du nur eine ArrayList, die diese "Punkte" aufnimmt. (du könntest auch die bereits vorhandene Klasse Point benutzen).
Aufjedenfall sollte man zusammengehörige und komplett voneinander abhängige Werte nicht trennen.

 

[icarus2]

Danke schon mal. Ich werde es dann mal mit der Klasse Point versuchen.

Also, ich habe eine Basisklasse Alien, die ein par Methoden zur Verfügung stellt, die alle Alientypen brauchen.

Die Klasse sieht so aus:
[Java]
package aliens;

import java.awt.*;
import java.util.*;

public class Aliens {


/**
* Calculates randomly in which direction an object moves.
*
* @param currentPosition
* @return if it moves upwards it returns a positive, if downwards a negative number.
*/

private static int calculateMovingDirection(int currentPosition){

int randomDirection = (int)(Math.random()*100+1);

if(randomDirection <= 50){
return -1;
}
else{
return 1;
}
}

protected void setNewXCoordinates(ArrayList<Integer> coordinate, int speedX){
for(int i = 0; i < coordinate.size(); i++){

if(calculateMovingDirection(0) > 0){

int position = coordinate.get(i) - speedX;

coordinate.remove(i);
coordinate.add(i, position);
}
else{
int position = coordinate.get(i) + speedX;

coordinate.remove(i);
coordinate.add(i, position);
}

}
}


protected ArrayList setNewYCoordinates(ArrayList<Integer> coordinate, int speedY){


for(int i = 0; i <= coordinate.size(); i++){

if(calculateMovingDirection(0) > 0){

int position = coordinate.get(i) - speedY;

coordinate.remove(i);
coordinate.add(i, position);
}
else{
int position = coordinate.get(i) + speedY;

coordinate.remove(i);
coordinate.add(i, position);
}

}

return coordinate;
}

/*
protected void setNewYCoordinates(ArrayList<Integer> coordinate, int speedY){

//System.out.println("setNewYCoordinates called");

for(int i = 0; i < coordinate.size(); i++){

if(calculateMovingDirection(0) > 0){

int position = coordinate.get(i) - speedY;

coordinate.remove(i);
coordinate.add(i, position);
}
else{
int position = coordinate.get(i) + speedY;

coordinate.remove(i);
coordinate.add(i, position);
}

}
}

*/

/**
* Calculates randomly if a ship is going to shoot or not
* @return true if it shoots, false if it does not
*/
protected static boolean shipShoots(){

int randomShoot = (int)(Math.random()*100+1);

if(randomShoot == 10 || randomShoot == 50 || randomShoot == 40){
return true;
}
else{
return false;
}
}

/**
* Calculates randomly if a new ship is created or not
*
* @param currentAmountOfShips, the current amount of ships in the space
* @return true if a new ship is created, else false
*/
protected static boolean shipGoingToBeCreated(int currentAmountOfShips){

boolean status;

if(currentAmountOfShips <= 2){
status = true;
}
else if(currentAmountOfShips >= 10){
status = false;
}
else{

int randomShip = (int)(Math.random()*50+1);

if(randomShip == 5 || randomShip == 10){
status = true;
}
else{
status = false;
}
}
return status;
}

protected int calculateXCoordinatesNewShip(){
return 800;
}

protected int calculateYCoordinatesNewShip(){
return (int)(Math.random()*500-50);
}

/**
* Method to paint the ships of a class that inherits this method
*
* @param xCoordinates, ArrayList in which the coordinates are stored
* @param yCoordinates, ArrayList in which the coordinates are stored
* @param g, The Graphics object
* @param c, The Color of the Ship
* @param width
* @param height
*/
protected static void paintShips(ArrayList<Integer> xCoordinates,
ArrayList<Integer> yCoordinates,
Graphics g, Color c,
int width, int height){
for(int i = 0; i < xCoordinates.size(); i++){
g.fillRect(xCoordinates.get(i), yCoordinates.get(i), width, height);
}

}



}
[/Java]

Ausserdem ist da noch die erste Alientyp-Klassen Trins. Das soll das erste Alien sein.
[Java]
package aliens;

import java.awt.Color;
import java.awt.Graphics;
import java.util.*;
import tools.*;

public class Trins
extends Aliens
implements Updatable, Createable, Destroyable, Movable{


private final static int initialStructure = 50;
private final static int speedX = 5;
private final static int speedY = 2;
private static int currentAmountOfShips = 0;
private final static int shipDimension = 100;
private final static Color shipColor = Color.red;
//private final static int shotPower = 10;

//Attributes which store values during the game in Lists
private ArrayList<Integer> xCoordinates = new ArrayList<Integer>();
private ArrayList<Integer> yCoordinates = new ArrayList<Integer>();
private ArrayList<Integer> currentStructure = new ArrayList<Integer>();

//done: Updatable, Creatable, Movable
//missing: Destroyable
@Override
public void updateObjects(){

if(super.shipGoingToBeCreated(currentAmountOfShips)){
createNewObject();
currentAmountOfShips++;
}


setNewXCoordinates(xCoordinates, speedX);
setNewYCoordinates(yCoordinates, speedY);


}

@Override
public void createNewObject(){

xCoordinates.add(calculateXCoordinatesNewShip());
yCoordinates.add(calculateYCoordinatesNewShip());

}


@Override
public int getStructure(int shipNumber) {
return currentStructure.get(shipNumber);
}


@Override
public void removeDestroyedObject(int shipNumber) {
xCoordinates.remove(shipNumber);
yCoordinates.remove(shipNumber);
currentStructure.remove(shipNumber);
}


@Override
public void setDamage(int shipNumber, int damage) {

int newStructure = currentStructure.get(shipNumber)-damage;

currentStructure.set(shipNumber, newStructure);
}


@Override
public int getSpeed(int shipNumber) {
return 5;
}


@Override
public int getXCoordinate(int shipNumber) {
return xCoordinates.get(shipNumber);
}


@Override
public int getYCoordinate(int shipNumber) {
return yCoordinates.get(shipNumber);
}


@Override
public void paintObjects(Graphics g) {

//System.out.println("paintObjects called");

paintShips(xCoordinates, yCoordinates, g, shipColor, shipDimension, shipDimension);

}






}
[/Java]


Das ganze lauft in verschiedenen Threads ab. Einer aktualisiert das Bild all 30 ms, der andere führt die Berechnungen für neue Schiffe und alles durch. Aber wie gesagt, das speichern funktioniert irgendwie nicht.

 

[icarus2]

So, ich habe den Code jetzt abeändert, sodass die Klasse Point verwendet wird:

[Java]
package aliens;

import java.awt.Color;
import java.awt.Graphics;
import java.awt.Point;
import java.awt.geom.Point2D;
import java.util.*;
import tools.*;

public class Trins
extends Aliens
implements Updatable, Createable, Destroyable, Movable{


private final static int initialStructure = 50;
private final static int speedX = 5;
private final static int speedY = 2;
private static int currentAmountOfShips = 0;
private final static int shipDimension = 100;
private final static Color shipColor = Color.red;
//private final static int shotPower = 10;

//Attributes which store values during the game in Lists
private ArrayList<Integer> currentStructure = new ArrayList<Integer>();
private ArrayList<Point> coordinates = new ArrayList<Point>();

//done: Updatable, Creatable, Movable
//missing: Destroyable
@Override
public void updateObjects(){

if(super.shipGoingToBeCreated(currentAmountOfShips)){
createNewObject();
currentAmountOfShips++;
}


}

@Override
public void createNewObject(){

coordinates.add(new Point(calculateXCoordinatesNewShip(), calculateYCoordinatesNewShip()));
System.out.println("createNewObject: ArrayList-size:"+(coordinates.size()-1));

}


@Override
public int getStructure(int shipNumber) {
return currentStructure.get(shipNumber);
}


@Override
public void removeDestroyedObject(int shipNumber) {
//xCoordinates.remove(shipNumber);
//yCoordinates.remove(shipNumber);
currentStructure.remove(shipNumber);
}


@Override
public void setDamage(int shipNumber, int damage) {

int newStructure = currentStructure.get(shipNumber)-damage;

currentStructure.set(shipNumber, newStructure);
}


@Override
public int getSpeed(int shipNumber) {
return 5;
}


@Override
public int getXCoordinate(int shipNumber) {
return (int)(coordinates.get(shipNumber).getX());
}


@Override
public int getYCoordinate(int shipNumber) {
return (int)(coordinates.get(shipNumber).getY());
}


@Override
public void paintObjects(Graphics g) {

paintShips(coordinates, g, shipColor, shipDimension, shipDimension);

}






}
[/Java]

In Zeile 44 wird folgendes ausgegeben: createNewObject: ArrayList-size:0

 

[Final_Striker]

ja, du bekommt size = 1 zurück, dann rechnest du size - 1.
und 1 - 1 ist halt 0.

 

[icarus2]

Ja, aber das Problem ist, dass die ganze Zeit Elemente hinzugefügt werden sollten in Zeile 43.

Die Methode wird aber ganz sicher aufgerufen, habs mit System.out.println() ausprobiert.

Aber da tut sich nichts :-(

 

[icarus2]

So, ich habe jetzt alles ausprobiert um den Fehler beheben zu können, doch es funktioniert leider immer noch nicht :-(

Ich habe jetzt mal den ganzen Code angehängt, so dass ihr in kompilieren könnt.

Interressant ist dabei vor allem die Klasse Trins, wo die Aliens erzeugt werden und alles andere, was durch die Interfaces implementiert wird. Da ist immer noch das problem, dass nicht in der ArrayList geaddet wird, obwohl es eigentlich sollte.

Ich wäre echt froh, wenn mir da jmd helfen könnte, denn ich komme wirklich nicht mehr weiter :/

(Und falls die Struktur des Programms völlig sinnlos ist bitte auch sagen ;-))

 

[Landei]

Ich krieg das Zip-File nicht auf.

Davon abgesehen hast du ein sehr komisches Design: Jeder Alien-Typ bekommt ein Objekt und hält die "Einzelteile" seiner Schiffe in irgendwelchen Listen. Warum bekommt nicht jedes Alienschiff sein eigenes Objekt, das Position, Schaden usw. enthält? Im Konstruktor kann es sich auch in einer statischen Liste registrieren usw.

 

[icarus2]

Hmm, stimmt das wäre echt besser wenn für jedes Schiff ein Ojbekt erzeugt wird.

Aber ich bin mir etwas unsicher wie ich das am besten anstelle dynamisch Objekte erzeugen zu können, diese zu speichern und anschliessend auf die Werte zuzugreiffen.

Hättest du eventuell ein Beispiel (etwas was du selber mal in die Richtung gemacht hast) auf Lager?

 

[icarus2]

So, ich habe mir das hier jetzt gerade selber überlegt wie ich das machen könnte.

Ich erzeuge jetzt, wie du gesagt hast, ein Objekt für jedes Schiff und speichere dieses in einer statischen ArrayList. Ich habe gerade mit einfachen Beispielen etwas experimentiert und es scheint soweit zu funktionieren.

Ich probiere das jetzt mal umzusetzen.

Danke, war ein verdammt guter Tipp mit dem Objekte erzeugen