Operation sur les pointeurs

Version imprimable

Bonjour à toutes et à tous.

J'ai un petit (gros) problème au sujet des pointeurs et des fonctions.
Voila en gros à quoi ressemble mon code.

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
 
#include <stdio.h>
#include <dirent.h>
#include <sys/stat.h>
#include <string.h>
 
int getInfoFile(char *psNomRep, char *psNomFichier, char *psCheminFichier, int *piDateFichier);
 
int getInfoFile(char *psNomRep, char *psNomFichier, char *psCheminFichier, int *piDateFichier)
{
        struct stat s;
 
        psCheminFichier = (char *) malloc(strlen(psNomRep) + strlen(psNomFichier) + 1);
        if(psCheminFichier == NULL)
        {
                printf("ERR : Plus assez de mémoire dispo...");
                exit(1);
        }
 
        strcpy(psCheminFichier, psNomRep);
        strcat(psCheminFichier, psNomFichier);
 
        lstat(psCheminFichier, &s);
        *piDateFichier = s.st_mtime;
}

Mon souci là-dessus vient de ma variable de sortie psCheminFichier. Je ne sais pas comment faire pour bien la récupérer.

J'ai effectivement compris que pour récupérer une variable en sortie, il fallait en faire un pointeur et attribuer à sa valeur un certain contenu à l'intérieur même de la fonction. Puis, lorsqu'on appelle la fonction, il faut placer dans le bon paramètre l'adresse d'une variable. Mais là, avec strcpy, je ne sais que faire...

Dans mon main(), j'appelle cette fonction en plusieurs occasions.

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
int main()
{
        struct dirent *pDp;
        DIR *pRep_id;
        int iDateFichier = 0;
        int iDateFichierCourant = 0;
        char sCheminFichier;
        char sCheminFichierCourant;
        char *psNomRep = "/MonChemin/";
        int i=0;
 
        pRep_id = opendir(psNomRep);
        for(pDp = readdir(pRep_id); pDp != NULL; pDp = readdir(pRep_id))
        {
                if(strcmp(pDp->d_name, ".") != 0 && strcmp(pDp->d_name, "..") != 0)
                {
                        if (i == 0)
                        {
                                getInfoFile(psNomRep, pDp->d_name, &sCheminFichier, &iDateFichier);
                                i++;
                        }
                        else
                        {
                                getInfoFile(psNomRep, pDp->d_name, &sCheminFichierCourant, &iDateFichierCourant);
 
                                if(iDateFichierCourant < iDateFichier)
                                {
                                        iDateFichier = iDateFichierCourant;
                                        sCheminFichier =sCheminFichierCourant;
                                }
                        }
                }
        }
        closedir(pRep_id);
        printf("\n-----------\nLe plus ancien fichier est %s.\n-----------\n", sCheminFichier);
        return 0;
}

Ma fonction main() va me servir à ramener le fichier avec la date de modification la plus basse pour pouvoir ensuite faire un traitement sur ce fichier.

Je m'excuse d'avance pour la maladresse de mon code :mrgreen:.

Bahan

24/05/2006, 10h41
Swoög

Salut ;)

Comme tu l'as dit, il faut effectivement faire passer un pointeur sur la variable que tu veux modifier, or cette variable est déjà un pointeur, et plus, tu alloues de la mémoire pour lui (donc tu modifies le pointeur en lui-même et pas uniquement ce qui est pointé)

il te faut donc faire passer un pointeur de pointeur, et déclarer un char* (et non un char) dans ton main ;)

de plus, pense à faire un free sur ton pointeur, pour le libéré (puisqu'il est alloué avec malloc ;)

J'ai modifié le code de mon main() pour essayer de prendre en compte tes conseils.
Voici ce que cela donne (en rouge), sachant que ma fonction getInfoFile() n'a pas changé de celle du début du topic.

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
#include <stdio.h>
#include <dirent.h>
#include <sys/stat.h>
#include <string.h>
 
int main()
{
        struct dirent *pDp;
        DIR *pRep_id;
        int iDateFichier = 0;
        int iDateFichierCourant = 0;
       char *psCheminFichier;
        char *psCheminFichierCourant;
     char *psNomRep = "/MonChemin/";
        int i=0;
 
        pRep_id = opendir(psNomRep);
        for(pDp = readdir(pRep_id); pDp != NULL; pDp = readdir(pRep_id))
        {
                if(strcmp(pDp->d_name, ".") != 0 && strcmp(pDp->d_name, "..") != 0)
                {
                        if (i == 0)
                        {
                                getInfoFile(psNomRep, pDp->d_name, psCheminFichier, &iDateFichier);
                                i++;
                        }
                        else
                        {
                                getInfoFile(psNomRep, pDp->d_name, psCheminFichierCourant, &iDateFichierCourant);
 
                                if(iDateFichierCourant < iDateFichier)
                                {
                                        iDateFichier = iDateFichierCourant;
                                        realloc(psCheminFichier, strlen(psCheminFichierCourant)+1);
                                        strcpy(psCheminFichier, psCheminFichierCourant);
                                        free(psCheminFichierCourant);
                                }
                        }
                }
        }
   closedir(pRep_id);
   printf("\n-----------\nLe plus ancien fichier est %s.\n-----------\n", psCheminFichier);
   free(psCheminFichier);
   return 0;
}

Je n'ai pas d'erreur de compilation. Cependant, j'ai maintenant une erreur de type :

Code:

1
2
 
Segmentation fault(coredump).

Cela signifie en gros que j'attribue mal mes variables n'est-ce pas ?

Bahan

24/05/2006, 11h17
Swoög

comme je te l'ai dit, il faut passer un pointeur de pointeur, donc remettre les & au moment de l'appel, rajouter un * à char* dans la déclaration/définition de la fonction et penser à travailler sur *psCheminFichier au lieu de psCheminFichier
24/05/2006, 11h58
Emmanuel Delahaye

Citation:

Envoyé par Swoög

comme je te l'ai dit, il faut passer un pointeur de pointeur, donc remettre les & au moment de l'appel, rajouter un * à char* dans la déclaration/définition de la fonction et penser à travailler sur *psCheminFichier au lieu de psCheminFichier

Que c'est laid...
24/05/2006, 12h07
Swoög

Citation:

Envoyé par Emmanuel Delahaye

Que c'est laid...

Si tu as une autre solution... comme il doit manipuler plusieurs variables, le retour de fonction n'est pas envisageable... je ne vois, personnellement, rien d'autre, ça ne signifie, bien sûr, pas qu'il n'y en a pas d'autre...
24/05/2006, 13h53
Bahan

Merci pour votre aide.

Bahan
24/05/2006, 14h11
Emmanuel Delahaye

Citation:

Envoyé par Swoög

Si tu as une autre solution... comme il doit manipuler plusieurs variables, le retour de fonction n'est pas envisageable... je ne vois, personnellement, rien d'autre, ça ne signifie, bien sûr, pas qu'il n'y en a pas d'autre...

La structure. Je l'ai souvent évoquée...

Yop les gens.

Pour finir, j'utilise la fonction popen, bien sympathoche.

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
 
#include <stdio.h>
#include <string.h>
 
main()
{
        char    * pszExtension;
        char    * pszRadical;
        char    szBufferW[1000];
        char    szBuffer[1000];
        FILE *  pfPop;
        printf("DEBUT\n");
        pfPop   = popen ("ls -rt /tmp | sort", "r") ;
        if ( pfPop == (FILE *) NULL)
        {
                printf("Erreur sur ouverture du PIPE\n");
                exit(0);
        }
        while(fgets(szBuffer, sizeof(szBuffer), pfPop) != (char *) NULL)
        {
                szBuffer[strlen(szBuffer) - 1] = '\0';
                strcpy(szBufferW, szBuffer);
                pszRadical = strtok(szBufferW, ".");
                pszExtension = strtok(NULL, ".");
                printf("Buffer lu <%s> radical <%s> extension <%s>\n",
                        szBuffer,
                        pszRadical,
                        pszExtension);
        }
        printf("FIN\n");
        exit(0);
}

Cela me permet de ramener mes fichiers dans l'ordre que je désire vu que je peux coupler le ls avec ce que je veux, et m'extrait le radical et l'extension de mes fichiers.

Est-ce plus propre déjà ? ;)

Bahan