cellにつけたチェックマークをUserDefaultsに保存して、view読み込み時に保存したデータからcellにチェックマークをつけたい

Question

セルを表示してチェックマークをつけることは出来たのですが、そこからチェックマークをつけたセルのIndexPathをUserDefaultsに保存して、view読み込み時に、保存したデータからcellに再びチェックマークをつけることが出来ません。具体的なコードは以下のものです。

//ボタンを押すとチェックマークのついたセルのIndexPathをUserDefaultsに保存
 @IBAction func hozon(_ sender: UIBarButtonItem) {
        for x in 0...Name.count-1 { //Name.countはrowの数
            let indexPath: IndexPath = IndexPath(row: x, section: 0)
            let cell = nameList.cellForRow(at:indexPath)
            //nameListはTableViewの名前
            if cell?.accessoryType == .checkmark {
                check.append(indexPath) //var check = [IndexPath]()
            }
        }
            //UserDefaultsにcheckの内容を保存
            UserDefaults.standard.set(check, forKey: "GroupData")
    }


//読み込み時にUserDefaultsに保存されているデータからセルにチェックをつける
 override func viewDidLoad() {
        super.viewDidLoad()
       
        //UserDefaultsに保存されていたらIndexPath型で代入
        if UserDefaults.standard.object(forKey: "GroupData") != nil{
            
            check = UserDefaults.standard.object(forKey: "GroupData")! as! [IndexPath]
            
            //チェック入れる
            for x in 0...check.count-1 {
            let indexPath: IndexPath = check[x]
            let cell = nameList.cellForRow(at:indexPath) 
            cell!.accessoryType = .checkmark
            }
        }
}

これは初めに表示されるビューコントローラのプログラムではありません。このプログラムのビューコントローラにsegueでジャンプする時にアプリが落ちてしまいます。また上のプログラムでは下から3行目の「let indexPath: IndexPath = check[x]」の行にThread 1: Fatal error: Index out of rangeと表示されているので、試しにそこのfor分全体を消去すればこのビューコントローラにジャンプすることが出来ました。しかしその状態でセルにチェックをつけてhozonボタンを押してもアプリが落ちてしまいます。どこがダメでどう改善すべきか教えていただきたいです。お願いします。

Answer 1

大前提として、「UITableViewCellは画面に表示されている範囲しか存在しない可能性がある」という事を頭に入れておいてください。

そのため、チェックの状態はメンバー変数等で保持してください。
テーブルからセルの要求が有った時にメンバー変数を元にチェック状態を入れるなりしてください。

Comments

[clocker]

ご回答ありがとうございます。
チェックの状態はどのようなコードで表すことができるのですか？

[takabosoft]

ちょっと今出先で試せませんがSet<IndexPath>のような集合型を使えば良いかと思います。ただ、UserDefaultsにそのまま保存できるかどうかは忘れました（だめなら一度配列にmapしてから保存/読み込みすればよいかと）

[clocker]

let info = Set<IndexPath>() とすると、どこのセルにチェックが入っているかという情報がinfoに保持されると考えて良いのでしょうか。またそれを再び読み込むんでセルに表示する方法がよく分りません。言葉で教えて頂いてもそれをコードで表す能力がまだ足りないので、具体的なコードも一緒に教えて頂ければ幸いです。

[takabosoft]

letだと変更できないのでvarにしてください。そのような感じでよいです。現在セルの要求をテーブルビューから受け取っている箇所（func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: IndexPath) -> UITableViewCell）でセルを生成（または再利用）して中身をいじっていると思いますが、その中でチェックマークを復元すればいいです。コードは自分で書かないと意味がないのでがんばってください。

[clocker]

コードは頑張って自分で書きます。以下のコードで、どこのセルにチェックが入っているかという情報をinfoに保持、そしてUserDefaultsに保存しようと考えています。しかしhozonボタンを押した途端にアプリが落ちてしまいます。どこがいけないんでしょうか。

