質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.80%

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

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 362

score 1

前提・実現したいこと

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

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

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

該当のソースコード

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+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)

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/06/23 17:48

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.80%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • トップ
  • iOSに関する質問
  • Swiftで作成したアクションシートが表示されず、アクションシート内の項目が勝手に開く。