エラーとSwitch文について教えて下さい。

Question

前提・実現したいこと

1-7個あるimageを二個のボタンどちらを最初に押すかによってimageの切り替えの増え方を変えたいです。

例
最初にleftButtonを押した場合、
それからの処理はleftを押すとimageが1.3.5.7と変わり、rightを押すとimageは1-7のように始めにどちらのボタンを押すかにとって、それからの処理の条件を変化させたいです。

最初にrightuttonを押した時はそれからの処理はleftを押すとimageが1-7と変わり、rightを押すとimageは1.3.5.7です。

そして、ansewerNumberが設定してない８になったら画面遷移をしたいです。

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

１　こちらのコードだと交互にボタンを押した場合、どちらも1-7となってしまいます。

２　また7枚目まで行くと8枚目まで行かず画面遷移されずにそれで終わってしまいます。

該当のソースコード

import UIKit

class ViewController: UIViewController {
    
    //どちらのボタンを押すかで回数を確定させる
   var answerNumber = 0
    
    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view.
    }
    
    @IBOutlet weak var imageView: UIImageView!
    
    
    @IBOutlet weak var Label: UILabel!
    
    enum actionTag : Int {
        
        case action1 = 1
        case action2 = 2
    }
    
    @IBAction func left(_ sender: Any) {
        
        //条件分岐
        if let button = sender as? UIButton {
            if let tag = actionTag(rawValue: button.tag) {
                
                switch tag {
                case .action1:
                   answerNumber = answerNumber * 2
                    
                case .action2:
                   answerNumber = answerNumber + 1
                    
                default: break
                }
            }
            
        }
        
        //imageの画像を変換させたい
        //0枚目
        if answerNumber == 0 {
        
        Label.text = "hazime"
        imageView.image = UIImage(named:"0")
            
        } else if answerNumber == 1 {
            
            //1枚目
            Label.text = "itibann"
            imageView.image = UIImage(named:"1")
            
        } else if answerNumber == 2 {
            
            //2枚目
            Label.text = "nibann"
            imageView.image = UIImage(named:"2")
            
        } else if answerNumber == 3 {
            
            //3枚目
            Label.text = "sannban"
            imageView.image = UIImage(named:"3")
            
        }
        else if answerNumber == 4 {
            
            //4枚目
            Label.text = "yonban"
            imageView.image = UIImage(named:"4")
            
        }else if answerNumber == 5 {
            
            //5枚目
            Label.text = "goban"
            imageView.image = UIImage(named:"5")
            
        } else if answerNumber == 6 {
            
            //6枚目
            Label.text = "rokuban"
            imageView.image = UIImage(named:"6")
            
        } else if answerNumber == 7 {
        
        //7枚目
        Label.text = "nanaban"
        imageView.image = UIImage(named:"7")
        }
        
        answerNumber = answerNumber + 1
        
        
        ///imageが８に達したら画面遷移
        if answerNumber == 8 {
            
            answerNumber = 0
            
        performSegue(withIdentifier: "next", sender: nil)
        }
    }
    
    
    
    
    @IBAction func right(_ sender: Any) {
        
        
        
        if let button = sender as? UIButton {
            if let tag = actionTag(rawValue: button.tag) {
                
                switch tag {
                case .action1:
                   answerNumber = answerNumber + 1
                    
                case .action2:
                   answerNumber = answerNumber * 2
                    
                default: break
                }
            }
            
        }
        
        //imageの画像を変換させたい
        //0枚目
        if answerNumber == 0 {
        
        Label.text = "hazime"
        imageView.image = UIImage(named:"0")
            
        } else if answerNumber == 1 {
            
            //1枚目
            Label.text = "itiban"
            imageView.image = UIImage(named:"1")
            
        } else if answerNumber == 2 {
            
            //2枚目
            Label.text = "niban"
            imageView.image = UIImage(named:"2")
            
        } else if answerNumber == 3 {
            
            //3枚目
            Label.text = "sanban"
            imageView.image = UIImage(named:"3")
            
        }
        else if answerNumber == 4 {
            
            //4枚目
            Label.text = "yonban"
            imageView.image = UIImage(named:"4")
            
        }else if answerNumber == 5 {
            
            //5枚目
            Label.text = "goban"
            imageView.image = UIImage(named:"5")
            
        } else if answerNumber == 6 {
            
            //6枚目
            Label.text = "rokuban"
            imageView.image = UIImage(named:"6")
            
        } else if answerNumber == 7 {
        
        //7枚目
        Label.text = "nanaban"
        imageView.image = UIImage(named:"7")
        }
        
        answerNumber = answerNumber + 1
        
       
        ///imageが８に達したら画面遷移
        if answerNumber == 8 {
            
            answerNumber = 0
            
        performSegue(withIdentifier: "next", sender: nil)
        }
    }
}

