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

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

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) == '-' )){ ... }