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 intoCodeYourFuture:masterfrom
bireworld:master
Open

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

Conversation

@bireworld
Copy link

@bireworld bireworld commented Jun 20, 2020

Your Details

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

Homework Details

Module: JS
Week: 1

Copy link

@Rody-Kirwan Rody-Kirwan left a comment

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;

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;

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);

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