Table view cellと遷移先のtextfieldを紐付けて保存したいです

Question

前提・実現したいこと

プログラミングを始めて２ヶ月の者です。
リストアプリを作成しております。（iPhoneのメモアプリのような仕組み）
tableviewcellをタップした際に、画面遷移させることはできましたが、
遷移先のtextfieldに文字を入力して、それを保存すると別のcellに格納されてしまいます。

tableviewcellと遷移先のtextfieldを紐付けて保存したいです。

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

textfieldに文字を入力して保存すると、違うcellに保存されてしまう

該当のソースコード

import UIKit
import RealmSwift

class ViewController: UIViewController, UITableViewDelegate, UITableViewDataSource {
    
    private var labelName: String?
    private var text1Name: String?
    private var text2Name: String?
    
    @IBOutlet weak var tableView: UITableView!
    
    var itemList: Results<Item>!
    

    override func viewDidLoad() {
        super.viewDidLoad()
        
        // Realmからデータを取得
        do{
        let realm = try Realm()
        itemList = realm.objects(Item.self)
        }catch{
        }
        
        title = "リスト"
        
        tableView.dataSource = self
        tableView.delegate = self
        tableView.tableFooterView = UIView(frame: .zero)
        
        
        // Do any additional setup after loading the view.
    }
    

    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        if (segue.identifier == "showSecondView") {
            let secondVC = segue.destination as! SecondViewController
            secondVC.getName = self.labelName!
            let thirdVC = segue.destination as! SecondViewController
            thirdVC.getText1 = self.text1Name!
            let fourthVC = segue.destination as! SecondViewController
            fourthVC.getText2 = self.text2Name!
            
        }
    }
    
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return itemList.count
    }

    // 各行に表示するセルを返す
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell: UITableViewCell = tableView.dequeueReusableCell(withIdentifier: "NameCell", for: indexPath)
        let item = itemList[indexPath.row]
        cell.textLabel?.text = item.title
        return cell
    }
    
    // セルタップ時のイベント
    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        let item = itemList[indexPath.row]
        self.labelName = item.title
        self.text1Name = item.name1
        self.text2Name = item.name2
        performSegue(withIdentifier: "showSecondView",sender: nil)
        
    }
    
    //追加ボタンで新規作成
    @IBAction func addButton(_ sender: Any) {
      var textField = UITextField()
           let alert = UIAlertController(title: "アイテムを追加", message: "", preferredStyle: .alert)
           let action = UIAlertAction(title: "リストに追加", style: .default) { (action) in
            let newItem = Item()
            // textFieldに入力したデータをnewItemのtitleに代入
            newItem.title = textField.text!
           // インスタンスをRealmに保存
           do{
               let realm = try Realm()
               try realm.write({ () -> Void in
                realm.add(newItem)
               })
           }catch{
           }
           //データをリロードする
           self.tableView.reloadData()
           }
           alert.addTextField { (alertTextField) in
               alertTextField.placeholder = "NEWアイテム"
               textField = alertTextField
           }
           alert.addAction(action)
           present(alert, animated: true, completion: nil)
       }
    
    //スワイプしたセルを削除
    func tableView(_ tableView: UITableView, commit editingStyle: UITableViewCell.EditingStyle, forRowAt indexPath: IndexPath) {
              if editingStyle == UITableViewCell.EditingStyle.delete {
               // Realm内のデータを削除
               do{
                   let realm = try Realm()
                   try realm.write {
                       realm.delete(self.itemList[indexPath.row])
                   }
                   tableView.deleteRows(at: [indexPath], with: UITableView.RowAnimation.fade)
               }catch{
               }
              }
          }

}

import UIKit
import RealmSwift

class SecondViewController: UIViewController, UITextFieldDelegate {
    
    var getName: String = ""
    var getText1: String = ""
    var getText2: String = ""
    
    var itemList: Results<Item>!
    
    @IBOutlet weak var labelName: UILabel!
    
    @IBOutlet weak var textField1: UITextField!
    @IBOutlet weak var textField2: UITextField!
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        // Labelへ格納
        self.labelName.text = getName
        //テキストへ格納
        self.textField1.text = getText1
        self.textField2.text = getText2
        
        
        textField1.delegate = self
        textField2.delegate = self

        // Do any additional setup after loading the view.
    }
    
    //完了ボタンでキーボードを閉じる
    func textFieldShouldReturn(_ textField: UITextField) -> Bool {
        self.view.endEditing(true)
        // Itemクラスのインスタンス作成
        let newItem = Item()
        // TextFieldの値を代入
        newItem.name1 = textField1.text!
        newItem.name2 = textField2.text!
        // インスタンスをRealmに保存
        do{
            let realm = try Realm()
            try realm.write({ () -> Void in
                realm.add(newItem)
            })
        }catch{
        }
        return true

    }
    
    //他画面タッチでキーボードを閉じる
    override func touchesBegan(_ touches: Set<UITouch>, with event: UIEvent?) {
        self.view.endEditing(true)
        // Itemクラスのインスタンス作成
        let newItem = Item()
        // TextFieldの値を代入
        newItem.name1 = textField1.text!
        newItem.name2 = textField2.text!
        // インスタンスをRealmに保存
        do{
            let realm = try Realm()
            try realm.write({ () -> Void in
                realm.add(newItem)
            })
        }catch{
        }
        return
    }
    
}

import Foundation
import RealmSwift

class Item: Object {
    // タイトル
     @objc dynamic var title: String? = ""
    // 内容
     @objc dynamic var name1 = ""
     @objc dynamic var name2 = ""
}

試したこと

userdefaultの保存など他の方法がないか沢山調べましたが、わかりませんでした、、、
todoリストの作り方は多々あったのですが、画面遷移してその内容を保存するものが見つかりませんでした。
また自分の知識不足で理解することができない内容が多く困っています。

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

ここにより詳細な情報を記載してください。

Comments

[freemann]

保存というのは、メモリ上ではなく記憶媒体に保存したいということでしょうか。
もしそうであれば、SecondViewControllerでは保存せず、TextFieldに入力だけさせて、ViewControllerに戻って値を反映させて、そこで保存するようにしてはどうでしょうか。
以下のサイトでは前のViewControllerに戻るときに値を戻す方法が書かれています。

https://www.oborodukiyo.info/Swift/v50/Swift-PassDataBetweenViewControllers

[bbbone]

返答、誠にありがとうございます。
「メモリ上ではなく記憶媒体に保存」について、私の知識不足で自分でもどちらなのか分かっていません。申し訳ございません。
可能であれば、TextFieldに入力完了した段階で保存したいのですが、添付サイトのような方法もあるのですね！こちらの方法も検討いたします、ありがとうございます！

Answer 1

基本的にはtyobigorouさんのご指摘通り、遷移先の画面では追加ではなく更新を行う必要があるのですが（書き換えを行っているうちに話も進んでしまったようですが）、コードを眺めていたらいくつか気になる点を発見しました。

1. Results<Item> 内に保持されたオブジェクトの順番は追加順とはならない。同様に任意のオブジェクトを削除してもその順番は保証されない。
2. realm.delete(_)でオブジェクトを削除する際、プロパティ（titleやname1など）が重複するオブジェクトがあった場合、どのオブジェクトが削除されるのか明確ではない
3. tyobigorouさんご指摘のとおり、遷移先でアップデートしなければいけない。
4. 次の画面への値渡しは、Itemクラスのプロパティではなく、Itemクラスのインスタンスをそのまま渡す。

1） についてですが、追加ボタンで追加したオブジェクトの並び順は順番通りになるという保証はありません。これは、いわゆるデータベースと呼ばれるシステムの一般的特性のようです。

削除についても同じで、途中のデータを抜いた場合、それ以外のオブジェクトの順番が保持されるという保証はありません。

一番最初に提示していただいたコードだと、たとえば削除しても順番が保持されているように思えますが、実はコード内（tableView(_:commit:forRowAt:()内)でtableView.reloadData()を書き忘れていらっしゃるようなので、たまたま順番が保持されたように見えているだけだと思います。

2）ですが、たとえば

aaa
aaa
bbb
aaa

という具合にデータを追加し、二番目のaaaを左スワイプで削除した例を考えてみます。この場合、二番目のaaaが削除されたようにも見えますが、実際にはどのaaaが削除されたのか明確ではありません。

また、1）で指摘したreloadData()が抜けていた場合、削除を行うと下記のような順番になることがあります。

aaa
aaa
bbb

これを防ぐためには、Itemクラスにプライマリーキーを追加し、削除したいオブジェクトをユニークな（重複のない）オブジェクトとして指定する必要があります。

3）については、遷移先でrealm.addを使っていますが、これだと単に新しいオブジェクトを追加するだけとなってしまいます。したがって、どのオブジェクトに追加するのか明確にするためにやはりプライマリーキーをオブジェクトに追加し、どのオブジェクトをadd（実際には .modifiedなどを併記してオブジェクトのアップデートであることを指定する）のか明確にする必要があります。

4）ですが、プロパティを個別に渡しても構わないのですが、今後拡張する際のことを考えると、個別に渡すよりクラスとして渡した方が見通しが良いので、そのように変更するといいかと思います。

---

以上のことから、必要な修正内容は次のとおりとなります。

まず、オブジェクトモデルを次のように変更する必要があります。

import Foundation
import RealmSwift

class Item: Object {
    // 中略
    // MARK: id としてオブジェクトが作成された日付を使う
    @objc dynamic var id = Date().description
    
    // MARK: id （プロパティ名が "id"）をプライマリーキーとして使う
    override static func primaryKey() -> String? {
        return "id"
    }
}

変更点についてはMARK:と書かれた行をご参照ください。

この例では、プライマリーキーとしてオブジェクトが作成された瞬間の日時を用いることにしました。日時を使うことによって、今後オブジェクトを取得する際、idをキーとしてソートすることにより作成日付順で自動ソートさせることも可能となります。

ただし、Realmはオブジェクトモデルを変更した場合、そのまま実行すると実行時エラーとなりクラッシュするため、都度後述のマイグレーション処理が必要となります。

次にViewController.swiftですが、下記のように変更します。

import UIKit
import RealmSwift

class ViewController: UIViewController, UITableViewDelegate, UITableViewDataSource {
    // MARK: 遷移先にデータを付け足したい行のクラスを渡すためのプロパティ
    private var itemName: Item?
    
    @IBOutlet weak var tableView: UITableView!
    
    var itemList: Results<Item>!
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        // Realmからデータを取得
        do{
            let realm = try Realm()
            // MARK: id をキーとしてソートする
            itemList = realm.objects(Item.self).sorted(byKeyPath: "id")
        }catch{
        }
        // 中略
    }
    
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        if (segue.identifier == "showSecondView") {
            let secondVC = segue.destination as! SecondViewController
            // MARK: 該当する行の item だけ渡す
            secondVC.item = self.itemName
            // MARK: 以下の行は不要
            // ちなみに、thridVC, fourthVC ともに実際は SecondViewController なので毎回ダウンキャストする必要はない
//            secondVC.getName = self.labelName!
//            let thirdVC = segue.destination as! SecondViewController
//            thirdVC.getText1 = self.text1Name!
//            let fourthVC = segue.destination as! SecondViewController
//            fourthVC.getText2 = self.text2Name!
        }
    }
    
    // 中略
    
    // セルタップ時のイベント
    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        let item = itemList[indexPath.row]
        // MARK: 遷移先の画面に渡すクラス
        self.itemName = item
        // MARK: クラスごと次の画面に渡すので、下記の行は不要
//        self.labelName = item.title
//        self.text1Name = item.name1
//        self.text2Name = item.name2
        performSegue(withIdentifier: "showSecondView",sender: nil)
    }
    
    // 中略

    //スワイプしたセルを削除
    func tableView(_ tableView: UITableView, commit editingStyle: UITableViewCell.EditingStyle, forRowAt indexPath: IndexPath) {
        if editingStyle == UITableViewCell.EditingStyle.delete {
            // Realm内のデータを削除
            do{
                let realm = try Realm()
                try realm.write {
                    realm.delete(self.itemList[indexPath.row])
                    // MARK:
                    itemList = realm.objects(Item.self).sorted(byKeyPath: "id")
                }
                
                tableView.deleteRows(at: [indexPath], with: UITableView.RowAnimation.fade)
                
                // MARK: おそらく記載忘れ。削除したら tableView を更新する
                tableView.reloadData()
            }catch{
            }
        }
    }
}

なるべく変更点が最小となるように、ご提示していただいたコードのスタイルに沿うように変更してあります。なので、一部冗長とも思われる書き方がありますが、とりあえずはこのような感じでご利用ください。

変更点や注意点は同じくMARK:の部分に書きました。

たとえば、idがプライマリーキーであった場合、それをキーとしてソートする場合には

itemList = realm.objects(Item.self).sorted(byKeyPath: "id")

のような感じで記述します。
詳しくはRealmに関する各種資料をご参照ください。

次にSecondViewController.swiftですが、次のような感じとなります。

import UIKit
import RealmSwift

class SecondViewController: UIViewController, UITextFieldDelegate {
    // MARK: Item クラスの情報を渡してくるので下記3行は不要
//    var getName: String = ""
//    var getText1: String = ""
//    var getText2: String = ""
    // MARK:　Item クラスの item を引き取る
    var item: Item!
    
    // MARK: 未使用なのでコメントアウト
//    var itemList: Results<Item>!
    
    @IBOutlet weak var labelName: UILabel!
    
    @IBOutlet weak var textField1: UITextField!
    @IBOutlet weak var textField2: UITextField!
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        // Labelへ格納
        // MARK:
//        self.labelName.text = getName
        labelName.text = item.title
        //テキストへ格納
        // MARK:
//        self.textField1.text = getText1
//        self.textField2.text = getText2
        textField1.text = item.name1
        textField2.text = item.name2
        
        textField1.delegate = self
        textField2.delegate = self
        
        // Do any additional setup after loading the view.
    }
    
    //完了ボタンでキーボードを閉じる
    func textFieldShouldReturn(_ textField: UITextField) -> Bool {
        self.view.endEditing(true)
        // MARK: 不要なので削除
//        // Itemクラスのインスタンス作成
//        let newItem = Item()
//        // TextFieldの値を代入
//        newItem.name1 = textField1.text!
//        newItem.name2 = textField2.text!
//
        // インスタンスをRealmに保存
        do{
            let realm = try Realm()
            
            try realm.write({ () -> Void in
                // MARK: realm に関連づいたクラスのプロパティは write ブロック内部でのみ更新可能
                item.name1 = textField1.text!
                item.name2 = textField2.text!
                // 更新処理を行う
                // realm.add(newItem)
                realm.add(item, update: .modified)
            })
        }catch{
        }
        return true
    }
    
    //他画面タッチでキーボードを閉じる
    override func touchesBegan(_ touches: Set<UITouch>, with event: UIEvent?) {
        // ここの処理はtextFieldShouldReturn(_:)と同じなので関数化して共通化したほうがいいかも
        self.view.endEditing(true)
        // MARK: 削除
//        // Itemクラスのインスタンス作成
//        let newItem = Item()
//        // TextFieldの値を代入
//        newItem.name1 = textField1.text!
//        newItem.name2 = textField2.text!
        
        // インスタンスをRealmに保存
        do{
            let realm = try Realm()
            
            try realm.write({ () -> Void in
                // MARK: realm に関連づいたクラスのプロパティ（マネージドオブジェクト）は write ブロック内部でのみ更新可能
                item.name1 = textField1.text!
                item.name2 = textField2.text!
                // 更新処理を行う
                // realm.add(newItem)
                realm.add(item, update: .modified)
            })
        }catch{
        }
    }
}

こちらもやはりMARK:の部分を中心にご参照ください。
重要なのは、値の更新部分になるかと思います。

            try realm.write({ () -> Void in
                // MARK: realm に関連づいたクラスのプロパティは write ブロック内部でのみ更新可能
                item.name1 = textField1.text!
                item.name2 = textField2.text!
                // 更新処理を行う
                // realm.add(newItem)
                realm.add(item, update: .modified)
            })
        }catch{
        }

