Tp Commit - Question 1

Programmation Avancée

Modérateur : Douin

manuel
Messages : 11
Inscription : 25 oct. 2017 10:33

Tp Commit - Question 1

Message par manuel » 04 déc. 2017 20:42

Bonjour,

Je pensais avoir réussi les test JNEWS pour la question 1 mais une erreur est apparue :
1)test_instanceof_dans_nombreDeCotisants(question1) : l''opérateur instanceof ne doit pas être utilisé???

ce test est nouveau ???? il a été inséré dernièrement ?

_mat_
Messages : 7
Inscription : 05 nov. 2017 11:51

Re: Tp Commit - Question 1

Message par _mat_ » 04 déc. 2017 22:05

Bonsoir,

Oui manuel, on dirait bien.
J'ai repassé les tests JNEWS et j'ai eu 3 erreurs à cause du "instanceof"
Il semblerait qu'elle ne soit pas contrôlée dans la méthode solde().

J'ai réussi à résoudre ce problème grâce au post de Mr Douin :
"Attention ce n'est pas objet si vous devez faire un test sur le noeud du composite ... imaginez que l'on ajoute une dizaine de nouveaux types de noeuds et votre programme devrait contenir au moins 10 tests (instanceof) de plus, laissez la liaison dynamique s'exécuter"
public Void visite(GroupeDeContributeurs g){
for( Cotisant c : g)
c.accepter(this);
}

manuel
Messages : 11
Inscription : 25 oct. 2017 10:33

Re: Tp Commit - Question 1

Message par manuel » 05 déc. 2017 0:29

Salut Mat!

Je te remercie de m'avoir répondu.
Je ne suis pas fou alors, des nouveaux tests dans JNEWS ont donc été ajouté sans qu'on soit prévenus. super!
Je me suis pas pris la tête, j'ai enlevé le instanceof dans la méthode nombreDeCotisants() de la classe GroupeDeContributeurs

Douin
Messages : 201
Inscription : 18 mars 2009 15:46

Re: Tp Commit - Question 1

Message par Douin » 05 déc. 2017 9:27

Bonjour,

En effet les tests jnews sont inscrits dans un processus d'affinement afin que votre solution soit la plus proche de ce qui est attendu, en l'occurrence l'opérateur utilisé induit une erreur de programmation si l'on suit les paradigmes de la POO
des nouveaux tests dans JNEWS ont donc été ajouté sans qu'on soit prévenus. super!
je partage aussi votre conclusion, c'est super comme outil

Bonne journée

Répondre

Qui est en ligne ?

Utilisateurs parcourant ce forum : Aucun utilisateur inscrit et 1 invité