Home > Back-end >  Images from UICollectionView changes to wrong images while scrolling
Images from UICollectionView changes to wrong images while scrolling

Time:05-25

I found some similar questions but none of them worked for me. I have a UICollectionView acting as a feed with posts that can contain images. I fetch all the information (images included) from Firebase and when refreshing or scrolling up/down the collection, images change from one cell to another. I know this is for downloading the image & delays but I'm using SDWebImage already and tried prepareForReuse but none of this worked. Here's some code:

class HomeViewController: UICollectionViewController { 
        override func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
              let cell = collectionView.dequeueReusableCell(withReuseIdentifier: homeImageTextCellReuseIdentifier, for: indexPath) as! HomeImageTextCell
              cell.delegate = self

              //Here I have the information with the URL image downloaded from FB
              cell.viewModel = PostViewModel(post: posts[indexPath.row])

              return cell
        }
}

The HomeImageTextCell is a UICollectionViewCell with a collectionView inside to display the images, posts can have more than one to display. Inside this cell:

class HomeImageTextCell: UICollectionViewCell {

private let collectionView: UICollectionView = { ... }

func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
        let cell = collectionView.dequeueReusableCell(withReuseIdentifier: reuseIdentifier!, for: indexPath) as! ImageCollectionViewCell
        cell.viewModel = PostImageViewModel(imageString: imagesToDisplay[indexPath.row])
        return cell
    }

}

And finally, the ImageCollectionViewCell just has an image to display the image fetched from Firebase. This cell is the one that gets messy:

class ImageCollectionViewCell: UICollectionViewCell {
    
    var viewModel: PostImageViewModel? {
        didSet {
            configure()
        }
    }
    
    //MARK: - Properties
    
    lazy var postImageView: UIImageView = {
        let iv = UIImageView()
        iv.contentMode = .scaleAspectFit
        iv.clipsToBounds = true
        return iv
    }()
    
    //MARK: - Lifecycle
    
    override func prepareForReuse() {
        super.prepareForReuse()
        self.postImageView.isHidden = true
    }
    
    override init(frame: CGRect) {
        super.init(frame: frame)
        addSubview(postImageView)
        postImageView.anchor(top: topAnchor, left: leftAnchor, right: rightAnchor)
    }
    
    required init?(coder: NSCoder) {
        fatalError("init(coder:) has not been implemented")
    }
    
    func configure() {
        guard let viewModel = viewModel else { return }
        postImageView.sd_setImage(with: viewModel.imageToDisplay, placeholderImage: UIImage())
       
    }
}

CodePudding user response:

try postImageView.sd_cancelCurrentImageLoad() in your prepareForReuse()

CodePudding user response:

add cell.layoutIfNeeded() before returning cell.

func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
    let cell = collectionView.dequeueReusableCell(withReuseIdentifier: reuseIdentifier!, for: indexPath) as! ImageCollectionViewCell
    cell.viewModel = PostImageViewModel(imageString: imagesToDisplay[indexPath.row])
    cell.layoutIfNeeded()
    return cell
}
  • Related