サブフォルダを再帰的に訪問することはできません

Question

ディレクトリ内のすべてのフォルダとファイルをファイルサイズと共に再帰的にリストするプログラムを作成しようとしています。私はプログラムが深いサブフォルダの1つのレベルに行くように見えるので、私はまだ最初の部分に取り組んでいます。

誰でも問題を見つけることができますか？私は立ち往生している。

#include <stdio.h> 
#include <stdlib.h> 
#include <sys/types.h> 
#include <sys/stat.h> 
#include <string.h> 
#include <strings.h> 
#include <dirent.h> 
#include <unistd.h> 

void listdir(const char *name) { 
    DIR *dir; 
    struct dirent *entry; 
    int file_size; 

    if (!(dir = opendir(name))) 
     return; 
    if (!(entry = readdir(dir))) 
     return; 

    do { 
     if (entry->d_type == DT_DIR) { 
      char path[1024]; 
      if (strcmp(entry->d_name, ".") == 0 || strcmp(entry->d_name, "..") == 0) 
       continue; 
      printf("./%s\n", entry->d_name); 
      listdir(entry->d_name); 
     } 
     else 
      printf("./%s\n", entry->d_name); 
    } while (readdir(dir) != NULL); 
    closedir(dir); 
} 

int main(void) 
{ 
    listdir("."); 
    return 0; 
} 

Answer 1

最初の問題は、あなたがreaddirの戻り値を放棄され、while条件である、それは、エントリに割り当てる必要があります。

また、再帰的にlistdirを呼び出すときは、パスの前に親の名前を追加する必要があります。それ以外の場合は、現在の作業ディレクトリから常に検索します。 このバージョンをお試しください：

void listdir(const char *name) { 
    DIR *dir; 
    struct dirent *entry; 
    int file_size; 

    if (!(dir = opendir(name))) 
      return; 

    while ((entry = readdir(dir)) != NULL) { // <--- setting entry 
      printf("%s/%s\n", name, entry->d_name); 
      if (entry->d_type == DT_DIR) { 
        char path[1024]; 
        if (strcmp(entry->d_name, ".") == 0 || strcmp(entry->d_name, "..") == 0) 
          continue; 
        sprintf(path, "%s/%s", name, entry->d_name); // <--- update dir name properly by prepend the parent folder. 
        listdir(path); 
      } 
    } 
    closedir(dir); 
} 

int main(void) 
{ 
    listdir("."); 
    return 0; 
} 

Answer 2

以下は、コードに対する最小限の修正です。私はここで非標準のasprintfを使用する自由を取った。 glibcを使用していない場合は、代わりにsnprintfなどを使用してください。

特に、listdirに与えられたパスは、現在の作業ディレクトリからの完全な相対パスでなければなりません。または絶対パス。しかし、entry->d_nameのファイル名はファイルの基本名に過ぎません。したがって、それはlistdirに渡されたパスと連結されなければなりません。私はまた、不適切なdo ... whileをwhileループに変更しました。

#define _GNU_SOURCE 
#include <sys/types.h> 
#include <sys/stat.h> 
#include <string.h> 
#include <strings.h> 
#include <dirent.h> 
#include <unistd.h> 
#include <stdio.h> 
#include <stdlib.h> 

void listdir(const char *path) { 
    DIR *dir; 
    struct dirent *entry; 

    if (!(dir = opendir(path))) 
     return; 

    while ((entry = readdir(dir))) { 
     if (strcmp(entry->d_name, ".") == 0 || strcmp(entry->d_name, "..") == 0) 
      continue; 

     char *current; 
     if (asprintf(&current, "%s/%s", path, entry->d_name) < 0) { 
      // asprintf failed 
      fprintf(stderr, "asprintf failed, exiting"); 
      goto exit; 
     } 

     puts(current); 
     if (entry->d_type == DT_DIR) { 
      listdir(current); 
     } 

     free(current); 
    } 

exit: 
    closedir(dir); 
} 

int main(void) 
{ 
    listdir("."); 
}