SwiftでAttempt to insert non-property list objectが出た時の対処について

Question

前提・実現したいこと

Swiftでアプリを作成していますが、途中で（完了ボタンを押した時）エラーが出てしまいます。

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

2018-05-06 21:47:14.344143+0900 timeTableApp[5738:240658] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: 'Attempt to insert non-property list object (
    "timeTableApp.Class"
) for key TimeTable'

該当のソースコード　（一部省略）

import UIKit

class Class {
    let className: String
    let roomName: String
    let cellNumber: Int
    
    init(className: String, roomName: String, cellNumber: Int) {
        
        self.className = className
        self.roomName = roomName
        self.cellNumber = cellNumber
    }
}

class Edit: UIViewController, UITextFieldDelegate {

    @IBOutlet weak var classTextField: UITextField!
    @IBOutlet weak var roomTextField: UITextField!
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        classTextField.delegate = self
        roomTextField.delegate = self
                
        let tapRecognizer = UITapGestureRecognizer(target: self, action: #selector(tapGesture))
        self.view.addGestureRecognizer(tapRecognizer)
    }
    
    @objc func tapGesture() {
        classTextField.resignFirstResponder()
        roomTextField.resignFirstResponder()
    }
    
    func textFieldShouldReturn(_ textField: UITextField) -> Bool {
        classTextField.resignFirstResponder()
        roomTextField.resignFirstResponder()
        
        return true
    }
    
    
//    完了ボタンを押した時
    @IBAction func tappedButton(_ sender: UIButton) {
        
        let classData = Class(className:classTextField.text!, roomName: roomTextField.text!, cellNumber: ViewController.cellNumber)
        
        ViewController.testArray.append(classData)
        UserDefaults.standard.set(ViewController.testArray, forKey: "TimeTable")
        UserDefaults.standard.synchronize()
        
        print(ViewController.testArray.count)
        
        dismiss(animated: true) {}
    }

}

import UIKit

class ViewController: UIViewController, UICollectionViewDelegate, UICollectionViewDataSource, UICollectionViewDelegateFlowLayout  {
    
    static var cellNumber: Int = 0
    @IBOutlet weak var collectionView: UICollectionView!
    
    static var testArray: [Class] = []
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        collectionView.delegate = self
        collectionView.dataSource = self
        
        collectionView.register(UINib(nibName: "CollectionViewCell", bundle: nil), forCellWithReuseIdentifier: "CollectionViewCell")
    }
    
    override func viewWillAppear(_ animated: Bool) {
        
        UserDefaults.standard.array(forKey: "TimeTable")
        if let array: [Class] = UserDefaults.standard.array(forKey: "TimeTable") as? [Class] {
            ViewController.testArray = array
            print(array)
        } else {
            print("Error")
        }
        
        collectionView.reloadData()
    }
    

    func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) {
        
        ViewController.cellNumber = indexPath.row
        
        if ViewController.cellNumber == indexPath.row {
            print("セル番号:(indexPath.row)")
        }
        
        performSegue(withIdentifier: "Modal1", sender: self)
    }
  
    
    func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
        let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "CollectionViewCell", for: indexPath) as! CollectionViewCell
        cell.backgroundColor = UIColor.groupTableViewBackground
        
        for data in ViewController.testArray {
        if indexPath.row == data.cellNumber {
            cell.classLabel.text = data.className
            cell.roomLabel.text = data.roomName
        } else {
            cell.classLabel.text = ""
            cell.roomLabel.text = ""
        }
        }
        return cell
    }

}

試したこと

完了ボタンを押してUserDefaultsで保存する時にエラーが出ていると予想しましたが、対処法がわかりません。
ViewController.testArrayがClass型だから保存できないのでしょうか？

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

Answer 1

ViewController.testArrayがClass型だから保存できないのでしょうか？

その通りです、UserDefaultsにはカスタムクラスはそのままでは保存できません、保存するのでしたらNSCodingプロトコルに準拠させないといけません。

先日も同じような質問（カスタムクラスの配列を保存する）に答えていますので、以下を参考にしてみてください。

参考質問: ランキング表示がうまくいきません。Swift,配列,Double型
参考サイト(古いですが): NSUserDefaultsに自作クラスを保持（swift）

回答追記

取得するところのコードが間違えていたので修正、あとクラス方の初期化のところもdecodeIntegerを使うように修正しました。
違いを比べて自分のプロジェクトに当てはめてみてください。

import UIKit

class ViewController: UIViewController {
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        let classData1 = Class(className: "a", roomName: "b", cellNumber: 1)
        let classData2 = Class(className: "c", roomName: "d", cellNumber: 2)
        let classData3 = Class(className: "e", roomName: "f", cellNumber: 3)
        let classArray = [classData1, classData2, classData3]
        
        // 保存
        UserDefaults.standard.set(NSKeyedArchiver.archivedData(withRootObject: classArray), forKey: "TimeTable")
        UserDefaults.standard.synchronize()
        
        // 取得
        if let data = UserDefaults.standard.object(forKey: "TimeTable") as? Data,
            let classArray = NSKeyedUnarchiver.unarchiveObject(with: data) as? [Class] {
            print(classArray)
        }
    }
}

class Class: NSObject, NSCoding {
    
    let className: String
    let roomName: String
    let cellNumber: Int
    
    init(className: String, roomName: String, cellNumber: Int) {
        
        self.className = className
        self.roomName = roomName
        self.cellNumber = cellNumber
    }
    
    func encode(with aCoder: NSCoder) {
        aCoder.encode(self.className, forKey: "name")
        aCoder.encode(self.roomName, forKey: "room")
        aCoder.encode(self.cellNumber, forKey: "number")
    }
    