@IBAction func hozon(_ sender: UIBarButtonItem) {
var info = Set<IndexPath>()
//UserDefaultsにinfoの内容を保存
UserDefaults.standard.set(info, forKey: "GroupData")
}

[takabosoft]

UserDefaultsのset関数は何でも保存してくれるようなものではなくて、ある程度の型の制限があります。Set型はたしか対応していなかった気がするので、配列に変えてから格納してください（map関数使えばすぐにsetから配列へ変換できます）。UserDefaultsから取得するときも配列型として読み込んだものをSet型に変換してください（Set型のコンストラクタで配列を受け取れたような記憶があります）

[clocker]

下記のコードでチェックマークをつけずにhozonを押すとアプリは落ちませんが、チェックマークをつけてhozonを押すと落ちます。試行錯誤の結果、以下の行
UserDefaults.standard.set(IndexInfo, forKey: "GroupData") //配列型をUserDefaultsに保存
を削除すればチェックをつけていてもアプリは落ちないことが分かりました。この行が悪いんだと思いますが、IndexInfoは配列型なので問題はないと思います。どこに問題がありますか。

@IBAction func hozon(_ sender: UIBarButtonItem) {
var info = Set<IndexPath>() //Set型の変数を定義
for x in 0...Name.count-1 {
let indexPath: IndexPath = IndexPath(row: x, section: 0)
let cell = nameList.cellForRow(at:indexPath)
if cell?.accessoryType == .checkmark { //チェックマークがあれば
info.insert(indexPath) //Set型の変数infoにindexPathを足していく
}
}
//UserDefaultsにinfoの内容を保存
let IndexInfo:[IndexPath] = info.map{($0)} // Set型から配列に変換
UserDefaults.standard.set(IndexInfo, forKey: "GroupData") //配列型をUserDefaultsに保存
}

[takabosoft]

あ、IndexPath型もUserDefaultsは対応していませんでしたねｗ　IndexPath型は辞書型に変換してみてください。つまり辞書型の配列型を保存すればいけるとおもいます。UserDefaultsで何の型が対応しているかは公式リファレンスを見てみてください。

[takabosoft]

で、二個上のコードではhozon関数内でcell?.accessoryTypeを見ていますが、最初に私が回答したとおり、この考え方は駄目なので、セルをタップした段階でメンバー変数を更新する方式にしてください。（hozon関数を実行したタイミングでは画面外のcellは取得できない可能性が高いため）

[clocker]

rowの数字をInt型の配列にしてUserDefaultsに保存することが出来ました。その配列の数字をIndexPath型のrowに、sectionには0を代入してチェックを付けようと思っています。そのコード（下）を実行すると、またしてもアプリが落ちました。Xcodeを確認すると「order.section = 0」の行で「Thread 1: EXC_BAD_INSTRUCTION (code=EXC_I386_INVOP, subcode=0x0)」が表示されていました。試しに「order.section = 0」の行を消してみると、今度は「order.row = check」の行で同様の表示がありました。なのでIndexPath型への代入が原因だと考えてますが、どうすれば良いでしょうか。

//UserDefaultsに保存されていたら
if UserDefaults.standard.object(forKey: "GroupData(GN)") != nil{
var order = IndexPath()
check = UserDefaults.standard.object(forKey: "GroupData(GN)")! as! [Int]
order.section = 0
for check in check {
order.row = check
//チェック入れる
let cell = nameList.cellForRow(at:order)
cell?.accessoryType = .checkmark
}
}

[takabosoft]

sectionとrowを後から変更しようとすると落ちるようですね。原因は不明ですが、イニシャライザで毎回生成すれば落ちないと思いますので、そちらで回避してください。IndexPath(row: Int, section: Int)

[clocker]

アドバイス有難うございました。おかげでcellにつけたチェックマークをUserDefaultsに保存して、view読み込み時に保存したデータからcellにチェックマークをつけることが出来ました・