Znaleźć w tablicy wszystkie kolejne liczby

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

MA
To jest forma pytania? Jeżeli tak to zapomniałeś go zadać, jeżeli chcesz tylko się pochwalić to w sumie ok i fajnie, że piszesz kod :)
P1
To znaczy co robię ogólnie zastanawiam się co może być nie tak w tym kodzie?
NewUser2k13
  • Rejestracja:ponad 11 lat
  • Ostatnio:13 dni
  • Postów:210
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.

P1
No chce użyć pętli do przejście po wszystkich elementach tablicy a ifem sprawdzam każdy kolejny element w tablicy jest większy od poprzedniego i przekazuje to do nowej tablicy
NewUser2k13
A co jeśli tablica zaczyna się od -10? Co jeśli tablica ma 2 elementy i są to cyfry [1,20]. Twój kod wykona tylko 2 iteracje. Dodatkowo ten warunek if(numbers[i] < numbers[i] + 1) zawsze jest true. Każda cyfra jest mniejsza od tej samej cyfry powiększonej o 1
P1
No to co w tym kodzie powinienem zmienić. Chciałbym napisać swój kod czyli to co napisałem wyżej tylko co zmienić?
NewUser2k13
Przede wszystkim wywalić ifa. Skoro zawsze jest true to nie jest potrzebny. Zmienić warunki brzegowe pętli for. Albo skorzystać z mojej rady. Znaleźć min i max w tej tablicy i iterować w przedziale obustronnie domkniętym <min,max> , albo skorzystać z opcji z poniższego postu. Od pierwszego elementu numbers[0] do ostatniego numbers[numbers.length-1] .Trzeba też poprawić warunek zakończenia pętli i wartość dodawaną do nowej tablicy. Opcja 2 zadziała przy założeniu że przekazywana tablica będzie posortowana. Opcja pierwsza uniwersalna
lion137
  • Rejestracja:około 8 lat
  • Ostatnio:2 minuty
  • Postów:4930
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:ponad 7 lat
  • Ostatnio:28 dni
  • Postów:639
0

Ok. Dzięki za pomoc

LukeJL
  • Rejestracja:około 11 lat
  • Ostatnio:2 minuty
  • Postów:8423
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;
}

edytowany 1x, ostatnio: LukeJL
gk1982
  • Rejestracja:prawie 14 lat
  • Ostatnio:około miesiąc
  • 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 ;
}

Don't give up learning JavaScript!
It is enjoyable to make things visible which are invisible.
Każdy programista przybywający z innego miasta jest fachowcem.
Anyone who stops learning is old, whether at twenty or eighty.
Anyone who keeps learning stays young.
The greatest thing in life is to keep your mind young.

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.