Discussion :

[suNiz]

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 :

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
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;
 
	}	
}

[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.

[thelvin]

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()

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.

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().

[Robin56]

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.

[suNiz]

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

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
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 ..)

[thelvin]

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 ==

[Robin56]

Ton "parentesage" est un peu boiteux :

Code :

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

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

Quelque chose comme ceci c'est plus claire :

Code :

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

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

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

Donc :

Code :

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

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

[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 :

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

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

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

[Robin56]

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

Javadoc. C'est un réflexe à prendre.

[suNiz]

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

[Robin56]

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

Code :

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

int size = s.length();

[droggo]

Xoe,

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

Robin56 te l'a déjà rappelé : JavaDoc !

[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 ^^

[thelvin]

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 :

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

c - '0'

(ce qui est la même chose que

Code :

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

c - 48

)

[suNiz]

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

[thelvin]

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....

[suNiz]

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

[thelvin]

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.

[suNiz]

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
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 ?..

[suNiz]

Je dis n'importe quoi! Enfaite j'ai juste besoin de mettre (s.charAt(i)-48)*....