[ C ] Fonction et char*

Fonction et char* [ C ] - Programmation

Marsh Posté le 18-07-2002 à 01:03:26    

Voila dans ce petit programme la saisie marche mais j'ai un segmentation fault ensuite. Ca doit être un truc à la ocn mais je ne le vois pas.
 Si vous avez quelques instants...
 
 

Code :
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. #include <ctype.h>
  5. char* fonc_saisie()
  6. {
  7. char* saisie = (char*)malloc(30 * sizeof(char));
  8. int i;
  10. scanf("%s",saisie);
  12. while (saisie[i] != '\0')
  13. {
  14.  saisie[i] = tolower(saisie[i]);
  15.  i = i+1;
  16. }
  17. printf("Nouvelle chaine : %s",saisie);
  18. return saisie;
  19. free(saisie);
  20. ;
  21. }
  23. int main()
  24. {
  25.   struct machin{
  26.    char* nom;
  27.    char* prenom;
  28.   }mac;
  30.   mac.nom = (char*)malloc(30 * sizeof(char));
  31.   mac.prenom = (char*)malloc(30 * sizeof(char));
  33.   printf("\nentrer le nom\n" );
  34.   mac.nom = fonc_saisie();
  35.   printf("\nentrer le prenom\n" );
  36.   mac.prenom = fonc_saisie();
  37.   printf("\n%s %s\n",mac.nom,mac.prenom);
  39.   free(mac.nom);
  40.   free(mac.prenom);
  42.  return 0;
  43. }

 
 
Je fais une fonction parce qu'après je vais rendre la saisie plus restrictive.
 
CHaiCA

Reply

Marsh Posté le 18-07-2002 à 01:03:26   

Reply

Marsh Posté le 18-07-2002 à 01:30:23    

Euh là y a une double erreur, je pense :
 
     return saisie;
     free(saisie);
 
* un return provoque le retour direct donc le free n'est jamais executé
* Il ne faut pas libérer la zone mémoire si tu la retournes, malheureux :D
* tu reserves l'espace dans la structure puis tu la reserves dans la fonction => c'est pas logique => tu fais 2 fois un free sur un pointeur d'où la fameuse "segmentation fault"  ;)  
* aucune vérif sur la taille des chaines (mais ça tu comptes peut être l'ajouter plus tard) => "spabien"  
 
(Je corrige de tête, j'ai pas de compilo sous la main)
 

Code :
  1. char* fonc_saisie(char* chaineReservee)
  2. {
  3.     scanf("%s",chaineReservee);
  4.     char* ptr = chaineReservee;
  5.     while(*ptr != 0) {
  6.        *ptr = tolower(*ptr);
  7.        ptr++;
  8.     }
  9.     printf("Nouvelle chaine : %s",chaineReservee);
  10.     return chaineReservee; // même pas utile en fait
  11. }
  13. // le reste est ok sauf pour chaque appel à fonc_saisie :
  14.   printf("\nentrez le nom\n" );
  15.   fonc_saisie(mac.nom);


 
Ok ?


Message édité par smaragdus le 18-07-2002 à 02:02:24
Reply

Marsh Posté le 18-07-2002 à 01:50:49    

Code :
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. #include <ctype.h>
  5. int fonc_saisie(char* saisie)
  6. {
  7. int i;
  9. scanf("%s",saisie);
  10. while (saisie[i] != '\0')
  11. {
  12.  saisie[i] = tolower(saisie[i]);
  13.  i = i+1;
  14. }
  15. printf("Nouvelle chaine : %s",saisie);
  16. return 0;
  17. ;
  18. }
  20. int main()
  21. {
  22.   struct machin{
  23.    char* nom;
  24.    char* prenom;
  25.   }mac;
  27.   mac.nom = (char*)malloc(30 * sizeof(char));
  28.   mac.prenom = (char*)malloc(30 * sizeof(char));
  30.   printf("\nentrer le nom\n" );
  31.   fonc_saisie(mac.nom);
  32.   printf("\nentrer le prenom\n" );
  33.   fonc_saisie(mac.prenom);
  34.   printf("\n%s %s\n",mac.nom,mac.prenom);
  37.   free(mac.nom);
  38.   free(mac.prenom);
  40.  return 0;
  41. }

 
 
Voila la nouvelle versionmais elle ne marche pas plus pour l'instant
 
CHaiCA


Message édité par chaica le 18-07-2002 à 01:51:50
Reply

Marsh Posté le 18-07-2002 à 02:04:56    

Si tu avais utilisé :
 

Code :
  1. char* ptr = chaineReservee;
  2.         while(*ptr != 0) {
  3.              *ptr = tolower(*ptr);
  4.              ptr++;
  5.         }


 
ça marcherait sans problème :D  
(en plus c'est plus rapide : il y a 2 additions et une affectation en moins)

Reply

Marsh Posté le 18-07-2002 à 02:10:56    

Nan c'est bon ca marche très ben ta première réponse était tip top, merci de m'avoir aidé à une heure si tardive et @ la prochaine question!
 
CHaiCA

Reply

Marsh Posté le 18-07-2002 à 08:20:31    

Code :
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. #include <ctype.h>
  5. int fonc_saisie(char* saisie)
  6. {
  7. int i;
  9. scanf("%s",saisie);
  10.        i= 0;  
  11. while (saisie[i] != '\0')
  12. {
  13.  saisie[i] = tolower(saisie[i]);
  14.  i = i+1;
  15. }
  16. printf("Nouvelle chaine : %s",saisie);
  17. return 0;
  18. ;
  19. }
  21. int main()
  22. {
  23.   struct machin{
  24.    char* nom;
  25.    char* prenom;
  26.   }mac;
  28.   mac.nom = (char*)malloc(30 * sizeof(char));
  29.   mac.prenom = (char*)malloc(30 * sizeof(char));
  31.   printf("\nentrer le nom\n" );
  32.   fonc_saisie(mac.nom);
  33.   printf("\nentrer le prenom\n" );
  34.   fonc_saisie(mac.prenom);
  35.   printf("\n%s %s\n",mac.nom,mac.prenom);
  38.   free(mac.nom);
  39.   free(mac.prenom);
  41.  return 0;
  42. }

 
 
tu oublie l'init du i c pour ca

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed