Discussion :

[PilloBuenaGente]

Bonjour à tous !

Je m'entraine sur un petit programme windows en mode console.

Tout roule(couleurs et hauteur), sauf que l'affichage que je veux fixe et sans croix pour quitter, parfois, s'ouvre de manière standard (position standard et croix accessible). Cela viens sans doute du constructeur.

Ceci est le .cpp de mon affichage.
(Il ne contient que 2 membres : int font, height; qui sont les couleurs du texte et celle du background, et leurs fonctions pour les modifiers (inline))

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
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
#include "Display.h"
 
#include <string>
#include <sstream>
#include <Windows.h>
 
Display::Display() : font(0), text(7)
{
    SetConsoleTitle("Job Manager");//Nom de la console
 
    HWND window = FindWindow(NULL, "Job Manager");
 
    MoveWindow(window,200, 1, 1, 1, 1);//Positionner la console
 
    HMENU menu = GetSystemMenu(window, false);//Enlever la croix
    RemoveMenu(menu, SC_CLOSE, 0);//Aussi enlever la croix
 
    Colorize();//Mettre couleurs par défaut
    Height(25);//Mettre taille par défaut
}
Display::~Display()
{}
void Display::Height(const int height)
{
    std::stringstream sS;
    sS << "mode con cols=80lines=" << height;//Créer la ligne pour changement de hauteur
    std::string s = sS.str();
    const char *entry = s.c_str();
    system(entry);//Changer la hauteur
}
void Display::Colorize()
{
    Colorize(font, text);
}
void Display::Colorize(const int font, const int text)
{
    HANDLE screen = GetStdHandle(STD_OUTPUT_HANDLE);
    SetConsoleTextAttribute(screen, (font * 16) + text);//Colorier
}

Je ne comprends pas ce résultat hasardeux. (je ne comprends pas tout à fait non plus toutes les commandes du constructeur... :p).

Quelqu'un aurait-il une idée ?
Merci pour votre attention !

[Médinoc]

Bonjour,
Déjà, un peu de gestion d'erreur ferait du bien (FindWindow(), etc.).

Ensuite, tu peux utiliser GetConsoleWindow() au lieu de FindWindow(), ça devrait être plus fiable (surtout si tu ouvres deux fois le programme).

[PilloBuenaGente]

Merci !!!!!
En effet la gestion des erreurs peut être utile, je débute et je me doute bien qu'il faut faire des retours mais je ne vois pas bien la meilleur solution.

J'ai tester ce "FindWindow" et répète l'assignation si cela n'a pas été fait.

Code :

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

1
2
3
4
5
6
 
HWND window = FindWindow(NULL, title);
while(!window)
    window = FindWindow(NULL, title);
 
MoveWindow(window, 200, 1, 1, 1, 1);////Marche à tout les coups !!!

Bien des heures passées.... C'est cool ! Bonne soirée !

[Neckara]

Bonjour,

Pour éviter la duplication de code, plutôt que de faire :

Code :

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

1
2
3
4
5
6
 
HWND window = FindWindow(NULL, title);
while(!window)
    window = FindWindow(NULL, title);
 
MoveWindow(window, 200, 1, 1, 1, 1);////Marche à tout les coups !!!

Tu peux faire :

Code :

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

1
2
3
4
5
6
7
HWND window;
do
{
      window = FindWindow(NULL, title);
} while( ! window )
 
MoveWindow(window, 200, 1, 1, 1, 1);////Marche à tout les coups !!!

Sinon, je ne comprend pas l'intérêt de la boucle, en effet :
- si la fenêtre n'a pas été trouvée, pourquoi serait-elle trouvée au tour suivant ?
- on a un gros risque de boucle infinie.

Je pense que tu devrais mettre un compteur dans ta boucle et afficher la valeur de ton compteur à la sortie de ta boucle pour voir combien de tour tu as réellement fait et voir si cette boucle est réellement utile.

[Médinoc]

Cette boucle est horrible.
De plus, comme je l'ai dit, si tu as deux fenêtres avec le même titre...

Utilise GetConsoleWindow() à la place bon sang!

[PilloBuenaGente]

Utilise GetConsoleWindow() à la place bon sang!

"GetConsoleWindow()" ne marche pas chez moi (pas reconnue), quelle diplomatie! Serais-tu prof ? ;p

- si la fenêtre n'a pas été trouvée, pourquoi serait-elle trouvée au tour suivant ?

C'est un problème que je ne comprends pas. Sur internet j'ai tout cherché "sometimes hwnd movewindow",à part "GetConsoleWindow()"....
De plus je n'ai pas trop envie de me prendre la tête à apprendre leur systeme windows d'HANDLE et tout, je fais ca pour m'entrainer au code.

- on a un gros risque de boucle infinie.

J'ai testé la boucle, les resultats sont les suivants : 1, 6, 10, 7, 1, 5, 2, 2, 3.....LOL j'ai envie de dire ...

du coup, au cas où :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
    int nbr = 0;
    HWND window;
 
    do
    {
        window = FindWindow(NULL, title);
        nbr++;
    }while(!window && nbr < 100);
 
    if(nbr < 100)
        MoveWindow(window, 200, 1, 1, 1, 1);
 
    HMENU menu = GetSystemMenu(window, false);

Merci, bonne journée !

[Médinoc]

GetConsoleWindow() est disponible dès Windows 2000, donc si tu ne "l'as" pas, c'est que tu as mal configuré tes headers ou que tu utilises un vieux Visual 6.