tableViewCellのIndexPathを取得したい。

Question

こちら↓の質問から派生しています。
リンク

やりたいこと

データ表示しているtableViewCellのIndexPathをIntで取得したいです。

試したこと・試していること

• macOSのtableViewメソッドにあるselectAllのiOS版(tableViewで使用可能な)がないか探したり尋ねたりしました。

→結果：ないので自作、という結論に。

• selectRowメソッドをfor in文内で使用し、どのような動きになるのか確認したいと思い試している最中でIndexPathがIntで所得できずに詰まっています。(IndexPath(Int)ではなく[IndexPath]で取得してしまっています。)

以下、[IndexPath]で取得してエラーが出ているコードです。なお、tableViewCellに表示させるデータはfirebase firestoreから取得しています。(UITableViewDelegateは省略しております)

import UIKit
import Firebase

class DraftViewController: UIViewController, UITextFieldDelegate {
    
    let db = Firestore.firestore()
    let user = Auth.auth().currentUser
    private var draftData:[QueryDocumentSnapshot] = []

  override func viewDidLoad() {
        super.viewDidLoad()
        
        //複数選択できるようにする
        tableView.allowsMultipleSelectionDuringEditing = true
    　　//firestoreのdraftsデータを取得
        getDraftsData()
  }

  //firestoreのdraft data を取得
  func getDraftsData() {
        guard let uid = user?.uid else { return }
        db.collection("drafts").whereField("uid", isEqualTo: uid).getDocuments() { (querysnapshot, err) in
            if let err = err {
                print("error getting documents: (err)")
            } else {
                let personalDraftData = querysnapshot!.documents.sorted {personalDraftData,provisionalDraftData in
                    guard let personalDraftsData = personalDraftData.data()["timestamp"] as? Timestamp,
                        let provisionalDraftsData = provisionalDraftData.data()["timestamp"] as? Timestamp else { return false }
                    return (personalDraftsData.dateValue() > provisionalDraftsData.dateValue())
                }

                self.draftData = personalDraftData
                self.tableView.reloadData()
            }
        }
    }
}

extension DraftViewController: UITableViewDataSource {

    @IBAction func tapCloseButton(_ sender: Any) {
        guard let indexPaths = tableView.indexPathsForVisibleRows else { return }
        let allIndexPaths = indexPaths.count
        guard let imageView = UIImage(named: "check mark") else { return }

        if closeButton.image == nil && closeButton.title == "Check all" { //全てのtable cell を選択する
            for _ in 0 ..< allIndexPaths {
                tableView.selectRow(at: indexPaths, animated: true, scrollPosition: .none)
            }   
        } else if closeButton.image == nil && closeButton.title == "Unselect all" {
            // 選択しているcellのチェックマークを全て外す(クリアにする)   
        }
    }
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return draftData.count
    }
    
    //各cellの要素設定(インスタンスを生成する)
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        // cellを取得する
        guard let cell = tableView.dequeueReusableCell(withIdentifier: "draftTableCell", for: indexPath) as? CustomizedTableViewCell else {
            return UITableViewCell()
        }
        
        //cellに表示する値を設定する
        cell.titleLabel.text = draftData[indexPath.row].data()["title"] as? String
        cell.detailLabel.text = draftData[indexPath.row].data()["detail"] as? String
       
        cell.titleLabel.numberOfLines = 0
        cell.detailLabel.numberOfLines = 0
        cell.titleLabel.lineBreakMode = .byWordWrapping
        cell.titleLabel.lineBreakMode = .byWordWrapping
  
        return cell
    }
    
    func tableView(_ tableView: UITableView, commit editingStyle: UITableViewCell.EditingStyle, forRowAt indexPath: IndexPath) {
        if editingStyle == .delete {
        }
    }
}

Comments

[hayabusabusash]

どのあたりでエラーになりますか？
また、どんなエラーが出ているのか教えていただけないでしょうか？

[退会済みユーザー]

追記・修正依頼ありがとうございます！

