Skip to content

OscfPhase2 #34

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

NutanNimkar
Copy link

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).

…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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant