функция stat () не возвращается правильно

Question

функция stat () не возвращается правильно

0

У меня есть эта функция

long processFile(char * fileName)
{
    struct stat statBuf;
    mode_t mode;
    int result;
    result = stat(fileName, &statBuf);
    if (result == -1);
        return -1;
    if(S_ISDIR(mode))
        return(processDirectory(fileName));
    else
        return 1;
}

Если "fileName" является фактическим файлом, я хочу его вернуть, но если "fileName" - это каталог, я хочу, чтобы он processDirectory к моей функции, называемой processDirectory. По какой-то причине независимо от того, что я делаю, вызов stat() продолжает возвращать -1. Я пробовал жестко кодировать имя файла, например "test.txt", и каталог под названием "/test", которые оба находятся в том же каталоге, что и мой исполняемый файл. Любые указания будут оценены.

AlwaysLearning 17 март 2014, в 20:12

Источник

4

Значение errno равно?
Ignacio Vazquez-Abrams 17 март 2014, в 18:38
0

ERRNO? @ IgnacioVazquez-Abrams
AlwaysLearning 17 март 2014, в 18:42
1

Да. Как сказано в разделе ВОЗВРАЩАЕМОЕ ЗНАЧЕНИЕ stat(2) страницы stat(2) .
Ignacio Vazquez-Abrams 17 март 2014, в 18:43
2

заменить return -1; with { perror("stat"); return -1; };
Basile Starynkevitch 17 март 2014, в 18:59
0

... или (более познавательно): {fprintf(stderr, "stat() failed: %s\n", strerror(errno)); return -1;} @BasileStarynkevitch
alk 17 март 2014, в 19:04
0

Я заменил result = stat(fileName, &statBuf); if (result == -1);return -1; with if(stat(fileName, &statBuf) == -1){ perror("stat() error "); return -1; } и теперь это работает. Спасибо всем за помощь.
AlwaysLearning 17 март 2014, в 19:07
1

Вау, какая здесь большая слепота ... посмотри на мой ответ.
Jim Balter 17 март 2014, в 19:12
0

«теперь это работает» - потому что вы убрали фальшивку ; после оператора if, а не из-за вызова perror.
Jim Balter 17 март 2014, в 19:16
1

Кстати, вы находитесь в "хорошей" компании с этой ошибкой - Google "Goto Fail".
Jim Balter 17 март 2014, в 19:18
0

"заменить return -1; на {perror (" stat "); return -1;};" - фактически, эта точная замена всегда будет возвращать -1, а также добавит еще одну фальшивку ; ,
Jim Balter 17 март 2014, в 19:22
1

«Мой компилятор не прав», «операционная система возвращает неправильные результаты», по моему большому опыту , правильна один раз из миллиона или около того.
vonbrand 17 март 2014, в 19:23
0

@vonbrand Да, как только я увидел этот заголовок, я понял, что его автор - неофит. Мы получаем много этого здесь. С опытом приходит смирение, по крайней мере, в этой области - ошибки почти всегда являются нашими собственными, мы способны к их широкому кругу и часто полностью их игнорируем. (В этом случае я увидел точку с запятой в течение доли секунды после просмотра кода, но я потратил на это 40 лет.)
Jim Balter 17 март 2014, в 19:24
0

@JimBalter Во-первых, я просто хочу сказать спасибо, что нашли мою опечатку. Кроме того, как бы вы предложили мне назвать название, чтобы показать смирение такому опытному человеку?
AlwaysLearning 18 март 2014, в 19:18
0

Первое, что нужно сделать, это проверить утверждение, запустив отладчик или вставив в printf значение, которое фактически возвращает stat. Что касается заголовка, «stat не возвращает ожидаемое значение» позволит избежать того, что он работает неправильно.
Jim Balter 18 март 2014, в 23:17

Показать ещё 12 комментариев

Теги:

c++

c

stat

linux

unix

2 ответа

1

Из вашего сообщения похоже, что может быть какая-то проблема с безопасностью пользователя или путь к каталогу может не иметь соответствующего разрешения для поиска. В любом случае, вы можете выполнить быстрое тестирование с помощью команды "stat" и проверить, работает ли она или нет.

$ stat filename.txt

Да errno во время stat(), возвращающего -1, будет полезно понять, почему этот системный вызов терпит неудачу.

Mantosh Kumar 17 март 2014, в 16:02

0

Помимо того, что это не дает правильного решения (см. Мой ответ), это не ответ вообще, это комментарий. Пожалуйста, оставляйте комментарии как комментарии (у вас сейчас достаточно представителей), а не как ответы.
Jim Balter 17 март 2014, в 19:14

Ещё вопросы

Да. Как сказано в разделе ВОЗВРАЩАЕМОЕ ЗНАЧЕНИЕ stat(2) страницы stat(2) .
заменить return -1; with { perror("stat"); return -1; };
... или (более познавательно): {fprintf(stderr, "stat() failed: %s\n", strerror(errno)); return -1;} @BasileStarynkevitch
Я заменил result = stat(fileName, &statBuf); if (result == -1);return -1; with if(stat(fileName, &statBuf) == -1){ perror("stat() error "); return -1; } и теперь это работает. Спасибо всем за помощь.
Вау, какая здесь большая слепота ... посмотри на мой ответ.
«теперь это работает» - потому что вы убрали фальшивку ; после оператора if, а не из-за вызова perror.
Кстати, вы находитесь в "хорошей" компании с этой ошибкой - Google "Goto Fail".
"заменить return -1; на {perror (" stat "); return -1;};" - фактически, эта точная замена всегда будет возвращать -1, а также добавит еще одну фальшивку ; ,
«Мой компилятор не прав», «операционная система возвращает неправильные результаты», по моему большому опыту , правильна один раз из миллиона или около того.
@vonbrand Да, как только я увидел этот заголовок, я понял, что его автор - неофит. Мы получаем много этого здесь. С опытом приходит смирение, по крайней мере, в этой области - ошибки почти всегда являются нашими собственными, мы способны к их широкому кругу и часто полностью их игнорируем. (В этом случае я увидел точку с запятой в течение доли секунды после просмотра кода, но я потратил на это 40 лет.)
@JimBalter Во-первых, я просто хочу сказать спасибо, что нашли мою опечатку. Кроме того, как бы вы предложили мне назвать название, чтобы показать смирение такому опытному человеку?
Первое, что нужно сделать, это проверить утверждение, запустив отладчик или вставив в printf значение, которое фактически возвращает stat. Что касается заголовка, «stat не возвращает ожидаемое значение» позволит избежать того, что он работает неправильно.
Помимо того, что это не дает правильного решения (см. Мой ответ), это не ответ вообще, это комментарий. Пожалуйста, оставляйте комментарии как комментарии (у вас сейчас достаточно представителей), а не как ответы.

Jim Balter · Accepted Answer · 2014-03-17T16-38-00.000Z

if (result == -1);
    return -1;

Удалите точку с запятой из первой строки.

Кстати, компиляция с gcc -Wall бы предупредил вас ....
Да, то же самое касается Apple: zdnet.com/…