remove commented code

matthewoestreich · Aug 3, 2024 · 9ff9d38 · 9ff9d38
1 parent 4d91422
commit 9ff9d38
Showing 1 changed file with 2 additions and 34 deletions.
diff --git a/algos/quickSort.js b/algos/quickSort.js
@@ -7,8 +7,7 @@ function* partition(arr, start, end) {
 	// select the pivot
 	let pivot = Number(arr[mid].dataset.value);
 	// move pivot to end;
-	[arr[mid], arr[end]] = [arr[end], arr[mid]];
-	//yield new AlgoAnimation({ type: "swap", indexes: [mid, end] });
+	swap(arr, mid, end);
 	yield AlgoAnimation.swap(mid, end);
 
 	// set bounds
@@ -17,62 +16,32 @@ function* partition(arr, start, end) {
 
 	while (left <= right) {
 		yield AlgoAnimation.colors([{ index: left, value: BAR_COLORS.compare }, { index: right, value: BAR_COLORS.compare }])
-		//yield new AlgoAnimation({
-		//	type: "colors",
-		//	elements: [
-		//		{ index: left, value: BAR_COLORS.compare },
-		//		{ index: right, value: BAR_COLORS.compare },
-		//		//{ index: start, value: "orange" },
-		//		//{ index: end - 1, value: "orange" },
-		//	],
-		//});
 
 		// Once we have left value greater or equal to pivot AND a right value that is less than the pivot we swap those two values
 		// (or right bound crosses left bound, which would break the while condition)
 		if (Number(arr[left].dataset.value) >= pivot && Number(arr[right].dataset.value) < pivot) {
 			yield AlgoAnimation.colors([{ index: left, value: BAR_COLORS.incorrect }, { index: right, value: BAR_COLORS.incorrect }]);
 			yield AlgoAnimation.swap(left, right);
-			//yield new AlgoAnimation({
-			//	type: "colors",
-			//	elements: [
-			//		{ index: left, value: BAR_COLORS.incorrect },
-			//		{ index: right, value: BAR_COLORS.incorrect },
-			//	],
-			//});
-			//yield new AlgoAnimation({ type: "swap", indexes: [left, right] });
-			swap(arr, left, right)
-			//[arr[left], arr[right]] = [arr[right], arr[left]];
+			swap(arr, left, right);
 			continue;
 		}
 
 		if (Number(arr[left].dataset.value) < pivot) {
 			yield AlgoAnimation.color(left, BAR_COLORS.default);
-			//yield new AlgoAnimation({ type: "color", value: BAR_COLORS.default, index: left });
 			left++;
 		}
 		if (Number(arr[right].dataset.value) >= pivot) {
 			yield AlgoAnimation.color(right, BAR_COLORS.default);
-			//yield new AlgoAnimation({ type: "color", value: BAR_COLORS.default, index: right });
 			right--;
 		}
 
 		yield AlgoAnimation.colors([
 			{ index: left, value: BAR_COLORS.correct }, 
 			{ index: right, value: BAR_COLORS.correct }
 		]);
-		//yield new AlgoAnimation({
-		//	type: "colors",
-		//	elements: [
-		//		{ index: left, value: BAR_COLORS.correct },
-		//		{ index: right, value: BAR_COLORS.correct },
-		//	],
-		//});
 	}
 
 	// right bound has crossed left bound
-	//yield new AlgoAnimation({ type: "swap", indexes: [left, end] });
-	//yield new AlgoAnimation({ type: "color", index: end, value: BAR_COLORS.correct });
-	//[arr[left], arr[end]] = [arr[end], arr[left]];
 	yield AlgoAnimation.swap(left, end);
 	yield AlgoAnimation.color(end, BAR_COLORS.correct);
 	swap(arr, left, end);
@@ -89,7 +58,6 @@ function* sort(arr, start, end) {
 		if (start === 0 && end === arr.length - 1) {
 			for (let i = start; i <= end; i++) {
 				yield AlgoAnimation.color(i, BAR_COLORS.completed);
-				//yield new AlgoAnimation({ type: "color", value: BAR_COLORS.completed, index: i });
 			}
 		}
 	}