Enigma machine disque

**cyrodil** · 05/03/2009, 11h42

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

)

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

Code :

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

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 :

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

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 :

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

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 :

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

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

**jean.2edi** · 05/03/2009, 14h10

Envoyé par cyrodil

Code :

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

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

**Lorantus** · 08/03/2009, 09h16

En restant dans ton code complexe des tableaux:

Code :

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

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

**cyrodil** · 08/03/2009, 13h10

Cher jean.2edi,

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

Merci encore pour ton aide

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

**cyrodil** · 08/03/2009, 13h15

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

Envoyé par Lorantus

En restant dans ton code complexe des tableaux:

Code :

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

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

**Lorantus** · 08/03/2009, 13h43

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

comme dis l'erreur :

Code :

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

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.

**cyrodil** · 08/03/2009, 14h07

Cher Lorantus,

Merci encore pour ton aide

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

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

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 :

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

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.

**Lorantus** · 12/03/2009, 10h33

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

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

**cyrodil** · 12/03/2009, 11h23

Je suis desoler Lorantus de perdre ton temps

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.

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

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

**Lorantus** · 12/03/2009, 12h29

Je comprends pas là :

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 :

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

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

**cyrodil** · 12/03/2009, 12h54

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 anglais

. Donc il est difficile desfois de comprendre ce qu'ils demandent

Mais je m'accroche

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'Irlande

Je suis desole, si t'abandonne je comprendrais

**Lorantus** · 12/03/2009, 13h08

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 :

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

**cyrodil** · 12/03/2009, 13h21

Merci encore Lorantus pour ton aide

Enigma machine disque

Java

Discussions similaires

Partager

Partager