Discussion :

[OmistuSamaIsCoding]

Bonjour,

On me demande quel est le problème sur cette classe et je n'arrive pas à trouver les problèmes qu'on peut rencontrer.
Quelqu'un pour m'aider? Merci d'avance

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public static Integer convertToInteger(Object unObjet) {
if (unObjet instanceof Integer) {
return (Integer)unObjet ;
}
if (unObjet instanceof String) {
try {
Integer i = Integer.parseInt((String)unObjet);
return i ;
} catch(NumberFormatException e) {
return 0 ;
}
}
return 0 ;
}

[Badshade23]

Salut,
Pour moi ta méthode fonctionne ....
As-tu un énoncé ou autre ? Une description du rôle ou du fonctionnement de ta méthode ?
Le fonctionnement attendu etc...

[OmistuSamaIsCoding]

Salut,
Pour moi ta méthode fonctionne ....
As-tu un énoncé ou autre ? Une description du rôle ou du fonctionnement de ta méthode ?
Le fonctionnement attendu etc...

Salut voici l'énoncé :
Vous avez à votre disposition la classe suivante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
classe Assert
public à portée de classe :
// affiche le message d’erreur si la proposition est fausse
// ou rien si la proposition est vraie estVrai
(proposition : booléen, messageErreur : chaîne)

Vous avez écrit en Java la fonction suivante qui doit permettre de convertir un objet en Integer. Le but est de
retourner 0 si l’objet est null quel que soit son type, ou s’il n’est pas de type Integer ou String. Dans le cas
d’un objet de type Integer, il est retourné sans changement. Dans le cas d’un objet de type String, une tentative
de conversion en Integer est faite (avec Integer.parseInt qui tente cette conversion). Si la conversion de
String en Integer n’est pas possible, il faut retourner 0 (le catch s’exécute si la conversion n’est pas possible).

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public static Integer convertToInteger(Object unObjet) {
if (unObjet instanceof Integer) {
return (Integer)unObjet ;
}
if (unObjet instanceof String) {
try {
Integer i = Integer.parseInt((String)unObjet);
return i ;
} catch(NumberFormatException e) {
return 0 ;
}
}
return 0 ;
}

Questions :
1. Écrire toutes les lignes de tests nécessaires pour assurer la couverture de code, en utilisant la classe Assert.
2. Préciser les éventuels messages obtenus lors des tests précédents, identifier les problèmes et donner une nouvelle version de la fonction.

[Badshade23]

Ok, tu as déjà écrit tes différents tests grâce à assert ? Et tu as renseigné les messages obtenue avec ces tests ?

[OmistuSamaIsCoding]

Ok, tu as déjà écrit tes différents tests grâce à assert ? Et tu as renseigné les messages obtenue avec ces tests ?

Oui ! Les voici :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
Assert.estVrai(convertToInteger("3") = 3, "erreur pour 3")
Assert.estVrai(convertToInteger("0") = 0, "erreur pour 0")
Assert.estVrai(convertToInteger("toto") = 0, "erreur pour toto")
Assert.estVrai(convertToInteger(NULL) = 0, "erreur pour NULL")
Assert.estVrai(convertToInteger() = 0, "erreur pour aucun paramètre")
Assert.estVrai(convertToInteger(5) = 5, "erreur pour 5  " )
Assert.estVrai(convertToInteger(5.25) = 0, "erreur pour 5.25  " )
Assert.estVrai(convertToInteger(0) = 0, "erreur pour 0" )

[Badshade23]

Tes tests fonctionnent ?
La méthode "estVrai" écrit ainsi n'existe pas pour la class Assert.
Je verrais ça ainsi :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
import static org.junit.Assert.assertTrue;
 
import org.junit.jupiter.api.Test;
 
class UnitTest {
 
	@Test
	void test() {
		assertTrue("erreur pour le String 3", 3 == TestConvertInt.convertToInteger("3"));
		assertTrue("erreur pour le String 0", 0 == TestConvertInt.convertToInteger("0"));
		assertTrue("erreur pour le String toto", 0 == TestConvertInt.convertToInteger("toto"));
		assertTrue("erreur pour Null", 0 == TestConvertInt.convertToInteger(null));
		assertTrue("erreur pour l'int 5", 5 == TestConvertInt.convertToInteger(5));
		assertTrue("erreur pour le float 5.25", 0 == TestConvertInt.convertToInteger(5.25));
		assertTrue("erreur pour l'int 0", 0 == TestConvertInt.convertToInteger(0));
	}
}

Et l’exécution de ce code ne retourne aucune erreurs.

Par contre le test :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Assert.estVrai(convertToInteger() = 0, "erreur pour aucun paramètre")

Il n'est pas bon. Si tu regardes la signature de ta méthode convertToInteger, tu es obligé de passer un paramètre à moins de changer celle-ci.

Suite aux tests effectués, je ne vois pas d’erreurs pour ta méthode "convertToInteger" à moins que le test précédent doit fonctionner ... Tu as des tests obligatoires ?
Peut-être que quelqu'un d'autre saura mieux t'éclairer sur le sujet.

[professeur shadoko]

"Si la conversion de
String en Integer n’est pas possible, il faut retourner 0 (le catch s’exécute si la conversion n’est pas possible)."
voilà une spec très discutable: si on a quelque chose qui représente "0" on ne fait pas la différence avec une erreur... bizarre...bizarre
(bon d'acc... c'est un exercice mais pourquoi donner de mauvaises habitudes qui s'incrusteront dans la vie réelle?)