Skip to content
This repository has been archived by the owner on Jan 14, 2024. It is now read-only.

London10 || Hussein-Bahdon || JavaScript-Core-1-Coursework-Week2 #449

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

H-BAHDON
Copy link

@H-BAHDON H-BAHDON commented Mar 3, 2023

Volunteers: Are you marking this coursework? You can find a guide on how to mark this coursework in HOW_TO_MARK.md in the root of this repository

Your Details

  • Your Name:
  • Your City:
  • Your Slack Name:

Homework Details

  • Module:
  • Week:

Notes

  • What did you find easy?

  • What did you find hard?

  • What do you still not understand?

  • Any other notes?

@@ -9,9 +9,17 @@
*/

function factorial(input) {

Choose a reason for hiding this comment

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

Well done!


if (userAge >= 18 && isLoggedIn ===true ) {
return true;
}else if (userAge > 18 && isLoggedIn === true) {

Choose a reason for hiding this comment

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

This condition repeats the previous one , i think the previous one is enough. And ===true not necessary there .
if (userAge >= 18 && isLoggedIn ) does the same work

@@ -45,7 +84,11 @@ function canRegister(age) {}
)
*/

function countReverse(number) {}
function countReverse(number) {
for (let i = number; i >= 1; i--) {

Choose a reason for hiding this comment

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

Smart !

@H-BAHDON H-BAHDON changed the title Hussein-Bahdon--London10-Droves London10 || Hussein-Bahdon || JavaScript-Core-1-Coursework-Week2 Dec 7, 2023
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