Methoden Objekte werden anscheinend nicht erstellt

[Venom Ousz]

Guten Abend liebes Forum,
ich habe folgendes Problem, das ich alleine nicht gelöst bekomme.
Ich arbeite mit der folgenden Klasse ArrayList.java:

Edit:
Kleine Erklärung zur Klasse: Eine ArrayList besteht aus einem Array aus Integerzahlen. Für mein Problem ist eigentlich nur die Methode append(int newElement) wichtig. Diese Methode ergänzt ein Objekt der ArrayList um ein weiteres Element.

public class ArrayList{
	private int[] data;
	
	public ArrayList(){
		this.data = new int[0];
	}
	
	private ArrayList(int[] dataArg){
		this.data = dataArg;
	}
	
	private void error(String errorMessage){
		System.err.println(errorMessage);
	}
	
	public int length(){
		return this.data.length;
	}
	
	public boolean isEmpty(){
		return this.length()==0;
	}
	
	public int get(int index){
		if (index <0 || index >= this.length()){
			error("Unzulaessiger Index.");
			return -1;
		}
		else{
			return this.data[index];
		}
	}
	public ArrayList append(int newElement){
		if(newElement<0){
			error("Unzulaessiges Element.");
			return null;
		}
		int newLength = this.length()+1;
		int[] newData = new int[newLength];
		for(int i=0; i<this.length();i++){
			newData[i]=this.get(i);
		}
		newData[newLength-1]=newElement;
		return new ArrayList(newData);
	}
	
	public int head(){
		if(this.isEmpty()){
			error("Die Liste hat kein Element.");
			return -1;
		}
		int head = this.get(0);
		return head;
	}
	
	public ArrayList tail(){
		if(this.isEmpty()){
			error("tail: Die Liste hat kein Element.");
			return null;
		}
		ArrayList result = new ArrayList();
		for(int i=1; i<this.length();i++){
			result=result.append(this.get(i));
		}
		System.out.println(result);
		return result;
	}
	public ArrayList concat(ArrayList tail){
		ArrayList result = this;
		ArrayList remainder = tail;
		while(!remainder.isEmpty()){
			result=result.append(remainder.head());
			remainder=remainder.tail();
		}
		return result;
	}
	public boolean compare(ArrayList that){
		ArrayList remainderOfThis = this;
		ArrayList remainderOfThat = that;
		while(!remainderOfThis.isEmpty()&&!remainderOfThat.isEmpty()){
			if(remainderOfThis.head()!=remainderOfThat.head()){
				return false;
			}
			remainderOfThis=remainderOfThis.tail();
			remainderOfThat=remainderOfThat.tail();
		}
		return remainderOfThis.isEmpty()&&remainderOfThat.isEmpty();
	}
}

Nun habe ich zu Testzwecken eine Klasse ArrayListTest.java erstellt, um einige ArrayListen zu erstellen und den Code zu überprüfen. Die Klasse sieht wie folgt aus:

public class ArrayListTest {
	public static void main(String[] args){
		ArrayList test = new ArrayList();
		test.append(1);
		test.append(2);
		test.append(3);
		test.append(4);
		int wut =test.length();
		System.out.println(wut);
	}

}

Allerdings bekomme ich in der Konsole für die Variable "wut" den Wert 0 ausgegeben, heißt, die Länge meiner Liste beträgt 0 und es wurden keine Elemente hinzugefügt.
Entweder ist die append(int newElement) Methode fehlerhaft oder meine ArrayList test wird nicht erstellt. Wie lautet die Lösung zu diesen Problem? Ich bin echt am verzweifeln... Freue mich über jede denkbare Hilfe.

Mit freundlichen Grüßen
Venom Ousz

 

[Ruzmanz]

Tut mir leid, aber da funktioniert einiges nicht ... deine Methode append() ändert gar nichts in dem Objekt "test", da wird was verararbeitet und zurückgegeben. Das ist genau so wie

String s = "Hallo";
s.substring(3);
System.out.println(s);

vs.

String s = "Hallo";
s = s.substring(3);
System.out.println(s);

 

[knilch]

Hi,
Schau mal deine append Methode an:

public ArrayList append(int newElement){
    if(newElement<0){
        error("Unzulaessiges Element.");
        return null;
    }
    int newLength = this.length()+1;
    int[] newData = new int[newLength];
    for(int i=0; i<this.length();i++){
        newData[i]=this.get(i);
    }
    newData[newLength-1]=newElement;
    return new ArrayList(newData);
}

Du erstellst eine neue Instanz von ArrayList und gibst diese dann zurück.

In der Main- Methode erstellst du zuerst eine ArrayList- Instanz, appendest im 1. schritt den int- Wert 1.
diese Methode gibt dann eine neue instanz von ArrayList zurück. Dann wird im 2.Schritt der wert 2 appendet. Dies gibt dir wieder eine neue Instanz ArrayList zurück.
Insgesamt hast du dann 4 ArrayList- Instanzen, die zurückgegeben wurden, aber die eigentliche Instanz ArrayList wurde nie updated. Da newData das int- Array für die neue ArrayList instanz ist

Dh. Am ende wird ArrayList (von Main) immer 0 sein!

Hier mal ein Ansatz:

public void append(int newElement){
    if(newElement < 0){
        error("Unzulaessiges Element.");
        return;
    }
    int[] newData = new int[data.length + 1];
    for(int i=0; i < data.length; i++){
        newData[i] = get(i);
    }
    newData[data.length] = newElement;
    data = newData;
}

 

[arilou]

Außerdem ist es unschön, eigenen Klassen einen Namen zu geben, den es in den Java-Standardbibliotheken bereits gibt - insbesondere, wenn mehr als eine Person an dem Code arbeiten. Dann redet Programmierer_1 von der selbst-erstellten Klasse, und Programmierer_2 von der Java-internen, und "nichts stimmt"...

Anmerkung:
java.util.ArrayList<Integer> wäre angebracht;
alternativ sind ggf. auch die Methoden von java.util.Arrays interessant.