lecture chaîne de char avec espace Résolu

Question

Bonjour, 

Vu que je n'ai pas un très bon niveau en c, j'ai écrit mon code et je demande votre aide pour me signaler si vous constater d'éventuelle incohérences.

Voici mon problème de départ : 
Ecrire un programme qui affiche une invite et lit une chaîne de caractère contenant des espaces. Le programme ne doit pas être sujet à buffer overflow.

Mon programme : 
char * lire_chaine(char * message){ printf("%s", message); int MIN_BUFFER = 7; size_t taillbuff = MIN_BUFFER; char *buffer = malloc(taillbuff); if (buffer == NULL) return NULL; // échec de malloc() char *p; for(p=buffer ; (*p=getchar()) != '
' && *p!=EOF ; ++p)  if (p - buffer == taillbuff - 1) {   // buffer plein,   p = realloc(buffer, taillbuff *= 2); //on le double   if (p == NULL) {     // échec de realloc()    free(buffer);    return NULL;   }   else buffer = p; //bloc réalloué != buffer    p += taillbuff/2 - 1; // p reprend sa place dans  }                       // la nouvelle zone

p = 0; p = realloc(buffer, p - buffer + 1); //réajustement if (p == NULL) {          // échec de realloc  free(buffer);  return NULL; } else return p;}int main (int argc, char *argv[]){  char chaine[100]; printf("entrez votre chaine de char : 
"); scanf("%[^
]%*c", chaine); printf("la chaine de char lue est : 
"); lire_chaine(chaine);  return 0;}

Le programme compile bien et lit les chaînes que je met en entrée

Je vous remercie d'avance. 

Configuration: Windows / Chrome 50.0.2661.94

nocram18 · Answer

Yo,

C'est du C++ ?

Je commence aussi donc je pourrais pas d'aider la mais ce que je te conseille déjà c'est d'apprendre à un indenter c'estg plus beau et lisible surtout quand tu partage apres tes problemes :D.

fiddy · Answer

Bonjour,

Je n'ai pas tout lu.
Mais déjà : scanf("%[^
]%*c", chaine
Risque de buffer overflow. Utilise plutôt : scanf("%99[^
]", chaine);

Note :pourquoi ne pas utiliser fgets ?

fiddy · Answer

Bonsoir,

Un retour plus complet que tout à l'heure :
Attention, à  ne pas trop mélanger les déclarations de variables et les instructions.
Par exemple :

{
     printf("Salut ");
     int a=5;
     [...]
}

N'est pas terrible. En C, il vaut mieux (d'ailleurs dans l'ancienne norme, c'est une obligation) mettre les déclarations de variable en début de frame puis après le code.
Ce qui donne :
<code c>
{
     int a=5;
     printf("Salut ");
     [...]
}

Dans le main() :
   scanf("%[^
]%*c", chaine);
Comme dit dans mon post précédent, c'est sujet au buffer overflow.
Utilise plutôt fgets(). scanf() est très dangereux lorsque mal utilisé.

for(p=buffer ; (*p=getchar()) != '
' && *p!=EOF ; ++p)
Aucun contrôle de taille. Donc, si l'utilisateur tape une longue chaîne, cela fera planter le programme. Fais intervenir taillbuff dans ta condition de fin.

buffer = p; //bloc réalloué != buffer
Inutile. Ton realloc a déjà fait le job.

p = 0;
Que souhaites-tu faire ? Plutôt *p=0; (ou *p='\0'); afin de mettre le \0 en fin de chaîne, non ?

Cdlt,

Lecture chaîne de char avec espace

3 réponses

Discussions similaires

Newsletters