Utilisation de GetSaveFileName()

Version imprimable

Bonjour,

Je programme avec l'API Windows et SDL.
J'utilise GetOpenFileName() sans souci, mais j'ai un problème avec GetSaveFileName().

Dans le code ci-dessous (aucun warning) tout se déroule sans problème mais la boîte de sélection de fichiers ne s'ouvre pas.
Le programme continue sans que rien ne se passe.:?

Auriez-vous une idée du problème ?

Merci d'avance.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 OPENFILENAME Ofn; char szFile[MAX_PATH]; ZeroMemory(szFile, MAX_PATH); char fileTitle[MAX_PATH]; ZeroMemory(fileTitle, MAX_PATH); Ofn.lStructSize = sizeof(Ofn); memset(&Ofn, 0, sizeof(Ofn)); ZeroMemory(&Ofn, sizeof(Ofn)); Ofn.hwndOwner = NULL; Ofn.lpstrFilter = "Image File\0*.bmp\0\0"; Ofn.lpstrFile=szFile; //Ofn.nMaxFile = sizeof(szFile)/ sizeof(*szFile); Ofn.nMaxFile = MAX_PATH; Ofn.lpstrFileTitle = fileTitle; Ofn.nMaxFileTitle = sizeof(fileTitle); Ofn.lpstrInitialDir = "C:\\Users\\Alain\\Pictures"; //Ofn.Flags = OFN_SHOWHELP | OFN_OVERWRITEPROMPT; Ofn.Flags = OFN_EXPLORER | OFN_FILEMUSTEXIST | OFN_HIDEREADONLY; Ofn.lpstrDefExt = (LPSTR)L"bmp"; Ofn.lpstrTitle = "Save .bmp"; GetSaveFileName(&Ofn);

27/04/2015, 11h10
Médinoc

Quelle est la valeur de retour de GetSaveFileName(), quelle est la valeur de GetLastError()/CommDlgExtendedError()?

(aussi, je ne suis pas sûr, mais il est possible qu'il faille appeler CoInitialize() avant usage)

Aussi, à cette ligne: Ofn.lpstrDefExt = (LPSTR)L"bmp"; le cast est hautement suspect.
27/04/2015, 20h42
nanosoft

Merci pour ces pistes Médinoc, voici ce que j'obtiens:

GetSaveFileName = 0
GetLastError = 87 (The parameter is incorrect.)
CommDlgExtendedError = 1 (The lStructSize member of the initialization structure for the corresponding common dialog box is invalid.)

Ceci que j'appelle ou non "CoInitialize(NULL);" avant.

Cela ne change rien avec :

Code:

Ofn.lpstrDefExt = "bmp";
27/04/2015, 22h26
10_GOTO_10
Citation:

Envoyé par nanosoft

Auriez-vous une idée du problème ?

Mon idée, c'est que c'est le bazar.
Code:

1 2 3 4 Ofn.lStructSize = sizeof(Ofn); memset(&Ofn, 0, sizeof(Ofn)); ZeroMemory(&Ofn, sizeof(Ofn));
Après avoir mis deux fois la structure Ofn à zéro, penses-tu vraiment que la valeur Ofn.lStructSize est conservée ? Mon avis est plutôt qu'elle est assez vraisemblablement à zéro.

Code:

Ofn.lpstrDefExt = (LPSTR)L"bmp";

Tu mélanges allègrement les chaînes de caractères UNICODE et chaînes de char, tu les cast, ça ne peut que pas marcher.

Code:

Ofn.lpstrFilter = "Image File\0*.bmp\0\0"

Le deuxième "\0" est inutile (il est automatiquement ajouté en fin de chaîne littérale).
28/04/2015, 08h26
Médinoc

Citation:

Envoyé par 10_GOTO_10

Code:

Ofn.lpstrFilter = "Image File\0*.bmp\0\0"

Le deuxième "\0" est inutile (il est automatiquement ajouté en fin de chaîne littérale).

ATTENTION CETTE PARTIE DE TA RÉPONSE EST FAUSSE: ~~lpstrFilter est une liste de paires de chaînes, terminée par deux chaînes vides (et donc, trois caractères nuls consécutifs, dont deux explicites). La chaîne spécifiée est correcte.~~

Merci à tous les 2, cela fonctionne : :P

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
    OPENFILENAME Ofn;
    char szFile[MAX_PATH];
    ZeroMemory(szFile, MAX_PATH);
    char fileTitle[MAX_PATH];
    ZeroMemory(fileTitle, MAX_PATH);
 
    memset(&Ofn, 0, sizeof(Ofn));
    Ofn.lStructSize = sizeof(OPENFILENAME);
    Ofn.hwndOwner = NULL;
    Ofn.lpstrFilter = "Image File\0*.bmp\0\0";
    Ofn.lpstrFile=szFile;
    Ofn.nMaxFile = MAX_PATH;
    Ofn.lpstrFileTitle = fileTitle;
    Ofn.nMaxFileTitle = sizeof(fileTitle);
    Ofn.lpstrInitialDir = "C:\\Users\\Alain\\Pictures";
    Ofn.Flags = OFN_EXPLORER | OFN_FILEMUSTEXIST | OFN_HIDEREADONLY;
    Ofn.lpstrDefExt = "bmp";
    Ofn.lpstrTitle = "Save .bmp";
 
    int retour_GSFN = GetSaveFileName(&Ofn);

28/04/2015, 20h44
10_GOTO_10

Citation:

Envoyé par Médinoc

ATTENTION CETTE PARTIE DE TA RÉPONSE EST FAUSSE: lpstrFilter est une liste de paires de chaînes, terminée par deux chaînes vides (et donc, trois caractères nuls consécutifs, dont deux explicites). La chaîne spécifiée est correcte.

Mmmmmhh... Pas convaincu.

Si on regarde la doc MSDN de OPENFILENAME on y lit :

Citation:

The last string in the buffer must be terminated by two NULL characters.

Deux, donc pas trois... Mais juste en dessous il y a un exemple, et ils en mettent deux explicites, plus un implicite :

Citation:

To retrieve a shortcut's target without filtering, use the string "All Files\0*.*\0\0"

En cherchant d'autres sources

Citation:

This is the correct code:

Code:

lofn.lpstrFilter = L"Equalizer Preset (*.feq)\0*.feq\0";

Le mieux, c'est d'essayer. J'ai donc fait le test avec juste deux zéros et du texte après:

Code:

Ofn.lpstrFilter = "Image File\0*.bmp\0\0*.*\0\0";

Et je n'ai bien qu'une seule ligne de filtre. Je suis donc assez convaincu que deux \0 suffisent (un explicite et un implicite).

Après, dans la mesure où la doc n'est pas très claire, je te conseille quand même d'en mettre explicitement deux. Le code est plus lisible et en général on n'est pas à un octet gaspillé près.
29/04/2015, 06h45
Médinoc

En effet, ce n'est pas aussi clair que je le croyais, je me demande pourquoi j'en étais aussi certain (je me suis probablement brûlé dessus une fois).