Discussion :

[Taurnil]

Bonjour à tous, j'ai un petit problème au niveau d'un code que j'ai bricolé avec divers bout de programmes que j'ai récupéré en ligne ou dans des livres...

Le programme est en MFC Wizard et est sensé piloter mon port série en fonction de l'heure ( en envoyant 255 octets sur le fil 3 de mon DB9 ).

Après il y a pas mal de code qui sert à commander les quelques boutons de ma fenêtre.

Je compile et il m'indique 0 erreurs, 0 warnings mais ma fenêtre ne s'ouvre pas. C'est depuis que j'ai ajouté le code du port série, avant ma fenêtre s'ouvrait sans problème.

Peut être que la partie de commande du port série, c'est du grand n'importe quoi et à ce moment là, il faudrait me filer un coup de main. En effet, c'est ma première année de programmation et je suis en stage dans une entreprise ou il n'y a personne de calé sur le sujet. Si quelqu'un pouvait me dire ce qui ne va pas, je lui en serait (vraiment très) reconnaissant.

Merci beaucoup d'avance.

Je vous ai mis le code ci-dessous, j'ai essayé de mettre quelques commentaires, j'espère que ça vous aidera.
Je l'ai mis en "brut" (c'est pas très organisé je vous l'accorde) donc un copier coller et on peut compiler.

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
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
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
 
// Fenetre 1Dlg.cpp : implementation file
//
#include <bios.h>
#include "stdafx.h"
#include "Fenetre 1.h"
#include "Fenetre 1Dlg.h"
#include <iostream.h>
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#include <string.h>
#include <windows.h>
#include <conio.h>
#ifdef _DEBUG
#define new DEBUG_NEW
#undef THIS_FILE
static char THIS_FILE[] = __FILE__;
#endif
 
#define RX_SIZE 4096 /* taille tampon d'entrée */
#define TX_SIZE 4096 /* taille tampon de sortie */
#define MAX_WAIT_READ 5000 /* temps max d'attente pour lecture (en ms) */
BOOL OpenCOM    (int nId);
BOOL CloseCOM   ();
BOOL WriteCOM   (void* buffer, int nBytesToWrite, int* pBytesWritten);
HANDLE g_hCOM = NULL;
COMMTIMEOUTS g_cto =
{
MAX_WAIT_READ, /* ReadIntervalTimeOut */
0, /* ReadTotalTimeOutMultiplier */
MAX_WAIT_READ, /* ReadTotalTimeOutConstant */
0, /* WriteTotalTimeOutMultiplier */
0 /* WriteTotalTimeOutConstant */
};
DCB g_dcb =
{
    sizeof(DCB),        /* DCBlength            */
    9600,               /* BaudRate             */
    TRUE,               /* fBinary              */
    FALSE,              /* fParity              */
    FALSE,              /* fOutxCtsFlow         */
    FALSE,              /* fOutxDsrFlow         */
    DTR_CONTROL_ENABLE, /* fDtrControl          */
    FALSE,              /* fDsrSensitivity      */
    FALSE,              /* fTXContinueOnXoff    */
    FALSE,              /* fOutX                */
    FALSE,              /* fInX                 */
    FALSE,              /* fErrorChar           */
    FALSE,              /* fNull                */
    RTS_CONTROL_ENABLE, /* fRtsControl          */
    FALSE,              /* fAbortOnError        */
    0,                  /* fDummy2              */
    0,                  /* wReserved            */
    0x100,              /* XonLim               */
    0x100,              /* XoffLim              */
    8,                  /* ByteSize             */
    NOPARITY,           /* Parity               */
    ONESTOPBIT,         /* StopBits             */
    0x11,               /* XonChar              */
    0x13,               /* XoffChar             */
    '?',                /* ErrorChar            */
    0x1A,               /* EofChar              */
    0x10                /* EvtChar              */
};
 
int n;
/////////////////////////////////////////////////////////////////////////////
// CAboutDlg dialog used for App About
 
class CAboutDlg : public CDialog
{
public:
    CAboutDlg();
 
// Dialog Data
    //{{AFX_DATA(CAboutDlg)
    enum { IDD = IDD_ABOUTBOX };
    //}}AFX_DATA
 
    // ClassWizard generated virtual function overrides
    //{{AFX_VIRTUAL(CAboutDlg)
    protected:
    virtual void DoDataExchange(CDataExchange* pDX);    // DDX/DDV support
    //}}AFX_VIRTUAL
 
// Implementation
protected:
    //{{AFX_MSG(CAboutDlg)
    //}}AFX_MSG
    DECLARE_MESSAGE_MAP()
};
 
CAboutDlg::CAboutDlg() : CDialog(CAboutDlg::IDD)
{
    //{{AFX_DATA_INIT(CAboutDlg)
    //}}AFX_DATA_INIT
}
 
void CAboutDlg::DoDataExchange(CDataExchange* pDX)
{
    CDialog::DoDataExchange(pDX);
    //{{AFX_DATA_MAP(CAboutDlg)
    //}}AFX_DATA_MAP
}
 
BEGIN_MESSAGE_MAP(CAboutDlg, CDialog)
    //{{AFX_MSG_MAP(CAboutDlg)
        // No message handlers
    //}}AFX_MSG_MAP
END_MESSAGE_MAP()
 
/////////////////////////////////////////////////////////////////////////////
// CFenetre1Dlg dialog
 
CFenetre1Dlg::CFenetre1Dlg(CWnd* pParent /*=NULL*/)
    : CDialog(CFenetre1Dlg::IDD, pParent)
{
    //{{AFX_DATA_INIT(CFenetre1Dlg)
    m_activer = FALSE;
    m_heurefermeture = 0;
    m_minutefermeture = 0;
    m_heureouverture = 0;
    m_minuteouverture = 0;
    m_heure = 0;
    m_minute = 0;
    //}}AFX_DATA_INIT
    // Note that LoadIcon does not require a subsequent DestroyIcon in Win32
    m_hIcon = AfxGetApp()->LoadIcon(IDR_MAINFRAME);
}
 
void CFenetre1Dlg::DoDataExchange(CDataExchange* pDX)
{
    CDialog::DoDataExchange(pDX);
    //{{AFX_DATA_MAP(CFenetre1Dlg)
    DDX_Check(pDX, IDC_CHECK1, m_activer);
    DDX_Text(pDX, IDC_EDIT1, m_heurefermeture);
    DDV_MinMaxInt(pDX, m_heurefermeture, 0, 23);
    DDX_Text(pDX, IDC_EDIT2, m_minutefermeture);
    DDX_Text(pDX, IDC_EDIT3, m_heureouverture);
    DDX_Text(pDX, IDC_EDIT4, m_minuteouverture);
    DDX_Text(pDX, IDC_EDIT5, m_heure);
    DDX_Text(pDX, IDC_EDIT6, m_minute);
    //}}AFX_DATA_MAP
}
 
BEGIN_MESSAGE_MAP(CFenetre1Dlg, CDialog)
    //{{AFX_MSG_MAP(CFenetre1Dlg)
    ON_WM_SYSCOMMAND()
    ON_WM_PAINT()
    ON_WM_QUERYDRAGICON()
    ON_BN_CLICKED(IDC_apropos, Onapropos)
    ON_BN_CLICKED(IDC_CHECK1, OnCheck1)
    ON_EN_HSCROLL(IDC_EDIT2, Onheurefermeturemin)
    ON_EN_HSCROLL(IDC_EDIT1, Onheurefermetureh)
    ON_EN_CHANGE(IDC_EDIT1, OnChangeEdit1)
    ON_BN_CLICKED(IDC_BUTTON1, Onhautheurefermeture)
    ON_BN_CLICKED(IDC_BUTTON2, Onbahheurefermeture)
    ON_BN_CLICKED(IDC_BUTTON3, Onhautminutefermeture)
    ON_BN_CLICKED(IDC_BUTTON5, Onhautminuteouverture)
    ON_BN_CLICKED(IDC_BUTTON6, Onbasminuteouverture)
    ON_BN_CLICKED(IDC_BUTTON7, Onhautheureouverture)
    ON_BN_CLICKED(IDC_BUTTON8, Onbasheureouverture)
    ON_BN_CLICKED(IDC_BUTTON4, Onbasfermeturemin)
    //}}AFX_MSG_MAP
END_MESSAGE_MAP()
 
/////////////////////////////////////////////////////////////////////////////
// CFenetre1Dlg message handlers
 
BOOL CFenetre1Dlg::OnInitDialog()
{
    CDialog::OnInitDialog();
 
    // Add "About..." menu item to system menu.
 
    // IDM_ABOUTBOX must be in the system command range.
    ASSERT((IDM_ABOUTBOX & 0xFFF0) == IDM_ABOUTBOX);
    ASSERT(IDM_ABOUTBOX < 0xF000);
 
    CMenu* pSysMenu = GetSystemMenu(FALSE);
    if (pSysMenu != NULL)
    {
        CString strAboutMenu;
        strAboutMenu.LoadString(IDS_ABOUTBOX);
        if (!strAboutMenu.IsEmpty())
        {
            pSysMenu->AppendMenu(MF_SEPARATOR);
            pSysMenu->AppendMenu(MF_STRING, IDM_ABOUTBOX, strAboutMenu);
        }
    }
 
    // Set the icon for this dialog.  The framework does this automatically
    //  when the application's main window is not a dialog
    SetIcon(m_hIcon, TRUE);            // Set big icon
    SetIcon(m_hIcon, FALSE);        // Set small icon
 
    // TODO: Add extra initialization here
    m_heurefermeture=0;
    m_activer = TRUE;
    m_heureouverture=0;
    m_minuteouverture=0;
    m_minutefermeture=0;
 
 
 
GetDlgItem(m_Ouvrirelectrovanne)->EnableWindow(FALSE);
GetDlgItem(m_Fermerelectrovanne)->EnableWindow(FALSE);
            UpdateData(FALSE);
 
    int heure = 0;                            //initialisation des variables
    int minute = 0;
 
    CTime curTime = CTime::GetCurrentTime();//récupération de l'heure de l'horloge       
 
    heure = curTime.GetHour();                //séparation heures-minutes
    minute = curTime.GetMinute();
 
 
m_heure = heure;                            //affichage pour test
m_minute = minute;
UpdateData(FALSE);                            //actualise la fenêtre
 
/* Partie ouverture écriture fermeture du port série */
 
 
char buffer[256];
    int nId, nBytesWritten;
 
nId=3;
OpenCOM(nId);
 
do
{fflush(stdin);
            gets(buffer);
            nBytesWritten=255;
            WriteCOM(buffer, strlen(buffer), &nBytesWritten);
 
}
while(m_minutefermeture!=minute);
 
    CloseCOM();
 
 
 
 
    return TRUE;  // return TRUE  unless you set the focus to a control
}
 
// Fonction d'écriture sur le port COM
BOOL WriteCOM(void* buffer, int nBytesToWrite, int* pBytesWritten)
{
    /* écriture sur le port */
    return WriteFile(g_hCOM, buffer, nBytesToWrite, (DWORD*)pBytesWritten, NULL);
}
 
 
//Fonction d'ouverture du port COM
BOOL OpenCOM(int nId)
{
    /* variables locales */
    char szCOM[16];
 
    /* construction du nom du port, tentative d'ouverture */
    sprintf(szCOM, "COM%d", nId);
    g_hCOM = CreateFile(szCOM, GENERIC_READ|GENERIC_WRITE, 0, NULL,
                        OPEN_EXISTING, FILE_ATTRIBUTE_SYSTEM, NULL);
 
    /* affectation taille des tampons d'émission et de réception */
    SetupComm(g_hCOM, RX_SIZE, TX_SIZE);
 
    /* configuration du port COM */
    if(!SetCommTimeouts(g_hCOM, &g_cto) || !SetCommState(g_hCOM, &g_dcb))
    {
 
        CloseHandle(g_hCOM);
        return FALSE;
    }
 
    /* on vide les tampons d'émission et de réception, mise à 1 DTR */
    PurgeComm(g_hCOM, PURGE_TXCLEAR|PURGE_RXCLEAR|PURGE_TXABORT|PURGE_RXABORT);
    EscapeCommFunction(g_hCOM, SETDTR);
    return TRUE;
}
 
 
//Fonction de fermeture du port COM
 
BOOL CloseCOM()
{
    /* fermeture du port COM */
    CloseHandle(g_hCOM);
    return TRUE;
}
 
 
void CFenetre1Dlg::OnSysCommand(UINT nID, LPARAM lParam)
{
    if ((nID & 0xFFF0) == IDM_ABOUTBOX)
    {
        CAboutDlg dlgAbout;
        dlgAbout.DoModal();
    }
    else
    {
        CDialog::OnSysCommand(nID, lParam);
    }
}
 
// If you add a minimize button to your dialog, you will need the code below
//  to draw the icon.  For MFC applications using the document/view model,
//  this is automatically done for you by the framework.
 
void CFenetre1Dlg::OnPaint()
{
    if (IsIconic())
    {
        CPaintDC dc(this); // device context for painting
 
        SendMessage(WM_ICONERASEBKGND, (WPARAM) dc.GetSafeHdc(), 0);
 
        // Center icon in client rectangle
        int cxIcon = GetSystemMetrics(SM_CXICON);
        int cyIcon = GetSystemMetrics(SM_CYICON);
        CRect rect;
        GetClientRect(&rect);
        int x = (rect.Width() - cxIcon + 1) / 2;
        int y = (rect.Height() - cyIcon + 1) / 2;
 
        // Draw the icon
        dc.DrawIcon(x, y, m_hIcon);
    }
    else
    {
        CDialog::OnPaint();
    }
}
 
// The system calls this to obtain the cursor to display while the user drags
//  the minimized window.
HCURSOR CFenetre1Dlg::OnQueryDragIcon()
{
    return (HCURSOR) m_hIcon;
}
 
void CFenetre1Dlg::OnCancel()
{
    // TODO: Add extra cleanup here
 
    CDialog::OnCancel();
}
 
 
void CFenetre1Dlg::Onapropos()
{
    // TODO: Add your control notification handler code here
    MessageBox("Developed by Guillaume Libraire & Romain Quessada. Copyright 2008.");
}
 
void CFenetre1Dlg::OnCheck1()
{
 
if (IDC_CHECK1 == TRUE)
 
    { //grise les cases si la case est cochée
 
    GetDlgItem(m_Ouvrirelectrovanne)->EnableWindow(FALSE); // désactive les cases Ouvrir
GetDlgItem(m_Fermerelectrovanne)->EnableWindow(FALSE);       // et fermer électrovanne
UpdateWindow();
 
    }
 
else
    { //ne les grise pas dans le cas contraire
 
    GetDlgItem(m_Ouvrirelectrovanne)->EnableWindow(TRUE); // active les cases Ouvrir
GetDlgItem(m_Fermerelectrovanne)->EnableWindow(TRUE);      // et fermer électrovanne
UpdateWindow();
    }
UpdateWindow();
 
}
 
void CFenetre1Dlg::Onheurefermeturemin()
{
    // TODO: If this is a RICHEDIT control, the control will not
 
 
 
    // send this notification unless you override the CDialog::OnInitDialog()
    // function to send the EM_SETEVENTMASK message to the control
    // with the ENM_SCROLL flag ORed into the lParam mask.
 
    // TODO: Add your control notification handler code here
 
}
 
void CFenetre1Dlg::Onheurefermetureh()
{
    // TODO: If this is a RICHEDIT control, the control will not
 
    UpdateData(TRUE);
 
    // send this notification unless you override the CDialog::OnInitDialog()
    // function to send the EM_SETEVENTMASK message to the control
    // with the ENM_SCROLL flag ORed into the lParam mask.
 
    // TODO: Add your control notification handler code here
 
}
 
void CFenetre1Dlg::OnChangeEdit1()
{
 
 
    // TODO: If this is a RICHEDIT control, the control will not
    // send this notification unless you override the CDialog::OnInitDialog()
    // function and call CRichEditCtrl().SetEventMask()
    // with the ENM_CHANGE flag ORed into the mask.
 
    // TODO: Add your control notification handler code here
 
}
 
 
// ####################################################################################
//############incrémentation et décrémentation des editbox via les boutons#############
//#####################################################################################
 
void CFenetre1Dlg::Onhautheurefermeture()
{m_heurefermeture++;
 
if(m_heurefermeture>23)
{m_heurefermeture=0;}
UpdateData(FALSE);
    // TODO: Add your control notification handler code here
 
}
 
void CFenetre1Dlg::Onbahheurefermeture()
{m_heurefermeture--;
    // TODO: Add your control notification handler code here
if (m_heurefermeture<0)
{m_heurefermeture=23;}   
UpdateData(FALSE);
}
 
void CFenetre1Dlg::Onhautminutefermeture()
{m_minutefermeture=m_minutefermeture+10;    //incrémente les minutes de 10 en 10
if(m_minutefermeture>50)                    //compare l'heure à une valeur max
{m_minutefermeture=0;}                        //corrige si nécéssaire
 
    UpdateData(FALSE);                        //Rafraichit l'edit box
}
 
void CFenetre1Dlg::Onhautminuteouverture()
{m_minuteouverture=m_minuteouverture+10;
if(m_minuteouverture>50)
{m_minuteouverture=0;}   
 
// TODO: Add your control notification handler code here
    UpdateData(FALSE);
}
 
