FindFirstFileEx/FindNextFile

[Алексей++]

забавная проблема напала: получается, что функция FindNextFile нереентерабельна, из-за чего рекурсивный поиск содержимого папки происходит неверно

(слева - моё дерево, справа - проводниковое)
содержимое папки C:\1111111111\2\3\  показано верно (она первая была просмотрена) , а вот папка C:\1111111111\3  показывает то же самое, что в предыдущей, хотя там совсем не то

И как же этот рекурсивный поиск сделать, неужели вытаскивать содержимое по одной папке, затем по этим файлам пробегаться снова и заходить в каждую папку ?

[RXL]

Леш, вероятно ты использовал имя папки как идентификатор ассоциативного массива. Тогда уж используй в качестве ключа полный путь.
А как еще без рекурсии?

[Алексей++]

нет, я строю дерево вообще-то, а не просто ассоциативный массив заполняю. Я не против рекурсии, но как-то она тут не вышла )

Добавлено через 7 минут и 38 секунд:
кстати, если вторую папку 3 переименовать (скажем в 5) , то содержимое папки 5 совсем не удаётся узнать.

[Sla]

Покажи...

[Алексей++]

Sla, минутку, нужно функцию рафинировать, чтобы показать )

Добавлено через 27 минут и 21 секунду:
вот, убрал лишнее, результат выводится в трейс (без отступов)

Код:

	void PrintDirs(const WCHAR* path_)
	{
		if(!path_)
		{
			path_=L"";
		}

		if(path_[0])
		{
			std::wstring fullpath__;
			WIN32_FIND_DATA WFD={0};

			fullpath__=path_;
			if(!fullpath__.empty() && *fullpath__.rbegin()!=L'\\')
			{
				fullpath__.push_back(L'\\');
			}
			fullpath__.push_back(L'*');

			HANDLE F=::FindFirstFileEx
			(
				fullpath__.c_str()
				,FindExInfoStandard
				,&WFD,FindExSearchNameMatch
				,0
				,0
			);

			//убираем звёздочку
			if(!fullpath__.empty())
			{
				fullpath__.erase(fullpath__.size()-1);
			}

			do
			{
				const bool isdir=((WFD.dwFileAttributes&FILE_ATTRIBUTE_DIRECTORY)!=0);

				if(isdir)
				{
					if(WFD.cFileName[0]==L'.')
					{
						if(WFD.cFileName[1]==0)continue;
						if(WFD.cFileName[1]==L'.')
						{
							if(WFD.cFileName[2]==0)continue;
						}
					}
				}

				if(isdir)
				{
					if(!fullpath__.empty() && *fullpath__.rbegin()!=L'\\')
					{
						fullpath__.push_back(L'\\');
					}
					fullpath__+=WFD.cFileName;

					PrintDirs(fullpath__.c_str());
				}
				else
				{
					//WFD.cFileName
				}

				TRACE("\r\n");
				TRACE(WFD.cFileName);


			}
			while(::FindNextFile(F,&WFD));
			::FindClose(F);
		}
	}

Добавлено через 10 минут и 59 секунд:
блин, у меня такое чуйство, что у меня где-то косяк, но не пойму, где

Добавлено через 4 минуты и 54 секунды:
трейс тоже радует

Код:

4
_CrtDbgReport: String too long or IO Error
3
2

а там всего лишь имя файла выводится "текстовый документ.txt"


аааааа, я сегодня блондин )))  Где-то я явно что-но накосячил с памятью...

[Sla]

Алексей1153++,
шо ты здесь накрутил...

[Алексей++]

Sla, что не так ?

[Sla]

наверное все так...
Ты как-то не сильно и рафинировал код

Printdir()
{
findfirst
do
   if is_dir {Printdir}
while findnext
}

И весь код... А у тебя много, на мой взгляд, мусора

[Алексей++]

Sla, весь мусор там связан с тем, что с путём надо няньчится

Можно и так (только две лишние директории появятся в списке)

Код:

	void PrintDirs(const WCHAR* path_)
	{
			std::wstring fullpath__;
			WIN32_FIND_DATA WFD={0};

			fullpath__=path_;
			if(!fullpath__.empty() && *fullpath__.rbegin()!=L'\\')
			{
				fullpath__.push_back(L'\\');
			}
			fullpath__.push_back(L'*');

			HANDLE F=::FindFirstFileEx
			(
				fullpath__.c_str()
				,FindExInfoStandard
				,&WFD,FindExSearchNameMatch
				,0
				,0
			);

			//убираем звёздочку
			if(!fullpath__.empty())
			{
				fullpath__.erase(fullpath__.size()-1);
			}

			do
			{
				if(WFD.dwFileAttributes&FILE_ATTRIBUTE_DIRECTORY)
				{
					fullpath__+=WFD.cFileName;

					PrintDirs(fullpath__.c_str());
				}

				TRACE("\r\n");
				TRACE(WFD.cFileName);

				fullpath__.push_back(L'\\');
			}
			while(::FindNextFile(F,&WFD));
			::FindClose(F);
	}

[Dimka]

Алексей1153++, гонишь.

Код: (C++)

#include <iostream>
#include <string>
#include <windows.h>

using namespace std;

void outtree(int level, wstring &path);

void procitem(int level, wstring &path, WIN32_FIND_DATA &findFileData)
{
        wstring fileName = wstring(findFileData.cFileName);
        if(fileName.compare(L".") != 0 && fileName.compare(L"..") != 0)
        {
                for(int i = 0; i < level; i += 1)
                {
                        wcout << L" ";
                }
                wcout << fileName << endl;
                if((findFileData.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY) != 0)
                {
                        outtree(level + 1, path + wstring(L"\\") + fileName);
                }
        }
}

void outtree(int level, wstring &path)
{
        WIN32_FIND_DATA findFileData;
        wstring searchPath = path + wstring(L"\\*");
        HANDLE searchHandle = FindFirstFileEx(searchPath.c_str(), FindExInfoStandard, &findFileData, FindExSearchNameMatch, NULL, 0);
        if(searchHandle != INVALID_HANDLE_VALUE)
        {
                procitem(level, path, findFileData);
                while(FindNextFile(searchHandle, &findFileData) != 0)
                {
                        procitem(level, path, findFileData);
                }
                FindClose(searchHandle);
        }
}

int main(int argc, char *argv[])
{
        outtree(0, wstring(L"C:\\Test"));
        getchar();
        return 0;
}

11
 2
  3
   4
    tt.txt
 3
  t.txt

[Sla]

Алексей1153++, зачем нянчиться?
Понятно , что .. и . нужно пропускать
Мною показанный "код" - чем тебя не устраивает.
И причем здесь нереентерабелность findnext?   

[Алексей++]

Dimka, я в курсе, вот и хочу понять, в каком именно моменте

[Sla]

Алексей1153++, упрощай

[Алексей++]

не, Слав, там нечего упрощать, мне всё так надо.  Всё, ошибку нашёл, это я переработался, похоже

Димке спасибо за намёк

вот тут я херню сморозил

Код:

fullpath__+=WFD.cFileName;

путь портился для поиска следующих элементов

[zubr]

Кстати, если посмотреть в отладчике на работу проводника, то будет видно, что он использует те же FindFirst /FindNext

[Алексей++]

zubr, в АПИ вроде нет таких функций. Или я не нашёл

[RXL]

http://msdn.microsoft.com/en-us/library/windows/desktop/aa364418%28v=vs.85%29.aspx
http://msdn.microsoft.com/en-us/library/windows/desktop/aa364428%28v=vs.85%29.aspx

Леш, уже пятница! Пора отдыхать

[zubr]

Алексей1153++, ну я их сокращенно написал, опуская окончание File

[Алексей++]

RXL, Ром, эти то я нашёл ))

Да и какая тяпница, у меня круглый день - рабочий. Вот сижу слушаю одну из любимых композиций с psyradio
http://www.youtube.com/watch?v=W3q5sRcUwR0=player_embedded

[Dimka]

путь портился для поиска следующих элементов

Переменные делятся на входные и выходные (параметры), а также локальные. Есть хорошее правило - не менять значениях входных параметров.

Однако, чтобы этим хорошим правилом воспользоваться, ещё надо уметь делать декомпозицию по функциям. И не только уметь, но и хотеть этим заниматься.

Можно и с другой стороны зайти. Определить, какие переменные характеризуют состояние объекта или процесса в пространстве состояний, а какие - лишь вспомогательные для расчётов. Затем взять за правило делить код на расчётный и отвечающий за смену состояний (управляющий), вынося расчёты в отдельные функции или вспомогательные объекты.

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

[Алексей++]

Dimka, учтём )

[dark_rain]

Извиняюсь если не в тему, но для подобных вещей boost::filesystem мега круто.

[Алексей++]

бустом не пользуюсь как-то )