Merge pull request kodecocodes#934 from JMatharu/master

Updating Counting Sort for stability

Author: richard-ash

Counting Sort/CountingSort.playground/Contents.swift

@@ -29,12 +29,17 @@ func countingSort(array: [Int]) throws -> [Int] {
 
   // Step 3
   // Place the element in the final array as per the number of elements before it
+  // Loop through the array in reverse to keep the stability of the new array
+  // i.e. 7, is at index 3 and 6, thus in sortedArray the position of 7 at index 3 should be before 7 at index 6
   var sortedArray = [Int](repeating: 0, count: array.count)
-  for element in array {
+  for index in stride(from: array.count - 1, through: 0, by: -1) {
+    let element = array[index]
     countArray[element] -= 1
     sortedArray[countArray[element]] = element
   }
+    
   return sortedArray
 }
 
 try countingSort(array: [10, 9, 8, 7, 1, 2, 7, 3])
+

Counting Sort/CountingSort.playground/playground.xcworkspace/xcshareddata/IDEWorkspaceChecks.plist

@@ -0,0 +1,8 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
+<plist version="1.0">
+<dict>
+	<key>IDEDidComputeMac32BitWarning</key>
+	<true/>
+</dict>
+</plist>

Counting Sort/CountingSort.swift

@@ -29,8 +29,11 @@ func countingSort(_ array: [Int])-> [Int] {
 
   // Step 3
   // Place the element in the final array as per the number of elements before it
+  // Loop through the array in reverse to keep the stability of the new sorted array
+  // (For Example: 7 is at index 3 and 6, thus in sortedArray the position of 7 at index 3 should be before 7 at index 6
   var sortedArray = [Int](repeating: 0, count: array.count)
-  for element in array {
+  for index in stride(from: array.count - 1, through: 0, by: -1) {
+    let element = array[index]
     countArray[element] -= 1
     sortedArray[countArray[element]] = element
   }

Counting Sort/README.markdown

@@ -50,7 +50,9 @@ The code for step 2 is:
 
 ### Step 3:
 
-This is the last step in the algorithm. Each element in the original array is placed at the position defined by the output of step 2. For example, the number 10 would be placed at an index of 7 in the output array. Also, as you place the elements you need to reduce the count by 1 as those many elements are reduced from the array.
+This is the last step in the algorithm. Each element in the original array is placed at the position defined by the output of step 2. For example, the number 10 would be placed at an index of 7 in the output array. Also, as you place the elements you need to reduce the count by 1 as those many elements are reduced from the array. 
+We also have to loop through the array in reverse to keep the stability of the new sorted array.
+For Example: 7 is at index 3 and 6, thus in sortedArray the position of 7 at index 3 should be before 7 at index 6.
 
 The final output would be:
 
@@ -63,7 +65,8 @@ Here is the code for this final step:
 
 ```swift
   var sortedArray = [Int](repeating: 0, count: array.count)
-  for element in array {
+  for index in stride(from: array.count - 1, through: 0, by: -1) {
+    let element = array[index]
     countArray[element] -= 1
     sortedArray[countArray[element]] = element
   }

Counting Sort/Tests/Tests.xcodeproj/project.pbxproj

@@ -97,6 +97,7 @@
 			developmentRegion = English;
 			hasScannedForEncodings = 0;
 			knownRegions = (
+				English,
 				en,
 				Base,
 			);
@@ -226,7 +227,7 @@
 				LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/../Frameworks @loader_path/../Frameworks";
 				PRODUCT_BUNDLE_IDENTIFIER = swift.algorithm.club.Tests;
 				PRODUCT_NAME = "$(TARGET_NAME)";
-				SWIFT_VERSION = 3.0;
+				SWIFT_VERSION = 5.0;
 			};
 			name = Debug;
 		};
@@ -238,7 +239,7 @@
 				LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/../Frameworks @loader_path/../Frameworks";
 				PRODUCT_BUNDLE_IDENTIFIER = swift.algorithm.club.Tests;
 				PRODUCT_NAME = "$(TARGET_NAME)";
-				SWIFT_VERSION = 3.0;
+				SWIFT_VERSION = 5.0;
 			};
 			name = Release;
 		};

Counting Sort/Tests/Tests.xcodeproj/project.xcworkspace/xcshareddata/IDEWorkspaceChecks.plist

@@ -0,0 +1,8 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
+<plist version="1.0">
+<dict>
+	<key>IDEDidComputeMac32BitWarning</key>
+	<true/>
+</dict>
+</plist>