[Swift]Custom Classの配列重複バグ

Question

みなさま、下記、お力添え頂けますと幸いです... 1ヶ月立ち往生しています...

前提

LINEのようなメッセージングアプリを作っています。
Firebaseから該当メッセージを取得してカスタムクラスのmessages配列に入れています。

class Message {
    private var _senderId = "";
    private var _url = "";
    private var _timeStamp: NSNumber = 1
    private var _toId = "";
    private var _childkey = "";
    private var _listened = false
    //クラスを初期化
    init(senderId: String, url: String, toId: String, timeStamp: NSNumber , childkey: String, listened: Bool) {
        _senderId = senderId;
        _url = url;
        _toId = toId
        _timeStamp = timeStamp
        _childkey = childkey
        _listened = listened
        // 13classes and objects 14 initializers
    }
    var senderId: String?
        return _senderId;    
    }
    var url: String? {
        return _url;
    }
 //中略
}

Aくん、Bくんのの二人とのトークがあり、Aくんとのメッセージ数は18個、Bくんとのメッセージ数は10個です。

Aくん、BくんのそれぞれのトークルームのView ControllerのViewdidappearの中で、messages.countを出力すると、それぞれ18と10が出力され、画面上でも同数のセルが表示されています。

 override func viewDidAppear(_ animated:Bool) {
        print(self.messages.count) //Aくんのトークルームに入ったら18, Bくんのときは10
}
```### 問題点
ここからが非常に困っている箇所です。
別途、iPhone上部のセンサーを覆い隠すとトリガーされる処理を書いています。

```Swift
@objc func proximitySensorStateChanged(){
        print(self.messages.count)
　　　　//中略：音声メッセージを再生する処理が続く
}

ここで、Aくんのトークルームに入った状態でiPhone上部のセンサーを手で隠し、proximitysensorを実際に発動させます。print(self.messages.count)は18と出力されます。ここまでは問題ありません。一度Aくんのトークを出て、Bくんのトークに入りなおします。この状態でproximitysensorを発動させると、print(messages.count)は、18と10が連続で出力されます。先ほどまで入っていた方のトークルームの情報がなぜか残っています。

順番としてまず18を出力してから10を出力されます。確かに音声メッセージを聞いてみても、先にAくんのメッセージが流れ初めて、途中ですぐかぶせる形でBくんの音声が流れ始めます。message配列の箱が二つ作成されるのかと思います。

print(messages.count)という全く同じ処理をしているにも関わらず、proximitysensorの箇所で書くと異なるアウトプットとなってしまうのが不可解です。

ここでは音声再生処理の詳述は避けますが、上記により、Bくんの音声メッセージが聞きたいのに、同時に関係のないAくんのメッセージも流れてしまうという支障が生じています。

参考

Firebaseからメッセージ情報を取得して配列を作る処理は下記の通りです。
各友達との個別トークのViewControllerの中で書いています。
一番下に、proximitysensorに繋げるselectorも記載しています。

 var messages = [Message]()

//override func viewDidLoad()の中
 Database.database().reference().child("user-messages").child(uid!).observe(.childAdded, with: { (snapshot) in
            let messageId = snapshot.key
           DBProvider.Instance.mediaMessagesRef.child(messageId).observe(.value, with: { (snapshot) in
                if let d = snapshot.value as? [String:AnyObject]{
                        if let url = d["url"] as? String {
                            let senderId = d["sender_id"] as? String
                            let toId = d["toID"] as? String
                            let childkey = d["childkey"] as? String
                            let listened = d["listened"] as? Bool
                            let timeStamp = d["timeStamp"] as? NSNumber
                            let newContact = Message(senderId: senderId!, url: url, toId: toId!, timeStamp: timeStamp! , childkey: childkey!, listened: listened!);
                            //自分が送ったボイスかつ、送り先がContactsVCで選んだ友達のとき。
                            if senderId == (Auth.auth().currentUser?.uid)! && toId! == self.toID {
                                self.messages.append(newContact);
                            }
                                //友達が送ったボイスかつ、送り先が自分
                            else if senderId == self.toID && toId == Auth.auth().currentUser?.uid {
                                self.messages.append(newContact);
                            } else {
                              }
                            //自分からそれ以外に送ったボイスや、友達が自分以外に送ったボイスはスルーする。
                           self.messages.sort(by: {Double($0.timeStamp!) < Double($1.timeStamp!)})
                        }
                }
            }, withCancel: nil)
        }, withCancel: nil)


NotificationCenter.default.addObserver(self, selector: #selector(ChatVC.proximitySensorStateChanged), name: UIDevice.proximityStateDidChangeNotification, object: nil)

以上となります。もし何かお気付きの点がございましたら、コメント頂けますと大変幸いです。お手数ですが、よろしくお願いいたします。

Answer 1

もしかしたらですが、個別トークのViewControllerが解放されていない可能性があるかもです。
個別トークのViewControllerが閉じられる時にdeinitが呼ばれているか確認してみてください。
もし呼ばれていないようでしたらClosure内での強参照がその原因の可能性が高いです。
Closure内では弱参照させるために [weak self] をつけてみてください。
例えば以下のうようにしてみてください。

 Database.database().reference().child("user-messages").child(uid!).observe(.childAdded, with: { [weak self] (snapshot) in
    let messageId = snapshot.key
    DBProvider.Instance.mediaMessagesRef.child(messageId).observe(.value, with: { [weak self] (snapshot) in

このようにするとClosure内での自身への参照は弱参照となるので例えば
self.messages.append(newContact);
このような部分では以下のようにすることで参照可能となります。
self?.messages.append(newContact);

見当違いな回答でしたらすいません。

Comments

[ishiishiyay]

k17aさん
ご回答いただきありがとうございます。
ViewControllerから戻る際にdeinitが呼ばれておらず、教えていただいた通りに修正しましたら上手くいきました！！！

ViewDidAppear内に記載している方のprint(self.messages.count)は修正前から、配列の重複などせずにちゃんと出力されるのが理解できませんが、とりあえずやっと解決できて本当によかったです！！

本当にありがとうございます、やっと前に進めそうです！！