【swift】TablevVewで特定Cellを非表示(高さを0)にして、詰めて表示するにはどうしたらいいでしょうか。

Question

TableViewに関して質問があります。
勉強会やカンファレンス等のイベント情報を閲覧できるアプリをご想像下さい。

イベント一覧をTableViewに表示していますが、そのviewのまま、
UISwitchのON/OFFで、一覧の中より参加済イベントのみ表示する/全て表示する　と
切り替えられるように変更したいです。

参加済みか否かは「dict["PointPaid"]」に数値が入っているか否かで判断しています。

cell.isHiddenで非表示にすることはできたのですが、
非表示にした箇所が空白で空いており不格好です。

非表示かつ、高さを0にして詰めて表示したいです。
heightForRowAt内でできそうな気がするのですが、記述方法が探せませんでした、、、
アドバイスいただける幸いです。

    var onlyPaid:Bool = false　//スイッチ判断
    var items = [NSDictionary]()　//イベント情報辞書

    @IBOutlet var `switch`: UISwitch!　//スイッチ判断
    @IBOutlet var tableView: UITableView!

    @IBAction func onSwitch(_ sender: UISwitch) {　　//スイッチ切り替え、テーブル再読込
        onlyPaid = sender.isOn
        tableView.reloadData()
    }
    
    func numberOfSections(in tableView: UITableView) -> Int {
        return 1
    }
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return items.count
    }
    
    func tableView(_ tableView: UITableView, heightForRowAt indexPath: IndexPath) -> CGFloat {
        displayHeight = self.view.frame.height
        return displayHeight / 9
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "VotedContentTableViewCell") as! VotedContentTableViewCell

        let dict = items[(indexPath as NSIndexPath).row]

        let resultPoint = dict["PointPaid"] as? Int
        let resultButton = cell.viewWithTag(1) as! UIButton

        if onlyPaid == false{ 　//全て表示
            if resultPoint != nil{
                resultButton.backgroundColor = .purple
                }else{
                resultButton.backgroundColor = .gray
                }
                } else {　//参加済みのみ表示
                    if resultPoint == nil{
                        cell.isHidden = true //参加済みでない為、isHidden = true 
                    }else{
                        resultButton.backgroundColor = .purple　//参加済み
                    }
        return cell
    }
    
    override func viewDidLoad() {
        self.tableView.register(UINib(nibName: "VotedContentTableViewCell", bundle: nil), forCellReuseIdentifier: "VotedContentTableViewCell")

        //tableview
        tableView.delegate = self
        tableView.dataSource = self
        tableView.separatorStyle = .none
        
        //firebase
        if let userID = userid{
        firebaseFunction.observeDataByNSDict( FirebaseURL:"https://firebaseio.com/User/(userID)/ParticipationContent"){ items in
            self.items = items
            self.tableView.reloadData()
            }
        }
        super.viewDidLoad()
    }

<追記>高さを条件分岐

    func tableView(_ tableView: UITableView, heightForRowAt indexPath: IndexPath) -> CGFloat {
        
        if onlyPaid == false{
            return displayHeight / 9
            
        } else {
                let dict = items[(indexPath as NSIndexPath).row]
                if dict["PointPaid"] as? Int == nil{
                    return 0.01
                    } else{
                        return displayHeight / 9
                        }
                }
    }

Answer 1

高さを更新するときにはtableViewかcellを更新してください（reloadDataなど）
なお高さ0は許可されていなく、0.1などとします

Comments

[HNSZ]

いつもアドバイス頂きましてありがとうございます！
本文に追記しておりますが、高さを条件分岐でわけて非表示にすることができました。

ただし、heightForRowAtと、cellForRowAtで同じ記述をしており非効率に感じています。

「 let dict = items[(indexPath as NSIndexPath).row]
if dict["PointPaid"] as? Int == nil　」

こちら、heightForRowAt内で省略することはできないでしょうか？

[kosanai]

通常は、fuzzballさんの言う通りcellを出すかどうかを何らかの方法で決定し、reloadDataをかけます。section、cellの構成をいじることでcellの出し分けができるのがtableViewの利点の一つです。そして最も基本的な方法です（方法1）
構成の変更については、indexPathの管理が若干面倒です。
最近私がやってる管理方法（方法1の追加の工夫）
https://qiita.com/netetahito/items/2e340e5d8503731bd105

今ならStackViewにしてしまう手もあります。StackViewはViewをisHiddenにすると要素が詰められるので便利です（方法2）
ただUITableViewとはまたやり方がガラッと変わるので別のハマりポイントもあります。
ちなみに世の中に何故StackViewの解説が少ないかと言えば、使用できるのがiOS9以降でかなり新しいからです。

話をUITableViewに戻して、アニメーションをしようとし始めると難易度が上がります。
最近分かりやすかったのはこれです（方法3）
https://qiita.com/KikurageChan/items/71d4e1e84d039d1e2f5a

[HNSZ]

kosanai様
色々と方法のご教示ありがとうございます！
足りない知識ばかりで。。とても勉強になりました。

まずはできる範囲でfuzzball様の方法1で実装できました。
ありがとうございました。

[kosanai]

????
ここ結構ハマります

Answer 2

表示ONのアイテムを抽出して、それをTableViewに与えて下さい。

コード追記

var joinedItems = [NSDictionary]() //参加済アイテム保存用
var items = [NSDictionary]() {
    didSet {
        //itemsn更新時にjoinedItemsを生成し直す
        joinedItems = items.filter {$0["PointPaid"] as? Int != nil}
    }
}

という仕組みで、自動的に表示ONのアイテムを抽出できるので、これをTableViewに与えてください。

• セル数取得 items.count → joinedItems
• セル生成時 items[indexPath.row] →　joinedItems[indexPath.row]

動作確認

items += [
    ["id": 0, "PointPaid": 123],
    ["id": 1],
    ["id": 2, "PointPaid": "abc"],
]
print(joinedItems)
/*
[{
    PointPaid = 123;
    id = 0;
}]
*/

items.append(["id": 3, "PointPaid": 999])
print(joinedItems)
/*
[{
    PointPaid = 123;
    id = 0;
}, {
    PointPaid = 999;
    id = 3;
}]
*/

items.remove(at: 0) //id0削除
print(joinedItems)
/*
[{
    PointPaid = 999;
    id = 3;
}]
*/

ひとりごと

参考にした記事が古いんだと思いますが、NSDictionaryではなくDictionaryの方がいいし、いちいちIndexPathをNSIndexPathにキャストしなくていいです。

Comments

[HNSZ]

fuzzball様
いつもありがとうございます！

>var items = [NSDictionary]() {　　 didSet {
この書き方をできることを知りませんでした。。

とても勉強になりました。
追記いただいたコードで実装できました。

>NSDictionaryではなくDictionaryの方がいいし
そうだったのですね。。全体でNSDictionary使っていたので
見直したいと思います！
ありがとうございました。