Android MVP - 如何使用依赖于 Android 类的逻辑？

Question

我正在开发一个现有的应用程序，将其迁移到 MVP 架构。在其中一个 RecyclerView.Adapter 中，有很多关于标题 View 应该是什么样子的逻辑。根据 MVP 架构，这样的逻辑应该被移动到 Presenter 并且 Presenter 应该使用 View 来显示 things。此外，我已经阅读了很多关于如果您的演示者中有 Android 类的内容 - 这基本上表明出了点问题。

到目前为止，我的想法是为 ViewHolder 创建一个 Presenter，当 ViewHolder(也是 View 从 MVP 的角度来看)被创建并将我的逻辑移到那里。然而，我遇到的问题是逻辑基于 Android 类，例如 Paint、Rect、Spannable 等等。

那么如何以最干净的方式解决这个问题呢？我是否将所有逻辑都移到了 View 中？或者我应该将逻辑保留在我的 Presenter 中，而只将与特定 Android 对象相关的计算移动到 View 中吗？作为一个看起来像这样的例子:

在演示者中:

double textLines = mView.getTextLines(text, 0 , text.length());

在 View 中:

Paint paint = mTextView.getPaint();
Rect bounds = new Rect();
paint.getTextBounds(text, start, end, bounds);
double parentWidth = parentView.getWidth() - parentView.getPaddingLeft() - parentView
            .getPaddingRight();
return Math.ceil((double) bounds.width() / parentWidth);

或者还有其他更好的方法吗？

Answer 1

依赖于 Android 类的每一段代码都应该保留在 View 实现上(Activiy、Fragment、ViewHolder 等)- View 应该尽可能简单。我认为您不需要 ViewHolder 的演示者，因为它通常不应该有逻辑。

如果你有单元测试(你应该)，这个过程会变得更容易掌握，因为你将被迫将 Android 依赖类移动到其他地方以便能够测试它。

例如，我有一个关于 RecyclerView 的真实案例场景是这样的:一个 Activity 的 View 界面，​​一个 ViewHolder 的 View 界面，​​一个 Adapter 的 presenter 和一个 Activity 的 presenter。以下是仅用于演示的部分不可编译示例，您可以在其中看到层与层之间的关注点分离。

因此，在您的情况下，该方法是正确的: View 界面只有一个方法可以“从某处”返回一个字符串，而在真正的 android View 中，您使用 Android API 中的实际类和方法来检索请求的数据。

我的例子如下。

// Definitions for the event adapter stuff
public interface EventAdapterContract {
    interface View {
        void onFetchEventsFailed(String reason);
        void notifyDataSetChanged();
    }

    interface Presenter {
        void getItemViewType(int position);
        void getItemCount();
        void bindEventRow(int position, ViewHolder holder, int backgroundColor);
    }

    interface ViewHolder {
        void setTitle(String value);
        void setHour(String value);
        void setBackgroundColor(int color);
    }
}

// Adapter presenter implementation. Again, partial code for simplicity
public class EventAdapterPresenter implements EventAdapterContract.Presenter {
    private final EventAdapterContract.View view;
    private final List<Event> events;

    static int VIEW_FINISHED = 0;
    static int VIEW_pending = 0;

    public EventAdapterPresenter(EventAdapterContract.View view, EventAPI api) {
        this.view = view;
        this.events = new ArrayList<>();

        // EventAPI is an interface (Retrofit in this case), which can
        // also be tested with plain junit
    }

    @Override
    public int getItemCount() {
        return events.size();
    }

    @Override
    public int getItemViewType(int position) {
        Event item = events.get(position);
        return item.isFinished() ? VIEW_FINISHED : VIEW_PENDING;
    }

    @Override
    public void bindEventRow(int position, EventAdapterContract.ViewHolder holder, int backgroundColor) {
        Event event = events.get(position);
        holder.setTitle(event.getTitle());
        holder.setHour(String.format("Event time: %s", event.getStartTime()));
        holder.setBackgroundColor(backgroundColor);
    }
}



// The adapter itself. Please mind that this is a partial
// piece of code just for the sake of demonstration, 
// I ommited several parts to keep it simple
public class EventAdapter 
    extends RecyclerView.Adapter<RecyclerView.ViewHolder> 
    implements EventAdapterContract.View {

    private EventAdapterContract.Presenter presenter;

    public class EventAdapter(Contex context) {
        // Regular adapter contructor
        // ...
        presenter = new EventAdapterPresenter(this, someImplOfEventApi);
    }

    @Override
    public void onFetchEventsFailed(String reason) {
        // Show an AlertDialog
    }

    @Override
    public int getItemViewType(int position) {
        return presenter.getItemViewType(position);
    }

    @Override
    public void onBindViewHolder(RecyclerView.ViewHolder holder, int position) {
        int backgroundColor = ContextCompat.getColor(context, R.color.colorEventPending);
        presenter.bindEventRow(position, holder, backgroundColor);
    }
}

public class EventRowViewHolder 
    extends RecyclerView.ViewHolder
    implements View.OnClickListener, EventAdapterContract.ViewHolder.Row {

    private TextView title;
    private TextView hour;

    public EventRowViewHolder(View view) {
        super(view);
        // init widgets etc...
        view.setOnClickListener(this);
    }

    @Override
    public void setTitle(String value) {
        title.setText(value);
    }

    @Override
    public void setHour(String value) {
        hour.setText(value);
    }

    @Override
    public void setBackgroundColor(int color) {
        this.itemView.setBackgroundColor(color);
    }

    @Override
    public void onClick(View view) {
        EventBus.getDefault().post(new OpenEventDetailsMessage(orderId));
    }
}