Discussion :

[Batman90]

Bonjour ,
Je suis débutante en JAVA. j'ai trouvé la solution pour le problème mais elle est très mal faite. j'aurais besoin de conseils afin de fournir un code plus performant.
Voici, le sujet : le principe de base est de construire un tableau de 5 par 5. A chaque case est affectée une lettre de l'alphabet, excepté le W qui sera remplacé par un V dans le texte en clair. chaque ligne ou colonne est numérotée de 1 à 5. le texte chiffré devient un couple de chiffre construit à l'aide des abscisse et ordonnée du tableau pour une seule lettre.
voilà le code que j'ai réalisé :

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
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
 
public static void main(String [] args)
	{
		 char [][] tabLettres = {{'A','B','C','D','E'},{'F','G','H','I','J'},{'K','L','M','N','O'},{'P','Q','R','S','T'},{'U','V','X','Y','Z'}};
		 String [][] tabChiffres = new String[5][5];
		 String chaine = "";
		 int compteur = 0;
		 String texteClair = "JE PENSE DONC JE SUIS.";
		 String texteCode="";
 
		 //transformer les alphabets en nombre correspondant au nombre de la ligne et de au nombre de la colonne de l'alphabet
		 for(int i=0;i<tabLettres.length;i++)
		 {
			 for(int j=0;j<tabLettres.length;j++)
			 {
				 int ligne = (i+1);
				 int colonne = (j+1);
				 chaine = chaine + Integer.toString(ligne) + Integer.toString(colonne);
			 }
		 }
 
		 //System.out.println(chaine);
 
		 //construction du tableau des chiffres
			 for(int j=0;j<tabLettres.length ;j++)
			 {
 
						 tabChiffres[0][j] = chaine.substring(compteur, compteur +2 );
 
						chaine = chaine.substring(compteur + 2,chaine.length() - 1);
 
			 } 
			 for(int j=0;j<tabLettres.length ;j++)
			 {
 
						 tabChiffres[1][j] = chaine.substring(compteur, compteur +2 );
 
						chaine = chaine.substring(compteur + 2,chaine.length() - 1);
 
			 } 
 
			 for(int j=0;j<tabLettres.length ;j++)
			 {
 
						 tabChiffres[2][j] = chaine.substring(compteur, compteur +2 );
 
						chaine = chaine.substring(compteur + 2,chaine.length() - 1);
 
			 } 
 
			 chaine = "41424344455152535455";
 
			 for(int j=0;j<tabLettres.length ;j++)
			 {
 
						 tabChiffres[3][j] = chaine.substring(compteur, compteur +2 );
 
						chaine = chaine.substring(compteur + 2,chaine.length() - 1);
 
			 } 
 
			 for(int j=0;j<tabLettres.length ;j++)
			 {
 
						 tabChiffres[4][0] = "51";
						 tabChiffres[4][1] = "52";
						 tabChiffres[4][2] = "53";
						 tabChiffres[4][3] = "54";
						 tabChiffres[4][4] = "55";
			 } 
 
			 //affichage du tableau des chiffres
 
		 System.out.println();
		 for(int i=0;i<tabLettres.length;i++)
		 {
			 for(int j=0;j<tabLettres.length;j++)
			 {
				 System.out.println("tabChiffres["+(i+1)+"]["+(j+1)+"] = "+tabChiffres[i][j]);
			 }
		 }
 
 
		 for(int k=0;k<texteClair.length();k++)
		 {
			 for(int i=0;i<tabLettres.length;i++)
			 {
				 for(int j=0;j<tabLettres.length;j++)
				 {
					 if(texteClair.charAt(k) == 'W')
					 {
						 texteClair = texteClair.replace('W', 'V');
					 }
					 if(texteClair.charAt(k) == tabLettres[i][j])
					 {
						texteCode = texteCode + tabChiffres[i][j]; 
					 }
				 }
			 }
		 }
		 System.out.println(texteCode);
	}

Merci de votre aide car je pense que pour trouver le tableau des chiffres il existe une meilleure solution mais je ne la trouve pas.
Tout conseil est bienvenu.


Batman90

[thelvin]

Bah en fait il sert à rien, le tableau de chiffres.

