tableviewでセルを並べ替えた後のデータの保存

Question

前提・実現したいこと

tableviewで編集モード実装しました。
ソート機能で入れ替え時のデータの処理をコーディングしました。
すると以下のエラーが発生しました。

発生している問題・エラーメッセージ

Cannot convert value of type 'String' to expected argument type 

該当のソースコード

 override func tableView(_ tableView: UITableView, moveRowAt sourceIndexPath: IndexPath, to destinationIndexPath: IndexPath) {
        // TODO: 入れ替え時の処理を実装する（データ制御など）
       
        //移動されたデータを取得する。
        let moveData = tableView.cellForRow(at: sourceIndexPath)?.textLabel!.text
              
              //元の位置のデータを配列から削除する。
        dataList.remove(at: sourceIndexPath.row)
              
              //移動先の位置にデータを配列に挿入する。
        dataList.insert(moveData, at:destinationIndexPath.row)
          

    }

試したこと

最下行のmoveDataにエラーメッセージが出たためmoveData!に変更しました

補足情報（FW/ツールのバージョンなど）

dataListは配列です
データ取得時のmoveDataをdataListにinsertするためにmoveDataを記述したのですがこの認識は間違っていますか？

xcode11.1

Answer 1

わざわざセルから取ってこなくてもデータはメンバにある方から取ればいいんじゃないでしょうかね。

let moveData = dataList[sourceIndexPath.row]

---

一応書いておきますが、

let moveData = tableView.cellForRow(at: sourceIndexPath)?.textLabel!.text

でUILabel.textはString?型なので、アンラップしないとコンパイルエラーが出ると思われます。
例えば

let moveData = tableView.cellForRow(at: sourceIndexPath)?.textLabel!.text!

のようにアンラップするだけでもコンパイルエラーは消えるのではないかと思います。

Comments

[aulthy787]

ご回答ありがとうございます！
データ取得を
let moveData = dataList[sourceIndexPath.row]
に書き換えたところエラーは解消されましたが、並べ替え後のデータが保存されませんでした。
この場合、この関数以外の関数に問題がありますか？

[takabosoft]

そもそもどうやってdataListを保存しているのでしょうか？コードがあれば載せてください。

[aulthy787]

こちらのコードでよかったですか？

var dataList = [DataStore]()

class DataStore: NSObject, NSSecureCoding {

static var supportsSecureCoding: Bool {
return true
}

// ToDoのタイトル
var cellTitle: String?
// ToDoを完了したかどうかを表すフラグ
var cellDone: Bool = false
// コンストラクタ
override init() {
}
// NSCodingプロトコルに宣言されているデシリアライズ処理。デコード処理とも呼ばれる
required init?(coder aDecoder: NSCoder) {
cellTitle = aDecoder.decodeObject(forKey: "cellTitle") as? String
cellDone = aDecoder.decodeBool(forKey: "cellDone")
}
// NSCodingプロトコルに宣言されているシリアライズ処理。エンコード処理とも呼ばれる
func encode(with aCoder: NSCoder) {
aCoder.encode(cellTitle, forKey: "cellTitle")
aCoder.encode(cellDone, forKey: "cellDone")
}

}

[takabosoft]

そちらではないですね、 dataListをストレージに保存している（それともUserDefaults？）と思うので、そのコードを載せてほしいです。

[aulthy787]

返信遅くなってしまってすみません。
こちらのコードをセルの保存に使用しています。
let userDefaults = UserDefaults.standard
// Data型にシリアライズする
do {
let data = try NSKeyedArchiver.archivedData(withRootObject: self.dataList, requiringSecureCoding: true)
userDefaults.set(data, forKey: "dataList")
userDefaults.synchronize()
} catch {
// エラー処理なし
}

[takabosoft]

TODOを追加や削除したときなどは、その保存コードが動作するようになっていませんか？
ソート後も同様にその処理が呼び出されるようにすれば良いです。

[aulthy787]

削除後の処理は以下のコードを使用しています

// ToDoリストから削除
dataList.remove(at: indexPath.row)
// セルを削除
tableView.deleteRows(at: [indexPath], with: UITableView.RowAnimation.fade)
// データ保存。Data型にシリアライズする
do {
let data: Data = try NSKeyedArchiver.archivedData(withRootObject: dataList, requiringSecureCoding: true)
// UserDefaultsに保存
let userDefaults = UserDefaults.standard
userDefaults.set(data, forKey: "dataList")
userDefaults.synchronize()
} catch {
// エラー処理なし
}

[takabosoft]

「データ保存。Data型にシリアライズする」の処理は関数にして、Todo削除時やソート後にその関数を呼び出すようにすればいいです。

[aulthy787]

以下であっていますか？
func dataSave() {
let data: Data = try NSKeyedArchiver.archivedData(withRootObject: dataList, requiringSecureCoding: true)
}
// UserDefaultsに保存
let userDefaults = UserDefaults.standard
userDefaults.set(data, forKey: "dataList")
userDefaults.synchronize()
} catch {
// エラー処理なし
}

[takabosoft]

それコンパイル通らなくないですか？

[aulthy787]

そうですか、まだまだ知識が足りていないのでもう少し深めてから再チャレンジしようと思います。
数日間に渡り返信していただきありがとうございました！