LINUX.ORG.RU

История изменений

Исправление CYB3R, (текущая версия) :

Странный какой-то пример, названия директорий выводит дважды.
Сделал небольшой фикс: будет печатать названия директорий один раз, добавляя в конце «/», чтобы отличать от файлов, а ещё будет скрывать файлы, начинающиеся с точки.

#include <stdlib.h>
#include <stdio.h>
#include <sys/types.h>
#include <string.h>
#include <errno.h>
/* "readdir" etc. are defined here. */
#include <dirent.h>
/* limits.h defines "PATH_MAX". */
#include <limits.h>

/* List the files in "dir_name". */

static void
list_dir (const char * dir_name)
{
    DIR * d;

    /* Open the directory specified by "dir_name". */

    d = opendir (dir_name);

    /* Check it was opened. */
    if (! d) {
        fprintf (stderr, "Cannot open directory '%s': %s\n",
                 dir_name, strerror (errno));
        exit (EXIT_FAILURE); /* здесь exit вообще уместен? 0_о */
/* если вдруг ошибка, где-то в середине, программа завершится, не закончив досчитывать директории */
    }
    while (1) {
        struct dirent * entry;
        const char * d_name;

        /* "Readdir" gets subsequent entries from "d". */
        entry = readdir (d);
        if (! entry) {
            /* There are no more entries in this directory, so break
               out of the while loop. */
            break;
        }
        d_name = entry->d_name;
        /* Print the name of the file and directory. */
        if (strncmp(d_name, ".", strlen("."))) /* мой фикс #1: не показываем файлы, начинающиеся с "." */
/* не уверен, будет ли константа одинаково себя вести на всех системах (битность, все дела), потому использую strlen */
            printf ("\n%s/%s", dir_name, d_name);

        /* See if "entry" is a subdirectory of "d". */

        if (entry->d_type & DT_DIR) {

            /* Check that the directory is not "d" or d's parent. */
            
            if (strcmp (d_name, "..") != 0 &&
                strcmp (d_name, ".") != 0) {
                int path_length;
                char path[PATH_MAX];
 
                path_length = snprintf (path, PATH_MAX,
                                        "%s/%s", dir_name, d_name);
                printf ("/"); /* мой фикс #2: добавим слеш к именам директорий */
                if (path_length >= PATH_MAX) {
                    fprintf (stderr, "Path length has got too long.\n");
                    exit (EXIT_FAILURE); /* здесь exit вообще уместен? 0_о */
/* если вдруг ошибка, где-то в середине, программа завершится, не закончив досчитывать директории */
                }
                /* Recursively call "list_dir" with the new path. */
                list_dir (path);
            }
        }
    }
    /* After going through all the entries, close the directory. */
    if (closedir (d)) {
        fprintf (stderr, "Could not close '%s': %s\n",
                 dir_name, strerror (errno));
        exit (EXIT_FAILURE); /* здесь exit вообще уместен? 0_о */
/* если вдруг ошибка, где-то в середине, программа завершится, не закончив досчитывать директории */
    }
}

int main ()
{
    list_dir ("/dev");
    return 0;
}
Нормально написал? И про использование exit везде: это нормально?

Исходная версия CYB3R, :

Странный какой-то пример, названия директорий выводит дважды.
Сделал небольшой фикс: будет печатать названия директорий один раз, добавляя в конце «/», чтобы отличать от файлов, а ещё будет скрывать файлы, начинающиеся с точки.

#include <stdlib.h>
#include <stdio.h>
#include <sys/types.h>
#include <string.h>
#include <errno.h>
/* "readdir" etc. are defined here. */
#include <dirent.h>
/* limits.h defines "PATH_MAX". */
#include <limits.h>

/* List the files in "dir_name". */

static void
list_dir (const char * dir_name)
{
    DIR * d;

    /* Open the directory specified by "dir_name". */

    d = opendir (dir_name);

    /* Check it was opened. */
    if (! d) {
        fprintf (stderr, "Cannot open directory '%s': %s\n",
                 dir_name, strerror (errno));
        exit (EXIT_FAILURE); /* здесь exit вообще уместен? 0_о */
/* если вдруг ошибка, где-то в середине, программа завершится, не закончив пересчитывать директории */
    }
    while (1) {
        struct dirent * entry;
        const char * d_name;

        /* "Readdir" gets subsequent entries from "d". */
        entry = readdir (d);
        if (! entry) {
            /* There are no more entries in this directory, so break
               out of the while loop. */
            break;
        }
        d_name = entry->d_name;
        /* Print the name of the file and directory. */
        if (strncmp(d_name, ".", strlen("."))) /* мой фикс #1: не показываем файлы, начинающиеся с "." */
/* не уверен, будет ли константа одинаково себя вести на всех системах (битность, все дела), потому использую strlen */
            printf ("\n%s/%s", dir_name, d_name);

        /* See if "entry" is a subdirectory of "d". */

        if (entry->d_type & DT_DIR) {

            /* Check that the directory is not "d" or d's parent. */
            
            if (strcmp (d_name, "..") != 0 &&
                strcmp (d_name, ".") != 0) {
                int path_length;
                char path[PATH_MAX];
 
                path_length = snprintf (path, PATH_MAX,
                                        "%s/%s", dir_name, d_name);
                printf ("/"); /* мой фикс #2: добавим слеш к именам директорий */
                if (path_length >= PATH_MAX) {
                    fprintf (stderr, "Path length has got too long.\n");
                    exit (EXIT_FAILURE); /* здесь exit вообще уместен? 0_о */
/* если вдруг ошибка, где-то в середине, программа завершится, не закончив пересчитывать директории */
                }
                /* Recursively call "list_dir" with the new path. */
                list_dir (path);
            }
        }
    }
    /* After going through all the entries, close the directory. */
    if (closedir (d)) {
        fprintf (stderr, "Could not close '%s': %s\n",
                 dir_name, strerror (errno));
        exit (EXIT_FAILURE); /* здесь exit вообще уместен? 0_о */
/* если вдруг ошибка, где-то в середине, программа завершится, не закончив пересчитывать директории */
    }
}

int main ()
{
    list_dir ("/dev");
    return 0;
}
Нормально написал? И про использования exit везде: это нормально?