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

north-west-5/davood.moradi/js.week1#401

Open
clav00cl wants to merge 1 commit into
CodeYourFuture:masterfrom
clav00cl:master
Open

north-west-5/davood.moradi/js.week1#401
clav00cl wants to merge 1 commit into
CodeYourFuture:masterfrom
clav00cl:master

Conversation

@clav00cl
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?

Copy link
Copy Markdown

@bensonarafat bensonarafat left a comment

Choose a reason for hiding this comment

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

Good job!
Cheers

var greeting = "Hello world";

console.log(greeting);
console.log(greeting); 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.

Good job!!
But it should be printed not just once

const numberOfStudents=40;
const numberOfMentors = 8;

console.log(numberOfMentors + numberOfStudents); 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.

Good one.
Just a few details are missing about your solution.
Also, a console description of what you are calculating.

var numberOfMentors = 8;
var preciseNum=(numberOfMentors / numberOfStudents) *100;
var roughNum = Math.round(preciseNum);
console.log(roughNum , preciseNum); 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.

Same here

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