[swift 5] 新しい文章を追加した時にcellが２つになるのを避けたい。

Question

#質問したいこと
今メモ帳アプリを作成中です。
うまくいい表すことができないのですが、ホーム画面からcellをタップして詳細画面にいき詳細画面で文章を編集して保存するとcellが新たに増えてしまっています。（写真を下の方に掲示しています。）
#コード

//ホーム画面
import UIKit

class ViewController: UIViewController,UITableViewDataSource,UITableViewDelegate {
    
    @IBOutlet weak var memoTableView: UITableView!
    
    var memoArray = [String]()

    let ud = UserDefaults.standard

    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return memoArray.count
    }


    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "memoCell", for: indexPath)
        cell.textLabel?.text = memoArray[indexPath.row]
        return cell
    }

    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        self.performSegue(withIdentifier: "toDetail", sender: nil)
        //押したら押した状態を解除
        tableView.deselectRow(at: indexPath, animated: true)
    }


    override func viewDidLoad() {
        super.viewDidLoad()

        memoTableView.delegate = self
        memoTableView.dataSource = self
    }
    override func viewWillAppear(_ animated: Bool) {
        loadMemo()
    }
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        //destinationのクラッシュ防ぐ
        if segue.identifier == "toDetail"{
            //detailViewControllerを取得
            //as! DetailViewControllerでダウンキャストしている
            let detailViewController = segue.destination as! DetailViewController
            //遷移前に選ばれているCellが取得できる
            let selectedIndexPath = memoTableView.indexPathForSelectedRow!
            detailViewController.selectedMemo = memoArray[selectedIndexPath.row]
            detailViewController.selectedRow = selectedIndexPath.row
        }
    }
    func loadMemo(){
        if ud.array(forKey: "memoArray") != nil{
            //取得 またas!でアンラップしているのでnilじゃない時のみ
            memoArray = ud.array(forKey: "memoArray") as![String]
            //reloadしてくれる
            memoTableView.reloadData()
        }
    }
    func tableView(_ tableView: UITableView, commit editingStyle: UITableViewCell.EditingStyle, forRowAt indexPath: IndexPath) {

        if editingStyle == .delete {

            //resultArray内のindexPathのrow番目をremove（消去）する
            memoArray.remove(at: indexPath.row)

            //再びアプリ内に消去した配列を保存
            ud.set(memoArray, forKey: "memoArray")

            //tableViewを更新
            tableView.reloadData()
        }
    }
}

//詳細画面

import UIKit

class DetailViewController: UIViewController {

    @IBOutlet weak var memoTextView: UITextView!

    var selectedRow:Int!
    var selectedMemo : String!

    override func viewDidLoad() {
        super.viewDidLoad()
        memoTextView.text = selectedMemo
        let kbToolBar = UIToolbar(frame: CGRect(x: 0, y: 0, width: 320, height: 40))
        kbToolBar.barStyle = UIBarStyle.default  // スタイルを設定
        kbToolBar.sizeToFit()  // 画面幅に合わせてサイズを変更
        // スペーサー
        let spacer = UIBarButtonItem(barButtonSystemItem: UIBarButtonItem.SystemItem.flexibleSpace, target: self, action: nil)
        // 閉じるボタン
        let commitButton = UIBarButtonItem(barButtonSystemItem: UIBarButtonItem.SystemItem.done, target: self, action: #selector(self.commitButtonTapped))
        kbToolBar.items = [spacer, commitButton]
        memoTextView.inputAccessoryView = kbToolBar
    }
    
    @objc func commitButtonTapped() {
        self.view.endEditing(true)
    }
    
    //画面遷移する時にタップするボタン
    @IBAction func save(_ sender: Any) {

        let inputText = memoTextView.text
        let ud = UserDefaults.standard
        if ud.array(forKey: "memoArray") != nil{
            //saveMemoArrayに取得
            var saveMemoArray = ud.array(forKey: "memoArray") as! [String]
                //テキストに何か書かれているか？
            if inputText != ""{
                //配列に追加
                saveMemoArray.append(inputText!)
                ud.set(saveMemoArray, forKey: "memoArray")
            }else{
                showAlert(title: "何も入力されていません")

            }

        }else{
            //最初、何も書かれていない場合
            var newMemoArray = [String]()
            //nilを強制アンラップはエラーが出るから
            if inputText != ""{
                //inputtextはoptional型だから強制アンラップ
                newMemoArray.append(inputText!)
                ud.set(newMemoArray, forKey: "memoArray")
            }else{
                showAlert(title: "何も入力されていません")
            }
        }
        showAlert(title: "保存完了")
        ud.synchronize()
    }

    func showAlert(title:String){
        let alert = UIAlertController(title: title, message: nil, preferredStyle: .alert)

        alert.addAction(UIAlertAction(title: "OK", style: .default, handler: nil))

        alert.addAction(UIAlertAction(title: "キャンセル", style: .cancel, handler: nil))

        self.present(alert, animated: true, completion:nil)
    }
    //削除ボタン
    @IBAction func deleteMemo(_ sender: Any) {
        let ud = UserDefaults.standard
        if ud.array(forKey: "memoArray") != nil{
            var saveMemoArray = ud.array(forKey: "memoArray") as![String]
            saveMemoArray.remove(at: selectedRow)
            ud.set(saveMemoArray, forKey: "memoArray" )
            ud.synchronize()
            //画面遷移
            self.navigationController?.popViewController(animated: true)
        }
    }
    
    //シェアボタン
    @IBAction func showActivityView(_ sender: UIBarButtonItem) {
        let activitycontroller = UIActivityViewController(activityItems: [memoTextView], applicationActivities: nil)
        
        self.present(activitycontroller, animated: true, completion: nil)
    }

}

#画像

語彙力なくて本当にすみません。
少し急いでいます。よろしくお願いします。

Answer 1

//配列に追加
saveMemoArray.append(inputText!)

配列に追加しているから(updateしなければいけないのに、insertしている)、Cellが増えるのは当然です。

前の画面から持ってきている selectedRow を使って、対象の要素にアクセス、それを書き換えないといけません。(検証していないけど大体こんな感じです)

saveMemoArray[selectedRow] = inputText!
ud.set(saveMemoArray, forKey: "memoArray")

Comments

[Kaguya_4869]

迅速な対応ありがとうございます！！
とても助かりました！ありがとうございます！

[退会済みユーザー]

Rino-T_Cさん、Kaguya_4869さんについてTwitterで呟いとる。
(ごめんなさい、全く関係のないことなので、無視してください。むしろ無視していただいた方が助かります。)
なんか、ごめんなさい。衝撃を受けてたみたいですね・・・(これも無視して結構です。)