Arrays and Nested Arrays - Exercise

Привет gbsho,

първо в твоя код  let [row,col,step,count] =[0,0,0,0]; "step" в деструктурирането ти е излишно, никъде не го ползваш. В цикъла "step" е друга различна променлива.

Също имаш излишен код count === mtrx.length ? console.log("true") : console.log("false"),това просто можеш да го изпишеш като

console.log(count === mtrx.length) - това е boolean стойност.

Като цяло идеята ти е добра, но пробвай с критичен вход [[]], празен масив. Получаваш отговор, а би трябвало да ти гръмне грешка. А и [[1, 0, 1],[0, 1, 0],[1, 0, 1]] при този вход имаш отговор "true", а както би видял не е варно. Ти проверяваш сбора на всеки ред и колона което е row при index[0] ->1+0+1 = 2 и col = 1+0+1 row=2=col=2, при вторият цикъл row=3 и col=3, само че на теб реда ти е = 1,(втори ред [0,1,0]=1 !== първи ред[1,0,1]=2) което не е равно row=2, но тъй като ти проверяваш обща сума, а тя е row=3 и по ред и колона col=3 изпълняваш проверката и добавяш count++;

Трябва да сравняваш ред по ред и колона по колона, а не общ сбор. Ако запишеш сбора на първи ред и сравняваш по него всеки ред и колона ще вземеш 100/100. създай два масива като единият ти и по сбора на колоните, а другият ти е по сбора на редовете, то тогава можеш да сравниш двата масива като ползваш toString(), след като си проверил дали стойностите в двата масива на всички колони и редове са равни. Тогава ще си сигурен, в крайният резултат!

Успех!

А ето и твоя код 100/100 с малки поправки ;):

function magicMtrx(mtrx) {

    let [row, col, count] = [0, 0, 0];
    const equalBenchmark = mtrx[0].reduce((a, b) => {
        a += b;
        return a; 
    }, 0)

    for (let step = 0; step < mtrx.length; step++) {
        let rowCheck = 0;
        let colCheck = 0;

        mtrx[step].forEach(el => rowCheck += el);
        rowCheck === equalBenchmark ? row += rowCheck : row = null;
       
        mtrx.forEach(el => colCheck += el[step]);
        colCheck === equalBenchmark ? col += colCheck : col = null;
        
        if (row === col && row != null && col != null) { count++ };
    }

    console.log(count === mtrx.length);
}

Лично аз бих го решил доста различно, но мисля, че идеята е да ти помогна на теб с твоята логика, а ако искаш да видиш друго драсни два реда.

Успех!

Здравей, Tzigoriyn  ! Благодаря, времето и подробните обеснения!

Да, горе няма нужда да задавам step [row,col,step,count]

Аз задачата я реших (отдолу е кода), но не исках да я правя, както по упражненията - взимат два съседни реда и две колони, които ги сравняват.

function magicMtrx(mtrx){
    
    let [row1,row2,col1,col2] =[0,0,0,0];
    
    for (let step=0; step<mtrx.length-1; step++){
        mtrx[step].forEach(el=>row1+=el);
        mtrx[step+1].forEach(el=>row2+=el);
        mtrx.forEach(el=>col1+=el[step]);
        mtrx.forEach(el=>col2+=el[step+1]);
        if(row1 === row2 && col1 === col2){return true;}
        else{return false;}
    }
}
console.log(magicMtrx([[1, 0, 1],
 [0, 1, 0],
 [1, 0, 1]]))

Моята идея е, че на всяка стъпка сравнявам сумите от ред/колона, увеличавам row и col - ако са равни увеличавам count.

На следващата стъпка, пак ги сравнявам и добавям в row и col - ако са равни пак увеличавам count... и т.н

Накрая ако count e колкото е matrix.lenght, значи всички суми от ред/колона са еднакви...

Целта ми беше да стане с възможно най-малко код :)

За 4ти тест, който коментирам матрицата е:

    [3,        6,        3] 
    [6,        3,        6]
    [3,        6,        3] 

( от вида които ти написа: [1, 0, 1], [0, 1, 0], [1, 0, 1] )

judge очаква да получи false, но според мен тази матрица е меджик ( true ).

Може и да бъркам нещо - както споменах - не мога да им разбера задания понягока.

Аз така или иначе не си спомням за "магически матрици" в математиката... :)

Ще се радвам да видя и твоята интерпретация (решение).

Благодаря!

Ако искаш малко код виж това:

function magicMatrices(arr) {

    let sumRow = arr.map((col => col.reduce((a, b) => a + b)));

    let sumCol = arr.reduce((r, a) => r.map((b, i) => a[i] + b));

    let allEqual = array => array.every( v => v === array[0]);

    console.log(allEqual(sumRow) && sumRow.toString() === sumCol.toString());
    
}

и идеята на кода e не само кода да е малко, а да е reusable и възможно най бърз, защото се предполага че ще се рендва в browser. Имай в предвид, че колкото пъвече циклиш толкова повече бавиш! Ако погледнеш в един от коментарите съм редактирал твоя код, взимам един ред и с него проверявам всеки ред и всяка колона. Сумата от редовете и колоните изобщо не тие е нужна, просто допълнителна проверка за подсигуряване.

