swift プロトコルが呼ばれません

Question

プロフィールページからプロフィール編集ページへモーダル表示をしています。プロフィールを編集してモーダルを閉じた時に即座に変更点がプロフィールに反映されないためプロトコルを用いて変更をすぐに反映させようとしていますが、できません。

EditProfileViewController.swift

import UIKit
import NCMB
import NYXImagesKit
import PKHUD


protocol ProfileEditDelegate {
    func didEditProfile(profileText: String)
}

class EditProfileViewController: UIViewController, UITextViewDelegate, UITextFieldDelegate, UIImagePickerControllerDelegate, UINavigationControllerDelegate {
    @IBOutlet var mainImageView: UIImageView!
    @IBOutlet var subImageCollectionView: UICollectionView!
    @IBOutlet var selectMainImageButton: UIButton!
    @IBOutlet var displayNameTextField: UITextField!
    @IBOutlet var searchIdTextfield: UITextField!
    @IBOutlet var profileTextView: UITextView!
    @IBOutlet var logOutButton: UIButton!

    override func viewDidLoad() {
        super.viewDidLoad()

        if let user = NCMBUser.current(){
            displayNameTextField.text = user.object(forKey: "userName") as? String
            searchIdTextfield.text = user.object(forKey: "searchId") as? String
            profileTextView.text = user.object(forKey: "profileSentence") as? String
            let file = NCMBFile.file(withName: NCMBUser.current()?.objectId, data: nil) as! NCMBFile
            file.getDataInBackground { (data, error) in
                if error != nil{
                    print(error)
                }else{
                    if data != nil {
                        let image = UIImage(data: data!)
                        self.mainImageView.image = image
                        
                    }
                }
            }
            
        }else{
   
        }
    }
    
    
    
    
var editDelegate: ProfileEditDelegate?
    
    @IBAction func saveProfile(){
        let user = NCMBUser.current()
        user?.setObject(self.displayNameTextField.text, forKey: "userName")
        user?.setObject(self.searchIdTextfield.text, forKey: "searchId")
        user?.setObject(self.profileTextView.text, forKey: "profileSentence")
        
        
        editDelegate?.didEditProfile(profileText: profileTextView.text)
        print("ああああああああああああああああああ")
        
        
        user?.saveInBackground({ (error) in
            if error != nil {
                print(error)
            }else{
                
                print("いいいいいいいいいいいいいいいいいい")
                self.dismiss(animated: true, completion: nil)
            }
        })
    }
    
    @IBAction func cancel(){
        self.dismiss(animated: true, completion: nil)
    }
    
    func imagePickerController(_ picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [UIImagePickerController.InfoKey : Any]) {
        let selectedImage = info[UIImagePickerController.InfoKey.originalImage] as! UIImage
        let resizedImage = selectedImage.scale(byFactor: 0.1)
        picker.dismiss(animated: true, completion: nil)
        
        let data = resizedImage!.pngData()
        let file = NCMBFile.file(withName: NCMBUser.current()?.objectId, data: data) as! NCMBFile
        file.saveInBackground({ (error) in
            if error != nil{
                print(error)
            }else{
                self.mainImageView.image = resizedImage
            }
        }) { (progress) in
            print(progress)
        }
        
        
    }
    
    
    
    
    
    

}

UserPageViewController.swift

import NCMB
import PKHUD

class UserPageViewController: UIViewController, ProfileEditDelegate{
    func didEditProfile(profileText: String) {
        print("ーーーーーーー呼ばれてる")
        profileTextView.text = profileText
        print("ーーーーーーー呼ばれてる")
    }
    
   
    
    @IBOutlet var mainImageView: UIImageView!
    
    @IBOutlet var subImageCollectionView: UICollectionView!
    @IBOutlet var nameLabel: UILabel!
    @IBOutlet var followerButton: UIButton!
    @IBOutlet var followingButton: UIButton!
    @IBOutlet var profileTextView: UITextView!
    @IBOutlet var goToEdit: UIButton!
    
    

    

    override func viewDidLoad() {
        super.viewDidLoad()
        
        
        
        if let user = NCMBUser.current(){
            
            HUD.flash(.progress, delay: 0.1){
                finished in
                DispatchQueue.main.async {
                    self.nameLabel.text = user.object(forKey: "userName") as? String
                    self.profileTextView.text = user.object(forKey: "profileSentence") as? String
                    self.navigationItem.title = user.object(forKey: "searchId") as? String
                        let file = NCMBFile.file(withName: NCMBUser.current()?.objectId, data: nil) as! NCMBFile
                    
                        file.getDataInBackground { (data, error) in
                            if error != nil {
                                print(error)
                            }else{
                                if data != nil{
                                    let image = UIImage(data: data!)
                                    self.mainImageView.image = image
                                }
                                
                            }
                        }
                }
            }
            
            
        }else{
            let storyboard = UIStoryboard(name: "SignIn", bundle: Bundle.main)
            let rootViewController = storyboard.instantiateViewController(withIdentifier: "RootNavigationController")
            UIApplication.shared.keyWindow?.rootViewController = rootViewController
            
            let ud = UserDefaults.standard
            ud.set(false, forKey: "isLogin")
            ud.synchronize()
        }
    }
    
