VPF::работа со спроецированным файлом

feodorv

Дата 18.5.2012, 10:02 (ссылка)

(голосов:1)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 2214
Регистрация: 30.7.2011

Репутация: 9
Всего: 45

Цитата(GremlinProg @ 18.5.2012, 08:25

)

эта ошибка системная, очень распространенная

Сам её совершал неоднократно)))

Цитата(GremlinProg @ 18.5.2012, 08:25

)

выход за пределы блока - точно есть

Я имел в виду, что в коде записи в память нет, всё рид-онли smile

А так-то))))

Цитата(GremlinProg @ 18.5.2012, 08:25

)

в WordCount передается указатель на 1 символ, а читается fSize,

Мда. Казалось бы, начало неплохое, если CreateFile, GetFileSize, CreateFileMapping и MapViewOfFile отрабатывают без ошибок (что не очевидно):

Цитата

    case WM_CREATE:
  hFile = CreateFile(_T("My.txt"), GENERIC_READ, 0, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL);
  if(hFile != INVALID_HANDLE_VALUE)
  {
    fSize = GetFileSize(hFile, NULL);
    hFileMap = CreateFileMapping(hFile, NULL, PAGE_READONLY, 0, fSize, NULL);
    if(hFileMap)
    p = (TCHAR*)MapViewOfFile(hFileMap, FILE_MAP_READ, 0, 0, 0);
    break;
  }
  countW = WordCount(p, fSize);
  countT = fSize;

Остро нужен контроль ошибок.
Кроме того, если ни hFile, ни hFileMap (кроме как в высвобождении ресурсов) более не используются, почему бы их здесь же не освободить (и забыть про них)?
Кроме того:

Цитата

unsigned int countW, countT;

как и p тоже не static.

Однако, в WM_PAINT...

Цитата

_stprintf(str, _T("Word count = %i\n Simbols count = %i\n"), WordCount(p, SimbolsCount(fSize)), SimbolsCount(fSize));

При этом:

Цитата(wallstreet @ 16.5.2012, 12:54

)

отрисовывает правильный результат

Цитата(wallstreet @ 14.5.2012, 19:01

)

в объявлении:

1: TCHAR *p = new TCHAR;

помогло.

Жуть... Ну и лекарство...

Такой код

Цитата

TCHAR *p = new TCHAR;
...
UnmapViewOfFile(p);

разрушить кучу вполне способен...

--------------------

Напильник, велосипед, грабли и костыли - основные инструменты программиста...

GremlinProg

Дата 18.5.2012, 10:18 (ссылка)

(голосов:1)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 2706
Регистрация: 9.8.2005
Где: Тюмень

Репутация: 99
Всего: 106

Цитата(feodorv @ 18.5.2012, 12:02)

Такой код

Цитата

TCHAR *p = new TCHAR;
...
UnmapViewOfFile(p);

разрушить кучу вполне способен...

ну, в данном случае, куча портится раньше вызова UnmapViewOfFile,
так что все равно не понятно, где она успела это сделать

--------------------

"Гений всегда разумнее, чем умнее. Ум — это машина, разум — водитель этой машины."

feodorv

Дата 18.5.2012, 10:38 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 2214
Регистрация: 30.7.2011

Репутация: 9
Всего: 45

Цитата(GremlinProg @ 18.5.2012, 11:18

)

так что все равно не понятно, где она успела это сделать

Тогда ждём от wallstreet исправлений и дальнейших приключений smile

--------------------

Напильник, велосипед, грабли и костыли - основные инструменты программиста...

wallstreet

Дата 21.5.2012, 15:22 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 153
Регистрация: 11.8.2011

Репутация: нет
Всего: нет

Пока пришел вот к этому результату и дальше сдвинуться не могу никак.

Код


int WordCount(TCHAR *pStr, unsigned int fSize)
{
    unsigned count = 0;
    int prev_alpha = 0;
    for(unsigned i=0; i<fSize; ++i)
    {
        int now_alpha = _istalnum(pStr[i]);
        count +=now_alpha && !prev_alpha;
        prev_alpha = now_alpha;
    }
    return count;
LRESULT CALLBACK WndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
{
    int wmId, wmEvent;
    PAINTSTRUCT ps;
    HDC hdc;
    static HANDLE hFile, hFileMap;
    static DWORD fSize;
    static RECT rt;
    static TCHAR *str = new TCHAR;
    static TCHAR *p = 0;


    switch (message)
    {
    case WM_CREATE:
        hFile = CreateFile(_T("My.txt"), GENERIC_READ, 0, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL);
        if(hFile != INVALID_HANDLE_VALUE)
        {
            fSize = GetFileSize(hFile, NULL);
            hFileMap = CreateFileMapping(hFile, NULL, PAGE_READONLY, 0, fSize, NULL);
            if(hFileMap)
            {
                CloseHandle(hFile);
                p = (TCHAR*)MapViewOfFile(hFileMap, FILE_MAP_READ, 0, 0, 0);
                CloseHandle(hFileMap);
            }
            break;
        }
        break;
    case WM_COMMAND:
        wmId    = LOWORD(wParam);
        wmEvent = HIWORD(wParam);
        // Parse the menu selections:
        switch (wmId)
        {
        case IDM_ABOUT:
            DialogBox(hInst, MAKEINTRESOURCE(IDD_ABOUTBOX), hWnd, About);
            break;
        case IDM_EXIT:
            DestroyWindow(hWnd);
            break;
        default:
            return DefWindowProc(hWnd, message, wParam, lParam);
        }
        break;
    case WM_PAINT:
        GetClientRect(hWnd, &rt);
        hdc = BeginPaint(hWnd, &ps);
        if(*str)
        {
            _stprintf(str, _T("Word count = %i\nSimbols count = %i\n"), WordCount(p, fSize), fSize);
            DrawText(hdc, str, _tcslen(str), &rt, DT_LEFT);
        }
        EndPaint(hWnd, &ps);
        break;
    case WM_DESTROY:
        if(p!=NULL)
            UnmapViewOfFile(p);
        if(str!=NULL)
            delete str;
        PostQuitMessage(0);
        break;
    default:
        return DefWindowProc(hWnd, message, wParam, lParam);
    }
    return 0;
}

Прошу не судить строго и намекать потолще на суть ошибки.
Буду стараться все быстрее исправить.
Кстати сейчас уже ошибка выполнения выскакивает при попытке закрыть окно.
суть:

Цитата

First-chance exception at 0x77942a80 in HomeWork6.9.exe: 0xC0000005: Access violation reading location 0xdddddde1.
The program '[5648] HomeWork6.9.exe: Native' has exited with code 0 (0x0).

указывает на:

Код


return DefWindowProc(hWnd, message, wParam, lParam);

в разделе обработки сообщения WM_DESTROY

ссылка для скачивания проекта

feodorv

Дата 21.5.2012, 16:00 (ссылка)

(голосов:1)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 2214
Регистрация: 30.7.2011

Репутация: 9
Всего: 45