guard all the things! 👮

Stencil/Node.swift

@@ -118,27 +118,27 @@ public class ForNode : NodeType {
   public class func parse(parser:TokenParser, token:Token) throws -> NodeType {
     let components = token.components()
 
-    if components.count == 4 && components[2] == "in" {
+    guard components.count == 4 && components[2] == "in" else {
-      let loopVariable = components[1]
+      throw TemplateSyntaxError("'for' statements should use the following 'for x in y' `\(token.contents)`.")
-      let variable = components[3]
-
-      var emptyNodes = [NodeType]()
-
-      let forNodes = try parser.parse(until(["endfor", "empty"]))
-
-      if let token = parser.nextToken() {
-        if token.contents == "empty" {
-          emptyNodes = try parser.parse(until(["endfor"]))
-          parser.nextToken()
-        }
-      } else {
-        throw TemplateSyntaxError("`endfor` was not found.")
-      }
-
-      return ForNode(variable: variable, loopVariable: loopVariable, nodes: forNodes, emptyNodes:emptyNodes)
     }
     
-    throw TemplateSyntaxError("'for' statements should use the following 'for x in y' `\(token.contents)`.")
+    let loopVariable = components[1]
+    let variable = components[3]
+    
+    var emptyNodes = [NodeType]()
+    
+    let forNodes = try parser.parse(until(["endfor", "empty"]))
+    
+    guard let token = parser.nextToken() else {
+      throw TemplateSyntaxError("`endfor` was not found.")
+    }
+    
+    if token.contents == "empty" {
+      emptyNodes = try parser.parse(until(["endfor"]))
+      parser.nextToken()
+    }
+    
+    return ForNode(variable: variable, loopVariable: loopVariable, nodes: forNodes, emptyNodes:emptyNodes)
   }
 
   public init(variable:String, loopVariable:String, nodes:[NodeType], emptyNodes:[NodeType]) {
@@ -174,15 +174,15 @@ public class IfNode : NodeType {
 
     trueNodes = try parser.parse(until(["endif", "else"]))
 
-    if let token = parser.nextToken() {
+    guard let token = parser.nextToken() else {
-      if token.contents == "else" {
-        falseNodes = try parser.parse(until(["endif"]))
-        parser.nextToken()
-      }
-    } else {
       throw TemplateSyntaxError("`endif` was not found.")
     }
     
+    if token.contents == "else" {
+      falseNodes = try parser.parse(until(["endif"]))
+      parser.nextToken()
+    }
+
     return IfNode(variable: variable, trueNodes: trueNodes, falseNodes: falseNodes)
   }
 
@@ -193,15 +193,15 @@ public class IfNode : NodeType {
 
     falseNodes = try parser.parse(until(["endif", "else"]))
 
-    if let token = parser.nextToken() {
+    guard let token = parser.nextToken() else {
-      if token.contents == "else" {
-        trueNodes = try parser.parse(until(["endif"]))
-        parser.nextToken()
-      }
-    } else {
       throw TemplateSyntaxError("`endif` was not found.")
     }
     
+    if token.contents == "else" {
+      trueNodes = try parser.parse(until(["endif"]))
+      parser.nextToken()
+    }
+
     return IfNode(variable: variable, trueNodes: trueNodes, falseNodes: falseNodes)
   }
 

Stencil/Template.swift

@@ -9,11 +9,11 @@ public class Template {
   /// Create a template with the given name inside the given bundle
   public convenience init(named:String, inBundle bundle:NSBundle? = nil) throws {
     let useBundle = bundle ??  NSBundle.mainBundle()
-    if let url = useBundle.URLForResource(named, withExtension: nil) {
+    guard let url = useBundle.URLForResource(named, withExtension: nil) else {
-      try self.init(URL:url)
-    } else {
       throw NSError(domain: NSCocoaErrorDomain, code: NSFileNoSuchFileError, userInfo: nil)
     }
+    
+    try self.init(URL:url)
   }
 
   /// Create a template with a file found at the given URL

Stencil/TemplateLoader/Include.swift

@@ -8,7 +8,7 @@ public class IncludeNode : NodeType {
   public class func parse(parser:TokenParser, token:Token) throws -> NodeType {
     let bits = token.contents.componentsSeparatedByString("\"")
 
-    if bits.count != 3 {
+    guard bits.count == 3 else {
       throw TemplateSyntaxError("'include' tag takes one argument, the template file to be included")
     }
 
@@ -20,18 +20,16 @@ public class IncludeNode : NodeType {
   }
 
   public func render(context: Context) throws -> String {
-    if let loader =  context["loader"] as? TemplateLoader {
+    guard let loader = context["loader"] as? TemplateLoader else {
-      if let template = loader.loadTemplate(templateName) {
+      throw TemplateSyntaxError("Template loader not in context")
-        return try template.render(context)
+    }
-      }
 
-      let paths:String = loader.paths.map { path in
+    guard let template = loader.loadTemplate(templateName) else {
-        return path.description
+      let paths:String = loader.paths.map { $0.description }.joinWithSeparator(", ")
-      }.joinWithSeparator(", ")
       throw TemplateSyntaxError("'\(templateName)' template not found in \(paths)")
     }
     
-    throw TemplateSyntaxError("Template loader not in context")
+    return try template.render(context)
   }
 }
 

Stencil/TemplateLoader/Inheritence.swift

@@ -31,12 +31,12 @@ class ExtendsNode : NodeType {
   class func parse(parser:TokenParser, token:Token) throws -> NodeType {
     let bits = token.contents.componentsSeparatedByString("\"")
 
-    if bits.count != 3 {
+    guard bits.count == 3 else {
       throw TemplateSyntaxError("'extends' takes one argument, the template file to be extended")
     }
 
     let parsedNodes = try parser.parse()
-    if (any(parsedNodes) { ($0 as? ExtendsNode) != nil }) != nil {
+    guard (any(parsedNodes) { $0 is ExtendsNode }) == nil else {
       throw TemplateSyntaxError("'extends' cannot appear more than once in the same template")
     }
 
@@ -58,22 +58,20 @@ class ExtendsNode : NodeType {
   }
 
   func render(context: Context) throws -> String {
-    if let loader =  context["loader"] as? TemplateLoader {
+    guard let loader = context["loader"] as? TemplateLoader else {
-      if let template = loader.loadTemplate(templateName) {
+      throw TemplateSyntaxError("Template loader not in context")
-        let blockContext = BlockContext(blocks: blocks)
+    }
-        context.push([BlockContext.contextKey: blockContext])
-        let result = try template.render(context)
-        context.pop()
-        return result
-      }
     
-      let paths:String = loader.paths.map { path in
+    guard let template = loader.loadTemplate(templateName) else {
-        return path.description
+      let paths:String = loader.paths.map { $0.description }.joinWithSeparator(", ")
-        }.joinWithSeparator(", ")
       throw TemplateSyntaxError("'\(templateName)' template not found in \(paths)")
     }
 
-    throw TemplateSyntaxError("Template loader not in context")
+    let blockContext = BlockContext(blocks: blocks)
+    context.push([BlockContext.contextKey: blockContext])
+    let result = try template.render(context)
+    context.pop()
+    return result
   }
 }
 
@@ -84,7 +82,7 @@ class BlockNode : NodeType {
   class func parse(parser:TokenParser, token:Token) throws -> NodeType {
     let bits = token.components()
 
-    if bits.count != 2 {
+    guard bits.count == 2 else {
       throw TemplateSyntaxError("'block' tag takes one argument, the template file to be included")
     }