Форум программистов, компьютерный форум, киберфорум
C++
Форум программистов и сисадминов Киберфорум > Форум программистов > C/C++ > C++
Войти
Регистрация
Восстановить пароль
Правила Карта форума Темы раздела Блоги Социальные группы Поиск Сообщения за день  
 
Рейтинг 4.67/9: Рейтинг темы: голосов - 9, средняя оценка - 4.67
Заблокирован
Автор FAQ
1

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

06.04.2011, 11:06. Показов 1737. Ответов 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;
}
0
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
 06.04.2011, 11:06
Ответы с готовыми решениями:

Перевод из си в ассемблер.Сил моих больше нет.
Вообщем,задача состоит в следующем перевести сишный код в ассемблерный. Часть кода есть,но т.к....

Огненная обезьяна WebBrowser (нет сил моих больше)
Уважаемые программисты! Взываю к Вашей любви к искусству программировать. Столкнулся (и бьюсь об...

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

Сложный запрос, нету сил больше
Есть таблица в которой много полей, выбрать нужно все. Есть 2 поля с помощью которых нужно...

15
4226 / 1795 / 211
Регистрация: 24.11.2009
Сообщений: 27,562
11.04.2011, 07:54 2
Цитата Сообщение от -=ЮрА=- Посмотреть сообщение
ошибка вылетает в try - catch блоке
Ты б его хоть показал, а то телепат всего то один, да и тот на форуме игроделов.
0
В астрале
Эксперт С++
8049 / 4806 / 655
Регистрация: 24.06.2010
Сообщений: 10,562
11.04.2011, 09:06 3
taras atavin, 45-52 строчка во втором листинге
0
4226 / 1795 / 211
Регистрация: 24.11.2009
Сообщений: 27,562
11.04.2011, 09:24 4
Вылетает конкретно на 51-й? Если кач вообще сработал, то ищи, откуда выплыло само исключение. И iINDENT где то объявлен и мне просто опять не по глазам? Какой у него тип?
0
Заблокирован
Автор FAQ
11.04.2011, 09:39  [ТС] 5
taras atavin, программа вылетает при рекурсивном вызове функции MkPathList и то при большой вложенности подкаталогов, это сначало наталкнуло меня на мысль что realloc не справляется с выделением памяти, но если бы происходили косяки с выделением памяти блок-памяти отведенный под pDATA обнулялся, а так с трай кэтч прога работает дальше. Я специально не хочу вводить в СStorage лист-контрол куда и осуществляю выгрузку результатов поиска, этот класс хочу юзать как для консолей так и диалоговых приложений. Параметр iINDENT отвечает за сдвиг иконок в листбоксе, т.е. указывает на вложенность подкаталога, вот привожу скриншот работы проги
Миниатюры
Помогите выловить баг, сил моих больше нету(((  
0
4226 / 1795 / 211
Регистрация: 24.11.2009
Сообщений: 27,562
11.04.2011, 09:44 6
Так с отловленным исключением всё работает? Тогда сделай лог и вывод максимально расширенного сообщения об исключении и ищи, откуда оно лезет.
0
Заблокирован
Автор FAQ
11.04.2011, 09:46  [ТС] 7
PS: iINDENT = 0; - 22-ая строчка в конструкторе, в объяве мог не показать, вырезал класс с проги
ну и сокращал
0
4226 / 1795 / 211
Регистрация: 24.11.2009
Сообщений: 27,562
11.04.2011, 09:47 8
Каков полный путь к тому файл/подкаталогу, на котором выскакивает исключение? Какой он длины, сколько в нём уровней? Допустимо ли это с точки зрения как системы в целом, так и конкретно используемых функций.
1
Заблокирован
Автор 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;
}
0
4226 / 1795 / 211
Регистрация: 24.11.2009
Сообщений: 27,562
11.04.2011, 09:50 10
Может быть ты просто влез в ограничение?

Добавлено через 55 секунд
А i/o егор - это ошибка обращения к диску.
0
Заблокирован
Автор 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);
0
4226 / 1795 / 211
Регистрация: 24.11.2009
Сообщений: 27,562
12.04.2011, 08:22 12
Цитата Сообщение от -=ЮрА=- Посмотреть сообщение
обнуляет pFILE_DATA
То есть файл просто не найден? В сообщении об исключении выведи значение pFILE_DATA для проверки, потом сделай проверку ифом.
0
Модератор
Эксперт по электронике
8876 / 6651 / 912
Регистрация: 14.02.2011
Сообщений: 23,420
12.04.2011, 08:38 13
Может срабатывает ограничение на длину имени файла???
макрос есть MAXPACH по моему.
определен в 255
попробуй добавить.
где то писал рекурсивный поиск файлов
найду исходник вышлю

Добавлено через 2 минуты
GetLastError() что пишет
0
Заблокирован
Автор FAQ
12.04.2011, 09:47  [ТС] 14
Баг уже выловил - как оказалось при определённой структуре внутренних подкаталогов добавляемой дирректории, происходит перекрытие блока памяти в котором содержится LPCTSTR szPath, в резулльтате szPath == NULL, об этом говорит указанный скриншот, проблемму решил введением дополнительной строковой переменной для хранения пути к каталогу, все проблеммы исчезли!
Ребят спасибо всем за участие!
Миниатюры
Помогите выловить баг, сил моих больше нету(((  
0
Модератор
Эксперт по электронике
8876 / 6651 / 912
Регистрация: 14.02.2011
Сообщений: 23,420
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 и ошибка должна исчезнуть
хотя наверно можно найти и более длинное имя файла
0
Заблокирован
Автор 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Ещё раз всем спасибо за участие
0
13.04.2011, 10:26
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
 13.04.2011, 10:26
Помогаю со студенческими работами здесь

С++Builder сил моих не хватает
Задача. В одномерном массиве, состоящем из 14 вещественных элементов, вычислить: 1) минимальный...

Сил моих нет Денвер постоянно ругается на рабочие коды
Warning: session_start() : Cannot send session cookie - headers already sent by (output started at...

Сил больше нет!
Всем привет! Комп - мать asus p8p67 rex1.xx, i5, gtx650, 8gb оперативки Проблема следующая: 1....

Виснет компьютер, сил больше нет
Здравствуйте, видел уже много подобных ситуаций, но ни один из советов мою проблему не решил. ...


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

Или воспользуйтесь поиском по форуму:
16
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Поддержать - Реклама - Обратная связь
Powered by vBulletin
Copyright ©2000 - 2023, CyberForum.ru