回答編集履歴

誤字訂正

2021/05/09 10:39

投稿

hoshi-takanori

スコア7895

test CHANGED Viewed

@@ -308,7 +308,7 @@
         }
-        print("loadStars: (stars)")
+        print("loadStars: (stars)") // デバッグ用、確認したら消す

誤字訂正

2021/05/09 10:39

投稿

hoshi-takanori

スコア7895

test CHANGED Viewed

@@ -204,7 +204,7 @@
-3 つも必要でしょうか？ 保存したいのはお気に入りかどうかなので、animal.star を保存すれば良くて、okiniGray や okiniYellow を保存するかは animal.star の値が決まれば自動的に決まります。
+3 つも必要でしょうか？ 保存したいのはお気に入りかどうかなので animal.star を保存すれば充分で、okiniGray や okiniYellow の値は animal.star の値が決まれば自動的に決まります。
@@ -212,7 +212,7 @@
-また、これだと別の問題があって、どの動物をお気に入りにしたかが区別つかないですよね。
+また、これだと別の問題があって、どの動物をお気に入りにしたかの区別がつかないですよね。

追記

2021/05/09 10:35

投稿

hoshi-takanori

スコア7895

test CHANGED Viewed

@@ -185,3 +185,227 @@
 とりあえずこれでお気に入りの処理はご希望通りになると思います。UserDefaults への保存は、ここまでの修正でちゃんと動くことを確認できてから追記しますので、コメント欄で結果をお知らせください。
+---
+UserDefaults への保存ですが、まずどんな値をどのような形で保存するかを考える必要があります。
+> UserDefaults.standard.set(okiniGray.isHidden, forKey: "saveGray")
+> UserDefaults.standard.set(okiniYellow.isHidden, forKey: "saveYellow")
+> UserDefaults.standard.set(animal.star, forKey: "saveCellStar")
+3 つも必要でしょうか？ 保存したいのはお気に入りかどうかなので、animal.star を保存すれば良くて、okiniGray や okiniYellow を保存するかは animal.star の値が決まれば自動的に決まります。
+> UserDefaults.standard.set(animal.star, forKey: "saveCellStar")
+また、これだと別の問題があって、どの動物をお気に入りにしたかが区別つかないですよね。
+というわけで、保存したいのは「各動物をお気に入りにしたかどうか」になります。もっと言うと、「犬をお気に入りにしたか」「猫をお気に入りにしたか」「猿をお気に入りにしたか」「鳥をお気に入りにしたか」…、をそれぞれ保存する必要があるということです。
+方法はいくつか考えられますが、直感的なのは辞書を使うことではないでしょうか。
+```swift
+let stars: [String: Bool] = [
+    "犬": true,
+    "猫": false,
+    "猿": true,
+    "鳥": false,
+]
+```
+もう一つの方法として、お気に入りの動物の配列を使うこともできます。この場合、配列に含まれないものはお気に入りではない、ということになります。
+```swift
+let stars: [String] = ["犬", "猿"]
+```
+配列の方が簡単なので、配列を使うことにしましょう。まず保存のメソッドを ListViewController に書きます。一時的にお気に入りの動物の名前だけを集めた配列を作って、それを保存します。
+(デバッグビルドの場合に synchronize してますが、これは Xcode からプログラムを停止させると UserDefaults が保存されないことがあるのを防ぐためです。)
+```swift
+    func saveStars() {
+        var stars = [String]()
+        for animal in animals {
+            if animal.star {
+                stars.append(animal.nameJP)
+            }
+        }
+        print("saveStars: (stars)") // デバッグ用、確認したら消す
+        UserDefaults.standard.set(stars, forKey: "stars")
+        #if DEBUG
+            UserDefaults.standard.synchronize()
+        #endif
+    }
+```
+次に読み込みです。これも ListViewController に書きます。お気に入りの配列を読み込んで、それに含まれる動物の star を true にします。(含まれない動物の star は初期値 false なので、そのままで良い。)
+```swift
+    func loadStars() {
+        guard let stars = UserDefaults.standard.array(forKey: "stars") as? [String] else {
+            return
+        }
+        print("loadStars: (stars)")
+        for animal in animals {
+            if stars.contains(animal.nameJP) {
+                animal.star = true
+            }
+        }
+    }
+```
+これらを呼び出す方法ですが、loadStars は loadData で animals の配列を作り終わった後に一回だけ呼び出せば良いでしょう。
+問題は saveStars ですが、お気に入りボタンを押すたびに保存したいので、DetailViewController から ListViewController のメソッドを呼び出すことになります。このためには DetailViewController が ListViewController の参照を持つ必要がありますが、循環参照を防ぐために weak (弱参照) にする必要があります。(本来はプロトコルを定義して [delegate](https://qiita.com/chanNaru/items/326bd50a78cf34371169) にしたいところですが、今はまだいいでしょう。)
+```diff
+ class DetailViewController: UIViewController {
++    weak var listViewController: ListViewController?
+     // 略
+```
+次に、画面遷移の際に listViewController を設定します。
+```diff
+     override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
+         if segue.identifier == "showDetailSegue" {
+             if let indexPath = myTableView.indexPathForSelectedRow {
+                 let destination = segue.destination as! DetailViewController
++                destination.listViewController = self
+                 destination.animal = dupList[indexPath.row]
+             }
+         }
+     }
+```
+最後に、お気に入りボタンを押すたびに saveStars を呼び出せば完成です。
+```diff
+     @IBAction func tapOkiniGray(_ sender: Any) {
+         // 略
+         animal.star = true
++        listViewController?.saveStars()
+     }
+     @IBAction func tapOkiniYellow(_ sender: Any) {
+         // 略
+         animal.star = false
++        listViewController?.saveStars()
+     }
+```