Fonction qui renvoie une liste

**la_praline** · 18/04/2007, 10h48

Bonjour a tous, je désire réaliser une fonction qui crée une liste qui contient le nom des fichiers contenu dans un répertoire. A la fin je renvoie la liste cependant a la compilation j'ai deux erreurs qui résiste.

Voici le code de la fonction :

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
GList liste_fichiers()
{
    /// Variables

    GDir *repertoire;
    GtkListStore *liste;
    gchar *nom;

    /// Instructions

    liste = gtk_list_store_new(1, G_TYPE_STRING);
    repertoire = g_dir_open(Selection_repertoire(),0,NULL);

    if(repertoire)
    {
70 :   while (nom = g_dir_read_name(repertoire) ) 
            {
                nom = g_malloc(40);
                GtkTreeIter *iteration;
                gtk_list_store_append(liste, iteration);
                gtk_list_store_set(liste, iteration,TEXT_COLUMN, nom,-1);
                g_free(nom);
            }

        g_dir_close(repertoire);

81 :    return liste;
    }


} // fin de la fonction liste_fichiers

Voici les erreurs :

Code :

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

1
2
3
 
ligne 70 : warning: assignment discards qualifiers from pointer target type
ligne 81 : error : incompatible types in return

**Franck.H** · 18/04/2007, 11h32

Envoyé par la_praline

Code :

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

1
2
3
 
ligne 70 : warning: assignment discards qualifiers from pointer target type
ligne 81 : error : incompatible types in return

ligne 70 :
La fonction g_dir_read_name renvoie un const gchar *, toi ton pointeur nom est de type gchar * !

ligne 81 :
Normal, ta fonction renvoie une GList mais il faut en fait renvoyer un pointeur sur une GList donc la signature de ta fonction n'est pas correcte, voici la bonne:

Code :

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

GList * liste_fichiers (void)

Par ailleurs je ne vois pourquoi tu as déclaré liste ainsi:

Code :

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

GtkListStore *liste;

Tu veux remplir une GList mais tu utilises un GtkListStore ce qui est complètement différent, tu confond tout là, il faut reprendre des cours de toute urgence. Déclare ta liste ainsi:

Code :

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

GList * liste = NULL;

et adapte ton code par rapport aux fonctions de cette liste ! Ton code est par conséquent complètement faux

**la_praline** · 18/04/2007, 17h19

En réalité c'est une GtkListStore que je désire créer (c t completement faux en effet une fois kon relis) avec dedans tous les noms de fichiers récupérer a partir du répertoire voila ce que ca donne désormais mais j'ai un plantage lors de l'éxécution de la fonction.

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
 
GtkListStore *liste_fichiers()
{
    /// Variables
 
    GDir *repertoire;
    GtkListStore *liste;
    const gchar *nom=NULL;
 
    /// Instructions
 
    liste = gtk_list_store_new(1, G_TYPE_STRING);
    repertoire = g_dir_open(Selection_repertoire(),0,NULL);
 
    if(repertoire)
    {
        while (nom = g_dir_read_name(repertoire) )
            {
                GtkTreeIter *iteration;
                gtk_list_store_append(liste, iteration);
                gtk_list_store_set(liste, iteration,TEXT_COLUMN, nom,-1);
            }
 
        g_dir_close(repertoire);
 
        return liste;
    }
 
 
} // fin de la fonction liste_fichiers

**la_praline** · 18/04/2007, 17h20

J'ai du mal ac les listes donc il se pourrait qu'il y ai encore des betises dans ce code !!

**Franck.H** · 18/04/2007, 17h31

Envoyé par la_praline

J'ai du mal ac les listes donc il se pourrait qu'il y ai encore des betises dans ce code !!

Oui y'a quelques bétises, voici la correction:

Code :

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

1
2
3
4
 
                GtkTreeIter iteration;
                gtk_list_store_append(liste, & iteration);
                gtk_list_store_set(liste, & iteration,TEXT_COLUMN, nom,-1);

Voilà, en fait il ne faut pas déclarer de pointeur sur un GtkTreeIter mais passer l'adresse d'une variable de type GtkTreeIter aux fonctions

**la_praline** · 19/04/2007, 21h23

J'ai avancé ma fonction désormais ma liste est intégré a ma table mais j'ai deux soucis déja il faut ke je dois éxécuter la fonction a l'ouverture car sinon je trouve pas de moyen de recharger une table deja existante !! L'autre soucis c que la liste n'affiche aucun chemin de fichier je comprend pas pourquoi !!

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
 
GtkWidget *liste_fichiers(GtkWidget *table)
{
    /// Variables
 
    GDir *repertoire;
    GtkListStore *liste;
    GtkWidget *vue;
    GtkTreeViewColumn *vue_colonne;
    GtkCellRenderer *colonne;
    GtkWidget *barre_defil;
    const gchar *nom=NULL;
 
    /// Instructions
 
    liste = gtk_list_store_new(1, G_TYPE_STRING);
    repertoire = g_dir_open(Selection_repertoire(),0,NULL);
 
    if(repertoire)
    {
        while (nom = g_dir_read_name(repertoire) )
            {
                GtkTreeIter *iteration;
                gtk_list_store_append(liste,&iteration);
                gtk_list_store_set(liste,&iteration,TEXT_COLUMN, nom,-1);
            }
 
        g_dir_close(repertoire);
 
        vue = gtk_tree_view_new_with_model(GTK_TREE_MODEL(liste));
 
        /// Creation de la colonne
 
            colonne = gtk_cell_renderer_text_new();
            vue_colonne = gtk_tree_view_column_new_with_attributes("Titre",colonne,"text", TEXT_COLUMN,NULL);
 
        /// Ajout de la colonne à la vue
 
            gtk_tree_view_append_column(GTK_TREE_VIEW(vue), vue_colonne);
 
        /// Ajout de la vue a la fenetre
 
            barre_defil = gtk_scrolled_window_new(NULL, NULL);
            gtk_scrolled_window_set_policy(GTK_SCROLLED_WINDOW(barre_defil),GTK_POLICY_AUTOMATIC,GTK_POLICY_AUTOMATIC);
            gtk_container_add(GTK_CONTAINER(barre_defil), vue);
            gtk_table_attach_defaults(GTK_TABLE(table), barre_defil, 20,35,2,22);
 
        return table;
    }
} // fin de la fonction liste_fichiers

Est ce que les chemins de fichiers sont dans un format que je peux afficher ?? Dois je les caster ??

**Franck.H** · 19/04/2007, 21h35

En fait il faudrait voir pour faire un peu autrement car tu met transmet le pointeur nom mais celui-ci est mis à jour à chaque tout de boucle ici:

Code :

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

while (nom = g_dir_read_name(repertoire) )

Ca se répercute donc sur les autres éléments précédemment enregistrés ! Il faudrait peut-être allouer à chaque fois une nouvelle adresse pour la chaîne de caractères que tu récupères à chaque tour de boucle à l'intérieur de la boucle.

Un nouveau pointeur sur char suffit avec une allocation dynamique de la taille de la chaîne pointée par nom en ajoutant 1 au résultat pour le zéro de fin de chaîne puis tu passes alors ce pointeur à la fonction gtk_list_store_set !

Il ne faudra bien sûr pas oublier de libérer toutes ces chaînes de caractères à la fin du programme, tu peux très facilement créer une fonction pour le faire à partir bien sûr de ton GtkListStore.

**bit_o** · 19/04/2007, 22h02

il y a encore une erreur non ?
sur le GtkTreeIter *iteration. Il suffit de mettre GtkTreeIter iteration je crois ?

**Franck.H** · 19/04/2007, 22h16

Envoyé par bit_o

il y a encore une erreur non ?
sur le GtkTreeIter *iteration. Il suffit de mettre GtkTreeIter iteration je crois ?

Oui mais ca je te l'avais marqué dans un post précédent

**la_praline** · 20/04/2007, 19h03

J'ai crée un deuxieme widget pour le nom dont je définis la taille dynamiquement dois yavoir un élément qui va pas car dans la liste ca n'affiche aucun texte alors que si jaffiche le nom dans un label ca passe !! Cette fonction commence a me rendre fou !!!

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
51
52
53
54
55
56
GtkWidget *liste_fichiers(GtkWidget *table)
{
    /// Variables
 
    GDir *repertoire;
    GtkListStore *liste;
    GtkWidget *vue;
    GtkTreeViewColumn *colonne;
    GtkCellRenderer *type_colonne;
    GtkWidget *barre_defil;
    const gchar *nom=NULL;
    const gchar *nom2=NULL;
 
 
    /// Instructions
 
    liste = gtk_list_store_new(1, G_TYPE_STRING);
    repertoire = g_dir_open(Selection_repertoire(),0,NULL);
 
    if(repertoire)
    {
        while (nom = g_dir_read_name(repertoire) )
            {
                nom2=g_malloc(strlen (nom)+1);
                nom2=nom;
                GtkTreeIter iteration;
                gtk_list_store_append(liste,&iteration);
                gtk_list_store_set(liste,&iteration,TEXT_COLUMN, nom2,-1);
                g_free(nom2);
            }
 
        g_free(nom);
 
        g_dir_close(repertoire);
 
        vue = gtk_tree_view_new_with_model(GTK_TREE_MODEL(liste));
 
        /// Creation de la colonne
 
            type_colonne = gtk_cell_renderer_text_new();
            colonne = gtk_tree_view_column_new_with_attributes("Titre",type_colonne,"text", TEXT_COLUMN,NULL);
 
        /// Ajout de la colonne à la vue
 
            gtk_tree_view_append_column(GTK_TREE_VIEW(vue),colonne);
 
        /// Ajout de la vue a la fenetre
 
            barre_defil = gtk_scrolled_window_new(NULL, NULL);
            gtk_scrolled_window_set_policy(GTK_SCROLLED_WINDOW(barre_defil),GTK_POLICY_AUTOMATIC,GTK_POLICY_AUTOMATIC);
            gtk_container_add(GTK_CONTAINER(barre_defil), vue);
            gtk_table_attach_defaults(GTK_TABLE(table), barre_defil, 20,35,2,22);
 
        return table;
    }
} // fin de la fonction liste_fichiers

voila ce que ca donne visuellement la zone chansons devrait etre pleine de nom de fichiers:

Nom : mp3.JPG
Affichages : 111
Taille : 43,8 Ko

Je ne pense pas que le probleme vienne de la vue mais plus du contenu de la boucle while !!

**Franck.H** · 20/04/2007, 19h15

Bien sûr que ca marche pas comme tu as fait ... ton pointeur déjà n'est pas local à la boucle alors qu'il est préférable qu'il le soit donc tu peux déjà virer sa délcaration dans le corps de ta fonction !

Ensuite, en faisant:

Code :

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

nom2 = nom;

Si tu connais ton C, tu devrais savoir qu'ici tu ne copies pas la chaîne d'une adresse à une autre mais simplement le pointeur ce qui fait que nom2 et nom pointent tous deux sur la même adresse alors c'est normal que tu ais des problèmes !

Pour continuer, tu crois qu'en libérant ton pointeur nom2 à la fin de chaque tour de boucle tu fait les choses correctement ? Bin non du tout, il faut libérer les chaîne une fois que tu n'en as plus besoin donc soit quand tu vide ton magasin de données soit quand tu quittes le programme !

Voici la boucle corrigée:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
while (nom = g_dir_read_name(repertoire))
{
   gchar * nom2 = g_malloc (strlen (nom) + 1);
 
   if (nom2 != NULL)
   {
      GtkTreeIter iteration;
 
      strcpy (nom2, nom);
 
      gtk_list_store_append(liste,&iteration);
      gtk_list_store_set(liste,&iteration,TEXT_COLUMN, nom2,-1);
   }
}

**la_praline** · 20/04/2007, 19h29

javou que j'enchaine les boulettes dsl !!

**la_praline** · 20/04/2007, 20h00

je sais que je suis très étourdi et que je fais des betises des fois mais ca n'afficher toujours rien alors que tout est corrigé bien comme tu m'avais conseillé jsui dsl de te faire perdre ton tps mais cette fonction me prend la tete depuis bcp de tps jte l'avoue !!

**Franck.H** · 20/04/2007, 20h20

Oui c'est vrai, tu les enchaînes les bétises mais c'est pas fini

Puis tu sais ce qu'on dit, c'est qu'en faisant des bétises qu'on apprend

Pourquoi faire un:

Code :

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

return table;

