Форум программистов, компьютерный форум CyberForum.ru

C++

Войти
Регистрация
Восстановить пароль
 
Рейтинг: Рейтинг темы: голосов - 10, средняя оценка - 4.90
-=ЮрА=-
Заблокирован
Автор FAQ
#1

Помогите выловить баг, сил моих больше нету((( - C++

06.04.2011, 11:06. Просмотров 1235. Ответов 15
Метки нет (Все метки)

Программа глючит при добавлении больших структурированных каталогов, раньше думал, что ошибка выделения памяти, оказалось нет...Очень надо помощь спеца, СРОЧНО!!!
Конструктор класса
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
CStorage::CStorage()
{
    CoInitialize(NULL);
    GetTempPath(MAX_PATH,szTMP);
    GetShortPathName(szTMP,szTMP,MAX_PATH);
    lpszW = (WCHAR *)malloc(sizeof(WCHAR));
    lpBuf = malloc(sizeof(char));
    //MEM_TOP_DOWN
/*  pDATA = (WIN32_FIND_DATA *)VirtualAlloc
                        (
                            (void *)&pDATA,
                            sizeof(WIN32_FIND_DATA),
                            MEM_COMMIT  ,
                            PAGE_READWRITE
                        );*/
    pDATA = (WIN32_FIND_DATA *)malloc(sizeof(WIN32_FIND_DATA));
    pFile = NULL;
    m_hWnd= NULL;
    nStgElm= 0;
    uStgLen= 0;
    pDATA[nStgElm].dwReserved1 = 0;
    iINDENT = 0;
}
Рекурсивный поиск каталогов, ошибка вылетает в try - catch блоке
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
WIN32_FIND_DATA * CStorage::MkPathList(LPCTSTR szPath)
{
    WIN32_FIND_DATA pFILE_DATA;ULONG uIND = nStgElm; 
    pDATA[nStgElm].dwFileAttributes = GetFileAttributes(szPath);
    pDATA[nStgElm].dwReserved1      = iINDENT;
    HANDLE hFile = CreateFile(
        strcpy(pDATA[nStgElm].cFileName,szPath), 
            GENERIC_READ,
            FILE_SHARE_READ,
            NULL, 
            OPEN_EXISTING, 
            FILE_ATTRIBUTE_NORMAL,
            NULL);
    if(hFile != INVALID_HANDLE_VALUE)
    {
        uStgLen += (pDATA[nStgElm].nFileSizeLow = GetFileSize(hFile,NULL));
        CloseHandle(hFile);
    }
    else
    {
        if(strcmp(pDATA[nStgElm].cFileName,"") == 0)
           strcpy(pDATA[nStgElm].cFileName,"+");
        sprintf(pFILE_DATA.cFileName,"%s\\*.*",pDATA[nStgElm].cFileName);
        hFile = FindFirstFile(pFILE_DATA.cFileName,&pFILE_DATA);
        if(hFile != INVALID_HANDLE_VALUE)
        {
            iINDENT++;
            pDATA[(uIND = nStgElm)].nFileSizeLow = uStgLen;
            pDATA = (WIN32_FIND_DATA *)realloc
                        (
                            (void *)pDATA,
                            sizeof(WIN32_FIND_DATA)*(1 + (nStgElm = nStgElm + 1))
                        );
            do
            {
                if(strcmp(pFILE_DATA.cFileName,".")  == 0 ||
                   strcmp(pFILE_DATA.cFileName,"..") == 0)
                    continue;
                else
                {
                    pFILE_DATA.dwReserved1 = iINDENT;
                    pDATA[nStgElm] = pFILE_DATA;
                    sprintf(pDATA[nStgElm].cFileName,"%s\\%s",szPath,pFILE_DATA.cFileName);
                    if(FILE_ATTRIBUTE_DIRECTORY & pFILE_DATA.dwFileAttributes)
                    try
                    {
                        MkPathList(pDATA[nStgElm].cFileName);
                    }
                    catch(...)
                    {
                        iINDENT--;
                        //MessageBox(m_hWnd,strerror(GetLastError()),"ERROR",MB_OK);
                    }
                    else
                    {
                        uStgLen += pFILE_DATA.nFileSizeLow;
                        pDATA = (WIN32_FIND_DATA *)realloc
                                    (
                                        (void *)pDATA,
                                        sizeof(WIN32_FIND_DATA)*(2 + (nStgElm = nStgElm + 1))
                                    );
                    }
                    PumpMessages(m_hWnd);
                }
            }
            while(FindNextFile(hFile,&pFILE_DATA) != 0);
            FindClose(hFile);
            pDATA[uIND].nFileSizeLow = uStgLen - pDATA[uIND].nFileSizeLow;
            iINDENT--;
        }
    }
    return pDATA;
}
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
taras atavin
Ушёл с форума.
3569 / 1752 / 91
Регистрация: 24.11.2009
Сообщений: 27,619
11.04.2011, 07:54     Помогите выловить баг, сил моих больше нету((( #2
Цитата Сообщение от -=ЮрА=- Посмотреть сообщение
ошибка вылетает в try - catch блоке
Ты б его хоть показал, а то телепат всего то один, да и тот на форуме игроделов.
ForEveR
В астрале
Эксперт С++
7970 / 4732 / 320
Регистрация: 24.06.2010
Сообщений: 10,541
Завершенные тесты: 3
11.04.2011, 09:06     Помогите выловить баг, сил моих больше нету((( #3
taras atavin, 45-52 строчка во втором листинге
taras atavin
Ушёл с форума.
3569 / 1752 / 91
Регистрация: 24.11.2009
Сообщений: 27,619
11.04.2011, 09:24     Помогите выловить баг, сил моих больше нету((( #4
Вылетает конкретно на 51-й? Если кач вообще сработал, то ищи, откуда выплыло само исключение. И iINDENT где то объявлен и мне просто опять не по глазам? Какой у него тип?
-=ЮрА=-
Заблокирован
Автор FAQ
11.04.2011, 09:39  [ТС]     Помогите выловить баг, сил моих больше нету((( #5
taras atavin, программа вылетает при рекурсивном вызове функции MkPathList и то при большой вложенности подкаталогов, это сначало наталкнуло меня на мысль что realloc не справляется с выделением памяти, но если бы происходили косяки с выделением памяти блок-памяти отведенный под pDATA обнулялся, а так с трай кэтч прога работает дальше. Я специально не хочу вводить в СStorage лист-контрол куда и осуществляю выгрузку результатов поиска, этот класс хочу юзать как для консолей так и диалоговых приложений. Параметр iINDENT отвечает за сдвиг иконок в листбоксе, т.е. указывает на вложенность подкаталога, вот привожу скриншот работы проги
Миниатюры
Помогите выловить баг, сил моих больше нету(((  
taras atavin
Ушёл с форума.
3569 / 1752 / 91
Регистрация: 24.11.2009
Сообщений: 27,619
11.04.2011, 09:44     Помогите выловить баг, сил моих больше нету((( #6
Так с отловленным исключением всё работает? Тогда сделай лог и вывод максимально расширенного сообщения об исключении и ищи, откуда оно лезет.
-=ЮрА=-
Заблокирован
Автор FAQ
11.04.2011, 09:46  [ТС]     Помогите выловить баг, сил моих больше нету((( #7
PS: iINDENT = 0; - 22-ая строчка в конструкторе, в объяве мог не показать, вырезал класс с проги
ну и сокращал
taras atavin
Ушёл с форума.
3569 / 1752 / 91
Регистрация: 24.11.2009
Сообщений: 27,619
11.04.2011, 09:47     Помогите выловить баг, сил моих больше нету((( #8
Каков полный путь к тому файл/подкаталогу, на котором выскакивает исключение? Какой он длины, сколько в нём уровней? Допустимо ли это с точки зрения как системы в целом, так и конкретно используемых функций.
-=ЮрА=-
Заблокирован
Автор FAQ
11.04.2011, 09:49  [ТС]     Помогите выловить баг, сил моих больше нету((( #9
Та пишет i/o error и всё - мне этого мало чтобы выявить баг. Ввёл рапорт об ошибке, но тоже мало чем помогло(((

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
CString CStgMakerDlg::StrWinError(HRESULT hRes)
{
    CString str;
    str.Format("\r\nHRESULT : 0x%p\r\n(int)hRes : %d\r\n",hRes,hRes);
    str+="ERRMSG  : ";
    LPSTR errString = NULL;  // will be allocated and filled by FormatMessage
    FormatMessage(
        FORMAT_MESSAGE_ALLOCATE_BUFFER |
        FORMAT_MESSAGE_FROM_SYSTEM, // use windows internal message table
        0,       // 0 since source is internal message table
        hRes,    // this is the error code 
                          // Could just as well have been an error code from generic
                          // Windows errors from GetLastError()
        0,       // auto-determine language to use
        (LPSTR)&errString,
        0,       // min size for buffer
        0 ); 
    return str += errString;
}
taras atavin
Ушёл с форума.
3569 / 1752 / 91
Регистрация: 24.11.2009
Сообщений: 27,619
11.04.2011, 09:50     Помогите выловить баг, сил моих больше нету((( #10
Может быть ты просто влез в ограничение?

Добавлено через 55 секунд
А i/o егор - это ошибка обращения к диску.
-=ЮрА=-
Заблокирован
Автор FAQ
11.04.2011, 17:22  [ТС]     Помогите выловить баг, сил моих больше нету((( #11
taras atavin я перепаял прогу вобщем прога вылетает в try_catch который приведен в коде ниже, причём для каждого каталог в одном конкретном месте, в некоторых каталогах вообще исключения нет,
strerror пишет Improper link. Моё предположение что FindFirstFile обнуляет pFILE_DATA и вот здесь
C++
1
pDATA[nStgElm] = pFILE_DATA;
мы делаем
C++
1
 pDATA[nStgElm] = NULL
Помоги как можно исключение обрабоать???
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
do
            {
                if(strcmp(pFILE_DATA.cFileName,".")  == 0 ||
                   strcmp(pFILE_DATA.cFileName,"..") == 0)
                    continue;
                else
                {
                    pFILE_DATA.dwReserved1 = iINDENT;
                    try
                    {
                    pDATA[nStgElm] = pFILE_DATA;
                    sprintf(pDATA[nStgElm].cFileName,"%s\\%s",szPath,pFILE_DATA.cFileName);
                    }
                    catch(...)
                    {
                        sprintf
                            (
                                chERR,
                                "do-while: begin\r\n%s\r\n%s\r\n%s\r\n%s",
                                strerror(GetLastError()),
                                pDATA[nStgElm - 2].cFileName,
                                pDATA[nStgElm - 1].cFileName,
                                pFILE_DATA.cFileName
                            );
                        MessageBox(m_hWnd,chERR,strerror(GetLastError()),MB_OK);
                    }
                    if(FILE_ATTRIBUTE_DIRECTORY & pFILE_DATA.dwFileAttributes)
                        MkPathList(pDATA[nStgElm].cFileName);
                    else
                    {
                        uStgLen += pFILE_DATA.nFileSizeLow;
                        pDATA = (WIN32_FIND_DATA *)realloc
                                    (
                                        (void *)pDATA,
                                        sizeof(WIN32_FIND_DATA)*(1 + (nStgElm = nStgElm + 1))
                                    );
                    }
                    PumpMessages(m_hWnd);
                }
            }
            while(FindNextFile(hFile,&pFILE_DATA) != 0);
taras atavin
Ушёл с форума.
3569 / 1752 / 91
Регистрация: 24.11.2009
Сообщений: 27,619
12.04.2011, 08:22     Помогите выловить баг, сил моих больше нету((( #12
Цитата Сообщение от -=ЮрА=- Посмотреть сообщение
обнуляет pFILE_DATA
То есть файл просто не найден? В сообщении об исключении выведи значение pFILE_DATA для проверки, потом сделай проверку ифом.
ValeryS
Модератор
6550 / 5016 / 463
Регистрация: 14.02.2011
Сообщений: 16,729
12.04.2011, 08:38     Помогите выловить баг, сил моих больше нету((( #13
Может срабатывает ограничение на длину имени файла???
макрос есть MAXPACH по моему.
определен в 255
попробуй добавить.
где то писал рекурсивный поиск файлов
найду исходник вышлю

Добавлено через 2 минуты
GetLastError() что пишет
-=ЮрА=-
Заблокирован
Автор FAQ
12.04.2011, 09:47  [ТС]     Помогите выловить баг, сил моих больше нету((( #14
Баг уже выловил - как оказалось при определённой структуре внутренних подкаталогов добавляемой дирректории, происходит перекрытие блока памяти в котором содержится LPCTSTR szPath, в резулльтате szPath == NULL, об этом говорит указанный скриншот, проблемму решил введением дополнительной строковой переменной для хранения пути к каталогу, все проблеммы исчезли!
Ребят спасибо всем за участие!
Миниатюры
Помогите выловить баг, сил моих больше нету(((  
ValeryS
Модератор
6550 / 5016 / 463
Регистрация: 14.02.2011
Сообщений: 16,729
12.04.2011, 19:40     Помогите выловить баг, сил моих больше нету((( #15
C++
1
2
3
4
5
6
7
8
9
10
11
12
typedef struct _WIN32_FIND_DATA {  
DWORD dwFileAttributes; 
 FILETIME ftCreationTime; 
 FILETIME ftLastAccessTime; 
 FILETIME ftLastWriteTime; 
 DWORD nFileSizeHigh;
  DWORD nFileSizeLow; 
 DWORD dwReserved0; 
 DWORD dwReserved1;
 TCHAR cFileName[MAX_PATH]; 
TCHAR cAlternateFileName[14];
} WIN32_FIND_DATA
TCHAR cFileName[MAX_PATH]; !!!!
WinDef.h
C++
1
#define MAX_PATH          260
о чем я и писал
преопредели MAX_PATH
сделай 1024 и ошибка должна исчезнуть
хотя наверно можно найти и более длинное имя файла
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
13.04.2011, 10:26     Помогите выловить баг, сил моих больше нету(((
Еще ссылки по теме:
Помогите пож создать программу на С++ чтобы выводился список только тех студентов, у которых больше одной двойки C++
Структуры: Вывести все танки с 1940-1950 г. чей калибр , больше 50 мм. или же просто больше 50 C++
Visual C++ нету шаблонов !
нету входа в цикл C++
Почему нету хелпа? с++ C++ Builder

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
-=ЮрА=-
Заблокирован
Автор FAQ
13.04.2011, 10:26  [ТС]     Помогите выловить баг, сил моих больше нету((( #16
ValeryS, пасибо за участие, но как уже говорил выше я нашёл решение и даже без ввода доп строковой переменной, вот оно:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
WIN32_FIND_DATA * CStorage::MkPathList(LPCTSTR szPath)
{
    WIN32_FIND_DATA pFILE_DATA;ULONG uIND = nStgElm; 
    pDATA[nStgElm].dwFileAttributes = GetFileAttributes(szPath);
    pDATA[nStgElm].dwReserved1      = iINDENT;
    HANDLE hFile = CreateFile(
        strcpy(pDATA[nStgElm].cFileName,szPath), 
            GENERIC_READ,
            FILE_SHARE_READ,
            NULL, 
            OPEN_EXISTING, 
            FILE_ATTRIBUTE_NORMAL,
            NULL);
    if(hFile != INVALID_HANDLE_VALUE)
    {
        uStgLen += (pDATA[nStgElm].nFileSizeLow = GetFileSize(hFile,NULL));
        CloseHandle(hFile);
        pDATA = (WIN32_FIND_DATA *)realloc
            (
                    (void *)pDATA,
                    sizeof(WIN32_FIND_DATA)*(1 + (nStgElm = nStgElm + 1))
            );
    }
    else
    {
        pFILE_DATA.cFileName[0] = '\0';
        if(strcmp(pDATA[nStgElm].cFileName,"") != 0)
            sprintf(pFILE_DATA.cFileName,"%s\\*.*",pDATA[nStgElm].cFileName);
        hFile = FindFirstFile(pFILE_DATA.cFileName,&pFILE_DATA);
        if(hFile != INVALID_HANDLE_VALUE)
        {
            iINDENT++;
            pDATA[(uIND = nStgElm)].nFileSizeLow = uStgLen;
            pDATA = (WIN32_FIND_DATA *)realloc
                        (
                            (void *)pDATA,
                            sizeof(WIN32_FIND_DATA)*(1 + (nStgElm = nStgElm + 1))
                        );
            do
            {
                if(strcmp(pFILE_DATA.cFileName,".")  == 0 ||
                   strcmp(pFILE_DATA.cFileName,"..") == 0)
                    continue;
                else
                {
                    //GetShortPathName(pDATA[uIND].cFileName,pDATA[nStgElm].cFileName,MAX_PATH);
                    sprintf
                        (
                            pDATA[nStgElm].cFileName,
                            "%s\\%s",
                            pDATA[uIND].cFileName,
                            pFILE_DATA.cFileName
                        );
                    pDATA[nStgElm].dwReserved1 = iINDENT;
                    pDATA[nStgElm].nFileSizeLow = pFILE_DATA.nFileSizeLow;
                    if(FILE_ATTRIBUTE_DIRECTORY & pFILE_DATA.dwFileAttributes)
                        MkPathList(pDATA[nStgElm].cFileName);
                    else
                    {
                        uStgLen += pFILE_DATA.nFileSizeLow;
                        pDATA = (WIN32_FIND_DATA *)realloc
                                    (
                                        (void *)pDATA,
                                        sizeof(WIN32_FIND_DATA)*(1 + (nStgElm = nStgElm + 1))
                                    );
                    }
                    PumpMessages(m_hWnd);
                }
            }
            while(FindNextFile(hFile,&pFILE_DATA) != 0);
            FindClose(hFile);
            pDATA[uIND].nFileSizeLow = uStgLen - pDATA[uIND].nFileSizeLow;
            iINDENT--;
        }
    }
    return pDATA;
}
В заключение скажу что сама XP поддерживает имена в 256 символовов и имя длинней 256 символов дать файлу не удасться, ну как например дать новому каталогу имя conЕщё раз всем спасибо за участие
Yandex
Объявления
13.04.2011, 10:26     Помогите выловить баг, сил моих больше нету(((
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru