Tester un carré magique

**komat** · 20/01/2011, 02h27

je veut faire un programme que l utilsateur entre la taille du magic carré aprés il entre les numero de la magic carré et aprés le programme affiche le rsultas (est ce qu il est magic carré ou non) le programme fait seulemnt la recherche aprés le remplissage
voila le code et je sais pas ou la faute

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
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
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
#include<stdio.h>
 
#include<stdlib.h>
 
#include<conio.h>
 
main()
 
{
 
      int t[50][50],a=0,s[50],n,test=1,i,j;
 
      printf("Donner la taille du carre  \t");
 
      scanf("%d",&n);
 
      printf("Remplissez le tableau \t");
 
      for(i=0;i<n;i++)
 
      {
 
                      for(j=0;j<n;j++)
 
                      {
 
                                      printf("Tapez la case T[%d][%d]\n",i+1,j+1);
 
                                      scanf("%d",&t[i][j]);
 
                      }
 
      }
 
 
 
      for(j=0;j<n;j++)
 
                      {
 
                                      s[j]=0;
 
                      }
 
      for(j=0;j<n;j++)
 
      {a=0;
 
                      for(i=0;i<n;i++)
 
                      a=t[i][j]+a;
 
      s[j]=a;}
 
      for(j=0;j<n;j++)
 
      {
 
                        if(s[j]!=s[j+1])
 
                        test=1;
 
      }
 
 
 
      if(test==0)
 
      {for(i=1;i<n;i++)
 
      {a=0;
 
                      for(j=1;j<n;j++)
 
                      a=t[i][j]+a;
 
      s[i]=a;}}
 
 
 
      for(i=0;i<n-1;i++)
 
      {
 
                        if(s[i]!=s[i+1])
 
                        test=1;
 
      }
 
 
 
      if(test==0)
 
      {a=0;
 
                 for(i=0;i<n;i++)
 
                 a=t[i][n-i]+a;
 
                 if(s[j]!=a)
 
                 test=1;
 
                 }
 
 
 
      if(test==0)
 
      {a=0;
 
                 for(i=0;i<n;i++)
 
                 a=t[n-i][i]+a;
 
                 if(s[j]!=a)
 
                 test=1;
 
                 }
 
 
 
      if(test==1)
 
      printf("le carre n'est pas magique \n");
 
      else
 
      {
 
          printf("le carre est pas magique \n");
 
            for(i=0;i<n;i++)
 
      {
 
                      for(j=0;j<n;j++)
 
 
 
                                      printf("%d \t",t[i][j]);
 
                                      printf("\n");
 
 
 
      }
 
      }
 
          getche ();          }

**LittleWhite** · 21/01/2011, 02h02

Bonjour,

Je vous pris de reformuler le problème car là, je n'ai rien compris du tout de ce qui était en place et de ce qui ne fonctionner pas.
Par explication du problème, j'entends:
- Votre réflexion sur la résolution du programme (algorithme)
- La description du bug:
- - Le résultat attendu
- - Le résultat que vous avez actuellement (et en quoi il est faut)

Je tiens aussi à ajouter que si vous avez des problèmes dans votre code vous pouvez utiliser un debuggueur qui vous permettra de suivre l'exécution du programme pas à pas tout en regardant les valeurs des variables.

J'ai reformé le programme (je n'ai pas touché les code):

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
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
#include<stdio.h>
#include<stdlib.h>
#include<conio.h>
 
