-
Notifications
You must be signed in to change notification settings - Fork 166
AnJu Hyppolite and Auredy Sanchez #170
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
| @@ -1,48 +1,86 @@ | |||
| "use strict" | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice job using use strict
|
|
||
| // * Calculate the area based on the radius, and log "The area is `areaOfCircle`". | ||
| // * Hint: https://www.w3schools.com/jsref/jsref_pi.asp | ||
| let areaOfCircle = (Math.PI * radius**2) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good use of Math library. Can also do Math.pow(radius, 2)
| // * Print out if Dee's grade is higher than the class average | ||
| let deesResults = 89 | ||
| let classAvg = (alicesGrade + bobsGrade + camsGrade + deesResults) / 4 | ||
| console.log (("The class average of Alice's," + " " + "Bob's," + " " + "Cam's," + " " + "and " + " " + "Dee's" + " " + "grades is" + " " + classAvg)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You don't need to add spaces between each word, only between text and your vairables.
| // * Years that are divisible by 100 are not leap years if they are not also divisible by 400. | ||
| // * For example 1900 was not a leap year, but 2000 was. Print "Leap year!" or "Not a leap year!" depending on the year you are provided. | ||
|
|
||
| //function checkLeapYear(year) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice attempt!
No description provided.