Developer Guide
Table of Contents
- Table of Contents
- Acknowledgements
- Setting Up and Getting Started
- Design
- Implementation
- Documentation, Testing, Logging, Configuration, Dev-Ops
- Appendix: Requirements
- Appendix: Instructions for Manual Testing
- Appendix: Effort
Acknowledgements
- Libraries used: JavaFX, Jackson, JUnit5
- Original project: AddressBook Level-3 Project created as part of the SE-EDU initiative
- Application logo: Inspired by Source Academy
- Code snippet for getting jar file directory: Taken from this Stackoverflow post
- PlantUML sprite for rake symbol: Taken from this PlantUML forum post
Setting Up and Getting Started
Refer to the guide Setting Up and Getting Started.
Design
.puml
files used to create diagrams in this document can be found in the diagrams folder. Refer to the PlantUML Tutorial at se-edu/guides to learn how to create and edit diagrams.
Architecture
The Architecture Diagram given above explains the high-level design of Source Control.
Given below is a quick overview of main components and how they interact with each other.
Main components of the architecture
Main
has two classes called Main
and MainApp
. It is responsible for,
- At app launch: Initializes the components in the correct sequence, and connects them up with each other.
- At shut down: Shuts down the components and invokes cleanup methods where necessary.
Commons
represents a collection of classes used by multiple other components.
The rest of Source Control consists of four components.
-
UI
: The UI. -
Logic
: The command executor. -
Model
: Holds the data of Source Control in memory. -
Storage
: Reads data from, and writes data to, the hard disk.
How the architecture components interact with each other
The Sequence Diagram below shows how the components interact with each other for the scenario where the user executes the command delete 1
.
Each of the four main components (also shown in the diagram above),
- defines its API in an
interface
with the same name as the Component. - implements its functionality using a concrete
{Component Name}Manager
class (which follows the corresponding APIinterface
mentioned in the previous point.
For example, the Logic
component defines its API in the Logic.java
interface and implements its functionality using the LogicManager.java
class which follows the Logic
interface. Other components interact with a given component through its interface rather than the concrete class (reason: to prevent outside component’s being coupled to the implementation of a component), as illustrated in the (partial) class diagram below.
The sections below give more details of each component.
UI component
The API of this component is specified in Ui.java
The UI consists of a MainWindow
that is made up of parts e.g.CommandBox
, ResultDisplay
, StudentListPanel
, StatusBarFooter
etc. All these, including the MainWindow
, inherit from the abstract UiPart
class which captures the commonalities between classes that represent parts of the visible GUI.
The UI
component uses the JavaFX UI framework. The layout of these UI parts are defined in matching .fxml
files that are in the src/main/resources/view
folder. For example, the layout of the MainWindow
is specified in MainWindow.fxml
The UI
component,
- executes user commands using the
Logic
component. - listens for changes to
Model
data so that the UI can be updated with the modified data. - keeps a reference to the
Logic
component, because theUI
relies on theLogic
to execute commands. - depends on some classes in the
Model
component, as it displaysStudent
object residing in theModel
.
Logic component
API : Logic.java
Here’s a (partial) class diagram of the Logic
component:
How the Logic
component works:
- When
Logic
is called upon to execute a command, it uses theSourceControlParser
class to parse the user command. - This results in a
Command
object (more precisely, an object of one of its subclasses e.g.AddCommand
) which is executed by theLogicManager
. - The command can communicate with the
Model
when it is executed (e.g. to add a student). - The result of the command execution is encapsulated as a
CommandResult
object which is returned back fromLogic
.
The Activity Diagram below illustrates how user input is parsed by the Logic
component.
The Sequence Diagram below illustrates the interactions within the Logic
component for the execute("delete 1")
API call.
DeleteCommandParser
should end at the destroy marker (X) but due to a limitation of PlantUML, the lifeline reaches the end of diagram.
Here are the other classes in Logic
(omitted from the class diagram above) that are used for parsing a user command:
How the parsing works:
- When called upon to parse a user command, the
SourceControlParser
class creates anXYZCommandParser
(XYZ
is a placeholder for the specific command name e.g.,AddCommandParser
) which uses the other classes shown above to parse the user command and create aXYZCommand
object (e.g.,AddCommand
) which theSourceControlParser
returns back as aCommand
object. - All
XYZCommandParser
classes (e.g.,AddCommandParser
,DeleteCommandParser
, …) inherit from theParser
interface so that they can be treated similarly where possible e.g, during testing.
Model component
API : Model.java
The Model
component,
- stores the application data i.e. all
Student
objects (which are contained in aUniqueStudentList
object). - stores the currently ‘selected’
Student
objects (e.g. results of a search query) as a separate filtered list which is exposed to outsiders as an unmodifiableObservableList<Student>
that can be ‘observed’ e.g. the UI can be bound to this list so that the UI automatically updates when the data in the list change. - stores a
UserPref
object that represents the user’s preferences. This is exposed to the outside as aReadOnlyUserPref
objects. - does not depend on any of the other three components (as the
Model
represents data entities of the domain, they should make sense on their own without depending on other components)
Tag
list in the SourceControl
, which Student
references. This allows SourceControl
to only require one Tag
object per unique tag, instead of each Student
needing their own Tag
objects. Note that other classes such as Group
and Assessment
are omitted here for brevity.Storage component
API : Storage.java
The Storage
component,
- can save both application data and user preference data in json format, and read them back into corresponding objects.
- inherits from both
SourceControlStorage
andUserPrefStorage
, which means it can be treated as either one (if only the functionality of only one is needed). - depends on some classes in the
Model
component (because theStorage
component’s job is to save/retrieve objects that belong to theModel
)
Common classes
Classes used by multiple components are in the seedu.sourcecontrol.commons
package.
Implementation
This section describes some noteworthy details on how certain features are implemented.
Adding a student
The addstudent
feature adds a student with the provided name and NUSNET ID into the database. If the student comes with optionally specified groups and tags, these fields will be added accordingly.
How the addstudent
command works
- The user specifies the student name, NUSNET ID, and if applicable, the groups and tags belonging to the student.
- If the name or NUSNET ID is not provided, the user will be prompted to enter them via an error message.
- The ID is cross-referenced with the current students in the database, and an error is thrown if the student to add has the same ID as a pre-existing student.
- If the student to add has a unique NUSNET ID, the groups, if provided, will be parsed individually.
- If the provided group exists, the student will be added into that group.
- If the provided group does not exist, a new group will be added, and the student will be added into that group subsequently.
- A new
Student
object is created with the given name, NUSNET ID, groups, and tags, which is then added into the database.
The following activity diagram summarizes what happens when a user executes the addstudent
command to add a new student. In the case where the student is not added, an error message will be displayed with the reason.
The following sequence diagram summarizes what happens when the user inputs an addstudent
command together with the name and NUSNET ID of the student to be added.
AddStudentCommand
should end at the destroy marker (X) but due to a limitation of PlantUML, the lifeline reaches the end of diagram. This is also applicable to other sequence diagrams in this guide.
Creating a new group
The addgroup
feature allows users to create new groups, as well as specify students to be added to the group to be created.
How the addgroup
command works
- The user specifies the group name, as well as a list of names and/or IDs of the students to be added into the group.
- For each of the names and IDs, an
AllocDescriptor
is created. - For each of the
AllocDescriptors
, a search is done against the currentStudentList
to find students that match the descriptors.- If there is one and only one match, the student is added to the group.
- The group is added to the application if Step 3 completes without any exceptions.
The following activity diagrams summarizes what happens when a user executes the addgroup
command to add a new group. In the case where the group is not added, an error message will be displayed with the reason.
The following sequence diagram summarizes what happens when the user inputs an addgroup
command together with a student to be added.
Adding a student into a group
The addalloc
feature allows users to allocate a student into a group.
How the addalloc
command works
- The user specifies the group name, and the name or ID of the student to be allocated into the group.
- An
AllocDescriptor
containing info of the group and the student is created. - The
AllocDescriptor
is used to find the group and the student(s) as specified.- If there is only one matched student, the student is added to the group.
- If there are multiple matched students, the allocation is not made successfully, and the student list is updated with all matched students.
- The student is allocated into the group.
The following activity diagram summarizes what happens when a user executes the addalloc
command to allocate a student into a group. In the case where the student is not added into the group, an error message will be displayed with the reason.
The following sequence diagram summarizes what happens when the user inputs an addalloc
command together with a group and a student, specified by name, to be allocated.
Adding a score
The addscore
feature allows users to add score for an assessment of a student.
How the addscore
command works
- The user specifies the assessment name, the name or ID of the student, and the score to be added.
- An
ScoreDescriptor
containing info of the group, the student and the score is created. - The
ScoreDescriptor
is used to find the assessment and the student(s) as specified.- If there is only one matched student, the assessment of the student will be updated with the new score.
- If there are multiple matched students, the update is not made successfully, and the student list is updated with all matched students.
- The score is updated in the assessment of the student.
The following activity diagram summarizes what happens when a user executes the addscore
command to add score for an assessment of a student. In the case where the score is not added/updated, an error message will be displayed with the reason.
The following sequence diagram summarizes what happens when the user inputs an addscore
command together with an assessment, a student, specified by name, and a score to be added.
Searching for students
The search
feature allows user to filter student list by name, NUSNET ID, groups, or tags.
How the search
command works
The following sequence diagram summarizes what happens when the user inputs an search
command together with a name to be searched for.
A Predicate<Student>
object will be created for each search command.
It contains test(Student student)
function which checks if the given student matches the list of keywords given.
To support the differentiated search functionality for different flags (e.g. name, NUSNET ID, group name, or tag),
multiple classes extending from Predicate<Student>
can be created,
each with different implementation of the test(Student student)
function.
-
NameContainsKeywordsPredicate
: checks if any word in the full name of student matches exactly any word in the given keywords. e.g.Alex Yu
will matchAlex Yeoh
andBernice Yu
. Partial search is not supported e.g.Han
will not matchHans
. -
IdContainsKeywordsPredicate
: checks if the ID of student contains any word in the given keywords. Partial search is supported. e.g.E05
will matchE0523412
. -
GroupContainsKeywordsPredicate
: checks if any group of student contains any word in the given keywords. Partial search is supported. e.g.T02
will matchT02A
andT02B
. -
TagContainsKeywordsPredicate
: checks if the tag of student contains any word in the given keywords. Partial search is supported. e.g.beginner
will matchbeginners
.
The following activity diagrams summarizes what happens when a user executes the search
command to search for students with different filters.
Importing data
The import
feature allows users to load data as specified in the provided CSV file.
How the import
command works
The following activity diagram summarizes what happens when a user executes the import
command to import a CSV data file. In the case where the file is not imported, an error message will be displayed with the reason.
There are several important details left out of the activity diagram for the sake of clarity:
-
The import feature is reliant on having a correctly formatted csv file (which is to be exported from sites like LumiNUS and Source Academy, and modified to fit the format). The user needs to provide the number of
Groups
,Assessments
, andTags
since we can’t detect this automatically from the format of the file. The proper format of the file can be found in the user guide. -
A
CommandException
will be thrown if any input does not follow the formatting specified in the respective classes such asName
,ID
, andScore
. -
When reading a student’s groups, the command will try to use an existing
Group
if possible, to ensure that theGroup
holds a reference to allStudents
in the group. A newGroup
will only be created in the case where the group hasn’t already been created. -
When reading a student’s scores, the command will add the score to the
Student
, as well as theAssessment
created from reading the first row. -
Columns can be empty, except for the assessment name columns in the header row, and the name and ID columns of each student. Empty columns are assumed to be missing data.
Showing assessment result analysis
The show
feature allows users to show the performance analysis of a student, a group or the cohort in an assessment.
How the show
command works
- The user specifies the student (by either name, ID or index), the group name or the assessment name.
- An
ScoreDescriptor
containing info of the group, the student and the score is created. - The
ScoreDescriptor
is used to find the assessment and the student(s) as specified.- If there is only one matched student, the assessment of the student will be updated with the new score.
- If there are multiple matched students, the update is not made successfully, and the student list is updated with all matched students.
- The performance analysis of the student, the group or the cohort in an assessment is displayed.
The following activity diagrams summarize what happens when a user executes the show
command to show the performance analysis of a student, a group or the cohort in an assessment. In the case where the display is not presented successfully, an error message will be displayed with the reason.
The following sequence diagram summarizes what happens when the user inputs an show
command together with a student specified by name.
Setting customized aliases for commands
The alias
feature allows users to define their own aliases for commands. This is useful to shorten the input for commands that the user uses often.
An alias is basically just a mapping of a user-provided string to a command word. It works by directly replacing the first word in the user’s input command with the alias word, then parsing again.
An alias contains two strings: An aliasWord
which is the new user-defined word, and a commandWord
which is the command word of an existing default command (e.g. addstudent
, clear
, exit
).
How the alias
command works
Parsing of an alias command follows the following steps:
- The alias word is checked to ensure that it is one word long.
- The alias word is also checked to ensure that it does not overlap with any default command word. This is to prevent the re-mapping of any default command words and potentially losing the functionality of the application.
- The command word is checked for validity by attempting to parse the command word. If the parser does not recognize the command word, it is not valid.
- If the command word is an alias, it is replaced with the command word that the alias maps to.
- The new
Alias
andAliasCommand
is created, and executed. - The alias is added to both the parser, and the model (so that it can be saved in the
UserPrefs
)- If the alias word is already present in the parser, the command word it is mapped to is replaced with the new command word instead.
- If the alias word and the command word are identical, it is assumed that the user is trying to delete an existing alias. The existing alias is removed instead.
If the command word of any future user input matches the alias, the first word of the user input will be replaced by the command word of the alias. The activity diagram of how a command is parsed can be found in the Logic section of this guide.
Design considerations
Aspect 1: How aliases are stored and parsed
There were two ideas on how this could be done:
- Store the command’s
Parser
in each alias. The parser attribute of the alias can then be retrieved andParser::execute
can be called when parsing.- Pros:
- Very elegant and streamlined parsing. Almost no changes need to be made to the logic of
SourceControlParser
.
- Very elegant and streamlined parsing. Almost no changes need to be made to the logic of
- Cons:
- It is not easy to obtain the type of
Parser
simply through the command word provided by the user given the current implementation of the project. You’d need to iterate through all possibleCommands
, and match their command word with the provided word. This approach would not be very extensible if more commands are to be added in the future. - It is also not easy to save/load the alias to/from the
UserPrefs
. The best approach is likely to just save the command word, then generate theParser
when loading. However, this poses a similar problem as the previous point, which is that it is not easy to generate theParser
just from the command word.
- It is not easy to obtain the type of
- Pros:
- Store the command word in each alias. Replace the command word of the user input if the user inputs an aliased command, and parse it again.
- Pros:
- Very easy to store. Each alias is essentially just two strings.
- Cons:
- Have to be careful with the implementation to prevent any edge cases where the user can define aliases to create an infinite loop.
- Pros:
We decided to go with the easier implementation of storing each alias as two strings. However, there were still more aspects to be considered.
Aspect 2: How to handle aliases of aliases
That is, what happens when the user does alias -c <existing_alias> -as <new_alias>
? There are two choices for this:
- The new alias maps to the existing alias word directly.
- The command word stored in the new
Alias
would be the alias word of the existingAlias
. - Assuming that an alias
ag
already exists foraddgroup
, the following commands have the following effects:-
alias -c ag -as addg
would makeaddg
map toag
, which is in turn mapped toaddgroup
. When parsing,addg
will be replaced withag
, which is then replaced byaddgroup
. -
alias -c addalloc -as ag
would makeag
map toaddalloc
.addg
would also then map toaddalloc
, since it gets replaced byag
when parsing, which then gets replaced byaddalloc
.
-
- Pros:
- Intuitive approach to aliases: “I want
addg
to meanag
”.
- Intuitive approach to aliases: “I want
- Cons:
- Possible to make parsing a very expensive task due to having to replace the command word and parsing multiple times.
- Possible to have an infinite loop by mapping two aliases to each other.
- It’s not very useful to have two aliases always mean the same thing.
- The command word stored in the new
- The new alias maps to the command word that the existing alias maps to.
- The command word stored in the new
Alias
would be the command word of the existingAlias
. - Assuming that an alias
ag
already exists foraddgroup
, the following commands have the following effects:-
alias -c ag -as addg
would makeaddg
map toaddgroup
directly. -
alias -c addalloc -as ag
would makeag
map toaddalloc
.addg
would still map toaddgroup
.
-
- Pros:
- Naturally prevents the creation of infinite loops.
- Parsing is guaranteed to only take one recursive call to the
parse
method.
- Cons:
- It is a less intuitive approach to aliases: “I want
addg
to mean whatag
means at the current moment”
- It is a less intuitive approach to aliases: “I want
- The command word stored in the new
We decided to go with implementation 2 due to its ability to naturally handle infinite loops and better performance. Our target audience is also Computer Science professors, who should be very familiar with this style of referencing, since that is exactly how names refer to primitive values in programming.
Aspect 3: Removing aliases
We believe that there needs to be a way to remove aliases. Otherwise, there will eventually be a very large amount of aliases, and some typo might lead to executing a command you didn’t intend to execute. Hopefully, that command isn’t clear
.
Again, there are two choices we could take:
- Make a new command for removing an alias, e.g.
removealias
.- Pros:
- Intuitive to know what the new command does from its command word.
- Cons:
- Might end up having too many commands for a less prominent function like aliasing.
- Takes a little more effort to implement.
- Pros:
- Modify the existing
alias
command to accept a special case to remove aliases.- More specifically,
alias -c <existing_alias> -as <same_alias>
will remove the alias. - Pros:
- Easy to implement: Just check if alias word and command word are the same.
- One command for the whole alias functionality.
- Cons:
- Not particularly intuitive. The
alias
command feels like a command to use for adding aliases, but it is being used to remove one instead.
- Not particularly intuitive. The
- More specifically,
We ended up going with the second approach since the alias functionality was a very small part of the application. We also believe that it does make sense that mapping an alias to itself would remove it.
Furthermore, removing aliases is likely a very rare use case, and dedicating a whole command to it does feel like a bit of a waste.
Documentation, Testing, Logging, Configuration, Dev-Ops
Appendix: Requirements
Product scope
Target user profile:
Targets professors of CS1101S who:
- has a need to manage a significant number of students
- has a need to analyse students’ performance
- prefer desktop apps over other types
- can type fast
- prefers typing to mouse interactions
- is reasonably comfortable using CLI apps
Value proposition: This app will help CS1101S professors keep track of students’ performance after each assessment, doing so faster than a typical mouse/GUI driven app. It can analyse results of individual students, tutorial groups, or the whole cohort in each assessment, in order to identify students who may require additional help.
User stories
Priorities: High (must have) - * * *
, Medium (nice to have) - * *
, Low (unlikely to have) - *
Importing and exporting data:
Priority | As a … | I want to … | So that I can… |
---|---|---|---|
* * * |
CS1101S Professor | Upload large amounts of data from a file | Upload all assessment scores of all students at once |
* * |
Long-time user | Clear all data | Remove records from the previous semester |
* * |
CS1101S Professor | Export data | Share interesting findings with my colleagues |
* |
New user | Import data from the previous semester | Have an idea of how intakes of the previous cohort performed |
Adding and editing data fields:
Priority | As a … | I want to … | So that I can… |
---|---|---|---|
* * * |
CS1101S Professor | Add a new assessment | Keep track of assessment scores of students |
* * * |
CS1101S Professor | Add a new student | Add a new student without having to make a new file |
* * * |
CS1101S Professor | Allocate a student into existing group | Allocate groupings without having to make a new file |
* * * |
CS1101S Professor | Organize students into groups | Encourage peer learning |
* * |
CS1101S Professor | Remove a specific student | Update the system accordingly when a student drops the module |
* * |
CS1101S Professor | Annotate a student with a tag | See categories of students quickly |
* * |
CS1101S Professor | Add remarks to particular students | Be aware of any special conditions the student is facing |
* * |
CS1101S Professor | Edit assessment score for a particular student | Make changes after the initial grading |
* |
CS1101S Professor | Add attendance records for each class | Track the students present at each class |
* |
CS1101S Professor | Upload students’ feedback about their tutors | Provide timely feedback to the tutors |
Viewing and searching data:
Priority | As a … | I want to … | So that I can… |
---|---|---|---|
* * * |
CS1101S Professor | Search for specific student | Track his/her progress to facilitate better learning |
* * * |
CS1101S Professor | Search for students by classes and groups | Track them by groups easily |
* * * |
CS1101S Professor | Check a student’s grades | See individual performance |
* |
CS1101S Professor | Check attendance records for each student | Track if the student has been attending classes diligently |
Analysing data:
Priority | As a … | I want to … | So that I can… |
---|---|---|---|
* * * |
CS1101S Professor | Analyse the performances of individual students | Give special assistance to those in need |
* * * |
CS1101S Professor | Analyse performance of cohort in each assessment | See how the cohort is performing |
* * |
CS1101S Professor | Analyse the performances of students in groups | See which studio is under-performing and check in with the tutor |
* * |
CS1101S Professor | Calculate overall grades | Easily decide on grade ranges |
* |
Detailed user | View the performances under different kinds of graph | Have better visualisation about the performances of students |
* |
CS1101S Professor | Analyse the cohort performance for each question | Understand which are the topics students require more help with |
* |
CS1101S Professor | Compare between different batches of students | See if the module is too hard this semester as compared to previous semesters |
* |
CS1101S Professor | Check the overall performance of the tutors based on several indicators | Identify excellent tutors to be called back next semester |
Others:
Priority | As a … | I want to … | So that I can… |
---|---|---|---|
* * * |
New user | Get command prompts when typing | Do not have to keep checking user guide |
* * |
Expert user | Have shortcuts for commands | Save time |
* * |
Potential user | View the app populated with sample data | See how the app looks like when in use |
* |
Forgetful user | Access the user guide with an easy to remember command | Lookup how to use a command/what command to use |
* |
Long-term user | Store meeting timings | Not miss any meetings that have been planned |
* |
Forgetful user | Have reminders about upcoming meetings | Avoid missing any important events |
* |
CS1101S Professor | Add TODO bug fixes accumulated throughout the semester | Fix them during CP3108 |
Use cases
(For all use cases below, the System is Source Control
and the Actor is the user
, unless specified otherwise)
Use case: Import student roster
MSS
- User has a comma-separated values (
.csv
) file of the student roster. - User requests to import the file into Source Control.
- Source Control shows the list of students parsed and imported. Use case ends.
Extensions
- 2a. User enters an incorrectly formatted csv file.
- 2a1. Source Control shows an error message.
- 2a2. User fixes the csv file. Use case resumes at step 2.
Use case: Create a new group
MSS
- User requests to create a new group and enters the group name and optionally the students’ names or NUSNET IDs.
- Source Control creates the group with the specified students. Use case ends.
Extensions
- 1a. User enters an invalid group name.
- 1a1. Source Control shows an error message. Use case resumes at step 1.
- 1b. User enters a group name which coincides with an existing group in the database.
- 1b1. Source Control shows an error message. Use case resumes at step 1.
- 1c. User enters a student name which match with multiple students in the Source Control database.
- 1c1. Source Control shows the list of students with matching names, and prompts the user to resolve the conflict by specifying the target student’s NUSNET ID instead. Use case resumes at step 1.
- 1d. User enters a list of student name contains duplicated students.
- 1d1. Source Control shows an error message to inform user of the duplicates. Use case resumes at step 1.
Use case: Showing a student’s performance
MSS
- User requests to view the performance of a specified student by providing the index of the student in the list, or the student’s name or NUSNET ID.
- Source Control displays a line chart showing the specified student’s performance against the cohort performance for each assessment. Use case ends.
Extensions
- 1a. User enters a student name or ID that does not exist in the database.
- 1a1. Source Control shows an error message. Use case resumes at step 1.
- 1b. User enters an invalid index.
- 1b1. Source Control shows an error message. Use case resumes at step 1.
Use case: Showing a group’s performance
MSS
- User requests to view the performance of a specified group by providing the group name.
- Source Control displays a line chart showing the specified group’s performance against the cohort performance for each assessment. Use case ends.
Extensions
- 1a. User enters a group name that does not exist in the database.
- 1a1. Source Control shows an error message. Use case resumes at step 1.
Use case: Showing cohort performance for an assessment
MSS
- User requests to view the cohort performance for a specified assessment.
- Source Control displays a histogram showing the score distribution of the cohort in the specified assessment. Use case ends.
Extensions
- 1a. User enters an assessment that does not exist in the database.
- 1a1. Source Control shows an error message. Use case resumes at step 1.
Use case: Creating an alias for an existing command
MSS
- User requests to create an alias for a specified existing command.
- Source Control accepts the user-created alias.
Extensions
- 1a. User enters a command that does not exist.
- 1a1. Source Control shows an error message. Use case resumes at step 1.
- 1b. User tries to create an alias using a default command.
- 1b1. Source Control shows an error message. Use case resumes at step 1.
- 1c. User enters an invalid alias name.
- 1c1. Source Control shows an error message. Use case resumes at step 1.
Non-functional requirements
- Should work on any mainstream OS as long as it has Java
11
or above installed. - Should be able to hold up to 1000 students without a noticeable sluggishness in performance for typical usage.
- Should be able to analyse data of up to 1000 students without a noticeable sluggishness in performance for typical usage.
- A user with above average typing speed for regular English text (i.e. not code, not system admin commands) should be able to accomplish most of the tasks faster using commands than using the mouse.
- Should work without internet connections.
- New users can pick up the basic functionalities of Source Control easily.
Glossary
- Mainstream OS: Windows, Linux, Unix, OS-X.
- Student: A student in the database, identified by their name and NUSNET ID. Each student can be in multiple groups, and can have scores for multiple assessments.
- Group: A group of students, identified by the group name.
- Assessment: An assessment is identified by the assessment name.
- Score: The score that a student has attained for an assessment, stored in percentage and can be recorded up to 2 decimal places. Each student can only have 1 score per assessment.
- Student list: The list of students displayed on the right panel of Source Control. Student list can be filtered to display selected students only.
-
Flag: Arguments flags are used to indicate different types of user inputs e.g.
-n
for student name, and-g
for group. More about flags can be found here.
Appendix: Instructions for Manual Testing
Given below are instructions to test the app manually.
Launch and shutdown
-
Initial launch
-
Download the jar file and copy into an empty folder.
-
Double-click the jar file Expected: Shows the GUI with a set of sample contacts. The window size may not be optimum.
-
-
Saving window preferences
-
Resize the window to an optimum size. Move the window to a different location. Close the window.
-
Re-launch the app by double-clicking the jar file.
Expected: The most recent window size and location is retained.
-
-
Shutdown
- Test case:
exit
Expected: Source Control exits and shut down automatically.
- Test case:
Adding a group
-
Adding a group successfully.
-
Prerequisites: Group should not already exist in database. Database contain a student with name
Hong Fai
and another studentA
with IDE0123456
. -
Test case:
addgroup -g T01A
Expected: GroupT01A
will be added to the database. -
Test case:
addgroup -g T02B -n Hong Fai -i E0123456
Expected: GroupT02B
will be added to database with studentsHong Fai
andA
in the group.
-
-
Adding a group with incorrect formats.
-
Test case:
addgroup
Expected: No group is created. Error detail shown in the status message to inform user of the correct command format. -
Test case:
addgroup -g
Expected: No group is created. Error detail shown in the status message to inform user that group name cannot be blank. -
Test case:
addgroup -g Tutorial@Wednesday
Expected: No group is created. Error detail shown in the status message to inform user that group name must be alphanumeric.
-
-
Adding a group that already exists in database.
-
Prerequisite: Group specified already exists in database.
-
Test case:
addgroup -g T01A
Expected: The group will not be re-created. Error details shown in the status message to inform user group already exists.
-
-
Adding a group with duplicated student instances.
-
Prerequisites: Database contain a student with name
Hong Fai
with the IDE0123456
. -
Test case:
addgroup -g T01A -n Hong Fai -i E0123456
Expected: The group will not be created. Error detail shown in the status message to inform user that the studentHong Fai
has been specified more than once.
-
-
Adding a group with non-existent students.
-
Prerequisites: Database does not contain a student with name
Hong Fai
. -
Test case:
addgroup -g T01A -n Hong Fai
Expected: The group will not be created. Error detail shown in the status message to inform user that the studentHong Fai
cannot be found in the database.
-
Deleting a Student
-
Deleting a student while a list of all students are being shown.
-
Prerequisites: List all students using the
list
command. Some students are displayed in the list. -
Test case:
delete 1
Expected: First student is deleted from the list. Details of the deleted student is shown in the status message. Timestamp in the status bar is updated. -
Test case:
delete 0
Expected: No student is deleted. Error details shown in the status message. Status bar remains the same. -
Other incorrect delete commands to try:
delete
,delete x
(where x is an invalid number)
Expected: Similar to previous.
-
-
Deleting a student after searching.
-
Prerequisites: List selected students using the
search
command. Some students are displayed in the list. -
Test case:
delete 1
Expected: First student displayed on the sorted list is deleted. Details of the deleted student is shown in the status message. Timestamp in the status bar is updated. -
Other incorrect delete commands to try:
delete
,delete x
(where x is an invalid number)
Expected: Similar to previous.
-
Adding an alias
-
Adding an alias successfully.
-
Test case:
alias -c addstudent -as example
Expected: A new aliasexample
is added foraddstudent
.example -n Zhiying -i E1234567
will add studentZhiying
to the database. -
Test case:
alias -c example -as example2
Expected: A new aliasexamples
is added foraddstudent
(the commandexample
is mapped to). -
Test case:
alias -c addgroup -as example
Expected: The aliasexample
is mapped toaddgroup
and no longer representaddgroup
.example -g T02A
will create a new groupT02A
.
-
-
Deleting an alias successfully.
-
Prerequisites: An alias
example
exists. -
Test case:
alias -c example -as example
Expected: Removes the aliasexample
.example
will no longer be recognized as a command.
-
-
Deleting an alias unsuccessfully.
- Test case:
alias -c addstudent -as addstudent
Expected: The aliasaddstudent
will not be removed. Error detail shown in the status message to inform user that default command cannot be overwritten.
- Test case:
-
Adding an alias unsuccessfully.
-
Test case:
alias -c addgroup -as addstudent
Expected: No alias will be created. Error detail shown in the status message to inform user that default command cannot be overwritten. -
Test case:
alias -c addstudent -as add student
Expected: No alias will be created. Error detail shown in the status message to inform user that alias can only be one alphanumeric word.
-
Saving data
-
Dealing with missing data file
-
Prerequisites:
./data/sourceControl.json
(or the file path specified inpreferences.json
) has been deleted. -
Test case: Run the application.
Expected: Application is populated with sample data../data/sourceControl.json
will be created after exiting the application.
-
-
Dealing with corrupted data file
-
Prerequisites: Deleted the data file at
./data/sourceControl.json
, then launched the jar file to populate the data file with sample data. -
Test case: Add an
*
to thename
of the first person in the data file and launch the jar file.
Expected: No data will be loaded into the application. The data file will lose all its data when the app exits. -
Test case: Add the group
new group
into thegroups
array of the first person and launch the jar file.
Expected: The new group is not added to the first person. The data file will no longer contain the new group when the app exits. -
Test case: Add the assessment
new assessment
into theassessments
array of the first person and launch the jar file.
Expected: The new assessment is not added to the first person. The data file will no longer contain the new assessment when the app exits. -
Test case: Add an extra
{
as the first character of the data file.
Expected: No data will be loaded into the application. The data file will lose all its data when the app exits.
-
Appendix: Effort
In general, we believe that we have put in a lot of effort for this project. We managed to implement the target features we originally set out to do, plus a few extra nice-to-have features.
We also managed to keep the code coverage reasonably high at 76%, and there was only a small set of unique bugs found in the PE-D.
Challenges faced
The following features were relatively hard to implement:
- JavaFX Charts are a part of the JavaFX library which we had to learn how to use from scratch. However, it was an essential part of our goals for the application.
- It was not easy to customize the chart styles (such as the colour scheme) to fit with the overall design of the application.
- Many things about the format of the charts were difficult to customize, such as the direction and layout of axis labels.
- Aliases were deceptively hard to implement, and took more time than expected.
- Even though Jonas implemented an alias feature for his iP, the implementation had to be completely different since the parser was using a completely different design.
- Many aspects had to be considered and carefully weighed. You can read more about it in the alias section of this guide.
- Adapting the AB3 code to fit into the context of Source Control.
- While adding and removing classes wasn’t too bad, adapting the tests was on a whole new level.
- There were a lot of tests that we didn’t want to delete, and instead chose to adapt them as well. This proved to be highly time-consuming, since you had to read through the test code and completely understand what it was doing in order to properly adapt it.
- Our application has three different classes (
Student
,Assessment
,Group
) that are highly coupled to each other. AB3 did not have such problems as it only contained onePerson
class. It was difficult to ensure that our objects are sharing information with each other properly, while trying to keep the level of coupling as low as possible. This coupling also proved to be the source of several hard-to-find bugs.
Features scrapped
The following features were scrapped due to the high difficulty or error-prone nature of their implementations.
- Allowing the user to specify their file path to export data or graphs
- From the feedback received from the PE-D, we realized how easy it was to exploit the ability to specify your file path through user input.
- One problem was how different OSes handled file paths. Some paths were valid only on some OSes, and checking the user input was non-trivial.
- Java’s
Path
andFile
felt insufficient to properly check all possible user inputs for validity. Through testing, it seemed like some invalid file names were still leaking through any checks we had implemented. - These problems were discovered too late to settle with a good, bug-free solution. As such, we decided to drop the ability for users to specify files to write to.
- While one solution was to use the
FileChooser
class from JavaFX, that would violate the requirements of the application being CLI-based.
- Commands longer than one word in length
- Initially,
addstudent
wasadd student
(the same goes for other similar commands in theadd
family). This required some minor tweaking of the parser which was relatively straightforward. - However, when it came to implementing aliases, we realized that it was difficult to ensure that the aliases don’t overlap with the multi-worded commands without drastically changing the parser.
- Thankfully, the simple solution was to just enforce everything to be one word long.
- Initially,
- Allowing non-alphanumeric characters in student names
- As Source Control stores the official names of students, we had to consider that these names could contain non-alphanumeric characters, such as “Mary-Ann Tan”. This is in contrast with AB3, where names of contacts do not need to be official names and these non-alphanumeric characters can be left out.
- However, allowing non-alphanumeric characters in student names would lead to problems such as not being able to identify invalid names like “@@@”. It also leads to complications with parsing, for example when “-g” is meant to be part of the student name but would be parsed as an argument.
- As such, given the time constraint, we decided to maintain the original functionality of AB3, and only allow alphanumeric characters.