Discussion :

[nanosoft]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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);

[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.

[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 :

Sélectionner tout - Visualiser dans une fenêtre à part

Ofn.lpstrDefExt = "bmp";

[10_GOTO_10]

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

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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).

[Médinoc]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[nanosoft]

Merci à tous les 2, cela fonctionne :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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);

[10_GOTO_10]

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 :

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 :

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

En cherchant d'autres sources

This is the correct code:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[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).