UserDefaultsの永続化ができない

Question

前提・実現したいこと

ボールゲームを作成中です
ゲームが終了した後、ポイントが表示され、そのポイントをショップ画面で見れるように、今あるポイントにゲーム終了後にもらえるポイントを足すプログラムを作っているます。
UserDefaultsにポイントを保存させたのですが...
（Printでユーザーデフォルトの中身の値とlabelに表示させるようの変数の値を表示させています）

ポイントをもらえる画面からショップに行った時
UserDefaults 今のポイント
Label用Point 今のポイント

ショップ画面を閉じてもう一度ショップ画面を開くと
UserDefaults 0
Label用Point 今のポイント

もう一度ショップを開くと
UserDefaults 0
Label用Point 0

ポイントをもらえる画面からホーム画面に移動してからショップ画面を開いた時は
UserDefaults 0
Label用Point 0

になってしまいます...

UserDefaultsを永続化させる方法、考え方を教えてください

該当のソースコード

import UIKit

class ViewControllerShop: UIViewController {
    @IBOutlet weak var PointLabel: UILabel!
    
    var userDefaults = UserDefaults.standard
    var getPoint2 = 0
    var getPointUD = 0
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        saveData()
        PointLabel.text = String(getPointUD) + " PT"
        print("ゲットポイント" + String(getPointUD))
        print("ユーザーデフォルト" + String(userDefaults.integer(forKey: "DataGetPoint")))
}
    
    func saveData(){
        getPointUD = userDefaults.integer(forKey: "DataGetPoint")
        userDefaults.set(getPoint2, forKey: "DataGetPoint")
        userDefaults.synchronize()
        getPointUD = getPointUD + userDefaults.integer(forKey: "DataGetPoint")
    }
    
    
}

試したこと

userDefaults.synchronize()を入れるとUserDefaultsの永続化が可能だとネットで知ったので入れてみた。

データを保存し、永続化するメソッド。
データを保存した際に登場する。
よく忘れてしまうので注意が必要。

一旦UserDefaultsの値を保存するための変数getPointUDを用意した

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

MacBookPro2015

MacOSCatalina
10.15.2

XcodeVersion 11.2.1

Answer 1

viewDidLoad()内でやるべきなのはsaveDataではなくloadDataなのでは？？

Comments

[Amuy]

ロードというのは、UserDefaultの値を取得する「userDefaults.integer(forKey: "DataGetPoint")」ということでしょうか？

[takabosoft]

ですね。
UserDefaultはストレージデータと言っても良いと思いますので、
画面に入ったらストレージからメモリへポイントをロード、
なにかポイントに更新があったら、メモリからストレージへポイントをセーブ
というようにプログラムを単純化したほうがいいです。

[Amuy]

なるほど！やってみます！

[Amuy]

解決することができました！takabosoftさん、アドバイスありがとうございました！

Answer 2

ショップ画面を閉じてもう一度ショップ画面を開くと

UserDefaults 0

そういう処理になっています。

    var getPoint2 = 0 //画面破棄→メモリ上に再展開でゼロリセット
    override func viewDidLoad() { //メモリ上に展開されたときに呼び出される（画面破棄→再表示で発生）
        super.viewDidLoad()

        saveData()
        ...
    }
    func saveData(){
        ...
        userDefaults.set(getPoint2, forKey: "DataGetPoint") //ゼロで上書き
        ...
    }

初期化なら、"DataGetPoint"が登録済みか確認してから。

【iOS】UserDefaultsをSwiftらしく使う を参照
5 存在チェック
→初期化（登録）を実行していない場合、integer(forKey)はゼロが返ります。
6 Synchronize
→こちらはおまけ

Comments

[Amuy]

回答ありがとうございます。
getPoint2には、値渡しの値が入るので消すことができません...

それと、「初期化なら、"DataGetPoint"が登録済みか確認してから。」の意味を教えてください。
本当に何もわからなくてすいません...

[dsuzuki]

回答に追記しました。

[Amuy]

チェックしてみました「if UserDefaults.standard.object(forKey: "DataGetPoint") != nil {
print("nil")
}」
起動するとnilと表示されました。
getPoint2が起動された時から0だとuserdefaultsも0になってしまうとのことだったので、空の変数に「var getPoint2:Int!」に切り替えましたが、それでも2回目に開くとuserdefaultsは0になってしまいます.

[dsuzuki]

viewDidLoad内で、saveData()を実行しているのが問題です。（takabosoftさんの指摘と同じ）
何の為に、viewDidLoad内（画面表示時）で値を保存しているのでしょうか。
UserDefaultsへ保存するのは、ポイントが増えたときか、減ったとき。
ショップ画面を表示するだけで、ポイントは変わるのでしょうか。
（そういう仕様なら必要な処理ですが、その場合、getPoint2は保存すべき値になっているはずです。）

[Amuy]

getPointUD = getPoint2 + userDefaults.integer(forKey: "DataGetPoint")
PointLabel.text = String(getPointUD) + " PT"
プログラムを簡略化してみましたが、一回ショップ画面を閉じてもう一度開くと、（またはホーム画面からゲームをせずにショップ画面を開くと）
getPointUD = getPoint2 + userDefaults.integer(forKey: "DataGetPoint")
のところに「Thread 1: Fatal error: Unexpectedly found nil while implicitly unwrapping an Optional value」というエラーが出てしまいました...ゲームをプレイした後にボタンを押して画面繊維した時のショップ画面では普通に開くことができます。

[Amuy]

長文失礼しました

[dsuzuki]

saveData()の実行タイミングは変更しましたか？（変更していないなら、なぜviewDidLoadで保存が必要？）
getPoint2は、var getPoint2: Int! のまま？
（getPoint2の参照時にエラーが発生しているとは思いますが）

エラーが発生したとき、行だけでなく、どこが問題なのか示されているはずです。
そこから、どの変数（もしくはメソッド）の何が問題なのか理解することが必要です。
今回は、nilをアンラップしようとしているのが問題です。

[Amuy]

savedata()はコメントアウトしました。getPoint2はvar getPoint2: Int!ままです...それと、
if UserDefaults.standard.object(forKey: "DataGetPoint") != nil {
print("nil")
getPoint2 = 0
}
のプログラムもコメントアウトしました

[dsuzuki]

> getPoint2はvar getPoint2: Int!ままです
この場合、getPoint2の初期値は、nil です。
何も値を設定しないまま、参照すれば問題のエラーが発生します。

元々は、userdefaultsへゼロで保存していたのが問題であり、getPoint2 の初期値ゼロが問題ではありません。
saveData()をコメントアウトしたのなら、
var getPoint2 = 0
に戻すことで解決しませんか。

[Amuy]

なるほど！そういうことだったんですね！やってみます

[Amuy]

今回はホーム画面からショップを開いても落ちませんでしたが、
ゲットポイント もらったポイント
ユーザーデフォルト 0
になってしまいました...UserDefaultsに保存してないからですかね...

[Amuy]

getPointUD = getPoint2 + userDefaults.integer(forKey: "DataGetPoint")
UserDefaults.standard.set(getPointUD, forKey: "DataGetPoint")
PointLabel.text = String(getPointUD) + " PT"
このプログラムにしたらうまくいきました！

[Amuy]

長くお付き合い&アドバイスありがとうございました！