fix:regex error in selection sort insertion sort and merge sort (#39230)
This commit is contained in:
@ -30,7 +30,7 @@ tests:
|
|||||||
- text: <code>insertionSort</code> should return an array that is unchanged except for order.
|
- text: <code>insertionSort</code> should return an array that is unchanged except for order.
|
||||||
testString: assert.sameMembers(insertionSort([1,4,2,8,345,123,43,32,5643,63,123,43,2,55,1,234,92]), [1,4,2,8,345,123,43,32,5643,63,123,43,2,55,1,234,92]);
|
testString: assert.sameMembers(insertionSort([1,4,2,8,345,123,43,32,5643,63,123,43,2,55,1,234,92]), [1,4,2,8,345,123,43,32,5643,63,123,43,2,55,1,234,92]);
|
||||||
- text: <code>insertionSort</code> should not use the built-in <code>.sort()</code> method.
|
- text: <code>insertionSort</code> should not use the built-in <code>.sort()</code> method.
|
||||||
testString: assert(!code.match(/\.?[\s\S]*?sort/));
|
testString: assert(!code.match(/\.\s*sort\s*\(/));
|
||||||
|
|
||||||
```
|
```
|
||||||
|
|
||||||
|
@ -34,7 +34,7 @@ tests:
|
|||||||
- text: <code>mergeSort</code> should return an array that is unchanged except for order.
|
- text: <code>mergeSort</code> should return an array that is unchanged except for order.
|
||||||
testString: assert.sameMembers(mergeSort([1,4,2,8,345,123,43,32,5643,63,123,43,2,55,1,234,92]), [1,4,2,8,345,123,43,32,5643,63,123,43,2,55,1,234,92]);
|
testString: assert.sameMembers(mergeSort([1,4,2,8,345,123,43,32,5643,63,123,43,2,55,1,234,92]), [1,4,2,8,345,123,43,32,5643,63,123,43,2,55,1,234,92]);
|
||||||
- text: <code>mergeSort</code> should not use the built-in <code>.sort()</code> method.
|
- text: <code>mergeSort</code> should not use the built-in <code>.sort()</code> method.
|
||||||
testString: assert(!removeJSComments(code).match(/\.?[\s\S]*?sort\s*\(/));
|
testString: assert(!removeJSComments(code).match(/\.\s*sort\s*\(/));
|
||||||
|
|
||||||
```
|
```
|
||||||
|
|
||||||
|
@ -30,7 +30,7 @@ tests:
|
|||||||
- text: <code>selectionSort</code> should return an array that is unchanged except for order.
|
- text: <code>selectionSort</code> should return an array that is unchanged except for order.
|
||||||
testString: assert.sameMembers(selectionSort([1,4,2,8,345,123,43,32,5643,63,123,43,2,55,1,234,92]), [1,4,2,8,345,123,43,32,5643,63,123,43,2,55,1,234,92]);
|
testString: assert.sameMembers(selectionSort([1,4,2,8,345,123,43,32,5643,63,123,43,2,55,1,234,92]), [1,4,2,8,345,123,43,32,5643,63,123,43,2,55,1,234,92]);
|
||||||
- text: <code>selectionSort</code> should not use the built-in <code>.sort()</code> method.
|
- text: <code>selectionSort</code> should not use the built-in <code>.sort()</code> method.
|
||||||
testString: assert(!code.match(/\.?[\s\S]*?sort/));
|
testString: assert(!code.match(/\.\s*sort\s*\(/));
|
||||||
|
|
||||||
```
|
```
|
||||||
|
|
||||||
@ -76,17 +76,17 @@ function isSorted(arr) {
|
|||||||
function selectionSort(array) {
|
function selectionSort(array) {
|
||||||
for (let i = 0; i < array.length-1; i++) {
|
for (let i = 0; i < array.length-1; i++) {
|
||||||
let minimumIndex = i;
|
let minimumIndex = i;
|
||||||
for (let j = i+1; j < array.length; j++){
|
for (let j = i+1; j < array.length; j++){
|
||||||
if (array[j] < array[minimumIndex]) {
|
if (array[j] < array[minimumIndex]) {
|
||||||
minimumIndex = j;
|
minimumIndex = j;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
let value = array[minimumIndex];
|
let value = array[minimumIndex];
|
||||||
array[minimumIndex] = array[i];
|
array[minimumIndex] = array[i];
|
||||||
array[i] = value;
|
array[i] = value;
|
||||||
}
|
}
|
||||||
return array;
|
return array;
|
||||||
}
|
}
|
||||||
```
|
```
|
||||||
|
|
||||||
</section>
|
</section>
|
||||||
|
Reference in New Issue
Block a user