このビューコントローラは "willSet/didSet"ペアでリークしますか？

Question

あなたは、VC（緑）を持っており、それがパネル（黄色）「ホルダー」

あなたは10の異なるビュー・コントローラ...価格、販売、証券、トラック、ドライバー、パレットがあるとしていますあなたはイエローエリアに一度に1つずつ入れようとしています。これは、動的に我々がcurrentに現在のVC 1を保持するストーリーボード

instantiateViewController(withIdentifier: "PricesID") as! Prices 

から各VCをロードします。ここでは、それらの "スワップ"を可能にするコードです...

>>これは間違っています。このコードを使用しないでください< <

サルタンが以下で説明することをしなければなりません。

var current: UIViewController? = nil { 
    willSet { 
     // recall that the property name ("current") means the "old" one in willSet 
     if (current != nil) { 
      current!.willMove(toParentViewController: nil) 
      current!.view.removeFromSuperview() 
      current!.removeFromParentViewController() 
      // "!! point X !!" 
     } 
    } 
    didSet { 
     // recall that the property name ("current") means the "new" one in didSet 
     if (current != nil) { 
      current!.willMove(toParentViewController: self) 
      holder.addSubview(current!.view) 
      current!.view.bindEdgesToSuperview() 
      current!.didMove(toParentViewController: self) 
     } 
    } 
} 

>>>>>>>>重要！ < < < < < < < < <

また、あなたはこのような何かをすれば、緑のページが行われたときに、黄色のビューコントローラを取り除くために不可欠であり、注意してください。 currentがそれを保持し、緑のページが解放されることはありませんそれ以外の場合：

override func dismiss(animated flag: Bool, completion: (() -> Void)? = nil) { 
    current = nil 
    super.dismiss(animated: flag, completion: completion) 
} 

継続、あなたはこのようにcurrentプロパティを使用したい：

func showPrices() { 
    current = s.instantiateViewController(withIdentifier: "PricesID") as! Prices 
} 
func showSales() { 
    current = s.instantiateViewController(withIdentifier: "SalesID") as! Sales 
} 

しかし、予告「点X」を考えます。通常は、削除するView Controllerを必ずnilに設定します。

blah this, blah that 
blah.removeFromParentViewController() 
blah = nil 

しかし、私は実際には "willSet"コードブロックの中に現在の値を設定することはできません。そして、私はそれがちょうど何か（didSetの中）に設定されようとしていることに感謝します。しかし、それはちょっと変わったようです。何が欠けている？あなたは計算されたプロパティでこのようなことをすることができますか？

---

最終使用可能なバージョン..... Sulthanのアプローチを使用して

、これは、かなりのテストの後に完璧に動作します。だから、これはうまく機能

このような

// change yellow area to "Prices" 
current = s.instantiateViewController(withIdentifier: "PricesID") as! Prices 

// change yellow area to "Stock" 
current = s.instantiateViewController(withIdentifier: "StickID") as! Stock 

を呼び出す

...

var current: UIViewController? = nil { // ESSENTIAL to nil on dismiss 
    didSet { 
     guard current != oldValue else { return } 

     oldValue?.willMove(toParentViewController: nil) 
     if (current != nil) { 
      addChildViewController(current!) 

      holder.addSubview(current!.view) 
      current!.view.bindEdgesToSuperview() 
     } 
     oldValue?.view.removeFromSuperview() 

     oldValue?.removeFromParentViewController() 
     if (current != nil) { 
      current!.didMove(toParentViewController: self) 
     } 
    } 
    // courtesy http://stackoverflow.com/a/41900263/294884 
} 
override func dismiss(animated flag: Bool, completion: (() -> Void)? = nil) { 
    // ESSENTIAL to nil on dismiss 
    current = nil 
    super.dismiss(animated: flag, completion: completion) 
} 

Answer 1

：私は、これはあなたがやろうとしているものをはるかに優れていと思います。しかし、最大の問題は、コードをwillSetとdidSetの間で分割しようとしているということです。なぜなら、これはまったく必要ないからです。ところで

var current: UIViewController? = nil { 
    didSet { 
     guard current != oldValue else { 
      return 
     } 

     oldValue?.willMove(toParentViewController: nil)    
     if let current = current { 
      self.addChildViewController(current) 
     } 

     //... add current.view to the view hierarchy here... 
     oldValue?.view.removeFromSuperview() 

     oldValue?.removeFromParentViewController() 
     current?.didMove(toParentViewController: self) 
    } 
} 

、関数が呼び出される順序が重要です：あなたは、常にoldValuedidSetで使用することができます。したがって、私は機能をremoveとaddに分割するよう勧めません。そうでなければ、両方のコントローラのためにviewDidDisappearとviewDidAppearの順番が驚くかもしれません。

Answer 2

はのは、二つに質問を分割してみましょう：（1） "リーク" はありますか？ （2）これは良い考えですか？

最初に「リーク」。短い答え：いいえ。 currentをnilに設定しない場合でも、明らかに保持するビューコントローラは「リーク」しません。含まれているビューコントローラが存在しなくなると、ビューコントローラはcurrentを指します。

currentビューコントローラは、必要以上に長生きします。そのため、これは愚かなことです。子ビューコントローラーに強い参照が必要はありません。なぜなら、それはあなたのchildViewControllers[0]です（子ビューコントローラー「ダンス」を正しく行う場合）。したがって、あなたの財産を複製するだけで、childViewControllersプロパティはすでにとなります。

これは、2番目の質問につながります。それは良いアイデアをしているのですか？いいえ、どこから来ているのかわかります。子ビューコントローラの「ダンス」をカプセル化したいと思っています。しかし、あなたはどんな場合でもダンスを間違ってやっています。このようにView Controllerの階層を覆しています。 「ダンス」をカプセル化するために、私はあなたが正しくダンスをやって、それが存在する場合はchildViewController[0]を指し計算読み取り専用プロパティと一緒に、それを実行機能を供給オフにはるかに優れていると言うでしょう。

ここでは、一度に1つの子ビューコントローラしか持たないと仮定します。私はdidSetを使用して、実際に間違っているとは思わない

var current : UIViewController? { 
    if self.childViewControllers.count > 0 { 
     return self.childViewControllers[0] 
    } 
    return nil 
} 

func removeChild() { 
    if let vc = self.current { 
     vc.willMove(toParentViewController: nil) 
     vc.view.removeFromSuperview() 
     vc.removeFromParentViewController() 
    } 
} 

func createChild(_ vc:UIViewController) { 
    self.removeChild() // There Can Be Only One 
    self.addChildViewController(vc) // * 
    // ... get vc's view into the interface ... 
    vc.didMove(toParentViewController: self) 
}