Skip to content

Commit f0de68e

Browse files
committed
Adding comments to todolist solution.
1 parent d4f5ebe commit f0de68e

2 files changed

Lines changed: 5 additions & 9 deletions

File tree

source/projects/bookmarker.md

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -141,8 +141,6 @@ Note that it is ok if you cannot reproduce this output *exactly*.
141141
Please, follow our [guideline on project submission](./projects/submission).
142142
In particular, make sure you write your name and the date in a delimited comment at the beginning of your file.
143143

144-
<!--
145-
146144
## Solution
147145

148146
A possible solution is shared [in this archive](./code/projects/Bookmarker.zip):
@@ -159,4 +157,3 @@ A solution completing the bonus
159157
> "Have your class handle strings, so that, for example, your Bookmarker constructor would take three strings as input"
160158
161159
is shared [in this archive](./code/projects/Bookmarker_with_strings.zip).
162-
-->

source/projects/todo_list.md

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -89,13 +89,12 @@ In particular, make sure you write your name and the date in a delimited comment
8989

9090
### Simplest Solution
9191

92-
A possible solution, using arrays but not resizing them, is as follows:
92+
A possible solution, using arrays but not resizing them, is [in this archive](./code/projects/TodoList_Array.zip).
93+
This solution could be improved by:
9394

94-
```
95-
!include code/projects/TodoList_Array/TodoList/Program.cs
96-
```
97-
98-
You can [download it here](./code/projects/TodoList_Array.zip)
95+
- using a `MAXSIZE` constant variable to hold the maximum number of elements in the todo list instead of "hard-coding" the value 100,
96+
- making sure that the user cannot enter more than `MAXSIZE` number of elements,
97+
- fix the incorrect behavior when the user enters the same number twice: can you determine what would happen with the current implementation, and why it is incorrect?
9998

10099
### Using Classes
101100

0 commit comments

Comments
 (0)