Enigma machine disque

Version imprimable

Bonjour a tous,

Je travaille sur un project sur la machine Enigma. Un des exercise demande de changer sur un disque les lettres d'une place vers la gauche(je crois:roll:)

Mon disque s'appelle Rotor1 et voici dessous sa description:

Code:

1
2
 private char[][] rotor1 = {{'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z'}, 
                                 {'G', 'T', 'U', 'B', 'Z', 'Y', 'V', 'J', 'L', 'K', 'S', 'A', 'W', 'X', 'H', 'C', 'D', 'E', 'M', 'N', 'O', 'P', 'Q', 'R', 'I', 'F'}};

Si mon code fonctionne comme demander, Rotor1 dois changer comme ci-dessous:

Code:

1
2
{{'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z', 'A'}, 
                                 {'T', 'U', 'B', 'Z', 'Y', 'V', 'J', 'L', 'K', 'S', 'A', 'W', 'X', 'H', 'C', 'D', 'E', 'M', 'N', 'O', 'P', 'Q', 'R', 'I', 'F', 'G'}};

Ci-dessous est mon code pour que Rotor1 permute d'une place:

Code:

1
2
3
4
5
6
7
8
9
10
11
public void rotateRotor(char[][] aRotor)
    {
     int row = rotor1[0][1];
     int col = rotor1[1][0];
     for(row = 0; row < aRotor.length; row++)
     {    
        for(col = 0; col < aRotor[row].length; col++)    
        {  
          rotor1 = aRotor[row][col];
        } 
     }

Je dois tester le code comme ceci:

Code:

1
2
EnigmaMachine em = new EnigmaMachine();
em.rotateRotor(em.getRotor1());

Mais bien evidement quand je teste le code rien ne se passe.

Est-ce quelqu'un aurrais une idee du problem;)

05/03/2009, 14h10
jean.2edi
Citation:
Envoyé par cyrodil

Code:

1 2 3 int row = rotor1[0][1]; int col = rotor1[1][0];
C'est bizarre : tu calcules tes index à partir du contenu de ton tableau ??? rotor1[0][1] vaut 'B' c'est à dire 66 ??? Tu es sûr que rien ne se passe, pas d'exception ?

Le tableau est pratique pour stocker un ensemble de valeurs mais pas pour les déplacer, il vaudrait mieux une liste comme LinkedList (pour des insertions, suppressions à différentes positions).

En restant dans ton code complexe des tableaux:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public void rotateRotor(char[][] aRotor)
    {
     char carry;
     for(int row = 0; row < aRotor.length; row++)
     {    
        carry=aRotor[0];
        for(int col = 1; col < aRotor[row].length; col++)    
        {  
          aRotor[row][col-1]=aRotor[row][col];
        }
        aRotor[row][col-1]=carry;
     }

08/03/2009, 13h10
cyrodil

Cher jean.2edi,

Malheureusement sur cet exercise je ne peux pas utiliser LinkedList, ce n'est pas dans mon cours:cry:

Merci encore pour ton aide;):king:

Citation:

Envoyé par jean.2edi

C'est bizarre : tu calcules tes index à partir du contenu de ton tableau ??? rotor1[0][1] vaut 'B' c'est à dire 66 ??? Tu es sûr que rien ne se passe, pas d'exception ?

Le tableau est pratique pour stocker un ensemble de valeurs mais pas pour les déplacer, il vaudrait mieux une liste comme LinkedList (pour des insertions, suppressions à différentes positions).

Cher Lorantus,

Quand je compile ce code, un message d'error s'affiche sur la ligne carry = aRotor[0];

le message d'error est le suivant : imcompatible type. found char[] but expected char

:cry:
Citation:
Envoyé par Lorantus
En restant dans ton code complexe des tableaux:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 public void rotateRotor(char[][] aRotor) { char carry; for(int row = 0; row < aRotor.length; row++) { carry=aRotor[0]; for(int col = 1; col < aRotor[row].length; col++) { aRotor[row][col-1]=aRotor[row][col]; } aRotor[row][col-1]=carry; }

08/03/2009, 13h43
Lorantus

Autant pou rmoi, j'ai pas fais de complie sur ce source -j'ai donné à titre d'info...

comme dis l'erreur :

Code:

char carry=aRotor[row][0];

Mais, bon, on sait tous que tu as corriger cela par toi même... dans le cadre de ton exercice, dont la réponse dépasse de loin le copier-coller.
:bug:
08/03/2009, 14h07
cyrodil

Cher Lorantus,

Merci encore pour ton aide:D

Je debute avec Java, le cours que je prends actuellement est vraiment trop pauvre:roll:

J'ai recorriger mon code, mais quand je vais dans inspector mon array ne bouge pas, c'est toujours en position [0] avec la lettre A:bug:

Citation:

Envoyé par Lorantus

Autant pou rmoi, j'ai pas fais de complie sur ce source -j'ai donné à titre d'info...

comme dis l'erreur :

Code:

char carry=aRotor[row][0];

Mais, bon, on sait tous que tu as corriger cela par toi même... dans le cadre de ton exercice, dont la réponse dépasse de loin le copier-coller.
:bug:

Ce programme fonctionne, si j'ai bien compris ton pb de permutation.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
public class Main {
    private static char[][] rotor1 = {{'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z'},
                                      {'G', 'T', 'U', 'B', 'Z', 'Y', 'V', 'J', 'L', 'K', 'S', 'A', 'W', 'X', 'H', 'C', 'D', 'E', 'M', 'N', 'O', 'P', 'Q', 'R', 'I', 'F'}};
 
    public static void main(String[] args) {
        rotateRotor(rotor1);
    }
 
    public static void rotateRotor(char[][] aRotor) {
        for (int row = 0; row < aRotor.length; row++) {
            char carry = aRotor[row][0];
            for (int col = 1; col < aRotor[row].length; col++) {
                aRotor[row][col - 1] = aRotor[row][col];
            }
            aRotor[row][aRotor[row].length - 1] = carry;
        }
    }
}

12/03/2009, 11h23
cyrodil

Je suis desoler Lorantus de perdre ton temps:oops:

J'ai appeler mon prof et il m'a dis que je n'avais pas a implenter un nested loop, un For suffit car je ne travaille pas avec une 2D array. 8O

Il a precise aussi peut-etre de tester que 'A' est egal a la lettre suivante, ainsi de suite jusqu'a ce que 'Z' egal 'A'.

Alors j'ai rechange mon code mais avec l'aide du tien, je nage dans la soupe cela ne fonctionne toujours pas:evilred:

public void rotateRotor(char[][] aRotor)
{
char temp;
int i = 0;
if (aRotor == null && aRotor.length > 0)
{
throw new IllegalArgumentException("invalid rotor");
}
for (i = 0; i < aRotor.length - 1; i++)
{
aRotor[i] = aRotor[i+1];
}
temp = aRotor[i][0];
aRotor[i][aRotor[i].length - 1] = temp;
}
}

:sm:

Je comprends pas là :

Citation:

Il a precise aussi peut-etre de tester que 'A' est egal a la lettre suivante, ainsi de suite jusqu'a ce que 'Z' egal 'A'.

Regarde ton code, il posséde bcps d'erreurs:
public void rotateRotor(char[][] aRotor)
{
char temp;
int i = 0;
//Comment être .length>0 ET avoir == null ?
if (aRotor == null && aRotor.length > 0)
{
throw new IllegalArgumentException("invalid rotor");
}
for (i = 0; i < aRotor.length - 1; i++)
{
aRotor[i] = aRotor[i+1];
}
//Il s'agit d'un tableau 1D : aRotor[i][0] devient aRotor[0] etc
temp = aRotor[i][0];
aRotor[i][aRotor[i].length - 1] = temp;
}
}
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 public static void rotateRotor1D(char[] aRotor) { char temp; int i = 0; if (aRotor == null || aRotor.length < 1) { throw new IllegalArgumentException("invalid rotor"); } temp = aRotor[0]; for (i = 0; i < aRotor.length - 1; i++) { aRotor[i] = aRotor[i + 1]; } aRotor[aRotor.length - 1] = temp; }