Quand tu entres dans le if(texteClair.charAt(k) == tabLettres[i][j]), tu sais déjà que la colonne c'est i et la ligne c'est j. Donc les chiffres c'est i+1 et j+1.

... Mais ça reste pas très optimisé. Il vaudrait mieux faire deux tableaux de 25 éléments, le premier qui contient la colonne de chaque lettre, le second qui contient la ligne de chaque lettre.

[kolodz]

Bonjour,

Tu pourrai utiliser l'objet Map. Celui-ci lie une clé à une valeur.

Une implémentation pourrai ressemble à ceci :

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
package org.k.developpez.forum;
 
import java.util.HashMap;
import java.util.Map;
 
public class Encoder {
	public static void main(String[] args) {
		// Contruction de la map à partir du tableau
		String[][] tabLettres = { { "A", "B", "C", "D", "E" },
				{ "F", "G", "H", "I", "J" }, { "K", "L", "M", "N", "O" },
				{ "P", "Q", "R", "S", "T" }, { "U", "V", "X", "Y", "Z" } };
		Map<String,String> myMap = new HashMap<String,String>();
		for (int i = 0; i < tabLettres.length; i++) {
			String[] lineLettre = tabLettres[i];
			for (int j = 0; j < lineLettre.length; j++) {
				String key = i+""+j;
				myMap.put(key,lineLettre[j]);
			}
		}
		//Inversion de la map pour facilité l'encodage.
		// On pourrait très bien faire la contruction dans le bon sens directement
		Map<String, String> myEncodingMap = new HashMap<>();
		for(Map.Entry<String, String> entry : myMap.entrySet()){
			myEncodingMap.put(entry.getValue(), entry.getKey());
		}
 
		String texteClair = "JE PENSE DONC JE SUIS.";
		StringBuilder sb = new StringBuilder();
		char[] letter =texteClair.toCharArray();
		for (char c : letter) {
			String encodedChar = myEncodingMap.get(String.valueOf(c));
			if(encodedChar != null){
				sb.append(encodedChar);
			}else {
				sb.append(c);
			}
		}
		System.out.println(sb.toString());
	}
}

Résultat :

1404 3004233304 03242302 1404 33401333.

A noté l'utilisation du StringBuilder pour évité la création des String intermédiaire en mémoire.

Cordialement,
Patrick Kolodziejczyk.

Source :
http://docs.oracle.com/javase/7/docs.../util/Map.html
http://docs.oracle.com/javase/7/docs...ngBuilder.html

[Batman90]

Je vous remercie pour la réponse. ça m'a été très utile.

Rq : je voulais voter 10 / 10 pour la réponse de Patrick mais je ne sais pas comment faire.
Aussi je remercie le modérateur (Thelvin ) car sa réponse est aussi pertinente. je vais sans doute la tester. et je vous enverrai le réponse dès que possible afin d'enrichir le forum même si je suis si débutante.

[thelvin]

Merci.

Rq : je voulais voter 10 / 10 pour la réponse de Patrick mais je ne sais pas comment faire.

On ne peut donner qu'un seul point par message. Même les messages qu'on apprécie plus que les autres. Mais bon, tout le monde peut donner un point, et donc les messages vraiment appréciés ont beaucoup de points.

Et personnellement, je n'encouragerais pas les messages comme le sien. Il a juste fait ton travail à ta place, et je ne vois pas trop l'intérêt. Personne ici ne doutait qu'il est cap' de le faire, or le but ici, c'est de progresser ou faire progresser. Ce n'est pas de te donner un code à copier/coller sans le comprendre, qui te fera progresser.

[Mishulyna]

Et personnellement, je n'encouragerais pas les messages comme le sien. Il a juste fait ton travail à ta place, et je ne vois pas trop l'intérêt.

Là vous vous trompez (une fois n'est pas coutume)...

Vous restez mon idole (vous et tchize_) mais apparemment vous personnellement vous avez oublié ce que c'est d'être étudiant, débutant, et devoir TOUT apprendre pour le prochain test, examen, whatever. Mon niveau d'incompétence me permet de rester proche des débutants et sachez que c'était le code filé par mes collègues qui m'a appris à lire, comprendre et écrire du code.
Que celui qui n'a jamais lu le code d'autrui (et ne s'en est jamais servi) jette la première pierre!