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

London9 Lovelace - Oleksii Chepurnyi - JavaScript Core 2 Coursework - Week1#184

Open
OleksChep wants to merge 9 commits into
CodeYourFuture:mainfrom
OleksChep:main
Open

London9 Lovelace - Oleksii Chepurnyi - JavaScript Core 2 Coursework - Week1#184
OleksChep wants to merge 9 commits into
CodeYourFuture:mainfrom
OleksChep:main

Conversation

@OleksChep
Copy link
Copy Markdown

Volunteers: Are you marking this coursework? You can find a guide on how to mark this coursework in HOW_TO_MARK.md in the root of this repository

Your Details

  • Your Name:
  • Your City:
  • Your Slack Name:

Homework Details

  • Module:
  • Week:

Notes

  • What did you find easy?
    Add and change properties/

  • What did you find hard?
    In which area of the code to write "if" statement.

  • What do you still not understand?

  • Any other notes?

@maxf maxf self-requested a review January 5, 2023 22:15
Copy link
Copy Markdown

@maxf maxf left a comment

Choose a reason for hiding this comment

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

Excellent. It looks like you're all caught up!
You've also improved your "git hygiene", although you can still improve but not deleting lines randomly, improve your indentation. The "files changed" section in your PR should only show changes related to your solutions to the exercises, not random changes like some whitespace removed.
Congrats!

*/
// write code here
console.log(basketballTeam.topPlayers.sort().join("\n"));
let basketballTeamsTopPlayers = basketballTeam.topPlayers.sort().join("\n");
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 idea to use a variable to make your code more readable

};

console.log(myPet.getName());
// the getName is not a function , getName is property of object myPet
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

That's not correct: getName is a function, and myPet.getName() does call it. The reason why it outputs undefined is elsewhere. I didn't see it at first, either!

Comment thread 2-mandatory/1-recipes.js
console.log("Recipe 1", " ", recipe1.title);
console.log("Servings: ", recipe1.servings);
console.log("Ingredients:", "\n", recipe1.ingredients.join("\n"));
console.log(`\ `);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I'm not sure that line is necessary

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

or perhaps you meant console.log('\n') as a separator?

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.

2 participants