Znaleźć w tablicy wszystkie kolejne liczby

1

Zadanie które robie https://www.codewars.com/kata/56b29582461215098d00000f/train/javascript. Mam tablice z następującymi liczbami 1,3,5,6,7,8 i na wyjściu powiniennem dostać liczby 1,2,3,4,5,6,7,8,9. Napisałem kod

function pipeFix(numbers){
	let emptyArr = [];
	for(let i = 0; i < numbers.length; i++){
		if(numbers[i] < numbers[i] + 1){
			emptyArr.push(i+1);
		}
	}
	return emptyArr;
}
pipeFix([1,3,5,6,7,8])

i dostaje na wyjściu 1,2,3,4,5,6

1

To już Twój kolejny post z zadaniami z tej stronki. Nie bardzo rozumiem jaki jest sens ciągłego ich wklejania?
Kolejny raz Twój kod - bez urazy - jest bez sensu.
Dlaczego? Ano zobaczmy.

 for(let i = 0; i < numbers.length; i++){

Skąd takie warunki brzegowe? Przecież to nie ma żadnego uzasadnienia wynikającego z treści zadania...

if(numbers[i] < numbers[i] + 1){

Kolejny raz też w Twoim kodzie są jakieś dziwne warunki, które powiem szczerze nie bardzo wiem co mają sprawdzać. Wytłumaczysz?

Podpowiedź - żeby nie było offtopu.
Może warto iterować przez zakres min(numbers) - max(numbers) zamiast 0 - numbers.length ?

PS
Proszę - przedstaw mi Twój tok rozumowania, który towarzyszył Ci podczas pisania tego kodu. Chcę poznać co Tobą kierowało.

2

Czemu od zera, przecież tam są też ujemne liczby. Wystarczy wypełnić tablice liczbami w zakresie od pierwszego elementu do ostatniego.

function pipe_fix(nums):
    st = nums[0]
    end = nums[-1]  # the last element
    result = []
    for elem  = st to elem = end:
        result.append(elem)
    return result
0

Ok. Dzięki za pomoc

2

W treści zadania masz podpowiedź:
" you can assume every number in the sequence after the first index will be greater than the previous and that there will be no duplicates."

Czyli to znaczy, że możesz założyć, że pierwszy element tablicy jest najmniejszy, a ostatni największy. Czyli nie musisz w ogóle iterować po tablicy. Bierzesz pierwszą (numbers[0]) i ostatnią wartość z tablicy (numbers[numbers.length - 1]) , a resztę olewasz. Generujesz nową tablicę tylko na podstawie tych dwóch liczb:

function pipeFix(numbers){
   let result = [];
   for (let i = numbers[0]; i <= numbers[numbers.length - 1]; i++) {
       result.push(i);
   }
   return result;
}
0

Przymierzyłem się do tego zadania, wyszukałem i opracowałem takie działające rozwiązanie.

function pipeFix(numbers){
let result = Array(numbers[numbers.length - 1]-numbers[0]+1).fill().map((_, idx) => numbers[0] + idx)
   return result ;
}

1 użytkowników online, w tym zalogowanych: 0, gości: 1