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

homework - #107

Open
FranklinPTorres wants to merge 1 commit into
CodeYourFuture:mainfrom
FranklinPTorres:main
Open

homework#107
FranklinPTorres wants to merge 1 commit into
CodeYourFuture:mainfrom
FranklinPTorres:main

Conversation

@FranklinPTorres

Copy link
Copy Markdown

No description provided.

@rebwar1 rebwar1 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Good Job

@Sharifa1 Sharifa1 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Well done Franklin

// Example 4
// Example 4 // console.log is returning undefined as in JavaScript, arrays are zero-indexed with the first element of an array being 0,
// the second element being 1 etc. In this example, there are three elements, however, the index ends at 2. Therefore, arr[3] returns
// undefined as the index ends at 2.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nice explanations

}
index ++;
}
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Good!

let str = "codeyourfuture";
for (let char of str) {
console.log(char.toUpperCase());
} No newline at end of file

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

well done Franklin

Comment thread 2-mandatory/4-stocks.js
return sortedArr.map((prices, index) => {
return `The highest price of ${STOCKS[index].toUpperCase()} in the last 5 days was ${prices[0].toFixed(2)}`;
})
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Good job!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants