A Tiny JS World

[OlStani]

###A Tiny JS World

Demo |
Code base

The code is submitted in a dedicated feature branch.

Only code files are submitted.

Please, review.

[OleksiyRudenko]

Good job!
A couple of fixes and we are done.

[OleksiyRudenko]

Having props explicitly listed ({name, gender, legs, hands,saying}) is a good approach.
Please use #Array.map and #Array.join to build the presentation string in the callback function. Remember, you can pack arguments into an array, e.g. ({a1, a2}) => [a1, a2]......some array processing code....

Also why do you need to wrap every presentation string into an array? You have nested arrays as a result.

On your demo page, open dev console and type inhabitantsInformation to see the data structure.

Feel free seeking help in students' chat.

[OlStani]

Please use #Array.map and #Array.join to build the presentation string in the callback function. Remember, you can pack arguments into an array, e.g. ({a1, a2}) => [a1, a2]......some array processing code....

я не зовсім розумію що я маю тут зробити : Please use #Array.map and #Array.join to build the presentation string in the callback function. Remember, you can pack arguments into an array, e.g. ({a1, a2}) => [a1, a2]......some array processing code....
як будете мати час напишіть будь-ласка https://t.me/olstaniev

[OleksiyRudenko]

Let's start with answering questions.

Also why do you need to wrap every presentation string into an array? You have nested arrays as a result.

[OlStani]

Let's start with answering questions.

Also why do you need to wrap every presentation string into an array? You have nested arrays as a result.

я вже виправив це

[OlStani]

метод map повертає мені массив строк

[OleksiyRudenko]

я вже виправив це

I can only see what's on PR.

[OleksiyRudenko]

метод map повертає мені массив строк

That's how it works. You can glue them with e.g. .join

[OleksiyRudenko]

I am a peer developer on this project, and was asked to add 100 more objects with yet another 20 properties.
Please make my life easier.

[OleksiyRudenko]

Please make properties separated exactly with '; '

[OleksiyRudenko]

@OlStani this works, grats.

I guess your latest change was inspired by other soultions. However, your own initial approach might have been improved with a minor change reducing code repetition. E.g.

const inhabitantsInformation = inhabitants.map(({name, gender, legs, hands,saying}) => [name, gender, legs, hands, saying].join('; '))

[OlStani]

то було занадто просто, тому не прийшло мені в глову)