CoreDataで重複して保存が実行されてしまう

Question

質問失礼します。

日付ごとのデータをCoreDataに保存したいです。

当日のデータがない場合は新規作成をする様に記載したのですが、
正しい日付が入っているデータだけでなく、
同時に中身が空のデータも保存されてしまいます。

新しい日付のデータが作成される時だけ一緒に空データが作成されるのですが、
insert,save文の使用方法が間違っているのでしょうか。

どなたかご教授いただけますと嬉しいです。
よろしくお願い致します。

class CountViewController: UIViewController {
    
    ///現在の日付
    var nowDate = Date()
    static var nowDateString = String()
    
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        //日付のフォーマットを変更する
        let formatter: DateFormatter = DateFormatter()
        formatter.dateFormat = "yyyyMMdd"
        ///日付をStringにする
        CountViewController.nowDateString = formatter.string(from: nowDate)

        //もし今日の日付でデータが存在しなかったら
        if CoreDataModel.selectOne(date: CountViewController.nowDateString) == [] {
            //今日のデータを作成して保存する
            CoreDataModel.insertData(date: CountViewController.nowDateString)
            CoreDataModel.save()
            
        }
}

class CoreDataModel {

    static var context = (UIApplication.shared.delegate as! AppDelegate).persistentContainer.viewContext

    //データを挿入する
    static func insertData(date: String) {
        //Data型を作成して値を反映させる
        let newData = Data(context: self.context)
        newData.date = date
　　　　　//インサートする
        NSEntityDescription.insertNewObject(forEntityName: "Data", into: context)
    }
    
    //データを保存する
    static func save() {
        //コミットする
        (UIApplication.shared.delegate as! AppDelegate).saveContext()
    }

    //データを一件取得
    static func selectOne(date: String) -> [Data] {
        
        var data = [Data]()
        
        let request = NSFetchRequest<NSFetchRequestResult>(entityName: "Data")
        //条件指定
        request.predicate = NSPredicate(format: "date == %@", date)
        
        do{
            data = try context.fetch(request) as! [Data]
            
            return data
            
        }catch{
            print("エラー")
        }
        return data
    }

    //データを全件取得する
    static func selectAll() -> [Data] {
        
        var data = [Data]()

        let request = NSFetchRequest<NSFetchRequestResult>(entityName: "Data")
        do{
            data = try context.fetch(request) as! [Data]
            
            return data
        }catch{
            print("エラー")
        }
        return data
    }
}

class AppDelegate: UIResponder, UIApplicationDelegate {

//CoreData関連部分のみ抜粋
    lazy var persistentContainer: NSPersistentContainer = {

        let container = NSPersistentContainer(name: "CoreDataTest")
        container.loadPersistentStores(completionHandler: { (storeDescription, error) in
            if let error = error as NSError? {

                fatalError("Unresolved error (error), (error.userInfo)")
            }
        })
        return container
    }()

    func saveContext () {
        let context = persistentContainer.viewContext
        if context.hasChanges {
            do {
                try context.save()
            } catch {
                let nserror = error as NSError
                fatalError("Unresolved error (nserror), (nserror.userInfo)")
            }
        }
    }
}

Comments

[fuzzball]

>>正しい日付が入っているデータだけでなく、同時に中身が空のデータも保存されてしまいます。
これは現在起こっている症状（不具合）だと思いますが、

>>新しい日付のデータが作成される時だけ一緒に空データが作成されるのですが
というのは何でしょうか？

[Ka_ya_]

fuzzballさん、ご確認いただきありがとうございます。

>>新しい日付のデータが作成される時だけ一緒に空データが作成されるのですが
この箇所に関しては、
CoreDataModel.selectOne(date: CountViewController.nowDateString)
にて１件取得するコードとif文は正常に機能しており、
その内部のinsertもしくはsaveに問題があるのではという事をお伝えしたくて記載しました。

説明が下手で申し訳ありません。
お手数ですが、ご確認よろしくお願い致します。

[fuzzball]

前半の「空のデータ」と後半の「空データ」は意味が違いますか？
後半の「空データ」は、前半の「正しい日付が入っているデータ」のことですか？

[Ka_ya_]

fuzzballさん、ご確認ありがとうございます。

記載がややこしく、大変申し訳ありません。
空データに関しては、どちらも同じ意味となっております。

質問内容を簡潔に記載し直しますと、日付のデータがない場合に
・dateに値が入っているData型
・dateに値が入っていないData型
が保存されてしまう状態なのですが、
・dateに値が入っているData型
だけを保存したいというのが実現したい事です。

たびたびお手数をおかけして、申し訳ありません。
よろしくお願い致します。

[fuzzball]

データが2つ保存されている、というのはどうやって確認しましたか？
確認方法と併せて、ログまたはスクショを見せて下さい。

[Ka_ya_]

fuzzballさん、ご確認ありがとうございます。

データを全件取得してテーブルビューに表示し、中身を確認しました。

日付の入っているデータが作成される前に１つ空レコードが作成されている状態です。
（スクリーンショットの20200730の上にある空レコードの日付がブランクなのは、
デフォルト値の設定を削除しただけで20200729の上にある空レコードと同様の現象です）

たびたびお手数をおかけしますが、ご確認よろしくお願い致します。

[fuzzball]

実際には「空データ」ではないですね。こういうのは正しく伝えた方が良いです。
で、そのテーブル表示がバグってる可能性はないですかね？
ちょっと以下のことを試してみて下さい。

「//もし今日の日付でデータが存在しなかったら」の次の行に、
print(CountViewController.nowDateString, CoreDataModel.selectOne(date: CountViewController.nowDateString).count)
を追加して、どのように表示されるか教えてもらえますか？（データが追加されるときと、追加されないときの両方）

今のところ、質問のコードにおかしいところは無いと思っています。
他のデータベースを触ってるところは無いのでしょうか？

[Ka_ya_]

fuzzballさん、ご確認ありがとうございます。

伝え方が至らずにお手数をおかけして、大変申し訳ありません。
printを試してみたところ、

・データがない場合（当日初回起動時）
20200731 0
・データがある場合（２回目以降）
20200731 1

上記の様に表示されており、
printのselectOneを全件取得するメソッドに変更してcountしたところ、
20200731 2
と表示されました。
（if文を使用せずに毎回データ生成するようにすると、
　countが2ずつ増えて行く為、2件ずつ生成されていると思われます）

データベースの保存操作をしているのは、質問に記載の箇所だけとなります。
AppDelegateに記載しているCoreData関連のコードと、
全件取得に使用しているメソッドを追記いたしました。

何度もご迷惑をおかけして申し訳ありません。
よろしくお願い致します。

[fuzzball]

解決方法は分かりました。
insertNewObject()は必要ないようです。
CoreDataのことをよく知らないので、なぜ不要なのかは分かりません。
（let newData = Data(context: self.context) の時点で追加されているということでしょうか）

知らないままに色々聞いてしまいましたが、（たぶん）解決したということでご勘弁を。
不要の理由などはご自身で調べてみて下さい。

[Ka_ya_]

fuzzballさん、ご回答ありがとうございます。

insertをコメントアウトしたところ、
無事に一件だけ保存する事が出来ました！

何度もお手数をお掛けしてしまい申し訳ありません。
原因の調べ方や質問の記載方法なども、とても勉強になりました。
解決していただき、本当にありがとうございます＾＾

Answer 1

コメントにてご回答いただけたので、解決済みとさせていただきます。
ありがとうございました。