Gibt es eine Möglichkeit Builder Code wiederverwenden für die Nachrüstung

Question

I Retrofit und in jeder Aufgabe verwende ich so etwas wie dies zu tun:

public class MyTask extends AsyncTask<String, String, String> { 

    private void someMethod() { 
     final RestAdapter restAdapter = new RestAdapter.Builder() 
      .setServer("http://10.0.2.2:8080") 
      .build(); 
     final MyTaskService apiManager = restAdapter.create(MyTaskService.class); 
    } 

    // ... 

} 

Was ist ein guter Weg, um diesen Code DRY zu machen?

Answer 1

zuerst erklären Sie Eltern-Klasse mit allen gängigen Verhalten

public abstract class MyAbstractTask extends AsyncTask<String, String, String> { 

protected void someMethod() { //note that i change private to protected 
    final RestAdapter restAdapter = new RestAdapter.Builder().setServer("http://10.0.2.2:8080").build(); 
    final MyTaskService apiManager = restAdapter.create(MyTaskService.class); 
} 

} 

dann, können Sie es mit jeder Aufgabe erweitern

public class MyTask extends MyAbstractTask { 

//your someMethod() is available from everywhere in your class 

} 

public class MyOtherTask extends MyAbstractTask { 

//your someMethod() is available from everywhere in your class 

} 

---

aber ich weiß nicht, wo Sie restAdapter und apiManager verwenden , und wenn die tatsächlich einmal pro Aufgabe erstellt werden müssen, da Sie wahrscheinlich außerhalb dieser Aufgaben erstellen können.

Wenn Sie sie außerhalb erstellen, und dann müssen Sie etwas in Ihrer Aufgabe verwenden, ist es auch gut, das Muster Dependency_injection zu berücksichtigen.

---

Außerdem sollten Sie Hardcoding-Werte in Ihren Klassen, wie http://10.0.2.2:8080

sollten Sie verwenden mindestens ein final static final String server= "http://10.0.2.2:8080" und verwenden Sie dann das, oder besser, verwenden Sie einen Setter oder den Konstruktor in der innersten Klasse vermeiden und setzen Sie die Werte von der Aktivität oder dem Hauptcontroller.

Answer 2

Sowohl die RestAdapter als auch die generierte Instanz Ihrer Dienste (MyTaskService in diesem Fall) sind extrem teure Objekte und sollten als Singletons verwendet werden.

Dies bedeutet, dass Sie immer nur einmal restAdapter.create aufrufen und dieselbe Instanz von MyTaskService jedes Mal erneut verwenden sollten, wenn Sie interagieren müssen.

Ich kann das nicht genug betonen.

Sie können das normale Singleton-Muster verwenden, um sicherzustellen, dass es immer nur eine einzige Instanz dieser Objekte gibt, die Sie überall verwenden. Ein Dependency-Injection-Framework wäre auch etwas, das zur Verwaltung dieser Instanzen verwendet werden könnte, wäre aber etwas übertrieben, wenn Sie es nicht bereits nutzen.

Answer 3

Wie Jake sagte, sollten Sie die singleton pattern verwenden, um sicherzustellen, dass immer dieselbe Instanz verwendet wird.

Hier ist ein Beispiel:

ApiManager.getService().listRepos(...);