Почему элемент Staggered Grid Layout внутри RecyclerView автоматически меняет свое положение при загрузке изображений?

Я использую RecyclerView в своем приложении для отображения изображений с сервера, я использую Staggered Grid Layout для размещения изображений в шахматном формате, а также я использую Universal Image Loader для загрузки изображений. Моя проблема в том, что позиции изображений меняются местами при загрузке с сервера, например, изображение, показанное во втором столбце, автоматически изменится на первое и наоборот. Я не мог понять причину этого, кто-нибудь может помочь? Заранее спасибо.

Представление Recycler и StaggeredGridLayout

     RecyclerView rView = (RecyclerView)view.findViewById(R.id.recycler_view);
    rView.setHasFixedSize(true);
    rView.setItemAnimator(new DefaultItemAnimator());
    StaggeredGridLayoutManager stagGrid = new StaggeredGridLayoutManager(rowColumnValue,StaggeredGridLayoutManager.VERTICAL);
    rView.setLayoutManager(stagGrid);

Адаптер RecyclerView.java

  public class RecyclerViewAdapter extends RecyclerView.Adapter<RecyclerViewHolders> {

private ArrayList<String> itemList;
private ArrayList<String> imageStatusList;
private List<ItemObject> imageList;
private Context context;
String [] imageURLList=null;
FragmentActivity fragmentActivity;
ImageLoader imageLoader;

public RecyclerViewAdapter(FragmentActivity activity, ArrayList<String> itemList,ArrayList<String> imageStatusList,String [] imageURLs) {

    this.itemList = itemList;
    this.imageStatusList=imageStatusList;

    this.context = context;
    this.fragmentActivity=activity;
    imageLoader = ImageLoader.getInstance();
    this.imageURLList=imageURLs;

    DisplayImageOptions options = new DisplayImageOptions.Builder().cacheInMemory(true)
            .cacheOnDisc(true).resetViewBeforeLoading(true).build();

    // .showImageForEmptyUri(fallback)
    //  .showImageOnFail(fallback)
    // .showImageOnLoading(fallback).build();
    //imageList =  imageLoader.displayImage(itemList,);



}

@Override
public RecyclerViewHolders onCreateViewHolder(ViewGroup parent, int viewType) {

    View layoutView = LayoutInflater.from(parent.getContext()).inflate(R.layout.card_view_list, null);
    RecyclerViewHolders rcv = new RecyclerViewHolders(layoutView,imageURLList,fragmentActivity);
    return rcv;
}

@Override
public void onBindViewHolder(RecyclerViewHolders holder, int position) {


    imageLoader.displayImage(itemList.get(position), holder.recipeImage);
    imageStatusList.toString();

    try {

        if (imageStatusList.get(position).equals("approved")) {
            holder.rimage.setImageDrawable(fragmentActivity.getResources().getDrawable(R.drawable.varified));
        } else if (imageStatusList.get(position).equals("pending")) {
            holder.rimage.setImageDrawable(fragmentActivity.getResources().getDrawable(R.drawable.pending));

        }

    }
    catch (Exception e){
        e.printStackTrace();
    }


}

@Override
public int getItemCount() {

    return this.itemList.size();

}
}

RecyclerViewHolder.java

public class RecyclerViewHolders extends RecyclerView.ViewHolder implements View.OnClickListener{


public ImageView recipeImage,rimage;
int  likeButtonCount;
String [] imageUrls;
Context mContext;
public RecyclerViewHolders(View itemView,String [] urlList,Context mContext) {

    super(itemView);
    itemView.setOnClickListener(this);
    recipeImage = (ImageView)itemView.findViewById(R.id.recipe_image);
    rimage=(ImageView)itemView.findViewById(R.id.image_status_imageview);
    imageUrls=urlList;
    this.mContext=mContext;

}

@Override
public void onClick(View view) {
    int position = getPosition();

    ArrayList<String>  imageUrlArrayList = new ArrayList<>();
    imageUrlArrayList.add(imageUrls[position]);
    imagePopupWindow(imageUrlArrayList);
    Toast.makeText(view.getContext(), "Clicked = " + getPosition(), Toast.LENGTH_SHORT).show();
}

RecyclerViewAdapter (отредактировано)

public class RecyclerViewAdapter extends RecyclerView.Adapter<RecyclerViewHolders> {

private ArrayList<String> itemList;
private ArrayList<String> imageStatusList;
private List<ItemObject> imageList;
private Context context;
String [] imageURLList=null;
FragmentActivity fragmentActivity;
ImageLoader imageLoader;
DisplayImageOptions options ;

public RecyclerViewAdapter(FragmentActivity activity, ArrayList<String> itemList,ArrayList<String> imageStatusList,String [] imageURLs) {

    this.itemList = itemList;
    this.imageStatusList=imageStatusList;

    this.context = context;
    this.fragmentActivity=activity;
    imageLoader = ImageLoader.getInstance();
    this.imageURLList=imageURLs;

    options  = new DisplayImageOptions.Builder().cacheInMemory(true)
            .cacheOnDisc(true).resetViewBeforeLoading(true)
            .showImageForEmptyUri(fragmentActivity.getResources().getDrawable(R.drawable.wedcake))
            .showImageOnFail(fragmentActivity.getResources().getDrawable(R.drawable.wedcake))
            .showImageOnLoading(fragmentActivity.getResources().getDrawable(R.drawable.wedcake)).build();


}

onBindViewHolder.java

