проверить CFile

ValeryS · Регистрация: 14.02.2011

Author24 — интернет-сервис помощи студентам

у меня VS 2008 русская профессиональная
прошу проверить вот такой тестовый код на других версиях

C++

CFileDialog dlg(TRUE);
if(dlg.DoModal()==IDOK)
{
CString str=dlg.GetPathName();
CFile fl(str,CFile::typeText);
 
 
}

все дело в том что у меня эта конструкция падает во время исполнения
при падении вышел на такой код

C++

1	ASSERT((nOpenFlags & typeText) == 0); // text mode not supported

в файле filecore.cpp

в MFC вообще отказались от поддержки текстового режима CFile или только в моей версии?

@~~-=ЮрА=-~~ · 22.12.2012, 11:10

ValeryS, cуть проблемы вот в чём, в MSVS 2008 задачи выделения памяти, преобразования типов в основном возложены на программиста. Потому крах твоей программы происходил от того что под CFileDialog::m_ofn.lpstrFile не выделял память/либо не присваивал константной строковой переменной.
Посмотри этот проект, также как бонус обрати внимание на тело конструктора MFCOpenSaveDlg верней переменную OPENFILENAME и посмотри инициализацию её полей в BOOL CMFCOpenSaveDlg::OnInitDialog(),
как ты понял я подал ещё и код с диалогами открытия/сохранения в стиле API (причём будет тот же CFileDialog) мне как то GetOpen(Save)FileName больше нравятся, т.к можно не заморачваться на класс CFileDialog а просто передавать поля структуры в отдельные модули, причём одну и ту же структуру для обоих диалогов (очень функционально и компактно на мой взгляд)

C++

void CMFCOpenSaveDlg::OnOpenCfile() 
{
    // TODO: Add your control notification handler code here
    CFileDialog fDlg(true);//диалог выбора файла
    ////Вот самое главное в отличие от 6-ки надо выделить память///
    fDlg.m_ofn.lpstrFile = m_Path.GetBuffer(MAX_PATH);
    ///////////////////////////////////////////////////////////////
    fDlg.m_ofn.lpstrFilter="(*.*) Все файлы\0*.*\0";
    fDlg.m_ofn.lpstrTitle ="Выбор файла";
    if(fDlg.DoModal() == IDOK)
        m_Path = fDlg.m_ofn.lpstrFile;
    else
        m_Path = "Выбор файла отменён";
    UpdateData(false);
}
 
void CMFCOpenSaveDlg::OnSaveCfile() 
{
    // TODO: Add your control notification handler code here
    CFileDialog fDlg(false);//диалог сохранения файла
    ////Вот самое главное в отличие от 6-ки надо выделить память///
    fDlg.m_ofn.lpstrFile = m_Path.GetBuffer(MAX_PATH);
    ///////////////////////////////////////////////////////////////
    fDlg.m_ofn.lpstrFilter="(*.*) Все файлы\0*.*\0";
    fDlg.m_ofn.lpstrTitle ="Сохранение файла";
    if(fDlg.DoModal() == IDOK)
        m_Path = fDlg.m_ofn.lpstrFile;
    else
        m_Path = "Выбор файла отменён";
    UpdateData(false);
}
 
void CMFCOpenSaveDlg::OnOpenOfname() 
{
    // TODO: Add your control notification handler code here
    pFile.lpstrTitle= "Выбор файла";
    if(GetOpenFileName(&pFile))//Диалог выбора файла
        m_Path = pFile.lpstrFile;
    else
        m_Path = "Выбор файла отменён";
    UpdateData(false);
}
 
void CMFCOpenSaveDlg::OnSaveOfname() 
{
    // TODO: Add your control notification handler code here
    pFile.lpstrTitle= "Сохранение файла";
    if(GetSaveFileName(&pFile))//Диалог сохранения файла
        m_Path = pFile.lpstrFile;
    else
        m_Path = "Выбор файла отменён";
    UpdateData(false);
}

@~~-=ЮрА=-~~ · 22.12.2012, 11:14

PS: ValeryS, это пашет только для Release

Сообщение от ValeryS

CFile fl(str,CFile::typeText);

в Debug тоже всплыл Exception

@~~-=ЮрА=-~~ · 22.12.2012, 11:20

Доработай 1-ую функцю проекта

C++

void CMFCOpenSaveDlg::OnOpenCfile() 
{
    // TODO: Add your control notification handler code here
    CFile pFile;
    CString szFmt;
    CFileDialog fDlg(true);//диалог выбора файла
    ////Вот самое главное в отличие от 6-ки надо выделить память///
    fDlg.m_ofn.lpstrFile = m_Path.GetBuffer(MAX_PATH);
    ///////////////////////////////////////////////////////////////
    fDlg.m_ofn.lpstrFilter="(*.*) Все файлы\0*.*\0";
    fDlg.m_ofn.lpstrTitle ="Выбор файла";
    if(fDlg.DoModal() == IDOK)
    {
        pFile.Open(m_Path = fDlg.m_ofn.lpstrFile, CFile::typeText);
        szFmt.Format("File Length : %d bytes", pFile.GetLength());
        pFile.Close();
        AfxMessageBox(szFmt);
    }
    else
        m_Path = "Выбор файла отменён";
    UpdateData(false);
}

и посмотри Debug/Release (см скрины)

@~~-=ЮрА=-~~ · 22.12.2012, 11:21

Не по теме:

В заключение :ИМХО CStdioFile на много лучше CFile...

