-
-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
A recursive function in JavaScript that prints all elements of a multi-dimensional array. #1724
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1724 +/- ##
==========================================
- Coverage 84.76% 84.69% -0.08%
==========================================
Files 378 379 +1
Lines 19742 19760 +18
Branches 2955 2958 +3
==========================================
Hits 16735 16735
- Misses 3007 3025 +18 ☔ View full report in Codecov by Sentry. |
console.log(arr[i]) | ||
} | ||
} | ||
} |
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.
Always use return
in the function and console the function execution .
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.
It is the function used to print the element, why should I use "Return"?
IMO this algorithm is too simplistic to be worth adding to the repo. |
know more
Describe your change:
Checklist:
Example:
UserProfile.js
is allowed butuserprofile.js
,Userprofile.js
,user-Profile.js
,userProfile.js
are notFixes: #{$ISSUE_NO}
.