試したこと
switch文の使い方がまだ上手く分かっていないので、サイトから見様見真似で
if let button = sender as? UIButton {
if let tag = actionTag(rawValue: button.tag) {

            switch tag {
            case .action1:
               answerNumber = answerNumber + 1
                
            case .action2:
               answerNumber = answerNumber * 2
                
            default: break
            }
        }    
    }

のところを追加しました、こちらを追加する前の処理では、一枚ずつ増えて８で画面遷移がちゃんと行われました。

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

Comments

[TsukubaDepot]

ソースコードを ``` で囲ってもらえますか？
https://teratail.com/help#about-markdown

単にみやすくするだけではなく、コピーしやすくなるという利点もあります。

[TsukubaDepot]

この質問ですが、単に switch 文だけの話ではなく、
1. tag を伴う UI 部品の制御
2. イニシャライザを使った列挙型（enum型）のインスタンスの作成
3. switch文を使った流れの制御
4. オプショナルバインディング（if-let）文
5. as! 演算子によるダウンキャスト（と、ダウンキャストに失敗したときの処理）
の5つの理解が必要ですが、どこまで理解されていますか？
それによって説明の方法が異なります。

さらに突っ込むと、

> 最初にleftButtonを押した場合、それからの処理はleftを押すとimageが1.3.5.7と変わり、rightを押すとimageは1-7のように始めにどちらのボタンを押すかにとって、それからの処理の条件を変化させたいです。
> 最初にrightuttonを押した時はそれからの処理はleftを押すとimageが1-7と変わり、rightを押すとimageは1.3.5.7です。

この流れを、ご自身でフローチャートに書き起こすことはできますか？
頭の中で想像するだけではなく、具体的に流れとして記述できるか否かという質問です。

[Ytan]

質問の5つの項目ですが、5つとも理解していないと言えます。
その場合だと一回基礎を学んでからの方がよろしいのでしょうか？

[TsukubaDepot]

ソースコード見やすくなりました。ありがとうございます。

ネットや本などからソースコードを引用して自分のコードにしようとすると、やはりSwiftの文法の理解やアルゴリズム、データ構造の理解は避けて通れないと思いますので、合わせて習得されるのがいいと思います。

switch 文へのコメントはあとで書いておきますので（switch文やオプショナルバインディング、enum値なども使わない方法も）、合わせて参考にしてください。

Answer 1

switch文そのものは非常にシンプルです。
次のような感じで使います。
playgroundで変数や条件の値を変更しながら実行してみるのがいいと思います。

let n = 1

switch n {
case 1:
    print("最初のcase")
case 2:
    print("2つめのcase")
default:
    print("該当なし")
}

// 範囲演算子も使える
let s = 68.6

switch s {
case 80...:
    print("A")
case 70..<80:
    print("B")
case 60..<70:
    print("C")
case ..<60:
    print("不合格")
default:
    print("該当なし")
}

基本は caseに条件を書き、続く行に処理を書きます。
複数行の処理も可能です。

どの条件にも合わない処理はdefault:以降で行います。

基本的に（例外あり） default:が無い switch文はエラーが出るので、その点は注意が必要です（注意というか、条件漏れがないように配慮してくれる）。

さらにはパターンマッチやlet文を使った代入などもできますが、いまは必要なさそうなので説明は省略します。

switch 文を使わずに実現するには

そもそも実現されたいことについては、switch文を使わなくても、変数を2つ使ってボタンの状態を把握することで実現できると思います。

これであれば if文だけでも実現できます（簡単なswitch文であれば if-elseの組み合わせでも実現できるため）。

ボタンを押すことによる数値の変化については一部不明な点もありますが、概ねこんな感じなんだろうと思います。

import UIKit

class ViewController: UIViewController {

    @IBOutlet weak var tokutenLabel: UILabel!
    
    // 初回のタップが行われたか
    var isFirstTap = false
    var isLeftFirst = false
    var tokuten = 0
    
    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view.
        tokutenLabel.text = String(tokuten)
    }

    @IBAction func leftButton(_ sender: Any) {
        if isFirstTap == false {
            // 両方とも最初のタップの場合
            
            // 初回タップを記憶
            isFirstTap = true
            // 左側が押されたことを記憶
            isLeftFirst = true
            
            // 初回のラベルの値
            tokuten = 1
        } else {
            // 2回目以降のタップの場合
            if isLeftFirst == true {
                tokuten += 2
            } else {
                tokuten += 1
            }
        }
        
        tokutenLabel.text = String(tokuten)
        
        // ここにImageの更新やsegueの実行を入れる
    }
    
    @IBAction func rightButton(_ sender: Any) {
        if isFirstTap == false {
            // 両方とも最初のタップの場合
            
            // 初回タップを記憶
            isFirstTap = true
            // 押されたのは左側では「無い」ので false
            isLeftFirst = false
            
            tokuten = 1
        } else {
            // 2回目以降のタップの場合
            if isLeftFirst == true {
                tokuten += 1
            } else {
                tokuten += 2
            }
        }
        
        tokutenLabel.text = String(tokuten)
        
        // ここにImageの更新やsegueの実行を入れる
    }
    
}

あえて冗長に書きました。

なので、同じような処理は関数にすることでもっと簡素化することができますし、変数isLeftFirstをオプショナル型にすれば（変数の値にnil, true, falseの3つの値を持たせることで）、変数isFirstTapも省略することも可能です。

#質問への追記について

質問の5つの項目ですが、5つとも理解していないと言えます。
その場合だと一回基礎を学んでからの方がよろしいのでしょうか？

やはり、基本的な制御文（if-else,for-in, switch-caseなどなど）の存在とそれらの具体的な使い方、変数の使い方については押さえておいた方がいい、というか必須だと思います。

制御文を用いたプログラムの制御とその理解は、別にSwiftに限った話ではなく、あらゆるプログラミング言語の基本となります。

ネットで調べられたことや、teratail でアドバイスを受けたことをうまく活用するためにも必要ですし、今後余計な労力を払わずに開発を進めるためにも必要です。

私が訪ねた5つの質問ですが、

1. tag を伴う UI 部品の制御
2. イニシャライザを使った列挙型（enum型）のインスタンスの作成
3. switch文を使った流れの制御
4. オプショナルバインディング（if-let）文
5. as! 演算子によるダウンキャスト（と、ダウンキャストに失敗したときの処理）

このうち3. について最低限必須な知識になりますし、4-5については、Swiftを使ってiOSで動くグラフィカルなアプリを作る上では理解は必須だと思います。

３. については先にも書きましたが、これは基本的な制御文ですから押さえておく必要はあります（if-elseで代用可能ですが、より見通しの良いプログラムを書く上で必須です）。

４. ですが、変数やメソッドが nil か否か、nilでなければどうするかという処理は至るところにありますから、理解必須です。

５. ですが、たとえばセグエ（segue）を使って別のインスタンス（ビュー、あるいは画面）に値を渡す場合などには避けて通れないのかな、と思います。

それ以外にもUI部品に渡されるsenderという変数もAny型で定義されていますが（これは具体的な型にすることができるはずですが）、ここで渡された変数を使う場合にも必要です。

提示していただいたコードの中で

　       if let button = sender as? UIButton {

こういう行がありますが、まさにこれが型の変換です。

Any型で渡された変数をより具体的な型に変換（キャスト）するには as?演算子もしくはas!演算子を使いますが、それぞれ実行結果が異なるので、理解して使わないと実行時エラーを引き起こしてしまいます。

なかなかやることが多くて大変だったと思いますが、必要に迫られて知識を整理することで、いままで見えてこなかったことが明らかに違った視点で見えてくると思いますのでお勧めします。

Comments

[Ytan]

とても分かりやすい説明とていねいなご説明ありがとうございます。
やる事が多いですが、やはり基本なので一つずつやっていきたいと思います。
ありがとうございました。

[TsukubaDepot]

どんなに複雑なプログラムも、必ず小さい要素に分解できますから、まずは小さなサンプルをいくつも作り、それを組み合わせるようにするといいと思います。

[Ytan]

switch文を教えてもらったif文に書き換えて思い通りに遷移もされるのですが、Bクラスで戻るButtonをdismissにて行ったのですが、戻ったときに初期状態に戻らなく、始めに押されたボタンを継承したままで、右を最初に押したあと画面遷移して戻ると次に左を押しても左は1個づつしか増えません、初期化をしたい時はどうすればよいのですか？

[TsukubaDepot]

Bクラスに遷移しても、呼び出し元のクラスで使っている変数は初期化されませんから（生きたまま、という表現が適切かわかりませんが、呼び出し元のクラスは生き続けています）、Bクラスをdismissしたあと呼び出し元クラスに戻ったときには、改めて変数を自分でリセットする必要があります。

たとえば、viewが表示される「前」に呼び出されるメソッドに viewWillAppear がありますが、その内部でこんな感じでフラグなどをリセットする方法があるとおもいます。

override func viewWillAppear(_ animated: Bool) {
// 親クラスの同じ名前のメソッドを呼び出す
super.viewWillAppear(animated)

isFirstTap = false
isLeftFirst = false
tokuten = 0
tokutenLabel.text = String(tokuten)
}

viewDidAppear でも同じようなことはできますが、こちらは表示された後に呼び出されるので、一瞬古い表示のままのラベルが残ってしまいます。試してみるといいと思いますよ。

[Ytan]

自分もviewWillAppearかと思っていたのですがイマイチ実証の仕方がわかりませんでしたが、どうやらPresentationがFullscreanになっていない事が原因でした。試した所うまく動作しました！
本当に一から何まで教えてくださりありがとうございます。
色々新しいことも知る事ができました！