Punteros y estructura dirent

[davidtadeo]

Hola a todos, quiciera saber por que no puedo copiar las cadenas que me regresa la estructura dirent, d_name que vendria siendo el nombre del programa o la carpeta, al final los listo pero me salen todos del mismo nombre, en mi caso tengo la ultima carpeta llamada tad2 y es la unica que copia al apuntador de cadenas de caracteres, les agradeceria mucho que me ayuden, gracias

Código C:

#include <stdio.h>
#include <stdlib.h>
#include <dirent.h>
#include <sys/param.h>
#include <dir.h>
 
int main(int argc, char *argv[])
{
  
  FILE *ptr;
  struct dirent *dir, *dirp;
  char nombre [ MAXPATHLEN ], **carpetas, nom [ 100 ];
  int x = 0;
    
  getcwd ( nombre, MAXPATHLEN );
  
  carpetas = malloc ( sizeof ( char * ) * 100 );
  
  if ( ( dir = opendir ( nombre ) ) == NULL ) {
       printf ("No se pudo abrir el directorio %s\n", nombre );
     }
  else {
       
       for ( dirp = readdir ( dir ); dirp != NULL; dirp = readdir ( dir ) ) {
           printf ("%s\n", dirp->d_name ); 
           carpetas [ x ] = dirp->d_name;
           ++x;
       }   
       
       printf ("\n\n");
       for ( x = 0; carpetas [ x ] != NULL; x++ ) {
           printf ("%s\n", carpetas [ x ] );
       }
        
       }
  
  system("PAUSE");  
  return 0;
}
 

[fcoredo]

Según la documentación de readdir: The data returned by readdir() is overwritten by subsequent calls to readdir() for the same directory stream.

En tu ejemplo, lo que copias en "carpetas" son punteros (no el contenido de la cadena). Como el contenido se sobreescribe con cada llamada de readdir, en realidad terminas con 100 punteros a la misma posición de memoria (lo puedes comprobar haciendo un printf("%p\n", cadenas[x]); Verás que seguramente todos apuntan a la misma posición.

Lo que tienes que hacer, es cambiar:

carpetas [ x ] = dirp->d_name;

por

carpetas [ x ] = strdup(dirp->d_name);

Pero no olvides liberar la memoria cuando no la necesites más, ya que cada strdup tiene un malloc implícito:

for(...)
{
free(carpetas[x]);
}

[davidtadeo]

O si muchisimas gracias, me ayduast mucho, e ahi la respuesta, jejejeejjeejje que buen tip, pensare un poco mas, oye estaba pensando en hacer un programa que busque un archivo en la carpeta actual y si no se encontaba en esa que busque dentro de las otras que esten ahi, la funcion scandir no trabaja en windows verdad??? que me sugieres, lo unico que se me ocurre es intentar abrir cada nombre de archivo o carpeta dentro de la carpeta que los contiene como FILE * ( con fopen ) y si no se puede abrir supongo que es una carpeta y ya de ahi me baso para abrir las carpetas, pero que tal si por alguna razon falla el fopen para un archivo, el programa va a pensar que es una carpeta y e ahi un error logico, me sugieres algo mejor??? gracias

[fcoredo]

No se como es en Windows, pero en Unix si intentas abrir una carpeta no te va a dar error, sino que la va a abrir. Si intentas leer el contenido, obtienes números de i-nodos junto con el nombre de los ficheros que están en la carpeta.

Lo que debes hacer para saber si el fichero es una carpeta o un fichero normal, es usar la función fstat().

En este problema, la recursividad te puede ayudar a simplificar el código. Si es un fichero haces lo que quieres hacer y si es una carpeta vuelves a llamar a la función que estás ejecutando con el nombre de la nueva carpeta. Así recorres todo el árbol.

Buena suerte.

[davidtadeo]

ok muy buena, gracias de verdad que amable, eres muy listo....