Skip to content
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

Implementation of client app and server app #27

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

Conversation

erisevaline
Copy link

Unit tests are implemented too.

Arthur Becaud & Bruno Egremy

bruno and others added 30 commits March 18, 2020 14:39
They might be changed in the near future.
The client and server addresses need to be chosen.
Added pom.xml to git.
Changed Protocol.java a bit.
Undo merge
This reverts commit aa82666.
…a, modified pom.xml to remove junit version 5
Merging Dev into Master
…rovement

Number format :
Client and Server couldn't process number with a comma (ex: 5,7).
Correction done in Client side by replacing comma by point in user input.
Correction done in Server side by adding symbols parameters in format type.

Client type result :
Client::sendCalculationToCompute was returning Integer inside the code while
the prototype was returning Double.

User interface improvement :
Simplification of messages and improvement in user input.
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