Swiftで作成したアクションシートが表示されず、アクションシート内の項目が勝手に開く。

Question

前提・実現したいこと

カメラかアルバムかを選ぶアクションシートが開くクラスを作成して、
メソッドを呼ぶだけで各場所に表示させたい。

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

カメラのみが開いてしまう。

該当のソースコード

showAlert.swift

import Foundation
import UIKit

class showAlert{

func showAlert(camera :Void,album: Void,view :UIViewController){
    
    let alertController = UIAlertController(title: "プロフィール画像", message: "どちらを使用しますか？", preferredStyle: .actionSheet)
    
    let action1 = UIAlertAction(title: "カメラ", style: .default) { (UIAlertAction) in
        
        camera
        
    }
    let action2 = UIAlertAction(title: "アルバム", style: .default) { (UIAlertAction) in
        
        album
        
    }
    let action3 = UIAlertAction(title: "キャンセル", style: .cancel)
    
    alertController.addAction(action1)
    alertController.addAction(action2)
    alertController.addAction(action3)
    
    view.present(alertController, animated: true, completion: nil)
    
}

}

ViewController.swift

@IBAction func ImageViewTapGesture(_ sender: UITapGestureRecognizer) {

    let generator = UINotificationFeedbackGenerator()
    generator.notificationOccurred(.success)
    
    alert.showAlert(camera: openCamera(), album: openAlbum(), view: self)
    
}


func openCamera(){
    
    let picker = UIImagePickerController()
    picker.sourceType = .camera
    picker.delegate = self
    // UIImagePickerController カメラを起動する
    present(picker, animated: true, completion: nil)
    picker.allowsEditing = true
    
}

func openAlbum(){
    
    let picker = UIImagePickerController()
    picker.sourceType = .photoLibrary
    picker.delegate = self
    // UIImagePickerController カメラを起動する
    present(picker, animated: true, completion: nil)
    picker.allowsEditing = true
    
}

func imagePickerController(_ picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [UIImagePickerController.InfoKey : Any]) {
    
    if info[.originalImage] as? UIImage != nil{
        
        let selectedImage = info[.originalImage] as? UIImage
        
        UserDefaults.standard.set(selectedImage?.jpegData(compressionQuality: 0.1), forKey: "userImageData")
        
        userImageView.image = selectedImage
        
        
        
        picker.dismiss(animated: true, completion: nil)
        
    }
    
}

   // styleをActionSheetに設定
        let alertSheet = UIAlertController(title: "title", message: "message", preferredStyle: UIAlertControllerStyle.ActionSheet)

        // 自分の選択肢を生成
        let action1 = UIAlertAction(title: "1", style: UIAlertActionStyle.Default, handler: {
            (action: UIAlertAction!) in
        })
        let action2 = UIAlertAction(title: "aka", style: UIAlertActionStyle.Destructive, handler: {
            (action: UIAlertAction!) in
        })
        let action3 = UIAlertAction(title: "cancel", style: UIAlertActionStyle.Cancel, handler: {
            (action: UIAlertAction!) in
        })

        // アクションを追加.
        alertSheet.addAction(action1)
        alertSheet.addAction(action2)
        alertSheet.addAction(action3)

        self.presentViewController(alertSheet, animated: true, completion: nil)

試したこと

始めたばかりなのでどこが間違っているのかわからず、何をどう試せば良いのかわかりませんでした。
すみません。

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

Xcode 11.5

Answer 1

alert.showAlert(camera: openCamera(), album: openAlbum(), view: self) 

この記述だと、showAlert()を呼び出す段階で最初のopenCamera()関数を評価し、実行してしまいます。

これが原因で

発生している問題・エラーメッセージ
カメラのみが開いてしまう。

という現象が発生してしまいます。

また、showAlertの宣言部分で前二つの引数がVoid型になっていますが、これも誤りです。

加えて、引数として受け取った関数を実行する部分が、単なる変数の評価になってしまっています。

実現したいことは関数としてshowAlert()に渡すことだとおもいますので、

1. showAlert の引数を 引数が Void型で戻り値が Void型 の関数やクロージャ として指定する
2. 引数として受け取った関数を実行する
3. 上記を受けて、関数の実行部分を訂正する

という変更を行えば、おそらくご期待の結果になるかと思います。

具体的には

// 引数の型を変更
func showAlert(camera : @escaping () -> Void, album: @escaping () -> Void, view :UIViewController){
        
    // 略    
    let action1 = UIAlertAction(title: "カメラ", style: .default) { (UIAlertAction) in
        // 関数として実行するので () をつける
        camera()            
    }

    let action2 = UIAlertAction(title: "アルバム", style: .default) { (UIAlertAction) in
        // 関数として実行するので () をつける
        album()
    }

と変更した上、呼び出し側は

        // 関数名だけを渡す
        alert.showAlert(camera: openCamera, album: openAlbum, view: self)

として実行してみてもらえますでしょうか。

Comments

[wLike]

うまく行きました！本当にありがとうございます！天才！！！！！