main()
{
    int t[50][50],a=0,s[50],n,test=1,i,j;
 
    printf("Donner la taille du carre  \t");
    scanf("%d",&n);
 
    printf("Remplissez le tableau \t");
 
    for(i=0;i<n;i++)
    {
        for(j=0;j<n;j++)
        {
            printf("Tapez la case T[%d][%d]\n",i+1,j+1);
            scanf("%d",&t[i][j]);
        }
    }
 
    for(j=0;j<n;j++)
    {
        s[j]=0;
    }
 
    for(j=0;j<n;j++)
    {
        a=0;
 
        for(i=0;i<n;i++)
 
        a=t[i][j]+a;
        s[j]=a;
    }
 
    for(j=0;j<n;j++)
    {
        if(s[j]!=s[j+1])
            test=1;
    }
 
    if(test==0)
    {
        for(i=1;i<n;i++)
        {
            a=0;
            for(j=1;j<n;j++)
                a=t[i][j]+a;
 
            s[i]=a;
        }
    }
 
    for(i=0;i<n-1;i++)
    {
        if(s[i]!=s[i+1])
            test=1;
    }
 
    if(test==0)
    {
        a=0;
 
        for(i=0;i<n;i++)
            a=t[i][n-i]+a;
 
        if(s[j]!=a)
            test=1;
    }
 
    if(test==0)
    {
        a=0;
 
        for(i=0;i<n;i++)
            a=t[n-i][i]+a;
 
        if(s[j]!=a)
            test=1;
    }
 
    if(test==1)
        printf("le carre n'est pas magique \n");
    else
    {
        printf("le carre est pas magique \n");
        for(i=0;i<n;i++)
        {
            for(j=0;j<n;j++)
                printf("%d \t",t[i][j]);
            printf("\n");
        }
    }
    getche ();          
}

Note1: Le prototype de la fonction main() doit être un des suivant:

Code :

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

1
2
3
4
 
int main()
int main(int argc, char** argv)
int main(int argc, char** argv, char** arge)

Note2: la bibliothèque Conio est très désagréable car elle ne permet pas d'avoir du code qui fonctionne sous Linux. Surtout pour l'utilisation qui en est fait, je ne vois pas l'intérêt. Ainsi je vous conseillerai de remplacer toute référence à celle-ci par du code C classique.
(Oui oui, même getch() peut être fait dans le langage C, sans grande dificulté (voir la FAQ C)
Note3: Ce n'est pas getche() mais getch()
Note4: Depuis que j'ai réindenter le code, des erreurs (boucles :p) sont très visibles.

**komat** · 21/01/2011, 15h44

merci pour tous ça mais est ce que tu as compris ce que je veut réaliser
si tu as compris bien le but de se programme de magic carré tu peut me dire ou se trouve la faute est ce que je doit faire exactement sil vous plait ne travailler pas avec les pointeur je lé conné pas

**LittleWhite** · 21/01/2011, 16h20

Envoyé par komat

merci pour tous ça mais est ce que tu as compris ce que je veut réaliser
si tu as compris bien le but de se programme de magic carré tu peut me dire ou se trouve la faute est ce que je doit faire exactement sil vous plait ne travailler pas avec les pointeur je lé conné pas

Et bah, le truc c'est que je n'ai pas compris la question, ni meme le probleme.
Je veux la reponse aux choses suivantes:

Code :

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

1
2
3
4
5
6
7
 
Je vous pris de reformuler le problème car là, je n'ai rien compris du tout de ce qui était en place et de ce qui ne fonctionner pas.
Par explication du problème, j'entends:
- Votre réflexion sur la résolution du programme (algorithme)
- La description du bug:
- - Le résultat attendu
- - Le résultat que vous avez actuellement (et en quoi il est faut)

**komat** · 21/01/2011, 23h13

et bien tous simplement je veut faire un programme que le l utilisateur donne la taille du magic carré et aprés l utilisateur remplie le tableau a 2 dimension et aprés le programme affiche le resultas suivant: si les nombre qui ont eté saisie par l utisateur represente un magic carré le programme va afficher que c est un magic carré si le contraire il va affichier que c est pas une magic carré.
c est simple le programme fait seulement la recherche et affiche le resultas.
pour ma reflexion sur le code je l ai donné déja, mon code il n 'est pas complet je sait pas ou se trouve l erreur (pas erreur de compilation)
si vous avez une idé pour m aider a realiser mon programme ta qu a l dire
et merci d avance.

**LittleWhite** · 21/01/2011, 23h39

Pour les prochaines (effectif à partir d'ici) commentez votre code !

J'ai remarqué un bug:
- La variable de test n'est pas remise à zero avant chaque nouveau test.

Si vous pouviez me donner un carré magique de test, j'en serai heureux

Sinon, vous pouviez voir le problème avec un debuggueur, donc cela me ferai très plaisir que vous cherchiez à en utiliser un

**b_reda31** · 21/01/2011, 23h55

Bonjour,
1.
Etant donnée la longueur de votre code et que vous ne donnez aucune précision sur votre erreur, personne ne sera en mesure de vous aider.
Je simplement survolé le code et a première vu, je dirai qu'il y a un problème dans la variable "test", celle-ci est initialisé à 1 est aucune affection test=0 n'est faite (à moins que je l'ai loupé

), de cette manière la matrice ne pourra jamais être détectée comme matrice magique dans votre programme.

2.
+1 pour ce qu'a dit LittleWhite concernant le debogage : (Je suis 100% d'accord pour le reste aussi)
Essayer par exemple d'afficher la variable "test" aprés chaque étape de votre programme (Calcul de somme des lignes, des colonnes, puis des diagonales). Ceci vous aidera à repérer plus facilement ou est ce que l'erreur se situe.

Edit : Désolé pour la répétition je n'avais pas vu le précédent post

**komat** · 22/01/2011, 01h42

merci pour tous ça je vais vous donné un code qui fait le contraire l utilisateur entre la taille de la magic carré et le programme donne le resultas a partir de ce code je crois que vous pouver m aider a réaliser le contraire du programme que je veut realiser:

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
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
#include <stdio.h>
#include <stdlib.h>
#define N  100
 
int     a[N][N];// declarer un tableau de deux dimentions 
int     n;    //  declarer un entier n
 
void Init (int n) // Fonction qui initialise les cases du tableau a 0
{
    int i, j;
 
    for (i = 0 ; i < n; i++)
        for (j = 0; j < n; j++)
            a[i][j] = 0;  //Chaque case a 0
}
 
void Magique (int n)// Algorithme d'un carre magique
{
    int i, j, k;
 
    i = n - 1; j = n / 2;
    for (k = 1; k <=  n * n; k++) {
        a[i][j] = k;
        if ((k % n) == 0)
            i = i - 1;
        else {
            i = (i + 1) % n; 
            j = (j + 1) % n;
        }
    }
}
 
void Erreur (char s[])//Fonction qui affiche le message passer a la foncion
{
    printf ("Erreur fatale: %s\n", s);
    return;
}
 
void Lire (int *n) //fonction qui lit la taille du carre
{
    printf ("Taille du carre magique svp?:");
    scanf ("%d", n);
    if ((*n <= 0) || (*n > N) || (*n % 2 == 0)) 
        Erreur ("Taille impossible.");
}
 
void Imprimer (int n)//fonction qui affiche les diferentes cases du carre
{
    int i, j;
 
    for (i = 0; i < n; i++) {
        for (j = 0; j < n; j++) 
            printf ("%4d ", a[i][j]);
        printf ("\n");
    }
}
 
int main ()//Appele des differentes fonctions
{
    Lire(&n);
    Init(n);       /* - */
    Magique(n);
    Imprimer(n);
    system("pause");
    return 0;
}

**LittleWhite** · 22/01/2011, 02h03

Dites moi ? il est a vous ce code ?
De plus, je n'ai pas compris le principe de faire l'inverse à partir d'un code qui fait autre chose.

Et troisièmement, on vous a aidé sur le premier code ... pourquoi ne continuons pas sur votre premier code ?

**komat** · 22/01/2011, 13h48

ragarde le code. d abord le remplissage du tableau et aprés une boucle pour demander a lutisaleur de saisir des nombre essay de ragerder dans les boucles des traitement et la derniére condition je crois que la faute existe dans cet endroit si ta une idéé de changer quelque chose ta qua le faire

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
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
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
#include<stdio.h>
 
#include<stdlib.h>
 
#include<conio.h>
 
main()
 
{
 
      int t[50][50],a=0,s[50],n,test=1,i,j;
 
      printf("Donner la taille du carre  \t");
 
      scanf("%d",&n);
 
      printf("Remplissez le tableau \t");// remplissage des 2 tableaux
 
      for(i=0;i<n;i++)
 
      {
 
                      for(j=0;j<n;j++)
 
                      {
 
                                      printf("Tapez la case T[%d][%d]\n",i+1,j+1);// l utilisateur entre les nombres 
 
                                      scanf("%d",&t[i][j]);
 
                      }
 
      }
 
 
 
      for(j=0;j<n;j++)
 
                      {
 
                                      s[j]=0;
 
                      }
 
      for(j=0;j<n;j++)
 
      {a=0;
 
                      for(i=0;i<n;i++)
 
                      a=t[i][j]+a;
 
      s[j]=a;}
 
      for(j=0;j<n;j++)
 
      {
 
                        if(s[j]!=s[j+1])
 
                        test=1;
 
      }
 
 
 
      if(test==0)
 
      {for(i=1;i<n;i++)
 
      {a=0;
 
                      for(j=1;j<n;j++)
 
                      a=t[i][j]+a;
 
      s[i]=a;}}
 
 
 
      for(i=0;i<n-1;i++)
 
      {
 
                        if(s[i]!=s[i+1])
 
                        test=1;
 
      }
 
 
 
      if(test==0)
 
      {a=0;
 
                 for(i=0;i<n;i++)
 
                 a=t[i][n-i]+a;
 
                 if(s[j]!=a)
 
                 test=1;
 
                 }
 
 
 
      if(test==0)
 
      {a=0;
 
                 for(i=0;i<n;i++)
 
                 a=t[n-i][i]+a;
 
                 if(s[j]!=a)
 
                 test=1;
 
                 }
 
 
 
      if(test==1)
 
      printf("le carre n'est pas magique \n");
 
      else
 
      {
 
          printf("le carre est magique \n");
 
            for(i=0;i<n;i++)
 
      {
 
                      for(j=0;j<n;j++)
 
 
 
                                      printf("%d \t",t[i][j]);
 
                                      printf("\n");
 
 
 
      }
 
      }
 
          getch();          
          }

**LittleWhite** · 22/01/2011, 14h15

Non mais j'ai compris (et j'ai même commenté le code chez moi).
Ce que je demande c'est:

Commentez votre code, dans le code en lui-même et non dans le message sur le forum (vous verrez c'est bénéfique pour tous)
Écoutez-moi lorsque je vous dis que je suspecte une erreur avec la variable test (nous sommes deux à le dire)
Comme je prétends avoir une solution, j'aimerai avoir un carré magique valide. Pour faire un essai. Merci.

**Sve@r** · 22/01/2011, 22h52

Envoyé par komat

ragarde le code. d abord le remplissage du tableau et aprés une boucle pour demander a lutisaleur de saisir des nombre essay de ragerder dans les boucles des traitement et la derniére condition je crois que la faute existe dans cet endroit si ta une idéé de changer quelque chose ta qua le faire

Moi je changerais bien l'auteur de ce topic (qui a réussi quand-même l'exploit d'avoir -16 pts en 42 posts) mais bon, je sens que ça ne va pas être possible donc...

Envoyé par komat

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
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
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
#include<stdio.h>
 
#include<stdlib.h>
 
#include<conio.h>
 
main()
 
{
 
      int t[50][50],a=0,s[50],n,test=1,i,j;
 
      printf("Donner la taille du carre  \t");
 
      scanf("%d",&n);
 
      printf("Remplissez le tableau \t");// remplissage des 2 tableaux
 
      for(i=0;i<n;i++)
 
      {
 
                      for(j=0;j<n;j++)
 
                      {
 
                                      printf("Tapez la case T[%d][%d]\n",i+1,j+1);// l utilisateur entre les nombres 
 
                                      scanf("%d",&t[i][j]);
 
                      }
 
      }
 
 
 
      for(j=0;j<n;j++)
 
                      {
 
                                      s[j]=0;
 
                      }
 
      for(j=0;j<n;j++)
 
      {a=0;
 
                      for(i=0;i<n;i++)
 
                      a=t[i][j]+a;
 
      s[j]=a;}
 
      for(j=0;j<n;j++)
 
      {
 
                        if(s[j]!=s[j+1])
 
                        test=1;
 
      }
 
 
 
      if(test==0)
 
      {for(i=1;i<n;i++)
 
      {a=0;
 
                      for(j=1;j<n;j++)
 
                      a=t[i][j]+a;
 
      s[i]=a;}}
 
 
 
      for(i=0;i<n-1;i++)
 
      {
 
                        if(s[i]!=s[i+1])
 
                        test=1;
 
      }
 
 
 
      if(test==0)
 
      {a=0;
 
                 for(i=0;i<n;i++)
 
                 a=t[i][n-i]+a;
 
                 if(s[j]!=a)
 
                 test=1;
 
                 }
 
 
 
      if(test==0)
 
      {a=0;
 
                 for(i=0;i<n;i++)
 
                 a=t[n-i][i]+a;
 
                 if(s[j]!=a)
 
                 test=1;
 
                 }
 
 
 
      if(test==1)
 
      printf("le carre n'est pas magique \n");
 
      else
 
      {
 
          printf("le carre est magique \n");
 
            for(i=0;i<n;i++)
 
      {
 
                      for(j=0;j<n;j++)
 
 
 
                                      printf("%d \t",t[i][j]);
 
                                      printf("\n");
 
 
 
      }
 
      }
 
          getch();          
          }

Bon, donc comme tout le monde l'a déjà dit
1) main est de type int
2) test est initialisé à 1 et jamais mis à 0 alors que son évaluation sur 0 est faite plusieurs fois
3) a mon avis, ce code a beaucoup trop de commentaires

