swift - 按钮混淆了

Question

我想先说我是一名学生，我目前正在学习 SWIFT。我没有 100% 的理解。

嘿，所以，我的按钮总是与代码混淆。我告诉按钮在单击时在标签上显示一条消息，但它显示我告诉另一个按钮要做的其他事情。我不知道如何解释这个问题，抱歉。

我有顶部按钮(总共有四个)显示“你是正确的!”并且还会出现一个用代码编写的按钮，但是当我按下按钮时，当我运行该应用程序时，它会显示“你错了!”，这是所有其他按钮应该说的。

这是一个与音乐相关术语的问答游戏。

我已经尝试更改按钮 (1-4) 的标签但没有用，我还尝试断开它们并将它们从 Storyboard 上删除并重新添加所有内容，但这也没有用工作。

import UIKit

class QuizVC: UIViewController {

@IBOutlet weak var questionLabel: UILabel!
@IBOutlet weak var answerLabel: UILabel!




@IBOutlet weak var button1: UIButton!
@IBOutlet weak var button2: UIButton!
@IBOutlet weak var button3: UIButton!
@IBOutlet weak var button4: UIButton!

@IBOutlet weak var Next: UIButton!

var CorrectAnswer = String()

override func viewDidLoad() {
    super.viewDidLoad()
    // Do any additional setup after loading the view, typically from a nib.
    RandomQuestions()
}


func RandomQuestions() {
    var RandomNumber = arc4random() % 5
    var RandomQuestion = arc4random() % 4
    RandomQuestion += 1
    RandomNumber += 1
    Next.isHidden = true

    switch(RandomNumber) {

    case 1:
        if RandomQuestion == UInt32(1) {
            questionLabel.text = "What is the tail end of a piece of music?"
            button1.setTitle("Coda", for: UIControl.State.normal)
            button2.setTitle("Da Capo", for: UIControl.State.normal)
            button3.setTitle("Forte", for: UIControl.State.normal)
            button4.setTitle("Largo", for: UIControl.State.normal)
            CorrectAnswer = "1"
        } else if RandomQuestion == UInt32(2) {
            questionLabel.text = "What is the tail end of a piece of music?"
            button1.setTitle("Da Capo", for: UIControl.State.normal)
            button2.setTitle("Largo", for: UIControl.State.normal)
            button3.setTitle("Coda", for: UIControl.State.normal)
            button4.setTitle("Forte", for: UIControl.State.normal)
            CorrectAnswer = "3"
        } else if RandomQuestion == UInt32(3) {
            questionLabel.text = "What is the tail end of a piece of music?"
            button1.setTitle("Largo", for: UIControl.State.normal)
            button2.setTitle("Da Capo", for: UIControl.State.normal)
            button3.setTitle("Forte", for: UIControl.State.normal)
            button4.setTitle("Coda", for: UIControl.State.normal)
            CorrectAnswer = "4"
        } else {
            questionLabel.text = "What is the tail end of a piece of music?"
            button1.setTitle("Largo", for: UIControl.State.normal)
            button2.setTitle("Coda", for: UIControl.State.normal)
            button3.setTitle("Forte", for: UIControl.State.normal)
            button4.setTitle("Da Capo", for: UIControl.State.normal)
            CorrectAnswer = "2"
        }
        break
    case 2:
        if RandomQuestion == UInt32(1) {
            questionLabel.text = "Another name for Majestically?"
            button1.setTitle("Lunga", for: UIControl.State.normal)
            button2.setTitle("Dolce", for: UIControl.State.normal)
            button3.setTitle("Maestoso", for: UIControl.State.normal)
            button4.setTitle("Molto", for: UIControl.State.normal)
            CorrectAnswer = "3"
        } else if RandomQuestion == UInt32(2) {
            questionLabel.text = "Another name for Majestically?"
            button1.setTitle("Dolce", for: UIControl.State.normal)
            button2.setTitle("Lunga", for: UIControl.State.normal)
            button3.setTitle("Molto", for: UIControl.State.normal)
            button4.setTitle("Maestoso", for: UIControl.State.normal)
            CorrectAnswer = "4"
        } else if RandomQuestion == UInt32(3) {
            questionLabel.text = "Another name for Majestically?"
            button1.setTitle("Molto", for: UIControl.State.normal)
            button2.setTitle("Maestoso", for: UIControl.State.normal)
            button3.setTitle("Lunga", for: UIControl.State.normal)
            button4.setTitle("Dolce", for: UIControl.State.normal)
            CorrectAnswer = "2"
        } else {
            questionLabel.text = "Another name for Majestically?"
            button1.setTitle("Maestoso", for: UIControl.State.normal)
            button2.setTitle("Dolce", for: UIControl.State.normal)
            button3.setTitle("Lunga", for: UIControl.State.normal)
            button4.setTitle("Molto", for: UIControl.State.normal)
            CorrectAnswer = "1"
        }
        break
    case 3:
        if RandomQuestion == UInt32(1) {
            questionLabel.text = "Another name for Time/Speed?"
            button1.setTitle("Largo", for: UIControl.State.normal)
            button2.setTitle("Soli", for: UIControl.State.normal)
            button3.setTitle("Tenuto", for: UIControl.State.normal)
            button4.setTitle("Tempo", for: UIControl.State.normal)
            CorrectAnswer = "4"
        } else if RandomQuestion == UInt32(2) {
            questionLabel.text = "Another name for Time/Speed?"
            button1.setTitle("Soli", for: UIControl.State.normal)
            button2.setTitle("Tenuto", for: UIControl.State.normal)
            button3.setTitle("Tempo", for: UIControl.State.normal)
            button4.setTitle("Largo", for: UIControl.State.normal)
            CorrectAnswer = "3"
        } else if RandomQuestion == UInt32(3) {
            questionLabel.text = "Another name for Time/Speed?"
            button1.setTitle("Tempo", for: UIControl.State.normal)
            button2.setTitle("Largo", for: UIControl.State.normal)
            button3.setTitle("Soli", for: UIControl.State.normal)
            button4.setTitle("Tenuto", for: UIControl.State.normal)
            CorrectAnswer = "1"
        } else {
            questionLabel.text = "Another name for Time/Speed?"
            button1.setTitle("Largo", for: UIControl.State.normal)
            button2.setTitle("Tempo", for: UIControl.State.normal)
            button3.setTitle("Tenuto", for: UIControl.State.normal)
            button4.setTitle("Soli", for: UIControl.State.normal)
            CorrectAnswer = "2"
        }
        break
    case 4:
        if RandomQuestion == UInt32(1) {
            questionLabel.text = "Another name for Softly"
            button1.setTitle("Piano", for: UIControl.State.normal)
            button2.setTitle("Forte", for: UIControl.State.normal)
            button3.setTitle("Segno", for: UIControl.State.normal)
            button4.setTitle("Tacet", for: UIControl.State.normal)
            CorrectAnswer = "1"
        } else if RandomQuestion == UInt32(2) {
            questionLabel.text = "Another name for Softly"
            button1.setTitle("Forte", for: UIControl.State.normal)
            button2.setTitle("Tacet", for: UIControl.State.normal)
            button3.setTitle("Piano", for: UIControl.State.normal)
            button4.setTitle("Segno", for: UIControl.State.normal)
            CorrectAnswer = "3"
        } else if RandomQuestion == UInt32(3) {
            questionLabel.text = "Another name for Softly"
            button1.setTitle("Tacet", for: UIControl.State.normal)
            button2.setTitle("Segno", for: UIControl.State.normal)
            button3.setTitle("Forte", for: UIControl.State.normal)
            button4.setTitle("Piano", for: UIControl.State.normal)
            CorrectAnswer = "4"
        } else {
            questionLabel.text = "Another name for Softly"
            button1.setTitle("Forte", for: UIControl.State.normal)
            button2.setTitle("Piano", for: UIControl.State.normal)
            button3.setTitle("Segno", for: UIControl.State.normal)
            button4.setTitle("Tacet", for: UIControl.State.normal)
            CorrectAnswer = "2"
        }
        break
    case 5:
        if RandomQuestion == UInt32(1) {
            questionLabel.text = "Another name for Really Fast?"
            button1.setTitle("Staccato", for: UIControl.State.normal)
            button2.setTitle("Fermata", for: UIControl.State.normal)
            button3.setTitle("Presto", for: UIControl.State.normal)
            button4.setTitle("Mezzo", for: UIControl.State.normal)
            CorrectAnswer = "3"
        } else if RandomQuestion == UInt32(2) {
            questionLabel.text = "Another name for Really Fast?"
            button1.setTitle("Fermata", for: UIControl.State.normal)
            button2.setTitle("Presto", for: UIControl.State.normal)
            button3.setTitle("Staccato", for: UIControl.State.normal)
            button4.setTitle("Mezzo", for: UIControl.State.normal)
            CorrectAnswer = "2"
        } else if RandomQuestion == UInt32(3) {
            questionLabel.text = "Another name for Really Fast?"
            button1.setTitle("Fermata", for: UIControl.State.normal)
            button2.setTitle("Mezzo", for: UIControl.State.normal)
            button3.setTitle("Staccato", for: UIControl.State.normal)
            button4.setTitle("Presto", for: UIControl.State.normal)
            CorrectAnswer = "4"
        } else {
            questionLabel.text = "Another name for Really Fast?"
            button1.setTitle("Presto", for: UIControl.State.normal)
            button2.setTitle("Mezzo", for: UIControl.State.normal)
            button3.setTitle("Staccato", for: UIControl.State.normal)
            button4.setTitle("Fermata", for: UIControl.State.normal)
            CorrectAnswer = "1"
        }
    default:

        break
    }
}




@IBAction func NextAction(_ sender: Any) {
    RandomQuestions()
    answerLabel.text = ""
}


@IBAction func button1Action(_ sender: Any) {
    if (CorrectAnswer == "1"){
        answerLabel.text = "You Are Correct!"
        Next.isHidden = false
    } else {
        answerLabel.text = "You Are Wrong!"
        Next.isHidden = true
    }
}

@IBAction func button2Action(_ sender: Any) {
    if (CorrectAnswer == "2"){
        answerLabel.text = "You Are Correct!"
        Next.isHidden = false
    } else {
        answerLabel.text = "You Are Wrong!"
        Next.isHidden = true
    }
}

@IBAction func button3Action(_ sender: Any) {
    if (CorrectAnswer == "3"){
        answerLabel.text = "You Are Correct!"
        Next.isHidden = false
    } else {
        answerLabel.text = "You Are Wrong!"
        Next.isHidden = true
    }
}

@IBAction func button4Action(_ sender: Any) {
    if (CorrectAnswer == "4"){
        answerLabel.text = "You Are Correct!"
        Next.isHidden = false
    } else {
        answerLabel.text = "You Are Wrong!"
        Next.isHidden = true
    }
}

}

