java codage d'un losange

Version imprimable

Bonjour, voici ce que mon code est sensé executer :

XXXXXXXX
XXX (2espaces) XXX
XX (4espaces) XX
X (6espaces) X
X(6espaces) X
XX (4espaces)XX
XXX (2espaces)XXX
XXXXXXXX

Mais voici ce que cela me donne :
XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX

Voici mon code

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
   private static void losange(int n) {
 
      int nbrCroix = n;
      int nbrEspace = 0;
 
      for(int i=1; i<=n; i++){
 
      //nbrCroix
         for(int j=n/2; j>1; j--){
 
            for( j=1;j<=n;j++)
               System.out.print('X');
         }
 
 
      //nbrEspace
         for(int j=0; j<=n-1-1;j++){
            for( j=n*2-1; j<=n-1-i;j--){
 
               System.out.print(' '); 
            }
 
 
         } System.out.println(); 
      }     
   }    
  // TODO 
}

Où est l'erreur? Merci

13/10/2019, 16h18
joel.drigo
Salut,
1. Premièrement
  Code:
  
  1 2 3 for(int j=n/2; j>1; j--){ for( j=1;j<=n;j++)
  Les deux boucles sont sur la même variable. Donc quelque soit la valeur de j par le premier for, elle est reinitialisé dans le second, qui la fait varier. Et quand il est terminé, on repart sur le premier for, on fait j--, puis on teste la condition j>1 et on continue. Si n>2, à la fin du second for, j vaut n>2, j-- donne j>1, donc on reboucle, infiniment.
2. Deuxièmement :
  Un losange, c'est comme deux pyramides : une première puis la même en inversant les lignes. Donc le code d'un losange n'est pas très éloigné de celui d'une pyramide (à la limite, on peut faire deux boucles successives, la première est la même que la pyramide, et la seconde la même en inversant le premier for (en allant de n à 1).

Réussi!! Merci!

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
// TODO 
      int nombreCroix = n;
      int nombreEspace = 0;
      for (int i=1; i<=n; i++) {
         for (int j=1; j<=nombreCroix; j++) {
            System.out.print ('X');
         }
           for (int k=1; k<=nombreEspace; k++) {
            System.out.print(' ');
            }
 
         for (int l=1; l<=nombreCroix; l++) {
            System.out.print ('X');
         }
         nombreCroix--;
         System.out.println();
 
         nombreEspace+=2;
      }
 
 
  int nombreCroix2 = 1;
 int nombreEspace2 = (n*2)-2;
   for (int z=1; z<=n; z++) {
 
   for (int m=1; m<=nombreCroix2; m++) {
            System.out.print ('X');
         }
      for (int p=1; p<=nombreEspace2; p++) {
            System.out.print(' ');
            }
            for (int e=1; e<=nombreCroix2; e++) {
            System.out.print ('X');
            }
            nombreCroix2++;
            System.out.println();
            nombreEspace2-=2;
}

mon code losange

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
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
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */
package losange;
import java.util.Scanner;
 
/**
 *
 * @author Samsung
 */
public class Losange {
 
    /**
     * @param args the command line arguments
     */
    public static void main(String[] args) {
        // TODO code application logic here
 
        int nombreLigne = 0;
        String caractereAAfficher ="";
        int videExt = 0, videInter = 2;
 
        System.out.println("donner le caractère à afficher puis cliquer sur entrer :");
        Scanner clavier = new Scanner(System.in);
        caractereAAfficher = clavier.nextLine();
        //System.out.println("caratère = " + caractereAAfficher);
        System.out.println("donner le nombre de ligne puis cliquer sur entrer :");
        nombreLigne = clavier.nextInt();
        clavier.nextLine();
        //System.out.println("nombre de ligne  = " + nombreLigne);
 
        videExt = nombreLigne / 2;
        if (nombreLigne % 2 == 0) 
            videExt--;
        for ( int i = 0; i<videExt; i++)
            System.out.print(" ");
        System.out.print(caractereAAfficher + caractereAAfficher );
        System.out.println();
        int ligneparcouru = videExt;
 
        if (nombreLigne % 2 != 0) {// si nombre de ligne est impair
            for(int i =0; i < ligneparcouru; i++) { // dessine la partie haute du losange
                videExt--;
 
                    for(int j =0; j < videExt ;j++)
                        System.out.print(" ");    
                System.out.print(caractereAAfficher);
 
                for( int k = 0; k < videInter ; k++ )
                    System.out.print(" ");
 
                System.out.print(caractereAAfficher);
                videInter = videInter + 2;
                System.out.println();
            }
 
        } else {//si nombre de ligne est paire
            int j =0,k=0;
            for(int i =0; i < ligneparcouru; i++) { // dessine la partie haute du losange
                videExt--;
 
                    for( j =0; j < videExt ;j++)
                        System.out.print(" ");    
                System.out.print(caractereAAfficher);
 
                for( k = 0; k < videInter ; k++ )
                    System.out.print(" ");
 
                System.out.print(caractereAAfficher);
                videInter = videInter + 2;
                System.out.println();
            }
            videInter = videInter - 2;
            for( j =0; j < videExt ;j++)
                        System.out.print(" ");    
                System.out.print(caractereAAfficher);
 
                for( k = 0; k < videInter ; k++ )
                    System.out.print(" ");
 
                System.out.print(caractereAAfficher);
                System.out.println();
        }
        if (nombreLigne % 2 == 0) {// si nombre de ligne est paire
            for(int i =0; i < ligneparcouru; i++) { // dessine la partie bas du losange 
                    videExt++;
                    videInter = videInter - 2;
                        for(int j =0; j < videExt ;j++)
                            System.out.print(" ");    
                    System.out.print(caractereAAfficher);
 
                    for( int k = 0; k < videInter ; k++ )
                        System.out.print(" ");
 
                    System.out.print(caractereAAfficher);
                    System.out.println();
                }
        }else{// si nombre de ligne est impair
 
            videInter = videInter - 2;
            for(int i =0; i < ligneparcouru; i++) { // dessine la partie bas du losange
                    videExt++;
                    videInter = videInter - 2;
                        for(int j =0; j < videExt ;j++)
                            System.out.print(" ");    
                    System.out.print(caractereAAfficher);
 
                    for( int k = 0; k < videInter ; k++ )
                        System.out.print(" ");
 
                    System.out.print(caractereAAfficher);
                    System.out.println();
                }
 
        }
    }
 
}