Discussion :

[Diablo143]

bonjour à tous

je suis en train de faire un petit programme qui doit vérifier certaines conditions dont l'une étant

"les chiffres du tableau forme une suite décroissante"

j'ai donc mis en place la méthode suivante qui prend en paramètre d'entrée un tableau d'entier et qui renvoi un boolean si la condition est vérifiée.

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
 
     /**
     * méthode qui vérifie que les nombre d'un tableau sont une suite décroissante
     * @param tab le tableau de nombre
     * @return  true suite décroissante<br>
     *          false suite non décroissante
     */
    public static boolean isSuiteDecroissante(int[] tab){
        boolean isDecroissant = false;
        for (int index = 0; index < tab.length; index++) {
            int nb1 = tab[index];
            int nb2 = tab[index ++];
            if(nb1 >= nb2){
                isDecroissant = true;
            } else {
                isDecroissant = false;
            }
        }      
        return isDecroissant;
    }

seul soucis ma méthode me dis que des nombre comme 47141 respecte cette condition, alors que ce n'est pas ce qui est attendu.

Quelqu'un aurait il une idée ?

merci d'avance.

[yildiz-online]

tu fais 2 x ++, donc tu sautes chaque fois un index.

[Diablo143]

donc au lieu d'avoir

Code :

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

1
2
3
4
5
6
7
8
9
10
 
for (int index = 0; index < tab.length; index++) {
  int nb1 = tab[index];
  int nb2 = tab[index ++];
  if(nb1 >= nb2){
    isDecroissant = true;
  } else {
    isDecroissant = false;
  }
}

je devrai plutôt faire ?

Code :

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

1
2
3
4
5
6
7
8
9
10
 
for (int index = 0; index < tab.length - 1; index++) {
  int nb1 = tab[index];
  int nb2 = tab[index +1];
  if(nb1 >= nb2){
    isDecroissant = true;
  } else {
    isDecroissant = false;
  }
}

[yildiz-online]

presque, dans l'état actuel, lorsque tu seras sur la dernière itération, le +1 déclenchera un ArrayOutOfBoundException.

Egalement, comme optimisation, tu pourrais sortir de la boucle dès que ton résultat est faux.

[Diablo143]

résolue

je n'incrémente plus que d'un l'index en faisant attention à la condition pour pas avoir une exception comme avant, et je garde le else qui fait que des les premier chiffres qui respecte pas, on repasse à condition false et on sort de la boucle

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
 
/**
         * méthode qui vérifie que les nombre d'un tableau sont une suite décroissante
         * @param tab le tableau de nombre
         * @return      true suite décroissante<br>
         *                      false suite non décroissante
         */
	public static boolean isSuiteDecroissante(int[] tab){
		boolean isDecroissant = false;
		for (int index = 0; index < tab.length -1; index++) { 
			int nb1 = tab[index];
			int nb2 = tab[index +1];
			if(nb1 >= nb2){
				isDecroissant = true;
			} else {
				isDecroissant = false;
				break;
			}
		}		
		return isDecroissant;
	}

merci

[yildiz-online]

ou plus simplement:

Code :

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

1
2
3
4
5
6
7
8
9
 
public static boolean isSuiteDecroissante(int[] tab){
    for (int index = 0; index < tab.length -1; index++) { 
        if(tab[index] < tab[index +1]){
            return false
        }
    }		
    return true;
}