Firebase Realtime Databaseで値が更新された際に変数が大量更新される

以下のプログラムでFirebase Realtime Databaseから配列を取得し、TableViewを動的に更新しているのですが、データベース上にある元々の配列が更新された瞬間に変数が一瞬の間に大量に更新されてしまっています。

Swift4
1override func viewDidLoad() {
2            super.viewDidLoad()
3            //インスタンスを作成
4            DBRef = Database.database().reference()
5            
6            //繰り返し
7            Timer.scheduledTimer( //TimerクラスのメソッドなのでTimerで宣言
8                timeInterval: 1, //処理を行う間隔の秒
9                target: self,  //指定した処理を記述するクラスのインスタンス
10                selector: #selector(self.newArray(_:)), //実行されるメソッド名
11                userInfo: nil, //selectorで指定したメソッドに渡す情報
12                repeats: true //処理を繰り返すか否か
13            )
14            
15        }
16        
17        @objc func newArray(_ sender: Timer) { //(_ sender: Timer) Timerクラスのインスタンスを受け取る
18            self.hogearray = []
19            let defaultPlace = DBRef.child("table/orderorder")
20            defaultPlace.observe(.value, with: { snapshot in
21                for item in (snapshot.children) {
22                    let snapshot = item as! DataSnapshot
23                    let dict = snapshot.value as! String
24                    self.hogearray.append(dict)
25                }
26                print (self.hogearray)
27                
28                DispatchQueue.main.async {
29                    self.tableView.reloadData()
30                }
31                
32            })
33        }

printしたデータの一部は以下です。（元々["2", "4", "3"]が格納されていた配列に要素["9"]を足した例）

["2", "4", "3"]
["2", "4", "3"]
["2", "4", "3"]
["2", "4", "3"]
["2", "4", "3"]
["2", "4", "3"]
["2", "4", "3"]
["2", "4", "3"]
["2", "4", "3"]
["2", "4", "3", "2", "4", "3", "9"]
["2", "4", "3", "2", "4", "3", "9", "2", "4", "3", "9"]
["2", "4", "3", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9"]
["2", "4", "3", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9"]
["2", "4", "3", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9"]
["2", "4", "3", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9"]
["2", "4", "3", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9"]
["2", "4", "3", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9", "2", "4", "3", "9"]

数秒経つと正しい結果["2", "4", "3", "9"]に戻るのですが、原因はどこにあるのでしょうか。

追記：["002","006"]に要素["004"]を足した例

@["002", "006"]@
@["002", "006"]@
@["002", "006"]@
@["002", "006", "002", "006", "004"]@
@["002", "006", "002", "006", "004", "002", "006", "004"]@
@["002", "006", "002", "006", "004", "002", "006", "004", "002", "006", "004"]@
@["002", "006", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004"]@
@["002", "006", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004"]@
@["002", "006", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004"]@
@["002", "006", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004"]@
@["002", "006", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004"]@
@["002", "006", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004"]@
@["002", "006", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004", "002", "006", "004"]@
@["002", "006", "004"]@
@["002", "006", "004"]@
@["002", "006", "004"]@

追記

@["002", "006", "004", "004"]@
@["002", "006", "004", "004"]@
@["002", "006", "004", "004"]@
@["002", "006", "004", "004", "002", "006", "004", "004", "007"]@
@["002", "006", "004", "004", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007"]@
@["002", "006", "004", "004", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007"]@
@["002", "006", "004", "004", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007"]@
@["002", "006", "004", "004", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007"]@
@["002", "006", "004", "004", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007"]@
@["002", "006", "004", "004", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007", "002", "006", "004", "004", "007"]@
@["002", "006", "004", "004", "007"]@
@["002", "006", "004", "004", "007"]@
@["002", "006", "004", "004", "007"]@
@["002", "006", "004", "004", "007"]@
@["002", "006", "004", "004", "007"]@

追記

hoge
@["002", "006", "004"]@
hoge
@["002", "006", "004"]@
hoge
@["002", "006", "004"]@
@["002", "006", "004", "002", "006", "004", "008"]@
@["002", "006", "004", "002", "006", "004", "008", "002", "006", "004", "008"]@
@["002", "006", "004", "002", "006", "004", "008", "002", "006", "004", "008", "002", "006", "004", "008"]@
@["002", "006", "004", "002", "006", "004", "008"]@
@["002", "006", "004", "002", "006", "004", "008", "002", "006", "004", "008", "002", "006", "004", "008", "002", "006", "004", "008"]@
@["002", "006", "004", "002", "006", "004", "008", "002", "006", "004", "008"]@
@["002", "006", "004", "002", "006", "004", "008", "002", "006", "004", "008", "002", "006", "004", "008", "002", "006", "004", "008", "002", "006", "004", "008"]@
@["002", "006", "004", "002", "006", "004", "008", "002", "006", "004", "008", "002", "006", "004", "008"]@
@["002", "006", "004", "002", "006", "004", "008", "002", "006", "004", "008", "002", "006", "004", "008", "002", "006", "004", "008", "002", "006", "004", "008", "002", "006", "004", "008"]@
@["002", "006", "004", "002", "006", "004", "008", "002", "006", "004", "008", "002", "006", "004", "008", "002", "006", "004", "008"]@
@["002", "006", "004", "002", "006", "004", "008", "002", "006", "004", "008", "002", "006", "004", "008", "002", "006", "004", "008", "002", "006", "004", "008", "002", "006", "004", "008", "002", "006", "004", "008"]@
hoge
@["002", "006", "004", "008"]@
hoge
@["002", "006", "004", "008"]@

fuzzball

2018/09/13 07:40

print("@\(self.hogearray)@") に変えて出力を教えて下さい。

kkkkkkkt

2018/09/13 07:48

編集させていただいた通りの結果になりました。

fuzzball

2018/09/13 09:13

forからprintまで、全部 DispatchQueue.main.async { の中に入れるとどうなりますか？

kkkkkkkt

2018/09/13 09:27

もう一度追記した通りです。繰り返される回数が減り、動作が早くなった（軽くなった）感じがしますが、まだ代入は止まらないようです。

fuzzball

2018/09/13 09:59

期待したような結果にならなかったのでコードは元に戻しておいて下さい。で、newArray(_:)の呼び出し間隔を5秒にするとどうなりますか？

kkkkkkkt

2018/09/13 10:10

全く同じ結果でしたが、異常な値が出ている行は要素の追加直後にいっぺんに出力され、それから5秒経って更新後の正しい値が出力されました。

fuzzball

2018/09/13 11:31

一回のnewArray(_:)の処理時間ってどれくらいか分かりますか？

kkkkkkkt

2018/09/13 11:47

例えば今回の例ではどの行もきっちり5秒おきに出力されたので、異常があるときも特に処理時間はかかっていないように思います。

fuzzball

2018/09/13 12:45

newArray(_:)の先頭にprint文を追加して、そのprint文の出力とprint (self.hogearray)の間に、次のnewArray(_:)が呼び出されていないか確認してみて下さい。

kkkkkkkt

2018/09/13 13:15

編集させていただいた通り、異常のある処理の中でnewArray(_:)は一度しか呼び出されていないようです。forループの中でのエラーということでしょうか…

fuzzball

2018/09/13 14:17

どうも思っていたのと違うみたいで、observeが何をしているかも分からないのでちょっとお手上げですね。

kkkkkkkt

2018/09/13 14:31

そうですか… お力添え頂き、ありがとうございました。

fuzzball

2018/09/13 16:09

今度からFirebaseの質問には Firebase のタグ付けましょう。（この質問も付けていたらすぐに解決していたはず）

行動規範の内容に同意します

回答1件

ベストアンサー

Firebaseの DatabaseReference#observe メソッドは、第一引数にDataEventType.valueを渡した場合、指定したパス以下で値の変化があるたびにコールバックが呼び出されます。また、このobserveによるデータ監視は removeObserverWithHandle などのメソッドを呼ばない限り終了しません。

つまり、データが更新されるたびに self.hogearray に全要素が追加されていきますし、監視を終了しない限りこれはずっと続きます。

コードの意図がいまいちわからないのですが、おそらくobserveがやってくれることを自分でやろうとしておかしくなっているものと思います。以下の方針で修正してみてください。

Timerを使って自分で監視する必要はないので、observeを一度だけ呼ぶ
データ更新のたびにhogearrayに全要素を追加していく処理はやめる
データの監視を終えるときにremoveObserverWithHandleを呼ぶ

swift
1private var observer: UInt = 0
2
3// 修正例
4override func viewDidLoad() {
5    super.viewDidLoad()
6    
7    DBRef = Database.database().reference()
8
9    let defaultPlace = DBRef.child("table/orderorder")
10    self.observer = defaultPlace.observe(.value, with: { snapshot in
11        var array: [String] = []
12        for item in (snapshot.children) {
13            let snapshot = item as! DataSnapshot
14            let dict = snapshot.value as! String
15            array.append(dict)
16        }
17
18        DispatchQueue.main.async {
19            // ここでarrayの内容をhogearrayに代入
20            self.hogearray = array
21            print(self.hogearray)
22
23            self.tableView.reloadData()
24        }
25    })
26}
27
28// データ監視の終了の例（タイミングは適当です）
29override func viewDidUnload() {
30    super.viewDidUnload();
31    DBRef.child("table/orderorder").removeObserver(withHandle: self.observer)
32}