Discussion :

[shams]

je suis en train de recompiler avec xCode mes vieux programmes fait sur windows. Et j'ai une fonction qui part littéralement en sucette! En effet, pour lister tous les fichiers que j'avais dans un répertoire précis, je faisais ceci:

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
void trouverFichierPGM()
{
    int i=0;
    char tmp[50]; 
    struct dirent *lecture;
    DIR *rep;
 
    rep = opendir("pgm");
 
    while ((lecture = readdir(rep))) 
    {
        if (i > 1  )
        {                                                                                                                                                               
            strcpy (tmp , lecture->d_name) ;
            strcpy (string_list[nbImages], tmp) ;            
            nbImages++; 
        }    
        i++;       
    }
    closedir(rep); 
}

Ce qui marchait très bien! du moins sur windows. Sur mac os, au lieu d'avoir la liste de mes fichiers, il me fait un mix dans le nom des fichiers. Il me garde toujours les 4 premiers caractères du nom précedent puis rajoute le nom complet du fichier suivant...

un petit exemple s'impose:
dans mon dossier j'ai:
-afichier1
-bfichier2
-cfichier3

sous windows j'ai bien
string_list[0] => afichier1
string_list[1] => bfichier2
string_list[2] => cfichier3
nbImages => 3

sous mac, ca me donne:
string_list[0] => aficbficcfich3
string_list[1] => aficbfichier2
string_list[2] => cfichier3
nbImages => 3

ca doi être un truc con, mais....
Merci

[hegros]

Et en ajoutant

Code :

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

1
2
3
 
memset(tmp,0,sizeof tmp);
memset(string_list[nbImages],0,sizeof string_list[nbImages]);

avant

Code :

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

1
2
3
 
strcpy (tmp , lecture->d_name) ;
strcpy (string_list[nbImages], tmp) ;

[stephl]

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
void trouverFichierPGM()
{
    int i=0;
    char tmp[50]; 
    struct dirent *lecture;
    DIR *rep;
 
    rep = opendir("pgm");
 
    while ((lecture = readdir(rep))) 
    {
        if (i > 1  )
        {                                                                                                                                                               
            strcpy (tmp , lecture->d_name) ;
            strcpy (string_list[nbImages], tmp) ;            
            nbImages++; 
        }    
        i++;       
    }
    closedir(rep); 
}

Où nbImages est-elle initialisée? Pourquoi faire:

Code :

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

1
2
strcpy (tmp , lecture->d_name) ;
strcpy (string_list[nbImages], tmp) ;

et pas seulement:

Code :

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

strcpy(string_list[nbImages],lecture->d_name);

?

[shams]

nbImages est une variable globale, initialisé au début du programme

Pour la deuxième remarque, c'est très juste... aucune idée de pourquoi j'ai fait ca...!

[stephl]

Je ne vois pas trop... Pourriez-vous montrer la définition de string_list?

[shams]

Code :

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

1
2
char string_list[100][sizeof(GLUI_String)] ;
int nbImages = 0;

[stephl]

Faites voir un:

Code :

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

printf("%u\n",sizeof (GLUI_String));

[shams]

ha ben voilà...
ça retourne 4...

J'ai remplacé mes definitions en mettant 50 à la place... c'est pas très joli mais ca marche

[Emmanuel Delahaye]

J'ai remplacé mes definitions en mettant 50 à la place... c'est pas très joli mais ca marche

Utilise PATH_MAX, c'est fait pour...

Mais la pratique courante est soit une liste chainée, soit un tableau (éventuellement souple) de pointeur. Dans les 2 cas, le nom est stocké dans une chaine dynamique avec strdup()...

Penser à libérer quand c'est terminé...