Basic Algorithm Scripting exercise 5

[franklin0415]

Hello I’m currently in the challenge “Return Largest Numbers in Arrays” from the Basic Algorithm Scripting section in the JavaScript Algorithms and Data Structures certification and I came up with this solution:

I got curious and also checked the provided solutions in the challenge guide. I noticed that my solution is different from all those and I would like to get some feedback. I’m still new to this coding stuff and I don’t really know about performace on algorithms, I know it works but I have no idea how efficient it is compared to the other answers

[JeremyLT]

Please post your actual code instead of a picture.

I wouldn’t get caught up in ‘performance’. For most things, as long as you write sensible code, you don’t have to worry about writing a ‘high performance’ algorithm.

[hbar1st]

We have blurred this solution so that users who have not completed this challenge can read the discussion in this thread without giving away the solution.

[franklin0415]

My bad, here is the code:

function largestOfFour(arr) {
    let newArr = [];
    for (let i = 0; i < arr.length; i++) {
        let largestNum = 0;
        largestNum = Math.max(...arr[i]);
        newArr.push(largestNum);
    }

    return newArr;
}

largestOfFour([4, 5, 1, 3], [13, 27, 18, 26], [32, 35, 37, 39], [1000, 1001, 857, 1]);

[JeremyLT]

I’ve edited your code for readability. When you enter a code block into a forum post, please precede it with a separate line of three backticks and follow it with a separate line of three backticks to make it easier to read.

You can also use the “preformatted text” tool in the editor (</>) to add backticks around text.

See this post to find the backtick on your keyboard.
Note: Backticks (`) are not single quotes (').

[franklin0415]

Got it, thanks for the answer

[JeremyLT]

@franklin0415

let newArr = [];

Small quibble - arrays generally should be declared as const.

@franklin0415

        let largestNum = 0;
        largestNum = Math.max(...arr[i]);

Why not put this all on one line?