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

时间:2017-03-07 18:29:20

标签: android architecture mvp

我正在开发一个现有应用,将其迁移到MVP架构。在其中一个RecyclerView.Adapter中,关于标题View应该是什么样子,有很多逻辑。根据{{​​1}}架构,此类逻辑应移至MVP,演示者应使用Presenter来显示内容。此外,我已经阅读了很多关于如何在演示者中拥有View课程的内容 - 这基本上表明出现了问题。

所以到目前为止,我的想法是在Android(以及Presenter立场中的ViewHolder)创建时为ViewHolder创建View并把我的逻辑移到那里。然而,我遇到的问题是逻辑基于MVP类,例如AndroidPaintRect等等。

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

在演示者中:

View

在视图中:

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

还是有另一种更好的方法吗?

1 个答案:

答案 0 :(得分:3)

依赖于Android类的每一段代码都应该保存在视图实现上(Activiy,Fragment,ViewHolder等) - 视图应该尽可能地愚蠢。我认为你不需要ViewHolder的演示者,因为它通常不应该有逻辑。

如果您有单元测试(并且您应该),则该过程变得更容易掌握,因为您将被迫将Android依赖类移动到其他位置以便能够进行测试。

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

所以,在你的情况下,方法是正确的:视图接口只有一个方法,从某个地方"返回一个字符串"而在真正的android视图中,你使用实际的类和方法来自用于检索所请求数据的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));
    }
}