Zu viele leere Parameter

[RezaScript]

Hallo,

ich versuche somit JSON-Daten zu versenden:

Data data = new Data("nickname", "null", "null", "null", "null", new ArrayList<>(), new ArrayList<>(), "Nickname is already taken");
userSession.getBasicRemote().sendText(gson.toJson(data));

Und meine Klasse Data sieht so aus:

package ch.yourclick.zt.chat;

import java.util.ArrayList;

class Data {
    private String eventType;
    private String timestamp;
    private String nickname;
    private String message;
    private String member;
    private ArrayList<String> members;
    private ArrayList<String> users;
    private String loginError;

    Data(String eventType, String timestamp, String nickname, String message, String member, ArrayList<String> members, ArrayList<String> users, String loginError) {
        this.eventType = eventType;
        this.timestamp = timestamp;
        this.nickname = nickname;
        this.message = message;
        this.member = member;
        this.members = members;
        this.users = users;
        this.loginError = loginError;
    }
}

Es funktioniert zwar aber ich finde es ziemlich umständlich, da ich in jedem Parameter null oder leere Arrays eingeben muss. Gibt es da eine elegantere Lösung?

 

[mihe7]

Zum Beispiel Builder-Pattern oder fluent interface.

 

[kneitzel]

Also ganz trivial wäre:
Erzeuge auch einen Konstruktor ohne (oder nur mit den immer gegebenen Parametern) und den kannst Du dann nutzen und wenn noch etwas mehr gesetzt werden soll, nutzt Du noch einen Setter.

Man könnte auch generell mehrere Konstruktoren anbieten, aber das wird in Java schnell unübersichtlich. (Da sind dann Sprachen wie Kotlin etwas besser, da man hier dann default Werte hinterlegen kann, so dass man nicht alles angeben muss)

Builder-Pattern findet man dann in Java öfters. Das hat @mihe7 ja jetzt schon erwähnt.

 

[mrBrown]

Oder allerdings nicht eine Klasse für alles nutzen, sondern Klassen für die einzelnen Events, die dann auch nur die benötigten Daten enthalten.
Eine Nachricht braucht sicherlich andere Parameter als ein Logout-Event.

 

[RezaScript]

Also könnte ich nicht z.B. 1 Parameter für die Methode Data geben und das wäre ein Array? Dort könnte ich dann überprüfen welche Keys und Values im Array vorhanden sind.

 

[mrBrown]

Also könnte ich nicht z.B. 1 Parameter für die Methode Data geben und das wäre ein Array? Dort könnte ich dann überprüfen welche Keys und Values im Array vorhanden sind.

Könntest du schon.
Alle genannten Varianten sind aber sinnvoller, warum nutzt du denn keine von denen?

 

[RezaScript]

@mrBrown, hat mich nur Wunder genommen. Ansonsten finde ich auch, dass hierbei mehrere Klasse vielleicht sinnvoller wären. Müssen es denn unbedingt mehrere Klassen sein oder kann ich auch in derselben Klasse einfach mehrere Methoden hinzufügen?

 

[mrBrown]

Ansonsten finde ich auch, dass hierbei mehrere Klasse vielleicht sinnvoller wären. Müssen es denn unbedingt mehrere Klassen sein oder kann ich auch in derselben Klasse einfach mehrere Methoden hinzufügen?

Naja, wenn du mehrere Klassen sinnvoller findest, dann sollten das schon mehrere Klassen werden

Eine Klasse mit mehrere Konstruktoren/Fabrikmethoden sind aber auch eine Möglichkeit.

 

[mihe7]

Die Frage wäre für mich, ob diese Klasse irgendeinen tieferen Sinn hat oder es nur darum geht, an unterschiedlichen Stellen unterschiedliche JSON-Texte zu erzeugen.

Wenn ich es richtig sehe, dann dient die Klasse ja lediglich als dummer Datencontainer, um vom JSON-Text zu abstrahieren. Dann kann man das aber auch sehr abstrakt halten, z. B. in Form von Key-Value-Paaren.

JSR-374 (JSON-P) bringt die API dazu gleich mit:

String message = Json.createObjectBuilder()
    .add("eventType", "nickname")
    .add("loginError", "Nickname is already taken")
    .build()
    .toString();

Mit GSON kann man vermutlich ein ähnliches Verhalten erreichen, indem man eine Map serialisiert.