VPF::GetSaveFileName() - Форум программистов

Belous

Дата 31.1.2017, 15:47 (ссылка)

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

Загрузка ...

Новичок

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

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

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

Код



namespace e
{
    char *filename;
}

// ...
        case IDM_SAVE_AS:
            e::filename = NULL;
            FormSaveAsFile(hwnd, e::filename);
            MessageBox(hwnd, e::filename, "Message", MB_OK);
            break;
// ...

void FormSaveAsFile(HWND hwnd, char *fname)
{
    OPENFILENAME ofn;
    char filename[MAX_PATH];
    filename[0] = 0;
    memset(&ofn, 0, sizeof(OPENFILENAME));
    ofn.lStructSize = sizeof(OPENFILENAME);
    ofn.hwndOwner = hwnd;
    ofn.lpstrFilter = "Text files (*.txt)\0*.txt\0";
    ofn.lpstrFile = filename;
    ofn.nMaxFile = sizeof(filename);
    ofn.Flags = OFN_FILEMUSTEXIST | OFN_HIDEREADONLY;
    ofn.lpstrDefExt = "txt";
    if (GetSaveFileName(&ofn))
    {
        strcpy(fname, ofn.lpstrFile);
        MessageBox(hwnd, "SAVE_FILE OK", "Message", MB_OK);
    }
    else
    {
        strcpy(fname, ofn.lpstrFile);
        MessageBox(hwnd, "SAVE_FILE Cancel", "Message", MB_OK);
    }
}

Добавлено @ 15:51
В этой строке ошибка при выполнении программы

Код


strcpy(fname, ofn.lpstrFile);

Unhandled exception at 0x004021a2 in Scheme.exe: 0xC0000005: Access violation writing location 0x00000000.

Это сообщение отредактировал(а) Belous - 31.1.2017, 16:46

Belous

Дата 31.1.2017, 16:45 (ссылка)

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

Загрузка ...

Новичок

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

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

Нашел решение:

вместо

Код


e::filename = NULL;

нужно

Код


e::filename = new char[MAX_PATH];

xvr

Дата 1.2.2017, 11:21 (ссылка)

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

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 7046
Регистрация: 28.8.2007
Где: Дублин, Ирландия

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

И откуда такое повальное увлечение - втыкать везде (к месту и не к месту) new/malloc/GlobalAlloc и пр? smile

Так не проще, а ?

Код


namespace e
{
    char filename[MAX_PATH];
}

borisbn

Дата 1.2.2017, 16:56 (ссылка)

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

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 4875
Регистрация: 6.2.2010
Где: Ростов-на-Дону

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

Если у Вас не Си, а Си++ (а судя по namespace и new у Вас именно он), то настоятельно рекомендую использовать std::string:

Код


#include <string>
// ...
namespace e
{
    std::string filename;
}
// ...
void FormSaveAsFile(HWND hwnd, std::string & fname)
{
// ...

а вместо

Код


strcpy(fname, ofn.lpstrFile);

просто копировать:

Код


fname = ofn.lpstrFile;

Повторю, это только, если у Вас не Си.

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

Женщины отличаются от программистов тем, что у них чары состоят из стрингов

Belous

Дата 6.2.2017, 21:35 (ссылка)

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

Загрузка ...

Новичок

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

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

Цитата

namespace e
{
char filename[MAX_PATH];
}

xvr
Действительно проще, к этому я и пришел, методом проб и ошибок.

Цитата

настоятельно рекомендую использовать std::string

borisbn
Мне уже даже начало нравиться работать не со строками, а с массивами символов.

Всем спасибо за советы.

Это сообщение отредактировал(а) Belous - 6.2.2017, 21:36

1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Системное программирование и WinAPI \| Следующая тема »