Problème fonction Math.pow

Version imprimable

Bonsoir a tous, j'ai un petit problème avec mon programme java, je souhaite transformer une chaine de caractère en un entier, pour avoir le bon positionnement des chiffres j'utilise la fonction Math.pow ...
Or pour calculer j'ai une tonne d'erreur (18 en tout) qui me disent que mes variables i , res , signe etc... sont invalides, bref.. Voici mon code, si vous avez des suggestions à faire je suis preneur ;)
Code:

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 34 35 import java.io.*; public class Exo2{ public static void main (String args[]){ String s="-3842"; int i=0 , res=0 , signe=0; int p=(int)Math.pow(10, s.length-(i+1)); Sytem.out.println("L'ordinateur choisit une chaine de caractere :"+s+" a convertir en un entier"); System.out.println(Convertir(s)); } public static int Convertir(String s) { while (i<s.length) { if(i==0&&s.charAt(0)='-'){ signe=-1; } else if(i==0&&s.charAt(0)='+'){ signe=1; } else { res=res+s.charAt(i)*p; } i++; } res=res*signe; return res; } }

12/01/2012, 18h22
Robin56

Je vois que tu utilises tes variables i, res et signe dans ta méthode Convertir alors qu'ils ne sont déclarés qu'au sein de la méthode main. Pour ça, ils faudraient les déclarer en variable de classe ou en paramètre de la fonction Convertir. Les messages d'erreurs qui sont affichés doivent surement être assez parlant à ce niveau.
12/01/2012, 18h25
thelvin

Citation:

Envoyé par suNiz

je souhaite transformer une chaine de caractère en un entier,

Je suppose qu'il s'agit d'un exercice, sinon il fallait utiliser Integer.parseInt()

Citation:

Envoyé par suNiz

pour avoir le bon positionnement des chiffres j'utilise la fonction Math.pow ...

Mauvaise idée. Math.pow() travaille sur des flottants, pas des entiers. Il y a des risques d'erreur de précision, et puis il y a aussi la question des performances. Il vaut mieux multiplier le nombre en cours de construction par 10, à chaque chiffre.

De toute façon, si tu n'as pas droit à Integer.parseInt() pour faire ça, tu n'as pas droit non plus à un truc qui te fournit les puissances de 10 sans réfléchir.

Citation:

Envoyé par suNiz

Or pour calculer j'ai une tonne d'erreur (18 en tout) qui me disent que mes variables i , res , signe etc... sont invalides, bref..

Rien à voir avec Math.pow().

Tes variables sont déclarées dans main(), pas dans Convertir().
Elles sont donc visibles dans main(), pas dans Convertir().

Solution : Elles ne sont pas utilisées dans main(), ce n'est donc pas là qu'il fallait les déclarer, mais dans Convertir().

Suggestion : les noms de méthodes devraient commencer par des minuscules. C'est convertir(), pas Convertir().
12/01/2012, 18h30
Robin56

Citation:

Envoyé par thelvin

Solution : Elles ne sont pas utilisées dans main(), ce n'est donc pas là qu'il fallait les déclarer, mais dans Convertir().

En effet, c'était la troisième solution qui est la meilleure dans ton cas.

Ah oui, ça marche un peu mieux maintenant ...

Code:

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
34
35
36
import java.io.*;
public class Exo2{
	public static void main (String args[]){
		String s="-3842";
		System.out.println("L'ordinateur choisit une chaine de caractere :"+s+" a convertir en un entier");
		System.out.println(convertir(s));
 
	}
 
 
	public static int convertir(String s) {
		int i=0 , res=0 , signe=0;
		int p=(int)Math.pow(10, (s.length-(i+1)));
 
		while (i<s.length) {
			if(i==0&&(s.charAt(0)='-')){
				signe=-1;
			}
 
			else if(i==0&&s.charAt(0)='+'){
				signe=1;
			}
 
			else {
				res=res+s.charAt(i)*p;
				}
 
			i++;
 
		}
 
		res=res*signe;
		return res;
 
	}	
}

