Skip to content
This repository was archived by the owner on Jan 14, 2024. It is now read-only.

LND10 | Adrian Ilovan | JS1 Coursework Week3 #262

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

AdrianIlovan
Copy link

Only 2/3 mandatory completed so far.

@@ -12,6 +12,16 @@
*/

function getTemperatureReport(cities) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect! 👍

*/

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks very good 👍
For extra practice, can you try re-writing this using the filter array method?

}
return headline
}


/*
The editor of the FT likes short headlines with only a few words!
Implement the function below, which returns the title with the fewest words.
(you can assume words will always be seperated by a space)
*/
function titleWithFewestWords(allArticleTitles) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very good solution!

@@ -24,6 +44,13 @@ function titleWithFewestWords(allArticleTitles) {
*/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The next few exercises look good as well!
I would recommend you practice all of these using array methods now 😄

@moneyinthesky
Copy link
Contributor

Hey @AdrianIlovan! This is a very good start - when you have time, it would be good practice to come back and finish the remaining questions 😄

@moneyinthesky moneyinthesky added the reviewed A mentor has reviewed this code label Mar 15, 2023
@AdrianIlovan AdrianIlovan changed the title Lnd10/adrianilovan LND10 | Adrian Ilovan | JS1 Coursework Week3 Jun 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
reviewed A mentor has reviewed this code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants