Wie kann dieser Code gekürzt werden, um Doppelungen zu vermeiden?

Question

Ich hatte diese Frage ziemlich lange. Ich versuche das Problem der 3 Türen zu visualisieren, nur um Spaß zu haben und mit Swift zu üben. So habe ich:

3 Türen und daher 3 verschiedene IBActions & 3 Funktionen für alle Türen. Diese Funktionen sind alle genau gleich, jedoch ist nur die Anzahl der Türen in jedem Code unterschiedlich. Also ich frage mich, kann ich diesen Code kürzen ?:

func openSecondChoice(whatDoorIsClickedOn: Int) 
    { 
     if whatDoorIsClickedOn == 1 
     { 
      if whatDoorIsClickedOn == doorWithNumber 
      { 
       UIButtonDoor1.setBackgroundImage(UIImage (named: "doorWithMoney"), for: UIControlState.normal) 
      } 
      else 
      { 
       UIButtonDoor1.setBackgroundImage(UIImage (named: "doorWithGoat"), for: UIControlState.normal) 
      } 
     } 
     if whatDoorIsClickedOn == 2 
     { 
      if whatDoorIsClickedOn == doorWithNumber 
      { 
       UIButtonDoor2.setBackgroundImage(UIImage (named: "doorWithMoney"), for: UIControlState.normal) 
      } 
      else 
      { 
       UIButtonDoor2.setBackgroundImage(UIImage (named: "doorWithGoat"), for: UIControlState.normal) 
      } 
     } 
     if whatDoorIsClickedOn == 3 
     { 
      if whatDoorIsClickedOn == doorWithNumber 
      { 
       UIButtonDoor3.setBackgroundImage(UIImage (named: "doorWithMoney"), for: UIControlState.normal) 
      } 
      else 
      { 
       UIButtonDoor3.setBackgroundImage(UIImage (named: "doorWithGoat"), for: UIControlState.normal) 
      } 
     } 
    } 

Yuk! Dieser Code ist so hässlich! Wenn der Benutzer zum Beispiel auf door1 drückt, rufe ich die Funktion "openSecondChoise (whatDoorIsClickedOn: 1)" auf. Gibt es eine Möglichkeit, dies zu verkürzen? Vielen Dank! Ich benutze keine Kurse hier, sollte ich sie benutzen?

Answer 1

Normalerweise, wenn Sie beginnen, Ihre Variablennamen mit 1, 2, 3, etc. zu suffixieren, ist es Zeit, stattdessen ein Array zu verwenden. Dafür gibt es Arrays.

Mit einem Array uiButtonDoors das enthält Ihre UIButtonDoor1 ... UIButtonDoor3, Ihre Funktion könnte wie folgt aussehen:

func openSecondChoice(whatDoorIsClickedOn: Int) { 
    let imageName = whatDoorIsClickedOn == doorWithNumber ? "doorWithMoney" : "doorWithGoat" 
    uiButtonDoors[whatDoorIsClickedOn - 1].setBackgroundImage(UIImage(named: imageName), for: UIControlState.normal) 
} 

Answer 2

func openSecondChoice(whatDoorIsClickedOn: Int) { 
    let imageName = whatDoorIsClickedOn == doorWithNumber ? "doorWithMoney" : "doorWithGoat" 
    let image = UIImage(named: imageName) 

    let button: UIButton 

    switch whatDoorIsClickedOn { 
    case 1: 
    button = UIButtonDoor1 
    case 2: 
    button = UIButtonDoor2 
    case 3: 
    button = UIButtonDoor3 
    default: 
    fatalError("Cannot be. Switch must be exhaustive, that's why we need to use 'default' for a switch on Int.") 
    } 

    button.setBackgroundImage(image, for: .normal) 
} 

für eine kürzere Versionsprüfung @tuple_cat s Antwort.

Answer 3

Ein anderer Ansatz, zum Spaß.

import UIKit 

class DoorGame { 
    func setupButtons() { 
     let buttons = [UIButton(), UIButton(), UIButton()] 

     for (index, button) in buttons.enumerated() { 
      button.addTarget(self, action: #selector(buttonTapped(_:)), for: .touchUpInside) 
      button.setTitle("\(index)", for: .normal) 
     } 

     let winningIndex = Int(arc4random_uniform(UInt32(buttons.count))) 

     buttons[winningIndex].tag = 1 
    } 

    @objc func buttonTapped(_ sender: UIButton) { 
     let imageName = sender.tag == 1 ? "doorWithMoney" : "doorWithGoat" 
     let image = UIImage(named: imageName) 

     sender.setBackgroundImage(image, for: .normal) 
    } 
}