Les Électroniciens.com

Intéressant Walter.

Bonjour à quoi servent les cellules RC sur les BP ?

Puis j'ai essayé de compiler le code et je ne comprends pas pourquoi il y a au moins une dizaine de problèmes

• #include <ds3231.h> pose problème il faut remplacer cette ligne par #include <DS3231.h>
• les structures "  struct ts heure;" ne sont pas définies
• idem pour "struct ts chronoStop;"
• etc...

Tu n'as aucun problème de compilation chez toi ?

Et pour finir pourquoi autant de complications pour un code qui pourrait être somme toute être beaucoup plus simple ?

Si tu pouvais expliquer la logique que tu as utilisée se serait vraiment intéressant.

Bonjour,

Bon boulot,

tu as un but précis derrière la tête , a quoi il va servir .

tu vas le connecter a un réseau ?

Pour ce qui est de ton code ceux ce sont pas quelques points qui sont à optimiser mais à le réécrire entièrement, je ne sais pas qui est à l'origine de ce code, mais je ne vois pas trop comment on aurait faire plus compliqué.

Je n'ai pas le temps pour l'instant de le faire, mais je voudrais juste te faire quelques remarques.

• Tu utilises des interruptions qui ne servent à rien puisque tu ne fais que changer la valeur d'une variable que tu ne prends en compte que plus loin dans le code.
• Tu as un somme incroyables de variables pour quoi faire ?
• La sélection des différents modes est très loin d'être efficace.
• Le système anti rebond comme tu l'appelles n'est pas aussi efficace, ni aussi sûr que quelques éléments de code.
• Les fonctions sont trop longues et prennent trop de temps ce qui fait que tu ne peux pas avoir un réaction optimum pour l'action des boutons.
• Il y ad'autres points mais j'arrête là pour le moment

Tu prendras peut être mal ces critiques mais ce n'est pas pour te dénigrer, mais simplement pour te donner des pistes, il n'y a rien de méchant dans ces lignes.

Je pense que j'aurais un moment dans la journée pour réécrire un code plus adapté à ce que tu recherches. Mais dis moi c'est  vraiment toi qui est à l'origine de ce code ?

Désolé, je suis un peux surbooké en ce moment.

Comme je n'ai pas le même afficheur que toi peux tu me décrire le fonctionnement exacte de ton projet.

Je ferais le code avec un LCD I2C en utilisant que  5 digits (dont un pour " : ", tu devras l'adapter à ton afficheur que je ne connais pas.

Voilà mon code, il fait 58 lignes de moins, je le trouve plus clair, mais c'est juste mon avis car en plus j'utilise au maximum le français " COCORICO ! " :o)

Tu remarqueras des astuces de programmeur pour ajouter ou soustraire une heure ou une minute à la pendule ! Dans beaucoup de cas la fonction modulo ( % ) fait gagner beaucoup de temps et  de ligne de code !

Je me suis permis un changement en mode Chrono le BpMoins permet de remettre le compteur à 0 et BpPlus permet de retrouver le chrono Initial,

Comme je te l'ai annoncé j'utilise qu matériel différent :

• Carte Arduino UNO
• Schield RTC directement embrochable sur l'Arduino
• Un écran LCD I2C

Je ne trouve pas dans ce projet la nécessité d'utiliser les interruptions, je réserve ce genre de code à les besoin dévolutions aussi rapide de soudaines !

Si tu veux je peux te faire une vidéo et même commenter mon code.

#include <DS3231.h>
#include <Wire.h>
#include <LiquidCrystal_I2C.h>
LiquidCrystal_I2C lcd(0x27, 16, 2);
#include "DHT.h"
#define DHTPIN 11
#define DHTTYPE DHT11
DHT dht(DHTPIN, DHTTYPE);
#define BpMode  8
#define BpPlus  9
#define BpMoins 10
DS3231  RTC(SDA, SCL);
Time T;
unsigned long TIME, sTIME = millis() + 5000, Chrono, TopChrono;
char HEURE, MINUTE, SECONDE;
String sHeure;
int Mode = 0, M, S ;
bool Etat = true, Encore;
void setup() {
  Serial.begin(1000000);
  RTC.begin();
  //RTC.setTime(8, 44, 0);    // Set the time to 12:00:00 (24hr format)
  lcd.init();
  lcd.backlight();
  pinMode(BpMode  , INPUT_PULLUP);
  pinMode(BpPlus  , INPUT_PULLUP);
  pinMode(BpMoins , INPUT_PULLUP);
  TopChrono = millis();
  lcd.setCursor(4, 1);
  lcd.print("Mode : ");
  lcd.print(Mode);
}

void loop() {

  if (!digitalRead(BpMode)) {
    Mode ++ ;
    TIME = millis() + 1000;
    sTIME = millis() + 6000;
    Mode = Mode % 4;
    Serial.println(Mode);
    Etat = 1;
    lcd.clear();
    lcd.setCursor(4, 1);
    lcd.print("Mode : ");
    lcd.print(Mode);
    while (!digitalRead(BpMode)) {
      delay(20);
    }
  }

  switch (Mode) {
    case 0:
      if (TIME < millis()) {
        TIME = millis() + 1000;
        if (Etat)
          LitHeure();
        else
          LitTemperature();
        if (sTIME < millis()) {
          sTIME = millis() + 6000;
          Etat = !Etat;
        }
        break;

      case 1:
        if (Etat)
          LitChrono();
        else if (millis() % 1000 < 100)
          LitTemperature();

        if (sTIME < millis()) {
          sTIME = millis() + 5000;
          Etat = !Etat;
        }
      }
      break;
    case 2:
      ChangeHeure();
      break;
    case 3:
      ChangeMinute();
      break;
  }
}

void LitTemperature () {
  float t = dht.readTemperature();
  String Tmp = String(t, 4);
  lcd.setCursor(4, 0);
  lcd.print(Tmp);
  lcd.setCursor(8, 0);
  lcd.print("C ");

}

void LitHeure() {
  T  = RTC.getTime();
  SECONDE = T.sec;
  lcd.setCursor(6, 0);
  HEURE   = T.hour;
  MINUTE  = T.min;
  lcd.setCursor(4, 0);
  if (HEURE < 10)
    lcd.print(0);
  lcd.print(HEURE, DEC);
  if (SECONDE % 2 == 0)
    lcd.print(":");
  else
    lcd.print(" ");
  lcd.setCursor(7, 0);
  if (MINUTE < 10)
    lcd.print(0);
  lcd.print(MINUTE, DEC);
}
void LitChrono() {
  Chrono  = (millis() - TopChrono) / 1000;
  S = Chrono % 60;
  M = (Chrono % 3600) / 100;
  lcd.setCursor(4, 0);
  if (M == 0)
    lcd.print(" ");
  lcd.print(M);
  lcd.print(":");
  if (S < 10)
    lcd.print(0);
  lcd.print(S);
  if (!digitalRead(BpMoins))
    TopChrono = millis();
  if (!digitalRead(BpPlus))
    TopChrono = 0;
}

void ChangeHeure() {
  if (TIME < millis()) {
    LitHeure();
    TIME = millis() + 1000;
  }
  if (TIME < millis() + 500) {
    lcd.setCursor(4, 0);
    lcd.print("  ");
  }
  if (!digitalRead(BpPlus)) {
    RTC.setTime((HEURE + 1) % 24, MINUTE, 0);
  }
  if (!digitalRead(BpMoins)) {
    RTC.setTime((HEURE + 23) % 24, MINUTE, 0);
  }
  delay(100);
}

void ChangeMinute() {
  if (TIME < millis()) {
    LitHeure();
    TIME = millis() + 1000;
  }
  if (TIME < millis() + 500) {
    lcd.setCursor(7, 0);
    lcd.print("  ");
  }
  if (!digitalRead(BpPlus)) {
    RTC.setTime(HEURE, (MINUTE + 1) % 60 , 0);
  }
  if (!digitalRead(BpMoins)) {
    RTC.setTime(HEURE, (MINUTE + 59) % 60 , 0);
  }
  delay(100);
}

Tu dis que tu code exclusivement en Anglais pour le partage avec des anglophones, mais pourquoi ne partages tu rien ici ? (même en Anglais) ou peut être peux-tu nous donner le lien du site sur lequel tu partage tes codes !

Pour ce qui est des interruptions tant qu'à les utiliser, il faut le faire avec plus d'efficacité je pense. Pourquoi lancer toutes les interruptions au début du code, alors qu'à cet instant, seul le changement de mode est nécessaire. En plus si tu actionnes les Bp qui n'ont pas encore de raison de l'être au changement de mode tu risques d'avoir des modifications non voulues. Enfin si tu utilises des interruptions comme plus est moins elles devraient être attachées à des fonctions spécifiques à la position dans ton code ce qui te permettrait d'être beaucoup plus efficace, mais c'est juste mo avis.