为什么会这样？我该如何解决？我查看了代码，没有发现任何错误，但我可能经验不足，无法真正注意到真正的错误。

Answer 1

您的代码中有很多重复项。如果您使用按钮标签，您可以为所有的回答按钮共享一个@IBAction。此外，如果您 shuffle() 答案，您可以消除更多重复代码。

您在按钮中的混淆可能是您不小心将按钮连接到多个 @IBAction。如果您在 Storyboard 中有一个连接后复制一个按钮，就会发生这种情况。然后添加第二个连接导致按钮调用两个 @IBAction。更新后的代码更加直接，应该可以消除这种混淆。

import UIKit

// The format of the data is ["Question", "correct answer", "wrong answer 1", "wrong answer 2",
// "wrong answer 3"].  The answers will be shuffled after a random question has been chosen.
let quizData: [[String]] = [
    ["What is the tail end of a piece of music?", "Coda", "Da Capo", "Forte", "Largo"],
    ["Another name for Majestically?", "Maestoso", "Dolce", "Lunga", "Molto"],
    ["Another name for Time/Speed?", "Tempo", "Largo", "Soli", "Tenuto"],
    ["Another name for Softly", "Piano", "Forte", "Segno", "Tacet"],
    ["Another name for Really Fast?", "Presto","Mezzo", "Staccato", "Fermata"]
]

