Skip to content

js basic lab#19

Open
gee92 wants to merge 1 commit intocode-differently:mainfrom
gee92:main
Open

js basic lab#19
gee92 wants to merge 1 commit intocode-differently:mainfrom
gee92:main

Conversation

@gee92
Copy link
Copy Markdown

@gee92 gee92 commented Dec 15, 2020

No description provided.

@kburd kburd self-requested a review December 18, 2020 15:19
Copy link
Copy Markdown

@kburd kburd left a comment

Choose a reason for hiding this comment

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

Great Job! (96%)

@@ -0,0 +1,109 @@
// basic Math

let points = 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.

Remove testing logic after you finish

//Area

let side = 4;
function areaSquare(side){
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 need to use the parameter side (-3)


}

function areaSphere(radius){
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This won't run since math was not capitalized (-1)

}

// surface area
let side = 4;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

let side = 4 is not needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants