generated from mwc/lab_iteration
29 lines
1014 B
Markdown
29 lines
1014 B
Markdown
+++
|
|
method = "completion"
|
|
score = "complete"
|
|
+++
|
|
|
|
Your tile is lovely! This would be fun to draw on a pen plotter...
|
|
|
|
In your ranges, you went to some trouble to include the maximum in the
|
|
range (e.g. you interpreted "all numbers from 0 to 100" to include 100).
|
|
This is a reasonable interpretation, but the convention in CS
|
|
is that the lower number in a range is always included and the upper number
|
|
is never included--matching the behavior of `range`:
|
|
|
|
>>> for i in range(2, 5):
|
|
... print(i)
|
|
2
|
|
3
|
|
4
|
|
|
|
This is just a convention, not something fundamentally right or wrong,
|
|
but it ends up being convenient. Two nice properties:
|
|
|
|
- The size of the range is equal to `maximum - minimum`.
|
|
- If you connect adjacent ranges (e.g. `range(0, 10)` and `range(10, 20)`),
|
|
every number will be included exactly once.
|
|
|
|
Therefore, I would replace `maximum + 1` with `maximum` in your code--and I'd definitely
|
|
be consistent (`print_even_numbers` includes maximum, but `print_odd_numbers` does not).
|