-
Notifications
You must be signed in to change notification settings - Fork 11
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
Mon foobarqix étendu #2
base: master
Are you sure you want to change the base?
Conversation
Test de pull request !
…'ajout des critères de divisibilité
…'ajout des critères de divisibilité ->couverture de tests effectuée
Est ce que tu peux ajouter un README avec les instructions pour lancer le projet: ligne de commande, version de Java ou autre à installer ... |
Salut, Ok. Et je dois ajouter un commit pour faire simplement ce que tu demandes dans l'exercice. :) => Afficher de 1 à 100 Bon, encore une fois, je suis complètement hors sujet, mais je me suis amusé... Je te fais ça ce midi car je ne parviens pas à le faire du boulot. (erreur 401 sur mon push?) |
hou la la, te presses pas, t'as encore le temps ! un petit goal maven run pour faciliter enore plus la vie, tu crois que c'est possible ;-) ? |
Ok, je te ferai tout ça, j'en profite pour apprendre à utiliser maven et git, dinosaure que je suis! |
Ajout d'un readme. Ajout d'un goal exec
Et voila. |
Salut,
Bon... Je me suis laché, j'ai voulu aller au delà et faire en sorte que l'algo fonctionne peu importe la taille du nombre via la mise en place de critères de divisibilité. Ca a multiplié la quantité de code par 4...
Pour une version correspondant à la demande, il faut revenir au commit "String optimization". :)
Je n'ai pas pris le temps de faire une gestion de logs, ni de contrôler le taux de couverture.
L'exécution de la classe FooBarQix permet d'utiliser l'algo via la console.
La TestSuite globale est FooBarQixTestSuite.