Use double buffer

terminal/puzzle_colorFade/potiNano/src/main.cpp

@@ -1,29 +1,39 @@
 #include <Arduino.h>
-
 #include <Wire.h>
 
 #define I2C_SLAVE_ADDRESS 0x08
 
-uint16_t values[6] = {0};
+volatile uint16_t bufferA[6];
+volatile uint16_t bufferB[6];
+
+// Pointer to the buffer currently exposed to ISR
+volatile uint16_t* activeBuffer = bufferA;
 
 void onI2CRequest() {
-    Wire.write((uint8_t*)values, sizeof(values));
+    // Safe: ISR only reads active buffer
+    Wire.write((uint8_t*)activeBuffer, 6 * sizeof(uint16_t));
 }
 
 void setup() {
     Wire.begin(I2C_SLAVE_ADDRESS);
     Wire.onRequest(onI2CRequest);
-
     analogReference(DEFAULT);
 }
 
 void loop() {
-    values[0] = analogRead(A0);
-    values[1] = analogRead(A1);
-    values[2] = analogRead(A2);
+    // Choose the inactive buffer
+    volatile uint16_t* writeBuffer =
+        (activeBuffer == bufferA) ? bufferB : bufferA;
 
-    values[3] = analogRead(A3);
-    values[4] = analogRead(A6);
-    values[5] = analogRead(A7);
-    delay(50);
+    writeBuffer[0] = analogRead(A0);
+    writeBuffer[1] = analogRead(A1);
+    writeBuffer[2] = analogRead(A2);
+    writeBuffer[3] = analogRead(A3);
+    writeBuffer[4] = analogRead(A6);
+    writeBuffer[5] = analogRead(A7);
+
+    // Atomic pointer swap
+    noInterrupts();
+    activeBuffer = writeBuffer;
+    interrupts();
 }

terminal/puzzle_colorRemember/src/main.cpp

@@ -97,16 +97,21 @@ bool readButton(uint8_t i) {
 
 // ------------------ Neues Spiel ------------------
 
-void generateNewGame() {
+void generateNewGame(bool easy) {
 
-  for (uint8_t i = 0; i < 6; i++) solutionOrder[i] = i;
+  for (uint8_t i = 0; i < 6; i++)
+    solutionOrder[i] = i;
 
-  for (int i = 5; i > 0; i--) {
+  if (!easy)
+  {
+    for (int i = 5; i > 0; i--)
+    {
       int j = random(i + 1);
       uint8_t t = solutionOrder[i];
       solutionOrder[i] = solutionOrder[j];
       solutionOrder[j] = t;
     }
+  }
 
   for (uint8_t i = 0; i < 6; i++)
     expectedOrder[i] = solutionOrder[5 - i];
@@ -137,12 +142,16 @@ void generateNewGame() {
   uint8_t idx[18];
   for (uint8_t i = 0; i < 18; i++) idx[i] = i;
 
-  for (int i = 17; i > 0; i--) {
+  if (!easy)
+  {
+    for (int i = 17; i > 0; i--)
+    {
       int j = random(i + 1);
       uint8_t t = idx[i];
       idx[i] = idx[j];
       idx[j] = t;
     }
+  }
 
   for (uint8_t i = 0; i < 6; i++)
     buttonAssignment[idx[i]] = solutionOrder[i];
@@ -244,7 +253,7 @@ void setup() {
 
   if (powerOnState)
   {
-    generateNewGame();
+    generateNewGame(false);
   }
 }
 
@@ -271,7 +280,7 @@ void loop() {
   // Detecting turn-on
   if (!powerOnState && digitalRead(PIN_POWER_ON) == HIGH)
   {
-    generateNewGame();
+    generateNewGame(false);
     powerOnState = true;
   }
   if (!powerOnState)
@@ -279,14 +288,18 @@ void loop() {
     // We are turned-off and stop anything after here
     return;
   }
-  unsigned long now = millis();
 
-  bool trigger = digitalRead(PIN_TRIGGER_SHUFFLE);
-  if (trigger && !lastTriggerState) {
-    generateNewGame();
+  if (digitalRead(PIN_TRIGGER_SHUFFLE) == HIGH)
+  {
+    unsigned long resetTimer = millis();
+    while (digitalRead(PIN_TRIGGER_SHUFFLE) == HIGH)
+    {
+      delay(10);
+    }
+    generateNewGame(millis() - resetTimer > 150);
   }
-  lastTriggerState = trigger;
 
+  unsigned long now = millis();
   for (uint8_t i = 0; i < 18; i++) {
     bool pressed = readButton(i);