Bonjour,  
 
j'essaie d'écrire une méthode qui prend en argument une chaine de caractères correspondant au nom d'un fichier : par exemple "mon_fichier" et qui par la suite lui associe le dossier dans lequel il se situe ainsi que son extension.  
 
Voici mon tout petit bout de code :  
 
 

 
 
A la fin de la méthode, chemin devrait correspondre à la chaine "files\mon_fichier.jpg", mais cela ne marche pas lorsque je concatène chemin avec la chaine passée en paramètres (ligne 12)... cela marche seulement avec les chaines que j'ai créées à l'intérieur de ma fonction. Voilà si quelqu'un pouvait m'expliquer... je ne vois pas pourquoi cela ne fonctionne pas...
Merci pour votre aide,
 
bonne journée.

Peut-être parce que "nom_fichier" != "mon_fichier" ?
 
De rien

ahah non c'est une erreur , j'ai voulu bidouiller dans mon message...
Dans mon code les noms correspondent bien, et j'ai bien une erreur (du type segmentation fault (core dumped))
Je remets le VRAI code    
 

 
mercii

Tu as bien inclus "#include <string.h>" hein ?
 
Et tu fais bien de préciser que ça plante à l'exécution, et pas à la compilation.
Du coup, je pense à un débordement en mémoire où il ne doit pas y avoir assez de place de réservée pour agrandir ta variable "chemin".
 
Essaie de la déclarer avec  

Ouais, aucune chance que ton code fonctionne.
 
Tu déclares un pointeur sur une chaine de caractère (char * chemin) et tu lui assignes une chaine statique: non seulement il n'y a pas la place à la fin de la chaine pour rajouter quoi ce soit, mais en plus ces chaines sont en lectures seules, toute tentative d'écriture = plantage direct.
 
Pour contourner le problème tu peux faire comme Turkleton t'as dit : déclarer une tableau et l'initialiser dans la foulée :
 
char chemin[100] = "files\\";
 

En effet

ne réserve de la place que pour f-i-l-e-s-\-0, soit sept caractères.
Mais

metstous les caractères de nom_fichier dérrière, et cela déborde donc.
La solution est de réserver plus d'espace, par exemple :

strncpy/strncat éviteront même de dépasser les 250 caractères et d'avoir un comportement indéfini ;-)

 
A part que strncpy d'une part écrit toujours n caractères et d'autre part ne garanti pas que le résultat soit une chaîne terminée par \0.
strncat est un peu moins piégeux.  La taille disponible à la destination doit être strlen(dst)+n+1.

 
Jamais vu ça

Effectivement ça fonctionne!! j'ai tout le chemin affiché ! merci à vous !

Si l'utilisateur rentre une chaîne de 5000 caractère et que toi tu la strcpy comme un boeuf dans ton buffer de 250 octets ton programme risque de pas aimer. Bon c'est sûr sur une utilisation normale y'aura jamais de problèmes cependant je tends à penser qu'il faut toujours utiliser strncpy en copiant n - 1 caractère (pour laisser de la place au \0) par sécurité
 
L'autre solution étant de faire un strlen de l'argument passé puis de vérifier si la taille de la chaîne est bien inférieure à la taille du buffer. Ca a l'avantage de pouvoir faire une gestion d'erreur correcte, alors qu'avec un strncpy tu vas juste copier les x premiers caractères et donc peut être tronquer ce que l'utilisateur a rentré

Je n'ai pas dit qu'il fallait les utiliser sans se soucier de rien non plus.

Sinon y a snprintf() qui est très bien aussi (mais C99, ce qui ne devrait quand même plus trop poser de problème aujourd'hui) : pas besoin de se soucier de la taille du buffer vs. la taille de la chaîne initiale (strncpy), et pas besoin d'initialiser le buffer avant recopie (strncat). Bon, je parle là des tâches communes hein.

Mais sinon, je plussoie "Un Programmeur" : strncat est bien moins casse-gueule. À préférer à strncpy, qui est une fonction si fourbe que je me demande qui a bien pu spécifier ce... cette... chose.

 
L'histoire que j'ai lue est qu'elle était conçue pour remplir le champs nom dans la structure décrivant les répertoires des premières versions de Unix, quand les noms étaient limités à 14 caractères (elle n'est pas dans l'index de mon exemplaire du Lions' Commentary on UNIX 6th Edition et j'ai pas cherché dans le code lui-même mais la structure décrivant les répertoires serait bien remplie correctement avec un strncpy).