-
Notifications
You must be signed in to change notification settings - Fork 427
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Remove support for repeated parameters in commands
If a single-valued parameter is repeated multiple times in a command, only the last occurrence is taken. This allows the user to correct a mistake in a parameter value by simply typing it again, which may be faster than moving the cursor back to the original parameter values and editing it. This feature serves as a good example of 'going the extra mile to optimize the application for target users' (i.e., users who can type fast). However, this feature can cause certain user errors to go unnoticed e.g., unintended repetition caused by a typo in parameter name Alternatives considered: (a) Retain current behavior but give a friendly warning to the user that a parameter was 'overridden'. (b) Remove this feature i.e., treat repeated single-valued parameters as an error, and reject the command with an error message. Alternative (a) requires significantly more complicated code (see #181), steepening the learning curve for for new programmers. Given that this application is primarily meant for training novice programmers, the impact on the learning curve is not worth the benefit of retaining this feature. Therefore, let's do option (b).
- Loading branch information
1 parent
bd48d59
commit 3d2c2df
Showing
7 changed files
with
144 additions
and
50 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters