Skip to content
This repository was archived by the owner on Oct 26, 2020. It is now read-only.

Berhane's Homework - #30

Open
bireworld wants to merge 21 commits into
CodeYourFuture:masterfrom
bireworld:master
Open

Berhane's Homework#30
bireworld wants to merge 21 commits into
CodeYourFuture:masterfrom
bireworld:master

Conversation

@bireworld

@bireworld bireworld commented Jun 20, 2020

Copy link
Copy Markdown

Your Details

Your Name: Berhane
Your City: Birmingham
Your Slack Name: Berhane

Homework Details

Module: JS
Week: 1

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

Well done Berhane. Good work

@@ -1,2 +1,9 @@
var numberOfStudents = 15;
var numberOfMentors = 8;
var stuPercentile = numberOfStudents/23*100;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

For readability try to use full words unless it is a very well recognised acronym like ID for example
studentPercentile

var numberOfStudents = 15;
var numberOfMentors = 8;
var stuPercentile = numberOfStudents/23*100;
var mentorpercentile = numberOfMentors/23*100;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

use camelCase mentorPercentile


function formatCurrency() {}
function formatCurrency(price) {
var totPrice = calculateSalesTax(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.

use full name - totalPrice :)

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