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

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

ただいまの
回答率

88.92%

【Swift】配列へのフィルターのかけかた

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 635

ishicoro

score 24

●やりたいこと
チャットアプリで、まだ開いていないメッセージの数を表示したい。

●試みた方法
配列(複数のkeyとvalueをもつ独自クラス)のうち、filter機能を使って、特定のkeyとvalueの組み合わせの数(=未読のメッセージの数)を数える。

●問題
ちゃんとカウントできていない

以下コードです。
お助け頂けますと幸いです。よろしくお願いいたします!!

class New2: NSObject {

    var sender_id: String?

    var timeStamp: NSNumber?

    var toID: String?

    var listened: Bool? //Firebaseのデータベースから取得するこのlistenedフラグで、既読か未読かを区別

}
var news = [New2]()
var newsDictionary = [String: New2]()

以下viewDidload()内
Database.database().reference().child("Media_Messages").observe(.childAdded, with: { (snapshot) in
            if let dictionary = snapshot.value as? [String: AnyObject]{
                let new2 = New2()
                new2.sender_id = dictionary["sender_id"] as? String
                new2.toID = dictionary["toID"] as? String
                new2.listened = dictionary["listened"] as? Bool

  //送り主が自分なら、表示する名前とアイコンは、受信者。送り主が自分じゃないなら、表示するのは送信者。
                if new2.sender_id == Auth.auth().currentUser?.uid {
                    self.newsDictionary[new2.toID!] = new2
                    self.news = Array(self.newsDictionary.values)
                    self.NewTalkCollectionView.reloadData()

                }
                //送る主が自分以外で、かつ、送り先が自分であるとき
                else if new2.toID == Auth.auth().currentUser?.uid {
                    self.newsDictionary[new2.sender_id!] = new2
                    self.news = Array(self.newsDictionary.values)
                    self.NewTalkCollectionView.reloadData()
                }


            }
        }, withCancel: nil)
func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
        let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "NewTalkCollectionViewCell", for: indexPath) as! NewTalkCollectionViewCell

        let user = news[indexPath.row]
        let numberOfMessages: Int = news.filter{ _ in user.listened == false}.count

        cell.numberOfMessages.text = "\(numberOfMessages)"

上記コードですと、ちゃんとメッセージ数が1と表示され、開いたら、メッセージ数は0になります。
ただし、未読のメッセージが3つの状態でも、メッセージ数は1と表示され、三つのうち1つでも聞いたら、まだ未読が2つ残っているものの、メッセージ数が0と表示されてしまいます。

Filterのところを以下コードに変えてみたりしましたが、下記のエラーが。

 let numberOfMessages: Int = news.filter{ (key: String, value: Bool) -> Bool in return key == "listened" || value  == false}.count

Contextual closure type '(New2) -> Bool' expects 1 argument, but 2 were used in closure body


以下10/6追加

 var numberOfMessages: Int = news.filter {$0.listened == false}.count
 news.enumerated().forEach {
            print($0, $1.listened ?? "nil")
            if let listened = $1.listened {
                if listened {
                    numberOfMessages += 1
                }
            }
        }
        print("numberOfMessages =", numberOfMessages)


こちらの出力が、
0 false
numberOfMessages = 1

//var numberOfMessages: Int = news.filter {$0.listened == false}.count
var numberOfMessages = 0
 news.enumerated().forEach {
            print($0, $1.listened ?? "nil")
            if let listened = $1.listened {
                if listened {
                    numberOfMessages += 1
                }
            }
        }
        print("numberOfMessages =", numberOfMessages)


こちらの出力が、
0 false
numberOfMessages = 0

です。両者とも、開いていないメッセージ(listened = false)が複数ある状態です。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

0

user.listenedってnews[indexPath.row].listenedってことだから、newsの中をループしているのではなく、ずっと同じものを見てます。

news.filter({ (new: New2) -> Bool in new.listened == false}).count

短くすると、

news.filter {$0.listened == false}.count

 test

var numberOfMessages = 0
news.enumerated().forEach {
    print($0, $1.listened ?? "nil")
    if let listened = $1.listened {
        if listened {
            numberOfMessages += 1
        }
    }
}
print("numberOfMessages =", numberOfMessages)

 test(修正版)

var countRead = 0, countUnread = 0, countNil = 0
news.enumerated().forEach {
    print($0, $1.listened ?? "nil")
    if let listened = $1.listened {
        if listened {
            countRead += 1
        } else {
            countUnread += 1
        }
    } else {
        countNil += 1
    }
}
print("count:\(news.count) read:\(countRead) unread:\(countUnread) nil:\(countNil)")

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/10/04 09:26

    おかしな状況のときのログを書いて下さい。

    実行タイミングは私には分かりません。あなたが想定しているたびに表示されていれば正常で、そうでなければ何かがおかしいことになります。(「都度チェックしに行ってる」というのが、あなたが想定している状況なのであれば「正常」ということです)

    キャンセル

  • 2018/10/06 16:57

    fuzzballさん
    申し訳ございません、おかしな状況とは何を指しますでしょうか?
    本文の一番下に、テストコード2パターンの出力結果を追記しました。
    firebase上で、listenedフラグがfalseになっている未読メールが複数ある状態で、実行しています。
    お手数ですが、引き続きアドバイス頂けますと幸いです、よろしくお願いいたします。

    キャンセル

  • 2018/10/09 09:18

    まず、私のテストプログラムは判定が逆になっていて既読をカウントするようになっていました。
    なので、今回のテスト結果だと、
    ・データは1個
    ・既読は0通(未読が1通ということ)
    です。

    >>listenedフラグがfalseになっている未読メールが複数ある状態で、実行しています。
    ということは、フィルタリングが原因ではなく、データが1個しか取れていないことが原因です。
    その原因を調べて下さい。

    テストプログラムは少し修正しておきます。

    キャンセル

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

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

関連した質問

同じタグがついた質問を見る