Récupération du nom

**zennn** · 18/05/2007, 17h13

Bonjour.

Je débute en C et j'ai quelques soucis

Donc voilà, je voulais à partir d'un chemin récupérer le nom du fichier.
Exemple:
/home/here/here2/laphoto.bmp ==> laphoto.bmp

je vous montre le code source:

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
 
#include <stdlib.h>
#include <stdio.h>
 
char *nom (char *chemin) {
 
int i;
int j;
 
int position;
 
char *name;
 
for (i=0;i<1000;i++) {
 
    if(chemin[i] == '/' )  position=i;
 
}
 
for (j=0;j<1000-(position+1);j++) name[j]=chemin[j+position+1];
 
return name;
 
}
 
int main () {
 
char *chemin="/home/here/here2/latof.bmp";
 
char *lenom;
 
lenom=nom(chemin);
 
printf(lenom);
 
printf("\n");
 
return 0;
 
}

quand je met un autre caractère que '/' , par exemple 'h' ou autre je récupère bien le chemin mais j'ai une erreur de segmentation (classique quoi

)

Donc voilà, j'y suis dessus pendant un long moment, si vous voulez bien m'aider.

Merci d'avance.

**droggo** · 18/05/2007, 17h35

Gio,

Il faudrait penser à réserver de la mémoire pour "char *name", et plus tard, penser à libérer cette mémoire

**souviron34** · 18/05/2007, 17h39

tu as le choix entre 2 solutions :

Soit tu gardes le chemin original, et alors il faut que tu réserves de la place mémoire pour "name" et que tu copies dedans ce que tu veux
Soit tu ne gardes pas, et tu "déplaces" l'extrait que tu veux en début de chaîne..

**Kryptonaute** · 18/05/2007, 17h42

pas très propre de boucler 1000 fois :

Code :

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

1
2
3
4
5
6
 
for (j=0;j<1000-(position+1);j++) name[j]=chemin[j+position+1];
 
return name;
 
}

pourquoi tu boucles jusqu'à 1000 ?
tu sais que pour tester la fin d'une chaine de caractères il suffit de tester la valeur '\0'..
tu devrais utiliser la fonction strrchr pour trouver la dernière occurence de '\'
http://www.cppreference.com/stdstring/strrchr.html

Sers toi des fonctions de manipulation des chaînes de caractères :
http://www.cppreference.com/stdstring/index.html

**zennn** · 18/05/2007, 17h59

C'est bon j'ai pu résoudre le problème, merci pour votre aide.
Voilà le code source:

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
 
#include <stdlib.h>
#include <stdio.h>
 
char *nom (char *chemin) {
 
int i=0;
int j=0;
int position;
char *name;
 
name=malloc(4*100);
 
 
char c=chemin[0];
char d=chemin[0];
 
while (c != '\0' ) {
 
    if(c == '/' ) position=i;
    i++;
    c=chemin[i];
 
}
 
d=chemin[position+1];
 
while( d != '\0' ) {
 
    name[j]=d;
    j++;
    d=chemin[position+1+j];
}
 
 
return name;
free(name);
 
}
 
int main () {
 
char *chemin="/home/here/here2/latof.bmp";
char *lenom;
lenom=nom(chemin);
printf(lenom);
printf("\n");
return 0;
 
}

**droggo** · 18/05/2007, 19h58

Gei,

Tu n'as pas de chance, car si ça a marché, c'est par hasard:

- Ta fonction nom réserve de la mémoire pour le pointeur name, bien (au fait, pourquoi cette taille 4*100 ?)

- Puis, après avoir mis ce qu'il faut dans cette mémoire, elle renvoie ce pointeur

- ET libère la mémoire correspondant !!

Donc, dans le programme appelant, après réception du pointeur, les données pointées ne sont plus sensées être valides.

**zennn** · 18/05/2007, 20h26

jé pas très bien compris

Pour 4*100 c'est pour une chaine de 100 caractère.
sinon peux-tu m'expliquer le reste??

Merci.

**Freed0** · 18/05/2007, 20h31

Une chaine de caractères est composée de caractères qui ont une taille d'un byte. Donc il n'y a pas de raison de faire un * 4.

Au pire tu pourrais mettre 100 * sizeof(char) mais c'est complètement inutile.

gl · 18/05/2007, 21h51

Envoyé par droggo

- Puis, après avoir mis ce qu'il faut dans cette mémoire, elle renvoie ce pointeur

- ET libère la mémoire correspondant !!

Donc, dans le programme appelant, après réception du pointeur, les données pointées ne sont plus sensées être valides.

Le free() etant fait apre le return, il ne ser en jamais execute, le pointeur contient donc, au retour de la fonction, bien une adresse valide.

Par contre du coup la zone n'est jamais liberee et il y a une fuite memoire.

**droggo** · 18/05/2007, 22h34

Aui,

Envoyé par gl

Le free() etant fait apre le return, il ne ser en jamais execute, le pointeur contient donc, au retour de la fonction, bien une adresse valide.

Par contre du coup la zone n'est jamais liberee et il y a une fuite memoire.

Je n'avais même pas remarqué. J'ai vu un free, et je n'ai pas trop fait attention au reste.

**dj.motte** · 18/05/2007, 22h41

Salut,

Je vous soumet une autre possibilité, je ne sais pas trop si c'est bon ...

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
26
27
28
29
30
31
 
 char * extract_name( const char * p, int sep  )
 { char * z = strrchr( p ,  sep  ) ;
   if( z != NULL )  z++ ;
   if( z == NULL || *z == '\0' )  return NULL ;
   return z ;
 }
 
 void do2()
 {  char * z = NULL ;
    char * name = NULL ; 
    char * p = "/home/djm/linux/zorahpile/abc.c" ;
    printf("path= %s\n", p ) ;
    z = extract_name( p , '/');
    if( z != NULL  )
    { printf("res 1= %s len= %d\n", z, strlen(z) ) ;
      // ou encore ...
      name = ( char *) malloc( strlen(z)+1 ) ;
      strcpy( name, z ) ;
      printf("res 2= %s len= %d\n", name, strlen(name) ) ;
      if( name ) free( name ) ; 
    }
    else
    printf("%s\n", "too bad" );
 }
 
 int main( int argc , char * argv[]  )
 { do2() ;
   printf("terminé\n");
   return 0 ;
 }

J'ai testé ça a l'air ok sans avoir à faire des boucles.

**Emmanuel Delahaye** · 18/05/2007, 23h00

Envoyé par dj.motte

Je vous soumet une autre possibilité, je ne sais pas trop si c'est bon ...

J'ai testé ça a l'air ok sans avoir à faire des boucles.

Alors poste le code que tu as testé, parce que là...

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
26
27
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `extract_name':
main.c:3: warning: implicit declaration of function `strrchr'
main.c:4: error: `NULL' undeclared (first use in this function)
main.c:4: error: (Each undeclared identifier is reported only once
main.c:4: error: for each function it appears in.)
main.c: At top level:
main.c:10: warning: function declaration isn't a prototype
main.c: In function `do2':
main.c:10: error: `NULL' undeclared (first use in this function)
main.c:12: warning: initialization discards qualifiers from pointer target type
main.c:13: warning: implicit declaration of function `printf'
main.c:16: warning: implicit declaration of function `strlen'
main.c:18: warning: implicit declaration of function `malloc'
main.c:19: warning: implicit declaration of function `strcpy'
main.c:21: warning: implicit declaration of function `free'
main.c: At top level:
main.c:27: warning: unused parameter 'argc'
main.c:27: warning: unused parameter 'argv'
Process terminated with status 1 (0 minutes, 1 seconds)
4 errors, 10 warnings

**dj.motte** · 18/05/2007, 23h07

Salut,

En ajoutant les #include ( stdio.h, string.h, stdlib.h ) ça devrait peut-être résoudre le pb.

Avec linux, strrchr est compris. Je ne sais pas sur les autres plateformes.

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
26
27
28
29
30
31
32
33
34
35
 
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
 
char * extract_name( const char * p, int sep  )
 { char * z = strrchr( p ,  sep  ) ;
   if( z != NULL )  z++ ;
   if( z == NULL || *z == '\0' )  return NULL ;
   return z ;
 }
 
 void do2()
 {  char * z = NULL ;
    char * name = NULL ; 
    char * p = "/home/djm/linux/zorahpile/abc.c" ;
    printf("path= %s\n", p ) ;
    z = extract_name( p , '/');
    if( z != NULL  )
    { printf("res 1= %s len= %d\n", z, strlen(z) ) ;
      // ou encore ...
      name = ( char *) malloc( strlen(z)+1 ) ;
      strcpy( name, z ) ;
      printf("res 2= %s len= %d\n", name, strlen(name) ) ;
      if( name ) free( name ) ; 
    }
    else
    printf("%s\n", "too bad" );
 }
 
 int main( int argc , char * argv[]  )
 { do2() ;
   printf("terminé\n");
   return 0 ;
 }

Voilà mon grand, ce n'était pas difficile à deviner.

**Emmanuel Delahaye** · 18/05/2007, 23h20

Envoyé par dj.motte

En ajoutant les #include ( stdio.h, string.h, stdlib.h ) ça devrait peut-être résoudre le pb.