 @Override
public void onBindViewHolder(RecyclerViewHolders holder, int position) {


 //   imageLoader.displayImage(itemList.get(position), holder.recipeImage,options);


    holder.bind(itemList.get(position));
    imageStatusList.toString();

    try {

        if (imageStatusList.get(position).equals("approved")) {
            holder.rimage.setImageDrawable(fragmentActivity.getResources().getDrawable(R.drawable.varified));
        } else if (imageStatusList.get(position).equals("pending")) {
            holder.rimage.setImageDrawable(fragmentActivity.getResources().getDrawable(R.drawable.pending));

        }

    }
    catch (Exception e){
        e.printStackTrace();
    }


}

RecyclerViewHolder.java

public class RecyclerViewHolders extends RecyclerView.ViewHolder implements View.OnClickListener{


public ImageView recipeImage,rimage;
int  likeButtonCount;
String [] imageUrls;
Context mContext;
ImageLoader imageLoader;
DisplayImageOptions options ;
public RecyclerViewHolders(View itemView,String [] urlList,Context mContext) {

    super(itemView);
    itemView.setOnClickListener(this);
    recipeImage = (ImageView)itemView.findViewById(R.id.recipe_image);
    rimage=(ImageView)itemView.findViewById(R.id.image_status_imageview);
    imageUrls=urlList;
    this.mContext=mContext;
    imageLoader = ImageLoader.getInstance();
    options  = new DisplayImageOptions.Builder().cacheInMemory(true)
            .cacheOnDisc(true).resetViewBeforeLoading(true)
            .showImageForEmptyUri(mContext.getResources().getDrawable(R.drawable.wedcake))
            .showImageOnFail(mContext.getResources().getDrawable(R.drawable.wedcake))
            .showImageOnLoading(mContext.getResources().getDrawable(R.drawable.wedcake)).build();



}
 @Override
public void onClick(View view) {
    int position = getPosition();

    ArrayList<String>  imageUrlArrayList = new ArrayList<>();
    imageUrlArrayList.add(imageUrls[position]);
    imagePopupWindow(imageUrlArrayList);
    Toast.makeText(view.getContext(), "Clicked = " + getPosition(), Toast.LENGTH_SHORT).show();
}


public void bind(String items){

    imageLoader.displayImage(items, recipeImage, options);
}

comment
Можете ли вы попробовать это в своем методе onBindViewHolder(), прежде чем устанавливать исходное изображение, попробуйте holder.setImageBitmap(null), а затем установите исходное изображение. В настоящее время то, что происходит, связано с Переработка просмотра   -  person Satyen Udeshi    schedule 21.10.2015
comment
@Satyen Udeshi: Спасибо за ответ; Я попробовал приведенный выше фрагмент, но он не решил мою проблему. Можете ли вы помочь?   -  person Tom    schedule 21.10.2015


Ответы (2)


Как я подозреваю, это связано с View Recycling, вы можете найти много сообщений в самом SO по этому поводу, я публикую некоторый код, который вы можете попробовать и посмотреть, поможет ли это

Объявите DisplayImageOptions options на уровне класса и просто инициализируйте внутри конструктора RecyclerViewAdapter(), в настоящее время вы не используете этот options при загрузке изображений, поэтому UIL не будет загружать кэшированные изображения. Итак, при загрузке изображений вы должны сделать следующее:

imageLoader.displayImage(item,recipeImage,options);

В качестве альтернативы вы также можете попробовать метод ниже, если выше не работает.

Добавьте метод в свой класс RecyclerViewHolder, например:

public void bind(String item){
    // Set your image  here
    imageLoader.displayImage(item,recipeImage,options);
}

и в вашем onBindViewHolder() вызовите этот метод следующим образом ->

//  imageLoader.displayImage(itemList.get(position), holder.recipeImage);
holder.bind(itemList.get(position))

Дайте мне знать, если это поможет

person Satyen Udeshi    schedule 21.10.2015
comment
@Satyan Udeshi: Еще раз большое спасибо за повтор. Я пробовал эти два метода, я обновил приведенный выше код, но моя проблема все еще не решена. Можете ли вы помочь? - person Tom; 22.10.2015

Я столкнулся с таким же поведением в режиме Staggered Grid View, и, похоже, это связано с тем, как элементы размещаются на этапе макета. Стратегия по умолчанию — GAP_HANDLING_MOVE_ITEMS_BETWEEN_SPANS. Согласно документации элементы можно перемещать внутри макета, чтобы скрыть пробелы, которые могут появиться из-за разных размеров элементов.

Вы можете отключить это поведение, изменив стратегию гэпа с помощью setGapStrategy (GAP_HANDLING_NONE). Это остановит изменение столбцов элементов, но вы можете получить большие промежутки в начале или конце столбца.

person Sparga    schedule 24.02.2016