0

Короче, проблема в том, что не получается попадать в другие диски и папки(выше чем лежит исполняемый файл).
И вообще сильное ощущение, что написана функция очень криво и не правильно. Подскажите как бы ее переписать корректно

bool Search_File(TCHAR *buffer, char mas[])
{
HANDLE file;
WIN32_FIND_DATA ffd = {0};
TCHAR temp[MAX_PATH];
TCHAR str[MAX_PATH];

SetCurrentDirectory(buffer);//текуций католог 
file = FindFirstFile(TEXT(mas),&ffd);


if (file == INVALID_HANDLE_VALUE) {
    cout <<"Файл не был найден"<<endl;
    return false;
}
else{
    do{
        if(!strcmp(ffd.cFileName, TEXT("."))||!strcmp(ffd.cFileName, 
TEXT(".."))){
            continue;
        }   
        if (ffd.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY){//если не 
установлен бит, тогда файл
            strcpy(temp, buffer);

            strcat(temp, "\\");
            strcat(temp, ffd.cFileName);

            Search_File(buffer, mas);

            strcpy(buffer, temp);

            continue;
        }
        cout<<"найденый файл:"<<ffd.cFileName<<endl;

    }
    while(FindNextFile(file,&ffd) != 0);    
}
FindClose(file);

return true;    
}
1
  • можно filesystem:: recursive_directory_iterator использовать, пример
    – jfs
    11 мая 2018 в 5:10

1 ответ 1

3

Обратите внимание на следующие моменты:

Во-первых, неправильно используется функция SetCurrentDirectory(), что чревато дальнейшими ошибками по мере роста кода программы. Дело в том, что текущий каталог - это глобальная переменная процесса и изменять ее таким образом, в одной из рабочих функций, и тем более - не восстанавливать - некорректная практика. Если имеем дело с многопоточным приложением и в разных потоках используются файловые операции (а как правило в больших программах именно так и происходит - ведутся протоколы, логи, багрепорты, считываются/записываются опции и т.п.), то хаотичное изменение текущего каталога недопустимо. Если эта функция используется, то обычно в момент инициализации программы при запуске и большинство (если не все) файловых операций осуществляются по относительным путям (относительно текущего).

Поэтому рекомендую убрать из функции вызов SetCurrentDirectory() и передавать на вход функции Search_File() полный путь + маска искомого файла (помним, что в FindFirstFile() можно использовать символы * и ?) или образовывать этот полный путь динамически путем присоединения внутри функции mas[] к buffer.

Во-вторых, в определенных случаях (когда целевое имя содержит подстановочные символы, точку или имя каталога) необходимо иметь права доступа к корневому каталогу и ко всем подкаталогам в поисковом пути для успешного поиска.

2
  • как это динамически? 11 мая 2018 в 12:30
  • В том смысле, что полный путь поиска формируется прямо в функции Search_File() из mas[] и buffer на ее входах: 1) либо добавляем буфер памяти, который использует функция для сцепления строк, при необходимости увеличивая/уменьшая его размер; 2) либо передаем buffer с запасом свободного места в конце, чтобы в него можно прицепить mas[], а после окончания поиска снова обрезать, записав нулевой символ
    – Denys Save
    11 мая 2018 в 13:58

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.