Arduino Zeitschaltung von werten.



  • Guten Morgen,
    ihr beiden seit SUPER, echt toll dass ich euch die zeit nehmt und auch alles gut erläutert.
    Vielen Lieben Dank



  • @Ben
    Bitte 🙂 Das liegt nebenbei auch daran wie du die Frage hier gestellt hast. Also ein Beitrag mit

    • dem Code dabei den du schon hast
    • einer Beschreibung was er aktuell tut
    • einer Beschreibung was du anders haben möchtest
    • klar formulierten Fragen

    Und alles freundlich und nicht fordernd bzw. anmassend formuliert. Das hilft sehr freundliche und oft auch hilfreiche Antworten zu bekommen. Andere, speziell neue Forennutzer machen das z.T. sehr anders, und bekommen dann entsprechend auch sehr andere Antworten 🙂

    Ich hätte noch mal ne doofe Frage...
    Du hast in deinem Beitrag hier geschrieben

    Bei der Temperatursteuerung habe ich festgestellt, wenn diese erreicht wird, fängt die Heizung an sich ständig an und aus zu schallten

    Das impliziert dass du die Temperatursteuerung (Regelung) schon programmiert hast. Ich sehe in dem Code den du gepostet hast aber nix davon. Der scheint nur LEDs zu schalten. Oder ist die mittlere "LED" etwa ein Ausgang der die Heizung steuert?



  • Servus @hustbaer,
    da ich alles noch auf einem Steckbrett habe und ich später die Heizung über 230V steuern muss ( werden Heizmatten und Lampen), muss ich ja nach meinem wissen mit optokoppler Relais arbeiten. Statt den Relays habe ich einfach die LED eingebaut um eine optische Rückmeldung zu bekommen. Nach dem anstöpseln gefällt es mir aber nun so gut, dass ich wohl auch eine LED Anzeige realisieren werde. Dann sehe ich auf einen Blick was gerade passiert. Das selbe kommt dann noch mit der Luftfeuchtigkeit.

    Im Aufbau habe ich es so:

    LED Rot = zu warm = 2 Lüfter geht an
    LED Gelb = noch ok = 1 Lüfter geht an
    LED Grün = leicht erhöhte Temperater = keine Heizung, kein Lüfter
    LED Grün = Alles im soll = Normale Tagbeleuchtung ohne Heizung und Lüfter
    LED Grün = leicht zu niedrige Temperatur = 1 Wärmequelle geht an
    LED Gelb = noch kälter als bei grün = 2. Wärmequelle geht mit an Lüfter ist gesperrt
    LED Rot = zu kalt = Alle Wärmequellen gehen an und Lüfter sind gesperrt

    Ich hoffe du kannst mir so folgen und ich habe keinen kollosalen Denkfehler mit drin.

    Grüße
    Ben



  • @Ben
    Ja, ich kann dir folgen.
    Du musst dann entweder auch die LEDs mit Hysterese schalten. Was aber bei einer reinen Visualisierung unüblich ist.

    Oder du musst die LEDs getrennt von der Heizung/den Lüftern steuern.
    Dann brauchst du mehr Ausgänge. Und wenn du die Ausgänge in jedem Durchlauf schreiben willst wie in deinem Beispiel, dann musst du dir den aktuellen Zustand merken. Weil sonst das Schalten mit Hysterese eben nicht funktioniert.

    da ich alles noch auf einem Steckbrett habe und ich später die Heizung über 230V steuern muss ( werden Heizmatten und Lampen), muss ich ja nach meinem wissen mit optokoppler Relais arbeiten.

    Ob du das musst weiss ich nicht. Galvanische Trennung wäre angesagt, ja. Aber die solltest du auch mit einem normalen Relais bekommen. Optokoppler ist da glaube nicht muss. Allerdings wirst du ein normales Relais vermutlich nicht mit dem Arduino schalten können. Die Dinger ziehen ja ordentlich Strom.
    Gibt aber genügend Möglichkeiten. Optokoppler-Relais wie du schreibst oder Opto-Triac oder irgendsowas.

    Wobei ich dir da keine konkreten Tips geben kann. Da geht's um Strom und Heizung. Also Stromschlag und Haus abbrennen und so... das ist mir zu heikel, da hab ich zu wenig Plan von dem Thema 🙂

    Also diesbezüglich hol dir lieber Rat oder besser sogar Hilfe (vor Ort) von jemandem der sich damit wirklich auskennt.



  • Hallo @hustbaer,
    wollte euch gerade fragen ob ich da richtig liege dass dann so meine Steuerung nicht funktioniert. Also werde ich die Steuerung unterteilen in "Anzeige" und "Steuerung" dann müsste es ja klappen.

    Das Elektrische mache ich selber und lasse von einem Elektromeister drüberschauen. Bin auch am Überlegen Rauchmelder und eine temperaturabhängige Sicherung mit einzubauen, dass kommt aber erst später dran.

    Vielen Dank für deine Rückmeldung
    Grüße Ben

    PS: Noch eine Frage, kann ich bei der Hysterese auch angeben z.B. +2 == -1?



  • @Ben sagte in Arduino Zeitschaltung von werten.:

    PS: Noch eine Frage, kann ich bei der Hysterese auch angeben z.B. +2 == -1?

    Hä? Versteh ich nicht 🙂

    Aber egal was du damit meinst...
    Das kannst du dir doch ganz einfach selbst durchdenken bzw. auch selbst aussuchen wie du es genau angeben/einstellen willst.

    Du brauchst einen Schwellwert für's Ein- und einen Schwellwert für's Ausschalten.
    Du kannst beide Werte direkt angeben.
    Oder du kannst die Temperatur dazwischen und ein Delta angeben und dann die eine Schwelle ausrechnen mit x = temp + delta und die andere mit y = temp - delta.
    Oder du kannst eine der beiden Temperaturen angeben (z.B. die höhere) und dazu die Breite des Bereichs. Und die 2. Temperatur rechnest du dann z.B. mit y = x - width.

    Im Endeffekt muss ein unterer Schwellwert rauskommen bei dessen Unterschreitung du die Heizung einschaltest und ein oberer Schwellwert bei dessen Überschreitung du die Heizung ausschaltest.



  • Stimmt, danke. Da hat wohl was ausgesetzt bei mir.
    Schönes Wochenende noch
    Grüße
    Ben



  • So,
    muss mich bei euch mal wieder melden.
    @hustbaer : Dein Code mit der Hysterese funktioniert, leider verstehe ich ihn nicht wirklich. Daher habe ich mal was versucht was mir klar erschien. Hoffe es geht so auch.

    Schaut euch doch bitte mal Zeile an, dort steht alles drin.
    Den nächsten Schritt den ich versuchen werde (hoffe ohne fremde Hilfe) verschiedene Heizungen (4Stk.) nach Temperaturunterschieden zu steuern. Mal sehen ob das klappt.

    int Tempsteuerung()
    {
      // Temperatursensoren lesen
      int Temperatur1 = Sensor1.readTemperature();//Temperatur Sensor1 auslesen
      int Temperatur2 = Sensor2.readTemperature();//Temperatur sensor2 auslesen
      int Durchschnitttemp = ( Temperatur1 + Temperatur2 ) / 2; // Durchscnittstemperatur berechnen
    
      // Solltemperatur ueber Potentiometer einstellen fuer Tag
      int einstellungtemptag = analogRead(15); //lesen des Potis mit der Solltemperatur für Tag
      int solltemptag = map(einstellungtemptag, 0, 1022, 10, 30); // festlegung des Regelbereiches der Temperatur Tag
      // Solltemperatur ueber Potentiometer einstellen fuer Nacht
      int einstellungtempnacht = analogRead(14); //lesen des Potis mit der Solltemperatur für Nacht
      int solltempnacht = map(einstellungtempnacht, 0, 1022, 10, 25); // festlegen des Reglerbereiches der Temperatur Nacht
    
      Serial.println(); // Zeilenumbruch
      Serial.print("Solltemperatur Tag: ");
      Serial.print(solltemptag);                                        //diesen Wert möchte ich einstellen können und er soll sich zu einer bestimmten (einstellbaren Zeit) auf "solltempnacht" umstellen z.B. 20:00Uhr.
      Serial.println(" Grad Celsius"); //Einheit nach dem Wert
      Serial.print("Solltemperatur Nacht: ");
      Serial.print(solltempnacht);                                      //diesen Wert möchte ich einstellen können und er soll sich zu einer bestimmten (einstellbaren Zeit) auf "solltemptag" umstellen z.B. 07:00Ihr.
      Serial.println(" Grad Celsius");
      Serial.println(); // Zeilenumbruch
      Serial.print("Temperatur 1: "); //Text Sensor1
      Serial.print(Temperatur1); //Sensor1 Wert Temperatur
      Serial.println(" Grad Celsius"); //Einheit nach dem Wert
      Serial.print("Temperatur 2: "); //Text für Sensor2
      Serial.print(Temperatur2); //Sensor2 Wert Temperatur
      Serial.println(" Grad Celsius"); //Einheit nach dem Wert
      Serial.println(); // Zeilenumbruch
      Serial.print("Durchschnittstemperatur: "); //Text Durchschnittstemperatur
      Serial.print(Durchschnitttemp); // Schreiben des Wertes
      Serial.println(" Grad Celsius"); //Einheit nach dem Wert
      Serial.println(); // Zeilenumbruch
    
      if (Durchschnitttemp == solltemptag) leuchte(0); //startet Ausführung 0 aus dem Array
      else if (Durchschnitttemp == solltemptag - 1) leuchte(-1); //startet Ausführung 1 aus dem Array
      else if (Durchschnitttemp == solltemptag + 1) leuchte(1); //startet Ausführung 2 aus dem Array
      else if (Durchschnitttemp == solltemptag - 2) leuchte(-2); //startet Ausführung 3 aus dem Array
      else if (Durchschnitttemp == solltemptag + 2) leuchte(2); //startet Ausführung 4 aus dem Array
      else if (Durchschnitttemp <= solltemptag - 3) leuchte(-3); //startet Ausführung 5 aus dem Array
      else if (Durchschnitttemp >= solltemptag + 3) leuchte(3); //startet Ausführung 6 aus dem Array
    
      boolean Heizungon;
      boolean Heizungoff; 
      {
      if (Durchschnitttemp < solltemptag-2)
          Heizungon = true;
      if (Durchschnitttemp > solltemptag+2)
          Heizungoff = true;
      if (Durchschnitttemp < solltemptag-2)
          Heizungoff = false;
      if (Durchschnitttemp > solltemptag+2)
          Heizungon =false; 
        
        
      if (Heizungon)
      {
       digitalWrite(A10, HIGH);
       digitalWrite(A9, LOW);
      }
      if (Heizungoff)
      {
        digitalWrite(A10, LOW);
        digitalWrite(A9, HIGH);
      }}
    

    Vielen Dank nochmal, die nächsten Fragen kommen bestimmt. Freue mich schon darauf.
    Grüße
    Ben



  • @Ben sagte in Arduino Zeitschaltung von werten.:

      boolean Heizungon;
      boolean Heizungoff; 
      {
      if (Durchschnitttemp < solltemptag-2)
          Heizungon = true;
      if (Durchschnitttemp > solltemptag+2)
          Heizungoff = true;
      if (Durchschnitttemp < solltemptag-2)
          Heizungoff = false;
      if (Durchschnitttemp > solltemptag+2)
          Heizungon =false; 
        
        
      if (Heizungon)
      {
       digitalWrite(A10, HIGH);
       digitalWrite(A9, LOW);
      }
      if (Heizungoff)
      {
        digitalWrite(A10, LOW);
        digitalWrite(A9, HIGH);
      }}
    

    Und bei solltemptag -2 <= Durchschnitttemp && Durchnitttemp <= solltemptag + 2? Da sind dann Heitzungon und Heizungoff undefiniert. Bumm.
    Heizungoff ist doch auch einfach nur !Heizungon!?
    Was ist boolean?



  • Hallo @Swordfish,
    danke für deine Antwort.
    Wie ich es verstanden habe, ist boolean ein Vergleich zwischen 0 und 1 bzw. false und true.
    Vergleichen kann ich dann verschiedene if s.
    Zum Schluss muss ich dann definieren was bei welchem Ergebnis passieren soll.

    Hier sind es ja zwei unterschiedliche boolean. Einmal heizung an und einmal aus.
    solltemptag -2 <= Durchschnitttemp && Durchnitttemp <= solltemptag + 2
    Ich hätte mit dazu schreiben müssen dass die Anzeige gerade über zwei LED gehen, um zu sehen was passiert

    Garantie kann ich allerdings bei meiner Taktik nix, laut LED geht es. Wenn man da noch was kürzen könnte, das wäre toll



  • @Ben sagte in Arduino Zeitschaltung von werten.:

    Einmal heizung an und einmal aus.

    Das geht doch in einer. Wie du sagst, eine bool Variable kann einen true oder einen false Wert speichern. Also kann du sowohl heizung an als auch heizung aus mit einer Variable realisieren.

    bool heizungAn = true; // Heizung ist an 
    heizungAn = false; // Heizung aus
    heizungAn = true; // wieder an 
    
    if(heizungAn)
      // mache etwas wenn Heizung an ist 
    
    if(!heizungAn) // -> Das ! bedeutet nicht
      // mache etwas wenn heizung aus ist
    

    Gekürzt (ganz ohne Variable) sieht das ganze so aus:

    if(solltemptag -2 < Durchschnitttemp || Durchnitttemp > solltemptag + 2) {
        digitalWrite(A10, HIGH);
         digitalWrite(A9, LOW);
    } else {
           digitalWrite(A10, LOW);
        digitalWrite(A9, HIGH);
    }
    

    Also sprich: Wenn entweder die Durschnittstemperatur kleiner als soll -2 oder (!) größer als +2 ist, dann ist die Heizung an. In allen anderen Fällen, also wenn die Tempatur irgendwo dazwischen liegt ist sie aus.
    Ich weiß nicht genau, ob das die Logik ist, die du haben willst, aber so stand es zumindest in deinem Code 😉 Ansonsten entsprechend anpassen



  • @Ben sagte in Arduino Zeitschaltung von werten.:

    Wie ich es verstanden habe, ist boolean ein Vergleich zwischen 0 und 1 bzw. false und true.

    boolean ist (nachdem wie es in Deinem Code verwendet wird) ein Datentyp. boolean bestätigt das. Nimm - wie auch in dem Link geraten - bool, das ist Standart, zu finden in <stdbool.h>.

    @Ben sagte in Arduino Zeitschaltung von werten.:

    Hier sind es ja zwei unterschiedliche boolean. Einmal heizung an und einmal aus.
    solltemptag -2 <= Durchschnitttemp && Durchnitttemp <= solltemptag + 2
    Ich hätte mit dazu schreiben müssen dass die Anzeige gerade über zwei LED gehen, um zu sehen was passiert

    Versteh' ich nicht. Welche Werte haben denn Heizungon und Heizungoff wenn davon

    @Ben sagte in Arduino Zeitschaltung von werten.:

      if (Durchschnitttemp < solltemptag-2)
          Heizungon = true;
      if (Durchschnitttemp > solltemptag+2)
          Heizungoff = true;
      if (Durchschnitttemp < solltemptag-2)
          Heizungoff = false;
      if (Durchschnitttemp > solltemptag+2)
          Heizungon =false;
    

    nichts zutrifft?



  • Servus nochmal,
    ich werde es mal umschreiben. Das erste habe ich bereits gemacht und jeweils die -2 rausgeworfen.
    Mal sehen was da gleich passiert.

    So, nun sieht es so aus:

    int Tempsteuerung()
    {
      // Temperatursensoren lesen
      int Temperatur1 = Sensor1.readTemperature();//Temperatur Sensor1 auslesen
      int Temperatur2 = Sensor2.readTemperature();//Temperatur sensor2 auslesen
      int Durchschnitttemp = ( Temperatur1 + Temperatur2 ) / 2; // Durchscnittstemperatur berechnen
    
      // Solltemperatur ueber Potentiometer einstellen fuer Tag
      int einstellungtemptag = analogRead(15); //lesen des Potis mit der Solltemperatur für Tag
      int solltemptag = map(einstellungtemptag, 0, 1022, 10, 30); // festlegung des Regelbereiches der Temperatur Tag
      // Solltemperatur ueber Potentiometer einstellen fuer Nacht
      int einstellungtempnacht = analogRead(14); //lesen des Potis mit der Solltemperatur für Nacht
      int solltempnacht = map(einstellungtempnacht, 0, 1022, 10, 25); // festlegen des Reglerbereiches der Temperatur Nacht
    
      Serial.println(); // Zeilenumbruch
      Serial.print("Solltemperatur Tag: ");
      Serial.print(solltemptag);                                        //diesen Wert möchte ich einstellen können und er soll sich zu einer bestimmten (einstellbaren Zeit) auf "solltempnacht" umstellen z.B. 20:00Uhr.
      Serial.println(" Grad Celsius"); //Einheit nach dem Wert
      Serial.print("Solltemperatur Nacht: ");
      Serial.print(solltempnacht);                                      //diesen Wert möchte ich einstellen können und er soll sich zu einer bestimmten (einstellbaren Zeit) auf "solltemptag" umstellen z.B. 07:00Ihr.
      Serial.println(" Grad Celsius");
      Serial.println(); // Zeilenumbruch
      Serial.print("Temperatur 1: "); //Text Sensor1
      Serial.print(Temperatur1); //Sensor1 Wert Temperatur
      Serial.println(" Grad Celsius"); //Einheit nach dem Wert
      Serial.print("Temperatur 2: "); //Text für Sensor2
      Serial.print(Temperatur2); //Sensor2 Wert Temperatur
      Serial.println(" Grad Celsius"); //Einheit nach dem Wert
      Serial.println(); // Zeilenumbruch
      Serial.print("Durchschnittstemperatur: "); //Text Durchschnittstemperatur
      Serial.print(Durchschnitttemp); // Schreiben des Wertes
      Serial.println(" Grad Celsius"); //Einheit nach dem Wert
      Serial.println(); // Zeilenumbruch
    
      if (Durchschnitttemp == solltemptag) leuchte(0); //startet Ausführung 0 aus dem Array
      else if (Durchschnitttemp == solltemptag - 1) leuchte(-1); //startet Ausführung 1 aus dem Array
      else if (Durchschnitttemp == solltemptag + 1) leuchte(1); //startet Ausführung 2 aus dem Array
      else if (Durchschnitttemp == solltemptag - 2) leuchte(-2); //startet Ausführung 3 aus dem Array
      else if (Durchschnitttemp == solltemptag + 2) leuchte(2); //startet Ausführung 4 aus dem Array
      else if (Durchschnitttemp <= solltemptag - 3) leuchte(-3); //startet Ausführung 5 aus dem Array
      else if (Durchschnitttemp >= solltemptag + 3) leuchte(3); //startet Ausführung 6 aus dem Array
    
      bool Heizungon;
      bool Heizungoff; 
      {
      if (Durchschnitttemp < solltemptag)
          Heizungon = true;
      if (Durchschnitttemp > solltemptag+2)
          Heizungoff = true;
      if (Durchschnitttemp < solltemptag)
          Heizungoff = false;
      if (Durchschnitttemp > solltemptag+2)
          Heizungon =false; 
        
        
      if (Heizungon)
      {
       digitalWrite(A10, HIGH);
       digitalWrite(A9, LOW);
      }
      if (Heizungoff)
      {
        digitalWrite(A10, LOW);
        digitalWrite(A9, HIGH);
      }}
    

    Besser so?

    Grüße
    Ben



  • Hallo @Leon0402

    @Leon0402 sagte in Arduino Zeitschaltung von werten.:

    Also sprich: Wenn entweder die Durschnittstemperatur kleiner als soll -2 oder (!) größer als +2 ist, dann ist die Heizung an. In allen anderen Fällen, also wenn die Tempatur irgendwo dazwischen liegt ist sie aus.
    Ich weiß nicht genau, ob das die Logik ist, die du haben willst, aber so stand es zumindest in deinem Code Ansonsten entsprechend anpassen

    Nee so wollte ich es nicht. (Haste dir ja sicher schon gedacht. Ziel war es, die Heizung an zu schallten, wenn die Temperatur unter soll geht und ausschallten wenn die Temperatur Solltemperatur +2 Grad hat.

    Bin wohl voll dran vorbei geschossen



  • Déjà vu.

    Welche Werte haben denn Heizungon und Heizungoff wenn davon

    @Ben sagte in Arduino Zeitschaltung von werten.:

      if (Durchschnitttemp < solltemptag)
          Heizungon = true;
      if (Durchschnitttemp > solltemptag+2)
          Heizungoff = true;
      if (Durchschnitttemp < solltemptag)
          Heizungoff = false;
      if (Durchschnitttemp > solltemptag+2)
          Heizungon =false; 
    

    nichts zutrifft?



  • Sorry ich verstehe dich jetzt echt nicht.
    die if werden doch konntrolliert ob sie true oder false sind oder?
    Warum soll es nicht zutreffen Durchschnitttemp z.B. 19°C ist kleiner als solltemperatur z.B. 22°C
    wenn das true ist, soll er mit Heizungon weiter arbeiten.
    wäre

     if (Heizungon)
      {
       digitalWrite(A10, HIGH);
       digitalWrite(A9, LOW);
      }
      if (Heizungoff)
      {
        digitalWrite(A10, LOW);
        digitalWrite(A9, HIGH);
      }}
    

    oder bin ich da jetzt false unterwegs?



  • @Ben sagte in Arduino Zeitschaltung von werten.:

    Hallo @Leon0402

    @Leon0402 sagte in Arduino Zeitschaltung von werten.:

    Also sprich: Wenn entweder die Durschnittstemperatur kleiner als soll -2 oder (!) größer als +2 ist, dann ist die Heizung an. In allen anderen Fällen, also wenn die Tempatur irgendwo dazwischen liegt ist sie aus.
    Ich weiß nicht genau, ob das die Logik ist, die du haben willst, aber so stand es zumindest in deinem Code Ansonsten entsprechend anpassen

    Nee so wollte ich es nicht. (Haste dir ja sicher schon gedacht. Ziel war es, die Heizung an zu schallten, wenn die Temperatur unter soll geht und ausschallten wenn die Temperatur Solltemperatur +2 Grad hat.

    Bin wohl voll dran vorbei geschossen

    Na dann macht das doch mal als Formel:

    if(hier deine Formel einfügen)
      // heizung an 
    else
     // heizung aus
    


  • Was ist wenn
    Durchschnitttemp < solltemptag nicht wahr ist,
    Durchschnitttemp > solltemptag+2 nicht wahr ist,
    Durchschnitttemp < solltemptag nicht wahr ist und auch
    Durchschnitttemp > solltemptag+2 nicht wahr ist?
    Welche Werte haben dann Heizungon und Heizungoff?



  • Hallo @Swordfish,
    jetzt ist der groschen gefallen, das heißt es muss noch eine if mit rein, Durschnitttemp==solltemptag oder bei einer Temperaturspanne = mit rein. Richtig?



  • @Ben sagte in Arduino Zeitschaltung von werten.:

    So, nun sieht es so aus:

    int Tempsteuerung()
    {
      // Temperatursensoren lesen
      int Temperatur1 = Sensor1.readTemperature();//Temperatur Sensor1 auslesen
      int Temperatur2 = Sensor2.readTemperature();//Temperatur sensor2 auslesen
      int Durchschnitttemp = ( Temperatur1 + Temperatur2 ) / 2; // Durchscnittstemperatur berechnen
    
    [...]
    
      Serial.println(); // Zeilenumbruch
      Serial.print("Temperatur 1: "); //Text Sensor1
      Serial.print(Temperatur1); //Sensor1 Wert Temperatur
      Serial.println(" Grad Celsius"); //Einheit nach dem Wert
      Serial.print("Temperatur 2: "); //Text für Sensor2
      Serial.print(Temperatur2); //Sensor2 Wert Temperatur
    

    Hallo @Ben,

    ich möchte dich mal fragen, ob dir deine eigenen Kommentare im Code eigentlich weiterhelfen. Im hier zitierten Teil würde ich höchstens den einen Kommentar in Zeile 3 lassen und alle anderen entfernen. Diese Kommentare beschreiben doch nur 1:1 die jeweilige Codezeile und sind damit überflüssig.

    Ich würde dir raten, ganz anders zu kommentieren.

    Das fängt an mit einer Beschreibung, was die Funktion tut, wenn sie keinen eindeutigen Namen hat. Was "Serial.println()" macht, ist klar. Aber was macht Tempsteuerung genau? Die Funktion returnt einen int. Was bedeutet der Return-Wert? So etwas solltest du kommentieren.

    Außerdem kannst du Kommentare da verwenden, wo z.B. der Code kompliziert ist. Oder wenn du einen Block hast, der eine bestimmte Sache tut, dann kannst du dem gesamten Block einen Kommentar spendieren - oder dann den Block in eine Funktion auslagern.

    Jedenfalls mein genereller Rat: kommentieren nicht die sowieso lesbaren und klaren Zeilen, sondern die, wo das nicht der Fall ist. Und kommentiere Funktionen und Klassen, sofern es sich nicht um triviale Dinge handelt oder durch den Funktionsnamen schon alles klar ist.

    Hier zum Beispiel:

    int einstellungtemptag = analogRead(15); //lesen des Potis mit der Solltemperatur für Tag

    finde ich den Kommentar sinnvoll. Andererseits: was ist die 15? Könntest du nicht aus der 15 eine benannte Konstante machen? Sowas wie const int POTI_TEMP_SOLL = 15;? Und dann wird der Kommentar auch vielleicht überflüssig. Generell: aussagekräftiger Code ist besser als Kommentare.


Anmelden zum Antworten