質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.11%

CollectionViewにURLから画像を表示

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 1,015

sam3457

score 50

なんどもすみません。。
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
    }


}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+1

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()したらどうなりますでしょうか?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/04/07 23:01

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

    キャンセル

  • 2018/04/07 23:19

    質問を立て直した方がいいと思いますが、
    let dict = posts[(indexPath as IndexPath).row]

    let dict = posts[indexPath.item]
    の結果が同じなのですか?
    その場合
    let postUrl = URL(string: postStr!)
    print(postUrl)
    の表示結果も全く同じでしょうか。

    キャンセル

  • 2018/04/08 15:07

    let dict = posts[(indexPath as IndexPath).row]

    let dict = posts[indexPath.item]
    の結果は同じでした。

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

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

    キャンセル

+1

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

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

参考: 公式ドキュメント

// コード例

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

...

override func viewDidUnload() {
    super.viewDidUnload()

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/04/07 17:51

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.11%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる