update

Author: saadshams

Sources/PureMVC/core/Controller.swift

@@ -110,8 +110,8 @@ open class Controller: IController {
             if instanceMap[key] == nil {
                 instanceMap[key] = factory(key)
             }
+            return instanceMap[key]
         }
-        return instanceMap[key]
     }
 
     /**

Sources/PureMVC/core/Model.swift

@@ -93,8 +93,8 @@ open class Model: IModel {
             if instanceMap[key] == nil {
                 instanceMap[key] = factory(key)
             }
+            return instanceMap[key]
         }
-        return instanceMap[key]
     }
 
     /**

Sources/PureMVC/core/View.swift

@@ -98,8 +98,8 @@ open class View: IView {
             if instanceMap[key] == nil {
                 instanceMap[key] = factory(key)
             }
+            return instanceMap[key]
         }
-        return instanceMap[key]
     }
 
     /**
@@ -192,7 +192,7 @@ open class View: IView {
             return false
         }
 
-        if exists { return }
+        guard !exists else { return }
 
         mediator.initializeNotifier(multitonKey)
 

Sources/PureMVC/patterns/facade/Facade.swift

@@ -85,8 +85,8 @@ open class Facade: IFacade {
             if instanceMap[key] == nil {
                 instanceMap[key] = factory(key)
             }
+            return instanceMap[key]
         }
-        return instanceMap[key]
     }
 
     /**
@@ -312,11 +312,9 @@ open class Facade: IFacade {
     - returns: whether a Core is registered with the given `key`.
     */
     open class func hasCore(_ key: String) -> Bool {
-        var result = false
         instanceMapQueue.sync {
-            result = instanceMap[key] != nil
+            instanceMap[key] != nil
         }
-        return result
     }
 
     /**