Voici en sus deux remarques inédites:
1) laisser une ligne vide entre 2 lignes de code a sûrement beaucoup de style mais cela n'aide pas à la relecture
2) essaye de découper ton code en tâches précises. Par exemple une fonction pour saisir, une pour vérifier, une pour afficher serait une super, mais alors vraiment super idée. Comme ça, si un truc ne fonctionne pas, tu peux mieux détecter où cela se passe...

Envoyé par LittleWhite

Non mais j'ai compris (et j'ai même commenté le code chez moi).

Tu voudrais pas le commenter pour nous STP ? Allez, pour la bonne cause, pense à tous les intervenants de qualité qui voient cette horreur et qui doivent faire avec alors qu'ils ne l'ont pas mérité. Tu veux pas le faire pour eux ? C'est toute la communauté developpez.net qui parle par ma bouche...

Envoyé par LittleWhite

Comme je prétends avoir une solution, j'aimerai avoir un carré magique valide. Pour faire un essai. Merci.

Pour ça, pas de problème. Il existe une méthode très facile pour créer un carré magique d'ordre impair
1) on pose le nombre 1 dans la case du haut de la colonne milieu
2) on écrit ensuite les autres nombres (2; 3; ...) dans les cases suivantes en établissant le schéma suivant: si la case située au dessus et à droite est libre, alors le nombre va dans cette case sinon on le place juste en dessous de là où on se trouve
Bien entendu, quand on dépasse du carré on reprend de l'autre coté (comme si le carré était en fait un globe). Donc si on est en première ligne, la case au dessus sera alors en dernière ligne. De même si on est tout à droite, la case encore à droite sera alors en colonne de gauche.

Ce qui donne le carré suivant (pour 5)
Ligne 1: 17; 24; 1; 8; 15
Ligne 2: 23; 5; 7; 14; 16
Ligne 3: 4; 6; 13; 20; 22
Ligne 4: 10; 12; 19; 21; 3
Ligne 5: 11; 18; 25; 2; 9

**LittleWhite** · 23/01/2011, 00h12

Envoyé par Sve@r

1) laisser une ligne vide entre 2 lignes de code a sûrement beaucoup de style mais cela n'aide pas à la relecture

Je pensais à un problème d'editeur 2 DVP ... mais bon ... je suis entièrement d'accord avec vous.

Tu voudrais pas le commenter pour nous STP ? Allez, pour la bonne cause, pense à tous les intervenants de qualité qui voient cette horreur et qui doivent faire avec alors qu'ils ne l'ont pas mérité. Tu veux pas le faire pour eux ? C'est toute la communauté developpez.net qui parle par ma bouche...

Voilà:

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
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
 
#include <stdio.h>
#include <stdlib.h>
 
