Clarification #357
Open
Clarification #357
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Here I'm trying to make it more explicit that a global variable is needed as a container for the sum. I didn't pick up on this and tried to fit everything into the `for loop`. I might be spelling things out too much with this commit, but I had to look at the solution before I could understand this problem. I also tried to make the writing in this problem more concise.
Contributor
|
LGTM @workshopper/owners ? |
a0viedo
requested changes
Nov 6, 2016
| Think about how to loop through the number arguments so you can output only their sum. The first element of the `process.argv` array is always 'node', and the second element is always the path to your program.js file. You'll need to start at the 3rd element (index 2), adding each element to the total sum until you reach the end of the array. | ||
|
|
||
| Also be aware that all elements of `process.argv` are strings and you may need to *coerce* them into numbers. You can do this by prefixing the property with `+` or passing it to `Number()`. e.g. `+process.argv[2]` or `Number(process.argv[2])`. | ||
| Try to remember what kind of statement JavaScript uses to hold on to values. What scope such a container would need? Which value would it initially be assigned to? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Here I'm trying to make it more explicit that a global variable is needed as a container for the sum. I didn't pick up on this and tried to fit everything into the
for loop. I might be spelling things out too much with this commit, but I had to look at the solution before I could understand this problem.I also tried to make the writing in this problem more concise.