class QuizVC: UIViewController {

    @IBOutlet weak var questionLabel: UILabel!
    @IBOutlet weak var answerLabel: UILabel!

    @IBOutlet weak var button1: UIButton!
    @IBOutlet weak var button2: UIButton!
    @IBOutlet weak var button3: UIButton!
    @IBOutlet weak var button4: UIButton!

    @IBOutlet weak var nextButton: UIButton!

    var correctAnswer = 0

    override func viewDidLoad() {
        super.viewDidLoad()

        // Set tag for each button.  This could also be done in Storyboard
        for (idx, button) in [button1, button2, button3, button4].enumerated() {
            button?.tag = idx + 1
        }
        randomQuestions()
    }

    func randomQuestions() {
        // Pick a random question
        let questionData = quizData.randomElement()
        let rightAnswer = questionData[1]

        // Shuffle the answers
        let randomAnswers = questionData.dropFirst().shuffled()

        // Find out where the correct answer ended up
        correctAnswer = randomAnswers.firstIndex(of: rightAnswer)! + 1

        questionLabel.text = questionData[0]
        for (idx, button) in [button1, button2, button3, button4].enumerated() {
            button?.setTitle(randomAnswers[idx], for: .normal)
        }

        nextButton.isHidden = true
    }

    @IBAction func nextAction(_ sender: UIButton) {
        randomQuestions()
        answerLabel.text = ""
    }

    // Connect buttons 1 through 4 to this action
    @IBAction func buttonAction(_ sender: UIButton) {
        if correctAnswer == sender.tag {
            answerLabel.text = "You Are Correct!"
            nextButton.isHidden = false
        } else {
            answerLabel.text = "You Are Wrong!"
            nextButton.isHidden = true
        }
    }

}