Simulateur de présence avec UNO et DS3231

Bonjour à tous ,

Je vous sollicite car j'essaie de réaliser un simulateur de présence
(https://www.instructables.com/Arduin...d-alarm-based/ ) composé de :
-Arduino UNO
-DS3231 ( à la place de DS1307)
-LCD-IIC ( la place LCD )

Le programme permet normalement de commander 6 relais différents sur 7 jours avec des durées différentes
Menu et Submenu sont prévus pour permettre les mises à jour

==> Date
===> Heure
=====> Alarme 01 Heure
======> Alarme 01 Jour
=======> Alarme 01 Durée
========> Alarme 01 Relais
l'opération se répète ainsi pour les autres alarmes ....

Mon problème est le suivant :
Alors que l'ensemble du programme fonctionne correctement
je n'arrive pas à incrémenter les heures et les minutes à partir des sous-programmes

void plusAlarma() et void minusAlarma()

L'heure ne s'incrémente pas et pour les minutes elles s'incrémentent mais le changment s'effectue aussi sur les heures (?) avec une valeur - 1
ex : 00:01 si btn+ devient 01:02 puis 02:03 etc (?)

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 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 502 503 504 505 506 507 508 509 510 511 512 513 514 515 516 517 518 519 520 521 522 523 524 525 526 527 528 529 530 531 532 533 534 535 536 537 538 539 540 541 542 543 544 545 546 547 548 549 550 551 552 553 554 555 556 557 558 559 560 561 562 563 564 565 566 567 568 569 570 571 572 573 574 575 576 577 578 579 580 581 582 583 584 585 586 587 588 589 590 591 592 593 594 595 596 597 598 599 600 601 602 603 604 605 606 607 608 609 610 611 612 613 614 615 616 617 618 619 620 621 622 623 624 625 626 627 628 629 630 631 632 633 634 635 636 637 638 639 640 641 642 643 644 645 646 647 648 649 650 651 652 653 654 655 656 657 658 659 660 661 662 663 664 665 666 667 668 669 670 671 672 673 674 675 676 677 678 679 680 681 682 683 684 685 686 687 688 689 690 691 692 693 694 695 696 697 698 699 700 701 702 703 704 705 706 707 708 709 710 711 712 713 714 715 716 717 718 719 720 721 722 723 724 725 726 727 728 729 730 731 732 733 734 735 736 737 738 739 740 741 742 743 744 745 746 747 748 749 750 751 752 753 754 755 756 757 758 759 760 761 762 763 764 765 766 767 768 769 770 771 772 773 774 775 776 777 778

#include "Wire.h" //#include "LiquidCrystal.h" #include <hd44780.h> #include <hd44780ioClass/hd44780_I2Cexp.h> // include i/o class header #include "math.h" #include "EEPROM.h" #define DS1307_I2C_ADDRESS 0x68 #define PIN_STG 8 #define PIN_MENU 2 #define PIN_DR 10 #define PIN_COMMAND_A A1 //relais A #define PIN_COMMAND_B A2 //relais B #define PIN_COMMAND_C A3 //relais C #define PIN_COMMAND_D A0 //relais D #define PIN_COMMAND_E 7 //relais E #define PIN_COMMAND_F 9 //relais F #define PIN_COMMAND_G 13 //relais G #define DESCHIS LOW #define INCHIS HIGH #define NR_ALARME 6 #define MAX_MENU_POS NR_ALARME*4+3 //doit être NR_ALARMS*4+3 hd44780_I2Cexp lcd; // auto locate and autoconfig interface pins #define HD44780_LCDOBJECT byte menuPos=0; byte subMenuPos = 0; byte cursorPos = -1; char ligne1[17]; char ligne2[17]; char ligne[17]; boolean dataDirthy = false; boolean oraDirthy = false; long lastMillis=0; boolean bAlarma = false; typedef struct alarm{ byte hour; byte minute; byte days; byte duration; byte relee; byte unitAlarma; }; typedef struct ds1307_time{ byte second; byte minute; byte hour; byte dayOfWeek; byte dayOfMonth; byte month; byte year; }; struct ds1307_time time; alarm* alarme = (alarm*) malloc(sizeof(alarm) * NR_ALARME); byte relee[]={PIN_COMMAND_G, PIN_COMMAND_F,PIN_COMMAND_E,PIN_COMMAND_D,PIN_COMMAND_C,PIN_COMMAND_B,PIN_COMMAND_A}; byte decToBcd(byte val){ return ((val/10*16)+(val%10)); } byte bcdToDec(byte val){ return ((val/16*10)+(val%16)); } void setDateDs1307(struct ds1307_time time) { Wire.beginTransmission(DS1307_I2C_ADDRESS); Wire.write(0); Wire.write(decToBcd(time.second)); Wire.write(decToBcd(time.minute)); Wire.write(decToBcd(time.hour)); Wire.write(decToBcd(time.dayOfWeek)); Wire.write(decToBcd(time.dayOfMonth)); Wire.write(decToBcd(time.month)); Wire.write(decToBcd(time.year)); Wire.endTransmission(); } void getDateDs1307(struct ds1307_time *time) { Wire.beginTransmission(DS1307_I2C_ADDRESS); Wire.write(0); Wire.endTransmission(); Wire.requestFrom(DS1307_I2C_ADDRESS,7); (*time).second = bcdToDec(Wire.read() & 0x7f); (*time).minute = bcdToDec(Wire.read()); (*time).hour = bcdToDec(Wire.read() & 0x3f); (*time).dayOfWeek = bcdToDec(Wire.read()); (*time).dayOfMonth = bcdToDec(Wire.read()); (*time).month = bcdToDec(Wire.read()); (*time).year = bcdToDec(Wire.read()); } void showMenu(){ char *zile[7]; zile[0]="L\0"; zile[1]="M\0"; zile[2]="M\0"; zile[3]="J\0"; zile[4]="V\0"; zile[5]="S\0"; zile[6]="D\0"; if(menuPos==1){ // Menu Date strcpy(ligne1,"Date\0"); strcpy(ligne2,getDate(time)); strcat(ligne2," "); strcat(ligne2,getZiuaSaptamaniiShort(time.dayOfWeek)); }else if(menuPos==2){ // Menu Heure strcpy(ligne1,"Heure\0"); strcpy(ligne2,getTime(time)); }else if((menuPos+1)%4==0){ // Menu Alarme Heure strcpy(ligne1,"Alarme "); ligne1[7] = 48+(byte)((menuPos+1)/4)/10; ligne1[8] = 48+(byte)(((menuPos+1)/4)-((menuPos+1)/4)/10*10); ligne1[9]='\0'; strcat(ligne1," Heure\0"); strcpy(ligne2,getFmt2Char((alarme[(menuPos+1)/4-1]).hour)); // Alarme Heure strcat(ligne2,":"); strcat(ligne2,getFmt2Char((alarme[(menuPos+1)/4-1]).minute)); // Alarme Minute strcat(ligne2,""); }else if((menuPos)%4==0){ // Jour de l'Alarme strcpy(ligne1,"Alarme "); ligne1[7] = 48+(byte)((menuPos)/4)/10; ligne1[8] = 48+(byte)((menuPos)/4-(menuPos)/4/10*10); ligne1[9]='\0'; strcat(ligne1," Jour\0"); strcpy(ligne2,"L"); for(int i=6;i>=0;i--){ if((alarme[(menuPos)/4-1]).days & (byte)ceil(pow(2,i))){ strcat(ligne2,"<"); }else{ strcat(ligne2," "); } if(i>0)strcat(ligne2,zile[7-i]); } }else if((menuPos-1)%4==0){ // Duree de l'Alarme strcpy(ligne1,"Alarme "); ligne1[7] = 48+(byte)((menuPos-1)/4)/10; ligne1[8] = 48+(byte)((menuPos-1)/4-(menuPos-1)/4/10*10); ligne1[9]='\0'; strcat(ligne1," duree\0"); itoa(alarme[(menuPos-1)/4-1].duration,ligne2,10); if(alarme[(menuPos-1)/4-1].unitAlarma){ strcat(ligne2," min"); }else{ strcat(ligne2," sec"); } }else if((menuPos-2)%4==0){ // Relais de l'Alarme strcpy(ligne1,"Relais Alarme"); ligne1[13]= 48+(byte)((menuPos-2)/4)/10; ligne1[14]= 48+(byte)((menuPos-2)/4-(menuPos-2)/4/10*10); ligne1[15]='\0'; strcpy(ligne2,""); for(int i=6;i>=0;i--){ char chBuff[2]; chBuff[0]=6-i+65; chBuff[1]='\0'; strcat(ligne2,chBuff); if((alarme[(menuPos-2)/4-1]).relee & (byte)ceil(pow(2,i))){ strcat(ligne2,"<"); }else{ strcat(ligne2," "); } //if(i>0)strcat(ligne2,zile[7-i]); } } } //nous plaçons le curseur en fonction de notre position dans submenu void showSubMenu(){ if(menuPos==1 || menuPos==2){ //la date ou l'heure switch(subMenuPos){ case 1://jour cursorPos = 1; break; case 2://Lundi cursorPos = 4; break; case 3://année cursorPos = 7; break; case 4://jour de la semaine if(menuPos==1){//seulement dans le cas de la date, nous avons le jour de la semaine cursorPos=12; break; } default: subMenuPos = 0; cursorPos = -1; break; } }else if((menuPos+1)%4==0){ //heure d'alarme switch(subMenuPos){ case 1: //ora cursorPos = 1; break; case 2: //minutul cursorPos = 4; break; default: subMenuPos = 0; cursorPos = -1; break; } }else if((menuPos)%4==0){ //jours d'alarme if(subMenuPos>0 && subMenuPos<8){ cursorPos = subMenuPos*2-1; }else{ subMenuPos = 0; cursorPos = -1; } }else if((menuPos-1)%4==0){ //durée de l'alarme if(subMenuPos==1){ cursorPos = 0; }else if(subMenuPos==2){ cursorPos=4; }else{ subMenuPos=0; cursorPos = -1; } }else if((menuPos-2)%4==0){ //relais d'alarme if(subMenuPos>0 && subMenuPos<8){ cursorPos = subMenuPos*2-1; }else{ subMenuPos = 0; cursorPos = -1; } } } void menu(){ if((millis()-lastMillis)>150){ if(digitalRead(PIN_MENU)==LOW && menuPos==0){ menuPos=(menuPos+1)%(MAX_MENU_POS); showMenu(); }else if(digitalRead(PIN_MENU)==LOW){ subMenuPos++; showSubMenu(); } } lastMillis=millis(); } void readValues(){ //nous lisons les valeurs initiales de l'eeprom. //si rien n'est écrit dans l'EEPROM, nous considérons également que les heures et la durée sont égales à 0 et nous écrivons dans l'EEPROM. for(int i=0;i<NR_ALARME;i++){ byte b = EEPROM.read(i*5+1); if(b==0xFF){ b = 0; EEPROM.write(i*5+1,b); } (alarme[i]).hour = b & 31; //pointeurs 1, 6, 11 etc. (alarme[i]).unitAlarma = b >> 6; b = EEPROM.read(i*5+2); if(b==0xFF){ b = 0; EEPROM.write(i*5+2,b); } (alarme[i]).minute = b; //pointeurs 2, 7, 12 etc. b = EEPROM.read(i*5+3); if(b==0xFF){ b = 0; EEPROM.write(i*5+3,b); } (alarme[i]).days = b; //pointeurs 3, 8, 13 etc. b = EEPROM.read(i*5+4); if(b==0xFF){ b = 0; EEPROM.write(i*5+4,b); } (alarme[i]).duration = b; //pointeurs 4, 9, 14 etc. b = EEPROM.read(i*5+5); if(b==0xFF){ b = 0; EEPROM.write(i*5+5,b); } (alarme[i]).relee = b; //pointeurs 5, 10, 15 etc. } } void setup(){ pinMode(PIN_MENU,INPUT); //btn menu pinMode(PIN_DR,INPUT); //btn droit pinMode(PIN_STG,INPUT); //btn gauche for(int i=0;i<7;i++){ pinMode(relee[i],OUTPUT); digitalWrite(relee[i],DESCHIS); } digitalWrite(PIN_MENU,HIGH); digitalWrite(PIN_STG,HIGH); digitalWrite(PIN_DR,HIGH); attachInterrupt(0,menu,CHANGE); //alarmes readValues(); //initializam DS_1307 Wire.begin(); //lcd.begin(16, 2); lcd.init(); lcd.backlight(); lcd.home(); Serial.begin(9600); struct ds1307_time time; time.second = 00; time.minute = 33; time.hour = 15; time.dayOfWeek = 4; time.dayOfMonth = 03; time.month = 8; time.year = 23; //setDateDs1307(time); } char* getFmt2Char(byte val){ char *buf = "00\0"; buf[0] = 48+val/10; buf[1] = 48+val%10; return buf; } char* getDate(struct ds1307_time time){ char *buf=" \0"; buf[0]=48+time.dayOfMonth/10; buf[1]=48+time.dayOfMonth%10; buf[2]='.'; buf[3]=48+time.month/10; buf[4]=48+time.month%10; buf[5]='.'; buf[6]=48+time.year/10; buf[7]=48+time.year%10; return buf; } char* getTime(struct ds1307_time time){ char *buf=" \0"; buf[0]=48+time.hour/10; buf[1]=48+time.hour%10; buf[2]=':'; buf[3]=48+time.minute/10; buf[4]=48+time.minute%10; buf[5]=':'; buf[6]=48+time.second/10; buf[7]=48+time.second%10; return buf; } char* getZiuaSaptamaniiShort(byte val){ switch(val){ case 1: return "Lun\0"; //Mon break; case 2: return "Mar\0"; //Tue break; case 3: return "Mer\0"; //Wen break; case 4: return "Jeu\0"; //Thu break; case 5: return "Ven\0"; //Fri break; case 6: return "Sam\0"; //Sat break; case 7: return "Dim\0"; //Sun break; default: return "?\0"; break; } } char* getZiuaSaptamanii(byte val){ switch(val){ case 1: return "Lundi\0"; //replace with Monday if you want break; case 2: return "Mardi\0"; //replace with Tuesday if you want break; case 3: return "Mercredi\0"; //replace with Wednesday if you want break; case 4: return "Jeudi\0"; //replace with Thursday if you want break; case 5: return "Vendredi\0"; //replace with Friday if you want break; case 6: return "Samedi\0"; //replace with Saturday if you want break; case 7: return "Dimanche\0"; //replace with Sunday if you want break; default: return "?\0"; break; } } void minusData(){ if(subMenuPos==1 && time.dayOfMonth>1){ //jours time.dayOfMonth--; }else if(subMenuPos==2 && time.month>1){ time.month--; }else if(subMenuPos==3 && time.year>1){ time.year--; }else if(subMenuPos==4 && time.dayOfWeek>1){ time.dayOfWeek--; } dataDirthy=true; showMenu(); } void plusData(){ if(subMenuPos==1){ //jours ici l'utilisateur doit faire attention à ne pas mettre plus de jours qu'il n'a de mois, je l'ai limité à 31 seulement time.dayOfMonth=(time.dayOfMonth+1)%32; if(time.dayOfMonth==0){ time.dayOfMonth=1; } }else if(subMenuPos==2){ time.month=(time.month+1)%13; if(time.month==0){ time.month=1; } }else if(subMenuPos==3){ time.year=(time.year+1)%99; }else if(subMenuPos==4){ time.dayOfWeek=(time.dayOfWeek+1)%8; if(time.dayOfWeek==0){ time.dayOfWeek=1; } } dataDirthy=true; showMenu(); } void minusOra(){ if(subMenuPos==1 && time.hour>0){ //zilele time.hour=time.hour-1; }else if(subMenuPos==2 && time.minute>0){ time.minute=time.minute-1; }else if(subMenuPos==3 && time.second>1){ time.second=time.second-1; } oraDirthy = true; showMenu(); } void plusOra(){ if(subMenuPos==1){ //zilele time.hour=(time.hour+1)%24; }else if(subMenuPos==2){ time.minute=(time.minute+1)%60; }else if(subMenuPos==3){ time.second=(time.second+1)%60; } oraDirthy = true; showMenu(); } void minusAlarma(byte pos){ alarme[pos-1]; if(subMenuPos==1) { if (alarme[pos-1].hour>0){ alarme[pos-1].hour--; } }else if(subMenuPos==2) { if (alarme[pos-1].minute>0){ alarme[pos-1].minute--; } } showMenu(); } void plusAlarma(byte pos){ alarme[pos-1]; if(subMenuPos==1){ alarme[pos-1].hour=(alarme[pos-1].hour+1)%24; } else if(subMenuPos==2){ alarme[pos-1].minute=(alarme[pos-1].minute+1)%60; } showMenu(); } void minusZileAlarma(byte pos){ Serial.println(alarme[pos-1].days); alarme[pos-1].days =alarme[pos-1].days ^ (byte)ceil(pow(2,(7-subMenuPos))); Serial.println(alarme[pos-1].days); showMenu(); } void plusReleeAlarma(byte pos){ minusReleeAlarma(pos); } void minusReleeAlarma(byte pos){ Serial.println(alarme[pos-1].relee); alarme[pos-1].relee = alarme[pos-1].relee ^ (byte)ceil(pow(2,(7-subMenuPos))); Serial.println(alarme[pos-1].relee); showMenu(); } void plusZileAlarma(byte pos){ minusZileAlarma(pos); } void minusUnitAlarma(byte pos){ plusUnitAlarma(pos); } void plusUnitAlarma(byte pos){ if(subMenuPos==2){ alarme[pos-1].unitAlarma=(alarme[pos-1].unitAlarma+1)%2; } } void minusDurataAlarma(byte pos){ if(alarme[pos-1].duration>0 && subMenuPos==1){ alarme[pos-1].duration--; }else if(subMenuPos==2){ minusUnitAlarma(pos); } showMenu(); } void plusDurataAlarma(byte pos){ if(subMenuPos==1){ alarme[pos-1].duration=(alarme[pos-1].duration+1)%240; }else if(subMenuPos==2){ plusUnitAlarma(pos); } showMenu(); } void saveValues(){ for(int i=0;i<NR_ALARME;i++){ EEPROM.write(i*5+1,(alarme[i]).hour | (alarme[i].unitAlarma << 6)); EEPROM.write(i*5+2,(alarme[i]).minute); EEPROM.write(i*5+3,(alarme[i]).days); EEPROM.write(i*5+4,(alarme[i]).duration); EEPROM.write(i*5+5,(alarme[i]).relee); } } void applyNewDateHour(){ if(dataDirthy || oraDirthy){ if(oraDirthy){ Serial.println("Avec l'heure"); setDateDs1307(time); }else{ //si l'heure n'a pas changé, nous la laisserons telle quelle Serial.println("Pas d'heure"); struct ds1307_time tmp; getDateDs1307(&tmp); tmp.dayOfMonth = time.dayOfMonth; tmp.dayOfWeek = time.dayOfWeek; tmp.month = time.month; tmp.year = time.year; setDateDs1307(tmp); } dataDirthy=false; oraDirthy = false; } saveValues(); } long valAlarmaMin(byte index){ return (alarme[index]).hour*3600L+(alarme[index]).minute*60L; } long valAlarmaMax(byte index){ byte multipl = (alarme[index]).unitAlarma?60:1; return (alarme[index]).hour*3600L+(alarme[index]).minute*60L+multipl*(alarme[index]).duration; } void checkAlarms(){ long valComp = time.hour*3600L+time.minute*60L+time.second; bAlarma=false; strcpy(ligne,getTime(time)); strcat(ligne,";A"); char cAl[4]; long timeToFinish=0,vAlMax=0; boolean tipTimp=0; byte statusRelee[7]; for(int i=0;i<7;i++){ statusRelee[i]=0; } for(int i=0;i<NR_ALARME;i++){ struct alarm alarma = alarme[i]; vAlMax=valAlarmaMax(i); if(valComp>=valAlarmaMin(i) && valComp<=vAlMax){ //comparam valorile minime si maxime ale fiecarei alarme cu timpul actual if((byte)ceil(pow(2,7-time.dayOfWeek)) & (alarma).days){ bAlarma=true; if((timeToFinish==0 && vAlMax-valComp>timeToFinish) || (vAlMax-valComp<timeToFinish)){//setam durata pana la sf primei alarme tipTimp = alarma.unitAlarma; if(tipTimp && vAlMax-valComp>=60){ timeToFinish=(vAlMax-valComp)/60; }else{ timeToFinish=vAlMax-valComp; tipTimp=0; } } for(int j=6;j>=0;j--){ if((alarma).relee & (byte)(ceil(pow(2,j)))){ statusRelee[j]=statusRelee[j] | 1; } } itoa((i+1),cAl,10); if(strlen(ligne)<11){ strcat(ligne,cAl); } } } } for(int i=0;i<7;i++){ if(statusRelee[i]){ digitalWrite(relee[i],INCHIS); }else{ digitalWrite(relee[i],DESCHIS); } } if(bAlarma){ itoa(timeToFinish,cAl,10); strcat(ligne," "); strcat(ligne,cAl); if(tipTimp){ strcat(ligne,"m"); }else{ strcat(ligne,"s"); } } } void loop(){ //si nous cliquons sur le menu, nous affichons ce dont nous avons besoin et nous n'affichons plus la date et l'heure actuelles if(menuPos>0){ lcd.clear(); lcd.setCursor(0,0); lcd.print(ligne1); lcd.setCursor(0,1); lcd.print(ligne2); if(digitalRead(PIN_STG)==LOW && menuPos){ //avem apasat minus delay(100); if(subMenuPos){ if(menuPos==1){ //data minusData(); }else if(menuPos==2){ minusOra(); }else if((menuPos+1)%4==0){ minusAlarma((menuPos+1)/4); }else if((menuPos)%4==0){ //Jour alarme minusZileAlarma((menuPos)/4); }else if((menuPos-1)%4==0){ //Duree alarme minusDurataAlarma((menuPos-1)/4); }else if((menuPos-2)%4==0){ //Relais alarme minusReleeAlarma((menuPos-2)/4); } } else{ if(menuPos==1){ applyNewDateHour(); } menuPos--; showMenu(); } }else if(digitalRead(PIN_DR)==LOW && menuPos){ //nous avons appuyé sur plus delay(200); if(subMenuPos){ if(menuPos==1){ //data plusData(); }else if(menuPos==2){ plusOra(); }else if((menuPos+1)%4==0){ //heure d'alarme plusAlarma((menuPos+1)/4); }else if((menuPos)%4==0){ //jours d'alarme plusZileAlarma((menuPos)/4); }else if((menuPos-1)%4==0){ //durée de l'alarme plusDurataAlarma((menuPos-1)/4); }else if((menuPos-2)%4==0){ // relais de l'alarme plusReleeAlarma((menuPos-2)/4); } } else if(menuPos<MAX_MENU_POS-1){ ++menuPos; Serial.print("Menu pos: "); Serial.print(menuPos); showMenu(); }else{ applyNewDateHour(); menuPos=0; } }else if(subMenuPos>0 && cursorPos>-1){ // Serial.println(cursorPos); lcd.cursor(); lcd.setCursor(cursorPos,1); }else{ lcd.noCursor(); } delay(100); return; } lcd.clear(); char buf[3]; getDateDs1307(&time); Serial.print(getTime(time)); Serial.print(" "); Serial.print(getDate(time)); Serial.print(":"); Serial.print("Jour de la semaine : "); Serial.println(time.dayOfWeek, DEC); lcd.setCursor(0, 0); lcd.print(getZiuaSaptamanii(time.dayOfWeek)); lcd.setCursor(8,0); lcd.print(getDate(time)); if(bAlarma){ lcd.setCursor(0,1); lcd.print(ligne); }else{ lcd.setCursor(4,1); lcd.print(getTime(time)); lcd.setCursor(12,1); lcd.print(" "); lcd.noCursor(); } checkAlarms(); delay(1000); }

---

Bonjour,

J'ai regardé ce code. S'il tombe en marche c'est que programmer n'est pas un métier.

Au hasard :

• Méconnaissance des opérateurs binaires : alarme[pos-1].days = alarme[pos-1].days ^ (byte)ceil(pow(2,(7-subMenuPos))); au lieu de alarme[pos-1].days ^= 1 << (7-subMenuPos); qui, il est vrai, nous prive de deux calculs flottants 8-)
• Des break après des return (on ne sait jamais des fois qu'il passerait en dessous :D).
• Des modulos partout y compris à 240 sur un byte. Pourtant j'aurais juré qu'un simple test >= 240 aurait suffit car le max 255 ne tient qu'une seule fois 240.
• Les conversions en chaînes de caractères nécessitent beaucoup de ligne de code pour un résultat peu efficace là où des tableaux constants suffiraient.
• etc.

Pour le problème rencontré, à quoi sert la ligne alarme[pos-1]; ? Pourquoi ce pos - 1 systématique ? On remarque que l'incrémentation est cyclique (modulo oblige) : 22h, 23h, 0h, 1h ... mais pas la décrémentation : 2h, 1h, 0h, 0h, 0h... Ceci n'explique cependant pas la séquence rencontrée.
Je propose néanmoins :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

void minusAlarma(byte pos){ if( subMenuPos == 1) alarme[pos-1].hour -= alarme[pos-1].hour ? 1 : -23; else if(subMenuPos == 2) alarme[pos-1].minute -= alarme[pos-1].minute ? 1 : -59; showMenu(); }   void plusAlarma(byte pos) { if( subMenuPos == 1) alarme[pos-1].hour += alarme[pos-1].hour < 23 ? 1 : -23; else if(subMenuPos == 2) alarme[pos-1].minute += alarme[pos-1].minute < 59 ? 1 : -59; showMenu(); }

---

Je ne comprends pas la fonction menu(). D'une part l'anti-rebond est très long ( > 150 ms). D'autre part, le premier test réussit si menuPos est à 0 ce signifie que menuPos sera mis à 1 systématiquement (on se demande pourquoi le modulo).

Salutations

Merci Guesset
d'avoir pris le temps de regarder

Bon, il semblerait que çà ne soit pas gagné au vu des remarques formulées et judicieuses
il est vrai que mettre des break après des return , il fallait oser mais ne dit-on pas ceinture et bretelle? :D

malheureusement mon problème reste entier mais avec quelques progrès
la piste "void minusAlarma()" et "void plusAlarma()" n'est pas la bonne car l'incrémentation semble bien se faire

mais l'affichage à partir de showmenu() que la position et la valeur sont décalées
l'emploi du sous-programme :

Code:

---

1 2 3 4 5 6 7

char* getFmt2Char(byte val){ char *buf = "00\0"; buf[0] = 48+val/10; buf[1] = 48+val%10; return buf; }

---

en est peut-être la cause
je continue ..............

Bonjour Cobra38,

Citation:

---

Envoyé par cobra38

... showmenu() que la position et la valeur sont décalées

Code:

---

1 2 3 4 5 6

char* getFmt2Char(byte val){ char *buf = "00\0"; buf[0] = 48+val/10; buf[1] = 48+val%10; return buf; }

---

...

---

J'avais regardé showmenu() mais ça m'a fait peur :mouarf:

La fonction sprintf n'existe pas pour ce MCU ?

Le code de getFmt2Char(byte val) peut également concourir au jeu des erreurs de base. Renvoyer un pointeur sur des données sur la pile, donc non pérennes, est un grand classique. Le pire est que cela peut marcher car si les données ne sont plus protégées (dans une zone réaffectable de la pile), elles peuvent (ou pas) rester valides un (in)certain temps.

Je proposerais donc un truc du genre :

Code:

---

1 2 3 4 5

void Byte2Chars(byte v, char * p) { byte v10 = v / 10; *p = '0' + v10; // Améliorable en *p = v ? '0' + v10 : ' '; si on ne veut pas de 0 non significatif *(p+1) = '0' + v - v10 * 10; }

---

J'évite d'avoir deux divisions en calculant le modulo 10 avec une multiplication. Sur un MCU sans division native le gain de vitesse est proche de 2x. Si on veut introduire (écrasement) les deux caractères correspondant à une variable byte v, par exemple en position 6, dans un tableau de chars ligne[], il suffit d'écrire Byte2Chars(v, ligne + 6); Bien sûr, il est préférable que cet espace de 2 caractères soit légitime : entre 0 et length - 2 inclus de ligne. Naturellement, v ne doit pas dépasser 99. Les transtypages byte/char pourraient être explicités pour faire plus beau 8-)

Il y a du boulot.

Salut

Merci Guesset

mon problème semble insoluble, je n'arrive pas à comprendre le but de l'auteur
en ce qui concerne la mise en mémoire des valeurs d'heures et de minutes
pourtant tout le reste semble fonctionner sauf çà

je me demande si le fait d'avoir mis un LCD I2C à la place d'un simple LCD ne vient perturber les valeurs en mémoire
je vais donc essayer avec un LCD normal

Bonjour à tous

avant de tout refaire j'ai essayé çà ligne 24 :

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

void showMenu(){ char *zile[7]; zile[0]="L\0"; zile[1]="M\0"; zile[2]="M\0"; zile[3]="J\0"; zile[4]="V\0"; zile[5]="S\0"; zile[6]="D\0"; if(menuPos==1){ // Menu Date strcpy(ligne1,"Date\0"); strcpy(ligne2,getDate(time)); strcat(ligne2," "); strcat(ligne2,getZiuaSaptamaniiShort(time.dayOfWeek)); }else if(menuPos==2){ // Menu Heure strcpy(ligne1,"Heure\0"); strcpy(ligne2,getTime(time)); }else if((menuPos+1)%4==0){ // Menu Alarme Heure strcpy(ligne1,"Alarme "); ligne1[7] = 48+(byte)((menuPos+1)/4)/10; ligne1[8] = 48+(byte)(((menuPos+1)/4)-((menuPos+1)/4)/10*10); ligne1[9]='\0'; strcat(ligne1," Heure\0"); Serial.print(getFmt2Char((alarme[(menuPos+1)/4-1]).hour)); <============================== strcpy(ligne2,getFmt2Char((alarme[(menuPos+1)/4-1]).hour)); // Alarme Heure strcat(ligne2,":"); strcat(ligne2,getFmt2Char((alarme[(menuPos+1)/4-1]).minute)); // Alarme Minute strcat(ligne2," "); }else if((menuPos)%4==0){ // Jour de l'Alarme strcpy(ligne1,"Alarme "); ligne1[7] = 48+(byte)((menuPos)/4)/10; ligne1[8] = 48+(byte)((menuPos)/4-(menuPos)/4/10*10); ligne1[9]='\0'; strcat(ligne1," Jour\0"); strcpy(ligne2,"L"); for(int i=6;i>=0;i--){ if((alarme[(menuPos)/4-1]).days & (byte)ceil(pow(2,i))){ strcat(ligne2,"<"); }else{ strcat(ligne2," "); } if(i>0)strcat(ligne2,zile[7-i]); } }else if((menuPos-1)%4==0){ // Duree de l'Alarme strcpy(ligne1,"Alarme "); ligne1[7] = 48+(byte)((menuPos-1)/4)/10; ligne1[8] = 48+(byte)((menuPos-1)/4-(menuPos-1)/4/10*10); ligne1[9]='\0'; strcat(ligne1," duree\0"); itoa(alarme[(menuPos-1)/4-1].duration,ligne2,10); if(alarme[(menuPos-1)/4-1].unitAlarma){ strcat(ligne2," min"); }else{ strcat(ligne2," sec"); } }else if((menuPos-2)%4==0){ // Relais de l'Alarme strcpy(ligne1,"Relais Alarme"); ligne1[13]= 48+(byte)((menuPos-2)/4)/10; ligne1[14]= 48+(byte)((menuPos-2)/4-(menuPos-2)/4/10*10); ligne1[15]='\0'; strcpy(ligne2,""); for(int i=6;i>=0;i--){ char chBuff[2]; chBuff[0]=6-i+65; chBuff[1]='\0'; strcat(ligne2,chBuff); if((alarme[(menuPos-2)/4-1]).relee & (byte)ceil(pow(2,i))){ strcat(ligne2,"<"); }else{ strcat(ligne2," "); } //if(i>0)strcat(ligne2,zile[7-i]); } } }

---

peut-être que les dieux de l'informatique ont eu pitié mais maintenant çà fonctionne nickel ( pas dit çà marche ! )
mais je ne comprends pas pourquoi la mise en place de cette tempo à la lecture du buffer sur la fonction alarme()

Code:

---

1 2	alarm* alarme = (alarm*) malloc(sizeof(alarm) * NR_ALARME);

---

améliore le résultat ?

Citation:

---

Envoyé par cobra38

mon problème semble insoluble, je n'arrive pas à comprendre le but de l'auteur
en ce qui concerne la mise en mémoire des valeurs d'heures et de minutes

---

Qu'est ce que vous ne comprenez pas ?

sinon comme l'a dit @Guesset le code n'est vraiment pas bon. Il serait sans doute plus simple de repartir de zéro (et d'utiliser un encodeur rotatif avec bouton intégré au lieu de deux boutons - ça rend l'interface plus conviviale et rapide)

Bonjour Jay M

Oui il est sûr que Guesset a parfaitement raison mais avant de tout casser
je souhaitais comprendre comment le système fonctionnait mais à ma grande surprise en ajoutant une seule ligne "Serial.print()" ( au départ pour tester les erreurs)
tout est rentré dans l'ordre (?) un miracle ou un gros problème de gestion du temps dans le programme
toujours est-il que le prg fonctionne correctement et je cherche par exemple à y inclure des caractères CGRAM particulier et me retrouve à galérer avec les position des menus
par ex :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

byte Check[] = { B00000, B00001, B00011, B10110, B11100, B01000, B00000, B00000 };     void setup(){ ../.. lcd.createChar(0, Check);   }

---

en lieu et place de

Code:

---

1 2	//strcat(ligne2,"*");

---

je voulais mettre ceci

Code:

---

1 2	strcat(ligne2,lcd.write(byte(0)));

---

mais il m'efface le caractère spécial

Mais je suis bien conscient qu'il faille tout refaire et l'idéal serait comme vous l'avez décrit un bouton rotaif mais ceci dépasse très largement mes compétences en programmation

Citation:

---

Envoyé par cobra38

mais à ma grande surprise en ajoutant une seule ligne "Serial.print()" ( au départ pour tester les erreurs) tout est rentré dans l'ordre (?)

---

c'est une indication typique d'un programme qui a des souci de débordement mémoire ou d'une partie du code qui est "Undefined Behavior". Tout peut arriver.

pour votre question, ce code ne veut pas dire grand chose (si je peux me permettre).

Code:

---

strcat(ligne2,lcd.write(byte(0)));

---

on utilise strcat() quand on veut rajouter du texte à la fin d'une c-string
on utilise lcd.write() quand on veut envoyer un caractère à l'écran


si vous voulez imprimer un texte sur l'écran LCD qui contient des caractères spécifiques et spécialement le caractère qui a le code 0, vous ne pouvez plus le faire avec print car la fonction print va prendre ce 0 comme fin de la chaîne et ne pas l'envoyer à l'écran. C'est possible en utilisant write() avec le nombre d'octets à envoyer

voici un exemple dans le simulateur wokwi
https://wokwi.com/projects/372315691310542849

sinon l'alternative est de ne pas mettre votre caractère Check en position 0 parce que 0 a un rôle particulier dans les chaînes de caractères (il marque la fin)

Bonjour,

Jouer avec l'imprévisibilité d'un mauvais usage de la pile est toujours risqué. La moindre modification peut tout remettre en cause.

Pour changer a minima le code, il est possible de déclarer le char * buf = "00\0"; en variable globale (ie en dehors de la fonction) comme cela il ne disparaît pas à la sortie de la fonction. Ce n'est pas très propre mais la modification reste limitée.

Salutations

Citation:

---

Envoyé par cobra38

Mais je suis bien conscient qu'il faille tout refaire et l'idéal serait comme vous l'avez décrit un bouton rotaif mais ceci dépasse très largement mes compétences en programmation

---

voici un début de code qui met en jeu
- un encodeur rotatif et affiche un compteur (cptr) quand on "tourne" l'encodeur
- le bouton de l'encodeur qui traite l'appui / le relâchement et affiche un verrou ouvert ou fermé
- une RTC pour afficher l'heure
- un écran LCD en I2C

==> avec les bonnes bibliothèques c'est assez simple à coder

vous pouvez voir cela dans le simulateur :

https://wokwi.com/projects/372318521981383681

Pièce jointe 642251


ça vous donne les bases pour détecter le mouvement de l'encodeur ou l'appui sur le bouton et gérer le temps qui passe. Avec cela ce n'est pas super difficile de refaire un code qui permet de choisir une date etc...


voici le code pour mémoire.

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

#include <Wire.h> #include <hd44780.h> // main hd44780 header #include <hd44780ioClass/hd44780_I2Cexp.h> // i2c expander i/o class header #include <Encoder.h> // https://www.pjrc.com/teensy/td_libs_Encoder.html #include <Toggle.h> // https://github.com/Dlloydev/Toggle #include "RTClib.h" // https://github.com/adafruit/RTClib     const uint8_t nbColonnes = 20; const uint8_t nbLignes = 4; hd44780_I2Cexp lcd;   RTC_DS1307 rtc; const char* jours[] = {"Dim", "Lun", "Mar", "Mer", "Jeu", "Ven", "Sam"}; const uint8_t nbJours = sizeof jours / sizeof * jours;     const byte encoderCLKPin = 2; const byte encoderDTPin = 3; Encoder encoder(encoderDTPin, encoderCLKPin);   const byte encoderSWPin = 4; Toggle encoderSwitch(encoderSWPin);   const char lockBitmap[] PROGMEM = {0b01110, 0b10001, 0b10001, 0b11111, 0b11011, 0b11011, 0b11111, 0b00000}; const uint8_t lock = 0;   const char unlockBitmap[] PROGMEM = {0b01110, 0b10000, 0b10000, 0b11111, 0b11011, 0b11011, 0b11111, 0b00000}; const uint8_t unlock = 1;   const char OKBitmap[] PROGMEM = {0b00000, 0b00001, 0b00011, 0b10110, 0b11100, 0b01000, 0b00000, 0b00000}; const uint8_t OK = 2;   const char clocheBitmap[] PROGMEM = {0b00100, 0b01110, 0b01110, 0b01110, 0b11111, 0b00000, 0b00100, 0b00000}; const uint8_t cloche = 3;   long position = 0;     void afficheDate() { static DateTime avant(1900, 0, 0, 0, 0, 0); DateTime maintenant = rtc.now(); if (maintenant != avant) { avant = maintenant; lcd.setCursor(0, 0); lcd.print(jours[maintenant.dayOfTheWeek()]); lcd.write(' '); if (maintenant.day() < 10) lcd.write('0'); lcd.print(maintenant.day()); lcd.write('/'); if (maintenant.month() < 10) lcd.write('0'); lcd.print(maintenant.month()); lcd.write('/'); if (maintenant.year() - 2000 < 10) lcd.write('0'); lcd.print(maintenant.year() - 2000); lcd.write(' '); lcd.setCursor(nbColonnes - 1, 0); lcd.write(cloche); lcd.setCursor(nbColonnes - 8, 1); if (maintenant.hour() < 10) lcd.write(' '); lcd.print(maintenant.hour()); lcd.write(':'); if (maintenant.minute() < 10) lcd.write('0'); lcd.print(maintenant.minute()); lcd.write(':'); if (maintenant.second() < 10) lcd.write('0'); lcd.print(maintenant.second()); }   }     bool encoderChanged() { long nouvellePosition = encoder.read() >> 2; // my encoder generates 4 ticks per click so I divide by 4 if (nouvellePosition != position) { position = nouvellePosition; return true; } return false; }   void setup() { encoderSwitch.begin(encoderSWPin); Serial.begin(115200); Serial.println();   int result = lcd.begin(nbColonnes, nbLignes); if (result) { Serial.print("LCD initialization failed: "); Serial.println(result); hd44780::fatalError(result); }   lcd.createChar(lock, lockBitmap); lcd.createChar(unlock, unlockBitmap); lcd.createChar(OK, OKBitmap); lcd.createChar(cloche, clocheBitmap);   if (! rtc.begin()) { Serial.println("Horloge absente"); Serial.flush(); abort(); }   if (! rtc.isrunning()) { rtc.adjust(DateTime(F(__DATE__), F(__TIME__))); }   lcd.setCursor(0, 2); lcd.print("cptr : "); lcd.print(position); lcd.setCursor(0, 3); lcd.print("etat : "); lcd.write(lock); Serial.println(F("Ready")); }   void loop() {   afficheDate();   if (encoderChanged()) { lcd.setCursor(7, 2); for (byte i = 7; i < nbColonnes; i++) lcd.write(' '); // on efface lcd.setCursor(7, 2); lcd.print(position); }   encoderSwitch.poll(); if (encoderSwitch.onPress()) { lcd.setCursor(7, 3); lcd.write(unlock); } if (encoderSwitch.onRelease()) { lcd.setCursor(7, 3); lcd.write(lock); }   }

---

merci à vous deux Jay M et Guesset pour vos précieux conseils

pour Jay M

j'ai effectivement déclaré :
lcd.createChar(7, Check); en lieu et place de lcd.createChar(0, Check);
et çà va beaucoup mieux , le byte (0) étant semble-t-il à bannir
merci encore

A noter que j'ai voulu mettre sur woki le programme pour une meilleure explication mais que je me suis heurté au problème des boutons, aucun ne répondaient (?)
faut-il mettre des "delay(200)" pour la simulation ?

pour Guesset
j'ai noté la déclaration en global je vais essayer

Citation:

---

Envoyé par Jay M

.../....

---

çà m'a l'air vraiment plus super que mon truc (!)
je le mets de coté bien précieusement et je vais tout reprendre à zéro
et clôturer donc mon post puisque le problème demandé est résolu


un grand merci
Bien Cordialement et Bonne Journée
pascal

Citation:

---

Envoyé par cobra38

A noter que j'ai voulu mettre sur woki le programme pour une meilleure explication mais que je me suis heurté au problème des boutons, aucun ne répondaient (?)
faut-il mettre des "delay(200)" pour la simulation ?

---

Non dans l’absolu il faut éviter au maximum les délais quand on veut une interface utilisateur agréable.
Avez vous bien câblé le bouton?
Quel code?
Avez vous utilisé une bibliothèque?