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.

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

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

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

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

[Emmanuel Delahaye]

Salut,

Alors Emmanuel 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 ?

Je te laisse juge :

http://emmanuel-delahaye.developpez.com/goret.htm

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

[zennn]

Merci beaucoup pour vos réponses.
J'ai beaucoup appris aujourd'hui

Sinon, une autre question, dans une fonction GTK je veux récupérer le chemin à l'extérieur de la fonction, cependant la fonction donné par GTK garde le chemin à l'intérieur de la fonction, il n'y a pas de return.

voilà le code:

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
 
void OnExplorer1(GtkWidget *pWidget, gpointer data)
{
    GtkWidget *pFileSelection;
//    GtkWidget *pDialog;
    GtkWidget *pParent;
     gchar *sChemin1;
 
    pParent = GTK_WIDGET(data);
 
    /* Creation de la fenetre de selection */
    pFileSelection = gtk_file_chooser_dialog_new("Ouvrir l'image...",
	GTK_WINDOW(pParent),
	GTK_FILE_CHOOSER_ACTION_OPEN,
	GTK_STOCK_CANCEL, GTK_RESPONSE_CANCEL,
	GTK_STOCK_OPEN, GTK_RESPONSE_OK,
	NULL);
    /* On limite les actions a cette fenetre */
    gtk_window_set_modal(GTK_WINDOW(pFileSelection), TRUE);
 
    /* Affichage fenetre */
    switch(gtk_dialog_run(GTK_DIALOG(pFileSelection)))
    {
        case GTK_RESPONSE_OK:
            /* Recuperation du chemin */
            sChemin1 = gtk_file_chooser_get_filename(GTK_FILE_CHOOSER(pFileSelection));
//             pDialog = gtk_message_dialog_new(GTK_WINDOW(pFileSelection),
//                 GTK_DIALOG_MODAL,
//                 GTK_MESSAGE_INFO,
//                 GTK_BUTTONS_OK,
//                 "Chemin du fichier :\n%s", sChemin);
//            gtk_dialog_run(GTK_DIALOG(pDialog));
//            gtk_widget_destroy(pDialog);
//	    g_free(sChemin);
 
            break;
        default:
            break;
    }
 
 
 
    chemin1=sChemin1;
    gtk_widget_destroy(pFileSelection);
}

J'ai mis en commentaires quelques lignes de code car je ne veux pas afficher le chemin mais plutôt le récupérer dans une variable.

Merci.

[zennn]

Merci beaucoup pour vos réponses.
J'ai beaucoup appris aujourd'hui

Sinon, une autre question, dans une fonction GTK je veux récupérer le chemin à l'extérieur de la fonction, cependant la fonction donné par GTK garde le chemin à l'intérieur de la fonction, il n'y a pas de return.

voilà le code:

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
 
void OnExplorer1(GtkWidget *pWidget, gpointer data)
{
    GtkWidget *pFileSelection;
//    GtkWidget *pDialog;
    GtkWidget *pParent;
     gchar *sChemin1;
 
    pParent = GTK_WIDGET(data);
 
    /* Creation de la fenetre de selection */
    pFileSelection = gtk_file_chooser_dialog_new("Ouvrir l'image...",
	GTK_WINDOW(pParent),
	GTK_FILE_CHOOSER_ACTION_OPEN,
	GTK_STOCK_CANCEL, GTK_RESPONSE_CANCEL,
	GTK_STOCK_OPEN, GTK_RESPONSE_OK,
	NULL);
    /* On limite les actions a cette fenetre */
    gtk_window_set_modal(GTK_WINDOW(pFileSelection), TRUE);
 
    /* Affichage fenetre */
    switch(gtk_dialog_run(GTK_DIALOG(pFileSelection)))
    {
        case GTK_RESPONSE_OK:
            /* Recuperation du chemin */
            sChemin1 = gtk_file_chooser_get_filename(GTK_FILE_CHOOSER(pFileSelection));
//             pDialog = gtk_message_dialog_new(GTK_WINDOW(pFileSelection),
//                 GTK_DIALOG_MODAL,
//                 GTK_MESSAGE_INFO,
//                 GTK_BUTTONS_OK,
//                 "Chemin du fichier :\n%s", sChemin);
//            gtk_dialog_run(GTK_DIALOG(pDialog));
//            gtk_widget_destroy(pDialog);
//	    g_free(sChemin);
 
            break;
        default:
            break;
    }
 
 
 
    chemin1=sChemin1;
    gtk_widget_destroy(pFileSelection);
}

J'ai mis en commentaires quelques lignes de code car je ne veux pas afficher le chemin mais plutôt le récupérer dans une variable.

Merci.

La fonction je l'ai trouvé dans le Tutoriel de GTK, mais il parle pas d'une quelconque récupération de chemin

[Emmanuel Delahaye]

Sinon, une autre question, dans une fonction GTK je veux récupérer le chemin à l'extérieur de la fonction, cependant la fonction donné par GTK garde le chemin à l'intérieur de la fonction, il n'y a pas de return.

Code :

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

1
2
3
4
5
 
void OnExplorer1(GtkWidget *pWidget, gpointer data)
{
<...>
}

J'ai mis en commentaires quelques lignes de code car je ne veux pas afficher le chemin mais plutôt le récupérer dans une variable.

C'est là qu'il faut apprendre à utiliser le paramètre 'data'.

Lors de l'installation de l'évènement, tu passes l'adresse d'une variable qui va être utilisée dans la fonction callback pour placer le résultat. Si c'est une chaine de caractères, il y a plusieurs solutions.

Code :

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

1
2
3
4
 
   char chaine[BIG_ENOUGH];
 
   on_event(..., callback, chaine);

avec

Code :

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

1
2
3
4
5
6
7
 
void callback (..., gpointer data)
{
   char *chaine = data; /* gpointer est en fait un 'void *' */
 
   strcpy(data, mesinfos);
}

ou bien

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
struct mydata
{
   char *chaine;
};
 
   struct mydata mydata = {0};
 
   on_event(..., callback, &mydata);
 
   /* usage */
 
   free (mydata.chaine), mydata.chaine = NULL;

avec

Code :

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

1
2
3
4
5
6
7
 
void callback (..., gpointer data)
{
   struct mydata *p_data = data;
 
   p_data->chaine = strdup(mesinfos);
}

etc.