Variablen Variablenwert über Konstruktor ändern

[Ashtari]

Hallo Zusammen,

derzeit Studiere ich Informatik und habe im zweiten Semester Java Programmierung. Aufgrund eines Wahlbereichs habe ich im ersten Semester schon C++ gehabt und das lief eigentlich auch ganz gut, allerdings habe ich hier gerade ein Problem welches ich nicht gelöst bekomme...

Ich habe eine Klasse in dessen Konstruktor ich mir eine Zufallszahl ausgeben lasse. Das klappt soweit auch wunderbar.

Jetzt möchte ich genau diesen Wert aber abspeichern und als String ausgeben...
Wenn ich mir jetzt allerdings die Werte aus meinem Array zufallsGewicht ausgeben lasse, bekomme ich immer 0. Er übernimmt die Werte die ich im Konstruktor festlege nicht.

So sieht mein Code derzeit aus:

public class Geschenk extends GewichtigesWeihnachtsObjekt {

    private static int anzahlGeschenke = 1;
    private static int zufallsGewicht[] = new int[anzahlGeschenke];
   
    public Geschenk(){
        super(anzahlGeschenke);
        Random zufallsZahl = new Random();   
        int zufallsGewicht[] = new int[anzahlGeschenke];
       
        //Zufälliges Gewicht für jedes Geschenk im Konstruktor!
        for(int i = 0; i < anzahlGeschenke; i++) {
            zufallsGewicht[i] = zufallsZahl.nextInt(20);
            System.out.println(zufallsGewicht[i]);
            i++;
        }       
    }

    public String toString() {
        int i = 0;
        return String.format("%s und %s", anzahlGeschenke, zufallsGewicht[i]);
    }
   
    //Zur Überprüfung der Klasse jeweils eine Main
    public static void main(String[] args) {
        Geschenk geschenkEins = new Geschenk();
        Geschenk geschenkZwei = new Geschenk();
        System.out.println(geschenkEins.toString());
    }
}

Wie kann ich jetzt in meiner toString() - Methode auf meine im Konstruktor Werte zugreifen, bzw. ist das grundsätzlich überhaupt möglich? Wenn ich die Werte in der For-Schleife ausgebe, zeigt er mir auch die Random-Zahlen an, aber ich möchte gern das der Wert eben im Array gespeichert wird.

Lieben Dank vorab für eure Hilfe!

 

[Robat]

Du erstellst im Konstruktor ein lokales int[], in das du die Zufallszahl einträgst. In der toString() Methode greifst du aber auf das Array zu, welches du als Instanzvariable anlegst.
Anstatt im Konstruktor ein neues Array zu erstellen, solltest du mit der Instanzvariable arbeiten.

BTW: Nimm das static weg - brauchst du nicht

 

[httpdigest]

Du verwendest static Klassenvariablen, um die Zufallszahlen zu speichern. Das ist genau dasselbe wie static Member in einer C++ Klasse. Ein static Member existiert immer nur einmal pro Klasse und nicht pro Instanz der Klasse.
Lasse einfach das static bei der Deklaration vom zufallsGewicht[] Feld der Klasse weg.
Aktuell würde jeder Konstruktoraufruf die Zufallszahlen wieder überschreiben.

EDIT: @Robat war schneller und seine Antwort ist zudem auch noch korrekt.

 

[Ashtari]

*lach* - okay... Als ich die Antwort von Robat gelesen habe und mir meinen Code angeschaut habe hats klick gemacht... Manchmal braucht man eben einen kleinen Schupser :-D

Dankeschön an euch zwei!
Schönen Abend noch - Hat mir sehr geholfen