Clarify instructions for bird-watcher, exercise 3 #2834
Closed
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.
The description for Problem 3 of the Bird Watcher go exercise states,
The the Pearson Education dictionary of contemporary English's defines "every second" to mean:
Thus, applied to the problem, "every second day" means that the 1st day the bird was not in the garden, the 2nd day it was, the 3rd day it was not, the 4th day it was, and so on and so forth.
bird_watcher_test, however, is coded to test something different: It expects that the 1st day, the bird was in the garden, the 2nd day it was not, the 3rd day it was, the 4th day it was not.
Other dictionaries either confirm this, or point out that the wording currently provided is ambiguous, either of which is undesirable in an exercise such as this. @IsaacG was kind enough to provide an example of a conflicting dictionary entry pointing out the ambiguity of the current wording here. Thank you, Isaac!
Thus, this PR disambiguates the instructions such that everybody is happy :)