Discussion :

[flyman29600]

Bonjour,

quand je fais tourner ma boucle je ne comprend pas quand i passe de 0 a 1 la valeur de i=0 est modifie aussi
j'ai beau faire tourner a la main je ne trouve pas l'erreur
Si quelqu'un a une idee???

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
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
public class Crypto {
 
    static final String ALPHABET = "abcdefghijklmnopqrstuvwxyz";
    static final int DECALAGE = 4;
 
    static Scanner scanner = new Scanner(System.in);
 
    public static void main(String[] args) {
 
        System.out.println("Veuillez entrer une chaine de caracteres : ");
        //String s = scanner.nextLine();
       String s ="Avez-vous Vu mes 3 chats et mes 2 chiens ?";
        // la chaine a coder
        String aCoder = "";
        // la chaine codee
        String chaineCodee = "";
 
        /*******************************************
         * Completez le programme a partir d'ici.
         *******************************************/
        s.intern();
        String ALPHA=ALPHABET+" ";		
        String aCode;
        aCode=s;
 
        Pattern p = Pattern.compile("(-)|\\b!?(\\d.)|(\\p{Upper}?)|(!\\S)?|(\\W)") ;
        Matcher m = p.matcher(aCode) ; 
 
        String aCoderemplace = m.replaceAll("");
 
        aCoder=aCoderemplace;
 
 
        for(int i=0;i<aCoder.length();i++)
        {
        	for(int j=0;j<ALPHA.length();j++)
        	{      
        		if(aCoder.charAt(i) == ALPHA.charAt(j))
        		{        
 
        			if(j<22)
        			{    
          				chaineCodee=chaineCodee+aCoder.charAt(i);    
        				chaineCodee=chaineCodee.replace(chaineCodee.charAt(i), ALPHA.charAt(j+DECALAGE));	
        			}
        			if(j>22)
        			{
        				chaineCodee=chaineCodee+aCoder.charAt(i);
        				chaineCodee=chaineCodee.replace(chaineCodee.charAt(i), ALPHA.charAt((j+DECALAGE)%26));
        			}
        			if(ALPHA.charAt(26)==aCoder.charAt(i))
        			{
        				chaineCodee=chaineCodee.replace(chaineCodee.charAt(i),ALPHA.charAt(26));
        			}
        		}
        	}	
        }
 
 
        /*******************************************
         * Ne rien modifier apres cette ligne.
         *******************************************/
    	System.out.println(aCoder);
    	System.out.format("La chaine initiale etait : '%s'\n", s);
 
        if (aCoder.isEmpty()) {
            System.out.println("La chaine a coder est vide.\n");
        } else {
            System.out.format("La chaine a coder est : '%s'\n", aCoder);
            System.out.format("La chaine codee est : '%s'\n", chaineCodee);
        }
    }
}

Merci pour vos reponse...

[joel.drigo]

Salut,

Ce n'est pas clair : quand tu fais tourner ta boucle (je suppose que c'est le for(int i=0;i<aCoder.length();i++)),

• quand i passe de 0 a 1 : i varie de 0 à aCoder.length(), la taille de la chaîne aCoder, donc effectivement i prend les valeurs 0, 1, 2, 3, etc.
• la valeur de i=0 est modifie aussi : qu'est ce que tu appelles la valeur de "i=0" ? "i=0" est une affectation en Java, ce serait quoi la valeur d'une affectation ? Et si tu parles de la valeur de i, bah oui, elle est modifiée, puisque quand i passe de 0 à 1, i passe de 0 à 1, obviously.

[flyman29600]

salut

desole pour l'imprecision

en fait quand ma variable passe dans la premiere boucle passe de 0 à 1 puis 2 , a l'index 0 le decalage se fait passe a 1 puis 2 mais il se refait aussi pour l'index 1

si par exemple je rentre "fuyez" au clavier il m'affiche "jccid",au lieu de "jycid".et quand je le fais pas a pas

il passe bien le 'f 'a 'j' puis le 'u' a 'y' puis le 'y' a 'c' mais a cette etape il change aussi le 'u' en 'c'..

je pas bien si c'est plus clair,j'espere en tout cas merci pour ta reponse. )

[joel.drigo]

Tu as ce problème parce que la méthode replace() remplace toutes les occurences du caractères correspondant au premier paramètre par le second caractère.

Code :

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

System.out.println( "abacada".replace('a','_'));

affiche

_b_c_d_

Ainsi dans chaineCodee=chaineCodee.replace(chaineCodee.charAt(i), ALPHA.charAt(j+DECALAGE));, tu dis que le caractère situé en position i dans chaineCodee doit être remplacé par le caractère décalé, et non pas mettre en position i le caractère décalé.

Ainsi, si on exécute pas à pas ton programme, on obtient :

1. chaineCodee avant : f
   
   • on remplace tous les f par j
   • chaineCodee après : j
2. chaineCodee avant : ju
   
   • on remplace tous les u par y
   • chaineCodee après : jy
3. chaineCodee avant : jyy
   
   • on remplace tous les y par c
   • chaineCodee après : jcc
4. chaineCodee avant : jcce
   
   • on remplace tous les e par i
   • chaineCodee après : jcci
5. chaineCodee avant : jcciz
   
   • on remplace tous les z par d
   • chaineCodee après : jccid

On voit bien pourquoi dans l'étape 3 le remplacement de y par c s'applique aussi au deuxième caractère (qui est y à cause du décalage de l'étape précédente).

à la place de

Code :

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

1
2
chaineCodee=chaineCodee+aCoder.charAt(i);    
chaineCodee=chaineCodee.replace(chaineCodee.charAt(i), ALPHA.charAt(j+DECALAGE));

pourquoi ne pas simplement faire :

Code :

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

chaineCodee=chaineCodee+ALPHA.charAt(j+DECALAGE);

Et même principe pour j>22...

[flyman29600]

Tu as ce problème parce que la méthode replace() remplace toutes les occurences du caractères correspondant au premier paramètre par le second caractère.


.

ok, je pensais que cette methode ne remplaçais que l'element a l'index.

merci en tout cas

[joel.drigo]

ok, je pensais que cette methode ne remplaçais que l'element a l'index.

Ce n'est pas possible : l'utilisation de l'index dans ton code ne se fait pas au même "niveau".
Quand tu écris ça :

Code :

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

chaineCodee.replace(chaineCodee.charAt(i), ALPHA.charAt(j+DECALAGE));

Tu exécutes en fait plusieurs statements, sucessivement :

1. chaineCodee.charAt(i) : premier résultat (un caractère)
2. j+DECALAGE : deuxième résultat (un int)
3. ALPHA.charAt( deuxième résultat ) : troisième résultat (un caractère)
4. chaineCodee.replace(premier résultat, troisème résultat) : résultat final

donc l'index i utilisé dans la détermination du premier résultat n'est pas possiblement connu par la méthode replace.