我正在开发一个现有应用,将其迁移到MVP
架构。在其中一个RecyclerView.Adapter
中,关于标题View
应该是什么样子,有很多逻辑。根据{{1}}架构,此类逻辑应移至MVP
,演示者应使用Presenter
来显示内容。此外,我已经阅读了很多关于如何在演示者中拥有View
课程的内容 - 这基本上表明出现了问题。
所以到目前为止,我的想法是在Android
(以及Presenter
立场中的ViewHolder
)创建时为ViewHolder
创建View
并把我的逻辑移到那里。然而,我遇到的问题是逻辑基于MVP
类,例如Android
,Paint
,Rect
等等。
那么如何以最干净的方式解决这个问题呢?我是否将所有逻辑移至Spannable
?或者我应该在我的View
中保留逻辑,但只将与特定Presenter
对象相关的计算移到Android
?作为一个看起来像这样的例子:
在演示者中:
View
在视图中:
double textLines = mView.getTextLines(text, 0 , text.length());
还是有另一种更好的方法吗?
答案 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));
}
}