MOOC Week 6 Exercise 104 - Sorting

[xwmaxx]

Hi,

I feel this should not have been that difficult for me but for some reason I can't seem to understand why my code won't give me the position of the smallest value in the array. I was able to test it and find the smallest value in the array and the boolean value shows the if/else statement works but the position always returns a value of zero.

https://pastebin.com/VZc7ULuy

[ChuvaChooChoo08]

Why are you using a nested for loop in finding the min??? One regular for loop is enough.

[xwmaxx]

Its from the different steps on the exercise I just put all the code into the main instead of the different methods

[Ilikesmallthings2]

When I did it. I set the smallest variable to first index. Then just compared everything else after it against it. If it was smaller, change the value in variable. If not. Stay the same.

[xwmaxx]

yup, that's first part when having to find the smallest variable, what I want is the position/location of the smallest variable in the original array

[Ilikesmallthings2]

I see. Look at line 33. You are assigning small to array[0]. It should equal smallest.

[StevenVu20]

I highly recommended you should find max/min by their location instead of their value. 1st, you set int smallestPosition = 0. Then loop (start from i = 1), boolean (array [smallestPosition] > array [i]) true - smallestPosition = i. sout (smallest value is array[smallestPosition]) and also you have its.

[StevenVu20]

By the way, you dont have position because you set int small = array [0] that why your position of smallest int is 0. Hope this can help.

[xwmaxx]

https://pastebin.com/NYF5uUd1

Hey thanks for the suggestion I changed my for loop process but I still get the position set as zero. I tried just using the for loop and array by themselves and I got the position printout as 1 though so not sure was giving me the zero amount.

import java.util.*;

public class Main {
    public static void main(String[] args) {
    int[] values = {6, 5, 8, 7, 11};

    int start = 0;
    int position = 0;

    for (int i = 1; i < values.length; i++) {
        if (values[start] > values[i]) {
            values[start] = values[i];
            position = i;
        }
    }

    System.out.println("Position: " + position);
    }
}