    override func viewWillAppear(_ animated: Bool) {
        if let user = NCMBUser.current(){
            
                nameLabel.text = user.object(forKey: "userName") as? String
                profileTextView.text = user.object(forKey: "profileSentence") as? String
                navigationItem.title = user.object(forKey: "searchId") as? String
            
                let file = NCMBFile.file(withName: NCMBUser.current()?.objectId, data: nil) as! NCMBFile
            
                file.getDataInBackground { (data, error) in
                    if error != nil {
                        print(error)
                    }else{
                        if data != nil{
                            let image = UIImage(data: data!)
                            self.mainImageView.image = image
                        }
                        
                    }
                }
           
            print(profileTextView)
            
            
        }else{
           let storyboard = UIStoryboard.init(name: "SignIn", bundle: Bundle.main)
            
           let rootViewController = storyboard.instantiateViewController(withIdentifier: "RootNavigationController")
            UIApplication.shared.keyWindow?.rootViewController = rootViewController
            
            let ud = UserDefaults.standard
            ud.set(false, forKey: "isLogin")
            ud.synchronize()
            
        }
        
        
        
        
        
    }
    
    
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        if segue.identifier == "fromUserPageToFollower"{
            let destinationcontroller = segue.destination as! FollowerViewController
            
            destinationcontroller.passedTappedUser = NCMBUser.current()
        }
        
        if segue.identifier == "fromUserPageToFollowing"{
            let destinationcontroller = segue.destination as! FollowingViewController
            
            destinationcontroller.passedTappedUser = NCMBUser.current()
        }
    }
    
    
   
    
    
    
    

   
}

print()を使って確認してみたところ、EditProfileViewController.swiftの方は全て呼ばれるべき関数等は呼ばれていましたが、UserPageViewController.swiftの

func didEditProfile(profileText: String) {
        print("ーーーーーーー呼ばれてる")
        profileTextView.text = profileText
        print("ーーーーーーー呼ばれてる")
    }

の部分は呼ばれていることが確認できません。
また、今はとりあえずプロフィール文のみの変更をモーダルを閉じ次第反映させようとしています。よろしくお願いします。

以下の画像のようにプロフィールページから編集ページはモーダルで繋がれています

Comments

[TsukubaDepot]

UserPageViewController.swift には同じコードが何度も現れるのですが、これはコピーミスでしょうか。
また、EditProfileViewController　が呼び出されるであろう場所も見つからないのですが、これはどこで呼び出されていますでしょうか。

ところで、delegate が呼び出されない一番多くありがちな原因ですが、tableView でいえば tableView.delegate = self といった感じで移譲先のインスタンスを設定することを忘れているパターンかと思います。こちらは大丈夫でしょうか。

[退会済みユーザー]

お返事ありがとうございます。コピーミスでした、申し訳ありませんでした。
自分の勉強不足で移譲先のインスタンスを設定できているのかすら分からない状況です。

[TsukubaDepot]

EditProfileViewController には、どのようにして遷移しているのでしょうか。
ボタンに直接 Segue を設定して遷移していらっしゃるのでしょうか。

[退会済みユーザー]

右上の青い歯車のアイコンからモーダル表示させています

Answer 1

全体的な実装が見えていないのですが、概ねこんな感じになるかと思います。

動きが分かりやすくなるように、余計な機能（NMCBなど）はバッサリと削ってあります。

import UIKit

class ViewController: UIViewController, ProfileEditDelegate {
    func didEditProfile(profileText: String) {
        // MARK: EditViewController でセットした値を表示するだけ
        print(profileText)
    }
    
    override func viewDidLoad() {
        super.viewDidLoad()
    }
    
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        // MARK: EditViewController への Segue を選択した場合
        if segue.identifier == "EditVCSegue" {
            // MARK: 遷移先が　Navigaton Controller の場合
            // カスタムクラスの場合には UINavigationController を適切な名前に変更
            if let nextNC = segue.destination as? UINavigationController {
                if let nextVC = nextNC.viewControllers[0] as? EditProfileViewController {
                    // MARK: EditProfileViewController の editDelegate に移譲先のインスタンスをセット
                    nextVC.editDelegate = self
                }
            }
            
            // MARK: 遷移先が通常の View Controller の場合はこちら
        //  if let nextVC = segue.destination as? EditProfileViewController {
        //      nextVC.editDelegate = self
        //  }
        }
    }
}

ViewController では、ボタンから次の View Controller へ直接 Segue を張っているので、ボタンを押すと prepare(for:sender:)が直接呼ばれますが、performSegue(withIdentifier:sender:) でインスタンス化した場合でも基本的に同じとなります。

prepare(for:sender:) で次の View Controller のインスタンスが見えますので、そこで delegate のためのインスタンスをセットするような形になります。

おそらく、EditProfileViewController は Navigation Controller 経由となっていると思われるので上記のような実装になっていますが、もし直接 View Controller をインスタンス化しているのであれば、コメントアウトした方で大丈夫です（いわゆる値渡しと呼ばれる方法と同じです）。

import UIKit

// MARK: delegate として宣言するプロトコルは弱参照させるため　class を継承させる
protocol ProfileEditDelegate: class {
    func didEditProfile(profileText: String)
}

class EditProfileViewController: UIViewController {
    // MARK:　weak キーワードを使って弱参照させる
    weak var editDelegate: ProfileEditDelegate?

    override func viewDidLoad() {
        super.viewDidLoad()
    }
    
    @IBAction func saveProfile(){
        editDelegate?.didEditProfile(profileText: "Saveボタンがタップされた。")
        self.dismiss(animated: true, completion: nil)
    }
    
    @IBAction func cancel(){
        self.dismiss(animated: true, completion: nil)
    }
}

遷移先ですが、まず delegate として宣言するプロトコルは弱参照させる必要があるため、class を継承するようにします。

また、editDelegate は weak キーワードを付けることで弱参照させます。プロトコルが class を継承していなければ weak キーワードをつけることができずエラーとなりますのでご注意ください。

あとは、基本的にオリジナルのままで大丈夫かと思います。

Comments

[退会済みユーザー]

返信遅れて申し訳ありません、無事機能しました。もっと勉強します。

[退会済みユーザー]

ありがとうございました