Leger mieux...

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c:14: warning: function declaration isn't a prototype
main.c: In function `do2':
main.c:16: warning: initialization discards qualifiers from pointer target type
main.c: At top level:
main.c:31: warning: unused parameter 'argc'
main.c:31: warning: unused parameter 'argv'
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 1 seconds)
0 errors, 4 warnings

Ceci fonctionne, y compris sur des noms 'simples'....

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
26
27
28
29
30
31
32
33
34
 
#include <stdio.h>
#include <string.h>
 
char const *extract_name (const char *p, int sep)
{
   char const *z = strrchr (p, sep);
   if (z != NULL)
      z++;
   else
      z = p;
   return z;
}
 
void do2 (void)
{
   char const path[] = "abc.c.txt";
   printf ("path= %s\n", path);
   char const *z = extract_name (path, '/');
   if (z != NULL)
   {
      size_t const len = strlen (z);
      printf ("res 1= %s len= %d\n", z, len);
   }
   else
      printf ("%s\n", "too bad");
}
 
int main (void)
{
   do2 ();
   printf ("finished\n");
   return 0;
}

**dj.motte** · 18/05/2007, 23h37

Salut

Emmanuel, il n' y a plus d'erreur mais des avertissments :

Project : Forums
Compiler : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c:14: warning: function declaration isn't a prototype
main.c: In function `do2':
main.c:16: warning: initialization discards qualifiers from pointer target type
main.c: At top level:
main.c:31: warning: unused parameter 'argc'
main.c:31: warning: unused parameter 'argv'
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 1 seconds)
0 errors, 4 warnings

On peut améliorer extract_name ainsi :

Code :

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

1
2
3
4
5
6
7
8
 
char * extract_name( const char * p, int sep  )
 { if( p == NULL ) return NULL ;  // ajouté pour améliorer
   char * z = strrchr( p ,  sep  ) ;
   if( z != NULL )  z++ ;
   if( z == NULL || *z == '\0' )  return NULL ;
   return z ;
 }

**crocodilex** · 18/05/2007, 23h48

Envoyé par zennn

Bonjour.

Je débute en C et j'ai quelques soucis

Donc voilà, je voulais à partir d'un chemin récupérer le nom du fichier.
Exemple:
/home/here/here2/laphoto.bmp ==> laphoto.bmp

je vous montre le code source:

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
 
#include <stdlib.h>
#include <stdio.h>
 
char *nom (char *chemin) {
 
int i;
int j;
 
int position;
 
char *name;
 
for (i=0;i<1000;i++) {
 
    if(chemin[i] == '/' )  position=i;
 
}
 
for (j=0;j<1000-(position+1);j++) name[j]=chemin[j+position+1];
 
return name;
 
}
 
int main () {
 
char *chemin="/home/here/here2/latof.bmp";
 
char *lenom;
 
lenom=nom(chemin);
 
printf(lenom);
 
printf("\n");
 
return 0;
 
}

quand je met un autre caractère que '/' , par exemple 'h' ou autre je récupère bien le chemin mais j'ai une erreur de segmentation (classique quoi

)

Donc voilà, j'y suis dessus pendant un long moment, si vous voulez bien m'aider.

Merci d'avance.

Si tu es sous linux (vu le chemin dans le code), il existe la commande "basename" qui fait exactement le boulot que tu demandes.
Mais si tu tiens à le faire toi même, voici un code en peu plus simplifié :

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
#include <stdio.h>
#include <string.h>
 
const char *nom (const char *chemin)
{
    const char *baseName = chemin + strlen(chemin);
 
    while (*baseName != '/' )
    {
        baseName--;
    }
    baseName++;
 
    return baseName;
}
 
 
int main (void)
{
    printf("%s\n", nom("/home/here/here2/latof.bmp"));
    return 0;
}

Il y a peut être possibilité de simplifier encore plus...

**dj.motte** · 19/05/2007, 00h07

Salut,

Dans la version de crocodilex :

Code :

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

1
2
 
const char *nom (const char *chemin)

Que se passe-t-il, si chemin est NULL ?

**Emmanuel Delahaye** · 19/05/2007, 00h10

Envoyé par dj.motte

Salut,

Dans la version de crocodilex :

Code :

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

1
2
 
const char *nom (const char *chemin)

Que se passe-t-il, si chemin est NULL ?

Comportement indéterminé. C'était facile à deviner !

**crocodilex** · 19/05/2007, 00h25

Envoyé par dj.motte

Salut,

Dans la version de crocodilex :

Code :

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

1
2
 
const char *nom (const char *chemin)

Que se passe-t-il, si chemin est NULL ?

Eh ben il se passe la même chose que dans la première version de ton code....

PS ; A l'occasion il faudra que tu poses la même question aux gars qui ont écrit la libc.

**dj.motte** · 19/05/2007, 00h33

Salut,

Alors Emmanuel ou Crocodilex qu'est-ce qui ne va pas avec cette fonction d'extraction du dernier fichier ou de répertoire ?

Code :

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

1
2
3
4
5
6
7
8
 
char * extract_name( const char * p, int sep  )
 { if( p == NULL ) return NULL ;  
   char * z = strrchr( p ,  sep  ) ;
   if( z != NULL )  z++ ;
   if( z == NULL || *z == '\0' )  return NULL ;
   return z ;
 }

C'est la forme qui fait défaut ?

Récupération du nom

C

Discussions similaires

Partager

Partager