Manchester | 25-ITP-Sep | Hani Sadah| Sprint 1 | Coursework/sprint 1#810
Manchester | 25-ITP-Sep | Hani Sadah| Sprint 1 | Coursework/sprint 1#810HANISADAH wants to merge 30 commits intoCodeYourFuture:mainfrom
Conversation
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Wrong number of parts separated by |s If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Wrong number of parts separated by |s If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Wrong number of parts separated by |s If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
4 similar comments
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Wrong number of parts separated by |s If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Wrong number of parts separated by |s If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Wrong number of parts separated by |s If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Wrong number of parts separated by |s If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Wrong number of parts separated by |s If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Wrong number of parts separated by |s If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
|
Can you please read this PR Guide and update the title and the PR description accordingly. |
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Wrong number of parts separated by |s If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
2 similar comments
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Wrong number of parts separated by |s If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
|
Your PR's title isn't in the expected format. Please check the expected title format, and update yours to match. Reason: Wrong number of parts separated by |s If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). |
cjyuan
left a comment
There was a problem hiding this comment.
-
This is the Sprint-1 PR. There shouldn't be any modified files in the Sprint-2 folder. Can you keep this PR branch clean by reverting the changes made in the Sprint-2 folder? (Revert is not the same as delete, so don't delete the files in Sprint-2 folder)
-
If
number-game-errors.htmlis not related to Sprint-1, it should also be deleted. -
In the PR description, the checked boxes are not properly encoded in Markdown syntax. And if you don't have any question, you can delete the "Questions" section.
Sprint-1/1-key-exercises/1-count.js
Outdated
|
|
||
| // Line 1 is a variable declaration, creating the count variable with an initial value of 0 | ||
| // Describe what line 3 is doing, in particular focus on what = is doing | ||
| // line 3 returns the incurmented value of count after adding 1 to it and assigns this new value back to count No newline at end of file |
There was a problem hiding this comment.
Operation like count = count + 1 is very common in programming, and there is a programming term describing such operation.
Can you find out what one-word programming term describes the operation on line 3?
There was a problem hiding this comment.
I believe the term is assignment. It is the process of giving a value to a variable using an assignment operator.
There was a problem hiding this comment.
I was referring to the operation that increases the value of a variable by certain amount.
Sprint-1/1-key-exercises/3-paths.js
Outdated
| const dir = filePath.slice(lastSlashIndex+1) | ||
| const ext = filePath.slice(lastDotIndexOf(".") +1); |
There was a problem hiding this comment.
I don't think this script works as expected. Can you test it and fix the bug?
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
There was a problem hiding this comment.
This exercise expects you to explain how the expression on line 4 works.
|
|
||
| //step-1---> Math.random() generates a random decimal. | ||
| //step-2---> (maximum - minimum + 1) evaluates to 100. | ||
| //step-3---> Math.random() is multiplied by (maximum - minimum + 1) which is 100, resulting in a random decimal between 0 and 100. |
There was a problem hiding this comment.
Phrases like "a number between X and Y" are not precise enough in a program specification, because they do not clearly state whether the endpoints X and Y are included.
We can also use the concise and precise interval notation to describe a range of values.
[,]=> inclusion(,)=> exclusion
For example, we can describe a number, -5 < x <= 10, as "x is a number in (-5, 10]".
Can you use this notation to describe the return value of Math.random() (step-1) and the value produced in step-3?
| // Line 1 is a variable declaration, creating the count variable with an initial value of 0 | ||
| // Describe what line 3 is doing, in particular focus on what = is doing | ||
| // line 3 returns the incurmented value of count after adding 1 to it and assigns this new value back to count No newline at end of file | ||
| // line 3 is an assignment which returns the incurmented value of count after adding 1 to it. No newline at end of file |
There was a problem hiding this comment.
I was referring to the operation that increase the value of a variable by 1. For example, count = count + 1.
Can you look up the name of such operation?
Manchester | 25-ITP-Sep | Hani Sadah| Sprint 1 | Coursework/sprint 1
Self checklist
Changelist
This is the answered version of sprint 1
Hani Sadah