Frage zu "this"

[allesklar]

Hi ich hab mal eine Frage zu "this"
Hier erstmal der Code:

public class Test{
	
	LinkedList<AList> testlist = new LinkedList<>();
	
	public static void main(String[] args){

		AList obj1 = AList("string1");
		
		Alist obj2 = AList("string");
}
}


public class AList{
 String str;
 Test t = new Test();

public Alist(String str){
 this.str = str;
 t.testlist.add(this);
 }

}

warum funktioniert das nicht? es kommt auch kein fehler, es passiert einfach nichts mit der Liste.

 

[AquaBall]

Was sollte denn passieren?
Woran erkennst du dass nichts passiert?

Und: das ist shcon eine verflixt exotische Konstruktion.
Du machst ja in jedem AList ein NEUES anonymes t, und in Test.Testlist wird dadurch nie reingeschrieben!
Was hält's du von:

public class Test{
    LinkedList<AList> testlist = new LinkedList<>();
    public static void main(String[] args){
        testlist.add(new AList("string1"));
        testlist.add(new AList("string2"));
        // ... und dann wäre noch ein print oder irgendwas angebracht ...
    }
}
 
public class AList{
    String str;
    public Alist(String str){
         this.str = str;
    }
}

 

[allesklar]

Was sollte denn passieren?
Woran erkennst du dass nichts passiert?

Und: das ist shcon eine verflixt exotische Konstruktion.
Du machst ja in jedem AList ein NEUES anonymes t, und in Test.Testlist wird dadurch nie reingeschrieben!

ich wollte das jedesmal wenn ein Objekt von AList erzeugt wird, und dieses dann in die Liste der Klasse Test geschrieben wird.

Was hält's du von:

public class Test{
    LinkedList<AList> testlist = new LinkedList<>();
    public static void main(String[] args){
        testlist.add(new AList("string1"));
        testlist.add(new AList("string2"));
        // ... und dann wäre noch ein print oder irgendwas angebracht ...
    }
}
 
public class AList{
    String str;
    public Alist(String str){
         this.str = str;
    }
}

danke das hat funktioniert.

so funktionierts auch:

class Test{
static LinkedList<Alist> testlist = new LinkedList<>();
....
}

class AList{

    public Alist(String str){
         this.str = str;
         Test.testlist.add(this);
    }
}

 

[AquaBall]

Damit kannst du die Klasse aber nie kapseln, und zerstörst die Verwendung und den eigentlichen Sinn von Klassen.

Würd ich auf keinen Fall machen.
Wenn du es ungbedingt automatisieren willst, und sicherstellen willst, dass es kein AList ohne Eintrag gibt, dann mach's mit ParameterÜbergabe.
Aber keinesfalls Test.testlist verwenden!

class Test{
static LinkedList<Alist> testlist = new LinkedList<>();
.... new AList("String",testlist) ...;
}
 
class AList{
    String str;
    public AList(String str,LinkedList<Alist> collList){
         this.str = str;
         collList.add(this);
    }
}

 

[haui95]

Eine Frage,

@ allesklar

class Test{
static LinkedList<Alist> testlist = new LinkedList<>();
....
}
 
class AList{
                                         
    public Alist(String str){
         this.str = str;                            // Wie soll das gehen ? Die Instanzvariable "str" fehlt doch !? Worauf bezieht sich denn dann die Variable "this"  ? 
         Test.testlist.add(this);
    }
}

MfG

Hauke

 

[allesklar]

jo fehlt die variable fehlt, weil ich zu faul war sie in diesem beispiel reinzuschreiben aber die ist auch für das eigentliche problem nicht relevant.

 

[Mujahiddin]

Das war ein Fehler...
In allen anderen Codeschnipseln ist die Klassenvariable [c]str[/code] gegeben.

E @OP:
Ich glaube, was du machen willst, ist eine statische Klassenvariable, die alle Instanzen einer Klasse speichert. Sowas ist meistens in der Klasse selbst geeignet:

public class Boo{
	private static List<Boo> instances = new ArrayList<>();
	
	public Boo(){
		instances.add(this);
		// blabla
	}
}