Discussion :

[MOONKHE]

Bonjour,

Je viens de me lancer dans la programmation JAVA et est en train de suivre le cours "Programmation Java pour les enfants, les parents et les grands-parents" du site.
J'essaye de faire l'exercice ou il faut convertir des températures (F & C) mais j'ai des soucis lors de la compilation.

Lien disponible ci-dessous (l'exercice est situé tout en bas) :
http://java.developpez.com/livres-co...ts/?page=bases

Rappel de l'énoncé :

1. Crée une nouvelle classe nommée ConvertisseurTempératures dont une méthode a la signature suivante :

public String convertirTempérature (int température, char convertirEn)

Si la valeur de l'argument convertirEn est F, la température doit être convertie en degrés Fahrenheit ; si c'est C, en degrés Celsius. Quand tu appelles cette méthode, mets la valeur de l'argument de type char entre apostrophes.

Voici mes codes :

Températures.Java

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
package test;
 
public class Temperature {
 
	public static void main(String[] args) {
 
		ConvertisseurTempératures convertChoix = new ConvertisseurTempératures();
 
		convertChoix.convertirTempérature(50,'C');
 
	}
 
}

ConvertisseurTempératures.java

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
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
package test;
 
public class ConvertisseurTempératures {
 
	public String convertirTempérature (int température, char convertirEn){
 
 
		do {
 
			if (convertirEn=='F') {
 
				température = ((température*9)/5) + 32;
				System.out.println("La conversion en F est : " +température);
 
			}	
 
			else if	(convertirEn=="C") {
 
				température = (température-32)*(5/9);
			    System.out.println("La conversion en C est : " +température);
 
			}
 
			else {
 
				break;
 
			}
 
		} while(!((convertirEn.equals('F')) |(convertirEn.equals('C'))));
 
	}
}

Merci d'avance.

[abysr]

change

Code :

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

else if	(convertirEn=="C") {

en

Code :

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

else if	(convertirEn=='C') {

tu as mis des doubles quotes au lieu des quotes

de plus vu que ces variables sont de types : char (primitif), il faut les comparer avec : == et non .equals(...)

[MOONKHE]

J'ai changé ça et j'ai aussi changé le "equals" par == à la fin du do while.

Mais toujours un problème à la compilation :

Exception in thread "main" java.lang.Error: Unresolved compilation problem:
This method must return a result of type String

at test.ConvertisseurTempératures.convertirTempérature(ConvertisseurTempératures.java:5)
at test.Temperature.main(Temperature.java:9)

Apparemment, ma méthode doit retourner un string mais je vois pas ou.

[abysr]

C'est normal car tu as signé ta méthode avec String comme type retour, alors que tu n'en as pas besoin remplace par 'void'
et ça devrait etre bon.
Tu pourrais laisser le String , mais tu devras ce String dans ton programme de test (ie ta classe temperature)

[MOONKHE]

Effectivement c'est bon, merci.

Maintenant la compilation s'est bien déroulé, mais le résultat est faux

Je vais chercher un peu et je vous tiens au courant.

[joel.drigo]

Maintenant la compilation s'est bien déroulé, mais le résultat est faux

Je t'ai dit pourquoi : les divisions de int !

[joel.drigo]

Salut,

Code :

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

1
2
3
4
5
package test;
 
public class ConvertisseurTempératures {
 
	public String convertirTempérature (int température, char convertirEn){

En effet, avec ce String ici, tu déclares que la méthode doit retourner une valeur de type String. Il est donc indispensable d'avoir au moins un return. Toutefois, il serait plus logique de retourner un int (ou un float ou un double pour un nombre à virgule).
En revanche, il n'y a aucune raison d'avoir une boucle à cet endroit : s'il devait y avoir une boucle, ce serait éventuellement à l'endroit d'une saisie au clavier (demander à l'utilisateur de saisir soit 'c' soit 'f', tant qu'il saisit autre chose). En l'occurence, faire une boucle sur une condition, et un break sur la condition inverse est superfétatoire. On peut tester si le type de conversion n'est pas supporté et renvoyer une exception dans ce cas :

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
19
package test;
 
public class ConvertisseurTempératures {
 
   public int convertirTempérature (int température, char convertirEn){
 
      if ( convertirEn=='F' ) {
          température = ((température*9)/5) + 32;
          System.out.println("La conversion en F est : " +température);
      }
      else if	(convertirEn=='C') {
           température = (température-32)*(5/9);
           System.out.println("La conversion en C est : " +température);
      }
      else {
          throw new RuntimeException("Le type de températeur " + convertirEn + " n'est pas supporté");
      }
      return temperature;
}

Toutefois, il y a un problème dans ces calculs : en Java, un int divisé par int donne toujours un int. En particulier un int divisé par int plus grand donne toujours 0 (comme 5/9 par exemple). Il vaudrait mieux faire les calculs en float :

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
public class ConvertisseurTempératures {
 
   public flaot convertirTempérature (float température, char convertirEn){
 
      if ( convertirEn=='F' ) {
          température = ((température*9)/5) + 32;
          System.out.println("La conversion en F est : " +température);
      }
      else if	(convertirEn=='C') {
           température = (température-32)*(5f/9);
           System.out.println("La conversion en C est : " +température);
      }
      else {
          throw new RuntimeException("Le type de températeur " + convertirEn + " n'est pas supporté");
      }
      return temperature;
}

A noter qu'on ne peut pas appeler equals() sur une variable de type primitif (char est un type primitif). Et utilise plutôt l'opérateur || que | (|| est l'opérateur de combinaison ou, et si la première condition est vraie, la seconde ne sera même pas évaluée, alors que | est l'opérateur bitwise ou).