OscfPhase2 #34
Open
OscfPhase2 #34
Conversation
This file contains hidden or 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
…t custom. Made a class variable loginId to implement the loginId for the client. Modified the constructor to add the client loginId and with a try and catch. Also made changes to the args(function) for the client, in order recognize different inputs.
…r for processing data. Added a class varaible loginId to keep track of it and also to return in the serverUI. Modified the handleMessageFromClient method to make changes to recieve and record the loginId of the client. Implemented hook methods ClientConnected and ClientDisconnected from the AbstractServer.
…se of the application. Created class variables, logoff and loginId which help setting up the commands. Made appropriate changes to the constructors to implement the variables created. Modified the handleMessageFromClientUI file to implement the commands created. Implemented hook methods from Abstract Client connectionClosed and connectionException.
…ll the commands which the server uses. Created class variables to support the commands created.Modified the display method and added new fucntions to make the main method functional.
…es. These methods helped with the UI and the creation of commands.
…re tested to prove the functionality of the code and the class implementations. There were few test cases which were skipped due to not having an additional functional computer.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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 branch contents the final changes for the Assignment 1. The changes include the class implementations, method implementation and also contains a doc with the result of the test cases Phase 2(simplechat2).