millis() readout extrem hoch und Überlauf nach Sekunden

International Deutsch
1

Guten morgen zusammen,

ich habe eine kleines Projekt und möchte dafür die zeit zwischen 2 Events auslesen, was ja eigentlich eine einfache Kiste sein sollte.

Alle 5 sek. werden shiftregister ausgelesen, sollte einer offen sein wird der timer gestartet. Ist der Sensor länger als 10sec. offen soll das relay 1 geschalten werden und länger als 180sec. dann wird ein weiteres Relay geschalten.

Leider gibt es bei meinem Script das Problem dass bereits nachdem der timer das erste mal ausgelesen wird angeblich bereits 16843s vergangen sind und 5 sec später beim nächsten readout bereits 4294967s (entspricht 4294967276 = 2^32 was wohl der max. für millis() ist)

Hab ich irgendetwas vergessen zu deklarieren oder etwas vergessen anzuschliessen oder warum zähl mein Board zu schnell.

Ich benutzte entweder ein Arduino Nano oder ein Pro Mini und bei beiden habe ich das Verhalten beobachtet.

Unten angehangen habe eich einmal mein Seriell Read out und auch das Program. Für Tipps wäre ich verdammt dankbar :wink:

Lieben Gruss
Stefan

Starting reading out of ShiftRegister - (round #1)
    Checking Sensors
    Connected Sensors: 0
    Closed Sensors: 11111111
      Sensor: 1 is connected and closed.
      Sensor: 2 is connected and closed.
      Sensor: 3 is connected and closed.
      Sensor: 4 is connected and closed.
      Sensor: 5 is connected and closed.
      Sensor: 6 is connected and closed.
      Sensor: 7 is connected and closed.
      Sensor: 8 is connected and closed.
All Sensors closed.
---------Finished ----------
Starting reading out of ShiftRegister - (round #2)
    Checking Sensors
    Connected Sensors: 0
    Closed Sensors: 1111111
      Sensor: 1 is connected and closed.
      Sensor: 2 is connected and closed.
      Sensor: 3 is connected and closed.
      Sensor: 4 is connected and closed.
      Sensor: 5 is connected and closed.
      Sensor: 6 is connected and closed.
      Sensor: 7 is connected and closed.
      Sensor: 8 is connected but open.
  Sensor opened - Starting timer
  Start: 16843
---------Finished ----------
Starting reading out of ShiftRegister - (round #3)
    Checking Sensors
    Connected Sensors: 0
    Closed Sensors: 1111111
      Sensor: 1 is connected and closed.
      Sensor: 2 is connected and closed.
      Sensor: 3 is connected and closed.
      Sensor: 4 is connected and closed.
      Sensor: 5 is connected and closed.
      Sensor: 6 is connected and closed.
      Sensor: 7 is connected and closed.
      Sensor: 8 is connected but open.
  Timer too high for Relay #0 | elapsed time = 4294967 max allowed time = 10000
  Timer too high for Relay #1 | elapsed time = 4294967 max allowed time = 180000
---------Finished ----------
Starting reading out of ShiftRegister - (round #4)
    Checking Sensors
    Connected Sensors: 0
    Closed Sensors: 11111111
      Sensor: 1 is connected and closed.
      Sensor: 2 is connected and closed.
      Sensor: 3 is connected and closed.
      Sensor: 4 is connected and closed.
      Sensor: 5 is connected and closed.
      Sensor: 6 is connected and closed.
      Sensor: 7 is connected and closed.
      Sensor: 8 is connected and closed.
All Sensors closed.

FreezerLogger.ino (6.94 KB)

2

Hmmm....
Vielleicht:
Serial.print(elapsed/1000);
unsigned long temp = elapsed/1000L;
Serial.print(temp);
Serial Print wertet die Zahl wohl als einfache int Zahl

3

nee, das ist es nicht.

Irgendwie verstellt dein Code den Timer für millis().

Wenn man sich den aktuelle Timerstand ausgeben läßt mit

Serial.print("millis: ");Serial.println(millis());
dann sind bereits nach dem ersten loopdurchlauf 26min vergangen.
Es sind aber tatsächlich nur ca. 5Sek, delay scheint also zu funktionieren.

4

Es passiert irgendwo in der Funktion "CheckPorts()"

5

Gefunden:

boolean ConnectedPorts[] = {};      
boolean OpenSensors[] ={};

das legt arrays der Größe 0 an, und mit späteren Zugriffen schreibst/liest du irgendwo im Speicher.

Du musst eine initiale Größe angeben, du muß also vorher schon wissen wieviele Port und Sensoren du maximal verwenden willst

boolean ConnectedPorts[99] ;      
boolean OpenSensors[99];

So kommt dieser Fehler nicht mehr.

6

guntherb:
Es passiert irgendwo in der Funktion "CheckPorts()"

Stimmt!

Ein klassischer Bufferoverflow.
ConnectedPorts ist zu klein

7

Hallo,

Glückwunsch zur Fehlerfindung. :slight_smile:

Ich hätte jetzt gedacht das er mit dem & Pointer sonst wohin springt.

(ConnectedSensorsShiftIn[Register] & (1 << n) ) {
8

& Pointer

Das & ist dort kein Adressoperator, sondern ein Bitmanipulationsoperator
:confused:

9

Danke schön, so ein einfacher (stümperhafterfehler) Fehler.

Wenn ich den Array bei Deklaration schon vorbefülle läuft alles 1A ab.

Kleine Ursache grosse Wirkung.

Vielen lieben Dank für die schnell Hilfe

Stefan

10

combie:
Das & ist dort kein Adressoperator, sondern ein Bitmanipulationsoperator
:confused:

Hallo,

gut, dann hab ich das verwechselt. :confused:

11

Doc_Arduino:
gut, dann hab ich das verwechselt.

Das kann man auch leicht, weil sie exakt gleich aussehen.
Einzig der Kontext macht den Unterschied.