Discussion :

[Népomucène]

ah ben oui, il ne faut pas confondre "" et null
tu dois écrire :

Code :

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

if (laVariableString != null)

pour tester le non null !

Pour info, "" est une chaîne de caractères de longueur zéro. Pour Java c'est donc un objet
si tu testes laVariableString .equals("") tu demandes si laVariableString "contient/pointe vers" un objet chaîne de caractères de longueur zéro.
par contre si tu testes laVariableString != null tu demandes si laVariableString "contient/pointe vers" quelque chose.

[Pill_S]

PS aussi: ça c'est faux:

Code :

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

1
2
3
4
5
for(int i=0;i<tab[qIndex].length;i++){
            for(int j=0;j<tab[rIndex].length;j++){
                    System.out.println(tab[i][j]);
            }
        }

t'as bien de la chance si ça marche, c'est un coup de bol... le jour où tu auras des tableaux de tailles différentes dans les différents index, ça pètera méchamment...

ça devrait plutôt être comme ça:

Code :

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

1
2
3
4
5
6
 
for(int i=0;i<tab.length;i++){
            for(int j=0;j<tab[i].length;j++){
                    System.out.println(tab[i][j]);
            }
        }

[glmtsr82]

Bonjour à tous,

J'ai revu mon code en prenant en compte vos remarques (merci), ça devrait être mieux :

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
 public static void main(String[] args) {
        String[][] tab = new String[100][10]; // Tableau des questions et
                                                // des réponses
        int qIndex = 0; // Variable représentant le n° d'indice de la question
        int rIndex = 0; // Variable représentant le n° d'indice de la réponse
        String rep;
        String quest;
 
        do {
            System.out.println("Ecrire une question :");
            tab[qIndex][rIndex] = quest = Terminal.lireString();
 
            if (quest.length() != 0) {
                do {
                    System.out.println("Ecrire une réponse :");
                    rIndex = rIndex + 1;
                    tab[qIndex][rIndex] = rep = Terminal.lireString();
                 } while (rep.length() != 0);
            } else {
                qIndex = qIndex + 1;
              }
        } while (quest.length() != 0);
 
        for (int i = 0; i < tab.length; i++) {
            for (int j = 0; j < tab[i].length; j++) {
                System.out.println(tab[i][j]);
            }
        }
    }
 
}

@Pill_S, oui, je suis sous Eclipse, merci pour le Ctrl+Shift+F, je ne connaissais pas

[joel.drigo]

Bonjour,

Il y a plusieurs problèmes dans ton code (tu l'as testé avec mettons 4 questions et 3 réponses pour chaque question ?) :

• tu ne réinitialises pas rindex, à chaque nouvelle question (donc tu vas vite sortir du tableau genre 3 questions avec 4 réponses et patatras)
• tu incrémentes qindex quand on saisit une chaîne vide, mais pas quand on saisit une question : donc chaque nouvelle question devrait remplacer la précédente (en gros, le tableau ne contiendra que la dernière question si tu ajoutes l'initialisation de rindex, et sinon la première ligne contiendra question 1, réponse 1 de 1, réponse 2 de 1, question 2, réponse 1 de 2, réponse 2..., avec le risque même d'afficher question 2 comme une réponse à question 1...
• il faudrait ajouter des tests des bornes de tableau (pour éviter des dépassements d'index)
• par rapport à l'énoncé, il manque la contrainte "au moins 2 réponses par question" (un test sur rindex donc)

A noter que qindex pourrait être utilisé comme borne du for pour éviter d'afficher les "questions" null en fin de tableau. Pour rindex, non, à moins de stocker la valeur dans un second tableau.