Commit 1 - Verbal questions #1

Open
wants to merge 7 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@mlhegeman

mlhegeman commented Dec 14, 2018

questions completed.

}
return sum;
}
console.log(sumDigits(42));

This comment has been minimized.

@reuben

reuben Dec 20, 2018

Contributor

nice work

function calculateSide(x, y) {
return Math.sqrt(x * x + y * y);
}
console.log(calculateSide(8, 6));

This comment has been minimized.

@reuben

reuben Dec 20, 2018

Contributor

looks good

for(const j = i * i; j < value; j += i) {
primes[j] = false;
}
}

This comment has been minimized.

@reuben

reuben Dec 20, 2018

Contributor

this causes a syntax error. you are omitting several necessary curly braces. you should not submit code with syntax errors... that means you haven't tested it/run it. fix this ad resubmit

for(let i = 0; i < numbers.length; i++){
sum += numbers[i]
}
console.log(sum)

This comment has been minimized.

@reuben

reuben Dec 20, 2018

Contributor

this logic works but the problem wants this to be a function. make it a function and resubmit.

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