ここに示されているとおり、一度Realmに保存したオブジェクトや関連するオブジェクトはマネージドオブジェクトとなるため、Realmのwriteブロック内部でないとプロパティの変更ができません。
一方、オブジェクトにはプライマリーキーを追加したため、add(_:update:)メソッドを使って該当するオブジェクトのみを更新することが可能となりました。

最後に、AppDelegate.swiftです。
ここで前述のマイグレーション処理を行います。

    func application(_ application: UIApplication, didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?) -> Bool {
        // Override point for customization after application launch.
        
        // MARK: マイグレーション処理
        // Realm.io の記述ほぼそのまま流用
        let config = Realm.Configuration(
            // 新しいスキーマバージョンを設定します。以前のバージョンより大きくなければなりません。
            // （スキーマバージョンを設定したことがなければ、最初は0が設定されています）
            schemaVersion: 2,
            
            // マイグレーション処理を記述します。古いスキーマバージョンのRealmを開こうとすると
            // 自動的にマイグレーションが実行されます。
            migrationBlock: { migration, oldSchemaVersion in
                // 最初のマイグレーションの場合、`oldSchemaVersion`は0です
                if oldSchemaVersion < 2 {
                    // MARK: https://github.com/realm/realm-cocoa/issues/5132
                    migration.enumerateObjects(ofType: Item.className()) { oldObject, newObject in
                        newObject!["id"] = Date().description
                    }
                }
        })
        // デフォルトRealmに新しい設定を適用します
        Realm.Configuration.defaultConfiguration = config
        
        // Realmファイルを開こうとしたときスキーマバージョンが異なれば、
        // 自動的にマイグレーションが実行されます
        _ = try! Realm()
        
        return true
    }

必要な部分だけ抜き出したので、そっくりそのまま入れ替えて使ってもらえますでしょうか。

ほとんど決まり切った処理なのですが、もしかしたらここが一番厄介かもしれません。
もし、全て書き換えてうまくいかない場合には（実行時エラーが解決しない場合）、新たにプロジェクトを作成した方が早いかもしれません。その場合にはマイグレーション処理は必要ではなく、AppDelegate.swiftの更新も必要ありません。

Comments

[bbbone]

コードの修正だけでなく、それに付随する解説までして頂き誠にありがとうございます。
コードを書き直し実行したところ、自分の思い描いていた処理ができました！
プログラミング勉強のモチベーションが下がっていたのですが、これでまた頑張れそうです！
本当にありがとうございました。

Answer 2

SecondViewControllerのtextFieldShouldReturnの中で入力した情報を反映させるんでしょうが、そこでaddになっています、特定データに対する編集?更新処理にしてください。

で、戻ってきてtableViewをリロードしてください。

Comments

[bbbone]

```SecondViewController
//完了ボタンでキーボードを閉じる
func textFieldShouldReturn(_ textField: UITextField) -> Bool {
self.view.endEditing(true)
// Itemクラスのインスタンス作成
let newItem = Item()
// TextFieldの値を代入
newItem.name1 = textField1.text!
newItem.name2 = textField2.text!
// インスタンスをRealmに上書き
do{
let realm = try Realm()
try realm.write {
newItem.name1 = textField1.text!
newItem.name2 = textField2.text!
}
}catch{
}
return true
}
```
上記のような更新処理のコードを書いたのですが、保存できません。知識不足で申し訳ないのですが、ご教示のほどよろしくお願いいたします。

[退会済みユーザー]

編集対象のオブジェクトを取得して、オブジェクトのプロパティの変更をしたらいいのではないでしょうか？
オブジェクトの削除の部分で特定オブジェクトを取得することはできてるので、その方法を使えばいいのでは？

[bbbone]

ViewController上では[indexPath.row]を使用して、対象のオブジェクトを取得できるのですが、SecondViewController上で対象のオブジェクトを呼び出す方法が分かっておりません。度々申し訳ございませんが、教えてください、、、

[退会済みユーザー]

画面遷移時にオブジェクトを特定できる情報を遷移先に渡しては？
私適当なので、
TsukubaDepotさんのほうがためになるのでそっちで聞いてみてください。