London|May-2025|Alexandru Pocovnicu|Coursework|sprint 1 #494
Conversation
…bout what 'num' is
alexandru-pocovnicu
added
the
Needs Review
Sprint-1/1-key-exercises/3-paths.js
Outdated
| const dir = filePath.slice(1,4); | ||
| const ext = filePath.slice(-1) |
There was a problem hiding this comment.
Have you checked the variables to see if they hold the expected values?
Can you think of a solution that could work for any valid file path?
For examples, "/tmp/package.json", "/Sprint-1/1-key-exercises/2-initials.js"
| 0, | ||
| paddedPenceNumberString.length - 2 | ||
| );:it creates a string variable with the value of '3' | ||
| 17-19. it creates a string variable that has the value of the last 2 characters from '399' which is '99' |
There was a problem hiding this comment.
Could we expect this program to work as intended for any valid penceString if we deleted .padEnd(2, "0") from the code?
In other words, do we really need .padEnd(2, "0") in this script?
There was a problem hiding this comment.
in this script we don't need it , padStart(3,0) takes care of that
| What output do you get?---ƒ log() { [native code] } | ||
|
|
||
| Now enter just `console` in the Console, what output do you get back? | ||
| Now enter just `console` in the Console, what output do you get back?---- a drop down list |
There was a problem hiding this comment.
Did you press "Enter" key after typing console?
There was a problem hiding this comment.
yes , but i didn't copy pasted, I missunderstood the question,I reviewed my answer now
cjyuan
added
Reviewed
alexandru-pocovnicu
added
Needs Review
|
Changes look good! Well done. |
cjyuan
added
Complete
|
thank you |
illicitonion
changed the title
3 participants
Learners, PR Template
Self checklist
Changelist
I have completed sprint 1, which included exercises, errors, interpreting code, and exploring chrome console
Questions
Ask any questions you have for your reviewer.