RecyclerView 适配器采用错误的值
RecyclerView adapter taking wrong values
我有一个 RecyclerView，它显示两种 View，一种代表用户发布，另一种代表事件发布。两者都有共同的元素，例如显示时间戳的 TextView。所以我创建了一个 PublicationViewHolder 将这个 TextView 时间戳记到一个变量中并加载它。我的问题是适配器最初加载了正确的值，但是当我向下滚动并再次向上滚动时，位置中的值被另一个位置的值更改。这是代码：
public class PublicationViewHolder extends RecyclerView.ViewHolder {
private TextView vTimeStamp;
public PublicationViewHolder(View itemView) {
super(itemView);
this.vTimeStamp = (TextView) itemView.findViewById(R.id.txt_view_publication_timestamp);
}
public void load(Publication publication, int i) {
load(publication);
try {
if (Publication.TYPE_USER_PUBLICATION == publication.getType()) {
load((UserPublication) publication);
} else if (Publication.TYPE_EVENT_PUBLICATION == publication.getType()) {
load((EventPublication) publication);
}
} catch (ClassCastException e) {
throw new RuntimeException("Publication type cast fail. See PublicationViewHolder.");
}
}
public void load(Publication publication) {
vTimeStamp.setText(DateFormatter.getTimeAgo(publication.getTimeStamp()));
}
public void load( UserPublication publication) {
//This method is override by UserPublicationViewHolder
};
public void load( EventPublication publication) {
//This method is override by EventPublicationViewHolder
};
}
现在我将为用户制作我的 UserPublicationViewHolder 出版物。
public class UserPublicationViewHolder extends PublicationViewHolder {
private ImageView vImageView, vLikeButton, vDislikeButton, vFavButton, vEditPost, vDeletePost;
private TextView vText, vUsername, vLikeCount, vDislikeCount, vFavCount;
private PostImagesLayout vImagesContainer;
private TagCloudLocationFriends tagsView;
public UserPublicationViewHolder(View itemView) {
super(itemView);
vImageView = (ImageView) itemView.findViewById(R.id.img_view_publication_user);
vText = (TextView) itemView.findViewById(R.id.txt_view_publication_text);
vLikeCount = (TextView) itemView.findViewById(R.id.txt_view_like_count);
vFavCount = (TextView) itemView.findViewById(R.id.txt_view_fav_count);
vDislikeCount = (TextView) itemView.findViewById(R.id.txt_view_dislike_count);
vUsername = (TextView) itemView.findViewById(R.id.txt_view_publication_user_name);
vLikeButton = (ImageView) itemView.findViewById(R.id.img_view_like);
vDislikeButton = (ImageView) itemView.findViewById(R.id.img_view_dislike);
vFavButton = (ImageView) itemView.findViewById(R.id.img_view_fav);
vImagesContainer = (PostImagesLayout) itemView.findViewById(R.id.container_post_images);
tagsView = (TagCloudLocationFriends) itemView.findViewById(R.id.location_friends_tag);
// edit - remove icons
vDeletePost = (ImageView) itemView.findViewById(R.id.img_view_delete_post);
vEditPost = (ImageView) itemView.findViewById(R.id.img_view_edit_post);
}
@Override
public void load(UserPublication publication) {
//Load the UserPublicationViewHolder specific views.
}
}
现在我将做同样的事情，但对于活动出版物
public class EventPublicationViewHolder extends PublicationViewHolder {
private TextView vTextViewTitle;
private TextView vTextViewText;
public EventPublicationViewHolder(View itemView) {
super(itemView);
vTextViewTitle = (TextView) itemView.findViewById(R.id.txt_view_publication_event_title);
vTextViewText = (TextView) itemView.findViewById(R.id.txt_view_publication_event_text);
}
@Override
public void load(EventPublication publication) {
//Load the EventPublicationViewHolder specifics views
}
}
现在这是我的 RecyclerView 适配器：
public class PublicationAdapter extends RecyclerView.Adapter<PublicationViewHolder> {
public static final int USER_PUBLICATION_TYPE = 1;
public static final int EVENT_PUBLICATION_TYPE = 2;
private List<Publication> publications = new ArrayList<Publication>();
public List<Publication> getPublications() {
return publications;
}
public void setPublications(List<Publication> publications) {
this.publications = publications;
}
@Override
public int getItemViewType(int position) {
if (publications.get(position) instanceof UserPublication) {
return USER_PUBLICATION_TYPE;
}
if (publications.get(position) instanceof EventPublication) {
return EVENT_PUBLICATION_TYPE;
}
throw new RuntimeException("Unknown view type in PublicationAdapter");
}
@Override
public PublicationViewHolder onCreateViewHolder(ViewGroup viewGroup, int type) {
View v;
switch (type) {
case USER_PUBLICATION_TYPE:
v = LayoutInflater.from(getActivity()).inflate(R.layout.view_holder_user_publication, viewGroup, false);
return new UserPublicationViewHolder(v);
case EVENT_PUBLICATION_TYPE:
v = LayoutInflater.from(getActivity()).inflate(R.layout.view_holder_event_publication, viewGroup, false);
return new EventPublicationViewHolder(v);
}
return null;
}
@Override
public void onBindViewHolder(PublicationViewHolder aPublicationHolder, int i) {
aPublicationHolder.load(publications.get(i), i);
}
@Override
public long getItemId(int position) {
//Here I tried returning only position or 0 without luck.
//The id is unique BTW
return publications.get(position).getId();
}
@Override
public int getItemCount() {
return publications.size();
}
}
我不知道哪里出了问题，UserPublication 和 EventPublication 都继承自 Publication。我没有做一些请求或重新加载适配器。我只加载一次适配器。
更新：
顺便说一句，我在 Fragment 中使用这个 RecyclerView，Fragment 加载到 PageAdapter 中，而 PageAdapter 加载到 Fragment 中的 ViewPager 中，也许这是问题所在？
更新：
这是另一个绑定代码。
这是UserPublicationViewHolder的加载方法。
@Override
public void load(UserPublication publication) {
PicassoHelper.publicationUser(getActivity(), publication.getUser().getAvatarUrl(),
vImageView);
vText.setText(publication.getText());
vUsername.setText(publication.getUser().getName());
boolean hasLocation = false;
if (publication.getImages().length > 0) {
vImagesContainer.setImages(publication.getImages());
} else {
vImagesContainer.setVisibility(View.GONE);
}
tagsView.setTags(new ArrayList<MinikastTag>());
tagsView.drawTags();
if(publication.getLocation() != null || publication.getTaggedFriends().size() > 0){
if(publication.getLocation() != null){
hasLocation = true;
tagsView.add(new MinikastTag(1,"Post from ",1));
tagsView.add(new MinikastTag(2, publication.getLocation().getName(), 2));
}
if(publication.getTaggedFriends().size() > 0){
if(hasLocation)
tagsView.add(new MinikastTag(3," with ",1));
else
tagsView.add(new MinikastTag(3,"With ",1));
int i = 0;
for(User aUser: publication.getTaggedFriends()){
MinikastTag aTag;
if(i == publication.getTaggedFriends().size() - 1 ) {
aTag = new MinikastTag(4, aUser.getName(), 3);
aTag.setUserID(aUser.getId());
aTag.setUserName(aUser.getName());
tagsView.add(aTag);
} else {
aTag = new MinikastTag(4, aUser.getName() + ", ", 3);
aTag.setUserID(aUser.getId());
aTag.setUserName(aUser.getName());
tagsView.add(aTag);
}
i = i+1;
}
}
}
tagsView.drawTags();
// likes, dislikes, favs
if(publication.getLikesAmount() > 0)
vLikeCount.setText(String.valueOf(publication.getLikesAmount()));
if(publication.getDislikesAmount() > 0)
vDislikeCount.setText(String.valueOf(publication.getDislikesAmount()));
if(publication.getLovesAmount() > 0)
vFavCount.setText(String.valueOf(publication.getLovesAmount()));
// reset buttons
vFavButton.setPressed(false);
vDislikeButton.setPressed(false);
vLikeButton.setPressed(false);
if(publication.getRelationship().equals("LOVE"))
vFavButton.setPressed(true);
else if (publication.getRelationship().equals("LIKE"))
vLikeButton.setPressed(true);
else if (publication.getRelationship().equals("DISLIKE"))
vDislikeButton.setPressed(true);
// edit - remove icons
if(String.valueOf(publication.getUser().getId()).equals(StartupSharedPreferences.getProfileId())){
vEditPost.setVisibility(View.VISIBLE);
vDeletePost.setVisibility(View.VISIBLE);
}else{
vEditPost.setVisibility(View.INVISIBLE);
vDeletePost.setVisibility(View.INVISIBLE);
}
}
}
这是 EventPublicationViewHolder 的加载方法：
@Override
public void load(EventPublication publication) {
vTimeStamp.setVisibility(View.GONE);
itemView.setOnClickListener(new View.OnClickListener() {
@Override
public void onClick(View view) {
//GoTo.eventDetail(getActivity(), publication);
}
});
vTextViewTitle.setText(publication.getTitle());
vTextViewText.setText(publication.getText());
}
我评论了一些代码只是因为我正在测试，但正如你所看到的，我只做了 setTexts 和 assess 一些图像。
这就是我在片段的 onViewCreated 方法中设置适配器、LinearLayoutManager 等的方式。
vRecyclerView = (FixedRecyclerView) view.findViewById(R.id.recycler_view_publications);
vSwipeRefresh = (SwipeRefreshLayout) view.findViewById(R.id.swipe_container);
mFeedCallback.onScrollReady(vRecyclerView);
mLayoutManager = buildLayoutManager();
vRecyclerView.setLayoutManager(mLayoutManager);
vRecyclerView.addItemDecoration(new DividerItemDecoration(getActivity(), DividerItemDecoration.VERTICAL_LIST));
mAdapter = new PublicationAdapter();
vSwipeRefresh.setOnRefreshListener(this);
vSwipeRefresh.setColorSchemeResources(R.color._SWIPER_COLOR_1, R.color._SWIPER_COLOR_2,
R.color._SWIPER_COLOR_3, R.color._SWIPER_COLOR_4);
vRecyclerView.setAdapter(mAdapter);
顺便说一句，适配器在我的自定义方法中加载了数据集，称为 onHttpClientReady，但这似乎不是问题所在。
以下是部分截图：
第一次进入应用程序时的列表顶部：
然后当我回来的时候：
顺便说一句，喜欢、不喜欢和喜欢的按钮，如果有人点击它们不止一次，会显示一个数值，如果是，这个值也会放错地方。
更新：
现在我知道那不是因为嵌套的片段。我更改了我的代码，现在每个选项卡片段都在 Activity 内的 ViewPager 内的 PageStateAdapter 中。但是问题依旧。
更新：
我发现 getItemId 方法从未被执行，不知道为什么。
绑定代码中的一个大变量是日期格式：
DateFormatter.getTimeAgo(publication.getTimeStamp())
没有直接看到 class 很难确定，但看起来，如果时间戳是不可变的，但格式化程序是基于当前时间的，那么这将与文本一致视图反弹时更改。
我认为一个更大的问题（稍微有点旁白）是代码的可读性，这使得很难从视觉上轻松地发现问题。这里的继承模式和重载使得很难对代码进行推理并决定采用哪条路径以及它是否在做正确的事情。这是一些餐巾纸代码（尚未构建或 运行 它）使用更组合的方法，可能是更清晰的组织并更容易调试问题：
通用视图持有者代码的新助手 class，替换 PublicationViewHolder:
public class PublicationViewHolderHelper {
private final TextView vTimeStamp;
public PublicationViewHolder(View itemView) {
super(itemView);
this.vTimeStamp = (TextView) itemView.findViewById(R.id.txt_view_publication_timestamp);
}
/** Binds view data common to publication types. */
public void load(Publication publication) {
vTimeStamp.setText(DateFormatter.getTimeAgo(publication.getTimeStamp()));
}
}
以 EventPublicationViewHolder 为例（对 UserPublicationViewHolder 做同样的事情）：
public class EventPublicationViewHolder extends ViewHolder {
private final PublicationViewHolderHelper helper;
// View fields...
public EventPublicationViewHolder(View itemView) {
super(itemView);
helper = new PublicationViewHolderHelper(itemView);
// Populated view fields...
}
@Override
public void load(EventPublication publication) {
helper.load(publication);
//Load the EventPublicationViewHolder specifics views
}
}
请注意，您的适配器中现在没有基础 class，也不需要类型检查，因此代码少了很多。
现在适配器保持不变，除了通用类型和 onBindViewHolder:
public class PublicationAdapter extends RecyclerView.Adapter<ViewHolder> {
...
@Override
public void onBindViewHolder(ViewHolder viewHolder, int position) {
final Publication publication = publications.get(position);
final int viewType = getItemViewType(position);
switch (viewType) {
case USER_PUBLICATION_TYPE:
((UserPublicationViewHolder) viewHolder).load((UserPublication) publication);
break;
case EVENT_PUBLICATION_TYPE:
((EventPublicationViewHolder) viewHolder).load((EventPublication) publication);
break;
default:
// Blow up in whatever way you choose.
}
}
...
}
请注意，它与您的 onCreateViewHolder 保持着非常相似的模式，因此不仅总体代码更少，而且内部一致性更高。这当然不是唯一的方法，只是根据您的特定用例提出的建议。
我建议查看您的 class 层次结构和用法。通常，如果您在基 class 中执行 type == type 类操作，那么您就违背了抽象和继承的目的。这样的东西对你有用：
public abstract class PublicationViewHolder extends RecyclerView.ViewHolder {
private TextView mTimeStamp;
public PublicationViewHolder(View itemView) {
mTimeStamp = (TextView)itemView.findViewById(R.id. txt_view_publication_timestamp);
}
public void bindViews(Publication publication) {
mTimeStamp.setText(DateFormatter.getTimeAgo(publication.getTimeStamp()));
}
}
现在您的 "event" 或 "user publications" 只需从这个 class 派生并实现构造函数和 bindViews() 方法。 请务必在两种情况下调用超级 class。 此外，请确保您设置 every 在您的 bindViews() 方法中查看特定出版物的布局。
在您的适配器中，您只需根据数据集中该位置的出版物类型创建正确的持有人：
public class PublicationAdapter extends RecyclerView.Adapter {
private ArrayList<Publication> mPubs;
// Your other code here, like
// swapPublications(), getItemCount(), etc.
...
public int getItemViewType(int position) {
return mPubs.get(position).getType();
}
public PublicationViewHolder createViewHolder(ViewGroup parent, int type) {
PublicationViewHolder ret;
View root;
LayoutInflater inflater = LayoutInflater.from(parent.getContext());
if (type == USER_PUBLICATION_TYPE) {
root =
inflater.inflate(R.layout.view_holder_user_publication,
parent,
false);
ret = new UserPubHolder(root);
} else {
root =
inflater.inflate(R.layout.view_holder_event_publication,
parent,
false);
ret = new EventPubHolder(root);
}
return ret;
}
public bindViewHolder(PublicationViewHolder holder, int position) {
holder.bindViews(mPubs.get(position));
}
}
这通常发生在您有 "if (field != null) holder.setField(field)" 之类的东西时，没有其他东西。持有人被回收，这意味着它将在那里有值，所以你需要清理或替换每个值，如果它是空的，你应该 nullit，如果不是，你应该写它，总是。来晚了，但是，作为对其他人的回答。
异步加载图像有同样的问题，这些图像具有 不同的高度 。所以使用调试器你可以看到，回收的位置取决于视图的实际大小。
对我来说，简单的解决方案是指定不同的尺寸，这样系统就知道所有项目的确切尺寸。 https://developer.android.com/reference/android/support/v7/widget/RecyclerView.Adapter.html#getItemViewType(int)
例如横向、纵向和方形。
所以我创建了单独的视图并像这样使用它们：（简化）
public class YourAdapter extends RecyclerView.Adapter<RecyclerView.ViewHolder> {
// ...
public static class ViewHolderLandscape extends RecyclerView.ViewHolder { ... }
public static class ViewHolderPortrait extends RecyclerView.ViewHolder { ... }
public static class ViewHolderSquare extends RecyclerView.ViewHolder { ... }
@Override
public int getItemViewType(int position) {
return mDataset.get(position).getImageType();
}
@Override
public RecyclerView.ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
int mLayoutId = 0;
switch (viewType) {
case 0:
mLayoutId = R.layout.list_item_landscape;
break;
case 1:
mLayoutId = R.layout.list_item_portrait;
break;
case 2:
mLayoutId = R.layout.list_item_square;
break;
}
View v = LayoutInflater.from(parent.getContext()).inflate(mLayoutId, parent, false);
ButterKnife.inject(this, v);
return new ViewHolder(v);
}
}
RecycleView 终于不再对 different/dynamic 项目大小感到困惑。
对我来说设置setHasStableIds(false)解决了问题。
我有一个 RecyclerView，它显示两种 View，一种代表用户发布，另一种代表事件发布。两者都有共同的元素，例如显示时间戳的 TextView。所以我创建了一个 PublicationViewHolder 将这个 TextView 时间戳记到一个变量中并加载它。我的问题是适配器最初加载了正确的值，但是当我向下滚动并再次向上滚动时，位置中的值被另一个位置的值更改。这是代码：
public class PublicationViewHolder extends RecyclerView.ViewHolder {
private TextView vTimeStamp;
public PublicationViewHolder(View itemView) {
super(itemView);
this.vTimeStamp = (TextView) itemView.findViewById(R.id.txt_view_publication_timestamp);
}
public void load(Publication publication, int i) {
load(publication);
try {
if (Publication.TYPE_USER_PUBLICATION == publication.getType()) {
load((UserPublication) publication);
} else if (Publication.TYPE_EVENT_PUBLICATION == publication.getType()) {
load((EventPublication) publication);
}
} catch (ClassCastException e) {
throw new RuntimeException("Publication type cast fail. See PublicationViewHolder.");
}
}
public void load(Publication publication) {
vTimeStamp.setText(DateFormatter.getTimeAgo(publication.getTimeStamp()));
}
public void load( UserPublication publication) {
//This method is override by UserPublicationViewHolder
};
public void load( EventPublication publication) {
//This method is override by EventPublicationViewHolder
};
}
现在我将为用户制作我的 UserPublicationViewHolder 出版物。
public class UserPublicationViewHolder extends PublicationViewHolder {
private ImageView vImageView, vLikeButton, vDislikeButton, vFavButton, vEditPost, vDeletePost;
private TextView vText, vUsername, vLikeCount, vDislikeCount, vFavCount;
private PostImagesLayout vImagesContainer;
private TagCloudLocationFriends tagsView;
public UserPublicationViewHolder(View itemView) {
super(itemView);
vImageView = (ImageView) itemView.findViewById(R.id.img_view_publication_user);
vText = (TextView) itemView.findViewById(R.id.txt_view_publication_text);
vLikeCount = (TextView) itemView.findViewById(R.id.txt_view_like_count);
vFavCount = (TextView) itemView.findViewById(R.id.txt_view_fav_count);
vDislikeCount = (TextView) itemView.findViewById(R.id.txt_view_dislike_count);
vUsername = (TextView) itemView.findViewById(R.id.txt_view_publication_user_name);
vLikeButton = (ImageView) itemView.findViewById(R.id.img_view_like);
vDislikeButton = (ImageView) itemView.findViewById(R.id.img_view_dislike);
vFavButton = (ImageView) itemView.findViewById(R.id.img_view_fav);
vImagesContainer = (PostImagesLayout) itemView.findViewById(R.id.container_post_images);
tagsView = (TagCloudLocationFriends) itemView.findViewById(R.id.location_friends_tag);
// edit - remove icons
vDeletePost = (ImageView) itemView.findViewById(R.id.img_view_delete_post);
vEditPost = (ImageView) itemView.findViewById(R.id.img_view_edit_post);
}
@Override
public void load(UserPublication publication) {
//Load the UserPublicationViewHolder specific views.
}
}
现在我将做同样的事情，但对于活动出版物
public class EventPublicationViewHolder extends PublicationViewHolder {
private TextView vTextViewTitle;
private TextView vTextViewText;
public EventPublicationViewHolder(View itemView) {
super(itemView);
vTextViewTitle = (TextView) itemView.findViewById(R.id.txt_view_publication_event_title);
vTextViewText = (TextView) itemView.findViewById(R.id.txt_view_publication_event_text);
}
@Override
public void load(EventPublication publication) {
//Load the EventPublicationViewHolder specifics views
}
}
现在这是我的 RecyclerView 适配器：
public class PublicationAdapter extends RecyclerView.Adapter<PublicationViewHolder> {
public static final int USER_PUBLICATION_TYPE = 1;
public static final int EVENT_PUBLICATION_TYPE = 2;
private List<Publication> publications = new ArrayList<Publication>();
public List<Publication> getPublications() {
return publications;
}
public void setPublications(List<Publication> publications) {
this.publications = publications;
}
@Override
public int getItemViewType(int position) {
if (publications.get(position) instanceof UserPublication) {
return USER_PUBLICATION_TYPE;
}
if (publications.get(position) instanceof EventPublication) {
return EVENT_PUBLICATION_TYPE;
}
throw new RuntimeException("Unknown view type in PublicationAdapter");
}
@Override
public PublicationViewHolder onCreateViewHolder(ViewGroup viewGroup, int type) {
View v;
switch (type) {
case USER_PUBLICATION_TYPE:
v = LayoutInflater.from(getActivity()).inflate(R.layout.view_holder_user_publication, viewGroup, false);
return new UserPublicationViewHolder(v);
case EVENT_PUBLICATION_TYPE:
v = LayoutInflater.from(getActivity()).inflate(R.layout.view_holder_event_publication, viewGroup, false);
return new EventPublicationViewHolder(v);
}
return null;
}
@Override
public void onBindViewHolder(PublicationViewHolder aPublicationHolder, int i) {
aPublicationHolder.load(publications.get(i), i);
}
@Override
public long getItemId(int position) {
//Here I tried returning only position or 0 without luck.
//The id is unique BTW
return publications.get(position).getId();
}
@Override
public int getItemCount() {
return publications.size();
}
}
我不知道哪里出了问题，UserPublication 和 EventPublication 都继承自 Publication。我没有做一些请求或重新加载适配器。我只加载一次适配器。
更新：
顺便说一句，我在 Fragment 中使用这个 RecyclerView，Fragment 加载到 PageAdapter 中，而 PageAdapter 加载到 Fragment 中的 ViewPager 中，也许这是问题所在？
更新： 这是另一个绑定代码。
这是UserPublicationViewHolder的加载方法。
@Override
public void load(UserPublication publication) {
PicassoHelper.publicationUser(getActivity(), publication.getUser().getAvatarUrl(),
vImageView);
vText.setText(publication.getText());
vUsername.setText(publication.getUser().getName());
boolean hasLocation = false;
if (publication.getImages().length > 0) {
vImagesContainer.setImages(publication.getImages());
} else {
vImagesContainer.setVisibility(View.GONE);
}
tagsView.setTags(new ArrayList<MinikastTag>());
tagsView.drawTags();
if(publication.getLocation() != null || publication.getTaggedFriends().size() > 0){
if(publication.getLocation() != null){
hasLocation = true;
tagsView.add(new MinikastTag(1,"Post from ",1));
tagsView.add(new MinikastTag(2, publication.getLocation().getName(), 2));
}
if(publication.getTaggedFriends().size() > 0){
if(hasLocation)
tagsView.add(new MinikastTag(3," with ",1));
else
tagsView.add(new MinikastTag(3,"With ",1));
int i = 0;
for(User aUser: publication.getTaggedFriends()){
MinikastTag aTag;
if(i == publication.getTaggedFriends().size() - 1 ) {
aTag = new MinikastTag(4, aUser.getName(), 3);
aTag.setUserID(aUser.getId());
aTag.setUserName(aUser.getName());
tagsView.add(aTag);
} else {
aTag = new MinikastTag(4, aUser.getName() + ", ", 3);
aTag.setUserID(aUser.getId());
aTag.setUserName(aUser.getName());
tagsView.add(aTag);
}
i = i+1;
}
}
}
tagsView.drawTags();
// likes, dislikes, favs
if(publication.getLikesAmount() > 0)
vLikeCount.setText(String.valueOf(publication.getLikesAmount()));
if(publication.getDislikesAmount() > 0)
vDislikeCount.setText(String.valueOf(publication.getDislikesAmount()));
if(publication.getLovesAmount() > 0)
vFavCount.setText(String.valueOf(publication.getLovesAmount()));
// reset buttons
vFavButton.setPressed(false);
vDislikeButton.setPressed(false);
vLikeButton.setPressed(false);
if(publication.getRelationship().equals("LOVE"))
vFavButton.setPressed(true);
else if (publication.getRelationship().equals("LIKE"))
vLikeButton.setPressed(true);
else if (publication.getRelationship().equals("DISLIKE"))
vDislikeButton.setPressed(true);
// edit - remove icons
if(String.valueOf(publication.getUser().getId()).equals(StartupSharedPreferences.getProfileId())){
vEditPost.setVisibility(View.VISIBLE);
vDeletePost.setVisibility(View.VISIBLE);
}else{
vEditPost.setVisibility(View.INVISIBLE);
vDeletePost.setVisibility(View.INVISIBLE);
}
}
}
这是 EventPublicationViewHolder 的加载方法：
@Override
public void load(EventPublication publication) {
vTimeStamp.setVisibility(View.GONE);
itemView.setOnClickListener(new View.OnClickListener() {
@Override
public void onClick(View view) {
//GoTo.eventDetail(getActivity(), publication);
}
});
vTextViewTitle.setText(publication.getTitle());
vTextViewText.setText(publication.getText());
}
我评论了一些代码只是因为我正在测试，但正如你所看到的，我只做了 setTexts 和 assess 一些图像。
这就是我在片段的 onViewCreated 方法中设置适配器、LinearLayoutManager 等的方式。
vRecyclerView = (FixedRecyclerView) view.findViewById(R.id.recycler_view_publications);
vSwipeRefresh = (SwipeRefreshLayout) view.findViewById(R.id.swipe_container);
mFeedCallback.onScrollReady(vRecyclerView);
mLayoutManager = buildLayoutManager();
vRecyclerView.setLayoutManager(mLayoutManager);
vRecyclerView.addItemDecoration(new DividerItemDecoration(getActivity(), DividerItemDecoration.VERTICAL_LIST));
mAdapter = new PublicationAdapter();
vSwipeRefresh.setOnRefreshListener(this);
vSwipeRefresh.setColorSchemeResources(R.color._SWIPER_COLOR_1, R.color._SWIPER_COLOR_2,
R.color._SWIPER_COLOR_3, R.color._SWIPER_COLOR_4);
vRecyclerView.setAdapter(mAdapter);
顺便说一句，适配器在我的自定义方法中加载了数据集，称为 onHttpClientReady，但这似乎不是问题所在。
以下是部分截图：
第一次进入应用程序时的列表顶部：
然后当我回来的时候：
顺便说一句，喜欢、不喜欢和喜欢的按钮，如果有人点击它们不止一次，会显示一个数值，如果是，这个值也会放错地方。
更新： 现在我知道那不是因为嵌套的片段。我更改了我的代码，现在每个选项卡片段都在 Activity 内的 ViewPager 内的 PageStateAdapter 中。但是问题依旧。
更新： 我发现 getItemId 方法从未被执行，不知道为什么。
绑定代码中的一个大变量是日期格式：
DateFormatter.getTimeAgo(publication.getTimeStamp())
没有直接看到 class 很难确定，但看起来，如果时间戳是不可变的，但格式化程序是基于当前时间的，那么这将与文本一致视图反弹时更改。
我认为一个更大的问题（稍微有点旁白）是代码的可读性，这使得很难从视觉上轻松地发现问题。这里的继承模式和重载使得很难对代码进行推理并决定采用哪条路径以及它是否在做正确的事情。这是一些餐巾纸代码（尚未构建或 运行 它）使用更组合的方法，可能是更清晰的组织并更容易调试问题：
通用视图持有者代码的新助手 class，替换 PublicationViewHolder:
public class PublicationViewHolderHelper {
private final TextView vTimeStamp;
public PublicationViewHolder(View itemView) {
super(itemView);
this.vTimeStamp = (TextView) itemView.findViewById(R.id.txt_view_publication_timestamp);
}
/** Binds view data common to publication types. */
public void load(Publication publication) {
vTimeStamp.setText(DateFormatter.getTimeAgo(publication.getTimeStamp()));
}
}
EventPublicationViewHolder 为例（对 UserPublicationViewHolder 做同样的事情）：
public class EventPublicationViewHolder extends ViewHolder {
private final PublicationViewHolderHelper helper;
// View fields...
public EventPublicationViewHolder(View itemView) {
super(itemView);
helper = new PublicationViewHolderHelper(itemView);
// Populated view fields...
}
@Override
public void load(EventPublication publication) {
helper.load(publication);
//Load the EventPublicationViewHolder specifics views
}
}
请注意，您的适配器中现在没有基础 class，也不需要类型检查，因此代码少了很多。
现在适配器保持不变，除了通用类型和 onBindViewHolder:
public class PublicationAdapter extends RecyclerView.Adapter<ViewHolder> {
...
@Override
public void onBindViewHolder(ViewHolder viewHolder, int position) {
final Publication publication = publications.get(position);
final int viewType = getItemViewType(position);
switch (viewType) {
case USER_PUBLICATION_TYPE:
((UserPublicationViewHolder) viewHolder).load((UserPublication) publication);
break;
case EVENT_PUBLICATION_TYPE:
((EventPublicationViewHolder) viewHolder).load((EventPublication) publication);
break;
default:
// Blow up in whatever way you choose.
}
}
...
}
请注意，它与您的 onCreateViewHolder 保持着非常相似的模式，因此不仅总体代码更少，而且内部一致性更高。这当然不是唯一的方法，只是根据您的特定用例提出的建议。
我建议查看您的 class 层次结构和用法。通常，如果您在基 class 中执行 type == type 类操作，那么您就违背了抽象和继承的目的。这样的东西对你有用：
public abstract class PublicationViewHolder extends RecyclerView.ViewHolder {
private TextView mTimeStamp;
public PublicationViewHolder(View itemView) {
mTimeStamp = (TextView)itemView.findViewById(R.id. txt_view_publication_timestamp);
}
public void bindViews(Publication publication) {
mTimeStamp.setText(DateFormatter.getTimeAgo(publication.getTimeStamp()));
}
}
现在您的 "event" 或 "user publications" 只需从这个 class 派生并实现构造函数和 bindViews() 方法。 请务必在两种情况下调用超级 class。 此外，请确保您设置 every 在您的 bindViews() 方法中查看特定出版物的布局。
在您的适配器中，您只需根据数据集中该位置的出版物类型创建正确的持有人：
public class PublicationAdapter extends RecyclerView.Adapter {
private ArrayList<Publication> mPubs;
// Your other code here, like
// swapPublications(), getItemCount(), etc.
...
public int getItemViewType(int position) {
return mPubs.get(position).getType();
}
public PublicationViewHolder createViewHolder(ViewGroup parent, int type) {
PublicationViewHolder ret;
View root;
LayoutInflater inflater = LayoutInflater.from(parent.getContext());
if (type == USER_PUBLICATION_TYPE) {
root =
inflater.inflate(R.layout.view_holder_user_publication,
parent,
false);
ret = new UserPubHolder(root);
} else {
root =
inflater.inflate(R.layout.view_holder_event_publication,
parent,
false);
ret = new EventPubHolder(root);
}
return ret;
}
public bindViewHolder(PublicationViewHolder holder, int position) {
holder.bindViews(mPubs.get(position));
}
}
这通常发生在您有 "if (field != null) holder.setField(field)" 之类的东西时，没有其他东西。持有人被回收，这意味着它将在那里有值，所以你需要清理或替换每个值，如果它是空的，你应该 nullit，如果不是，你应该写它，总是。来晚了，但是，作为对其他人的回答。
异步加载图像有同样的问题，这些图像具有 不同的高度 。所以使用调试器你可以看到，回收的位置取决于视图的实际大小。
对我来说，简单的解决方案是指定不同的尺寸，这样系统就知道所有项目的确切尺寸。 https://developer.android.com/reference/android/support/v7/widget/RecyclerView.Adapter.html#getItemViewType(int)
例如横向、纵向和方形。
所以我创建了单独的视图并像这样使用它们：（简化）
public class YourAdapter extends RecyclerView.Adapter<RecyclerView.ViewHolder> {
// ...
public static class ViewHolderLandscape extends RecyclerView.ViewHolder { ... }
public static class ViewHolderPortrait extends RecyclerView.ViewHolder { ... }
public static class ViewHolderSquare extends RecyclerView.ViewHolder { ... }
@Override
public int getItemViewType(int position) {
return mDataset.get(position).getImageType();
}
@Override
public RecyclerView.ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
int mLayoutId = 0;
switch (viewType) {
case 0:
mLayoutId = R.layout.list_item_landscape;
break;
case 1:
mLayoutId = R.layout.list_item_portrait;
break;
case 2:
mLayoutId = R.layout.list_item_square;
break;
}
View v = LayoutInflater.from(parent.getContext()).inflate(mLayoutId, parent, false);
ButterKnife.inject(this, v);
return new ViewHolder(v);
}
}
RecycleView 终于不再对 different/dynamic 项目大小感到困惑。
对我来说设置setHasStableIds(false)解决了问题。