Pour le temps passé à ma façon de coder de rien c'est avec plaisir !

J'ai commandé le même afficheur que toi (c'est toujours intéressant de voir d'autre système) et si j'ai un moment je te donnerai ma solution qu'en à l'utilisation des interruptions selon mon idée.

Jetfuzz je ne comprends que ne sois pas venu mettre le boxon ici ?

Jetfuzz je ne comprends pas que tu ne sois pas venu mettre le boxon ici ?

et c'est qui qui cherche a foutre le boxon !!!!!

parce que jusque la, tu as été agréable !!!
et que le code c'est pas mon truc.
pourvu que ca dure.

Cordialement

Jetfuzz

J'adore tu démarres au quart de tour je connais tes adresses IP et je sais que tu es connecté pratiquement tout le temps mais comme tes agressions manquent un peu de temps en temps je les relances de temps à autres, ça marche au top !

Oui chez toi ....

Si on compare le nombre de mes posts constructifs et tes posts pseudo constructifs, comme dit mon gamin il n'y a pas photo, 80% au moins des tiens sont agressifs même quand je ny réponds pas !

Regarde sans parti pris ce que j'apporte ici et ce que toi tu donnes ! De plus je sais que tu regardes tous mes tutos !

Walter où as-tu pris le fait que tu ne partages rien ? Il est vrai que j'ai dit qu'ici je n'ai pas vu grand-chose dans ces pages. Mais comme tu affirmes je cite :

Par habitude de partagé mon code avec des anglophones, je code ou commente en utilisant uniquement l'anglais, même la documentation :)

Je n'ai trouver que cela :

"https://www.les-electroniciens.com/discussions/petite-sonde-temperature-humidite"
J'ai dû mal cherché car je n'ai pas trouvé le code et les commentaires dont tu nous parles ... J'ai pourtant bien fouillé. Mais je suis certainement passé à coté !

Je pense donc que tu partages tes projets ailleurs avec tes amis anglophones puisqu’ici je n'ai pas vu d'anglophone participé à tes créations ... Ce serait sympa comme je te le disais que tu partages aussi le site sur lequel vous échangez. Donc je réitère ma demande.

Tu dis encore que je n'ai rien publié; ???????. C'est vrai que dans cette catégorie je n'ai pas créer de sujet, j'ai préféré les poster sous forme de tutos, il y en a plus d'une douzaine il me semble, sans compter ce que j'ai donné dans d'autres catégories sous forme d'aide.

Tu me traites de menteur, puis je savoir quels mensonges ai-je fait ?

Bien revenons à ce sujet, Walter je viens de recevoir l'afficheur, il faut reconnaître qu'en choisissant ce module tu n'as pas choisi la facilité.

Je reconnais avoir mis un certain temps pour arrivé obtenir l'affichage que je voulais.

Sur le net je n'ai trouvé aucune explication sur les commandes de la library, juste des exemples n'expliquant pas les paramètres des diverses fonctions. Je pense que tu as dû en bavé autant que moi, autrement un grand bravo!, et alors je suis complètement nul.

Maintenant, dans un premier temps je vais faire exactement ton montage, comme toi, avec ton code pour que je puisse voir comment il réagi et par la suite pouvoir coller au maximum à ta réalisation.

Puis j'essayerai de coder ce montage à ma manière avec le même matériel dans le but que tu puisses comparer, nos différences dans la manière de coder comme tu dis trouver cela intéressant.

Bonjour,

Toujours pas de modérateur sur ce forum ? Walter tu n'étais pas intéressé par la fonction ? C'est lourd de voir tous les posts finir sur le même ton, et ca n'avance a rien a part faire fuir les gens matures...

Alex ? tu vois bien qu'il n'y à pas moyen. Il y a  une tumeur sur le forum.

Au point ou nous en sommes,même un mauvais modérateur serait moins pire qu'aucun modérateur.

Walter plus besoin de document sur la library du TM1637 j'ai fait ma fonction qui permet d'afficher n'importe quelle valeur sur cette afficheur et d'une manière très simple

Exemple :