12/03/2009, 12h54
cyrodil

Ok, je crois que j'ai compris. La bonne blague c'est que j'habite en Irlande et je suis a ma seconde annee avec java et le cour est en anglais8O. Donc il est difficile desfois de comprendre ce qu'ils demandent:evilred:

Mais je m'accroche:mouarf:

L'exercise donne le titre de ma methode: public static void rotateRotor1D(char[][] aRotor) donc si j'ai bien compris [][] veut dire une 2D array? j'ai relu l'exercise, je dois changer la ligne 0 qui:

{'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z'}

Mais je ne change pas la deuxieme:

{'G', 'T', 'U', 'B', 'Z', 'Y', 'V', 'J', 'L', 'K', 'S', 'A', 'W', 'X', 'H', 'C', 'D', 'E', 'M', 'N', 'O', 'P', 'Q', 'R', 'I', 'F'}

(mille excuse persuade qu'il fallait changer les deux lignes, c'est pour cela que ma premiere citation avait un nested loop)

Donc au telephone j'ai rien compris a ce que le prof m'a dit en plus il a un fort accent du sud de l'Irlande8O

Je suis desole, si t'abandonne je comprendrais:oops:

Citation:

L'exercise donne le titre de ma methode: public static void rotateRotor1D(char[][] aRotor) donc si j'ai bien compris [][] veut dire une 2D array ?

