Discussion :

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

[droggo]

Gio,

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

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

[Kryptonaute]

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]

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]

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]

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]

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]

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

Aui,

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]

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]

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]

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]

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]

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]

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]

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]

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]

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]

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 ?