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

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

ただいまの
回答率

90.47%

  • Swift

    7479questions

    Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

@IBAction funcを書く位置が間違っているのか?

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 538
退会済みユーザー

退会済みユーザー

@IBAction funcを書く位置が間違っているのでしょうか?
SendControllerに

import UIKit

class SendController:UIViewController,
UINavigationControllerDelegate,UIImagePickerControllerDelegate{

    @IBAction func ButtonCamera(_ sender: Any) {
    }

    //定数
    let ButtonCamera = 0
    let ButtomRead = 1
    let ButtonWrite = 2


    //変数
    var imageView:UIImageView  = UIImageView()
    var btnCamera:UIButton = UIButton(type: .custom)
    var btnRead:UIButton  = UIButton(type: .custom)
    var btnWrite:UIButton  = UIButton(type: .custom)

    //ロード完了時に呼ばれる
    override func viewDidLoad() {
        super.viewDidLoad()
    }

    //ボタンクリック時に呼ばれる
       func onClick(sender:UIButton){
       if sender.tag == ButtonCamera {
          showAlert(title: nil, text: "利用できません")
          openPicker(sourceType: UIImagePickerControllerSourceType.camera)


         }else if sender.tag == ButtomRead {
           openPicker(sourceType: UIImagePickerControllerSourceType.photoLibrary)
        }

    }

    //アラートの表示
    func showAlert(title: String?, text: String?) {
        let alert = UIAlertController(title: title, message: text, preferredStyle: UIAlertControllerStyle.alert)
        alert.addAction(UIAlertAction(title: "OK", style: UIAlertActionStyle.default, handler: nil))
        present(alert, animated: true, completion: nil)
    }

  //イメージピッカーのオープン
    func openPicker(sourceType:UIImagePickerControllerSourceType){
        if !UIImagePickerController.isSourceTypeAvailable(sourceType){
            showAlert(title: nil, text: "利用できません")
            return
        }

        //イメージピッカーの生成
        let picker = UIImagePickerController()
        picker.sourceType = sourceType
        picker.delegate = self

        //ビューコントローラーのビューを開く
        present(picker, animated: true, completion: nil)
           }

//   // イメージピッカーのイメージ取得時に呼ばれる
    func imagePickerController(_ picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [String : Any]) {
        let image = info[UIImagePickerControllerOriginalImage]as! UIImage
        imageView.image = image
        //ビューコントローラーのビューを閉じる
        picker.presentingViewController?.dismiss(animated: true,completion:nil)
    }

//    //イメージピッカーのキャンセル取得時に呼ばれる
    func imagePickerControllerDidCancel(_ picker: UIImagePickerController) {
        picker.presentingViewController?.dismiss(animated: true, completion: nil)
    }


}


ButtonCameraと名付けた”写真を撮る”ボタンを押すと、カメラが起動するようにしたいです。MainStoryBoardのそのボタンをコントロールキーを押しながら

 @IBAction func ButtonCamera(_ sender: Any) {
    }


の部分でドラックアンドドロップしました。
その時のConnection欄をActionにし@IBAction func ButtonCamera〜のコードを追加しました。
ですが、自分のiPhoneでアプリを立ち上げて、”写真を撮る”ボタンを押してもカメラが起動しません。
@IBAction func ButtonCamera〜書く位置が間違っているのか、と思いましたがどう直せばいいのかわからず。
どう直せば良いのでしょうか?

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+1

アクションは実際には通常のメソッドと同じもので、何か動作をさせるには{}内に処理を記述しなければなりません。
ボタンを押したときに実行したいのはonClickでしょうか?それでしたら、その内容をButtonCamera内に移す必要があるかと思います。
あるいは、onClickの頭に@IBActionを付けて、ButtonCameraの代わりにこちらをボタンと接続してもいいかもしれません。

[コメントに関して追記]
「中身を移す」の申し上げたのはfunc onClick(sender:UIButton){}で挟まれた部分を@IBAction func ButtonCamera(_ sender: Any) {}で挟まれた部分に移動するということを意図しました。
さらに、ButtonCameraの中でsenderのtagを見て分岐していますので、tagを見ることができるようsenderの型をAnyからUIButtonにしてみました。
このアクションを、ButtonCameraとButtomReadに対応する2つのボタンに繋いでみてください(ButtonCameraとButtomReadのどちらを押しても同じアクションが実行されるようにする)。

import UIKit

