UITableViewで表示される順がおかしい

Question

##前提・実現したいこと
tableViewへ受信したメッセージを順に表示したいのですが、うまく実装できません。
送信者は一定周期(動作時5秒ごと)でメッセージを送信、受信者は一定周期でメッセージを表示しています。
しかし、メッセージを表示する際に、めちゃくちゃな順番で表示されてしまい、原因がわからず困っています。
少なくとも1から順にメッセージが表示されるように、
もしできるなら、新しいものが上に(5回送るなら5 4 3 2 1の順)に表示されるようにしたいです。

##発生している問題
メッセージを5つ順に送った場合、
表示が上から 5 2 1 4 3 となります。
3件だと上から 1 3 2 の順で、
毎回ランダムにメッセージの表示順が入れ替わってしまいます。

##該当のソースコード

var mssList: [String: String] = [String: String]() // 受信メッセージ[送信者名： 受信メッセージ]
var viewList: [String] = [String]() // UIへ表示するリスト
var timer = Timer.scheduledTimer(withTimeInterval: 5, repeats: true){ timer in
        updateView() }

......

extension ViewController: UITableViewDelegate, UITableViewDataSource {
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "cell", for: indexPath)
        cell.textLabel?.font = UIFont(name: "cell", size:9)

        for (key,value) in mssList {
            let text = key + ": \t" + value

            viewList.append(text)
            /// 表示リストの管理
            if (viewList.count > 30) {
                viewList.remove(at: 0)
            }
        }
        
        cell.textLabel?.text = viewList[indexPath.row]
        viewList.removeAll()
        
        return cell
    }

    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return viewItemList.count
    }


    func updateView(){
        DispatchQueue.global().async {
            DispatchQueue.main.async {
                self.viewer.reloadData()
            }
        }
    }
}

##自分で調べたことや試したこと
送信者は間違いなく1から順にメッセージを送っています。
また、途中でviewList.removeAll()を使っていますが、これを外すと5回メッセージを送ると
1 1 2 1 2 と表示されます。

##使っているツールのバージョンなど補足情報
Swift 5.0
Xcode 11.0
iOS 12.3
MacOS Mojave 10.14.6

都合上、アップデートができません。
ご助力いただけますと幸いです。

Answer 1

mssListは辞書なので、順序が保証されません。一度配列にコピーしてから、キーでソートしてみてはどうでしょうか。

Comments

[saku_panda]

ありがとうございます。
追加でメッセージの受信時間を取れるようにして、
こちらhttps://teratail.com/questions/189748を参考に、
構造体の配列にして時間でソート試みます。

12/26 追記
無事に動作しました。
ありがとうございました。

Answer 2

とりあえず解決に向けた道順を記します。

セルの数はviewItemList.countを使っているにも関わらず、cellの生成にはviewList[indexPath.row]を使っており、（やりたいことはなんとなくわかりますが）この方法は事故りやすいので、同じ配列を参照するようにしてみてください。

また、func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath)はセルを生成する事に特化し、中でなんらかのリストを更新するのはやめてください。

①「受信したメッセージを表示するための配列」が正しく作れてから、②それをテーブルビューに反映させるというイメージで組んでください。
①と②を同時に実装しようとしているのでうまくいっていないと思います。
①が完璧に動作してから、②を実装してください。

Comments

[saku_panda]

詳細な説明ありがとうございます。
①を解決するのに手間取り、返事が遅れてしまいました。申し訳ありません。
表示する構造体の配列作成後、教えていただいたセルの数・生成時の参照配列を見直したところ、無事に動作しました。
ありがとうございました。