Skip to content

loops lab#4

Open
gee92 wants to merge 1 commit into
Return-Ready-2021-JavaScript-Evening:mainfrom
gee92:main
Open

loops lab#4
gee92 wants to merge 1 commit into
Return-Ready-2021-JavaScript-Evening:mainfrom
gee92:main

Conversation

@gee92
Copy link
Copy Markdown

@gee92 gee92 commented Dec 15, 2020

No description provided.

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, just missing number 4 (81%)

Comment thread JavaScript-Loops-Lab.js

/*let answer = prompt("Are we there yet?")
if(answer === "yes"){
console.log("good");
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Yes displays twice if you answer yes on the first prompt (-3)

Comment thread JavaScript-Loops-Lab.js
let triangle = " ";

for (let i=1; i <= 12; i++) {
triangle += i + " ";
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Should be * instead of numbers (-1)

Comment thread JavaScript-Loops-Lab.js
}

/*part 4
numberSum = (n,m) => {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Incomplete (-12)

Comment thread JavaScript-Loops-Lab.js
console.log(output);

//8
let output="";let count = 0;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Should work of a variable n, similar to what you did on problem 3 (-3)

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