void CFenetre1Dlg::Onbasminuteouverture()
{m_minuteouverture=m_minuteouverture-10;
if (m_minuteouverture<0)
{m_minuteouverture=50;}
    // TODO: Add your control notification handler code here
    UpdateData(FALSE);
}
 
void CFenetre1Dlg::Onhautheureouverture()
{m_heureouverture++;
    // TODO: Add your control notification handler code here
if(m_heureouverture>23)
{m_heureouverture=0;}
    UpdateData(FALSE);
}
 
void CFenetre1Dlg::Onbasheureouverture()
{m_heureouverture--;
    // TODO: Add your control notification handler code here
if (m_heureouverture<0)
{m_heureouverture=23;}   
 
UpdateData(FALSE);
}
 
void CFenetre1Dlg::Onbasfermeturemin()
{m_minutefermeture=m_minutefermeture-10;
 
if (m_minutefermeture<0)
{m_minutefermeture=50;}
 
    // TODO: Add your control notification handler code here
        UpdateData(FALSE);
}
// ####################################################################################
//############fin d'incrémentation et décrémentation des editbox via les boutons#######
//#####################################################################################

[Flow_75]

salut

1 - attention a mettre ton code entre balises.
2 - Il est où ta classe dérivé de CWinApp ?
3 - CWinApp::Run() ?

[Taurnil]

Euh, je suis nouveau sur le site alors désolé si mon message était pas correct. La, il est dans une fenêtre, j'imagine que c'est ça entre balises mais c'est pas moi qui l'ai fait.

Pour tes questions, si tu penses qu'il y a des trucs à ajouter/retirer/modifier, tu pourrais être plus explicite car je suis pas vraiment bon en programmation.

Merci d'avance^^

[Flow_75]

pour les balises, c'est pas grave, la prochaine fois tu le sauras.

Concernant ton programme, à vrai dire, je ne connais pas le MFC.
Mais en fait j'ai regarder dans le tutoriel du site (http://farscape.developpez.com/tutor...arrer-mfc-vc6/)
Il parle d'une classe CWinApp à dériver, etc...
regarde cette doc.
car si ca compile, c'est que ca vient pas de ton code (enfin, je te préviens encore que je ne connais le mfc que via les premieres pages du tuto...).

[Taurnil]

Bon bah je vais lire la doc. merci à toi.
Je reposterais pour dire si c'était suffisant ou pour reposter des questions si besoin est.

Voila, à très bientôt j'imagine.

[Flow_75]

une question,

tu n'as seulement que ce fichier à montrer ?

[Taurnil]

Je comprend pas bien ta question, juste ce fichier à montrer, ça veut dire quoi?

J'ai un peu changé de méthode, en fait j'utilise la broche DTR du COM pour commander mon électrovanne (car en fait le programme sert à ça). J'arrive sans problème avec 2 boutons à ouvrir ou fermer l'electrovanne en cliquant sur 2 boutons : voici la modif dans le code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
void CFenetre1Dlg::OnOuvrirelectrovanne() 
{
	PurgeComm(g_hCOM, PURGE_TXCLEAR|PURGE_RXCLEAR|PURGE_TXABORT|PURGE_RXABORT);
    EscapeCommFunction(g_hCOM, SETDTR);
}
 
void CFenetre1Dlg::OnFermerelectrovanne() 
{
  EscapeCommFunction(g_hCOM, CLRDTR);
 PurgeComm(g_hCOM, PURGE_TXABORT|PURGE_RXABORT|PURGE_TXCLEAR|PURGE_RXCLEAR); 
}

Ça, c'est les mises à 0 ou 1 du fil DTR, je les commande avec les boutons ouvrir et fermerelctrovanne.

