ArrayList und toString

[Wuast]

Hallo mal wieder,
nach längerer Pause versuche ich wieder zurück zu finden.
Ich wurschtel mich gerade durch die Observer Patterns und möchte dazu eine ArrayList in einer Klasse haben. Einen Singleton habe ich auch - der soll zufällige IDs generieren. Die sollen dann in einer ArrayList abgelegt werden.
Jetzt dachte ich mir, ich gebe die ArrayList mal aus um zu schauen, ob soweit alles geklappt hat. Hat es natürlich nicht.
Kann mir jemand meinen Fehler aufzeigen? Auch für allg. Tipps in Hinblick auf mein Vorhaben bin ich dankbar.

package DesignPattern;

public class Main {

    public static void main(String[] args) {
       
        Nummerierungen nummer = Nummerierungen.getInstance();
       
       
        System.out.println("Die Rechnungsnummer lautet " + nummer.getId());
       
       
        Rechnungsliste rechnungsliste = new Rechnungsliste();
        rechnungsliste.ArrayListVerwalten();
       
        System.out.println(rechnungsliste.toString());
       
    }

}



package DesignPattern;

import java.util.Random;

// Singleton Pattern


public class Nummerierungen {

    Random randomNumber = new Random();
    private int min = 100000000;
    private int max = 999999999;
    private int id = randomNumber.nextInt(max + min) + min; //mind. 9 Ziffern
   
   
   
    private Nummerierungen() {
    }
   
    private static Nummerierungen nummer = new Nummerierungen();
   
   
    public static Nummerierungen getInstance() {
       
        if (nummer == null) {
            nummer = new Nummerierungen();
        }
       
        return nummer;
    }
   
   
    public int getId() {      
        return id;
    }
   
}



package DesignPattern;

import java.util.ArrayList;

public class Rechnungsliste {

   
    public void ArrayListVerwalten() {
   
    Nummerierungen nummer = Nummerierungen.getInstance();
   
    ArrayList <Nummerierungen> rechnungsIds = new ArrayList <Nummerierungen> ();
   
    rechnungsIds.add(nummer);
    rechnungsIds.add(nummer);
    rechnungsIds.add(nummer);
    }

    @override
    public String toString(Nummerierungen rechnungsIds) {
        return String.format("%s", rechnungsIds);
    }
   
   
}

Danke vorab

 

[Jw456]

private int id;

// Keine Berechnung im Klassenrumpf das sollte in die getID Methode.

 randomNumber.nextInt(max + min) + min; //mind. 9 Ziffern

 

[Wuast]

private int id;

// Keine Berechnung im Klassenrumpf das sollte in die getID Methode.

 randomNumber.nextInt(max + min) + min; //mind. 9 Ziffern

erledigt. Danke für den Hinweis

Kommen wir auf das toString zurück..

 

[httpdigest]

Der Code ist syntaktisch und grammatikalisch korrekt, so wie er ist. randomNumber.nextInt(max + min) + min ist einfach nur die Initialisierungs-Expression für die Instanzvariable id der Klasse Nummerierungen.

 

[Jw456]

deine Arrayliste ist ja nur local in der Methode ArrayListVerwalten() du kannst also von ausen nicht darauf zugreifen.
Mache das als Instanzvariable.

 

[Wuast]

Der Code ist syntaktisch und grammatikalisch korrekt, so wie er ist. randomNumber.nextInt(max + min) + min ist einfach nur die Initialisierungs-Expression für die Instanzvariable id der Klasse Nummerierungen.

funktioniert vorher wie nachher einwandfrei. Also ein Missverständnis von JW456?

deine Arrayliste ist ja nur local in der Methode ArrayListVerwalten() du kannst also von ausen nicht darauf zugreifen.
Mache das als Instanzvariable.

das hatte ich zuerst so aber dann beschwert sich Eclipse ob fehlerhafter Syntax bei den Listenfeldern. "error on ' ;'' , @ expected"
das heißt ich muss bei der arraylist immer die felder in die { } setzen ?! dann gehts ja von der Syntax her wieder.

Aber die ArrayListrechnungsIds gibt mir die Konsole trotzdem als Hashwert aus. Stimmt etwas mit dem toString nicht?

 

[Jw456]

    public void ArrayListVerwalten() {
 
    Nummerierungen nummer = Nummerierungen.getInstance();
 
    ArrayList <Nummerierungen> rechnungsIds = new ArrayList <Nummerierungen> ();
 
    rechnungsIds.add(nummer);
    rechnungsIds.add(nummer);
    rechnungsIds.add(nummer);
    }

aber was speicherst du in deiner lokalen Liste immer nur die Instanz deiner Klasse , und nicht eine der zufälligen nummern.

du solltes die liste zurückgenen um sie auch anzeigenlassen zu können in der main.

du zeigst mit System.out.println(rechnungsliste.toString()); nur die Instanz an nicht die liste.

 

[KonradN]

Ein paar kleine Punkte:

a) Bitte immer genau schreiben, was nicht geht: Was passiert, was erwartest Du?

b) Wenn ein Objekt ausgegeben werden soll, dann muss man die toString() Methode überschreiben. Sonst wird nur die die Klasse@HashCode ausgegeben.

c) Du hast eine ArrayList von Nummerierung und packst die eine Nummerierung mehrfach hinein. Deine Klasse Nummerierung ist ja keine Nummerierung sondern ein NummernGenerator. Und da willst Du dann eine Methode "generate" haben.

 

[KonradN]

du zeigst mit System.out.println(rechnungsliste.toString()); nur die Instanz an nicht die liste.

Nein, das ist falsch! Seine Rechnungsliste Klasse überschreibt toString und in dieser gibt es die ArrayList aus. Das ist also soweit korrekt.

Nur eben seine Elemente sind von einer Klasse, die toString nicht überschreibt, so dass er eben in der ausgegebenen Liste nur Klassenname@Hashcode sieht. (Und da mehrfach den gleichen Hashcode, da ja mehrmals eine Referenz hinterlegt wurde.

 

[httpdigest]

Seine Rechnungsliste Klasse überschreibt toString und in dieser gibt es die ArrayList aus. Das ist also soweit korrekt.

Das ist leider auch falsch. Seine Rechnungsliste Klasse überschreibt toString() NICHT!
Er erzeugt eine eigene Methode mit Signatur public String toString(Nummerierungen rechnungsIds)
Witzigerweise sollte hier aber die IDE auch bei der @Override Annotation meckern (die auch falsch geschrieben ist).

 

[KonradN]

Stimmt, das hatte ich tatsächlich beim überfliegen des Codes übersehen.

 

[Jw456]

ohne IDE in etwa so

public class Rechnungsliste {

    ArrayList <int> rechnungsIds;
   
    public void ArrayListVerwalten() {
 
    Nummerierungen nummer = Nummerierungen.getInstance();
 
    rechnungsIds = new ArrayList <> ();
 
    rechnungsIds.add(nummer.getId());
    rechnungsIds.add(nummer.getId());
    rechnungsIds.add(nummer.getId());
   
    }

    @Override
    public String toString( )
      String st = "";
      for (int i : rechnungsIds) {
        st = st + i + "/n";
      }
      return st;
    }
 
 
}

das es drei geiche sind wissen wir ja das sollte er dann machen

 

[KonradN]

Eine kleine Verbesserung (Ehe @Wuast daran hängen bleibt): Bei Generics ist ein int nicht erlaubt - da muss also die Wrapper-Klasse herhalten: ArrayList<Integer>

 

[Jw456]

Ok ja stimmt. Hatte halt nur einen Editor keine IDE zum tippen. Danke.

 

[Wuast]

Ein paar kleine Punkte:

a) Bitte immer genau schreiben, was nicht geht: Was passiert, was erwartest Du?

b) Wenn ein Objekt ausgegeben werden soll, dann muss man die toString() Methode überschreiben. Sonst wird nur die die Klasse@HashCode ausgegeben.

c) Du hast eine ArrayList von Nummerierung und packst die eine Nummerierung mehrfach hinein. Deine Klasse Nummerierung ist ja keine Nummerierung sondern ein NummernGenerator. Und da willst Du dann eine Methode "generate" haben.

