Learn Basic Algorithmic Thinking by Building a Number Sorter - Step 27

Tell us what’s happening: it is daying decalre temp variable after nexted for loop , i did that but still it is giving me same problem again and again

Your code so far

const selectionSort = (array) => {
for (let i = 0; i < array.length; i++) {
let minIndex = i;

for (let j = i + 1; j < array.length; j++) {
  console.log(array, array[j], array[minIndex]);
  if (array[j] < array[minIndex]) {
    minIndex = j;
  }
}
let temp = array[i]
array[i] = array[minIndex]
array[minIndex] = temp

}
}

WARNING

The challenge seed code and/or your solution exceeded the maximum length we can port over from the challenge.

You will need to take an additional step here so the code you wrote presents in an easy to read format.

Please copy/paste all the editor code showing in the challenge from where you just linked.

Replace these two sentences with your copied code.
Please leave the ``` line above and the ``` line below,
because they allow your code to properly format in the post.

Your browser information:

User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/122.0.0.0 Safari/537.36

Challenge Information:

Learn Basic Algorithmic Thinking by Building a Number Sorter - Step 27

It appears that you are trying to implement a selection sort algorithm. The error you’re encountering might be due to the placement of the temp variable declaration.

for (let i = 0; i < array.length - 1; i++) {
let minIndex = i;
for (let j = i + 1; j < array.length; j++) {
console.log(array, array[j], array[minIndex]);
if (array[j] < array[minIndex]) {
minIndex = j;
}
}
let temp = array[i];
array[i] = array[minIndex];
array[minIndex] = temp;
}

  1. Moved the declaration of the temp variable inside the outer loop to ensure it’s declared after the nested loop.
  2. Added initialization of minIndex variable inside the outer loop. This ensures that minIndex starts fresh for each iteration of the outer loop.

i hope With these corrections, your selection sort implementation should work as expected. If you’re still encountering issues, please provide more details or error messages for further assistance.

Best regard
Danish Hafeez | QA Assistant
ICTInnovations

1 Like

Hi @rahulchourase223

Declare the temp variable using the const keyword.
The rest of your code looks fine.

Happy coding

1 Like

thank you @teller890 , it worked

1 Like

This topic was automatically closed 182 days after the last reply. New replies are no longer allowed.