CODEWITHSUNDEEP
androidretrofitandroid-glide
Christopher
Christopher

Reputation: 149

Long display time for pictures using Glide

I am doing an API request to Imgur using Retrofit in order to retrieve about 40 URLs and then display them in a RecyclerView using Glide like such:

@Override
public void onBindViewHolder(ViewHolder holder, int position) {
    Glide
            .with(context)
            .load(list.get(position).getLink()) // Gets the URL in my list
            .centerCrop()
            .placeholder(R.mipmap.placeholder)
            .crossFade()
            .into(holder.imageView);
}

The request gets answered quickly, but most of my images stay as placeholders as some of them appear one by one later on.

What is causing this delay in the display of my images? Is it perhaps linked to the speed of my Internet connection?

Additionally is my approach a correct one when it comes to "large" amounts of pictures?

Please note that most ImageViews do not load, even the ones that are visible to my user.

Upvotes: 0

Views: 544

Answers (4)

Hookah_Smoka
Hookah_Smoka

Reputation: 374

Once you've set the url for the image in Glide, the image itself still needs to be downloaded and shown in the ImageView, which is causing the delay. If the images are not loading, can you check if the image url loads the image in your browser for example?

A better approach to load images in an adapter with Glide is to use a RequestManager which you pass in the constructor of your adapter. Glide will then subscribe to the lifecycle of your activity of fragment. This will prevent images from being downloaded when your activity of fragment has been destroyed, which is especially useful when Glide has to download large images.

public MyAdapter(List<Object> items, RequestManager requestManager) {
    this.items = items;
    this.requestManager = requestManager;
}    

@Override
public void onBindViewHolder(ViewHolder holder, int position) {
    requestManager
            .load(list.get(position).getLink()) // Gets the URL in my list
            .centerCrop()
            .placeholder(R.mipmap.placeholder)
            .crossFade()
            .into(holder.imageView);
}

And then you can call the constructor of your adapter in an acitivty or a fragment as such:

MyAdapter myAdapter = new MyAdapter(items, Glide.with(this));

I've been using this approach after I've found this answer from TWiStErRob.

Upvotes: 1

megaturbo
megaturbo

Reputation: 677

This might be depending on your internet connection. If so, you can use imgur smaller sizes, you can add one of those letter at the end of your filename:

  • s = Small Square (90×90)
  • b = Big Square (160×160)
  • t = Small Thumbnail (160×160)
  • m = Medium Thumbnail (320×320)
  • l = Large Thumbnail (640×640)
  • h = Huge Thumbnail (1024×1024)

i.e. this:

http://i.imgur.com/bF8zPs4.jpg

becomes this (for the small square):

http://i.imgur.com/bF8zPs4s.jpg

Upvotes: 1

Harshil
Harshil

Reputation: 1000

In Recyclerview, Glide only loads images in visible imageViews. ImageViews that are not visible will be loaded once you scrolldown. This is due to recycler property of RecyclerView.

Add following line in you onBindViewHolder and see the logs, you will understand it: 

Log.w("Link", list.get(position).getLink());

Upvotes: 0

Ramzy Hassan
Ramzy Hassan

Reputation: 966

it is depend on your internet connection speed and the size of images.
if the size of images it to large,it may cause of of Memory exception.

Upvotes: 0

Related Questions