Attribut Telefonnummer - eigene Klasse oder String

[DrPils]

Moin
Für eine Telefonbuch entwerfe ich eine Klasse Contact, bei dem Attribut phoneNumber stelle ich mir jetzt die Frage ob es guter Stil wäre, diese doch in einer eigene Klasse abzubilden statt einfach nur ein String. Ich finde aber keine Argumente die für das eine oder andere Vorgehen sprechen.

Pro eigene Klasse: Mit der Klasse PhoneNumber könnte ich dort festlegen was eine Telefonummer ist.
Contra String: Mit einem String wäre es erstmal eine Zeichenkette und ich müsste eigentlich dort wo ich sie verwende immer schauen, dass es eine Telefonnummer ist.

Wie würdet ihr das lösen?

 

[Kirby.exe]

Naja du könntest es ein eine eigene Klasse auslagern und für die Vorwahl der Telefonnummer ein Enum benutzen
Das wäre zu dem auch Hilfreich, da meines Wissens jede Telefonnummer in einer Datenbank des Staats an eine Person geknüpft ist. Dies war früher bei z.B. Prepaid Karten nicht so

 

[DrPils]

Naja du könntest es ein eine eigene Klasse auslagern und für die Vorwahl der Telefonnummer ein Enum benutzen

Ist das dein Ernst?
5202 Ortsvorwahlen +37 Mobilfunkvorwahlen und das nur für Deutschland....

 

[Kirby.exe]

Ist das dein Ernst?
5202 Ortsvorwahlen +37 Mobilfunkvorwahlen und das nur für Deutschland....

Du kannst die auch aus irgendeiner DB ziehen

 

[MoxxiManagarm]

Wie würdet ihr das lösen?

Telefonnummern können sehr komplex werden. Ggf. willst du dir dafür eine Lib anschauen, https://github.com/google/libphonenumber kommt auch mit einer `PhoneNumber` Klasse

 

[DrPils]

Du kannst die auch aus irgendeiner DB ziehen

Achso
ich möchte es erstmal nicht mit einer DB machen.
Aber das Enum muss doch sowieso geschrieben werden? Also ich kann doch nicht zur Laufzeit ein Enum von einer DB erstellen?

Telefonnummern können sehr komplex werden. Ggf. willst du dir dafür eine Lib anschauen, https://github.com/google/libphonenumber kommt auch mit einer `PhoneNumber` Klasse

Danke schau ich mir mal an. Also eher die eigene Klasse statt den String?

 

[Kirby.exe]

Aber das Enum muss doch sowieso geschrieben werden? Also ich kann doch nicht zur Laufzeit ein Enum von einer DB erstellen?

Not really du kannst dir die Vorwahlen direkt per SQL Anfrage holen :

SELECT VORWAHL FROM WHATEVER WHERE COUNTRY = 'Germany' and CITY = 'Berlin';

 

[Thallius]

Telefonnummer sollte vor allem nicht nur eine Klasse oder ein String sein sondern eine Liste davon. Schließlich kann ein Kontakt beliebig viele Telefonnummern besitzen

 

[DrPils]

Also meine Ursprüngliche Frage war ja ob ich es als eigene Klasse umsetzen soll, nicht wie.
Habe ich das richtig verstanden dass ich es als eigene Klasse umsetzen sollte?

 

[LimDul]

Meine Frage: Was ist im Kontext deiner Anwendung eine Telefonnummer? Nur ein String der irgendwo angezeigt werden soll? Oder passieren da fachliche Dinge damit (muss man z.B. feststellen, dass 0049 12345 die gleiche Nummer ist wie +4912345). Muss man die verschiedene Formate normalisieren (0049 => +49). Muss man Landes/Ortsvorwahl ggf. separat anzeigen können? Muss man prüfen können ob ein eingegebener Text eine gültige Telefonummer ist?

Je mehr dieser oder ähnlicher Fragen du mit "JA" beantwortest um so mehr spricht für eine eigene Klasse.

 

[LimDul]

Um vielleicht mal zwei Varianten darzustellen aus dem beruflichen Umfeld.

* Ein System was Personen-Stammdaten verwendet um diese zu verwalten mit ihren Kontakt-Information: Hier würde ich für eine Klasse PhoneNumber plädieren
* Ein System was das System für die Personen-Stammdaten anbindet und die Daten die daraus kommen nur zur Anzeige in irgendwelchen Masken nutzt => Hier würde ich die PhoneNumber als String ablegen

 

[DrPils]

Also es soll einfach ein ganz billiges Telefonbuch werden, wo ich Einträge in einer csv Datei speichern und laden kann.
Die Anzeige erfolgt über eine JavaFX gui.
Normalisierung der Vorwahlen möchte ich umsetzen.

 

[thecain]

Aus DDD Sicht macht es sowieso Sinn eine eigene Value Class dafür zu machen. Meiner Meinung nach verlierst durch eine eigene Klasse nichts, hast aber ggf Vorteile bei der Weiterentwicklung

 

[MoxxiManagarm]

Normalisierung der Vorwahlen möchte ich umsetzen.

Ich würde sagen ja, nimm eine Klasse sobald du mit der Telefonnummer Logik (hier Normalisierung) verbindest.