Premiere procedure en C

Version imprimable

Bonjour à tous

Je viens de creer mes premiers boutons dans ma premiere fenetre, et deja la joie m'a extorqué un petit cri de bonheur, qui s'est subreptissement echappé de ma bouche, sans crier gare :D

Grace au super TUTO de MELEM, j'ai compris noir de choses :ccool:

Seulement voila...le bonheur viens de s'arretter a la premiere fonction que je voulais creer :cry:

J'ai voulu essayer d'exporter la creation de fenetre et le BEEP dans chacun une procedure pour voir comment manupuler les procedures :roll:

Et j'obtiens quand meme un exe, mais je n'arrive pas a me débarasser du message d'erreur avec PELLES C :cry:

Mais le plus fou c'est que avec VC6++ j'ai pas d'erreur 8O
Quelqu'un sait pourquoi les deux compilateurs ne reagissent pas pareil ?? :roll:

Citation:

Building D:\Reception\FenetreEtBouton\output\Fenetre.obj.
D:\Reception\FenetreEtBouton\Fenetre.c(82): warning #2027: Missing prototype for 'Bip'.
Building D:\Reception\FenetreEtBouton\FenetreEtBouton.exe.
Done.

Code:

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
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
#include <windows.h>
 
LRESULT CALLBACK WndProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
{
    static HINSTANCE hInstance;
 
    switch (message)
    {
    case WM_CREATE:
 
        hInstance = ((LPCREATESTRUCT)lParam)->hInstance;
        CreateWindow("BUTTON", "OK", WS_CHILD | WS_VISIBLE, 0, 0, 100,24, hwnd, (HMENU)1, hInstance, NULL);
 
        break;
 
    case WM_COMMAND:
        /***************************************************\
        * LOWORD(wParam) = ID du contrôle ou du menu        *
        * HIWORD(wParam) = Raison du message (notification) *
        \***************************************************/
 
        switch(LOWORD(wParam))
        {
 
        case 1:
 
            switch(HIWORD(wParam))
            {
 
            case BN_CLICKED:
                Beep(1000, 100);
                break;
 
            default:
                break;
            }
 
            break;
 
        case 2:
 
            switch(HIWORD(wParam))
            {
 
            case BN_CLICKED:
 
                Beep(100, 100);
                break;
 
            default:
 
                break;
 
            }
 
            break; 
 
        default:
 
            break;
 
        }
 
        break; /* case WM_COMMAND */
 
    case WM_DESTROY:
 
        PostQuitMessage(0);
        break;
 
    default:
 
        return DefWindowProc(hwnd, message, wParam, lParam);
 
    }
 
    return 0;
 
}
 
void Bip(/*paramètres ou non*/)
{
 Beep(100, 100);
}
 
void Principal(HINSTANCE hInstance, int nCmdShow)
{
 
    HWND hWnd;
 
    hWnd = CreateWindow("Classe 1", "Notre première fenêtre", WS_OVERLAPPEDWINDOW,100, 100, 600, 300, NULL,NULL,hInstance,NULL);
    CreateWindow("BUTTON", "OK", WS_CHILD | WS_VISIBLE, 100,100, 100,24, hWnd, (HMENU)1, hInstance, NULL);
    CreateWindow("BUTTON", "OK", WS_CHILD | WS_VISIBLE, 200,200, 100,24, hWnd, (HMENU)2, hInstance, NULL);
    ShowWindow(hWnd, nCmdShow);
    Bip();
 
}
 
int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow)
{
 
    MSG msg;
 
    WNDCLASS wc;
    wc.cbClsExtra     = 0;
    wc.cbWndExtra     = 0;
    wc.hbrBackground  = (HBRUSH)(COLOR_WINDOW + 1);
    wc.hCursor        = LoadCursor(NULL, IDC_ARROW);
    wc.hIcon          = LoadIcon(NULL, IDI_APPLICATION);
    wc.hInstance      = hInstance;
    wc.lpfnWndProc    = WndProc;
    wc.lpszClassName  = "Classe 1";
    wc.lpszMenuName   = NULL;
    wc.style          = CS_HREDRAW | CS_VREDRAW;
 
    RegisterClass(&wc);
 
    Principal(hInstance, nCmdShow);
 
    while (GetMessage(&msg, NULL, 0, 0))
    {
        TranslateMessage(&msg);
        DispatchMessage(&msg);
    }
 
    return (int)msg.wParam;
}

Merci de votre aide

31/10/2010, 00h19
Obsidian

Bonjour,

Citation:

Envoyé par andrebernard

Et j'obtiens quand meme un exe, mais je n'arrive pas a me débarasser du message d'erreur avec PELLES C :cry:

Mais le plus fou c'est que avec VC6++ j'ai pas d'erreur 8O
Quelqu'un sait pourquoi les deux compilateurs ne reagissent pas pareil ?? :roll:

Oui, c'est parce que l'un est configuré par défaut pour être plus strict que l'autre.

Un prototype de fonction, c'est le nom de la fonction avec son type et ses arguments, mais sans le code qui lui est associé. Cela permet au compilateur de savoir comment les appeler même si elles ne sont pas encore disponibles. Et typiquement, ce sont ces prototypes qui, entre autres choses, vont dans les fameux fichiers « *.h ».

En l'occurrence, tu reçois un message d'avertissement parce que tu utilises ta fonction « bip() » alors qu'elle n'est pas définie, ni déclarée. Ton compilo ne la connaît pas encore. Bon, elle n'est certes pas bien loin (quelques lignes plus bas dans le même fichier) mais si tu devais compiler ton programme en une seule passe, cela poserait problème quand même.

Essaie de copier-coller la ligne « void Bip(/*paramètres ou non*/) » en haut de ton fichier juste après les « #include ».
31/10/2010, 00h28
andrebernard

Bonjour OBSIDIAN j'suis super content de te lire :D

C'est un truc de "guedin", tu as raison, une fois ça marche une fois ça marche pas....
Il doit y avoir un sens dans les "Rebuid all", "rebuild" , "Compile" etc ...

D'accord, donc un avertissement n'est pas grave

Bon j'ai essayé comme t'as dit, j'ai mis
void Bip(/*paramètres ou non*/) ;
avant le
#include <windows.h>
et meme apres et c'est pareil :(

Alors ce qui est encore plus fou...c'est que si je fais "Rebuild" dans PELLES C, j'ai l'erreur, et si je fais un "Build" j'ai pas d'erreur 8O
Mais dans les deux cas j'obtiens l'EXE :D
31/10/2010, 00h39
Obsidian

Citation:

Envoyé par andrebernard

Il doit y avoir un sens dans les "Rebuid all", "rebuild" , "Compile" etc ...

La différence entre toutes ces fonctions est que certaines ne vont compiler que ce qui est nécessaire, et d'autres vont tout effacer et reconstruire depuis le départ. Donc, si tu avais des choses déjà compilées en suspens et que les dépendances étaient mal configurées, il est possible que ton compilateur n'ait pas pris tout de suite en compte les modifs que tu as faites.

D'accord, j'ai compris pour les Buid, Rebuild...merci 8-)

Pour essayer de comprendre je suis retourné sur VC6++ et la tout est nickel

Je suis meme arrivé a deplacer la boucle d'attente d'evenement
Et tout ça sans erreur...mais rien a faire avec PELLES C :cry:

Code:

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
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
void Bip(/*paramètres ou non*/);
#include <windows.h>
 
LRESULT CALLBACK WndProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
{
    static HINSTANCE hInstance;
 
    switch (message)
    {
    case WM_CREATE:
 
        hInstance = ((LPCREATESTRUCT)lParam)->hInstance;
        CreateWindow("BUTTON", "OK", WS_CHILD | WS_VISIBLE, 0, 0, 100,24, hwnd, (HMENU)1, hInstance, NULL);
 
        break;
 
    case WM_COMMAND:
        /***************************************************\
        * LOWORD(wParam) = ID du contrôle ou du menu        *
        * HIWORD(wParam) = Raison du message (notification) *
        \***************************************************/
 
        switch(LOWORD(wParam))
        {
 
        case 1:
 
            switch(HIWORD(wParam))
            {
 
            case BN_CLICKED:
                Beep(1000, 100);
                break;
 
            default:
                break;
            }
 
            break;
 
        case 2:
 
            switch(HIWORD(wParam))
            {
 
            case BN_CLICKED:
 
                Beep(100, 100);
                break;
 
            default:
 
                break;
 
            }
 
            break; 
 
        default:
 
            break;
 
        }
 
        break; /* case WM_COMMAND */
 
    case WM_DESTROY:
 
        PostQuitMessage(0);
        break;
 
    default:
 
        return DefWindowProc(hwnd, message, wParam, lParam);
 
    }
 
    return 0;
 
}
 
void Bip(/*paramètres ou non*/)
{
 Beep(100, 100);
}
 
int Principal(HINSTANCE hInstance, int nCmdShow)
{
 
    HWND hWnd;
	MSG msg;
 
    hWnd = CreateWindow("Classe 1", "Notre première fenêtre", WS_OVERLAPPEDWINDOW,100, 100, 600, 300, NULL,NULL,hInstance,NULL);
    CreateWindow("BUTTON", "OK", WS_CHILD | WS_VISIBLE, 100,100, 100,24, hWnd, (HMENU)1, hInstance, NULL);
    CreateWindow("BUTTON", "OK", WS_CHILD | WS_VISIBLE, 200,200, 100,24, hWnd, (HMENU)2, hInstance, NULL);
    ShowWindow(hWnd, nCmdShow);
	Bip();
 
	while (GetMessage(&msg, NULL, 0, 0))
    {
        TranslateMessage(&msg);
        DispatchMessage(&msg);
    }
 
	return (int)msg.wParam;
 
}
 
int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow)
{
 
 
    WNDCLASS wc;
    wc.cbClsExtra     = 0;
    wc.cbWndExtra     = 0;
    wc.hbrBackground  = (HBRUSH)(COLOR_WINDOW + 1);
    wc.hCursor        = LoadCursor(NULL, IDC_ARROW);
    wc.hIcon          = LoadIcon(NULL, IDI_APPLICATION);
    wc.hInstance      = hInstance;
    wc.lpfnWndProc    = WndProc;
    wc.lpszClassName  = "Classe 1";
    wc.lpszMenuName   = NULL;
    wc.style          = CS_HREDRAW | CS_VREDRAW;
 
    RegisterClass(&wc);
 
    return Principal(hInstance, nCmdShow);    
 
}

31/10/2010, 03h37
Obsidian

Citation:

Envoyé par andrebernard

Je suis meme arrivé a deplacer la boucle d'attente d'evenement. Et tout ça sans erreur...mais rien a faire avec PELLES C :cry:

Comment ça, « rien à faire » ? Qu'est-ce que tu obtiens ?

Et bien je n'y comprend rien

Voici le code

Code:

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
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
void Bip(/*paramètres ou non*/);
 
#include <windows.h>
 
LRESULT CALLBACK WndProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
{
    static HINSTANCE hInstance;
 
    switch (message)
    {
    case WM_CREATE:
 
        hInstance = ((LPCREATESTRUCT)lParam)->hInstance;
        CreateWindow("BUTTON", "OK", WS_CHILD | WS_VISIBLE, 0, 0, 100,24, hwnd, (HMENU)1, hInstance, NULL);
 
        break;
 
    case WM_COMMAND:
        /***************************************************\
        * LOWORD(wParam) = ID du contrôle ou du menu        *
        * HIWORD(wParam) = Raison du message (notification) *
        \***************************************************/
 
        switch(LOWORD(wParam))
        {
 
        case 1:
 
            switch(HIWORD(wParam))
            {
 
            case BN_CLICKED:
                Beep(1000, 100);
                break;
 
            default:
                break;
            }
 
            break;
 
        case 2:
 
            switch(HIWORD(wParam))
            {
 
            case BN_CLICKED:
 
                Beep(100, 100);
                break;
 
            default:
 
                break;
 
            }
 
            break; 
 
        default:
 
            break;
 
        }
 
        break; /* case WM_COMMAND */
 
    case WM_DESTROY:
 
        PostQuitMessage(0);
        break;
 
    default:
 
        return DefWindowProc(hwnd, message, wParam, lParam);
 
    }
 
    return 0;
 
}
 
void Bip(/*paramètres ou non*/)
{
 Beep(100, 100);
}
 
int Principal(HINSTANCE hInstance, int nCmdShow)
{
 
    HWND hWnd;
    MSG msg;
    HWND hWndB1;
    HWND hWndB2;
 
    hWnd = CreateWindow("Classe 1", "Notre première fenêtre", WS_OVERLAPPEDWINDOW,100, 100, 600, 300, NULL,NULL,hInstance,NULL);
    hWndB1 = CreateWindow("BUTTON", "OK1", WS_CHILD | WS_VISIBLE, 100,100, 100,24, hWnd, (HMENU)1, hInstance, NULL);
    hWndB2 = CreateWindow("BUTTON", "OK2", WS_CHILD | WS_VISIBLE, 200,200, 100,24, hWnd, (HMENU)2, hInstance, NULL);
    ShowWindow(hWnd, nCmdShow);
   Bip();
 
	while (GetMessage(&msg, NULL, 0, 0))
    {
       TranslateMessage(&msg);
       DispatchMessage(&msg);
    }
 
    return (int)msg.wParam;
 
}
 
int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow)
{
 
 
    WNDCLASS wc;
    wc.cbClsExtra     = 0;
    wc.cbWndExtra     = 0;
    wc.hbrBackground  = (HBRUSH)(COLOR_WINDOW + 1);
    wc.hCursor        = LoadCursor(NULL, IDC_ARROW);
    wc.hIcon          = LoadIcon(NULL, IDI_APPLICATION);
    wc.hInstance      = hInstance;
    wc.lpfnWndProc    = WndProc;
    wc.lpszClassName  = "Classe 1";
    wc.lpszMenuName   = NULL;
    wc.style          = CS_HREDRAW | CS_VREDRAW;
 
    RegisterClass(&wc);
 
    return Principal(hInstance, nCmdShow);
 
}

Quand je BUILD ce code avec PELLES C, j'ai pas de message d'erreur
Quand je REBUILD ce meme code avec PELLES C apres, ou je compile mon fichier "Fenetre.c", j'ai le message d'erreur

Citation:

Reception\FenetreEtBouton\output\Fenetre.obj.
D:\Reception\FenetreEtBouton\Fenetre.c(84): warning #2027: Missing prototype for 'Bip'.
Building D:\Reception\FenetreEtBouton\FenetreEtBouton.exe.
Done.

Et ça marche a tous les coups, si je recommence
Je BUILD a nouveau ce code avec PELLES C, j'ai pas de message d'erreur
Je REBUILD avec PELLES C apres, ou je compile mon fichier "Fenetre.c",j'ai de nouveau le message d'erreur

Donc en un mot, j'ai ce message d'erreur que quand je REBUILD ou je compile mon fichier "Fenetre.c" et seulement avec PELLES C, car avec VC aucun probleme, ni en BUILD, ni REBUILD 8-)
Et pourtant je suis sur que c'est le meme code car je copie/colle tout le code de la "fenetre.c" de l'un a l'autre :roll:

31/10/2010, 12h23
Obsidian

Oui. En plus, c'est moi qui t'ai dit une ânerie. Tu définis bien « bip() » avant de l'utiliser, donc mon prototype en début de fichier est inutile (retire-le).

Qu'est-ce que cela donne si tu retires le commentaire « /*paramètres ou non */ » dans la définition de ta fonction ? En principe, ça ne devrait rien changer non plus, mais sait-on jamais. L'analyseur syntaxique de Pelles C est peut-être un poil trop tatillon.
31/10/2010, 14h15
andrebernard

Oui, j'ai pris cette habitude avec mon langage proche du C
Car cela m'evite d'avoir à declarer les procedures avant de les utiliser.
Mais comme je ne connais pas les habitudes de C, j'ai pensé comme toi que ce pouvait etre ça.

Mais tu reste quand meme le WINNER parce que tu as trouvé le probleme....ou presque. :zoubi:
Tu m'as mis sur la voie, j'ai retiré le commentaire et c'etait encore pareil :?

Alors j'ai mis une grande baffe dans mon ecran.....non j'rigole :mouarf:

J'ai vu dans un tutoriel C que lui il mettait un void entre les parentheses 8O
Alors n'ecoutant que mon courage...j'en ai fait de meme :mrgreen:

Code:

void Bip(void)

Et la ....il est né le divin enfant...ou presque car le miracle s'est réalisé sous mes yeux ebahis 8O plus d'erreur 8-)

T'as raison ce PELLES C il est un peu trop chatouilleux :?
En fin de compte...je crois que je prefere de loin VC6....car j'ai vu que l'on pouvais debugger ligne a ligne, et voir la valeur des variables en cours :ccool:, ce que l'on ne peut pas faire avec PELLES car lui il m'ouvre une page ASM a chaque fois :?
La seule chose qu'est moins bien dans VC6 c'est qu'on peut pas "plier" les fonctions au "{" :(

Je suis aussi en train d'essayer un C-Free qui a l'air pas mal non plus en version PRO :roll:

T'es trop fort OBSIDIAN :ccool:
On fait une fine equipe tous les deux (enfin surtout toi :oops:) on a vaincu l'ennemi :lol:

Je te remercie beaucoup et te souhaite un excelent dimanche ;)