Powtarzająca się wartość - funkcja

Question

Stwórz funkcję indexOfRepeatedValue(array). Prześlij do niej tablicę z 10 liczbami które są zapisane w zmiennej numbers.

Stwórz w tej funkcji zmienną firstIndex. W pętli for sprawdź, która z liczb powtarza się jako pierwsza i przypisz jej indeks do zmiennej firstIndex. Następnie wypisz w konsoli tą zmienną – poza pętlą for. Zwróć wartość firstIndex z funkcji. const numbers = [2, 4, 5, 2, 3, 5, 1, 2, 4];

mój kod nie działa bo totalnie nie wiem jak wypisać to w funkcji:

return firstIndex;
console.log(firstIndex);

a to cały mój kod:

function indexOfRepeatedValue(array) {
    let firstIndex = "";
    for (let i = 0; i < numbers.length; i++) {
        for (let j = i + 1; j < numbers.length; j++) {
            if (numbers[i] === numbers[j]) {

                if (firstIndex === "") {

                    firstIndex = array.indexOf(numbers[i]);
                    break;
                }
            }
        }

        if (firstIndex !== "") {

            break;
        }
    }
    return firstIndex;
    console.log(firstIndex);
}
indexOfRepeatedValue(numbers);

pytanie nowicjusza wiec proszę o wyrozumiałość 

Comments

Na przyszłość proszę umieszczać kod w odpowiednich bloczkach, zamiast jako cytat.

(zdaje się, że wystąpił tu błąd z formatowaniem bloczka z kodem, który już wcześniej się pojawiał)

Answer 1

Parametr który przyjmujesz w funkcji nazywasz array ale później w kodzie operujesz na zmiennej numbers którą pewnie masz zadeklarowaną wcześniej jako zmienną globalną. Poprawiłem trochę Twój kod żeby działał poprawnie. Zamieniłem również zmienną firstIndex aby początkowo miała wartość -1. To taka dobra praktyka żeby nie zmieniać typu zmiennej (u Ciebie początkowo był typ string a w późniejszym etapie int).

function indexOfRepeatedValue(array) {
    let firstIndex = -1;
    for (let i = 0; i < array.length; i++) {
        for (let j = i + 1; j < array.length; j++) {
            if (array[i] === array[j]) {
                firstIndex = array.indexOf(array[i]);
                break;
            }
        }
        if(firstIndex > -1)
            break;
    }
    
    return firstIndex;
}

Kod możesz wywołać w następujący sposób

const numbers = [2, 4, 5, 2, 3, 5, 1, 2, 4];
let foundedFirstIndex = indexOfRepeatedValue(numbers);
console.log(foundedFirstIndex);

 

Comments

dziękuję za odpowiedź! bardzo mi pomogła. I masz rację z góry miałam narzuconą zmienną numbers z tabelą, dlatego próbowałam wcisnąć ją w pętle. Twoje rozwiązanie jest dużo lepsze

Answer 2

juz wiem... 

    console.log(firstIndex);
return firstIndex;