Par contre la condition && dans if, else if ne marche par avec les 'char' ? (d'apres le compilateur, non ..)

12/01/2012, 18h51
thelvin

Citation:

Envoyé par suNiz

Par contre la condition && dans if, else if ne marche par avec les 'char' ? (d'apres le compilateur, non ..)

Si, mais = ne marche ni avec les chars, ni avec rien.

= est l'opérateur d'affectation.
L'opérateur d'égalité, c'est ==
12/01/2012, 18h53
Robin56

Ton "parentesage" est un peu boiteux :

Code:

if(i==0&&(s.charAt(0)='-')){ ... }

Quelque chose comme ceci c'est plus claire :

Code:

if((i == 0) && (s.charAt(0) = '-')){ ... }

Ensuite, tu n'utilises pas = comme il faut.
= : assignation
== : test d'égalité

Donc :

Code:

if((i==0) && (s.charAt(0) == '-' )){ ... }
12/01/2012, 18h55
suNiz

Ah oui, j'avais pas fait attention à ce détail... Bon, apparement il ne reste plus qu'une seule erreur dans mon Math.pow ..

Code:

int p=(int)Math.pow(10, (s.length-(i+1)));

le s.length ne semble pas être apprécié... :/
12/01/2012, 18h57
Robin56

Citation:

Envoyé par suNiz

le s.length ne semble pas être apprécié... :/

Javadoc. C'est un réflexe à prendre.
12/01/2012, 19h06
suNiz

Il faut donc que j'utilise int length(s) ? :?
12/01/2012, 19h08
Robin56

Citation:

Envoyé par suNiz

Il faut donc que j'utilise int length(s) ? :?

Code:

int size = s.length();
12/01/2012, 19h23
droggo

Xoe,

Citation:

Envoyé par suNiz

Il faut donc que j'utilise int length(s) ? :?

Robin56 te l'a déjà rappelé : JavaDoc ! :)
12/01/2012, 19h26
suNiz

Merci beaucoup, mon programme est valide, reste plus qu'a regler le probleme de mon calcul, qui ne m'affiche pas du tout le bon resultat ^^
12/01/2012, 19h39
thelvin

Citation:

Envoyé par suNiz

Merci beaucoup, mon programme est valide, reste plus qu'a regler le probleme de mon calcul, qui ne m'affiche pas du tout le bon resultat ^^

C'est parce que tu additionnes des chars et des ints sans te poser de question.

- Regarde combien font 1 + '0'
- Regarde quel est le code ASCII du caractère '0'
=> Donc le problème, c'est de transformer tes chars en ints.

- Regarde à nouveau la table ASCII et constate que les chiffres de 0 à 9 sont dans l'ordre et consécutifs.
- Déduis-en que la valeur numérique d'un char c qui représente un chiffre, est

Code:

c - '0'

(ce qui est la même chose que

Code:

c - 48

)
13/01/2012, 01h20
suNiz

Thelvin, je vois de quoi tu veux parler, mais je ne comprends pas ou l'appliquer dans mon programme.. :/
13/01/2012, 12h35
thelvin

Citation:

Envoyé par suNiz

Thelvin, je vois de quoi tu veux parler, mais je ne comprends pas ou l'appliquer dans mon programme.. :/

Chaque fois que tu as besoin de transformer un char en nombre....
13/01/2012, 12h59
suNiz

Donc si j'ai bien compris mon s.charAt(i) donne quelque chose sous forme de caractère (et non de nombre)
13/01/2012, 13h11
thelvin

Citation:

Envoyé par suNiz

Donc si j'ai bien compris mon s.charAt(i) donne quelque chose sous forme de caractère (et non de nombre)

Oui... Comme son nom et sa documentation l'indiquent.

Code:

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
34
import java.io.*;
public class Exo2{
	public static void main (String args[]){
		String s="+10";
		System.out.println("L'ordinateur choisit une chaine de caractere :"+s+" a convertir en un entier");
		System.out.println(convertir(s));
 
	}
 
 
	public static int convertir(String s) {
		int i=0 , res=0 , signe=0 , lgtab=s.length();
		while (i<lgtab) {
			if((i == 0) && (s.charAt(0) == '-')){
				signe=-1;
			}
 
			else if((i == 0) && (s.charAt(0) == '+')){
				signe=1;
			}
 
			else {
				res+=s.charAt(i)*(int)Math.pow(10, lgtab-(i));
				}
 
			i++;
 
		}
 
		res=signe*res;
		return res;
 
	}	
}

Oui mais apres un (int) devant s.charAt(i) ne suffit pas ?..

13/01/2012, 13h36
suNiz

Je dis n'importe quoi! Enfaite j'ai juste besoin de mettre (s.charAt(i)-48)*....
13/01/2012, 13h40
thelvin

Citation:

Oui mais apres un (int) devant s.charAt(i) ne suffit pas ?..

Non, cela se fait automatiquement, et te donnera le code ASCII du chiffre, ce qui ne t'avance pas à grand-chose.

Citation:

j'ai juste besoin de mettre (s.charAt(i)-48)*....

Personnellement je préfère

Code:

s.charAt(i) - '0'

ça évite de se demander d'où vient le 48.
13/01/2012, 13h47
suNiz

D'accord, et bien merci beaucoup à toi Thelvin, j'ai compris pas mal de choses qui me passait par dessus.

Bon weekend ;):ccool: