programme du suivi de croissance de komate

bonjours à tous,
voilà je suit une formation de programmeur, j'apprend le C et C++. Dans le cadre de cette formation j'ai entrepris d'écrire un rpogramme pour suivre la croissance de ma jeune tortue komate. Le code source que j'ai écrit fonctionne, mais à mon avis, il y a moyen de épurer un peu. J'aurai souhaité vos conseilles pour améliorer ce scrript. Merci :D

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

#include<iostream> #include<stdio.h> #include<stdlib.h> #include <string.h> using namespace std; main() { char NOM_FICHIER[100] = "/var/www/komate/crois.txt"; FILE *FICHIER; int DATE; int POID; int TAILLE; do{ cout<<"Quelles date sommes nous ?: format JJMMAA\n"; cin>>DATE; }while((DATE<010112)||(DATE>311214)); do{ cout<<"Combien de gramme pese komate ?: \n"; cin>>POID; }while((POID<89)||(POID>500)); do{ cout<<"Combien de millimetre mesure komate ?: \n"; cin>>TAILLE; }while((TAILLE<0)||(TAILLE>500)); //affichage resultat cout<<"Au " << DATE <<" komate pese "<<POID <<" grammes et mesure "<< TAILLE<<" millimetres \n" <<FICHIER; FICHIER = fopen(NOM_FICHIER, "a"); fprintf(FICHIER, "%d | %d | %d\n",DATE, POID,TAILLE);     return 0; }

---

Une chose qui pourrait être bien, ce serait de pouvoir revenir en arrière.
Une faute de frappe peut toujours arriver.

Tu pourrais soit proposer un menu pour modifier les paramètres entrés juste avant d'écrire dans le fichier, soit si l'utilisateur tape -1, on revient en arrière.

Ensuite, pour l'écriture dans ton fichier, je te conseillerais de mettre des \t plutôt que des espaces.

EDIT : pour ta date, tu ne vérifie pas la validité de la date ex : 12/58/2013 n'est pas valide

Merci pour ta réponse, je vais continuer à améliorer ce petit programme.

En effet c'est une super idée de pouvoir modifier le fichier txt si l'utilisateur a commit une faute de frappe. Je vais aussi réfléchir à vérifier de manière efficace la date.
Merci
++

Un petite remarque de style : Il est rare de voir des MAJUSCULES pour des noms de variable, on réserve plus souvent ça pour des constantes.

Sinon, tu utilises les flux (cin, cout) pour lire/écrire des informations à l'écran, mais pour écrire dans un fichier, tu utilises une autre technique, issue du C, alors que les flux marchent aussi :

Code:

---

1 2 3 4 5 6 7

#include <fstream> using namespace std; int main() { ofstream fichier("/var/www/komate/crois.txt"); fichier << "Aujourd'hui " << 42 << endl; }

---

Merci, encore pour ces quelques conseils et explications. Du coup j'ai apporté un certain nombre de modifications, notamment au niveau de la saisi par l'utilisateur de la date de mesure (Partie de code qui assure la validité de saisi de la date).
Je n'ai pas trop compris comment utilisé le bout de code que tu as laissé JolyLoïc, c'est dommage parce qu'il m'a l'aire plus simple d'utilisation que le tout le remue ménage de mon code.
Du coup j'ai réécrit mon code source, compilé, et la...
le programme se lance
c'est pas mal j'aime bien. :D
Peut-être voyez vous encore des améliorations à apporter ?

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

#include<iostream> #include<stdio.h> #include<stdlib.h> #include <string.h> #include <fstream>   using namespace std; main() { char NOM_FICHIER[100] = "/home/sk8trasher/komate/komate_pousse.txt"; FILE *FICHIER; int jour; int mois; int annee; int POID; int TAILLE; do{ cout<<"Quelle date sommes nous ? \n"; cout<<"Saisir le jour (01, 02, 03, ...30,31) \n"; cin>>jour; }while((jour<=0)||(jour>31)); do{ cout<<"Saisir le mois (01, 02, 03, ...11, 12) \n"; cin>>mois; }while((mois<=0)||(mois>31)); do{ cout<<"Saisir l'annee (2012, 2013, 2014...2049, 2050) \n"; cin>>annee; }while(annee<2012);   cout<< "Nous sommes le" <<jour<<"/"<<mois<<"/"<<annee<<endl; do{ cout<<"Combien de gramme pese komate ?: \n"; cin>>POID; }while((POID<89)||(POID>500)); do{ cout<<"Combien de millimetre mesure komate ?: \n"; cin>>TAILLE; }while((TAILLE<0)||(TAILLE>500)); //affichage resultat cout<<"le " <<jour<<"/"<<mois<<"/"<<annee<<" komate pese "<<POID <<" grammes et mesure "<<TAILLE<<" millimetres \n" <<FICHIER; FICHIER = fopen(NOM_FICHIER, "a"); fprintf(FICHIER,"%d\t%d\t%d\t%d\t%d\t\n",jour, mois, annee, POID,TAILLE); return 0; }

---

Code:

---

1 2	char * NOM_FICHIER = "/home/sk8trasher/komate/komate_pousse.txt";

---

à la place de :

Code:

---

1 2	char NOM_FICHIER[100] = "/home/sk8trasher/komate/komate_pousse.txt";

---

me semble plus intéressant car tu ne prend que l'espace dont tu as besoin (en plus, il ne me semble pas que tu modifie ta chaîne de caractère).

Bonjour,

Quitte à faire de C++, autant utiliser un string.

Citation:

---

Envoyé par sk8trasher

Peut-être voyez vous encore des améliorations à apporter ?

---

Fermer le fichier: fclose(FICHIER) à la fin?
Pour mieux comprendre le code de JolyLoic, tu peux regarder la :faq: Manipulation des fichiers. La syntaxe pour écrire reste la même que pour cout.

S'il fait un fopen, quel utilité d'utiliser un string pour ensuite utiliser :

mon_string.c_str() pour convertir en char * pour le fopen?

La solution de JolyLoic n'est pourtant pas trop chinoise, à la place de :

Code:

---

1 2	FICHIER = fopen(NOM_FICHIER, "a"); fprintf(FICHIER,"%d\t%d\t%d\t%d\t%d\t\n",jour, mois, annee, POID,TAILLE);

---

tu mets (et tu n'oublies pas d'inclure <fstream>) :

Code:

---

1 2	ofstream fichier(NOM_FICHIER); fichier << jour << '\t' << mois << '\t' << annee << '\t' << poid << '\t' << taille << std::endl;

---

le std::endl remplace le '\n' (que tu peux quand même utiliser si besoin...)

D'autre part, pour améliorer ton programme il serait peut-être intéressant d'utiliser des fonctions pour isoler les saisies utilisateurs.

En même temps, s'il apprend le C/C++, je pense qu'il va essayer d'utiliser ces deux langages dans un même programme pour tester ses connaissances.
D'où l'utilisation de son "fopen"

Si je peux me permettre, il vaudrait mieux qu'il étudie ces deux langages séparément.

Car le jour où il faudra pour une raison X ou Y compiler en C, il ne pourra plus utiliser les fonctions C++ et il risque de se retrouver assez déboussolé.

Puis à faire des mixtes des deux langages, certes ça compile et ça marche, mais, c'est assez dangereux de les mélanger comme cela en effet, un

Code:

---

1 2	string *test = malloc(sizeof(string));

---

a toute les chances de faire un SEGFAULT plus tard.

Citation:

---

Envoyé par Neckara

Puis à faire des mixtes des deux langages, certes ça compile et ça marche, mais, c'est assez dangereux de les mélanger comme cela en effet, un

Code:

---

1 2	string *test = malloc(sizeof(string));

---

a toute les chances de faire un SEGFAULT plus tard.

---

C'est surtout n'importe quoi de mettre dans un pointeur typé classe l'adresse d'une zone allouée avec malloc sur laquelle aucune initialisation n'a été faite vu que malloc n'appelle pas le constructeur...d'ailleurs pour une chaine locale pas besoin de faire de l'allocation dynamique avec tous les risques de bévues que cela comporte

Citation:

---

Envoyé par therwald

C'est surtout n'importe quoi de mettre dans un pointeur typé classe l'adresse d'une zone allouée avec malloc sur laquelle aucune initialisation n'a été faite vu que malloc n'appelle pas le constructeur...d'ailleurs pour une chaine locale pas besoin de faire de l'allocation dynamique avec tous les risques de bévues que cela comporte

---

C'est justement pour cela qu'il ne faut pas mélanger le C et le C++.
En C++ certaines variables sont initialisées à leur déclaration ce qui n'est pas le cas en C.
Ensuite, certes, il n'y a pas besoin de faire de l'allocation dynamique pour une chaine locale, mais ce n'est qu'un exemple d'erreur possible lorsqu'on mélange C et C++ comme cela. Qu'on ai pas besoin de le faire n'importe peu, ce n'est qu'un exemple de ce qu'il ne faut pas faire.

/!\ Ne pas utiliser ce message si l'on ne sais pas exactement ce que l'on fait /!\

Juste pour contredire (mais le principe reste bien le même) :

Code:

---

1 2	std::string *test = reinterpret_cast<std::string *>(malloc(sizeof(std::string))); new (test) std::string;

---

Certes...mais le placement new (qui reste un new) est un cas particulier pour traiter des besoins particuliers de gestion d'allocation...peu pédagogique de citer ça à un débutant en parlant d'une chaine dans la main d'une application aussi simple.

Là, vous l'embrouillez plus qu'autre chose les gars ;)

Si tu veux vraiment être clair avec la date, il faudrait vérifier d'autres choses (31 février est possible avec ton code).
Si tu souhaites être au point avec ton code, je te conseille de créer une fonction qui vérifiera ta date (selon le mois, et même selon l'année (bissextile ou non)). C'est beaucoup plus poussé, mais ca pourrait t'apprendre à créer une fonction et à l'utiliser ;)

Entièrement d'accord avec toi

Citation:

---

peu pédagogique de citer ça à un débutant

---

@sk8trasher:
Pour faire simple, quand tu fais de la réservation manuelle de mémoire (allocation dynamique) soit tu fais du C et tu utilises malloc (! libérer avec free()), soit tu fais du C++ et tu utilises new (! libérer avec delete). Mais surtout pas de mélange des deux...

Que tu fasses du C ou du C++, évites au maximum de réserver et libérer manuellement la mémoire, c'est source de bugs car on a vite fait de commettre une erreur....

Dans ton cas, le plus efficace est d'utiliser une constante en laissant le compilateur s'occuper de son stockage, comme suggéré par Neckara:

Citation:

---

Code:

---

1 2	char * NOM_FICHIER = "/home/sk8trasher/komate/komate_pousse.txt";

---

---

Citation:

---

Envoyé par therwald

Certes...mais le placement new (qui reste un new) est un cas particulier pour traiter des besoins particuliers de gestion d'allocation...peu pédagogique de citer ça à un débutant en parlant d'une chaine dans la main d'une application aussi simple.

---

C'est sûr que c'était une mauvaise idée. Mais je ne peux pas m'empêcher de contredire quand je vois l'histoire de segfault.
Bref, je vais ajouter un panneau d'attention à mon message précédent, ça réglera le problème.

euh j'avais un peu oublié ce topic ! et je viens de lire toutes les réponses qui ont été émises depuis mon absence et ces dernières offrent beaucoup de possibilité d'évolution (formation)de ce programme . Notamment le contrôle de la date si l'année est bissextile

Je suit toujours ma formation, et je reviens plus en détail sur le C, n'étant pas déjà d'attaque pour le C++.

Cet après midi j'ai réalisé les mesures de komate, et je me suis replongé sur mon script source. J'ai voulu le convertir en C(sans toutes les extensions farfelu que je vais développer), mais je n'arrive pas à mettre a ajouté une nouvelle ligne pour chaque nouvelles saisie.
Lorsque je lance le programme efface les données précédemment écrite.

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

#include<iostream> #include<stdio.h> #include<stdlib.h> #include <string.h> #include <fstream>   using namespace std; main() { char NOM_FICHIER[100] = "/home/user/komate/komate_pousse.txt"; FILE *FICHIER; int jour; int mois; int annee; int POID; int TAILLE; do{ cout<<"Quelle date sommes nous ? \n"; cout<<"Saisir le jour (01, 02, 03, ...30,31) \n"; cin>>jour; }while((jour<=0)||(jour>31)); do{ cout<<"Saisir le mois (01, 02, 03, ...11, 12) \n"; cin>>mois; }while((mois<=0)||(mois>31)); do{ cout<<"Saisir l'annee (2012, 2013, 2014...2049, 2050) \n"; cin>>annee; }while(annee<2012);   cout<< "Nous sommes le" <<jour<<"/"<<mois<<"/"<<annee<<endl; do{ cout<<"Combien de gramme pese komate ?: \n"; cin>>POID; }while((POID<85)||(POID>500)); do{ cout<<"Combien de millimetre mesure komate ?: \n"; cin>>TAILLE; }while((TAILLE<0)||(TAILLE>500)); //affichage resultat cout<<"le " <<jour<<"/"<<mois<<"/"<<annee<<" komate pese "<<POID <<" grammes et mesure "<<TAILLE<<" millimetres \n" <<FICHIER; FICHIER = fopen(NOM_FICHIER, "a"); fprintf(FICHIER,"%d\t%d\t%d\t%d\t%d\t\n",jour, mois, annee, POID,TAILLE);   return 0; }

---

Avant d'étudier le C++, mon prof ma conseillé de revoir le C tout court. Voici une version C du suivi de croissance de komate mais le ne programme n'écrit pas à la suite du fichier :arf:<br>
J'ai essayé d'utilisé une structure, il n'y a aucun intérêt, mais je voulais essayé.

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

#include <stdio.h> #include <stdlib.h> typedef struct { char date [9]; int poid; int taille; } komate; int main() { FILE *p_fichier; /* pointeur fichier */ /* Créer et remplir le fichier */ p_fichier = fopen("komate_croissance.txt","w"); if (p_fichier == NULL) { printf("\aImpossible de créer le fichier \n"); exit(-1); // Abandonner le programme } komate p; printf("Veuillez entrer la date des mesures (JJ/MM/AA):"); scanf("%s",p.date); printf("Veuillez entrer le poid de Komate ( en gramme):"); scanf("%d",&p.poid);/* ne pas oublier le & !!! */ printf("Veuillez entrer la taille de Komate ( en millimetre ):"); scanf("%d",&p.taille); /* ne pas oublier le & !!! */ fprintf(p_fichier, "%s ",p.date); fprintf(p_fichier, "%d ",p.poid); fprintf(p_fichier, "%d ",p.taille); fclose(p_fichier); return 0; }

---

Et quand même , pour ceux que ca intérressent voici les résultats observés ces derniers mois:

Code:

---

1 2 3 4 5

14 1 2012 85 59 20 1 2012 85 59 4 2 2012 85 59 28 3 2012 85 61

---

Citation:

---

Code:

---

1 2	FICHIER = fopen(NOM_FICHIER, "a");

---

---

Théoriquement, l'écriture devrait se faire à la suite, je suis perplexe.

As-tu essayer de faire un ftell pour savoir où se situe la tête de lecture lors de l'ouverture du fichier en append?

Si tu n'es pas à la fin de ton fichier, tu pourra toujours te positionner à la fin de celui-ci avec fseek()

Par contre n'oublie pas le

Code:

---

1 2	fclose(FICHIER)

---

NB : désolé, je regardais la mauvaise version,

Code:

---

1 2	p_fichier = fopen("komate_croissance.txt","w");

---

va effacer ton fichier dès que tu commencera à écrire si je ne dit pas de bêtises

Code:

---

1 2	p_fichier = fopen("komate_croissance.txt","a");

---

J'en ai profité des réponses postées pour les prendre en considération, pour le développement de mon application du suivi de croissance de ma tortue.
J'ai modifier les variable écrite en majuscule en minuscule. remplacement de la ligne

Code:

---

1 2	FICHIER = fopen(NOM_FICHIER, "a"); fprintf(FICHIER,"%d\t%d\t%d\t%d\t%d\t\n",jour, mois, annee, POID,TAILLE);

---

par

Code:

---

1 2	ofstream fichier(NOM_FICHIER); fichier << jour << '\t' << mois << '\t' << annee << '\t' << poid << '\t' << taille << std::endl;

---

C'est cool, ça compile par contre j'ai le même problème que dans mon post précédant: lorsqu'on exécute le prog, les nouvelles saisies écrase le fichier existant.

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

#include<iostream> #include<stdio.h> #include<stdlib.h> #include <string.h> #include <fstream>   using namespace std; main() { char *NOM_FICHIER = "/home/user/komate/komate_pousse.txt"; FILE *FICHIER; int jour; int mois; int annee; int poid; int taille; do{ cout<<"Quelle date sommes nous ? \n"; cout<<"Saisir le jour (01, 02, 03, ...30,31) \n"; cin>>jour; }while((jour<=0)||(jour>31)); do{ cout<<"Saisir le mois (01, 02, 03, ...11, 12) \n"; cin>>mois; }while((mois<=0)||(mois>31)); do{ cout<<"Saisir l'annee (2012, 2013, 2014...2049, 2050) \n"; cin>>annee; }while(annee<2012);   cout<< "Nous sommes le" <<jour<<"/"<<mois<<"/"<<annee<<endl; do{ cout<<"Combien de gramme pese komate ?: \n"; cin>>poid; }while((poid<85)||(poid>500)); do{ cout<<"Combien de millimetre mesure komate ?: \n"; cin>>taille; }while((taille<0)||(taille>500)); //affichage resultat cout<<"le " <<jour<<"/"<<mois<<"/"<<annee<<" komate pese "<<poid <<" grammes et mesure "<<taille<<" millimetres \n" <<FICHIER; ofstream fichier(NOM_FICHIER); fichier << jour << '\t' << mois << '\t' << annee << '\t' << poid << '\t' << taille << std::endl;   return 0; }

---

Citation:

---

Envoyé par Neckara

NB : désolé, je regardais la mauvaise version,

Code:

---

1 2	p_fichier = fopen("komate_croissance.txt","w");

---

va effacer ton fichier dès que tu commencera à écrire si je ne dit pas de bêtises

Code:

---

1 2	p_fichier = fopen("komate_croissance.txt","a");

---

---

Ok cool Le programme à été compilé, et fonctionne (j'ai encore un hic, comment faire pour qu'à chaque nouveau lancemnt , le programme saute une ligne)
Reste à le mettre un peu plus performant dans le contrôle des saisies et le coup de la date pour les année bissextiles.

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

#include <stdio.h> #include <stdlib.h> typedef struct { char date [40]; int poid; int taille; } komate; int main() { FILE *p_fichier; /* pointeur fichier */ /* Créer et remplir le fichier */ p_fichier = fopen("komate_croissance.txt","a"); if (p_fichier == NULL) { printf("\aImpossible de créer le fichier \n"); exit(-1); // Abandonner le programme } komate p; printf("Veuillez entrer la date des mesures (JJ/MM/AA):"); scanf("%s",p.date); printf("Veuillez entrer le poid de Komate ( en gramme):"); scanf("%d",&p.poid); printf("Veuillez entrer la taille de Komate ( en millimetre ):"); scanf("%d",&p.taille); /* ne pas oublier le & !!! */ fprintf(p_fichier, "%s ",p.date); fprintf(p_fichier, "%d ",p.poid); fprintf(p_fichier, "%d \n",p.taille); fclose(p_fichier); return 0; }

---

je retire mon précédant post
ça marche , voici un petite mise à jour, dans le contrôle des saisies

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

#include <stdio.h> #include <stdlib.h> typedef struct { char date [40]; int poid; int taille; } komate; int main() { FILE *p_fichier; /* pointeur fichier */ /* Créer et remplir le fichier */ p_fichier = fopen("komate_croissance.txt","a"); if (p_fichier == NULL) { printf("\aImpossible de créer le fichier \n"); exit(-1); // Abandonner le programme } komate p; printf("Veuillez entrer la date des mesures (JJ/MM/AA):"); scanf("%s",p.date); do { printf("Veuillez entrer le poid de Komate ( en gramme):"); scanf("%d",&p.poid); }while((p.poid<85)||(p.poid>500)); do { printf("Veuillez entrer la taille de Komate ( en millimetre ):"); scanf("%d",&p.taille); /* ne pas oublier le & !!! */ }while((p.taille<0)||(p.taille>500)); fprintf(p_fichier, "%s ",p.date); fprintf(p_fichier, "%d ",p.poid); fprintf(p_fichier, "%d ",p.taille); fclose(p_fichier); return 0; }

---

encore une version le partisan du moindre effort qui sombre en moi a solutionné mon problème concernant le contrôle de la saisie de la date. et j'ai simplifié le scripte en enlevant le typedef struct(){...}

Voici ma mise à jour

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

#include <stdio.h> #include <stdlib.h> #include <time.h>     time_t t = time(NULL); int poid; int taille;   int main() { FILE *p_fichier; /* pointeur fichier */ /* Créer et remplir le fichier */ p_fichier = fopen("komate_croissance.txt","a"); if (p_fichier == NULL) { printf("\aImpossible de créer le fichier \n"); exit(-1); // Abandonner le programme }   do { printf("Veuillez entrer le poid de Komate ( en gramme):"); scanf("%d",&poid); }while((poid<85)||(poid>500)); do { printf("Veuillez entrer la taille de Komate ( en millimetre ):"); scanf("%d",&taille); /* ne pas oublier le & !!! */ }while((taille<0)||(taille>500)); fprintf(p_fichier, "%s\t%d\t%d\n",ctime(&t),poid,taille); fclose(p_fichier); return 0; }

---

Plutôt que de mettre :

Code:

---

1 2	fprintf(file, "%d ", x);

---

Tu peux faire :

Code:

---

1 2	fprintf(file, "%d\t", x);

---

\t est une tabulation, ainsi tu pourras aligner tes valeurs correctement.

C'est top, il devient bien ce petit programme. Un dernier truc, ou un avant dernier, est-il possible de forcer le programme à écrire sur une seule ligne. Pour le moment dans le fichier généré voici comment sont présenté les résultats:

Code:

---

1 2	Wed Mar 28 23:06:43 2012 85 61

---

http://www.sk8-4-life.fr/images/komate2.jpg

Code:

---

1 2 3 4 5

//#include <string.h> char mon_tableau [] = ctime(&t); mon_tableau[strlen(mon_tableau)-1] = '\0'; fprintf(file, "%s", mon_tableau);

---

Tu enregistre ta date dans un tableau de char et tu remplace le '\n' par '\0' qui indique la fin de ta chaine.

Edit : \zéro

Citation:

---

Envoyé par Neckara

Code:

---

1 2 3 4 5

//#include <string.h> char mon_tableau [] = ctime(&t); mon_tableau[strlen(mon_tableau)-1] = '\0'; fprintf(file, "%s", mon_tableau);

---

Tu enregistre ta date dans un tableau de char et tu remplace le '\n' par '\o' qui indique la fin de ta chaine.

---

J'ai appliquer cette modification, mais ça ne compile pas:
user@K73E:~/komate$ g++ komate3.c -o komate3
komate3.c:12:33: erreur: array must be initialized with a brace-enclosed initializer

Ai-je bien inséré le bloc de commande ?

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

#include <stdio.h> #include <stdlib.h> #include <time.h> #include <string.h>         int poid; int taille; time_t t = time(NULL); char mon_tableau [10] = ctime(&t); int main() { FILE *p_fichier; /* pointeur fichier */ /* Créer et remplir le fichier */ p_fichier = fopen("komate_croissance.txt","a"); if (p_fichier == NULL) { printf("\aImpossible de créer le fichier \n"); exit(-1); // Abandonner le programme }   mon_tableau[strlen(mon_tableau)-1] = '\0';   do { printf("Veuillez entrer le poid de Komate ( en gramme):"); scanf("%d",&poid); }while((poid<85)||(poid>500)); do { printf("Veuillez entrer la taille de Komate ( en millimetre ):"); scanf("%d",&taille); /* ne pas oublier le & !!! */ }while((taille<0)||(taille>500)); fprintf(p_fichier, "%s\t%d\t%d\n", mon_tableau, poid,taille);     fclose(p_fichier); return 0; }

---

En faisant :

Code:

---

1 2 3

char tab[10]; tab = ctime(&time);

---

ça devrait marcher.

Et bien j'ai suivi ton conseil , mais ça marche pas. Rrrh, je comprend pas pourquoi, à la compilation le termianl dit:

erreur: cannot convert ‘time_t (*)(time_t*)throw () {aka long int (*)(long int*)throw ()}’ to ‘const time_t* {aka const long int*}’ for argument ‘1’ to ‘char* ctime(const time_t*)’



Voici mon script

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

#include <stdio.h> #include <stdlib.h> #include <time.h> #include <string.h>     int main() { int poid; int taille; time_t t = time(NULL); char tab[50]; tab = ctime(&time); FILE *p_fichier; /* pointeur fichier */ /* Créer et remplir le fichier */ p_fichier = fopen("komate_croissance.txt","a"); if (p_fichier == NULL) { printf("\aImpossible de créer le fichier \n"); exit(-1); // Abandonner le programme }   tab[strlen(tab)-1] = '\0';   do { printf("Veuillez entrer le poid de Komate ( en gramme):"); scanf("%d",&poid); }while((poid<85)||(poid>500)); do { printf("Veuillez entrer la taille de Komate ( en millimetre ):"); scanf("%d",&taille); /* ne pas oublier le & !!! */ }while((taille<0)||(taille>500)); fprintf(p_fichier, "%s\t%d\t%d\n", tab, poid,taille);     fclose(p_fichier); return 0; }

---

Je ne serai dire si c'est du C tout court ou du ++
Quelqu'un pourrait-il me commenter cette ligne

Code:

---

tab[strlen(tab)-1] = '\0';

---

, SVP ?

Je pense que tu t'es trompé :

Code:

---

1 2 3 4

time_t t = time(NULL); char tab[50]; tab = ctime(&time);

---

Essaye plutôt ceci :

Code:

---

1 2 3 4

time_t t = time(NULL); char tab[50]; tab = ctime(&t);

---

'\0' marque la fin d'une chaine de caractère
int strlen(char *) retourne le nombre de caractère contenu dans une chaine (sans le '\0')
strlen(tab) - 1 : retourne donc le nombre de caractère contenu dans la chaîne tab - 1
tab[strlen(tab) - 1] est donc la position du dernier caractère de tab ie '\n' ici.
donc tab[strlen(tab) - 1] = '\0' remplace le '\n' par '\0' enlevant le '\n' de ta chaine de caractère.

J'ai effectué la modification.
Je m'excuse mais j'ai encore un souci de compilation:
"komate3.c:12:15: erreur: incompatible types in assignment of ‘char*’ to ‘char [50]’"
Piou c'est dur

Code:

---

1 2 3

char tab[20]; strcpy(tab, ctime(&t));

---

strcpy permet de copier une chaine dans une autre.

Nikel, marche bien le petit programme, Merci de ton aide Neckara

Quelle commande puis-je utiliser pour afficher à la fin de la saisie le fichier texte qui à été généré et même mieux, je pense que cela doit être possible: qu'un graphique de la croissance en fonction de la date?

voici l'état actuel de la source

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

#include <stdio.h> #include <stdlib.h> #include <time.h> #include <string.h>     int main() { int poid; int taille; time_t t = time(NULL); char tab[50]; strcpy(tab, ctime(&t));   FILE *p_fichier; /* pointeur fichier */ /* Créer et remplir le fichier */ p_fichier = fopen("komate_croissance.txt","a"); if (p_fichier == NULL) { printf("\aImpossible de créer le fichier \n"); exit(-1); // Abandonner le programme }   tab[strlen(tab)-1] = '\0';   do { printf("Veuillez entrer le poid de Komate ( en gramme):"); scanf("%d",&poid); }while((poid<85)||(poid>500)); do { printf("Veuillez entrer la taille de Komate ( en millimetre ):"); scanf("%d",&taille); /* ne pas oublier le & !!! */ }while((taille<0)||(taille>500)); fprintf(p_fichier, "%s\t%d\t%d\n", tab, poid,taille);     fclose(p_fichier); return 0; }

---

Pour afficher le fichier, rien de plus simple :

- tu le ferme
- tu l'ouvres en lecture seule "r"
- tu lis une ligne avec fgets(char *, int taille - 1, File *)
- et tu continu de lire tant que fgets ne renvois pas NULL

Pour afficher ton graphique, il me semble qu'il existe des fonctions pour dessiner sur la console, donc après c'est assez simple de positionner quelques points.
Tu peux même essayer de les relier entre eux mais ça sera plus compliqué.

L'autre solution est de se trouver une bibliothèque graphique qui fait déjà des graphiques.

rien de plus simple... sûrement
j'ai essayé j'ai introduit à partir du fclose(p_fichier); (ligne 35)
le code suivant, bien entendu ça ne fonctionne pas

Code:

---

1 2	fopen(p_fichier,"r"); fgets(p_fichier, int 50,komate_croissance.txt);

---

j'aai tester sans réussite final d'autre syntaxe; le terminal me dit lors de la compilation:"komate4.c: In function ‘int main()’:
komate4.c:37:43: erreur: cannot convert ‘FILE* {aka _IO_FILE*}’ to ‘char*’ for argument ‘1’ to ‘char* fgets(char*, int, FILE*)’"

Code:

---

1 2	fopen("komate_croissance.txt","r"); fgets(p_fichier,50,"komate_croissance.txt");

---

Salut,

"p_fichier" est un pointeur vers le fichier en mémoire, si tu veux c'est l'adresse du fichier. Tu n'as donc besoin de savoir comment le fichier s'appelle qu'à l'ouverture de celui-ci.

De plus, bien que ça soit un forum d'entraide, il faudrait que tu te dépatouilles un peu plus pour chercher d'où viennent tes erreurs sinon tu n'apprendras jamais :) Simplement en lisant le message d'erreur :

Code:

---

komate4.c:37:43: erreur: cannot convert ‘FILE* {aka _IO_FILE*}’ to ‘char*’ for argument ‘1’ to ‘char* fgets(char*, int, FILE*)

---

Tu sais donc que fgets attend trois arguments, un char*, un int et un pointeur sur fichier (ton p_fichier). Il faut bien entendu passer ses arguments dans l'ordre dans lequel les attends fgets. A ce propos, tu peux consulter les documentations des fonctions en tapant "man fgets" dans google ou dans un terminal Linux. Tu apprendras entre-autre que char* est le "réceptacle" (appelé un buffer) dans lequel fgets va placer ce qu'il a lu.

Malgré ce que j'ai dit au dessus, n'hésite pas à venir poser tes questions quand tu n'en sors vraiment pas (car parfois il manque des notions théoriques sous-jacentes indispensables et on gagne beaucoup de temps en demandant aux gens).

Super, Merci de ta réponse, Cet après midi je me mets dessus et je solutionne ce problème.

Je bloque un peu sur la syntaxe exacte de l'instruction fgets que je veux utiliser. J'ai pensé (ça m'arrive :lol: ) qu'il faudrait que je crée un pointeur pour pouvoir utiliser cette instruction.

Es-ce une bonne idée, ou je plane complet ?