From 572e20471d81c812f5e78a8ef25864e710ddda13 Mon Sep 17 00:00:00 2001
From: Mijin Jeon <mayihelpu27@gmail.com>
Date: Mon, 14 May 2018 16:51:45 +0900
Subject: [PATCH] Refactor: Declare static constants or use enum value for
 setting frames of View #26

---
 CardGameApp/CardGameApp/View/CardDeckView.swift  |  6 +++---
 .../CardGameApp/View/CardStacksView.swift        | 16 +++++++++-------
 .../CardGameApp/View/FoundationView.swift        |  4 ++--
 CardGameApp/CardGameApp/ViewController.swift     | 11 +++++++----
 4 files changed, 21 insertions(+), 16 deletions(-)

diff --git a/CardGameApp/CardGameApp/View/CardDeckView.swift b/CardGameApp/CardGameApp/View/CardDeckView.swift
index d28c74b8..ff6fbd69 100644
--- a/CardGameApp/CardGameApp/View/CardDeckView.swift
+++ b/CardGameApp/CardGameApp/View/CardDeckView.swift
@@ -23,11 +23,11 @@ class CardDeckView: UIView {
 
     required init?(coder aDecoder: NSCoder) {
         super.init(coder: aDecoder)
-        self.frame = CGRect(x: 0, y: 0, width: 414, height: 100)
+        self.frame = CGRect(x: 0, y: 0, width: ViewController.widthOfRootView, height: PositionY.bottom.value)
     }
 
     convenience init() {
-        self.init(frame: CGRect(x: 0, y: 0, width: 414, height: 100))
+        self.init(frame: CGRect(x: 0, y: 0, width: ViewController.widthOfRootView, height: PositionY.bottom.value))
         self.deckManager = gameManager.getDeckDelegate()
     }
 
@@ -83,7 +83,7 @@ class CardDeckView: UIView {
     @objc func cardDoubleTapped(sender: UITapGestureRecognizer) {
         if sender.state == .ended {
             let deckview = sender.view?.superview as! CardDeckView
-            NotificationCenter.default.post(name: .doubleTappedOpenedDeck, object: self, userInfo: ["from": deckview])
+            NotificationCenter.default.post(name: .doubleTappedOpenedDeck, object: self, userInfo: [ViewController.fromViewKey: deckview])
         }
     }
 
diff --git a/CardGameApp/CardGameApp/View/CardStacksView.swift b/CardGameApp/CardGameApp/View/CardStacksView.swift
index 26327534..48ed60de 100644
--- a/CardGameApp/CardGameApp/View/CardStacksView.swift
+++ b/CardGameApp/CardGameApp/View/CardStacksView.swift
@@ -9,6 +9,7 @@
 import UIKit
 
 class CardStacksView: UIView {
+    static let numberOfStacks: Int = 7
     private var wholeStackManager: (CardStackDelegate & Stackable)!
     private var oneStackViews = [OneStack]()
 
@@ -19,7 +20,7 @@ class CardStacksView: UIView {
 
     convenience init() {
         self.init(frame: CGRect(x: 0, y: PositionY.bottom.value,
-                                width: 414, height: 736 - PositionY.bottom.value))
+                                width: ViewController.widthOfRootView, height: ViewController.heightOfRootView - PositionY.bottom.value))
         self.wholeStackManager = CardGameManager.shared().getWholeStackDelegate()
     }
 
@@ -28,7 +29,7 @@ class CardStacksView: UIView {
     }
 
     func setup() {
-        for i in 0...6 {
+        for i in 0..<CardStacksView.numberOfStacks {
             oneStackViews.append(OneStack(column: i, manager: wholeStackManager))
             addSubview(oneStackViews[i])
             oneStackViews[i].setup()
@@ -46,6 +47,7 @@ class CardStacksView: UIView {
 }
 
 class OneStack: UIView {
+
     private var column: Int!
     private var wholeStackManager: (CardStackDelegate & Stackable)!
     private var stackManager: StackDelegate!
@@ -61,8 +63,8 @@ class OneStack: UIView {
     convenience init(column: Int, manager: CardStackDelegate & Stackable) {
         self.init(frame: CGRect(x: PositionX.allValues[column].value,
                                 y: 0,
-                                width: 414 / 7,
-                                height: 736 - PositionY.bottom.value))
+                                width: ViewController.widthOfRootView / CGFloat(CardStacksView.numberOfStacks),
+                                height: ViewController.heightOfRootView - PositionY.bottom.value))
         self.column = column
         self.wholeStackManager = manager
         self.stackManager = wholeStackManager.getStackDelegate(of: column)
@@ -72,8 +74,8 @@ class OneStack: UIView {
         super.init(coder: aDecoder)
         self.frame = CGRect(x: PositionX.allValues[column].value,
                             y: 0,
-                            width: 414 / 7,
-                            height: 736 - PositionY.bottom.value)
+                            width: ViewController.widthOfRootView / CGFloat(CardStacksView.numberOfStacks),
+                            height: ViewController.heightOfRootView - PositionY.bottom.value)
     }
 
     func setup() {
@@ -104,7 +106,7 @@ class OneStack: UIView {
     @objc func cardDoubleTapped(sender: UITapGestureRecognizer) {
         if sender.state == .ended {
             let oneStackView = sender.view?.superview as! OneStack
-            NotificationCenter.default.post(name: .doubleTappedStack, object: self, userInfo: ["from": oneStackView])
+            NotificationCenter.default.post(name: .doubleTappedStack, object: self, userInfo: [ViewController.fromViewKey: oneStackView])
         }
     }
 
diff --git a/CardGameApp/CardGameApp/View/FoundationView.swift b/CardGameApp/CardGameApp/View/FoundationView.swift
index 6931c3f2..74907d68 100644
--- a/CardGameApp/CardGameApp/View/FoundationView.swift
+++ b/CardGameApp/CardGameApp/View/FoundationView.swift
@@ -19,11 +19,11 @@ class FoundationView: UIView {
 
     required init?(coder aDecoder: NSCoder) {
         super.init(coder: aDecoder)
-        self.frame = CGRect(x: 0, y: 0, width: 414, height: 100)
+        self.frame = CGRect(x: 0, y: 0, width: ViewController.widthOfRootView, height: PositionY.bottom.value)
     }
 
     convenience init() {
-        self.init(frame: CGRect(x: 0, y: 0, width: 414, height: 100))
+        self.init(frame: CGRect(x: 0, y: 0, width: ViewController.widthOfRootView, height: PositionY.bottom.value))
         self.foundationManager = gameManager.getFoundationDelegate()
     }
 
diff --git a/CardGameApp/CardGameApp/ViewController.swift b/CardGameApp/CardGameApp/ViewController.swift
index 93960444..398e51f5 100644
--- a/CardGameApp/CardGameApp/ViewController.swift
+++ b/CardGameApp/CardGameApp/ViewController.swift
@@ -15,11 +15,14 @@ class ViewController: UIViewController {
     private var stackView: CardStacksView!
     private var foundationView: FoundationView!
 
+    static let fromViewKey: String = "from"
+    static let widthOfRootView: CGFloat = 414
+    static let heightOfRootView: CGFloat = 736
     static let spaceY: CGFloat = 15.0
     static let widthDivider: CGFloat = 8
     static let cardHeightRatio: CGFloat = 1.27
-    static let cardSize = CGSize(width: 414 / ViewController.widthDivider,
-                                 height: (414 / ViewController.widthDivider) * ViewController.cardHeightRatio)
+    static let cardSize = CGSize(width: widthOfRootView / widthDivider,
+                                 height: (widthOfRootView / widthDivider) * cardHeightRatio)
 
     override var preferredStatusBarStyle: UIStatusBarStyle {
         return .lightContent
@@ -107,7 +110,7 @@ class ViewController: UIViewController {
     // from deck view to foundation || stack view
     @objc func deckViewDidDoubleTap(notification: Notification) {
         guard let userInfo = notification.userInfo else {return}
-        guard let from = userInfo["from"] else { return }
+        guard let from = userInfo[ViewController.fromViewKey] else { return }
         guard (from as? CardDeckView) != nil else { return }
         let targetCard = deckView.lastCardView!
 
@@ -148,7 +151,7 @@ class ViewController: UIViewController {
 
     @objc func stackViewDidDoubleTap(notification: Notification) {
         guard let userInfo = notification.userInfo else { return }
-        guard let from = userInfo["from"] else { return }
+        guard let from = userInfo[ViewController.fromViewKey] else { return }
         guard let fromView = from as? OneStack else { return }
         let fromIndex = fromView.getColumn()
         let targetCard = fromView.lastCardView!