class SendController:UIViewController,
UINavigationControllerDelegate,UIImagePickerControllerDelegate{

    //このアクションとボタンを繋ぐ
    @IBAction func ButtonCamera(_ sender: UIButton) { // senderの型をUIButtonに変更
        if sender.tag == ButtonCamera {
            showAlert(title: nil, text: "利用できません")
            openPicker(sourceType: UIImagePickerControllerSourceType.camera)

        }else if sender.tag == ButtomRead {
            openPicker(sourceType: UIImagePickerControllerSourceType.photoLibrary)
        }
    }

    //定数
    let ButtonCamera = 0
    let ButtomRead = 1
    let ButtonWrite = 2


    //変数
    var imageView:UIImageView  = UIImageView()
    var btnCamera:UIButton = UIButton(type: .custom)
    var btnRead:UIButton  = UIButton(type: .custom)
    var btnWrite:UIButton  = UIButton(type: .custom)

    //ロード完了時に呼ばれる
    override func viewDidLoad() {
        super.viewDidLoad()
    }

    //ボタンクリック時に呼ばれる
    //以下のメソッドの中身をButtonCameraに移動
    /*
    func onClick(sender:UIButton){
        if sender.tag == ButtonCamera {
            showAlert(title: nil, text: "利用できません")
            openPicker(sourceType: UIImagePickerControllerSourceType.camera)


        }else if sender.tag == ButtomRead {
            openPicker(sourceType: UIImagePickerControllerSourceType.photoLibrary)
        }

    }
    */

    //アラートの表示
    func showAlert(title: String?, text: String?) {
        let alert = UIAlertController(title: title, message: text, preferredStyle: UIAlertControllerStyle.alert)
        alert.addAction(UIAlertAction(title: "OK", style: UIAlertActionStyle.default, handler: nil))
        present(alert, animated: true, completion: nil)
    }

    //イメージピッカーのオープン
    func openPicker(sourceType:UIImagePickerControllerSourceType){
        if !UIImagePickerController.isSourceTypeAvailable(sourceType){
            showAlert(title: nil, text: "利用できません")
            return
        }

        //イメージピッカーの生成
        let picker = UIImagePickerController()
        picker.sourceType = sourceType
        picker.delegate = self

        //ビューコントローラーのビューを開く
        present(picker, animated: true, completion: nil)
    }

    //   // イメージピッカーのイメージ取得時に呼ばれる
    func imagePickerController(_ picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [String : Any]) {
        let image = info[UIImagePickerControllerOriginalImage]as! UIImage
        imageView.image = image
        //ビューコントローラーのビューを閉じる
        picker.presentingViewController?.dismiss(animated: true,completion:nil)
    }

    //    //イメージピッカーのキャンセル取得時に呼ばれる
    func imagePickerControllerDidCancel(_ picker: UIImagePickerController) {
        picker.presentingViewController?.dismiss(animated: true, completion: nil)
    }


}

onClickをアクションに変えるパターンはこのような感じです。

import UIKit

class SendController:UIViewController,
UINavigationControllerDelegate,UIImagePickerControllerDelegate{

    //これは削除
    /*
    @IBAction func ButtonCamera(_ sender: Any) {
    }
    */

    //定数
    let ButtonCamera = 0
    let ButtomRead = 1
    let ButtonWrite = 2


    //変数
    var imageView:UIImageView  = UIImageView()
    var btnCamera:UIButton = UIButton(type: .custom)
    var btnRead:UIButton  = UIButton(type: .custom)
    var btnWrite:UIButton  = UIButton(type: .custom)

    //ロード完了時に呼ばれる
    override func viewDidLoad() {
        super.viewDidLoad()
    }

    //ボタンクリック時に呼ばれる
    @IBAction func onClick(sender:UIButton){ // @IBActionを追加
        if sender.tag == ButtonCamera {
            showAlert(title: nil, text: "利用できません")
            openPicker(sourceType: UIImagePickerControllerSourceType.camera)


        }else if sender.tag == ButtomRead {
            openPicker(sourceType: UIImagePickerControllerSourceType.photoLibrary)
        }

    }

    //アラートの表示
    func showAlert(title: String?, text: String?) {
        let alert = UIAlertController(title: title, message: text, preferredStyle: UIAlertControllerStyle.alert)
        alert.addAction(UIAlertAction(title: "OK", style: UIAlertActionStyle.default, handler: nil))
        present(alert, animated: true, completion: nil)
    }

    //イメージピッカーのオープン
    func openPicker(sourceType:UIImagePickerControllerSourceType){
        if !UIImagePickerController.isSourceTypeAvailable(sourceType){
            showAlert(title: nil, text: "利用できません")
            return
        }

        //イメージピッカーの生成
        let picker = UIImagePickerController()
        picker.sourceType = sourceType
        picker.delegate = self

        //ビューコントローラーのビューを開く
        present(picker, animated: true, completion: nil)
    }

    //   // イメージピッカーのイメージ取得時に呼ばれる
    func imagePickerController(_ picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [String : Any]) {
        let image = info[UIImagePickerControllerOriginalImage]as! UIImage
        imageView.image = image
        //ビューコントローラーのビューを閉じる
        picker.presentingViewController?.dismiss(animated: true,completion:nil)
    }

    //    //イメージピッカーのキャンセル取得時に呼ばれる
    func imagePickerControllerDidCancel(_ picker: UIImagePickerController) {
        picker.presentingViewController?.dismiss(animated: true, completion: nil)
    }


}

こちらも同じく、ButtonCameraとButtomReadの両方のボタンに接続してみてください。

接続図

注意点として、ストーリーボード上のButtonCameraボタンが削除したアクションButtonCameraに繋がったままだと、ボタンを押した時にonClickとButtonCameraの両方のアクションを実行しようとしてクラッシュすると思いますので、コネクションインスペクタで古いアクションへの接続は切ってください。

古い接続を切る

このような感じでいかがでしょうか?

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/06/04 21:42

    ありがとうございます。

    キャンセル

  • 2017/06/04 21:43

    import UIKit

    class KenshinSendController:UIViewController,
    UINavigationControllerDelegate,UIImagePickerControllerDelegate{

    @IBAction func ButtonCamera(_ sender: Any) {
    //イメージピッカーのオープン
    func openPicker(sourceType:UIImagePickerControllerSourceType){
    if !UIImagePickerController.isSourceTypeAvailable(sourceType){
    showAlert(title: nil, text: "利用できません")
    return
    }

    //イメージピッカーの生成
    let picker = UIImagePickerController()
    picker.sourceType = sourceType
    picker.delegate = self

    //ビューコントローラーのビューを開く
    present(picker, animated: true, completion: nil)
    }
    }

    //定数
    let ButtonCamera = 0
    let ButtomRead = 1
    let ButtonWrite = 2


    //変数
    var imageView:UIImageView = UIImageView()
    var btnCamera:UIButton = UIButton(type: .custom)
    var btnRead:UIButton = UIButton(type: .custom)
    var btnWrite:UIButton = UIButton(type: .custom)


    //ロード完了時に呼ばれる
    override func viewDidLoad() {
    super.viewDidLoad()

    }


    //アラートの表示
    func showAlert(title: String?, text: String?) {
    let alert = UIAlertController(title: title, message: text, preferredStyle: UIAlertControllerStyle.alert)
    alert.addAction(UIAlertAction(title: "OK", style: UIAlertActionStyle.default, handler: nil))
    present(alert, animated: true, completion: nil)
    }


    // // イメージピッカーのイメージ取得時に呼ばれる
    func imagePickerController(_ picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [String : Any]) {
    let image = info[UIImagePickerControllerOriginalImage]as! UIImage
    imageView.image = image
    //ビューコントローラーのビューを閉じる
    picker.presentingViewController?.dismiss(animated: true,completion:nil)
    }

    // //イメージピッカーのキャンセル取得時に呼ばれる
    func imagePickerControllerDidCancel(_ picker: UIImagePickerController) {
    picker.presentingViewController?.dismiss(animated: true, completion: nil)
    }


    }

    キャンセル

  • 2017/06/04 21:44

    と実行したいonClickをButtonCamera内に移しました。しかし、動きません。私の解釈が間違っているのでしょうか?

    キャンセル

  • 2017/06/05 06:08

    図を挿入するため回答欄の方に追記しました。
    ちなみに質問者さんの記述ですと、「ButtonCameraメソッドの内部でopenPickerメソッドを定義する」というような形になり、openPickerメソッドは定義されるのみで実行されず動作しなかったものと思われます。

    キャンセル

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

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

関連した質問

  • 解決済

    NSUserDefaultに画像を格納したいのですができません・

    テキストデータはNSUserDefaultに格納できたのですが、画像ができません。 画像をライブラリーから選択して、指定のimageviewに表示させることはできました。

  • 解決済

    iPadでのUIAlertControllerの使用について

    前提・実現したいこと iPad pro でactionsheetをタップするとクラッシュする件について 今、「世界一受けたいiPhoneアプリ開発の授業」の最新版を読んでいます

  • 解決済

    Swiftでの画面遷移について

    イタリックテキストいつもお世話になっております。 Swift3での画面遷移でうまく動かないので質問させてください。 やりたいことは、Photoボタンが押された後にUIIma

  • 受付中

    カメラアクセスと写真のアップロードができない

    作ったアプリでカメラアクセスと写真のアップロードができません。 Xcodeから自分のiPhoneでエミュレータを起動したのですが Camera access ボタンとPhoto

  • 解決済

    現在のmodalを閉じて、別のmodalを開く方法

    前提・実現したいこと ベースとなるビューを"A" "A"からpresentで開くことのできる画面を"B" "B"と同じく"A"から開くことのできる画面を"C"と仮定します。

  • 解決済

    tableViewを利用したlabelの表示について

    swift4.1 import UIKit class ViewController: UIViewController,UITableViewDataSource,UITa

  • 解決済

    割り勘アプリを作成しています。

     リセットボタンを実装したいです。 割り勘アプリを作成していて、 計算して結果を出した後に、 リセットボタンを押すと 人数と金額に入力するされた数字が 空欄の状態になるようにしたい

  • 解決済

    アラートで入力した文章を読み上げたい

    アラートで入力した値を変数textFieldに代入し、関数voice()で読み上げたいのですが、変数textFieldをどのようにして関数voice()に渡せばいいのかわかりません

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

  • Swift

    7479questions

    Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています