Discussion :

[Flophx]

Bonjour à tous,
j'essaie de concaténer trois chaînes de caractères tel que je puisse obtenir:
Image_XX.ppm

mon code est le suivant:

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
 
char* s1 = "Image_";
   char* s2 = (char*)num;
   char* s3 = ".ppm";
   char* s4 = NULL;
   char* nom_image=NULL;
   s4 = malloc (strlen (s1) + strlen (s2) + 1);
   s4 = strcat(s1,s2);
   printf("%s",s4);
   nom_image = malloc (strlen (s4) + strlen (s3) + 1);
   nom_image = strcat(s4,s3);
   printf("%s",nom_image);
 
printf("En cours d'ecriture de l'image...\n");
fp = fopen(nom_image, "w");

Le problème, c'est qu'il me renvoie une erreur du type: erreur de segmentation
je ne comprends pas l'erreur...
Quelqu'un saurait-il ce qui cloche dans mon code?
Merci
Bonne journée

[salseropom]

Salut, quand tu fais

Code :

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

1
2
 
strcat(s1,s2)

tu colles à la fin de s1 la string s2. Mais ta string s1 n'a pas suffisament d'espace pour contenir la string s2 à sa suite d'où ton segmentation fault.

Je te propose la chose suivante (non testée) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
char* s1 = "Image_";
   char* s2 = (char*)num;
   char* s3 = ".ppm";
   char s4[strlen(s1)+strlen(s2)+strlen(s3)+10];
   strcat(s4,s1);
  strcat(s4,s2);
  strcat(s4,s3);
   printf("%s",s4);
   char * nom_image = strdup(s4);
   printf("%s",nom_image);

n'oublie pas de faire un

Code :

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

1
2
 
free(mon_image); mon_image=NULL;

et n'oublie pas qu'après un appel à malloc, il faut vérifier que ton pointeur ne soit pas NULL

[Flophx]

Je débute en C, ca sert à quoi les dernières lignes de code (le free et remettre image à null)?

[scorpion.os]

le "free" c pour liberer la memoire que tu as allouer avec "malloc"

[scorpion.os]

deja,tu dois convertir ce que malloc te renvoi.
ecri ca :

Code :

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

1
2
 
s4 = (char* ) malloc (strlen (s1) + strlen (s2) + 1);

c ar s4 est de type char *

[zooro]

deja,tu dois convertir ce que malloc te renvoi.
ecri ca :

Code :

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

1
2
 
s4 = (char* ) malloc (strlen (s1) + strlen (s2) + 1);

c ar s4 est de type char *

Justement non, il ne faut pas convertir le retour de malloc.
Pour apprendre à utiliser correctement malloc(), cf. l'excellent site d'Emmanuel : http://emmanuel-delahaye.developpez....tes.htm#malloc

Cela dit, ton code initial semble trop compliqué. Pourquoi n'utilises-tu pas sprintf(), ou, mieux, snprintf() ?
Par exemple, comme ça :

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
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
 
int main(void)
{
    const char* num = "123";
    char* nom_image = NULL;
    int longueur = 11+strlen(num);
 
    nom_image = malloc(longueur);
    if (nom_image == NULL) {
        perror("Erreur sur malloc");
        exit(EXIT_FAILURE);
    }
    memset(nom_image, '\0', sizeof(nom_image));
 
    snprintf(nom_image, longueur, "Image_%s.ppm", num);
 
    printf("nom_image = '%s'\n",nom_image);
 
    free(nom_image); nom_image = NULL;
 
    exit(EXIT_SUCCESS);
}

[Flophx]

Je sais pas, ca marche toujours pas, je vois pas d'ou vient l'erreur (j'ai essayé le code tel quel)....

Même en donnant le type:

Code :

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

1
2
 
s4 = (char*)malloc (strlen (s1) + strlen (s2) + 10);

Ca ne marche pas non plus