CollectionViewにURLから画像を表示

Question

なんどもすみません。。
FirebaseRDBに保存した画像のURLをCollectionViewに表示する処理を書いています。
以下のコードで起動してもcollectionviewには表示されませんでした。

RDBからURLの取得自体ができていないのかと思い、print(画像のURL)で確認してみたのですが、URLは取得できていました。

残るは原因はcellForItemAtのところくらいだと思うのですが自分には何が原因なのかわかりませんでした。

ご存知の方がいらっしゃいましたらよろしくお願いいたします。

import UIKit
import Firebase
import SDWebImage

class PostList: UICollectionViewController {

    var posts = [NSDictionary]()
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        //ログインチェック
        if UserDefaults.standard.object(forKey: "check") != nil {
            //何もしない
        } else {
            let loginViewController = self.storyboard?.instantiateViewController(withIdentifier: "login")
            self.present(loginViewController!, animated: true, completion: nil)
        }
        
        loadAllData()
        collectionView?.reloadData()
        
    }
    
    @IBAction func loginOrOut(_ sender: Any) {
        performSegue(withIdentifier: "goLogin", sender: nil)
    }
    
    
    //RDBにある投稿画像の参照(URL)を取ってくる
    func loadAllData() {
        
        let RDBRef = Database.database().reference().child("posts")
        RDBRef.queryLimited(toLast: 10).observe(DataEventType.childAdded, with: { (snapshot) in
            var tempPosts = [NSDictionary]()
            for post in(snapshot.children) {
                
                let child = post as! DataSnapshot
                let dict = child.value
                tempPosts.append(dict as! NSDictionary)
                print(tempPosts)
            }
            self.posts = tempPosts
            
        })
 
    }

    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
    }

    /*
    // MARK: - Navigation

    // In a storyboard-based application, you will often want to do a little preparation before navigation
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        // Get the new view controller using [segue destinationViewController].
        // Pass the selected object to the new view controller.
    }
    */

    // MARK: UICollectionViewDataSource

    override func numberOfSections(in collectionView: UICollectionView) -> Int {
        return 1
    }


    override func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
        return posts.count
    }

    override func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
        let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "cell", for: indexPath)
        
        let dict = posts[(indexPath as IndexPath).row]
        
        //投稿画像(postimageを表示)
        let postImageView = cell.contentView.viewWithTag(1) as! UIImageView
        let postStr = dict["postimage"] as? String
        let postUrl = URL(string: postStr!)
        postImageView.sd_setImage(with: postUrl)
        
        //let iconImageView = cell.contentView.viewWithTag(2) as! UIImageView
        //let nameLabel = cell.contentView.viewWithTag(3) as! UIImageView
        
        
        
        
        return cell
    }

    
}

Answer 1

516kさんのご回答の内容で動作はすると思いますので、1点だけ補足します。

ご質問のコードではメモリリークを起こしてしまっています。
Firebase RDBのデータ監視リスナーはデタッチしなければイベントがずっと送られてくる状態になってしまいますので、 viewDidUnload あたりでデタッチするようにすべきです。

参考: 公式ドキュメント

// コード例

let handle = RDBRef
    .queryLimited(toLast: 10)
    .observe(...) { ... }

...

override func viewDidUnload() {
    super.viewDidUnload()

    // リスナーのデタッチ
    RDBRef.removeObserver(withHandle: handle)
}

Comments

[sam3457]

お返事が遅くなって申し訳ありません。
回答ありがとうございます！いつもお世話になっております。
メモリリーク、、動作のことばかり調べたりしていたのでそちらの方は完全に蔑ろになっておりました、、
メモリリークについてはほとんど知識が無かったので、これを機に詳しく調べて他の処理でも意識してみることにします。非常にありがたいです。。
ありがとうございますm(_ _)m

Answer 2

collectionView?.reloadData()

のタイミングの問題が気になりました。
上記の質問のコードではおそらく非同期処理にてデータベースにデータを問い合わせ、実際にデータが返ってくる前にcollectionView?.reloadData()が実施されているように見受けられます。

RDBRef.queryLimited(toLast: 10).observe(DataEventType.childAdded, with: { (snapshot) in
            var tempPosts = [NSDictionary]()
            for post in(snapshot.children) {

                let child = post as! DataSnapshot
                let dict = child.value
                tempPosts.append(dict as! NSDictionary)
                print(tempPosts)
            }
            self.posts = tempPosts
            self.collectionView?.reloadData()
        })

このようにメソッドの完了時クロージャ内でデータをpostsにセットした後にreloadData()したらどうなりますでしょうか？

Comments

[516k]

この完了時クロージャがメインスレッドでない場合は、メインスレッドにDispatchしてreloadData()して下さい。

[sam3457]

お返事が遅くなってしまい申し訳ありません。
回答ありがとうございます！
非同期処理でデータが帰ってくる前にcollectionView?.reloadData()が呼ばれていたのですね、、
それを防ぐためにクロージャ内でreloadData()を呼ぶ必要があったのですね！
515k様にお教えいただいた様に、メソッドの完了時クロージャ内でデータをpostsにセットした後にreloadData()をすることでcollectionviewに表示される様になりました！
ありがとうございますm(_ _)m

もう一つお聞きしたいのですが、投稿画像は違うものを複数投稿していたはずなのですが表示される画像は全部同じものになっていました。。
postsに入れるURLが全部同じになってしまっているか、cellForItemAt内で最初のURLだけを何個も表示しているのかだと思ったのですが、自分にはどちらも異常ない様に見えてしまいます、、
質問のコードから516k様の回答のご指摘部分以外は変えていないのですが、もしよろしければそちらの方もご覧いただけると幸いです。

[516k]

let dict = posts[(indexPath as IndexPath).row]
のIndexPathのキャストは何の意味がありますかね？ひょっとしたらここで毎回ゼロが返ってきてるかもしれません。

[sam3457]

少し前に普通にこの様に
let dict = posts[indexPath.row]
記述したらなぜか画面が真っ白になって、いろいろ試しているうちにキャストをして普通に画面が表示される様になったのですが、

今、キャストを解いて
let dict = posts[indexPath.row]
で起動してみたのですが今度はちゃんと表示されました、、笑

多分違う部分の修正の効果をそこの効果と勘違いしてその様に記述していたものと思われます、、

let dict = posts[indexPath.row]
に直して起動してみた結果なのですが、先ほどと同じく同じ画像がずらっと並んでいました、、

[sam3457]

分かり難かったかもしれないので一応、
let dict = posts[indexPath.row]に直して起動してみた結果はやはり同じ画像がずらっと表示されていました。
ということです！

[516k]

今回のviewはtableではなくてcollectionなので、indexPath.rowではなくてindexPath.itemではいかがでしょう？

[sam3457]

今、.itemで確認してみたのですがやはり結果は同じでした、、
posts配列の中身が同じになっているのかなとも思ったのですが、print()で確認してURLが違うことを確認していますし、それもないですよね、、

[516k]

質問を立て直した方がいいと思いますが、
let dict = posts[(indexPath as IndexPath).row]
と
let dict = posts[indexPath.item]
の結果が同じなのですか？
その場合
let postUrl = URL(string: postStr!)
print(postUrl)
の表示結果も全く同じでしょうか。

[sam3457]

let dict = posts[(indexPath as IndexPath).row]
と
let dict = posts[indexPath.item]
の結果は同じでした。

print(postUrl)の表示結果は異なるURLがpostsの数分だけコンソールに表示されました。

そうですね、、申し訳ありません。
質問を立て直してみます。
最後までありがとうございますm(_ _)m