Merge branch 'master' into master

Author: YeralYamil

.github/CONTRIBUTING.md

@@ -0,0 +1,57 @@
+# Contributing Guidelines
+
+Want to help out with the Swift Algorithm Club? Great! While we don't have strict templates on the format of each contribution, we do have a few guidelines that should be kept in mind:
+
+**Readability**
+
+The `README` file is the cake, and the sample code is the cherry on top. Readablity is really important for us. A good contribution has succinct explanations supported by diagrams. Code is best introduced in chunks, weaved into the explanations where relevant. 
+
+**API Design Guidelines**
+
+A good contribution abides to the [Swift API Guidelines](https://swift.org/documentation/api-design-guidelines/). We review the pull requests with this in mind.
+
+**Swift Language Guidelines**
+
+We follow the following Swift [style guide](https://github.com/raywenderlich/swift-style-guide). 
+
+## Contribution Categories
+
+### Refinement
+
+Unit tests. Fixes for typos. No contribution is too small. :-)
+
+The repository has over 100 different data structures and algorithms. We're always interested in improvements to existing implementations and better explanations. Suggestions for making the code more Swift-like or to make it fit better with the standard library is most welcomed. 
+
+### New Contributions
+
+Before writing about something new, you should do 2 things:
+
+1. Check the main page for existing implementations
+2. Check the [pull requests](https://github.com/raywenderlich/swift-algorithm-club/pulls) for "claimed" topics. More info on that below. 
+
+If what you have in mind is a new addition, please follow this process when submitting your contribution:
+
+1. Create a pull request to "claim" an algorithm or data structure. This is to avoid having multiple people working on the same thing.
+2. Use this [style guide](https://github.com/raywenderlich/swift-style-guide) for writing code (more or less).
+3. Write an explanation of how the algorithm works. Include **plenty of examples** for readers to follow along. Pictures are good. Take a look at [the explanation of quicksort](../Quicksort/) to get an idea.
+4. Include your name in the explanation, something like *Written by Your Name* at the end of the document. 
+5. Add a playground and/or unit tests.
+
+For the unit tests:
+
+- Add the unit test project to `.travis.yml` so they will be run on [Travis-CI](https://travis-ci.org/raywenderlich/swift-algorithm-club). Add a line to `.travis.yml` like this:
+
+```
+- xctool test -project ./Algorithm/Tests/Tests.xcodeproj -scheme Tests
+```
+
+- Configure the Test project's scheme to run on Travis-CI:
+    - Open **Product -> Scheme -> Manage Schemes...**
+    - Uncheck **Autocreate schemes**
+    - Check **Shared**
+
+![Screenshot of scheme settings](../Images/scheme-settings-for-travis.png)
+
+## Want to chat?
+
+This isn't just a repo with a bunch of code... If you want to learn more about how an algorithm works or want to discuss better ways of solving problems, then open a [Github issue](https://github.com/raywenderlich/swift-algorithm-club/issues) and we'll talk!

.github/ISSUE_TEMPLATE.md

@@ -0,0 +1,13 @@
+### Category
+
+- [ ] Bug
+- [ ] Feature Request
+- [ ] Question
+
+### Brief Intro
+
+<!-- Describe the issue briefly.-->
+
+### More Details
+
+<!-- If necessary, start off with the code snippet. Put the code in between ``` to format them into code blocks.-->

.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1,11 @@
+<!-- Thanks for contributing to the SAC! Before you submit your pull request, please make sure to check the following boxes by putting an x in the [ ] -->
+
+### Checklist
+
+- [ ] I've looked at the [contribution guidelines](https://github.com/raywenderlich/swift-algorithm-club/blob/master/.github/CONTRIBUTING.md).
+- [ ] This pull request is complete and ready for review.
+
+### Description
+
+<!-- In a short paragraph, describe the PR --> 
+

.swiftlint.yml

@@ -17,8 +17,8 @@ disabled_rules:
 custom_rules:
   smiley_face:
     name: "Smiley Face"
-    regex: "(\:\))"
-    match_kinds: 
+    regex: '( :\))'
+    match_kinds:
       - comment
       - string
     message: "A closing parenthesis smiley :) creates a half-hearted smile, and thus is not preferred. Use :]"

.travis.yml

@@ -1,5 +1,5 @@
 language: objective-c
-osx_image: xcode8
+osx_image: xcode9
 # sudo: false
 
 install:
@@ -8,35 +8,40 @@ install:
 
 script:
 
-# - xcodebuild test -project ./All-Pairs\ Shortest\ Paths/APSP/APSP.xcodeproj -scheme APSPTests
+- xcodebuild test -project ./All-Pairs\ Shortest\ Paths/APSP/APSP.xcodeproj -scheme APSPTests
 - xcodebuild test -project ./Array2D/Tests/Tests.xcodeproj -scheme Tests
 - xcodebuild test -project ./AVL\ Tree/Tests/Tests.xcodeproj -scheme Tests
 - xcodebuild test -project ./Binary\ Search/Tests/Tests.xcodeproj -scheme Tests
-# - xcodebuild test -project ./Binary\ Search\ Tree/Solution\ 1/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./Boyer-Moore/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./Binary\ Search\ Tree/Solution\ 1/Tests/Tests.xcodeproj -scheme Tests
 - xcodebuild test -project ./Bloom\ Filter/Tests/Tests.xcodeproj -scheme Tests
-# - xcodebuild test -project ./Bounded\ Priority\ Queue/Tests/Tests.xcodeproj -scheme Tests
-# - xcodebuild test -project ./Breadth-First\ Search/Tests/Tests.xcodeproj -scheme Tests
-# - xcodebuild test -project ./Bucket\ Sort/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./Bounded\ Priority\ Queue/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./Breadth-First\ Search/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./Bucket\ Sort/Tests/Tests.xcodeproj -scheme Tests
 - xcodebuild test -project ./B-Tree/Tests/Tests.xcodeproj -scheme Tests
-# - xcodebuild test -project ./Counting\ Sort/Tests/Tests.xcodeproj -scheme Tests
-# - xcodebuild test -project ./Depth-First\ Search/Tests/Tests.xcodeproj -scheme Tests
-# - xcodebuild test -project ./Graph/Graph.xcodeproj -scheme GraphTests
-# - xcodebuild test -project ./Heap/Tests/Tests.xcodeproj -scheme Tests
-# - xcodebuild test -project ./Heap\ Sort/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./Comb\ Sort/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./Counting\ Sort/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./Depth-First\ Search/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./Graph/Graph.xcodeproj -scheme GraphTests
+- xcodebuild test -project ./Heap/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./Heap\ Sort/Tests/Tests.xcodeproj -scheme Tests
 - xcodebuild test -project ./Insertion\ Sort/Tests/Tests.xcodeproj -scheme Tests
-# - xcodebuild test -project ./K-Means/Tests/Tests.xcodeproj -scheme Tests
-# - xcodebuild test -project ./Linked\ List/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./K-Means/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./Linked\ List/Tests/Tests.xcodeproj -scheme Tests
 - xcodebuild test -project ./Longest\ Common\ Subsequence/Tests/Tests.xcodeproj -scheme Tests
-# - xcodebuild test -project ./Minimum\ Spanning\ Tree\ \(Unweighted\)/Tests/Tests.xcodeproj -scheme Tests
-# - xcodebuild test -project ./Priority\ Queue/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./Minimum\ Spanning\ Tree\ \(Unweighted\)/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./Priority\ Queue/Tests/Tests.xcodeproj -scheme Tests
 - xcodebuild test -project ./Queue/Tests/Tests.xcodeproj -scheme Tests
-# - xcodebuild test -project ./Quicksort/Tests/Tests.xcodeproj -scheme Tests
-# - xcodebuild test -project ./Run-Length\ Encoding/Tests/Tests.xcodeproj -scheme Tests
-# - xcodebuild test -project ./Select\ Minimum\ Maximum/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./Quicksort/Tests/Tests-Quicksort.xcodeproj -scheme Tests-Quicksort
+- xcodebuild test -project ./Radix\ Sort/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./Rootish\ Array\ Stack/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./Select\ Minimum\ Maximum/Tests/Tests.xcodeproj -scheme Tests
 - xcodebuild test -project ./Selection\ Sort/Tests/Tests.xcodeproj -scheme Tests
-# - xcodebuild test -project ./Shell\ Sort/Tests/Tests.xcodeproj -scheme Tests
-# - xcodebuild test -project ./Shortest\ Path\ \(Unweighted\)/Tests/Tests.xcodeproj -scheme Tests
-# - xcodebuild test -project ./Single-Source\ Shortest\ Paths\ \(Weighted\)/SSSP.xcodeproj -scheme SSSPTests
+- xcodebuild test -project ./Shell\ Sort/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./Shortest\ Path\ \(Unweighted\)/Tests/Tests.xcodeproj -scheme Tests
+- xcodebuild test -project ./Single-Source\ Shortest\ Paths\ \(Weighted\)/SSSP.xcodeproj -scheme SSSPTests
 - xcodebuild test -project ./Stack/Tests/Tests.xcodeproj -scheme Tests
 - xcodebuild test -project ./Topological\ Sort/Tests/Tests.xcodeproj -scheme Tests
 - xcodebuild test -project ./Treap/Treap/Treap.xcodeproj -scheme Tests
+- xcodebuild test -project ./Palindromes/Test/Test.xcodeproj -scheme Test
+- xcodebuild test -project ./Ternary\ Search\ Tree/Tests/Tests.xcodeproj -scheme Tests

AVL Tree/AVLTree.playground/Contents.swift

@@ -1,5 +1,10 @@
 //: Playground - noun: a place where people can play
 
+// last checked with Xcode 9.0b4
+#if swift(>=4.0)
+print("Hello, Swift 4!")
+#endif
+
 let tree = AVLTree<Int, String>()
 
 tree.insert(key: 5, payload: "five")

AVL Tree/AVLTree.playground/Sources/AVLTree.swift

@@ -99,17 +99,11 @@ open class AVLTree<Key: Comparable, Payload> {
 
 extension TreeNode {
   public func minimum() -> TreeNode? {
-    if let leftChild = self.leftChild {
-      return leftChild.minimum()
-    }
-    return self
+    return leftChild?.minimum() ?? self
   }
 
   public func maximum() -> TreeNode? {
-    if let rightChild = self.rightChild {
-      return rightChild.maximum()
-    }
-    return self
+    return rightChild?.maximum() ?? self
   }
 }
 
@@ -120,11 +114,7 @@ extension AVLTree {
   }
 
   public func search(input: Key) -> Payload? {
-    if let result = search(key: input, node: root) {
-      return result.payload
-    } else {
-      return nil
-    }
+    return search(key: input, node: root)?.payload
   }
 
   fileprivate func search(key: Key, node: Node?) -> Node? {
@@ -351,11 +341,11 @@ extension AVLTree {
       }
     } else {
       // Handle stem cases
-      if let replacement = node.leftChild?.maximum() , replacement !== node {
+      if let replacement = node.leftChild?.maximum(), replacement !== node {
         node.key = replacement.key
         node.payload = replacement.payload
         delete(node: replacement)
-      } else if let replacement = node.rightChild?.minimum() , replacement !== node {
+      } else if let replacement = node.rightChild?.minimum(), replacement !== node {
         node.key = replacement.key
         node.payload = replacement.payload
         delete(node: replacement)
@@ -364,7 +354,6 @@ extension AVLTree {
   }
 }
 
-
 // MARK: - Debugging
 
 extension TreeNode: CustomDebugStringConvertible {
@@ -385,11 +374,7 @@ extension TreeNode: CustomDebugStringConvertible {
 
 extension AVLTree: CustomDebugStringConvertible {
   public var debugDescription: String {
-    if let root = root {
-      return root.debugDescription
-    } else {
-      return "[]"
-    }
+    return root?.debugDescription ?? "[]"
   }
 }
 
@@ -409,10 +394,6 @@ extension TreeNode: CustomStringConvertible {
 
 extension AVLTree: CustomStringConvertible {
   public var description: String {
-    if let root = root {
-      return root.description
-    } else {
-      return "[]"
-    }
+    return root?.description ?? "[]"
   }
 }

AVL Tree/AVLTree.swift

@@ -29,7 +29,7 @@ public class TreeNode<Key: Comparable, Payload> {
   internal var leftChild: Node?
   internal var rightChild: Node?
   fileprivate var height: Int
-  weak fileprivate var parent: Node?
+  fileprivate weak var parent: Node?
 
   public init(key: Key, payload: Payload?, leftChild: Node?, rightChild: Node?, parent: Node?, height: Int) {
     self.key = key
@@ -99,17 +99,11 @@ open class AVLTree<Key: Comparable, Payload> {
 
 extension TreeNode {
   public func minimum() -> TreeNode? {
-    if let leftChild = self.leftChild {
-      return leftChild.minimum()
-    }
-    return self
+    return leftChild?.minimum() ?? self
   }
 
   public func maximum() -> TreeNode? {
-    if let rightChild = self.rightChild {
-      return rightChild.maximum()
-    }
-    return self
+    return rightChild?.maximum() ?? self
   }
 }
 
@@ -120,11 +114,7 @@ extension AVLTree {
   }
 
   public func search(input: Key) -> Payload? {
-    if let result = search(key: input, node: root) {
-      return result.payload
-    } else {
-      return nil
-    }
+    return search(key: input, node: root)?.payload
   }
 
   fileprivate func search(key: Key, node: Node?) -> Node? {
@@ -351,11 +341,11 @@ extension AVLTree {
       }
     } else {
       // Handle stem cases
-      if let replacement = node.leftChild?.maximum() , replacement !== node {
+      if let replacement = node.leftChild?.maximum(), replacement !== node {
         node.key = replacement.key
         node.payload = replacement.payload
         delete(node: replacement)
-      } else if let replacement = node.rightChild?.minimum() , replacement !== node {
+      } else if let replacement = node.rightChild?.minimum(), replacement !== node {
         node.key = replacement.key
         node.payload = replacement.payload
         delete(node: replacement)
@@ -364,7 +354,6 @@ extension AVLTree {
   }
 }
 
-
 // MARK: - Debugging
 
 extension TreeNode: CustomDebugStringConvertible {
@@ -385,11 +374,7 @@ extension TreeNode: CustomDebugStringConvertible {
 
 extension AVLTree: CustomDebugStringConvertible {
   public var debugDescription: String {
-    if let root = root {
-      return root.debugDescription
-    } else {
-      return "[]"
-    }
+    return root?.debugDescription ?? "[]"
   }
 }
 
@@ -409,10 +394,6 @@ extension TreeNode: CustomStringConvertible {
 
 extension AVLTree: CustomStringConvertible {
   public var description: String {
-    if let root = root {
-      return root.description
-    } else {
-      return "[]"
-    }
+    return root?.description ?? "[]"
   }
 }