JS loop-y proste zadanie do sprawdzenia.

Question

Witam, czy ktoś mógłby poświecić chwile i sprawdzić moje rozwiązanie zadania?

Treść zadania:

Construct a for loop that sorts the array [2, 3, 13, 18, -5, 38, -10, 11, 0, 104] into two new arrays:

1. Define an empty evens array to hold the even numbers and an odds array for the odd numbers.
2. In the loop, determine if each number is even or odd, then put that number into evens or odds, as appropriate.
3. Print the arrays to confirm the results. Print evens first. Example: console.log(evens);

moj kod:

const arr = [2, 3, 13, 18, -5, 38, -10, 11, 0, 104];
let evens = [];
let odds = [];

for (let i = 0; i < arr.length; i++) {
if (arr[i]%2 == 0) {
    evens += arr[i] + "\n";
} else {
    odds += arr[i] + "\n";
}
}
console.log(evens);
console.log(odds);

dodalem /n aby log drukowal w nowych liniach.

Zaznaczam, ze jestem super swiezy w temacie wiec bede bardzo wdzieczny za porady co moglem zrobic lepiej/inaczej.

Dzieki za poświęcony czas! ;)

Answer 1

Skoro evens i odds to tablice, to dlaczego przerabiasz je na stringi w 7 i 9 linii? Powinieneś prędzej użyć metody push, żeby dodawać kolejne wartości do tablicy - wtedy też dodanie znaku \n było by zbędne.

Nie wiem za bardzo jak rozumieć fragment o sortowaniu:

Construct a for loop that sorts the array [2, 3, 13, 18, -5, 38, -10, 11, 0, 104] into two new arrays:

, bo jeśli tu faktycznie trzeba sortować tablicę, to w Twoim kodzie tego nie widzę.

Co do samego kodu, to ja bym skusił się na użycie metody reduce, która zwróci obiekt z obiema tablicami (evens i odds):

const { evens, odds } = [2, 3, 13, 18, -5, 38, -10, 11, 0, 104]
  .sort((a,b) => a - b)
  .reduce((evensAndOdds, num) => {
    if (num % 2 === 0) {
        evensAndOdds.evens.push(num);
    } else {
        evensAndOdds.odds.push(num)
    }

    return evensAndOdds;
  }, { evens: [], odds: [] });

console.log(evens); // [-10, 0, 2, 18, 38, 104]
console.log(odds); // [-5, 3, 11, 13]

 

Comments

Cześć, dzięki za odpowiedz, nie znam jeszcze funkcji wiec ich nie dodawałem, tak jak wspomniałem jestem we wczesnej fazie nauki - obecnie pętle i na napisaniu pętli która wyrzuci wartości polegało zadanie 

Zgodnie z Twoja uwaga poprawiłem kod aby dodawał wartości do zdeklarowanych stringow.

const arr = [2, 3, 13, 18, -5, 38, -10, 11, 0, 104];
let evens = [];
let odds = [];
 
for (let i = 0; i < arr.length; i++) {
if (arr[i]%2 == 0) {
    evens.push(arr[i]);
} else {
    odds.push(arr[i]);
}
}
console.log(evens);
console.log(odds);

 

---

Ok. Teraz zmienne evens i odds mogą spokojnie być zadeklarowane jako const, ponieważ wartość przypisujesz im tylko raz. Lepiej też jest używać potrójnego zamiast podwójnego operatora porównania.

---

Wielkie dzieki za Twoj czas ;)

const arr = [2, 3, 13, 18, -5, 38, -10, 11, 0, 104];
const evens = [];
const odds = [];
 
for (let i = 0; i < arr.length; i++) {
if (arr[i]%2 === 0) {
    evens.push(arr[i]);
} else {
    odds.push(arr[i]);
}
}
console.log(evens);
console.log(odds);

 

---

const arr = [2, 3, 13, 18, -5, 38, -10, 11, 0, 104];
const evens = [], odds = [];
  
for (item of arr)
  if (item % 2 === 0)
    evens.push(item);
  else
    odds.push(item);

console.log(evens, odds);