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

JavaScript-Core-1-Coursework-Week1 - #446

Open
laxmikckarki wants to merge 3 commits into
CodeYourFuture:masterfrom
laxmikckarki:master
Open

JavaScript-Core-1-Coursework-Week1#446
laxmikckarki wants to merge 3 commits into
CodeYourFuture:masterfrom
laxmikckarki:master

Conversation

@laxmikckarki

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?

  • What did you find hard?

  • What do you still not understand?

  • Any other notes?

@laxmikckarki

Copy link
Copy Markdown
Author

done excercise and mandatory

@mattietea

Copy link
Copy Markdown

Great work @laxmikckarki!

@mattietea
mattietea self-requested a review November 25, 2022 09:45
Comment thread mandatory/2-logic-error.js Outdated
function trimWord(word) {
return wordtrim();
function trimWord (word) {
return trimWord();

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@laxmikckarki Hi, well done!
In think in line 4 (function 1), it is better to use word.trim(), because you use the trim() method to trim the string. If you look at the tests below the exercises, it can give you a clue.

Comment thread mandatory/4-tax.js Outdated
const priceWithTax = 1.2 * price;
return priceWithTax;
function calculateSalesTax(price) {
return price + (price / 100 * 20);

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 have done this exercise same as you but I think its better to use parentheses in a way to make your math more clear

}
const greeting = createLongGreeting("Daniel", 30);

console.log(greeting);

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 ,I have just an suggestion in line 3 you use two way: and " ". I think its better use one of them.

return 'hello ' + mentor3;
}
let greeting = greetingUpperCase();
console.log(greeting.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.

GREAT. I learn from this one I did in different way(I use twice toUpperCase) I will change mine to yours

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.

4 participants