[swift 5]写真を保存すると他の内容に違う写真が反映されてしまっていることで困っている。

Question

#質問したいこと
語彙力がなくてどう言えばいいのかわからないのですが、写真を保存してからホーム画面へ戻ると他の内容に違う写真が反映されてしまっています。（下記に写真が掲示してあります。）
#コード

//追加画面
import UIKit

class AddViewController: UIViewController, UIImagePickerControllerDelegate, UINavigationControllerDelegate {

    @IBOutlet weak var memoTextView: UITextView!
    @IBOutlet weak var imageView: UIImageView!
    @IBOutlet weak var imageButton: UIButton!
    
    let saveData: UserDefaults = UserDefaults.standard
    
    //カメラボタンがタップされた時の処理
    @IBAction func launchCamera(_ sender: UIButton) {
        let camera = UIImagePickerController.SourceType.camera
        if UIImagePickerController.isSourceTypeAvailable(camera) {
            let picker = UIImagePickerController()
            picker.sourceType = camera
            picker.delegate = self
            self.present(picker, animated: true)
        }
        imageView.isHidden = false
        imageButton.isHidden = true
    }
    
    //ユーザーが撮影し終わった時の処理
    func imagePickerController(_ picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [UIImagePickerController.InfoKey : Any]) {
        let image = info[UIImagePickerController.InfoKey.originalImage] as! UIImage
        self.imageView.image = image
        UIImageWriteToSavedPhotosAlbum(image, nil, nil, nil)
        self.dismiss(animated: true)
        
        UserDefaults.standard.set(image.jpegData(compressionQuality: 0.8), forKey: "MemoImage")
        saveData.synchronize()
    }
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        imageView.isHidden = true
    }

    @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{
            
            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]()
            
            if inputText != ""{
                
                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)
    }

}

import UIKit


class DetailViewController: UIViewController, UIImagePickerControllerDelegate, UINavigationControllerDelegate {

    @IBOutlet weak var memoTextView: UITextView!
    @IBOutlet weak var imageView: UIImageView!

    var selectedRow:Int!
    var selectedMemo : String!
    
    let saveData: UserDefaults = UserDefaults.standard

    //カメラボタンがタップされた時の処理
    @IBAction func launchCamera(_ sender: UIBarButtonItem) {
        let camera = UIImagePickerController.SourceType.camera
        if UIImagePickerController.isSourceTypeAvailable(camera) {
            let picker = UIImagePickerController()
            picker.sourceType = camera
            picker.delegate = self
            self.present(picker, animated: true)
        }
    }
    
    //ユーザーが撮影し終わった時の処理
    func imagePickerController(_ picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [UIImagePickerController.InfoKey : Any]) {
        
        let image = info[UIImagePickerController.InfoKey.originalImage] as! UIImage
        self.imageView.image = image
        UIImageWriteToSavedPhotosAlbum(image, nil, nil, nil)
        self.dismiss(animated: true)
        UserDefaults.standard.set(image, forKey: "MemoImage")
        saveData.synchronize()
    }
    
    

    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
        
        let imageData:NSData = UserDefaults.standard.object(forKey: "MemoImage") as! NSData
        imageView.image = UIImage(data: imageData as Data)
//        imageView = saveData.object(forKey: "MemoImage") as! UIImageView
    }
    
    @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[selectedRow] = 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)
    }

}

#写真

急いでおります。
よろしくお願いします。

Comments

[hayabusabusash]

画像の保存はUserDefaultsで行っているようですが、
もしかして保存する時のキーは"MemoImage"で統一していますか？

[Kaguya_4869]

そうです。memoimageで統一してます。

Answer 1

保存する時のキーをMemoImageで統一しているのが原因です。

UserDefaultsはplist形式になっていて、
あるキーに対して値が対応する形になっています。(Key-Value)
なので現状はMemoImageというキーに対して毎回上書きをかけている状態になっていると思います。

もし違う画像を保存したいならMemoImageで統一するのではなく、キーをユニークなものにする必要があると思います。

2019/11/28追記(コードについて)

ちょっと回りくどいかもしれませんが、現状Stringになっているメモを以下のような構造体にします。

struct Memo: Codable {
    let id: String
    let content: String
}

contentにはメモの内容を入れて、
idにはUUID().uuidStringを保存して画像の読み込みの時に使用します。
メモを保存する時に、このidをキーとして画像を保存してしまえばメモと画像が紐づく形で保存ができると思います。

