[JAVA] Es fühlt sich gut an, unterschiedliche Klassen für dieselben Tabellenwerte zu haben

Einführung

Fassen Sie die Geschichte zusammen, dass es nicht ausreichte, dieselbe Klasse zu verwenden, nur weil die Werte gleich sind.

Schreiben wir drei leicht unterschiedliche Beispiele. Ich habe den Verarbeitungsinhalt entsprechend geändert, aber die Konfiguration so angepasst wie der Teil, in dem ich mit dem von mir gepflegten Dienst zu kämpfen hatte.

Eigentlich wird das Klassendesign basierend auf der Idee von DDD entschieden, aber dieses Mal werde ich nur sagen, dass es ziemlich gut sein wird, wenn man nur den Typ entwickelt.

Artikel Prämisse

• Es ist kein Framework, das eine Primärschlüsselklasse wie aktive Datensätze vorbereitet
• Der gebuchte Code ist super-super vereinfacht und nur die wichtigsten Punkte werden extrahiert.
• Alle haben tatsächlich mehr Parameter und die Klassen sind kompliziert unterteilt
• Der Prozess ist auch komplizierter, da E-Mails übersprungen und die Rechnungsverarbeitung eingeschlossen werden
• Andere Klassen als "Service" und "Tabelle" werden der Einfachheit halber als "Domänen" bezeichnet.
• "Domains" dürfen nicht von "Service" oder "Tabelle" abhängen (ziehen Sie keinen Pfeil von "Domains" zu etwas anderem als "Domains").

Fall 1 Behandeln Sie zwei Primärschlüssel

Angenommen, ein an einen Mitarbeiter verliehenes Firmenhandy ist defekt und wird ersetzt. Erhalten Sie die aktuell verwendete Seriennummer und ersetzen Sie sie durch die neu vorbereitete Seriennummer.

Klassen und Implementierungen

(SerialNumber) und (SerialNumber, SerialNumber) sind anfällig für Fehler.

Da die beiden Seriennummern nur durch den Variablennamen unterschieden werden, kann sie auch mit "(geändert, geändert)" kompiliert werden. .. Dann, wahrscheinlich durch Durchsuchen der Daten mit dem ersten Argument, fühlt es sich wie eine Ausführungsausnahme an, wie "Eine solche Seriennummer ist nicht ausgeliehen". (Abgesehen davon ist mir gerade aufgefallen, dass ich einen Fehler machen könnte, wenn ich ihn mit der Ergänzung des Editors schreibe, weil sowohl "geändert" als auch "aktuell" von "c" stammen. Es ist unerwartet, dass ein solcher Fehler ein langes Wort ist.)

Refactor

Da "Domänen" nur eine Zahlenklasse hat, ist die Handhabung einfach und bezieht sich auf die Methode zur Identifizierung aller Teile, die hart mit dem Variablennamen "aktuell" und "geändert" mit den Klassen "aktuell" und "geändert" arbeiten. ..

Es fühlt sich gut an.

tatsächlich

Schließlich wird es in dieselbe Spalte derselben Tabelle verschoben. Wenn Sie also zu der Ebene gehen, die so nah wie möglich an der Datenbank liegt, können Sie aus "CurrentSerialNumber" eine "SerialNumber" erstellen, um den Abstraktionsgrad zu erhöhen.

"SerialNumber" sollte jedoch in der Nähe von "Table" anstelle von "Domains" erstellt werden und "SerialNumber.from (CurrentSerialNumber)" anstelle von "CurrentSerialNumber.toSerialNumber ()" sein. Wenn Sie es mit "to" generieren, hängt "Domains" von "Table" ab, und "Domains" ermöglichen die Verwendung von "SerialNumber".

Fall 2 Aktualisieren Sie den Status einer Klasse

Angenommen, es gibt ein Mitgliederverwaltungssystem, das Mitgliederabhebungen akzeptiert. Stellen Sie beim Austritt aus der Mitgliedschaft den Status als zurückgezogen ein und geben Sie das leere Rücktrittsdatum ein, um den Status zu aktualisieren.

Klassen und Implementierungen