Maintenant la boucle, je sais pas pourquoi j'arrive à exécuter maintenant mais bon :

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
do{
CTime curTime = CTime::GetCurrentTime();//récupération de l'heure de l'horloge		
	heure = curTime.GetHour();				//séparation heures-minutes
	minute = curTime.GetMinute();
	UpdateData(FALSE);
 
nId=3;
/* variables locales */
    char szCOM[16];
 
    /* construction du nom du port, tentative d'ouverture */
    sprintf(szCOM, "COM%d", nId);
    g_hCOM = CreateFile(szCOM, GENERIC_READ|GENERIC_WRITE, 0, NULL,
                        OPEN_EXISTING, FILE_ATTRIBUTE_SYSTEM, NULL);
 
    /* affectation taille des tampons d'émission et de réception */
    SetupComm(g_hCOM, RX_SIZE, TX_SIZE);
 
    /* configuration du port COM */
    if(!SetCommTimeouts(g_hCOM, &g_cto) || !SetCommState(g_hCOM, &g_dcb))
    {
 
        CloseHandle(g_hCOM);
        return FALSE;
    }
 
    /* on vide les tampons d'émission et de réception, mise à 1 DTR */
    PurgeComm(g_hCOM, PURGE_TXCLEAR|PURGE_RXCLEAR|PURGE_TXABORT|PURGE_RXABORT);
    EscapeCommFunction(g_hCOM, SETDTR);
    return TRUE;
 
 
UpdateData(FALSE);
 CloseCOM();
 
}	while(m_minutefermeture!=minute);
 
UpdateData(FALSE);

En fait ici, plutôt que d'appeler la fonction OpenCOM (qui fait en même temps la mise à 1 DTR), je l'ai recopié et ça marche. Me demandez pas pourquoi et à la limite, si personne ne m'explique, ça va pas m'empêcher de dormir, le principal étant que ça marche.

Par contre, mon heure ne se met pas à jour dans la boucle et je sais pas comment on fait, si quelqu'un pouvait m'aider de ce côté là, ce serait (vraiment très) gentil.

Merci beaucoup d'avance.

[Taurnil]

Desolé je reposte mais je fais part au fur et à mesure de mes avancées.

En fait, j'ai pas vraiment avancé mais j'ai testé quelque chose, ça ne marche pas et j'aimerais bien savoir pourquoi (ou comment ça marcherait) :

Voila le code que j'ai utilisé :

do{CTime curTime = CTime::GetCurrentTime();//récupération de l'heure de l'horloge
heure = curTime.GetHour(); //séparation heures-minutes
minute = curTime.GetMinute();
UpdateData(FALSE);
if(heure==m_heurefermeture)
{
if(minute==m_minutefermeture)
{EscapeCommFunction(g_hCOM, CLRDTR);
PurgeComm(g_hCOM, PURGE_TXABORT|PURGE_RXABORT|PURGE_TXCLEAR|PURGE_RXCLEAR); }
}

if(heure==m_heureouverture)
{
if(minute==m_minuteouverture)
{PurgeComm(g_hCOM, PURGE_TXCLEAR|PURGE_RXCLEAR|PURGE_TXABORT|PURGE_RXABORT);
EscapeCommFunction(g_hCOM, SETDTR);}
Sleep(30000);
}
}while(1);

En gros, je teste en cascade et en boucle (while 1, toutes les 30 secondes) les valeurs respectives : des qu'on atteint l'heure de fermeture : mise à 0 et dès qu'on atteint l'heure d'ouverture : mise à 1.

Mais ça marche pas, quand on arrive à l'heure de fermeture, mon électrovanne se ferme pas.

J'ai l'impression que l'heure est mise à jour en plus car on va la chercher toutes les 30 secondes.

Donc si quelqu'un peut m'aider, qu'il n'hésite pas

[escafr]

mieux?

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
void test() {
    CTime curTime;
    int heure = 0;
    int minute = 0;
    while (true) {
        curTime = CTime::GetCurrentTime();
        heure = curTime.GetHour(); //séparation heures-minutes
        minute = curTime.GetMinute();
        UpdateData(TRUE);
        if (heure == m_heurefermeture && minute == m_minutefermeture) {
            EscapeCommFunction(g_hCOM, CLRDTR);
            PurgeComm(g_hCOM, PURGE_TXABORT|PURGE_RXABORT|PURGE_TXCLEAR|PURGE_RXCLEAR);
        } else if (heure == m_heureouverture && minute == m_minuteouverture) {
            PurgeComm(g_hCOM, PURGE_TXCLEAR|PURGE_RXCLEAR|PURGE_TXABORT|PURGE_RXABORT);
            EscapeCommFunction(g_hCOM, SETDTR);
        }
        Sleep(30000);
    }
}

Ne voudrais tu pas indiquer des intervalles d heure, plutot que dire si 17h00 pile poile alors faire cela.
Genre ouvert entre 08h00 et 17h00 et fermer le reste du temps?

EDIT:
De plus pense à bien indenter ton code, c est bcp plus simple pour le lire.

[Flow_75]

mieux?
EDIT:
De plus pense à bien indenter ton code, c est bcp plus simple pour le lire.

Penses également à la balise code, ca sera encore plus simple a te lire

[Taurnil]

Merci beaucoup les gars, je teste ça de suite et je reposte dans quelques minutes pour vous dire quoi.

A tout de suite

[Taurnil]

Bon, l'électrovanne reste ouverte après l'heure de fermeture le problème doit venir d'ailleurs.

Par contre escafr, tu as parlé d'un truc intéressant, en fait l'opérateur entre une heure de fermeture et une heure d'ouverture et l'électrovanne doit rester ouverte entre ces heures.
Tu crois que ça serais plus simple de voir les choses avec une plage de temps ou avec le déclenchement à heure fixe?

De toute façon avec le déclenchement à heure fixe, il n'y a pas de raison pour que ça ne marche pas, le truc étant que ça ne marche pas.

Je n'ai aucune idée d'où ça vient, vous auriez une idée?
Merci d'avance.

PS : Dites le si vous n'en avez pas d'idée, c'est pas grave, c'est déjà sympa de prendre du temps pour moi, en plus ça m'évitera de me dire : "c'est bon, je peux glandouiller jusqu'à ce qu'on me réponde... (je me connais)

[escafr]

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
void test() {
    CTime curTime;
    int heure = 0;
    int minute = 0;
    while (true) {
        curTime = CTime::GetCurrentTime();
        heure = curTime.GetHour(); //séparation heures-minutes
        minute = curTime.GetMinute();
        UpdateData(TRUE);
        if (heure == m_heurefermeture && minute == m_minutefermeture) {            
         std::cout << "Fermeture de la vanne" << std::endl;
        } else if (heure == m_heureouverture && minute == m_minuteouverture) {
            std::cout << "Ouverture de la vanne" << std::endl;
        }
        Sleep(30000);
    }
}

si ca affiche rien en sortie, c est qu il y a un probleme dans les tests.
Si ca affiche qquechose c est que ta gestion de vanne marche pas.


L'avantage d'avoir une plage horaire est que tu peux verifier toutes les x minutes au lieu de toutes les 30sc, a part si c est très important de pour toi d etre precis dans l ouverture et la fermeture. L avantage aussi est que si tu as oublie de lancer ton programme, ben ca marchera qd meme.

Ou mieux encore, tu fais le programme qui verifie l intervalle sans boucle, puis tu le mets dans le lanceur automatique de tache windows et pouf pouf ca t evite d avoir ton prog qui tourne tout le temps.

PS: la balise code c'est la balise #

[Taurnil]

Dans le bout de code que tu m'as donné, tu mets des cout mais moi, je suis pas en console, mon appli est une MFC. Et aussi, c'est quoi les std:: que tu mets devant ces mêmes cout?

Le problème avec la plage horaire est qu'elle n'est pas sensée être fixe :

Le lundi de 8h à 19h, le mardi pareil mais le vendredi on pourrait vouloir qu'elle s'allume de 8h à 15h par exemple, et pas du tout allumée le week-end...

(en fait pour le week-end, il suffit de fermer le programme, l'électrovanne s'éteint automatiquement dans ce cas)

Si tu penses que c'est quand même possible de faire ça avec les plages de temps (et donc que je n'ai pas besoin de changer mon interface) tu pourrais me mettre un petit bout d'algo ou de code ? (même blindé de fautes, c'est juste pour le principe)


Merci beaucoup d'avance.


PS: Je vais faire un mini programme en console pour pouvoir faire le test que tu m'as donné, on verra bien si j'y arrive^^

[escafr]

[QUOTE=escafr;3185893]

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
void test() {
    CTime curTime;
    int heure = 0;
    int minute = 0;
    while (true) {
        curTime = CTime::GetCurrentTime();
        heure = curTime.GetHour(); //séparation heures-minutes
        minute = curTime.GetMinute();
        UpdateData(TRUE);
        if (heure == m_heurefermeture && minute == m_minutefermeture) {            
        AfxMessageBox("Fermeture de la vanne", MB_ICONSTOP);
        } else if (heure == m_heureouverture && minute == m_minuteouverture) {
           AfxMessageBox("Ouverture de la vanne", MB_ICONSTOP);
        }
        Sleep(30000);
    }
}

std c est le namespace pour la librairie standard, STL.

Tu crées un fichier avec les plages horaires de tous les jours, puis tu parses ton fichier pour savoir quelle plage horaire prendre suivant le jour
genre:

Code :

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

1
2
3
4
5
6
 
lundi;0800;1700
mardi;0800;1700
mercredi;0800;1800
jeudi;0800;1900
...etc

mais bon tu me sembles être très débutant
donc moi je serais toi, je lirais la FAQ C++ developpez.com pendant une journée, histoire de bien intégrer les concepts, puis je commencerais.

[Taurnil]

Salaud, je sais que je suis débutant c'est pas la peine de le dire ...

Bon, de toute façon tu dois avoir raison, je vais lire ça et je vais tester le morceau que tu m'as filé avec mon programme, je reposterais pour dire ce que ça m'as donné.

Merci beaucoup, a très bientôt

[Taurnil]

J'ai fait le test de ton bout de programme avant de commencer à lire (désolé j'ai pas pu m'empêcher) et aucune messagebox ne s'ouvre. Donc, le problème n'est pas au niveau du pilotage de l'électrovanne.

Le truc, c'est que je ne sais pas du tout où il est le problème.

En fait, j'ai l'impression que le programme lance la fenêtre mais ne boucle pas pour vérifier l'heure toute les 30 secondes.

Du coup, j'ai mis 2 edit box dans lesquels j'écris les valeurs de l'heure de l'horloge. Et en fait je crois que j'ai un peu cerné le problème : heure et minute n'avancent pas en même temps que l'horloge.

Voici le bout de code que j'ai mis pour le test :

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
 while (true) {
        curTime = CTime::GetCurrentTime();
        heure = curTime.GetHour(); //séparation heures-minutes
        minute = curTime.GetMinute();
 
		UpdateData(TRUE);
        if (heure == m_heurefermeture && minute == m_minutefermeture) {            
        AfxMessageBox("Fermeture de la vanne", MB_ICONSTOP);
        } else if (heure == m_heureouverture && minute == m_minuteouverture) {
           AfxMessageBox("Ouverture de la vanne", MB_ICONSTOP);
        }
		m_heure = heure;		//affichage pour test			
		m_minute = minute;
		UpdateData(FALSE);
        Sleep(30000);
 
 
	}

Si y a pas d'erreur la dedans, tes messages box ne s'affichent pas parce que l'heure n'avance pas en même temps. Pourquoi le programme ne boucle pas?

[escafr]

euh bizarre

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
 while (true) {
        curTime = CTime::GetCurrentTime();
        heure = curTime.GetHour(); //séparation heures-minutes
        minute = curTime.GetMinute();
        AfxMessageBox("Si ca s'affiche pas c'est que je rentre pas dans la boucle", MB_ICONSTOP);
		UpdateData(TRUE);
        if (heure == m_heurefermeture && minute == m_minutefermeture) {            
        AfxMessageBox("Fermeture de la vanne", MB_ICONSTOP);
        } else if (heure == m_heureouverture && minute == m_minuteouverture) {
           AfxMessageBox("Ouverture de la vanne", MB_ICONSTOP);
        }
		m_heure = heure;		//affichage pour test			
		m_minute = minute;
		UpdateData(FALSE);
        Sleep(30000);
 
 
	}

[Taurnil]

Ca s'affiche pas mais en fait y a un autre problème. Quand je mets ça :

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
     AfxMessageBox("Si ca s'affiche pas c'est qu'il y a une merde quelque part", MB_ICONSTOP);
		UpdateData(TRUE);
 
 
  while (true) {
        curTime = CTime::GetCurrentTime();
        heure = curTime.GetHour(); //séparation heures-minutes
        minute = curTime.GetMinute();
        AfxMessageBox("Si ca s'affiche pas c'est que je rentre pas dans la boucle", MB_ICONSTOP);
		UpdateData(TRUE);
        if (heure == m_heurefermeture && minute == m_minutefermeture) {            
        AfxMessageBox("Fermeture de la vanne", MB_ICONSTOP);
        } else if (heure == m_heureouverture && minute == m_minuteouverture) {
           AfxMessageBox("Ouverture de la vanne", MB_ICONSTOP);
        }
		m_heure = heure;		//affichage pour test			
		m_minute = minute;
		UpdateData(FALSE);
        Sleep(30000);}
	return TRUE;  // return TRUE  unless you set the focus to a control
}

Donc une première ligne qui s'exécute même si on rentre pas dans la boucle, et bien la message box associée ne s'affiche pas.... Faut associer les message box à quelque chose ou pas (dans Class Wizard je veux dire)?

[escafr]

ben c est que tu rentres pas dans la fonction.. sinon je vois pas du tout...