Wie heißt dieses Anti-Muster?

Question

Sagen wir, ich habe eine Funktion send_welcome_email() und eine Klasse namens User genannt (implementiert in Python, aber hoffentlich einfach für Nicht-Python-Entwickler zu verstehen):

class User: 
    email = TextField() 
    first_name = TextField() 
    last_name = TextField() 


def send_welcome_email(user): 
    msg = EmailMessage(recipient=user.email) 
    msg.send() 

In diesem Fall wäre es das gewesen, besser zu definieren haben Funktionsschnittstelle als:

def send_welcome_email(email) 

so dass die Funktion mit der User Klasse nicht gekoppelt.

Ich weiß, das ist ein bekanntes Anti-Pattern, aber ich kann es nicht finden. Wer weiß, wie es heißt?

Answer 1

Ich glaube nicht, dass es einen Namen für das Anti-Muster gibt, aber die Regel, die dies verletzt, wird Law of Demeter genannt.

LoD kann als das Prinzip angesehen werden, "geringstes strukturelles Wissen" anzunehmen (etwas, das sein Schöpfer "strukturscheue Programmierung" nennt). Die Idee besteht darin, Kenntnis von der inneren Struktur eines Objekts anzunehmen, die sich von Ihrem eigenen unmittelbaren Selbst unterscheidet.

So wie ich gehört habe es beschrieben: in einem Geschäft, wenn Sie einen Kassierer Sie Ihre Kreditkarte aus der Brieftasche bekommen bezahlen und es ihnen geben, anstatt die Brieftasche übergeben und lassen Sie sie durch sie kramen. Ihr Beispiel zeigt den Suchansatz. Die Funktion ruft das Benutzerobjekt ab und sie muss wissen, wie sie auf die E-Mail-Eigenschaft zugreifen kann, über die sie nicht informiert werden sollte.

Answer 2

Es klingt wie die Anemic Domain Model.

Answer 3

Ist nicht der ganze Zweck von send_welcome_email eine E-Mail für den speziellen Zweck der Begrüßung des Benutzers zu generieren (ich brauche Informationen über den Benutzer wie Name, Benutzername, E-Mail-Adresse)?

Ihre Alternative tun Sie das nicht.

Die Methode sollte wahrscheinlich nicht in der Benutzerklasse aber etwas wie UserEmailSender oder ähnlich sein.

Meine Antwort ist daher nein: Es sieht nicht wie ein Anti-Muster aus.

Answer 4

Die Funktion hat einen Hauch von the Feature Envy code smell. ("Code smells" sind Antipattern, dokumentiert in einem Kapitel von Fowler und Beck in , genannt, bevor das Wort "antipattern" weit verbreitet war.) Feature Envy ist, wenn ein Modul an ein anderes Modul gekoppelt ist, indem zu viele andere aufgerufen werden Methoden des Moduls. In diesem Fall ist einer zu viel.

Die übliche Lösung ist, eine Methode auf das andere Modul zu extrahieren, aber das trifft hier nicht zu, da es nur einen stinkigen Methodenaufruf gibt. Die Fehlerbehebung, den Aufrufer der Funktion anzurufen, die Methode stattdessen aufzurufen, funktioniert, solange es für den Aufrufer OK ist, diese Kopplung mit der Klasse User zu haben.

Dies erinnert an die Law of Demeter, eine Richtlinie, die Kopplung reduziert, aber das gilt am spezifischsten für einen Aufrufer Aufruf einer Methode auf das Ergebnis der Aufruf einer anderen Methode, die hier nicht passiert.