- Notifications
You must be signed in to change notification settings - Fork6.7k
[BER-FT-032023] Gunnar Miklis#3687
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
arekiu left a comment
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.
Very elegant solutions!
| } else { | ||
| sum += typeof entry === 'string' ? entry.length : entry | ||
| } | ||
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.
Super good solution! The only thing I think it would be even better is to write the statement with the types we know will be supported, first (in if). And all the rest going to else.
But anyway, in this case, the array to test is already there and the solution works perfectly
This pull request has been automatically marked as stale because it didn't have any recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This pull request is closed. Thank you. |
Done: Iteration#1 to#7