Discussion :

[Bahan]

Bonjour à toutes et à tous.

Alors j'ai un petit problème avec la fonction strcat. Débutant en C, voici le code que je réalise :

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
 
 
#include <stdio.h>
#include <dirent.h>
main()
{
        struct dirent *pDp;
        DIR *pRep_id;
        char *pcNomFichier;
        char *pcDossierCourant = ".", *pcDossierParent = "..", *pcNomRep = "/MonChemin/";
 
        pRep_id = opendir(pcNomRep);
        for(pDp = readdir(pRep_id); pDp != NULL; pDp = readdir(pRep_id))
        {
              pcNomFichier = strcat(pcNomRep, pDp->d_name);
              printf("%s.\n", pcNomFichier);
        }
        closedir(pRep_id);
        return 0;
}

J'ai alors l'erreur suivante :

Code :

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

1
2
3
4
 
 
"MonFichier.c", line 14.28: 1506-068 (S) Operation between types "char*" 
and "int" is not allowed.

La ligne 14 correspond à mon utilisation de strcat. Et je ne comprends pas cette erreur, car pcNomRep est bien un pointeur de char, et pDp->d_name aussi, non ?

Bahan

[fearyourself]

Est-ce que tu as l'impression d'allouer de la mémoire pour tes chaînes de caractères?

Toute déclaration :

Code :

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

char *s = "J'ai faim, c'est bientôt le déjeuner";

doit être considérée comme étant une chaîne constante, il y a une option dans la plupart des compilateurs qui permet de s'assurer que tu les déclares comme tel.

Jc

[Gruik]

Héhé, c'est la meprise des débutants qui croient que "char *" ca veut dire "chaine"

[Bahan]

Ok merci de votre aide .
En fait je commettais plusieurs erreurs dont le fait de ne pas allouer assez de mémoire dans mon strcat.

Il vaut mieux en fait que je déclare une chaine de caractère assez grande pour ensuite stocker dedans mes deux variables qui vont être concaténer.

Bahan

[Gruik]

La vraie formulation serait plutot "je declare un tableau de caractere de taille fixe pour y ecrire et concatner des chaines"

[Médinoc]

Bonjour,

1. L'erreur elle-même indique que tu as oublié d'inclure <string.h> : strcat(), non-définie, retourne un int, ce qui ne peut donc pas être affecté à pcNomFichier
2. Cela montre que tu n'as aucun warning d'activé. Quel est ton compilateur ?
3. Les chaînes entre guillemets sont en mémoire constante: Utilise des const char * (avec le compilateur GCC, l'option -Wwrite-strings permet de vérifier cela. Sous les autres compilateurs, aucune idée).
4. Oui, il y aura de l'allocation de mémoire à faire (soit en déclarant un tableau de caractères (allocation statique ou automatique), soit avec de l'allocation dynamique)