ValeryS · 22.12.2012, 11:39 **[ТС]**

Сообщение от -=ЮрА=-

PS: ValeryS, это пашет только для Release

разумеется в Release ASSERT выбрасывается
но ведь зачем то он там стоит?

Сообщение от -=ЮрА=-

Потому крах твоей программы происходил от того что под CFileDialog::m_ofn.lpstrFile не выделял память/либо не присваивал константной строковой переменной.

нет здесь все в порядке

Сообщение от -=ЮрА=-

как ты понял я подал ещё и код с диалогами открытия/сохранения в стиле API

диалог я дал только для того чтобы не привязываться к конкретным файлам
если написать так

C++

CFileDialog dlg(TRUE);
if(dlg.DoModal()==IDOK)
{
CString str=dlg.GetPathName();
CFile fl(str);
 }

то все в порядке
проблема в CFile::typeText

началось все с того что я решил открыть текстовый файл в CFile,
не получилось, плюнул, сделал через fopen fread, но осадочек то остался

насколько я помню в шестерке это работало(правдо давно я с ней не работаю)
сейчас скачал 2012, поставил на виртуалку попробую с ней

Добавлено через 7 минут

Сообщение от -=ЮрА=-

посмотри Debug/Release

я правильно понял, что шестерка тоже выбрасывает?

Сообщение от -=ЮрА=-

:ИМХО CStdioFile на много лучше CFile...

не знаю не работал
я вообще CFile и прочее пытаюсь не использовать, посему в этом классе не силен

@~~-=ЮрА=-~~ · 22.12.2012, 11:40

Сообщение от ValeryS

насколько я помню в шестерке это работало(правдо давно я с ней не работаю)

- нет в Debug она тоже светит исключение. Вобщем typeText как бы для Release, хмм страно, по моему там какого-то флага CFile не хватает, ща погляжу я вначале не придал этому значения

ValeryS · 22.12.2012, 11:44 **[ТС]**

Сообщение от -=ЮрА=-

Вобщем typeText как бы для Release, хмм страно,

понимаешь он может просто игнорировать этот флаг
насколько я помню разница между текстовым и бинарным режимом в обработке 0x0D 0x0A в бинарном читает 2 байта в текстовом заменяет на один перевод строки (а ну еще ^Z конец файла)
посему логика может нарушится

@~~-=ЮрА=-~~ · 22.12.2012, 11:45

ValeryS, http://msdn.microsoft.com/en-u... s.80).aspx
CFile::typeText Sets text mode with special processing for carriage return–linefeed pairs (used in derived classes only). CFile в автомате(как скажем i(o)fstream же стоит в текстовом режиме, на двоичный сбрасываем typeBinary а для текстового ничего не надо).

ValeryS · 22.12.2012, 11:55 **[ТС]**

ща попробую открыть CFile c typeBinary и без (то же сделаю с CStdioFile)
и сравню буфера
о результатах сообщу дополнительно

@~~-=ЮрА=-~~ · 22.12.2012, 12:07

ValeryS, делай typeBinaryпри таком режиме перевод на новую строку буде \r\n т.е 2 символа вместо привычного \n

@~~-=ЮрА=-~~ Заблокирован
	22.12.2012, 11:14	3
	PS: ValeryS, это пашет только для Release Сообщение от ValeryS CFile fl(str,CFile::typeText); в Debug тоже всплыл Exception 0

@~~-=ЮрА=-~~
	22.12.2012, 11:21 #5
	Не по теме: В заключение :ИМХО CStdioFile на много лучше CFile... 0

@~~-=ЮрА=-~~ Заблокирован
	22.12.2012, 11:40	7
	Сообщение от ValeryS насколько я помню в шестерке это работало(правдо давно я с ней не работаю) - нет в Debug она тоже светит исключение. Вобщем typeText как бы для Release, хмм страно, по моему там какого-то флага CFile не хватает, ща погляжу я вначале не придал этому значения 0

ValeryS Модератор 8950 / 6716 / 921 Регистрация: 14.02.2011 Сообщений: 23,699
	22.12.2012, 11:44 [ТС]	8
	Сообщение от -=ЮрА=- Вобщем typeText как бы для Release, хмм страно, понимаешь он может просто игнорировать этот флаг насколько я помню разница между текстовым и бинарным режимом в обработке 0x0D 0x0A в бинарном читает 2 байта в текстовом заменяет на один перевод строки (а ну еще ^Z конец файла) посему логика может нарушится 0

@~~-=ЮрА=-~~ Заблокирован
	22.12.2012, 11:45	9
	ValeryS, http://msdn.microsoft.com/en-u... s.80).aspx CFile::typeText Sets text mode with special processing for carriage return–linefeed pairs (used in derived classes only). CFile в автомате(как скажем i(o)fstream же стоит в текстовом режиме, на двоичный сбрасываем typeBinary а для текстового ничего не надо). 0

ValeryS Модератор 8950 / 6716 / 921 Регистрация: 14.02.2011 Сообщений: 23,699
	22.12.2012, 11:55 [ТС]	10
	ща попробую открыть CFile c typeBinary и без (то же сделаю с CStdioFile) и сравню буфера о результатах сообщу дополнительно 0

@~~-=ЮрА=-~~ Заблокирован
	22.12.2012, 12:07	11
	ValeryS, делай typeBinaryпри таком режиме перевод на новую строку буде \r\n т.е 2 символа вместо привычного \n 0