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

ZA2_Nishka-Kisten-JavaScript-Core-1-Coursework-Week1 - #293

Closed
Nishka-Kisten wants to merge 1 commit into
CodeYourFuture:masterfrom
Nishka-Kisten:master
Closed

ZA2_Nishka-Kisten-JavaScript-Core-1-Coursework-Week1#293
Nishka-Kisten wants to merge 1 commit into
CodeYourFuture:masterfrom
Nishka-Kisten:master

Conversation

@Nishka-Kisten

@Nishka-Kisten Nishka-Kisten commented Jul 1, 2022

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: Nishka Kisten
  • Your City: Cape Town
  • Your Slack Name: Nishka-Kisten

Homework Details

  • Module: JavaScript-Core-1
  • Week: 1

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
View rendered exercises/F-strings-methods/README.md
View rendered exercises/G-numbers/README.md

@MarvinLeeMarshall

Copy link
Copy Markdown

This is very well done and it is also refreshing to see a different view on how you've written your code.

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

Amazing understanding of the basic Javascript syntax and naming conventions. No need to know all words, as long as you understand the concepts.

@@ -1 +1,4 @@
console.log("Hello world");
console.log("Nishka Kisten");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Amazing use of the javascript syntax.

@@ -1,3 +1,5 @@
// Start by creating a variable `message`

var message = "This is a string";

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 neat code here. Keep that up!

var numberOfMentors = 8;
var studentsAndMentors = numberOfStudents + numberOfMentors;

var messageStudents = "Number of students: " + numberOfStudents;

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 is great javascript basics. You understanding the basic concept to declaring variables and assigning arguments to them. Well done!

Comment thread mandatory/4-tax.js
*/

function addTaxAndFormatCurrency() {}
function addTaxAndFormatCurrency(num) {

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 is amazing code. Making use of good naming conventions and ignoring unclear function names.

@dschouw-CYF dschouw-CYF 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.

Thank you and well done for submitting your coursework Nishka.
Try and commit more often ( there is only one commit in the pull request ), and if you have time, try and complete the extra challenges.


// console.log(result2);

function shoutyGreeting(name, string){

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 Nishka 💯
Maybe consider moving shoutyMessage variable into the shoutyGreeting function , since its only used in the function

Suggested change
function shoutyGreeting(name, string){
function shoutyGreeting(name, string){
let shoutyMessage = "hello"
... rest of function

// Add comments to explain what this function does. You're meant to use Google!
function getRandomNumber() {
return Math.random() * 10;
return Math.random() * 10; // it returns a random number that can be anywhere between 0 and 1. The 0 is included and 1 is excluded.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Comment is clear, well done.
You've described the function Math.random well, but note the multiply by 10. What do you think the function returns with the multiply by 10?

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.

5 participants