Относно ->

[3,        6,        3]->12
[6,        3,        6]->15
[3,        6,        3]->12

12       15        12

определено е false, по условие " Write a function that checks if a given matrix of numbers is magical. A matrix is magical if |-> the sums of the cells of every row and every column are equal |->(сумите от клетките на всеки един ред и всяка една колона трябва да са равни). "!

Ако ти харесва моя код и коментара не забравяй да удариш един палец .

Успех!

П.П. Матриците са във висшата математика и има магически матрици, дори при тях са равни и диагоналите

                            15

[8,        1,        6]->15
[3,        5,        7]->15
[4,        9,        2]->15

15       15      15   15

100/100

function magic(matrix) {

    let sum = matrix[0].reduce((sum, x) => sum + x);

    for (let row = 0; row < matrix.length; row++) {

        let rowSum = 0;

        let colSum = 0;

        for (let col = 0; col < matrix[row].length; col++) {

            if (col < matrix.length) {

                rowSum += matrix[col][row];

            }           

            colSum += matrix[row][col];

        }

        if (rowSum != sum ||

            colSum != sum) {

            return false;

        }

    }

    return true;

}

Доста нечетим код Tzigoriyn.

VladeyMoney Няма страшно ще се научиш, но това става с времето.

Пояснавам

sumRow

е масив от сумите на всеки ред, а

sumCol

е масив от сумите на всяка колона

allEqual(sumRow)

е проверка коята проверява дали всички суми от пададеният масив са равни.

console.log(allEqual(sumRow) && sumRow.toString() === sumCol.toString());

Ако си задаваш въпроса "Защо не проверявам и колоните за еднаквост?" -няма смисъл да циклиш още веднъж през every() и да бавиш кода допълнително, просто ако първият масив е true сравняваш два стринга, примерно - "2, 2, 2" === "2, 2, 2". Целият израз връща boolean и печатиш резултата.

Но ако трябва да ти разяснявам какво прави map(), reduce(), every() и каво означава let allEqual = array => .... и че това не е променлива, а

Function expressions

на която подавш масив, тогава яко трябва да налягаш на ученето. Виждам, че си избрал правилното място SoftUni.

;)

Успех приятел!

Tzigoriyn тези функции отдолу не работят с цикли и са по-оптимизирани, така ли да го разбирам?

Да точно така VladeyMoney, те се наричат Methods, погледни "Instance methods" MDN те също обхождат масива в зависимост от това което търсим от него, разбира се доста по оптимизирано. Като цяло това което наричаме цикли са само три:

• стандартен фор цикъл - for ([initialExpression]; [conditionExpression]; [incrementExpression])
    statement

• while(condition)
• do

    statement
  while (condition);
  
  
  
  
  
  
   

  Tzigoriyn аз не мисля така

https://leanylabs.com/blog/js-forEach-map-reduce-vs-for-for_of/

Един преподавател някога в SoftUni каза "На Вас като програмисти никой няма да Ви плаща за брой редове код, а за работещ код".

Един реторичен въпрос: Колко реда код ще трябва да напишеш ако трябва да подредиш поименно един масив от именна, защото посредством метода sort() ще го направиш буквално с един ред. Да той отдолу като цяло е цикъл, но за да разбера какво искаш да направиш при обикновен for() ще трябва да анализирам целият ти код, а при метода сорт веднага ще знам, че този код ще сортира и ще видя по къкъв критерий. А сега си представи няколко хиляди реда код и десетина човека като теб написали фор-цикълчета и на теб се пада честа да го тестваш. Ай ся кажи кой for() какво прави, сумира ли, филтрира ли, редуцира ли или пък прави нещо друго??? Също имай предвид, че този код не е на едно място, а е разпръснат в десетки файлове които се експортват и импортват на пред на зад(опа това с export май още не сте го учили?).

А иначе, за това се учат циклите, за да ръзбереш същноста на работа на МЕТОДИТЕ, които ги ползват underground. Да, те до някъде са по-бавни, но за сметка на това са и далеч по-мощни и интуитивни, но това ще го научиш с времето.

Всъщност тук имаше въпрос на който отговорих, защото колега изпитваше затруднения при разбирането на задачата и му помогнах. Моя код в никакъв случай не е критерий просто е кратък буквално е 6 реда със скобите, срещу 12 или 15, че и повече. 

Пожелавам ти успех VladeyMoney, извини ме, но не съм учител, поне не все още! Останалото ще го научиш в SoftUni, когато му дойде времето!

П.П. При въпрос на интервю за работа, циклите са тези които ти описах, другото, въпреки че изпълняват LOOP се наричат МЕТОДИ.

Прав си за "за плащането на ред код" това важи и за писането по форуми според мен, но свързано с полза не заплащане.

Прав си VladeyMoney , аз отговорих на конкретното запитването на gbsho , колегата явно имаше проблем с условието на задачата, а не с решението и. Незная какво общо имаше неговият въпрос с кода който изплющя ти, при наличието, че той вече беше намерил своето решение със собствена логика, което е най важното. На теб ти отговорих от вежливост и в никакъв случай не съм се опитвал да те поучавам, ти явно си ги знаеш нещата, извини ме моля!

Благодаря Axiomatik , гоуем си  .