Es ist schwierig, sich der Konsistenz von "Status" bewusst zu sein, und "0..1" ist sehr anfällig für Ausführungsausnahmen.

Ich weiß nicht, welches ** Mitglied ich von find (id) bekommen habe, also überprüfe ich es.

Selbst wenn der Mitgliedsstatus zurückgezogen und dauerhaft gemacht wird, wird danach etwas, das nicht zurückgezogen wird, dauerhaft, wenn die Einstellung falsch ist.

Refactor

Wenn ein solches Zustandsaktualisierungssystem für jeden Staat in eine andere Klasse eingeteilt wird, kann eine beträchtliche Anzahl von Landminen beseitigt werden.

"Nicht abonnierte Mitglieder" können nur aus "in Gebrauch befindlichen Mitgliedern" erstellt werden, "in Gebrauch befindliche Mitglieder" haben nicht immer ein Widerrufsdatum und "zurückgezogene Mitglieder" haben immer ein Widerrufsdatum.

Sie können darauf vertrauen, dass das Ergebnis von "find (id)" ein "Mitglied in Verwendung" ist. (Natürlich ist Table stattdessen für die Prüfung verantwortlich)

Da "Table.unSubscribe ()" nur eine Methode zum Abbestellen ist, muss "Status.UnSubscribe" nicht mehr übergeben werden, und der Einstellungswert für die Systemfreundlichkeit stammt nicht mehr von "Domains".

tatsächlich

Es gibt verschiedene Situationen wie Antragstellung, Änderung der Vertragsform, Nichtzahlung von Gebühren, Einfrieren usw., und nicht nur Mitglieder, sondern auch verschiedene Klassen können nicht ohne Management auskommen. Wenn Sie versuchen, alle Zustände ordnungsgemäß nur mit dem Einstellungswert in derselben Klasse zu verwenden, ist dies so schmerzhaft, dass die Ausführungsausnahme stirbt, da sie mit der Inkonsistenzprüfung des Einstellwerts und der Prüfung "Optional" abgedeckt ist.

Selbst wenn ich eine getippte Sprache verwende, kann ich nicht zuerst kompilieren, also habe ich das Gefühl, welche Sprache ich schreibe ... ??

Hier gibt es verschiedene Designs, und angesichts des Eventdesigns wird es zu lang sein, also habe ich es gerade vorgestellt. (Eher studiere ich noch Eventdesign ...)

Fall 3 Haben Sie eine gemeinsame Elternklasse

Angenommen, alle Benutzeroperationen müssen als Operationsverlauf aufgezeichnet werden. Wenn Sie einen Kauf tätigen, speichern Sie den Verlauf der Kaufanwendung. Wenn abgebrochen, aktualisieren Sie den Anwendungsverlauf auf storniert und erstellen Sie einen Abbruchvorgang.

Es ist kompliziert, aber es sieht so aus.

Klassen und Implementierungen

Da "Anwendung" und "Stornierung" unterschiedlich behandelt werden, werden die Klassen getrennt. Es ist jedoch auch zweckmäßig, sie für allgemeine Zwecke als Operationsverlauf zu behandeln. Dies ist beispielsweise ein Beispiel für das Erstellen einer übergeordneten Klasse.

Da dies kompliziert ist, gibt es viele Einstellwerte, und da alles als ".save ()" gespeichert ist, besteht die Möglichkeit, dass der Wert und die Häufigkeit falsch sind.

Refactor

Um die Anwendung und den Abbruch beim Kompilieren beim Verlassen der gemeinsamen Klasse korrekt zu behandeln, scheint es gut, die übergeordnete Klasse zu trennen.

Wenn Sie eine dedizierte Methode wie die Auszahlung verwenden, können Sie den eingestellten Wert fast eliminieren. Durch Ändern von ".save ()" in ".cancel (angewendet, abgebrochen)" können Sie die beiden immer korrekt aktualisieren.

tatsächlich

Schließlich fällt der Operationsverlauf ab, um abzubrechen oder abzuschließen. Nach der Bewerbung können Sie es verwenden, nachdem Sie bestätigt haben, dass es angekommen ist. Der Grund für die Erstellung eines separaten Stornierungsverlaufs besteht darin, dass von nun an Rückerstattungen akzeptiert werden müssen und die Stornierung abgeschlossen ist, wenn die erforderliche Verarbeitung abgeschlossen ist.

In Bezug auf das Bild sieht die endgültige Form so aus.

Grundsätzlich liegt es in dieser Form vor, es gibt jedoch ebenso viele Betriebsverläufe wie Dienste, z. B. das Aktualisieren der Mitgliederinformationen und den Kauf zusätzlicher Geräte.

Es ist äußerst schwierig, diese jedes Mal korrekt zu aktualisieren, daher möchte ich die Kompilierungsleistung erhalten, während ich einige übergeordnete Klassen vorbereite.

Zusammenfassung

Grundlegende Richtlinien

• Beseitigen Sie den Einstellungswert selbst, indem Sie anstelle des Einstellwerts eine dedizierte Klasse oder Methode vorbereiten
• Denken Sie, dass es in Ordnung ist, wenn der Variablenname einfach ist
• Stellen Sie sicher, dass Variablen desselben Typs nicht im selben Bereich vorhanden sind

Ein Zeichen, das gefährlich zu sein scheint

• Methode hat mehrere Argumente des gleichen Typs
• Erstellen Sie vorerst eine übergeordnete Klasse
• Auch wenn Sie es versehentlich an die Methode übergeben, wird es kompiliert.
• Um dies zu verhindern, müssen Sie mit "Instanz von" wirken, aber das ist auch subtil
• Es ist ein gängiges Muster, ein "Mitglied" als Elternteil eines "verwendeten Mitglieds" und eines "Mitglieds, das sich zurückgezogen hat" zu erstellen, aber es ist besser, zu beenden.
• Versuchen Sie, den Zweck durch den Variablennamen zu vermitteln
• Viele Einstellungen

Bewältigung

• Erstellen Sie immer mehr Klassen für jede Spezifikation
• Sie könnten denken, dass die Anzahl der Klassen zunehmen würde, aber es sollte erhöht werden.
• Es ist zu schmerzhaft, alle Attribute in einer Summe zu halten
• Wenn Sie versuchen, alles mit "Benutzer" zu tun, müssen Sie "Benutzer" immer reparieren, unabhängig von Änderungen an den Spezifikationen, und Sie werden am Ende alle Spezifikationen erneut testen.
• Das Erstellen von "XxxUser" für jede Spezifikation hat einen geringeren Einflussbereich
• Entscheiden Sie, wo die Ausführungsausnahme auftreten soll, und drücken Sie sie dort hinein
• Ich habe den Code dieses Mal weggelassen, aber da "Tabelle" Dateninkonsistenz und Kommunikationsunterbrechung aufweist, werde ich die Ausnahmeprüfung ordnungsgemäß durchführen
• Service macht das, was es basierend auf den Spezifikationen tun sollte. Konzentrieren Sie sich also auf das Verhalten, anstatt nach Ausführungsausnahmen zu suchen
• Berücksichtigen Sie Attribute, indem Sie sie in Parameter unterteilen, die basierend auf Benutzereingaben und Parametern, die für den Systemkomfort erforderlich sind, korrekt eingestellt werden sollten.
• Denken Sie, dass die Informationen ausreichend und typsicher sind, auch wenn der Variablenname einfach ist

Was als Grundhaltung zu tun

• Ich denke, dass Fehler auftreten werden, bei denen Sie Fehler machen können
• Das einzige Mal, dass ich denke, dass ich einen solchen Fehler machen kann, ist der Moment, in dem ich ihn schreibe
• Vergessen Sie nicht, dass das Beispiel nur sehr vereinfacht ist und tatsächlich irgendwo im Meer des Codes vergraben ist.
• Wenn ein solcher Code von einer anderen Person stammt, ist es ziemlich schmerzhaft zu verstehen, es sei denn, Sie lesen verschiedene Dinge unerwartet
• Gehen Sie herum und denken Sie, dass Sie sich in Zukunft nicht trauen können
• Zuverlässiger Code kann nicht verwechselt werden

So ähnlich.