Aidez moi pour un programme en C
Résolu
stephe8
Messages postés
65
Date d'inscription
mercredi 23 juillet 2008
Statut
Membre
Dernière intervention
24 octobre 2009
-
20 févr. 2009 à 19:57
stephe8 Messages postés 65 Date d'inscription mercredi 23 juillet 2008 Statut Membre Dernière intervention 24 octobre 2009 - 22 févr. 2009 à 00:48
stephe8 Messages postés 65 Date d'inscription mercredi 23 juillet 2008 Statut Membre Dernière intervention 24 octobre 2009 - 22 févr. 2009 à 00:48
A voir également:
- Aidez moi pour un programme en C
- Écrire un programme qui calcule la moyenne des notes en c - Forum C
- Programmer en c - Conseils pratiques -C / C++ / C++.NET
- Applications et programme d'interface - Conseils pratiques -Graphisme
- Un programme tente d'envoyer un courrier en votre nom ✓ - Forum VBA Office (Excel, Word ...)
- Programme python jeu mario ✓ - Forum Python
3 réponses
cs_rt15
Messages postés
3874
Date d'inscription
mardi 8 mars 2005
Statut
Modérateur
Dernière intervention
7 novembre 2014
13
21 févr. 2009 à 23:20
21 févr. 2009 à 23:20
Salut,
Rooooh. La fausse joie. Un moment je me suis dit : "stephe -> Stéphanie". Il se trouve que je suis un grand fan des Stéphanies. Mais là c'est raté. Bref.
Impressionant le nombre d'étudiants étrangés qui postent ces derniers temps.
Et avec un peu toujours la même façon de coder et de poster le message. Encore que toi tu as pas trop mal posé la question. Si si. Le petit "corrigé le svp ou bien donnez moi des conseils". On aime.
Mais je m'égare. Et je vais encore en faire des kilos.
Sache, jeune stephe8, qu'un code ce n'est pas un texte. C'est un poème. Une oeuvre d'art. Faut que chaque ligne parle de celui qui l'a écrite. Faut que ça ait un début et une fin. Faut que ce soit beau. Faut pas bacler le boulot. Faut respecter les normes et les habitudes, dans la mesure du possible. Bref, faut au moins aimer son code juste après l'avoir écrit. On le détestera un an plus tard mais ce n'est pas le problème.
1) Indentation et aspect du code :
## La première chose qui choque dans ton code, comme ça, de visu, c'est l'indentation. L'indentation n'a aucun sens pour le compilateur. Pourtant elle est fondamentale pour la relecture du code. Une mauvaise indentation peut sans problème provoquer une mauvaise compréhension du code. Faut qu'elle soit réalisée au quart de poil. Et faut pas se dire "j'indenterai à la fin". On indente tout de suite.
Et là ton indentation est hasardeuse, bien qu'on remarque quelques efforts. Je ne sais pas si tu as utilisé le caractère tabulation, mais c'est à éviter. On aboutit vite à du n'importe quoi car les tabulation sont traitées différement d'un éditeur à l'autre. Il faut faire des espaces. On peut souvent configurer les éditeurs pour qu'une pression sur la touche tab soit transformée en un certain nombre d'espace. Concernant la taille de l'indentation, on utilise généralement 2 ou 4 espaces. Utiliser deux espaces permet de ne pas avoir du code trop à droite sur l'écran et est plus rapide à l'édition. Utiliser 4 espaces est considéré par certains comme plus lisible.
## De même que pour l'indentation, il faut toujours positionner les espaces optionnels de la même façon. for() ou for (). Mais pas for avec ou sans espace entre for et () dans le même code. Les {} si optionnelles, on les met généralement pas. Si on les mets quand elles sont optionnelles on les met toujours. Bref, il faut suivre son propre guide de style à la lettre.
## Les commentaires, c'est important aussi.
2) Remarques syntaxiques :
## Valeur de retour du main :
return(0);
return 0; Une sortie de fonction n'est pas un appel de fonction.
## Type de retour du main :
Ton main dois renvoyer un int.
3) Structure générale du code :
On essaie généralement de faire des fonctions de petites tailles dédiées à un travail précis. Cela augmente de beaucoup la lisibilité, la maintenabilité, la réusabilité...
4) Bonnes pratiques :
## Il faut généralement tester les codes d'erreur des fonctions suceptibles d'échouer (Ouvertures de fichier...) et on informe l'utilisateur en cas de pépin.
## Il faut éviter absolument de rédiger deux fois le même code. Les copier coller sont interdits. Faire une boucle, un fonction... Mais jamais deux fois un code similaire !
## Il faut éviter d'utiliser des headers non standards. Comme conio.h.
## Il faut compiler au moins avec -Wall si tu utilises gcc, pour avoir les warnings, qui sont souvent des erreurs ou des imprécision génantes.
5) Erreurs :
## Boucle en trop :
for(i=0;i<dimA;i++)
fscanf(fichierB,"%d",&dimB);
La tu risque quelques ennuis...
## Un peu partout :
fscanf(fichierB, "%d", &dimB);
fscanf(fichierA, "%d", &dimA);
Pourquoi relire les tailles ? Elles sont plusieurs fois dans le fichier ? Inutile. Et attention, la lecture d'un fichier ne recommence pas au début par l'opération du saint esprit.
## k non initialisé :
k n'est pas initialisé avant d'être utilisé.
## Condition de sortie de boucle :
while((i<dimA)&&(i<dimB))
Tu aurais du mettre un j quelque part.
## Affectation bizarre :
dimA = A[0];
dimB = B[0];
D'une part A et B ne sont pas encore initialisé, mais en plus tu ne te sert pas de dimA et dimB avant des les réaffecter.
## Deuxième et troisième boucle while :
Elles servent à récupérer les valeurs du tableau que l'on a pas entièrement parcouru après la première boucle while. Il ne faut donc pas toucher à i et j avant de rentrer dedans.
C[k]=A[j];
Dans la deuxième boucle. Tu voulais mettre B ici.
## Globalement :
D'une manière générale : de la rigueur !!! Sinon tu n'arrivera qu'à faire des bugs !!!
6) Ma solution :
Je propose d'utiliser la première case des tableaux pour stocker leurs tailles.
Je suppose A croissant, B décroissant, et je produit C croissant.
<hr size="2" width="100%" />Fichier A :
6 5 9 12 22 33 806
Fichier B :
6 500 21 15 9 3 1
Fichier C :
12 1 3 5 9 9 12 15 21 22 33 500 806
<hr size="2" width="100%" />#include <stdio.h>
#define FILE_A "fichierA.TXT"
#define FILE_B "fichierB.TXT"
#define FILE_C "fichierC.TXT"
void PrintArray(char *lpArrayName, int *lpVals)
{
int i;
printf("%s : ", lpArrayName);
for (i = 1; i <= lpVals[0]; i++)
printf("%d ", lpVals[i]);
printf("\n");
}
int ReadArrayFromFile(char *lpFileName, int *lpVals)
{
FILE* lpFile;
int i;
lpFile = fopen(lpFileName, "r");
if (lpFile == NULL) return 0;
fscanf(lpFile, "%d", &lpVals[0]);
for (i = 1; i <= lpVals[0]; i++)
fscanf(lpFile, "%d", &lpVals[i]);
fclose(lpFile);
return -1;
}
int WriteArrayToFile(char *lpFileName, int *lpVals)
{
FILE* lpFile;
int i;
lpFile = fopen(lpFileName, "w");
if (lpFile == NULL) return 0;
for (i = 0; i <= lpVals[0]; i++)
fprintf(lpFile, "%d ", lpVals[i]);
fclose(lpFile);
return -1;
}
int main()
{
int A[50], B[50], C[100];
int i, j, k;
if (! ReadArrayFromFile(FILE_A, A))
{
printf("Echec de l'ouverture de %s\n", FILE_A);
return 1;
}
if (! ReadArrayFromFile(FILE_B, B))
{
printf("Echec de l'ouverture de %s\n", FILE_B);
return 1;
}
PrintArray("A", A);
PrintArray("B", B);
C[0] = A[0] + B[0];
i = 1;
j = B[0];
k = 1;
while ((i <= A[0]) && (j > 0))
{
if (A[i] < B[j])
{
C[k] = A[i];
i++;
k++;
}
else
{
C[k] = B[j];
j--;
k++;
}
}
while(i <= A[0])
{
C[k] = A[i];
i++;
k++;
}
while (j > 0)
{
C[k] = B[j];
j--;
k++;
}
PrintArray("C", C);
if (! WriteArrayToFile(FILE_C, C))
{
printf("Echec de l'ouverture de %s\n", FILE_C);
return 1;
}
return 0;
}
Rooooh. La fausse joie. Un moment je me suis dit : "stephe -> Stéphanie". Il se trouve que je suis un grand fan des Stéphanies. Mais là c'est raté. Bref.
Impressionant le nombre d'étudiants étrangés qui postent ces derniers temps.
Et avec un peu toujours la même façon de coder et de poster le message. Encore que toi tu as pas trop mal posé la question. Si si. Le petit "corrigé le svp ou bien donnez moi des conseils". On aime.
Mais je m'égare. Et je vais encore en faire des kilos.
Sache, jeune stephe8, qu'un code ce n'est pas un texte. C'est un poème. Une oeuvre d'art. Faut que chaque ligne parle de celui qui l'a écrite. Faut que ça ait un début et une fin. Faut que ce soit beau. Faut pas bacler le boulot. Faut respecter les normes et les habitudes, dans la mesure du possible. Bref, faut au moins aimer son code juste après l'avoir écrit. On le détestera un an plus tard mais ce n'est pas le problème.
1) Indentation et aspect du code :
## La première chose qui choque dans ton code, comme ça, de visu, c'est l'indentation. L'indentation n'a aucun sens pour le compilateur. Pourtant elle est fondamentale pour la relecture du code. Une mauvaise indentation peut sans problème provoquer une mauvaise compréhension du code. Faut qu'elle soit réalisée au quart de poil. Et faut pas se dire "j'indenterai à la fin". On indente tout de suite.
Et là ton indentation est hasardeuse, bien qu'on remarque quelques efforts. Je ne sais pas si tu as utilisé le caractère tabulation, mais c'est à éviter. On aboutit vite à du n'importe quoi car les tabulation sont traitées différement d'un éditeur à l'autre. Il faut faire des espaces. On peut souvent configurer les éditeurs pour qu'une pression sur la touche tab soit transformée en un certain nombre d'espace. Concernant la taille de l'indentation, on utilise généralement 2 ou 4 espaces. Utiliser deux espaces permet de ne pas avoir du code trop à droite sur l'écran et est plus rapide à l'édition. Utiliser 4 espaces est considéré par certains comme plus lisible.
## De même que pour l'indentation, il faut toujours positionner les espaces optionnels de la même façon. for() ou for (). Mais pas for avec ou sans espace entre for et () dans le même code. Les {} si optionnelles, on les met généralement pas. Si on les mets quand elles sont optionnelles on les met toujours. Bref, il faut suivre son propre guide de style à la lettre.
## Les commentaires, c'est important aussi.
2) Remarques syntaxiques :
## Valeur de retour du main :
return(0);
return 0; Une sortie de fonction n'est pas un appel de fonction.
## Type de retour du main :
Ton main dois renvoyer un int.
3) Structure générale du code :
On essaie généralement de faire des fonctions de petites tailles dédiées à un travail précis. Cela augmente de beaucoup la lisibilité, la maintenabilité, la réusabilité...
4) Bonnes pratiques :
## Il faut généralement tester les codes d'erreur des fonctions suceptibles d'échouer (Ouvertures de fichier...) et on informe l'utilisateur en cas de pépin.
## Il faut éviter absolument de rédiger deux fois le même code. Les copier coller sont interdits. Faire une boucle, un fonction... Mais jamais deux fois un code similaire !
## Il faut éviter d'utiliser des headers non standards. Comme conio.h.
## Il faut compiler au moins avec -Wall si tu utilises gcc, pour avoir les warnings, qui sont souvent des erreurs ou des imprécision génantes.
5) Erreurs :
## Boucle en trop :
for(i=0;i<dimA;i++)
fscanf(fichierB,"%d",&dimB);
La tu risque quelques ennuis...
## Un peu partout :
fscanf(fichierB, "%d", &dimB);
fscanf(fichierA, "%d", &dimA);
Pourquoi relire les tailles ? Elles sont plusieurs fois dans le fichier ? Inutile. Et attention, la lecture d'un fichier ne recommence pas au début par l'opération du saint esprit.
## k non initialisé :
k n'est pas initialisé avant d'être utilisé.
## Condition de sortie de boucle :
while((i<dimA)&&(i<dimB))
Tu aurais du mettre un j quelque part.
## Affectation bizarre :
dimA = A[0];
dimB = B[0];
D'une part A et B ne sont pas encore initialisé, mais en plus tu ne te sert pas de dimA et dimB avant des les réaffecter.
## Deuxième et troisième boucle while :
Elles servent à récupérer les valeurs du tableau que l'on a pas entièrement parcouru après la première boucle while. Il ne faut donc pas toucher à i et j avant de rentrer dedans.
C[k]=A[j];
Dans la deuxième boucle. Tu voulais mettre B ici.
## Globalement :
D'une manière générale : de la rigueur !!! Sinon tu n'arrivera qu'à faire des bugs !!!
6) Ma solution :
Je propose d'utiliser la première case des tableaux pour stocker leurs tailles.
Je suppose A croissant, B décroissant, et je produit C croissant.
<hr size="2" width="100%" />Fichier A :
6 5 9 12 22 33 806
Fichier B :
6 500 21 15 9 3 1
Fichier C :
12 1 3 5 9 9 12 15 21 22 33 500 806
<hr size="2" width="100%" />#include <stdio.h>
#define FILE_A "fichierA.TXT"
#define FILE_B "fichierB.TXT"
#define FILE_C "fichierC.TXT"
void PrintArray(char *lpArrayName, int *lpVals)
{
int i;
printf("%s : ", lpArrayName);
for (i = 1; i <= lpVals[0]; i++)
printf("%d ", lpVals[i]);
printf("\n");
}
int ReadArrayFromFile(char *lpFileName, int *lpVals)
{
FILE* lpFile;
int i;
lpFile = fopen(lpFileName, "r");
if (lpFile == NULL) return 0;
fscanf(lpFile, "%d", &lpVals[0]);
for (i = 1; i <= lpVals[0]; i++)
fscanf(lpFile, "%d", &lpVals[i]);
fclose(lpFile);
return -1;
}
int WriteArrayToFile(char *lpFileName, int *lpVals)
{
FILE* lpFile;
int i;
lpFile = fopen(lpFileName, "w");
if (lpFile == NULL) return 0;
for (i = 0; i <= lpVals[0]; i++)
fprintf(lpFile, "%d ", lpVals[i]);
fclose(lpFile);
return -1;
}
int main()
{
int A[50], B[50], C[100];
int i, j, k;
if (! ReadArrayFromFile(FILE_A, A))
{
printf("Echec de l'ouverture de %s\n", FILE_A);
return 1;
}
if (! ReadArrayFromFile(FILE_B, B))
{
printf("Echec de l'ouverture de %s\n", FILE_B);
return 1;
}
PrintArray("A", A);
PrintArray("B", B);
C[0] = A[0] + B[0];
i = 1;
j = B[0];
k = 1;
while ((i <= A[0]) && (j > 0))
{
if (A[i] < B[j])
{
C[k] = A[i];
i++;
k++;
}
else
{
C[k] = B[j];
j--;
k++;
}
}
while(i <= A[0])
{
C[k] = A[i];
i++;
k++;
}
while (j > 0)
{
C[k] = B[j];
j--;
k++;
}
PrintArray("C", C);
if (! WriteArrayToFile(FILE_C, C))
{
printf("Echec de l'ouverture de %s\n", FILE_C);
return 1;
}
return 0;
}
stephe8
Messages postés
65
Date d'inscription
mercredi 23 juillet 2008
Statut
Membre
Dernière intervention
24 octobre 2009
20 févr. 2009 à 23:05
20 févr. 2009 à 23:05
svp aidez moi
stephe8
Messages postés
65
Date d'inscription
mercredi 23 juillet 2008
Statut
Membre
Dernière intervention
24 octobre 2009
22 févr. 2009 à 00:48
22 févr. 2009 à 00:48
merci bcp pour ta gentillesse et de ton aide anisi pour tes conseil