    required init?(coder aDecoder: NSCoder) {
        self.className = aDecoder.decodeObject(forKey: "name") as! String
        self.roomName = aDecoder.decodeObject(forKey: "room") as! String
        self.cellNumber = aDecoder.decodeInteger(forKey: "number")
    }
}

Comments

[s1209]

ありがとうございます。

コードを書いてみたのですが、UserDefaultsで配列（ViewController.testArray）を保存できていないようです。。。
保存する時のコードが間違っているのでしょうか？

```
class Class: NSObject, NSCoding {

let className: String
let roomName: String
let cellNumber: Int

init(className: String, roomName: String, cellNumber: Int) {

self.className = className
self.roomName = roomName
self.cellNumber = cellNumber
}

func encode(with aCoder: NSCoder) {
aCoder.encode(self.className, forKey: "name")
aCoder.encode(self.roomName, forKey: "room")
aCoder.encode(self.cellNumber, forKey: "number")
}

required init?(coder aDecoder: NSCoder) {
self.className = aDecoder.decodeObject(forKey: "name") as! String
self.roomName = aDecoder.decodeObject(forKey: "room") as! String
self.cellNumber = aDecoder.decodeObject(forKey: "number") as! Int
}
}
```

```
class Edit: UIViewController, UITextFieldDelegate {

// 完了ボタンを押した時
@IBAction func tappedButton(_ sender: UIButton) {

let classData = Class(className:classTextField.text!, roomName: roomTextField.text!, cellNumber: ViewController.cellNumber)
ViewController.testArray.append(classData)

// 保存
UserDefaults.standard.set(NSKeyedArchiver.archivedData(withRootObject: ViewController.testArray), forKey: "TimeTable")
UserDefaults.standard.synchronize()
print(UserDefaults.standard.set(NSKeyedArchiver.archivedData(withRootObject: ViewController.testArray), forKey: "TimeTable"))

dismiss(animated: true) {}
}
}
```

```
class ViewController: UIViewController, UICollectionViewDelegate, UICollectionViewDataSource, UICollectionViewDelegateFlowLayout {

static var testArray: [Class] = []

override func viewWillAppear(_ animated: Bool) {

// 取得
let userDefault = UserDefaults.standard.array(forKey: "TimeTable")
if let array = userDefault {

let myClass = NSKeyedUnarchiver.unarchiveObject(with: (userDefault as! NSData) as Data) as! Class
print(array)

} else {
print("Error")
}

collectionView.reloadData()
}
}
```

[_Kentarou]

コードに間違えている部分があったので、回答の方に記載しました。

[s1209]

ありがとうございます。
取得コードを書き直してみました。

```
// 取得
if let data = UserDefaults.standard.object(forKey: "TimeTable") as? Data, let array = NSKeyedUnarchiver.unarchiveObject(with: data) as? [Class] {
print("取得:(array)")
}
```

[_Kentarou]

値が取得でき、解決したということでしょうか？

[s1209]

無事に取得できました！
本当にありがとうございました。

[s1209]

解決済み後にコメントしてしまい大変申し訳ないのですが、xcodeのシミュレーターを起動し、値の入力をした後にシミュレーターを終了、再度シミュレーターを起動すると入力した値が消えてしまうため、うまく保存ができていなかったようです。。。

取得部分のprintではarrayがちゃんと出力されるので、解決したと思い込んでしまいました。
再起動するとarrayの中身が初期化されるので保存ができていないということでしょうか？

私の確認不足でご迷惑おかけしてすみません。

[_Kentarou]

> 取得部分のprintではarrayがちゃんと出力される

でしたら、保存はできていると思いますね、空で上書き等していないでしょうか？

後は直接値が入っているか確認してみるのもいいと思います。
https://qiita.com/dondoko-susumu/items/85f5e1449df94be20fa9

[s1209]

返信が遅くなり申し訳ありません。
おそらく上書きはしていないと思います。

参考記事のやり方で値が入っているか確認してみようとしましたが、find ./ ...の後に`-bash: com.~.timeTableApp: No such file or directory`と出てしまいます。

[_Kentarou]

もっとシンプルなプロジェクトを作成して（今回私が書いた程度）、動くことを確認してから自分のプロジェクトに移してみてください。

そうすればどこが悪いかあたりがつけやすいと思いますので。

[s1209]

ありがとうございます。
上記の方法でやってみます。

[_Kentarou]

それでも分からなければまた聞いてください。

[s1209]

なぜNo such file or directoryが出るか分かりました。
この方法だと実機を接続しないとダメなんですね。。。
MacBookProのUSB-Cに対応したケーブルを持っていないので、この方法で確認できない状況です（近々購入します）

[_Kentarou]

実機でなくてもいけるはずですが、状況がわからないので、、、

実機を繋いでいるのでしたら古い記事ですが、以下の方法でアプリの中の情報にアクセスできます。
UserDefaultsも格納されているのでみてみてください。

https://qiita.com/snaka/items/f10994af66549a4c2c5a

[s1209]

てっきり実機がないとダメだと思ったのですが、古い記事だったんですね。

いろいろ試してみたのですが、シミュレーター再起動してもUserDefaults取得時のprint(array)の内容は保存されたままなので、セルに表示させるところ(cellForItemAt)が間違っていると推測しました。

func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "CollectionViewCell", for: indexPath) as! CollectionViewCell
cell.backgroundColor = UIColor.groupTableViewBackground

if let data = ViewController.testArray.first(where: { $0.cellNumber == indexPath.row }) {
cell.classLabel.text = data.className
cell.roomLabel.text = data.roomName
} else {
cell.classLabel.text = ""
cell.roomLabel.text = ""
}
return cell
}