From 2a7c18f36ce4c2e4300632a25b5d88bad3e08ca1 Mon Sep 17 00:00:00 2001
From: CoderXpert <adnan.iiui@gmail.com>
Date: Sat, 19 Nov 2016 22:20:46 +0000
Subject: [PATCH] Refactor to avoid nesting by using guard + by removing some
 unnecessary `{}`

---
 ObjectiveKit/ObjectiveClass.swift | 88 +++++++++++++------------------
 ObjectiveKit/RuntimeClass.swift   | 19 ++++---
 2 files changed, 47 insertions(+), 60 deletions(-)

diff --git a/ObjectiveKit/ObjectiveClass.swift b/ObjectiveKit/ObjectiveClass.swift
index a9cd787..a9fedeb 100644
--- a/ObjectiveKit/ObjectiveClass.swift
+++ b/ObjectiveKit/ObjectiveClass.swift
@@ -28,20 +28,17 @@ public class ObjectiveClass <T: NSObject>: ObjectiveKitRuntimeModification {
     ///
     /// - Returns: An array of instance variables.
     public var ivars: [String] {
-        get {
-            var count: CUnsignedInt = 0
-            var ivars = [String]()
-            let ivarList = class_copyIvarList(internalClass, &count)
-            for i in (0..<Int(count)) {
-                let unwrapped  = ivarList?[i].unsafelyUnwrapped
-                if let ivar = ivar_getName(unwrapped) {
-                    let string = String(cString: ivar)
-                    ivars.append(string)
-                }
-            }
-            free(ivarList)
-            return ivars
+        var count: CUnsignedInt = 0
+        var ivars = [String]()
+        let ivarList = class_copyIvarList(internalClass, &count)
+        for i in 0..<Int(count) {
+            let unwrapped  = ivarList?[i].unsafelyUnwrapped
+            guard let ivar = ivar_getName(unwrapped) else { continue }
+            let string = String(cString: ivar)
+            ivars.append(string)
         }
+        free(ivarList)
+        return ivars
     }
 
 
@@ -49,58 +46,49 @@ public class ObjectiveClass <T: NSObject>: ObjectiveKitRuntimeModification {
     ///
     /// - Returns: An array of selectors.
     public var selectors: [Selector]  {
-        get {
-            var count: CUnsignedInt = 0
-            var selectors = [Selector]()
-            let methodList = class_copyMethodList(internalClass, &count)
-            for i in (0..<Int(count)) {
-                let unwrapped  = methodList?[i].unsafelyUnwrapped
-                if let selector = method_getName(unwrapped) {
-                    selectors.append(selector)
-                }
-            }
-            free(methodList)
-            return selectors
+        var count: CUnsignedInt = 0
+        var selectors = [Selector]()
+        let methodList = class_copyMethodList(internalClass, &count)
+        for i in 0..<Int(count) {
+            let unwrapped  = methodList?[i].unsafelyUnwrapped
+            guard let selector = method_getName(unwrapped) else { continue }
+            selectors.append(selector)
         }
+        free(methodList)
+        return selectors
     }
 
     /// Get all protocols implemented by the class.
     ///
     /// - Returns: An array of protocol names.
     public var protocols: [String] {
-        get {
-            var count: CUnsignedInt = 0
-            var protocols = [String]()
-            let protocolList = class_copyProtocolList(internalClass, &count)
-            for i in (0..<Int(count)) {
-                let unwrapped  = protocolList?[i].unsafelyUnwrapped
-                if let protocolName = protocol_getName(unwrapped) {
-                    let string = String(cString: protocolName)
-                    protocols.append(string)
-                }
-            }
-            return protocols
+        var count: CUnsignedInt = 0
+        var protocols = [String]()
+        let protocolList = class_copyProtocolList(internalClass, &count)
+        for i in 0..<Int(count) {
+            let unwrapped  = protocolList?[i].unsafelyUnwrapped
+            guard let protocolName = protocol_getName(unwrapped) else { continue }
+            let string = String(cString: protocolName)
+            protocols.append(string)
         }
+        return protocols
     }
 
     /// Get all properties implemented by the class.
     ///
     /// - Returns: An array of property names.
     public var properties: [String] {
-        get {
-            var count: CUnsignedInt = 0
-            var properties = [String]()
-            let propertyList = class_copyPropertyList(internalClass, &count)
-            for i in (0..<Int(count)) {
-                let unwrapped  = propertyList?[i].unsafelyUnwrapped
-                if let propretyName = property_getName(unwrapped) {
-                    let string = String(cString: propretyName)
-                    properties.append(string)
-                }
-            }
-            free(propertyList)
-            return properties
+        var count: CUnsignedInt = 0
+        var properties = [String]()
+        let propertyList = class_copyPropertyList(internalClass, &count)
+        for i in 0..<Int(count) {
+            let unwrapped  = propertyList?[i].unsafelyUnwrapped
+            guard let propretyName = property_getName(unwrapped) else { continue }
+            let string = String(cString: propretyName)
+            properties.append(string)
         }
+        free(propertyList)
+        return properties
     }
 
 }
diff --git a/ObjectiveKit/RuntimeClass.swift b/ObjectiveKit/RuntimeClass.swift
index 13ab0c4..0f88af7 100644
--- a/ObjectiveKit/RuntimeClass.swift
+++ b/ObjectiveKit/RuntimeClass.swift
@@ -44,10 +44,9 @@ public class RuntimeClass: NSObject, ObjectiveKitRuntimeModification {
 
     /// Register class. Required before usage. Happens automatically on allocate.
     public func register() {
-        if registered == false {
-            registered =  true
-            objc_registerClassPair(internalClass)
-        }
+        guard registered == false else { return }
+        registered =  true
+        objc_registerClassPair(internalClass)
     }
 
     /// Allocate an instance of a new custom class at runtime.
@@ -79,12 +78,12 @@ public enum ObjectiveType: Int {
 
     func encoding() -> String {
         switch self {
-            case .NSString: return "@"
-            case .NSObject: return "@"
-            case .Float: return "f"
-            case .Int: return "i"
-            case .Double: return "d"
-            case .Void: return "v"
+        case .NSString: return "@"
+        case .NSObject: return "@"
+        case .Float: return "f"
+        case .Int: return "i"
+        case .Double: return "d"
+        case .Void: return "v"
         }
     }