search

matthieu637 / cpp-2a-info

CPP - Prépa des INP - Nancy | Projet Informatique 2ème année
https://matthieu637.github.io/cpp-2a-info/client.Reseau-class.html
MIT License
1 stars 4 forks source link

Ajout limite taille liste_ventes et liste_achats envoyé par le serveur #14

Closed david540 closed 7 years ago

david540 commented 7 years ago

C'est le code que j'ai, celui avec lequel j'ai les issues que j'ai envoyé (je les avais aussi avant modifications)

matthieu637 commented 7 years ago

J'ai essayé de corriger les exceptions que tu as trouvé (9d82199e3f4eca2121c2475aa1381964ffca5c2a), il faut donc que tu adaptes un peu ton code.

matthieu637 commented 7 years ago

Sinon sur le code proposé, tu peux juste définir la valeur par défaut de nbMaxElemListe à 0 : nbMaxElemListe=0 dans la déclaration de la fonction client.py.

david540 commented 7 years ago

Oui ça marche ! Je vais mettre mes modifs dans mon pull request.

david540 commented 7 years ago

Voilà c'est fait. Par contre pour avoir une sous liste des listesAchats et listesVentes j'ai fais un cast avec une linkedList mais il y a peut être moins couteux maintenant.

david540 commented 7 years ago

Voilà j'ai refait quelques modifications, maintenant la complexité de la création de la sous liste est en O(nbMaxAchats) je crois. J'ai fais un seul grand if(userInput.startsWith(ACHATS)) puis j'ai géré les deux cas arguments.length == 2 et arguments.length == 3 ensuite mais du coup j'ai du ajouter pas mal de "else envoyer(out,-4)" donc le code perd en lisibilité, je sais pas ce qui est le mieux J'ai aussi ajouter des else envoyer(out,-4) là ou il n'y en avait pas (mais bon c'était dans les cas ou les demandes n'étaient pas faites par client.py donc ça ne posait pas de problèmes)

matthieu637 commented 7 years ago

C'est bien comme ça, merci. A terme, le code devrait être séparé dans différentes classes pour chaque requête, mais ce n'est pas très important.