Comment retourner rapidement à partir d'un init dans swift 3

Question

J'ai une application dans laquelle je voudrais créer un objet basé sur s'il y a déjà un objet équivalent sauvegardé aux valeurs par défaut de l'utilisateur. Je voudrais détecter cela dans la classe init et revenir tôt, si un objet est détecté. Voilà ce que je suis en train de faire:

init() { 

    /* There are two possibilities when creating a hero: 
    1. The hero is brand new and needs to be built from scratch 
    2. The hero is loaded from defaults */ 

    // Check to see if there is existing game data: 

    if defaultExistsForGameData() { 
     // This means there is a hero to load and no need to create a new one 
     self = extractHeroFromDefaults() // This just loads from UserDefaults 
     print("Loading hero from defaults with name of: \(hero.heroName).") 

     return self 
    } 

    // These actions are for creating a brand new hero 
    let size = CGSize(width: 32, height: 32) 
    let heroTexture = SKTexture(imageNamed: "hero2.ico") 
    super.init(texture: heroTexture, color: .clear, size: size) 

    self.isUserInteractionEnabled = true 
    self.name = "hero" 

    self.zPosition = 50 

} 

Il y a des erreurs de couple dans la console, l'auto est immuable, etc. Ce que je veux savoir si cela est un motif valable ou si je devrais prendre un tout une approche différente.

Answer 1

Dans Swift (contrairement à ObjC), init ne peut pas retourner un objet différent de lui-même. Une manière courante de réaliser ce que vous essayez de faire ici est d'utiliser une méthode de fabrication de classe (en rendant éventuellement init privé si vous ne voulez pas que d'autres objets puissent l'appeler directement).

Par exemple, quelque chose le long de ces lignes:

class func loadOrCreate() -> Hero { 
    if defaultExistsForGameData() { 
     // This means there is a hero to load and no need to create a new one 
     print("Loading hero from defaults with name of: \(hero.heroName).") 
     return extractHeroFromDefaults() // This just loads from UserDefaults 
    } else { 
     return Hero() 
    } 
} 

private init() { 
    let size = CGSize(width: 32, height: 32) 
    let heroTexture = SKTexture(imageNamed: "hero2.ico") 
    super.init(texture: heroTexture, color: .clear, size: size) 

    self.isUserInteractionEnabled = true 
    self.name = "hero" 

    self.zPosition = 50 
} 

Une autre approche qui est plus proche de votre API actuelle est de créer un séparé (éventuellement privé) désigné initialiseur comme ceci:

private init(name: String, zPosition: Int) { 
    let size = CGSize(width: 32, height: 32) 
    let heroTexture = SKTexture(imageNamed: "hero2.ico") 
    super.init(texture: heroTexture, color: .clear, size: size) 

    self.isUserInteractionEnabled = true 
    self.name = name 
    self.zPosition = zPosition 
} 

public convenience init() { 
    let name: String 
    let zPosition: Int 
    if defaultExistsForGameData() { 
     name = defaultName() // Read it out of user defaults 
     zPosition = defaultZPosition 
    } else { 
     name = "hero" 
     zPosition = 50 
    } 
    self.init(name: name, zPosition: zPosition) 
} 

Le Un problème avec cette approche est que cela peut être un peu surprenant. Ce n'est pas très clair ce qui est supposé se produire si vous créez plusieurs objets Hero. quelque chose comme loadOrCreate() rend très clair qu'il existe des influences externes.