Bestes Muster, um eine util-Klasse zu ersetzen

Question

Ich habe eine class wie util, Es enthält alle vorbereitenden und aufrufenden Anforderungen (Ich möchte einfach Anforderungen verwalten). Ex:

public void reqLogin(String emailAddress, String password) { 
    JsonRequestBody parameters = new JsonRequestBody(); 
    parameters.put(USER_NAME, emailAddress); 
    parameters.put(PASSWORD, password); 
    mService.login(parameters.createRequestBody()); 
} 
public void reqRegister(LocationWrapper location) { 
    JsonRequestBody jsonRequestBody = new JsonRequestBody(); 
    jsonRequestBody.put(DEVICE_TOKEN, "myToken"); 
    jsonRequestBody.put(DEVICE_TOKEN_TYPE, "type"); 
    mService.register(jsonRequestBody.createRequestBody()); 
} 

... 

Aber Problem dieser Konstruktion ist, dass LoginFragment in I Register request nennen kann, und in RegisterFragment kann ich Login request nennen. Also, ist es eine schlechte Lösung, um diese util Klasse zu erstellen? Gibt es ein Muster, um meine issue zu lösen?

Answer 1

Also, ist es eine schlechte Lösung, um diese Util-Klasse zu erstellen?

Util Klassen delegieren die Verarbeitung nicht an andere Klassen. Sie arbeiten an den Eingaben, berechnen etwas und geben das Ergebnis zurück. In Ihrem Beispiel delegieren Sie Aufgaben an Serviceklassen von util. Das ist einfach nicht richtig. Stattdessen sollten Serviceklassen zu util-Methoden aufrufen.

Und Utils sollte keine Geschäftslogik haben. Schauen Sie sich Math Klasse an.

Answer 2

eine der Lösungen könnten zwei verschiedene Schnittstellen für Ihre Klasse sein:

class RequestManager implemets LoginRequest, RegistrationRequest{ 
    @Override //method from LoginRequest 
    public void reqLogin(String emailAddress, String password) { 
     JsonRequestBody parameters = new JsonRequestBody(); 
     parameters.put(USER_NAME, emailAddress); 
     parameters.put(PASSWORD, password); 
     mService.login(parameters.createRequestBody()); 
    } 
    @Override //method from RegistrationRequest 
    public void reqRegister(LocationWrapper location) { 
     JsonRequestBody jsonRequestBody = new JsonRequestBody(); 
     jsonRequestBody.put(DEVICE_TOKEN, "myToken"); 
     jsonRequestBody.put(DEVICE_TOKEN_TYPE, "type"); 
     mService.register(jsonRequestBody.createRequestBody()); 
    } 
} 

andere Lösung könnte nur 2 separate Klassen tun Anmeldung und Registrierung sein.

Wie oben erwähnt, ist diese Klasse keine Utils-Klasse, sondern eher Netzwerkprozessor oder Request Manager.

Answer 3

Um zu vermeiden, GodObject Antipattern, nachdem Sie 60 API-Aufrufe haben, können Sie zum Beispiel so etwas wie Command Entwurfsmuster verwenden.

interface APIOperation { 
    void execute(); 
} 

class LoginAPIOperation implements APIOperation { 
    private final String mEmail, mPass; 

    public LoginAPIOperation(String emailAddress, String password) { 
     mEmail = emailAddress; 
     mPass = password; 
    } 

    public void execute() { 
     JsonRequestBody parameters = new JsonRequestBody(); 
     parameters.put(USER_NAME, mEmail); 
     parameters.put(PASSWORD, mPass); 
     mService.login(parameters.createRequestBody()); 
    } 
} 

new LoginAPIOperation("user@gmail.com","qwerty").execute(); 

Sie können dies in verschiedenen Klassen wiederverwenden.