DigitAff("12:48"), DigitAff("__:36) ou encore DigitAff("24°C") etc... Si tu es intéressé fait moi le savoir !

Je n'ai pas encore testé ton code je étais sur l'affichage digital.

J'ai fini par trouver le manuel de la library TM1637Display  ici robojax-tm1637_display_manual.rar

Mais je te donne ma fonction sous TM1637Display.h qui, pour moi tout du moins, est beaucoup plus simple à utiliser.

Il suffit de créer une Chaîne de 5 caractères au choix dans "0123456789ABCDEF:°-_" puis

1. On remplace un espace par "_" (Plus facile à visualiser que " ")
2. Le 3 ème caractère ( obligatoire ) doit être ":" si on veut afficher ce séparateur, tout autre caractère est ignoré, le mieux est de le remplace par "_" ou même un espace
3. On appelle la fonction DigitAff( paramètre ); // Avec la Chaîne voulue comme paramètre
4. Au besoin il est possible de rajouter des caractères supplémentaires dans la fonctions. exemple pour "H":

5.  String Alpha = "0123456789ABCDEF:*-_H"
   et

6. Ajouter un élément a Car[28] par Car[29]

7.   byte Car[29] = {63, 6, 91, 79, 102, 109, 125, 7, 127, 111, 119, 124, 57, 94, 121, 113, 128, 99, 64,0, 118}; //le code décimale des segments BCEFG soit (0 + 2 + 8 + 0 + 16 + 32 + 64  =118

#include <TM1637Display.h>
// Instantiation and pins configurations
// Pin 2 - > DIO
// Pin 3 - > CLK
int Delais = 1000;
TM1637Display display(2, 3);

void setup()
{

// Exemples :
  display.setBrightness(5);

  for (int n = 0; n < 10; n++) {
    DigitAff(String(n) + "____");
    delay(300);
    DigitAff("-" + String(n) + "___");
    delay(300);
    DigitAff("__-" + String(n) + "__");
    delay(300);
    DigitAff("___-" + String(n) );
    delay(600);
  }

  DigitAff("12:45");
  delay(Delais);
  DigitAff("01:00");
  delay(Delais);
  DigitAff("24 °F");
  delay(Delais);
  DigitAff("94__A");
  delay(Delais);
  DigitAff("64_-B");
  delay(Delais);
  DigitAff("-4_°C");
  delay(Delais);
  DigitAff("88:88");  // Pour tester tous les segments
 delay(Delais);
DigitAff("_____"); // Pour éteindre !!
}

void loop() {}

void DigitAff(String Aff) {
  byte Car[29] = {63, 6, 91, 79, 102, 109, 125, 7, 127, 111, 119, 124, 57, 94, 121, 113, 128, 99, 64, 0};
  String Alpha = "0123456789ABCDEF:*-_"; Aff.replace("°","*");
  byte D[4];
  D[0] = Car[Alpha.indexOf(Aff.substring(0, 1))];
  D[1] = Car[Alpha.indexOf(Aff.substring(1, 2))];if(Aff.substring(2,3)==":")D[1]+=128;
  D[2] = Car[Alpha.indexOf(Aff.substring(3, 4))];
  D[3] = Car[Alpha.indexOf(Aff.substring(4, 5))];
  byte data[] = {D[0], D[1], D[2], D[3]};
  display.setSegments(data);
}

Tu as bien fait toute amélioration est bonne à prendre !

Par contre j'ai abandonné ce que j'avais prévu de faire avec les interruptions, je ne comprends pas pourquoi quand j'appuie sur un bouton avec l'interruption "1" c'est de temps en temps l'interruption "2" ou "3" qui est déclenchée donc comme ce n'est pas fiable j'abandonne.

Ca ma fichu les boules j'ai tout effacé !

Je reprendrai peut être dans quelques temps !

J'ai jeté un oeil sur ta modif, mais je vois que tu ne cherches pas la simplicité, pourquoi aller bricoler avec les adresses mémoires ?

Si tu ne te fâches pas je vais essayer de te proposer ma solution à ton afficheur (si j'y arrive, en ce moment rien est écrit )

Non je ne parle pas de bricolage, j'aurais dû employer le mot utiliser (j'ai dit bricolage par ce que ce n'est pas à la portée de tout le monde de comprendre )

Voilà ma solution pour ton afficheur (Merci de le tester et de me donner le résultat de ton test) par contre il ne faut plus rien mettre en lieu et place des ":" ou "."  s'il ne doivent pas être affichés.

J'ai aussi rajouté des caractères dont j'avais besoin pour une autre appli !

Enfin j'ai testé ton code sur mon afficheur, j'obtiens des "r" à la place des "_" et tu as mis les deux premières lignes et dehors de la fonction !

byte Car[29] = {63, 6, 91, 79, 102, 109, 125, 7, 127, 111, 119, 124, 57, 94, 121,
113, 128, 99, 64, 0};
String Alpha = "0123456789ABCDEF:*- .";

J'ai trouvé le problème il vient du fait que tu utilises des 'espace' et moi des 'underscore' pour effacer les caractères donc pas de problème après modif !

Voilà ma solution pour ton afficheur (Merci de le tester et de me donner le résultat de ton test) par contre il ne faut plus rien mettre en lieu et place des ":" ou "."  s'il ne doivent pas être affichés apr rapport à mon premier code.

#include <TM1637Display.h>
// Instantiation and pins configurations
// Pin 2 - > DIO
// Pin 3 - > CLK
int Delais = 1000;
TM1637Display display(2, 3);

void setup()
{

  // Exemples :
  display.setBrightness(5);
  for (int n = 0; n < 11; n++) {
    DigitAff(String(n) + "___");
    delay(100);
    DigitAff("-" + String(n) + "__");
    delay(100);
    DigitAff("_-" + String(n) + "_");
    delay(100);
    DigitAff("___-" + String(n) );
    delay(200);

  }

  DigitAff("12:34");
  delay(Delais);
  DigitAff("5678");
  delay(Delais);
  DigitAff("90AB");
  delay(Delais);
  DigitAff("CDEF");
  delay(Delais);
  DigitAff("6420");
  delay(Delais);
  DigitAff("-4°C");
  delay(Delais);
  DigitAff("88:88");
  delay(200);
  DigitAff("8.888");
  delay(200);
  DigitAff("88:88");
  delay(200);
  DigitAff("888.8");
  delay(200);
  DigitAff("_HOT");
  delay(Delais);
  DigitAff("COOL");
  delay(Delais * 3);
  DigitAff("____");
}

void loop() {
}

//************************************************************
void DigitAff(String Aff) {
  byte Car[32] = {63, 6, 91, 79, 102, 109, 125, 7, 127, 111, 119, 124, 57, 94, 121, 113, 99, 64, 0, 118, 92, 56, 120, 128, 128};
  String Alpha = "0123456789ABCDEF*-_HOLT.: "; Aff.replace("°", "*");
  byte D[4] = {0, 0, 0, 0}, Digit = 3;
  int Caract = Aff.length() - 1;
  while (Caract >= 0) {
    D[Digit] += Car[Alpha.indexOf(Aff.substring(Caract, Caract + 1))];
    if (D[Digit] != 128)  Digit --;
    Caract--;
  }
  byte data[] = {D[0], D[1], D[2], D[3]};
  display.setSegments(data);
}

//************************************************************

Si tu mets plus de quatre caractères c'est normal que cela plante l'afficheur n'en a que quatre !

Tu peux utiliser le caractère qui te plait pour éteindre tous les segments, j'utilise les 'underscore' car, pour moi c'est plus facile à compter (visuellement) que des espaces.

Pour les variables c'est comme tu veux, je les ai incluses dans la fonction pour faire une fonction autonome voilà tout. le temps de rafraîchissement et négligeable !

J'ai trouvé le problème, tout fonctionne parfaitement et d'une manière très simple, mais vu ton changement d'attitude je ne diffuserai pas le code !

Le problème c'était chez moi qu'il avait lieu, comme je te l'ai dit.

Tout fonctionne chez toi : C'est parfait !

Tu sai très bien quel était mon problème sur les interruptions ! tu y as même fait allusion dans un de tes messages.

Tu n'as rien corrigé du tout, tu l'as adapté à ta façon. C'est à dire rendre transparentes les erreurs de codage, c'est complètement différent  ! Moi je préfère que ça plante ou que cela ne donne pas satisfaction, pour que l'on cherche et supprime les erreurs et ne pas laissé un code faux ce qui pour moi est digne d'un développeur du DIMANCHE. A chacun sa façon de voir les choses.

Il faut pensé industriel c'est à dire qu'une tierce personne peut intervenir sur ce code et ne plus rien comprendre.