Oui, enfin, là, c'est moi qui decide de l'appeler 1D ou 2D... 1D : 1 dimension / 2D : deux dimensions (ou indices) etc.

Avec cela, tu as tous les éléments necessaires:
L'appel de rotateRotor(char[][]) avec un rotor en char[][]...

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
37
38
39
40
41
42
43
44
45
46
 
public class Main {
    private static char[][] rotor1 = {{'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z'},
                                      {'G', 'T', 'U', 'B', 'Z', 'Y', 'V', 'J', 'L', 'K', 'S', 'A', 'W', 'X', 'H', 'C', 'D', 'E', 'M', 'N', 'O', 'P', 'Q', 'R', 'I', 'F'}};
 
    /**
     * @param args the command line arguments
     */
    public static void main(String[] args) {
        rotateRotor(rotor1);
    }
 
    public static void rotateRotor(char[][] aRotor) {
        if (aRotor == null || aRotor.length < 1) {
            throw new IllegalArgumentException("invalid rotor");
        }
 
        rotateRotor1D(aRotor[0]);
    }
 
    public static void rotateRotor1D(char[] aRotor) {
        if (aRotor == null || aRotor.length < 1) {
            throw new IllegalArgumentException("invalid rotor");
        }
 
        char temp = aRotor[0];
        for (int i = 0; i < aRotor.length - 1; i++) {
            aRotor[i] = aRotor[i + 1];
        }
        aRotor[aRotor.length - 1] = temp;
    }
 
    public static void rotateRotor2D(char[][] aRotor) {
        for (int row = 0; row <
                aRotor.length; row++) {
            char carry = aRotor[row][0];
            for (int col = 1; col <
                    aRotor[row].length; col++) {
                aRotor[row][col - 1] = aRotor[row][col];
            }
 
            aRotor[row][aRotor[row].length - 1] = carry;
        }
 
    }
}

Je ne pourrai pas faire mieux.

12/03/2009, 13h21
cyrodil

Merci encore Lorantus pour ton aide:yaisse2::lol::king:;)