エラーが出ているのは
２つ目のコードの６行目あたり↓
if closeButton.image == nil && closeButton.title == "Check all" { //全てのtable cell を選択する
for _ in 0 ..< allIndexPaths {
tableView.selectRow(at: indexPaths, animated: true, scrollPosition: .none)
}
の、

`tableView.selectRow(at: indexPaths, animated: true, scrollPosition: .none)`

部分です。
エラー内容は↓です。

Cannot convert value of type '[IndexPath]' to expected argument type 'IndexPath?

[hayabusabusash]

返信ありがとうございます！
みにくくて申し訳ないですが、こんな感じにしたらエラーは消えませんか？

if let indexPaths = tableView.indexPathsForVisibleRows {
for indexPath in indexPaths {
tableView.selectRow(at: indexPath, animated: true, scrollPosition: .none)
}
}

[退会済みユーザー]

遅くなりました！
やってみたところ、エラーも出なくなった上にやりたかったことがやれるようになっていっぺんに全部解決致しました!!!!
ありがとうございます!!!( ；∀；)

Answer 1

indexPathsForVisibleRowsを対象に選択を行っても、「見えてる部分」しかチェックが入りませんが、大丈夫でしょうか？

1000行で試した結果を載せておきます。

import UIKit

class ViewController: UITableViewController {

    
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        tableView.register(UITableViewCell.self, forCellReuseIdentifier: "cell")
        tableView.isEditing = true
        tableView.allowsMultipleSelectionDuringEditing = true
    }

    override func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return 1000
    }
    
    override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "cell", for: indexPath)
        cell.textLabel?.text = "(indexPath.row)"
        return cell
    }

    @IBAction func hoge(_ sender: Any) {
        guard let indexPaths = tableView.indexPathsForVisibleRows else { return }
        for indexPath in indexPaths {
            tableView.selectRow(at: indexPath, animated: false, scrollPosition: .none)
        }
        
        /*for row in 0..<1000 {
            tableView.selectRow(at: IndexPath(row: row, section: 0), animated: false, scrollPosition: .none)
        }*/
    }
}

「すべて選択」を実現したいのであれば、indexPathsForVisibleRowsではなく、全データに対応するIndexPathを自分で生成し、選択を行う必要があります。
コードだとたぶん以下のようになると思います。

for row in 0..<draftData.count {
    let indexPath = IndexPath(row: row, section: 0)
    tableView.selectRow(at: indexPath, animated: true, scrollPosition: .none)
}

Comments

[fuzzball]

見えてないセルはnilですね‥。

[takabosoft]

今回はセルへのアクセスではなく、selectRowを使った選択処理の話ですので、生成されていないCellでも大丈夫です。

[fuzzball]

あ、ごめんなさい。完全に勘違いです！

[退会済みユーザー]

takabosoftさん！返信が大変遅くなりましたorz
昨日ご回答下さっていた
for row in 0..<draftData.count{ ... }
で試したところ見えていない部分までちゃんと一括選択できて、本来やりたかった実装になりました
( ；∀；)ありがとうございます!!!
(以前、for in文で <draftData.count 使った時にエラーが出て上手く行かなかったのでdraftData.countは使えないものと思い込んでおりました... :_|
追加編集で例まであげて丁寧にご解説くださり大変わかりやすかったです！ありがとうございます!!!)

[fuzzball]

直接 draftData.count を使うのではなく tableView(tableView, numberOfRowsInSection: 0) を呼び出すほうが良いと思います。

[退会済みユーザー]

>fuzzballさん
不勉強で大変恐縮なのですが、差し支えなければ「 draftData.count を使うよりも tableView(tableView, numberOfRowsInSection: 0) を呼び出すほうが良い」と考えられている理由を教えていただけませんか？
orz

[fuzzball]

必要なのはセル数であって、データ数ではありません。
現在は、たまたま「セル数 == データ数」なだけです。

例えば、もし（なんらかの理由で）セル数 != draftData.count になったとき、今回の箇所も変更しないといけなくなります。

[takabosoft]

保守性を考えたときに、って意味ですよね。
私もその方が良いと思います。

極論を言えばextension UITableViewでselectAllRow関数などを作って汎用的にうごくものにした方が使いまわしも出来て良いとは思いますが、まあ今回の本題ではないので割愛します。

[退会済みユーザー]

なるほど。。
ありがとうございます！

Answer 2

”質問への追記・修正の依頼”をしてくださったhayabusabusashさんが書いてくださった方法を試みたところ、この質問に対して解決致しました!!!!hayabusabusashさん、ありがとうございます！！！

また、”回答”してくださった** takabosoftさんのおかげで、本来行いたかった実装まで一気に解決いたしました！丁寧な解説でわかりやすくとても助かりました!! takabosoft**さん、ありがとうございます!!!

御二方とも、本当にありがとうございます！( ；∀；)