funkcja przyjmująca dwie tablicę, zwracająca nową tablicę zawierającą sumę każdego elementu z tych tablic

Question

Witam,

rozwiązuję zadanie o następującej treści:

Napisz funkcję addArrays, która ma przyjmować dwa argumenty – dwie tablice zawierające liczby całkowite. Funkcja ta ma zwracać również tablicę, która będzie wynikiem dodawania każdego elementu o tym samym indeksie. Jeżeli, tablice nie są równe, elementy nadmiarowe są przepisywane do nowej tablicy.

stworzyłem coś takiego:

function addArrays(arr1, arr2) {
  var newArray = [];
  var newArray2 = [];
  for(i = 0; i < arr1.length; i++) {
    for(j = 0; j < arr2.length; j++) {
      if(arr1.length === arr2.length) {
        newArray = [i+j];
        return newArray;
      }
    }
  }
}
console.log(addArrays([1,2,3], [2,4,4]));

jednak konsola zwraca tablicę [0]. Domyślam się, że problemem jest tutaj dodawanie elementów z obu tablic.

nie ruszyłem jeszcze drugiej części zadania, w której jeśli tablice nie są równe to ma powstać następna tablica z tymi nadmiarowymi elementami.

pomożecie?

Answer 1

Jeżeli chcesz coś dodać do tablicy to użyj metody push - myArray.push(something).

return sprawi, że pętla zatrzyma się po pierwszej iteracji.

W dodatku nie dodajesz do siebie elementów tablicy a ich indeksy.

Chyba nie do końca wiesz co dzieje się, gdy pętle są zagnieżdżone, proponuje Ci debugowanie swojego kodu. Sprawdź co dzieje się ze zmiennymi i & j w drugiej pętli(użyj console.log). Dodatkowo zapomniałeś o słowie var przy tych zmiennych.

Comments

tak, to była jakaś porażka, stworzyłem coś nowego na podstawie informacji w internecie:

var addArrays = function(arr1, arr2) {
  var newArray = [];
  var newArray2 = [];
  var longer = '';
  if (arr1.length > arr2.length) {
    longer = arr1;
  }
  else if (arr2.length > arr1.length) {
    longer = arr2;
  }
  else {
    arr1 = arr2;
  }
  for (var i = 0; i < longer.length; i++) {
    newArray.push(arr1[i] + arr2[i]);
  }
  return newArray;
  return newArray2;
}
console.log(addArrays([1,2,3,4], [1,2,3]));

Jednakże consola wyświetla [2,4,6,NaN] czyli dodaje pierwsze trzy elementy z obu tablic a 4 zapisuje jako NaN - dlaczego tak się dzieje i jak zrobić, aby elementy z tablicy dłuższej zapisywały się do nowej tablicy?

---

No dzieje się tak dlatego, że druga tablica ma 3 elementy a próbuje dodać element 4 z tablicy pierwszej i 4 element z tablicy drugiej(ale go nie ma).

Możesz napisać funkcje, która będzie usuwać ostatni element większej tablicy tak długo dopóki nie będzie równa drugiej tablicy. Usuwane elementy możesz przypisać do jakiejś zmiennej i dodawać je do tablicy.

Answer 2

Na początku musisz określić, która tablica jest większa? oraz ile ma elementów? - Nowa tablica będzie miała długość większej tablicy.

var maxLength = Math.max(arr1.length, arr2.length)

Następnie pętla for

newArr[i] = arr1[i] + arr2[i]

Musisz tylko pamiętać, żeby przed dodaniem arr1[i] i arr2[i], sprawdzić co w nich siedzi. Jeśli któraś tablica będzie dłuższa, w drugiej tablicy będziesz próbował odwołać się do indexu, który nie istnieje

arr[i] zwróci wtedy undefined - trzeba zamienić tę wartość na 0 i dopiero przejść do dodawania

na koniec funkcji: return newArr

Comments

ok, ale to nie bardzo wiem gdzie to zamieścić i czy to co piszesz anuluje moje poprzednie zapisy?

---

Na razie napisz kod, który doda tablice o tej samej długości. Później później zajmiemy się undefiend

---

No dobrze, ale po napisaniu kodu, który poleciłeś konsola wyrzuca jedną tablicę z jednym elementem zero ([0]) stąd moje pytanie czy mam go dodać w którymś momęcie mojego kodu czy nie.

---

pokaż kod

---

var addArrays = function(arr1, arr2) {
  var newArray = [];
  var newArray2 = [];
  var maxLength = Math.max(arr1.length, arr2.length);

  for (i = 0; i < newArray.length; i++) {
    newArray[i] = arr1[i] + arr2[i];
  }
  return newArray
}
console.log(addArrays([1,2,3], [1,2,3,4]));

 

---

Po co Ci dwie puste tablice?

i < newArray.length

newArray.length - długość pustej tablicy to 0. 

pętla for musi wykonać się tyle razy, ile elementów ma dłuższa tablica. Po to jest liczone maxLength

---

Ach, pie***le to dziadostwo...

W każdym razie dzięki za pomoc

Answer 3

Możesz stworzyć funkcję - generator przy użyciu yield

 https://developer.mozilla.org/pl/docs/Web/JavaScript/Reference/Operators/yield

Answer 4

function sumArray(arr1, arr2) {
  let arr3 = [];
  for (var i = 0; i < Math.max(arr1.length, arr2.length); i++) {
    arr3.push(arr1[i] + arr2[i]);
    if (arr1.length > arr2.length) {
      arr1.pop();
    }
    if (arr2.length > arr1.length) {
      arr2.pop();
    }
  }
  return arr3;
}

console.log(sumArray([1, 2, 3], [1, 2, 3, 4]));

Wydaje mi się, że to może być rozwiązanie na Twój problem ;) pozdrawiam bootcampu CodersLabu hehe 

Comments

"Odkopałeś" wpis, 

druga sprawa Twój kod nie "robi"  

Jeżeli, tablice nie są równe, elementy nadmiarowe są przepisywane do nowej tablicy.

---

No wiem, że odkopałem, ale co w związku z tym ? Szukałem rozwiązania tego problemu i natrafiłem na ten post z przed pięciu lat. Mi udało się znaleźć rozwiązanie, więc je udostępniłem jakby ktoś jeszcze miał podobne dylematy  a tutaj masz kod który już "coś robi" : 

function sumArray(arr1, arr2) {
  let arr3 = [];
  for (let i = 0; i < Math.max(arr1.length, arr2.length); i++) {
    arr3.push((arr1[i] || 0) + (arr2[i] || 0));
  }
  return arr3;
}

console.log(sumArray([1, 2, 3], [1, 2, 3, 4]));