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
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.
JavaScript 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.
-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
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
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
Ok. Dzięki za pomoc
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;
}
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 ;
}