用户在我的应用程序中滚动UIViewCollectionCells列表时的奇怪行为(每个单元格中的数据随机变化)

时间:2016-04-08 22:11:07

标签: ios swift uicollectionview uicollectionviewcell uicollectionreusableview

在我的应用中,我正在使用UICollectionView,我决定在下面的代码中使用它:

class UserList: UIViewController, UICollectionViewDataSource, UICollectionViewDelegate {

@IBOutlet weak var tview: UICollectionView!
let reuseIdentifier = "cell" 
var items = NSMutableArray()

func collectionView(collectionView: UICollectionView, cellForItemAtIndexPath indexPath: NSIndexPath) -> UICollectionViewCell {

    let cell = tview.dequeueReusableCellWithReuseIdentifier(reuseIdentifier, forIndexPath: indexPath) as! MyCollectionViewCell

    let user:SingleUser =  self.items[indexPath.item] as! SingleUser   

    cell.username.text = user.name


        if let checkedUrl = NSURL(string: user.photo) {
            cell.userImg.contentMode = .ScaleAspectFit

            getDataFromUrl(checkedUrl) { (data, response, error)  in
                dispatch_async(dispatch_get_main_queue()) { () -> Void in
                    guard let data = data where error == nil else { return }
                    print(response?.suggestedFilename ?? "")
                    cell.userImg.image = UIImage(data: data)
                }
            }

        }

    return cell
}

所以我有一个UILabelUIImage的单元格,对于从json获取的每个对象,我解析它并将用户的数据分配给该单元格。

当我加载窗口时,我会看到带有照片的用户名,但是当我开始滚动浏览集合视图时,用户的照片会发生变化,用户会得到不同的照片。

我读过它可能是与细胞重用有关的东西(到目前为止我一无所知),我只是假设应该加载一次(当用户打开这个面板时),然后保持原样。但是,每次用户滚动该列表时,似乎都会“获取”此数据。

那么我怎样才能确切地解决我的问题并确保每次用户滚动列表时 - 数据是否正确?

这里可能有用的另一件事 - 方法getDataFromUrl从网址中提取照片:

func getDataFromUrl(url:NSURL, completion: ((data: NSData?, response: NSURLResponse?, error: NSError? ) -> Void)) {
    NSURLSession.sharedSession().dataTaskWithURL(url) { (data, response, error) in
        completion(data: data, response: response, error: error)
        }.resume()
}

1 个答案:

答案 0 :(得分:1)

尤金所说的是正确的,但如果你不想改变你已经太多的代码。你可以在SingleUser上添加一个可选的UIImage属性,称之为“image”,然后你可以这样说:

if user.image == nil{
  getDataFromUrl(checkedUrl) { (data, response, error)  in
    dispatch_async(dispatch_get_main_queue()) { () -> Void in
     guard let data = data where error == nil else { return }
     print(response?.suggestedFilename ?? "")
     let image  = UIImage(data: data)
     user.image = image
     //Check if this cell still has the same indexPath, else it has been dequeued, and image shouldn't be updated
     if collectionView.indexPathForCell(cell) == indexPath
     {
       cell.userImg.image = image
     }
    }
   }
  }else{
    cell.userImg.image = user.image!
  }
}