main()
{
    int t[50][50],a=0,s[50],n,test=1,i,j;
 
    printf("Donner la taille du carre  \t");
    scanf("%d",&n);
 
    printf("Remplissez le tableau \t");
 
    for(i=0;i<n;i++)
    {
        for(j=0;j<n;j++)
        {
            printf("Tapez la case T[%d][%d]\n",i+1,j+1);
            scanf("%d",&t[i][j]);
        }
    }
 
     // Initialisation tableau de solution
    for(j=0;j<n;j++)
    {
        s[j]=0;
    }
 
 
    // Somme des cases en lignes
    for(j=0;j<n;j++)
    {
        a=0;
 
        for(i=0;i<n;i++)
        {
            a=t[i][j]+a;
        }
 
        s[j]=a;
    }
 
    // On regarde si tout les lignes sont équivalents
    for(j=0;j<n;j++)
    {
        if(s[j]!=s[j+1])
            test=1;
    }
 
    if(test==0)
    {
        // Tests des colonnes
        for(i=1;i<n;i++)
        {
            a=0;
            for(j=1;j<n;j++)
            {
                a=t[i][j]+a;
            }
 
            s[i]=a;
        }
    }
 
 
    // On regarde les solutions (faut remettre le test à zero, sinon il sera toujours à 1)
    test = 0;
    for(i=0;i<n-1;i++)
    {
        if(s[i]!=s[i+1])
            test=1;
    }
 
    if(test==0)
    {
        test=0;
        a=0;
 
        for(i=0;i<n;i++)
            a=t[i][n-i]+a;
 
        if(s[j]!=a)
            test=1;
    }
 
    if(test==0)
    {
        test=0;
        a=0;
 
        for(i=0;i<n;i++)
            a=t[n-i][i]+a;
 
        if(s[j]!=a)
            test=1;
    }
 
    if(test==1)
        printf("le carre n'est pas magique \n");
    else
    {
        printf("le carre est pas magique \n");
        for(i=0;i<n;i++)
        {
            for(j=0;j<n;j++)
                printf("%d \t",t[i][j]);
            printf("\n");
        }
    }
}

Ce qui donne le carré suivant (pour 5)
Ligne 1: 17; 24; 1; 8; 15
Ligne 2: 23; 5; 7; 14; 16
Ligne 3: 4; 6; 13; 20; 22
Ligne 4: 10; 12; 19; 21; 3
Ligne 5: 11; 18; 25; 2; 9

Merci beaucoup ... mais bon ... vous faites un peu comme moi ... vous travaillez pour l'ouvreur de la discussion.

---

En faisant les tests, j'ai remarqué que la problème avec test était vraiment beaucoup plus profond que cela.
Je propose de faire une incrémentation à chaque fois qu'un test est réussi.

Je pense même qu'il faut arrêter tout ces if sur test afin de faciliter le programme.
J'ai enfin une solution (réelle) je pense

... vu que je perd la tête dans cette multitude de test

(ma concentration est complètement partie en voyage)

EDIT: Ah non ... je me suis vraiment perdu dans le programme

**quetzacoatl** · 23/01/2011, 00h22

Je pense qu'une partie du problème peut-être résolue en initialisant test à 0.
En effet ici on l'initialise à 1.
Puis on fait:

for(j=0;j<n;j++)
{
if(s[j]!=s[j+1])
test=1;
}

if(test==0)

C'est la première utilisation de la variable test, donc étant donné qu'on l'a initialisé à 1, aucune chance qu'elle passe le test.

**Sve@r** · 23/01/2011, 14h55

Envoyé par LittleWhite

Merci beaucoup ... mais bon ... vous faites un peu comme moi ... vous travaillez pour l'ouvreur de la discussion.

Ben oui. Ca fait tellement mal au bide de voir un truc aussi horrible qu'on a forcément envie de réécrire un code élégant à la place de cette horreur rien que pour ne plus nous faire saigner les yeux. Par exemple déjà en créant une fonction "sommeLigne()" permettant d'avoir la somme d'une ligne x et une fonction "sommeCol()" permettant l'équivalent pour une colonne y (voire même une seule fonction à laquelle on passe le sens lig/col) ça donnerait déjà un premier point de départ. Mais bon, l'ouvreur ayant disparu de la scène, suis pas certain de revenir ici bien souvent. Pas grave. L'important c'est que moi (et toi et la majorité des intervenants habituels de la cat. C) nous nous sachions tous capables de réussir cet exo et en plus de le réussir avec style quoi...

Tester un carré magique

C

Discussions similaires

Partager

Partager