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

CAPE TOWN- CLASS 1 - SINOMBULELO_PUNGULA - JS_CORE_1 - WEEK_7 - #60

Closed
sinopungula wants to merge 4 commits into
CodeYourFuture:masterfrom
sinopungula:master
Closed

CAPE TOWN- CLASS 1 - SINOMBULELO_PUNGULA - JS_CORE_1 - WEEK_7#60
sinopungula wants to merge 4 commits into
CodeYourFuture:masterfrom
sinopungula:master

Conversation

@sinopungula

Copy link
Copy Markdown

completed all the work and attempted J-not successsful yet.

Your Details

  • Your Name: Sinombulelo Pungula
  • Your City: Cape Town
  • Your Slack Name: sinopungula

Homework Details

  • Module: JS Core 1
  • Week: 7

completed all the work and attempted J-not successsful yet
improvements made to folder L-functions
Comment thread mandatory/3-function-output.js Outdated
Comment thread mandatory/3-function-output.js Outdated
return Math.random() * 10;
}
return Math.random() * 10;
} //getNumber will have Math.random() return a random number between 0 (incl.) and 1(excl) and then is multiplied 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.

returns random float between 0 and 10

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Hi. Thank you for your comment.

Great catch with the word float, attention to detail makes all the difference. Thanks!
Google says it returns a random float between 0 and 1-not 10, perhaps this is a typo?

function percentageNumber(numOfPeople, totalNumOfPeople) {
return numOfPeople * 100 / totalNumOfPeople;
}
var numOfStudents = percentageNumber(15, 23);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

you should use variables instead of hardcoding the values

var mentor3 = "Mimi";
var mentor4 = "Rob";
var mentor5 = "Yohannes";
var mentor1 = greeting("Daniel");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

you didnt have to redeclare the variables

@chandredewet

Copy link
Copy Markdown

Well done Sino on this week's work, for getting a grip on javascript and Wowzers to your name being 11 characters long! The only feedback I would like to give you is to ask you to commit more and often, maybe after every second exercise, etc. Also to complete the instructions even though you know the function or command, to be mindful to add the full sentences required in the suggested answer on the readme. Getting in a habit of this will help you in your future career as you are working in teams, where expected guidelines are required and also, some programs will require those full detailed sentences. Here is just a few things I picked up and suggestions to work on:

G | Add all detail required not just the sum.

I | There is a mistake in your final answer for the mentors, can you spot it? the value is incorrect
also the way the answer is displayed as mentioned before, do also make sure you include the extra detail, e.g
"Percentage of mentors: " and "%"

L exercise 2 | great initiative in creating two files, as this question was definitely not clear, in fact it had errors, and you did manage to solve both exercises. but again simply remember to add the %

Well done on a whole! You doing well! Keep committing, remember to include all the details and just check that error in I :)

@aashiqabrahams123

Copy link
Copy Markdown

Hi Sino, Well done on your work and code, keep up the good work!!!

@github-actions

Copy link
Copy Markdown

Your coursework submission has been closed because nobody has interacted with it in six weeks. You are welcome to re-open it to get more feedback.

@github-actions github-actions Bot added the Stale label May 13, 2021
@github-actions github-actions Bot closed this May 13, 2021
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