Android 应用程序架构我这样做正确吗？

Question

我在此处遵循此指南https://developer.android.com/jetpack/docs/guide?hl=en

为了学习关注点分离，我创建了一个简单的应用程序，该应用程序从 API 中获取一些数据并将其以 activity.

的形式显示在屏幕上

我的activity是下面它的职责是向用户显示信息。

public class MainActivity extends AppCompatActivity implements View.OnClickListener {
    private Button myButt;
    private MainViewModel mvw;
    private TextView myView;
    private MutableLiveData<Orders> mld;


    @Override
    protected void onCreate(Bundle savedInstanceState) {
        mvw = new ViewModelProvider(this).get(MainViewModel.class);

        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        myButt = findViewById(R.id.button);
        myView = findViewById(R.id.textview);
        myButt.setOnClickListener(this);

        mvw.getMld().observe(this, new Observer<Orders>() {
            @Override
            public void onChanged(Orders orders) {
                Log.i("livedata","got an updt");
                myView.setText(mvw.extractDate(orders));
            }
        });
    }

    @Override
    public void onClick(View view) {
        switch(view.getId()){
            case R.id.button:
                Log.i("tag","hello");
                mvw.updateData();
        }
    }
}

下面是 ViewModel，它关注的是从存储库中获取数据 class 并在将数据显示在 activity 之前对该数据进行一些计算。

public class MainViewModel extends ViewModel {
    private GetOrder getOrderRepo;
    private MutableLiveData<Orders> mld;

    public MainViewModel(){
        getOrderRepo = new GetOrder();
        mld = getOrderRepo.getAllOrders();
    }

    public MutableLiveData<Orders> getMld() {
        return mld;
    }

    public void setMld(MutableLiveData<Orders> mld) {
        this.mld = mld;
    }





    public void updateData(){
        getOrderRepo.getAllOrders(); //discard the return value
    }

    public String extractDate(Orders orders){
        ArrayList<Order> listOfOrders = orders.getOrders();
        String date = listOfOrders.get(0).getOrderTime();
        return date;
    }

}

接下来是存储库，它处理来自 API 的 GET 请求并将其放入 MutableLiveData 容器 "allOrders"

public class GetOrder {
    private ApiService mAPIService;
    MutableLiveData<Orders> allOrders;
    private Orders orders;
    public GetOrder(){
        mAPIService = ApiUtils.getAPIService();
    }
    public MutableLiveData<Orders> getAllOrders(){
        Log.i("func","starting func");
        allOrders = new MutableLiveData<Orders>();
        mAPIService.getOrders().subscribeOn(Schedulers.io()).observeOn(AndroidSchedulers.mainThread())
                .subscribe(new Subscriber<Orders>() {
                    @Override
                    public void onCompleted() {
                        Log.i("func","onComplete");

                    }

                    @Override
                    public void onError(Throwable e) {
                        Log.i("onError",e.toString());

                    }

                    @Override
                    public void onNext(Orders orders) {
                        Log.i("Repo",orders.toString());
                        allOrders.setValue(orders);
                    }
                });
        return allOrders;
    }
}

这是正确的实施方式吗？还是我误解了什么？

我担心的一个问题是按钮有 mvw.orderData().observe(this, new Observer() 因为这每次都会创建一个新的观察者。还是在每次 onChanged 后它都会死？

反馈后更新

Answer 1

ViewModel 应负责存储数据。

因此，更好的方法是在 ViewModel 而不是 Repository 中使用 MutableLiveData<Orders> allOrders;。这样 Repository 只负责 get 数据。它不应该存储它。

然后，在您的 Activity 中，您应该像这样观察 allOrders 实时数据：mvw.allOrders.observe(...) 在 onCreate() 中。

然后单击按钮，您只需使用 ViewModel 的方法 orderData().

将 Repository 方法调用到 get/update allOrders LiveData

Answer 2

您在这里实现的是 MVVM 架构。模型（存储库）-视图（Activity）-VM（视图模型）。基本结构看起来不错，但我发现您可以改进一些地方以更好地遵循最佳实践。

如您所说，每次单击 Button 都会创建一个新的 Observer。您应该改为在 onCreate 中观察 LiveData。在您的 ViewModel 中添加一个函数，告诉存储库更新数据并在 onClick 方法中调用它。
您应该将 MutableLiveData 实现为私有并将其包装在一个 LiveData 中，然后在 Activity 中观察。所以它不会发生突变。此外，ViewModel 应该存储获取的 MutableLiveData。

更新(2): 检查 MainActivityViewModel under this link。您在 ViewModel 中设置的 MutableLiveData 应该是私有的。

 private MutableLiveData<List<String>> fruitList;

然后通过普通 LiveData 返回 MutableLiveData。这个 LiveData 不是可变的，不能为 null，因此可以安全地暴露给其他类，比如您的 MainActivity，您可以在其中观察它。

LiveData<List<String>> getFruitList() {
        if (fruitList == null) {
            fruitList = new MutableLiveData<>();
            loadFruits();
        }
        return fruitList;
    }

在您的案例中，

loadFruits() 函数是从存储库中调用的 api。

您应该使用 ViewModelProvider and/or ViewModelFactory 实例化 ViewModel。是更详细的解释。它应该看起来像这样：

private val viewModel = ViewModelProvider(this).get(MainViewModel::class.java)

它现在是生命周期感知的。如果愿意，您可以稍后在片段中注入相同的实例。

只是为了好玩：我喜欢你的按钮变量被称为 myButt :D

Android 应用程序架构我这样做正确吗？

Android application architecture am i doing this correctly?

java

android

retrofit