Codableに準拠しているので
JSONEncoder、JSONDecoderを使用してData型にしたり元のMemoの構造体に戻すことができます。
少し面倒ですが一度Data型として取り出し、その後Memoの配列として取り出すような流れになります。

    // AddViewController

    @IBAction func save(_ sender: Any) {
        let inputText = memoTextView.text

        // 新しいメモの構造体を生成
        let memo = Memo(id: UUID().uuidString, content: inputText)
        
        // 一度Dataとして取り出し、そのあとにMemoの配列にデコード
        if let data = UserDefaults.standard.data(forKey: /*Memoの配列を保存するキー*/),
             let storedMemoArray = try? JSONDecoder().decode([Memo].self, from: data) {
            var newMemoArray = storedMemoArray
            newMemoArray.append(memo)

            // Dataに戻して保存
            if let newData = try? JSONEncoder().encode(newMemoArray) {
                UserDefaults.standard.set(newData, forKey: /*Memoの配列を保存するキー*/)
            }
        } else {
            // 何も保存されていなかった時の処理
        }

        // ImagePickerから選択された画像を取り出し、Dataに変換
        if let image = imageView.image,
           let imageData = image.jpegData(compressionQuality: 0.8) {
            // キーには上で生成したMemoのid(UUID)を使う
            UserDefaults.standard.set(imageData, forKey: memo.id)
        }
    }

    func imagePickerController(_ picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [UIImagePickerController.InfoKey : Any]) {
        let image = info[UIImagePickerController.InfoKey.originalImage] as! UIImage
        self.imageView.image = image
        UIImageWriteToSavedPhotosAlbum(image, nil, nil, nil)
        self.dismiss(animated: true)

        // ここで保存はせずに、imageViewに選択した画像をセットするだけにします
    }

保存はこんな感じになると思います。

Memoのidが保存された画像と紐づいているので、
画像の読み込みに特定のMemoのidを使って読みこめば期待する動作になるはずです。

簡単にですがサンプルを作ってみたので、よかったら参考にしてください。

Comments

[Kaguya_4869]

import UIKit

class TableViewCell: UITableViewCell {

@IBOutlet weak var cellImage: UIImageView!
@IBOutlet weak var cellLabel: UILabel!

// @IBOutlet weak var label: UILabel!
// @IBOutlet weak var tablecell: UITableViewCell!

let uuid = NSUUID().uuidString

override func awakeFromNib() {
super.awakeFromNib()

let imageData:NSData = UserDefaults.standard.object(forKey: uuid) as! NSData
cellImage.image = UIImage(data: imageData as Data)
// Initialization code
}

override func setSelected(_ selected: Bool, animated: Bool) {
super.setSelected(selected, animated: animated)

// Configure the view for the selected state
}

//ここを追加
func fill(image: UIImageView, titleLb: String, date: String, siteName: String){
cellImage.image = imageView?.image

}

}

こんな感じで付け足してみたんですけど、Unexpectedly found nil while unwrapping an Optional valueというエラーが出ていて、それが
let imageData:NSData = UserDefaults.standard.object(forKey: uuid) as! NSData
のコードのところに出ています。

どうすればいいでしょうか

[hayabusabusash]

UUIDは生成する度に違う値になります。
なので保存したときに使ったキーのUUIDは一致せず、UserDefaultsで読み込む際に値を取り出せずにエラーになっている感じですね。

UUIDを使うのはありだと思いますが、
今回の場合メモみたいな情報と一緒に保存する必要があると思います。
なのでメモとUUID等をプロパティとしてもつクラス、もしくは構造体を定義して、それを保存するようにすれば解決できると思います。
クラスの場合はUserDefaultsで保存する場合はデコードの処理が必要になるので注意してください。
構造体でやる場合はCodable準拠にしてData型として扱えばUseDefaultsで保存できるはずです。

[Kaguya_4869]

具体的にどんな感じでコードを書いてみればいいか教えていただけますか？
書き方がよくわからなくて…

[hayabusabusash]

長くなってしまいましたが追記しました！

[Kaguya_4869]

遅くなってしまい、大変申し訳ありませんでした。いつもいつもありがとうございます！！

[hayabusabusash]

解決されたようでよかったです！
もしサンプルで不明な点があれば、
Githunbの方でissueとして立ててもらえればいくらでも答えるのでよろしくお願いします????‍♂️