Only instance methods can be declared @IBActionを解決したい

前提・実現したいこと

while構文中にボタンを配置したかったのですが、Main.storyboardからボタンを紐すけでwhile中に出そうとしたら出来ず、コードを打ち込んであとで紐付けしようとしたらこのようなエラーが出ました
crass { }の中ではボタンの部分にエラーは出なかったのでwhile構文をoverride[]内からcrass[]内に移そうと考えたがcrass[]内に直接処理は書けないのでXcode経験の浅い自分からしたら詰み状態に陥りました。
何か良い改善策があればお教えいただきたいです。

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

Only instance methods can be declared @IBAction

該当のソースコード



import UIKit
import AVFoundation
import Darwin

class ViewController: UIViewController , AVAudioPlayerDelegate {
  
    var a = 15.0
    var A = 0.0
    var d = 0.0
    var X = 0.0
    var Y = 0.0
    var Xm = 0.0
    var XM = 0.0
    var Ym = 0.0
    var YM = 0.0
    var total = 0.0
    
    let koukaon = Bundle.main.bundleURL.appendingPathComponent("button04b.mp3")
    var buttonsaisei = AVAudioPlayer()
    var i = 0    // View
    
    var myImageView1: UIImageView!
    var myImageView2: UIImageView!
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
  
        myImageView1 = UIImageView(frame: CGRect(x:-50,y:-50,width:5.195*6,height:5.195*6))//固視点
        myImageView2 = UIImageView(frame: CGRect(x:-50,y:-50,width:5.195*2.2567,height:5.195*2.2567))
       
 
        let myImage1 = UIImage (named:"zyuuzi")
        let myImage2 = UIImage (named:"blue")
        
        myImageView1.image = myImage1
        myImageView2.image = myImage2
        
        // UIImageViewをViewに追加する
        self.view.addSubview(myImageView1)
        self.view.addSubview(myImageView2)
        
        //固視点の座標
        let position0 = CGPoint(x:300, y:300)


        myImageView1.layer.position = position0
        
        d = 12.0 / 11.0
        X = 300 * tan(15.0)
        Y = 300 * tan(3.0)
        
        Xm = 300 * tan(15.0 - 5.0 * d)
        XM = 300 * tan(15.0 + 5.0 * d)
        
        Ym = 300 * tan(3.0 - 5.0 * d)
        YM = 300 * tan(3.0 + 5.0 * d)
        
       
    
        
    
    while total > 0{
        
        let positionA1 = CGPoint(x:300+Xm, y:300+Y)
        let positionB1 = CGPoint(x:300+X, y:300+YM)
        let positionC1 = CGPoint(x:300+Xm, y:300+Y)
        let positionD1 = CGPoint(x:300+X, y:300+Ym)
        
        var positions1 =  [positionA1,positionB1,positionC1,positionD1]
        
        @IBAction func No(_ sender: Any) {
        }
    
    }

}
}

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

hoshi-takanori

2021/01/17 08:49

「crass[]内に直接処理は書けない」の意味がよく分かりません。ソースコードが省略しすぎで情報が少なすぎるので、できれば省略せずにソースを貼って欲しいです。

kikizi

2021/01/17 13:57

https://qiita.com/hyoutann/items/76513fc40ab5881f84a1 自分でどうにかしようと調べて見つけたのがこの記事で、内容がclass ViewController: UIViewController{ } 内では処理を直接書いてはいけないとありました。whileがcrass内で使おうとしてエラーになったのが上記の理由であると判断しました。編集でコードを追記しましたがまだ作成途中です。自分が作っているアプリは画像を表示してそれが見えるか否かを判断し、ボタンを押すといったものです。A1の位置に画像を表示→ボタンを押すと今度はB2の座標に画像が表示→繰り返して4点全て見えたらwhileが終了するシステムを考えています。方法はボタンを二個設置し、[見えた]ボタンを押すとtotal ＝ + 0,[見えなかった]ボタンを押すとtotal = + 1し、4点が見えたならtotal = 0で繰り返し終了と言ったシステムです。『whileでループしたいのに override func viewDidLoad() { }内にボタンが設置できず、逆にcrass{}内でwhileを使おうとすると上記の記事のエラーが発生します。改善案、何卒よろしくお願い申し上げます。』

hoshi-takanori

2021/01/18 05:26

4 枚の絵を順番に表示して、それぞれに対して「見えた」「見えない」ボタンを押して欲しいからループの中でメソッドを定義したかったんですね。気持ちは分かりますが、それは iOS をはじめとする GUI プログラミングの考え方とは違ってしまいます。この場合は、4 枚の絵をループで順番に表示することはできなくて、現在何枚目の絵を表示しているかを変数に入れて覚えておいて、ボタンが押されたら次の絵を表示する、というコードを書く必要があります。このため、@IBAction はボタンの数だけメソッドを用意することになります。

kikizi

2021/01/18 06:03

根本的にプログラムの書き方が間違っていたのですね。ありがとうございます。プログラムの見直しをしてみようと思います。