a) Hatte ich das nicht?
b) Genau, soweit waren wir ja auch in meinem letzten Thread zum Thema toStrings schon... ich weiß auch nicht, warum ich mich so schwer tue mit dem toStrings-Überschreiben. Genau das wollte ich nämlich machen in der Klasse Rechnungsliste
c) danke für den Hinweis,du hast natürlich völlig recht. Nur für mein Verständnis: Ich habe die Klasse als Singleton-Pattern gedacht, die ich jetzt im ganzen Projekt aufrufen kann, wenn ich eine neue ID brauche. Die getter spuckt mir dann doch jedes mal eine Zufalls-ID aus. Jetzt muss ich doch letztenendes nur dafür sorgen, dass die Ids im Array gespeichert werden und eine neue Position bekommen. - Soweit zumindest die Theorie in meinem Kopf. Oder ist das Quatsch?

Das ist leider auch falsch. Seine Rechnungsliste Klasse überschreibt toString() NICHT!
Er erzeugt eine eigene Methode mit Signatur public String toString(Nummerierungen rechnungsIds)
Witzigerweise sollte hier aber die IDE auch bei der @Override Annotation meckern (die auch falsch geschrieben ist).

Sie meckert, wenn ich sie richtig schreibe

ohne IDE in etwa so

public class Rechnungsliste {

    ArrayList <int> rechnungsIds;
  
    public void ArrayListVerwalten() {
 
    Nummerierungen nummer = Nummerierungen.getInstance();
 
    rechnungsIds = new ArrayList <> ();
 
    rechnungsIds.add(nummer.getId());
    rechnungsIds.add(nummer.getId());
    rechnungsIds.add(nummer.getId());
  
    }

    @Override
    public String toString( )
      String st = "";
      for (int i : rechnungsIds) {
        st = st + i + "/n";
      }
      return st;
    }
 
 
}

das es drei geiche sind wissen wir ja das sollte er dann machen

also quasi das, was ich oben schon angedacht hatte - nämlich in der ArrayList die zufällig generierten IDs jeweils einem neuen Feld zuweisen?! Dann müsste die toString (so wie du sie formuliert hast) ja über die List laufen und alle ausgeben?

Eine kleine Verbesserung (Ehe @Wuast daran hängen bleibt): Bei Generics ist ein int nicht erlaubt - da muss also die Wrapper-Klasse herhalten: ArrayList<Integer>

Also nicht so wie ich es angedacht habe die ArrayList<Nummerierungen> ... sondern als <Integer>? Wie kriege ich denn dann das Objekt Zufalls-ID in die ArrayList? :/

Danke für alle Antworten!!

 

[Wuast]

public class Rechnungsliste {

    ArrayList <int> rechnungsIds;
 
    public void ArrayListVerwalten() {
 
    Nummerierungen nummer = Nummerierungen.getInstance();
 
    rechnungsIds = new ArrayList <> ();
 
    rechnungsIds.add(nummer.getId());
    rechnungsIds.add(nummer.getId());
    rechnungsIds.add(nummer.getId());
 
    }

    @Override
    public String toString( )
      String st = "";
      for (int i : rechnungsIds) {
        st = st + i + "/n";
      }
      return st;
    }

[/QUOTE]

Wobei ich hier noch feststellen muss, dass bei der Methodenüberschriebung das Element rechnungsIds in der for-Schleife nicht auf die ArrayList rechnungsIds zugreifen kann. Einfach überladen geht auch nicht

    @Override
    public String toString (<Integer> rechnungsIds) {    //Syntax error token on "(", type expected after this token
 
      String st = "";
      for (int i : rechnungsIds) {
        st = st + i + "/n";
      }
      return st;

 

[KonradN]

Gehen wir den Code einfach einmal Stück für Stück durch.

1. Deine Klasse Nummerierungen. Was macht diese Klasse eigentlich? Doch in erster Linie das Generieren von Ids. Also machen wir daraus einfach mal eine entsprechende Klasse IdGenerator:

package DesignPattern;

import java.util.Random;

// Singleton Pattern


public class IdGenerator {

    Random randomNumber = new Random();
    private int min = 100000000;
    private int max = 999999999;



    private IdGenerator() {
    }

    private static IdGenerator nummer = new IdGenerator();


    public static IdGenerator getInstance() {

        if (nummer == null) {
            nummer = new IdGenerator();
        }

        return nummer;
    }


    public int getNextId() {
        return randomNumber.nextInt(max - min) + min; //mind. 9 Ziffern;
    }

}

(Noch paar Hinweise: Hier bräuchte es kein Singleton - das könnte auch eine Utility-Klasse sein und getNextId eine statische Methode. Und das Singleton ist nicht Thread sicher. Und zu guter letzt: Ich habe da aus dem max+min ein max-min gemacht - das dürfte das sein, was Du vermutlich wolltest.)

Da braucht man dann auch keine toString Methode oder so.

2. Rechnungsliste

package DesignPattern;

import java.util.ArrayList;

public class Rechnungsliste {

    private ArrayList <Integer> rechnungsIds = new ArrayList <> ();

    public void ArrayListVerwalten() {

        IdGenerator idGenerator = IdGenerator.getInstance();
        
        rechnungsIds.add(idGenerator.getNextId());
        rechnungsIds.add(idGenerator.getNextId());
        rechnungsIds.add(idGenerator.getNextId());
    }

    @Override
    public String toString() {
        return String.format("%s", rechnungsIds);
    }
}

Was gibt es hier an Änderungen:
a) rechnungsIds ist nun eine Instanzvariable. Wenn Du die Variable innerhalb der Konstruktors deklarierst, dann ist es eine lokale Variable und nur innerhalb des Blockes (also bei Dir der Konstruktor) gültig. So ist die Liste rechnungsIds innerhalb der Instanz von Rechnungsliste dauerhaft gültig.
b) Die rechnungsIds speichert nun wirklich die Ids. Das sind also Integer Werte.
c) @override wird mit großem O geschrieben -> @Override
d) Die Methode toString hat keinem Parameter - es wird jetzt also schlicht die Instanzvariable ausgegeben.

Noch nicht angepasst: ArrayListVerwalten sagt nicht aus, was die Methode macht und startet mit einem großen Buchtaben. Das könnte also etwas sein wie "addThreeMoreIds" oder so.

Dann bleibt nur noch die Main:

package DesignPattern;

public class Main {

    public static void main(String[] args) {

        IdGenerator nummer = IdGenerator.getInstance();

        System.out.println("Die Rechnungsnummer lautet " + nummer.getNextId());


        Rechnungsliste rechnungsliste = new Rechnungsliste();
        rechnungsliste.ArrayListVerwalten();

        System.out.println(rechnungsliste);
    }

}

Hier war eigentlich alles schon soweit ok. getId() ist ja zu getNextId() geworden. Und bei der Ausgabe muss man kein explizites toString() aufrufen - es reicht ein System.out.println(rechnungsliste);

Und wenn man das nun aufruft bekommt man eine Ausgabe wie:

Die Rechnungsnummer lautet 883960219
[595271656, 697488971, 930262368]

Noch ein paar Hinweise:

- packages bestehen in der Regel nur aus kleinen Buchstaben, also statt DesignPattern eher designpattern.

- Wenn man eine Methode überschreiben will, dann nutz ruhig die Entwicklungsumgebung dafür. Also in IntelliJ kann ich sagen, dass ich Code generiert bekommen möchte -> Methode überschreiben und dann zeigt er mir an, welche Methoden ich überschreiben könnte. Und dabei gibt es dann auch noch so Optionen wie Übernehmen der JavaDoc Kommentare - dann hat man auch in der Regel direkt die Beschreibung, was auch weiterhin für die Methode gelten sollte.

 

[Wuast]

Super, danke!! Das klingt gut, viele Tipps und wie immer sehr verständlich.

Und die ArrayList kann ich ja nun auch dynamisch über Benutzereingaben erweitern, oder? --> Dazu würde ich dann einfach ein Objekt der Klasse erzeugen, die Methode aufrufen und dann .add(get.nextId();, korrekt?

Dann würde ich dafür eine seperate Klasse schrieben oder kommt das in die Main? Soweit ich das bisher verstanden habe empfiehlt sich eine seperate Klasse mit einer entsprechenden Methode? Deswegen habe ich die Methode anfangs auch arrayListVerwalten genannt, eben in der Absicht, diese dann aufzurufen und über eine Konsoleneingabe Elemente zu addieren, zu löschen oder zu verändern (was nat. bei den Ids erstmal quatsch ist, das rührt noch aus einer Übungsaufgabe her - ebenso der Singleton-Pattern, den habe ich einfach für die Folgeübung behalten)