?? Je retournais plutôt le widget GtkTreeView:

Code :

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

return vue;

Sinon je ne vois pas le but de retourner un pointeur que tu transmet déjà en arguemnt

De plus, quelle valeur est représentée par ta constante TEXT_COLUMN ?

**la_praline** · 20/04/2007, 20h34

ben dans mon main ma fenetre principale dont tu a la photo précédemment est basée sur une table une fois le répertoire choisit la liste apparait dans la table c'est pour ca que je trouvais plus judicieux de retourner la table !!

Et puis pour le TEXT_COLUMN duquel parles tu ??

Celui-ci :

Code :

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

1
2
 
colonne=gtk_tree_view_column_new_with_attributes("Chansons",type_colonne,"text", TEXT_COLUMN,NULL);

ou de celui ci :

Code :

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

1
2
 
gtk_list_store_set(liste,&iteration,TEXT_COLUMN, nom2,-1);

**Franck.H** · 20/04/2007, 20h42

Envoyé par la_praline

ben dans mon main ma fenetre principale dont tu a la photo précédemment est basée sur une table une fois le répertoire choisit la liste apparait dans la table c'est pour ca que je trouvais plus judicieux de retourner la table !!

Si tu modifies directement la table en y ajoutant un widget et ce à partir du pointeur passé en paramètre, retourner ce même pointeur ne sert strictement à rien, autant ne pas faire renvoyer de données.

Envoyé par la_praline

Et puis pour le TEXT_COLUMN duquel parles tu ??

Peu importe, c'est une constante unique. Comment est-elle créée ?

**la_praline** · 20/04/2007, 20h56

Nan la tout a fait d'accord autant mettre en void mais c lorsque je bossais sur la liste je renvoyai la liste jai fais une modif et jai po eu le réflexe sinon voila l'enum

Code :

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

1
2
3
4
enum {
TEXT_COLUMN,
N_COLUMN
};

**la_praline** · 20/04/2007, 21h02

ca marche je te donne le code fonctionnel il se pourrait que ca puisse servir a quelqu'un !!

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
51
52
53
54
55
56
57
58
59
60
61
enum {
TEXT_COLUMN,
N_COLUMN
};
 
void *liste_fichiers(GtkWidget *table)
{
    /// Variables
 
    GDir *repertoire;
    GtkListStore *liste;
    GtkWidget *vue;
    GtkTreeViewColumn *colonne;
    GtkCellRenderer *type_colonne;
    GtkWidget *barre_defil;
    const gchar *nom=NULL;
 
 
    /// Instructions
 
    liste = gtk_list_store_new(1, G_TYPE_STRING);
    repertoire = g_dir_open(Selection_repertoire(),0,NULL);
 
    if(repertoire)
    {
        while (nom = g_dir_read_name(repertoire))
        {
            const gchar * nom2 = g_malloc (strlen (nom) + 1);
 
            if (nom2 != NULL)
                {
                    GtkTreeIter iteration;
                    strcpy (nom2, nom);
                    gtk_list_store_append(liste,&iteration);
                    gtk_list_store_set(liste,&iteration,TEXT_COLUMN, nom2,-1);
                }
        }
 
        g_free(nom);
 
        g_dir_close(repertoire);
 
        vue = gtk_tree_view_new_with_model(GTK_TREE_MODEL(liste));
 
        /// Creation de la colonne
 
            type_colonne = gtk_cell_renderer_text_new();
            colonne = gtk_tree_view_column_new_with_attributes("Chansons",type_colonne,"text",TEXT_COLUMN,NULL);
 
        /// Ajout de la colonne à la vue
 
            gtk_tree_view_append_column(GTK_TREE_VIEW(vue),colonne);
 
        /// Ajout de la vue a la fenetre
 
            barre_defil = gtk_scrolled_window_new(NULL, NULL);
            gtk_scrolled_window_set_policy(GTK_SCROLLED_WINDOW(barre_defil),GTK_POLICY_AUTOMATIC,GTK_POLICY_AUTOMATIC);
            gtk_container_add(GTK_CONTAINER(barre_defil), vue);
            gtk_table_attach_defaults(GTK_TABLE(table), barre_defil, 20,35,2,22);
    }
} // fin de la fonction liste_fichiers