Znaleźć w tablicy wszystkie kolejne liczby

Znaleźć w tablicy wszystkie kolejne liczby
P1
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 640
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

Kopiuj
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

NewUser2k13
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 217
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.

Kopiuj
 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...

Kopiuj
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.

lion137
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 5023
2

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

Kopiuj
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
P1
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 640
0

Ok. Dzięki za pomoc

LukeJL
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 8487
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:

Kopiuj
function pipeFix(numbers){
   let result = [];
   for (let i = numbers[0]; i <= numbers[numbers.length - 1]; i++) {
       result.push(i);
   }
   return result;
}
gk1982
  • Rejestracja: dni
  • Ostatnio: dni
  • Lokalizacja: Łódź
  • Postów: 541
0

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

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

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.