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

ZA_2-Marvin_Marshall-JS-Core-1-WK1 - #295

Closed
MarvinLeeMarshall wants to merge 1 commit into
CodeYourFuture:masterfrom
MarvinLeeMarshall:master
Closed

ZA_2-Marvin_Marshall-JS-Core-1-WK1#295
MarvinLeeMarshall wants to merge 1 commit into
CodeYourFuture:masterfrom
MarvinLeeMarshall:master

Conversation

@MarvinLeeMarshall

@MarvinLeeMarshall MarvinLeeMarshall commented Jul 1, 2022

Copy link
Copy Markdown

I have passed all test in the JS files.

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?


View rendered exercises/B-hello-world/README.md

I have passed all test in the JS files.
@Getson47

Getson47 commented Jul 1, 2022

Copy link
Copy Markdown

Keep pushing!

@chezlinbenson chezlinbenson 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.

These exercises were good challenges to understand the basic Javascript language. declaring functions, variables, parameters, arguments and naming conventions, were the goals for week1, and you have shown great skills in the javascript basic language. Well Done! Continue following the basic structures and concepts moving forward.

var total = numberOfStudents + numberOfMentors;
/*calculation for students*/

var text1 = "Percentage of Students ";

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 understanding of basic javascript syntax and naming conventions. No need to understand all the words, as long as you understand the concepts.

@@ -1,4 +1,7 @@
// Declare your function here
function createLongGreeting(name, age){

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Understanding string concaternation takes basic exercises like these. You did well in this exercise analysing your code and code stucture.

var mentor5 = "Yohannes";


function shoutyGreeting(name){

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This exercise had many trainees get to use more of their brains. There are many solutions to any code. Your basic functions show great understanding on Declaring Functions and returning them. Well done!

return Math.random() * 10;
}

//The above function grabs a random number by using the Math.random() method and multiplies that random number by 10.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Very good comments. They are clear and uses the Javascript proper commenting.

Comment thread mandatory/4-tax.js
*/

function addTaxAndFormatCurrency() {}
function addTaxAndFormatCurrency(price){

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Very clean code, well done! There are various options to the functions you are coding to make it shorter. Making use of the correct methods in your code, are best practise, Well done there!

@riaanduplessis riaanduplessis 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.

Awesome work Marvin! 🎉

total = a ++ b;

return "The total is total";
return "The total is " + (a + b);

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 call using the brackets here + (a + b) 👌

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants