虽然循环停止没有明显的原因

时间:2013-10-19 06:25:30

标签: c loops

我有以下C代码:

#include <dirent.h>
#include <stdio.h>
#include <string.h>
#include <signal.h>
#include <unistd.h>


int main(int argc, const char *argv[]){
    const char *directory_path;
    directory_path = argv[1];
    char *dirname;
    dirname = strdup(directory_path);
    recursive_ls(dirname);

}


int recursive_ls(char *dirname){
    printf("%s\n",dirname);
    DIR *d;
    struct dirent *dir;

    if ((d=opendir(dirname))==-1) {
        perror("Oops");
        return(0);
    }
    if(d){
        while((dir = readdir(d)) !=NULL){
            char *dname = dir->d_name;
            puts(dname);
/*          if(strcmp(dir->d_type,DT_REG)){
                puts("I am a regular file");
            } else if(strcmp(dir->d_type,DT_DIR)){
                puts("I am a directory.");
                            recursive_ls(strcat(strcat(dirname,"/"),dir->d_name));
                } else {
                    puts("I am not a file");
            }
*/    
        }

        closedir(d);
    }
return(0);
}

如果我在while循环中注释掉if(){} else if(){} else(){},我会得到以下输出:

Debug
.project
pipe_test.c
.
..

目录中的所有内容。另一方面,如果我取消注释,那么我得到以下内容:

/home/wilbert/workspace/pipe_test
.cproject

我想我至少应该得到“我不是一个档案”的部分,如果所有的事情,如果递归步骤搞砸了;但是,由于某种原因,我不这样,我感到困惑。这个东西编译并显示没有语法错误所以我真的很困惑。为什么会这样?

2 个答案:

答案 0 :(得分:4)

由于Jonathon建议d_typeunsigned char,所以改为:

if(dir->d_type == DT_REG){
   puts("I am a regular file");
} else if(dir->d_type == DT_DIR){
   puts("I am a directory.");
   recursive_ls(strcat(strcat(dirname,"/"),dir->d_name));
} else {
   puts("I am not a file");
}

答案 1 :(得分:2)

d_type struct dirent成员不是字符串(char *),而是unsigned char。这实际上只是一个8位整数,它是一种基本类型。比较基元类型时,我们使用==运算符。 strcmp严格用于比较字符串。

if (dir->d_type == DT_REG) {
   puts("I am a regular file");
} else if (dir->d_type == DT_DIR) {
   puts("I am a directory.");
   recursive_ls(strcat(strcat(dirname,"/"),dir->d_name));
} else {
   puts("I am not a file");
}

有关详细信息,请参阅manual

而且,即使您的原始代码编译没有错误,当您包含注释的正文时,它会提供很多警告。一个好的建议是始终将编译器警告视为错误。