Récupération du nom

Version imprimable

Voir 40 message(s) de cette discussion en une page

Récupération du nom

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:

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 :aie: )

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

Merci d'avance.

18/05/2007, 17h35
droggo

Gio,

Il faudrait penser à réserver de la mémoire pour "char *name", et plus tard, penser à libérer cette mémoire :)
18/05/2007, 17h39
souviron34
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..
18/05/2007, 17h42
Kryptonaute
pas très propre de boucler 1000 fois :
Code:

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

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

Code:

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;
 
}

18/05/2007, 19h58
droggo

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.
18/05/2007, 20h26
zennn

[Résolu] Récupétation du nom

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.
18/05/2007, 20h31
Freed0

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.
18/05/2007, 21h51
gl

Citation:

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.
18/05/2007, 22h34
droggo

Aui,

Citation:

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. :D

Salut,

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

Code:

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.

Citation:

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:

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

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:

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.

Citation:

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:

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:

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;
}

Salut

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

Citation:

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:

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 ; }

Citation:
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:

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 :aie: )

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:

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...

19/05/2007, 00h07
dj.motte
Salut,

Dans la version de crocodilex :
Code:

1 2 const char *nom (const char *chemin)
Que se passe-t-il, si chemin est NULL ?
19/05/2007, 00h10
Emmanuel Delahaye
Citation:
Envoyé par dj.motte

Salut,

Dans la version de crocodilex :

Code:

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

Que se passe-t-il, si chemin est NULL ?
Comportement indéterminé. C'était facile à deviner !
:dehors:
19/05/2007, 00h25
crocodilex
Citation:
Envoyé par dj.motte

Salut,

Dans la version de crocodilex :

Code:

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.... :lol:

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

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:

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 ?

Voir 40 message(s) de cette discussion en une page