Soundausgabe führt zum Stoppen meines Programmes

Hallo liebe Community,

ich bin ganz frisch in der Arduino Welt. Habe die letzten 15 Jahre mir nur mit Bascom Basic beschäftigt und Versuche nun umzusteigen.

Ich habe ein Programm, genauer ein Spiel geschrieben, ich glaube in eurer Welt nennt man das Sketch.

Soweit so gut, das funktioniert einwandfrei.

Wenn der Spieler gewinnt, möchte ich gerne das eine kleine Melody abgespielt wird und daran verzweilfe ich gerade etwas :slightly_frowning_face:

Wenn ich das Beispiel auf mein Arduino übertrage, dann wird die Melody ohne Probleme abgespielt und ist auch zu hören.

Wenn ich den Quellcode jetzt aber in mein Programm einfüge dann passiert einfach garnichts!

den ersten Teil des Beispielprogrammes habe ich am Anfang meines Programmes eingefügt:

/* 
                                   
                                              
                                              Robson Couto, 2019
*/

#define NOTE_B0  31
#define NOTE_C1  33
#define NOTE_CS1 35
#define NOTE_D1  37
#define NOTE_DS1 39
#define NOTE_E1  41
#define NOTE_F1  44
#define NOTE_FS1 46
#define NOTE_G1  49
#define NOTE_GS1 52
#define NOTE_A1  55
#define NOTE_AS1 58
#define NOTE_B1  62
#define NOTE_C2  65
#define NOTE_CS2 69
#define NOTE_D2  73
#define NOTE_DS2 78
#define NOTE_E2  82
#define NOTE_F2  87
#define NOTE_FS2 93
#define NOTE_G2  98
#define NOTE_GS2 104
#define NOTE_A2  110
#define NOTE_AS2 117
#define NOTE_B2  123
#define NOTE_C3  131
#define NOTE_CS3 139
#define NOTE_D3  147
#define NOTE_DS3 156
#define NOTE_E3  165
#define NOTE_F3  175
#define NOTE_FS3 185
#define NOTE_G3  196
#define NOTE_GS3 208
#define NOTE_A3  220
#define NOTE_AS3 233
#define NOTE_B3  247
#define NOTE_C4  262
#define NOTE_CS4 277
#define NOTE_D4  294
#define NOTE_DS4 311
#define NOTE_E4  330
#define NOTE_F4  349
#define NOTE_FS4 370
#define NOTE_G4  392
#define NOTE_GS4 415
#define NOTE_A4  440
#define NOTE_AS4 466
#define NOTE_B4  494
#define NOTE_C5  523
#define NOTE_CS5 554
#define NOTE_D5  587
#define NOTE_DS5 622
#define NOTE_E5  659
#define NOTE_F5  698
#define NOTE_FS5 740
#define NOTE_G5  784
#define NOTE_GS5 831
#define NOTE_A5  880
#define NOTE_AS5 932
#define NOTE_B5  988
#define NOTE_C6  1047
#define NOTE_CS6 1109
#define NOTE_D6  1175
#define NOTE_DS6 1245
#define NOTE_E6  1319
#define NOTE_F6  1397
#define NOTE_FS6 1480
#define NOTE_G6  1568
#define NOTE_GS6 1661
#define NOTE_A6  1760
#define NOTE_AS6 1865
#define NOTE_B6  1976
#define NOTE_C7  2093
#define NOTE_CS7 2217
#define NOTE_D7  2349
#define NOTE_DS7 2489
#define NOTE_E7  2637
#define NOTE_F7  2794
#define NOTE_FS7 2960
#define NOTE_G7  3136
#define NOTE_GS7 3322
#define NOTE_A7  3520
#define NOTE_AS7 3729
#define NOTE_B7  3951
#define NOTE_C8  4186
#define NOTE_CS8 4435
#define NOTE_D8  4699
#define NOTE_DS8 4978
#define REST      0


// change this to make the song slower or faster
int tempo = 108;

// change this to whichever pin you want to use
int buzzer = 11;

// notes of the moledy followed by the duration.
// a 4 means a quarter note, 8 an eighteenth , 16 sixteenth, so on
// !!negative numbers are used to represent dotted notes,
// so -4 means a dotted quarter note, that is, a quarter plus an eighteenth!!
int melody[] = {
  
  // Dart Vader theme (Imperial March) - Star wars 
  // Score available at https://musescore.com/user/202909/scores/1141521
  // The tenor saxophone part was used
  
  NOTE_AS4,8, NOTE_AS4,8, NOTE_AS4,8,//1
  NOTE_F5,2, NOTE_C6,2,
  NOTE_AS5,8, NOTE_A5,8, NOTE_G5,8, NOTE_F6,2, NOTE_C6,4,  
  NOTE_AS5,8, NOTE_A5,8, NOTE_G5,8, NOTE_F6,2, NOTE_C6,4,  
  NOTE_AS5,8, NOTE_A5,8, NOTE_AS5,8, NOTE_G5,2, NOTE_C5,8, NOTE_C5,8, NOTE_C5,8,
  NOTE_F5,2, NOTE_C6,2,
  NOTE_AS5,8, NOTE_A5,8, NOTE_G5,8, NOTE_F6,2, NOTE_C6,4,  
  
  NOTE_AS5,8, NOTE_A5,8, NOTE_G5,8, NOTE_F6,2, NOTE_C6,4, //8  
  NOTE_AS5,8, NOTE_A5,8, NOTE_AS5,8, NOTE_G5,2, NOTE_C5,-8, NOTE_C5,16, 
  NOTE_D5,-4, NOTE_D5,8, NOTE_AS5,8, NOTE_A5,8, NOTE_G5,8, NOTE_F5,8,
  NOTE_F5,8, NOTE_G5,8, NOTE_A5,8, NOTE_G5,4, NOTE_D5,8, NOTE_E5,4,NOTE_C5,-8, NOTE_C5,16,
  NOTE_D5,-4, NOTE_D5,8, NOTE_AS5,8, NOTE_A5,8, NOTE_G5,8, NOTE_F5,8,
  
  NOTE_C6,-8, NOTE_G5,16, NOTE_G5,2, REST,8, NOTE_C5,8,//13
  NOTE_D5,-4, NOTE_D5,8, NOTE_AS5,8, NOTE_A5,8, NOTE_G5,8, NOTE_F5,8,
  NOTE_F5,8, NOTE_G5,8, NOTE_A5,8, NOTE_G5,4, NOTE_D5,8, NOTE_E5,4,NOTE_C6,-8, NOTE_C6,16,
  NOTE_F6,4, NOTE_DS6,8, NOTE_CS6,4, NOTE_C6,8, NOTE_AS5,4, NOTE_GS5,8, NOTE_G5,4, NOTE_F5,8,
  NOTE_C6,1
  
};

// sizeof gives the number of bytes, each int value is composed of two bytes (16 bits)
// there are two values per note (pitch and duration), so for each note there are four bytes
int notes = sizeof(melody) / sizeof(melody[0]) / 2;

// this calculates the duration of a whole note in ms
int wholenote = (60000 * 4) / tempo;

int divider = 0, noteDuration = 0;

Nur mit diesen Code funktioniert mein Programm weiterhin. Füge aber ich jetzt den Unterenteil dort in mein Programm ein wo die Melody abgespielt werden soll, dann passiert einfach garnichts mehr:

// iterate over the notes of the melody. 
  // Remember, the array is twice the number of notes (notes + durations)
  for (int thisNote = 0; thisNote < notes * 2; thisNote = thisNote + 2) {

    // calculates the duration of each note
    divider = melody[thisNote + 1];
    if (divider > 0) {
      // regular note, just proceed
      noteDuration = (wholenote) / divider;
    } else if (divider < 0) {
      // dotted notes are represented with negative durations!!
      noteDuration = (wholenote) / abs(divider);
      noteDuration *= 1.5; // increases the duration in half for dotted notes
    }

    // we only play the note for 90% of the duration, leaving 10% as a pause
    tone(buzzer, melody[thisNote], noteDuration*0.9);

    // Wait for the specief duration before playing the next note.
    delay(noteDuration);
    
    // stop the waveform generation before the next note.
    noTone(buzzer);
  }
}

Es lässt sich fehlerfrei Kompilieren und übertragen, aber das gesamte Programm läuft dann nicht mehr, selbst der Anfangteil wo der untere Bereich noch garnicht abgearbeitet wird, es scheint so als ob sich mein Gesamtes Programm dann weigert zu Starten.

Habt ihr eine Idee was ich falsch mache?
Würde mich super freuen etwas von euch zu hören, irgendwie ein Tipp wo ich weiter suchen kann, wäre es klasse.

Vielen lieben dank
Stefan

delay() blockiert den Controller.
Du mußt Das Timing mit millis() machen.
Grüße Uwe

Danke für den Tipp, ich hab zwar noch keine Ahnung was mit millis() gemeint ist, aber da werde ich jetzt Tante Google mal bemühen, so hab ich schonmal ein Anhaltspunkt, vielen vielen dank

Wobei mir noch nicht ganz klar ist warum das delay ein Problem ist, es wäre ja nicht schlimm wenn das Programm solange der sound abgespielt wird nichts anderes macht, aber es läuft ja überhaupt nicht mehr, nicht einmal bis zu diesem Punkt, das ist für mich etwas unbegreiflich....

Dann mußt Du schon ein kompilierbaren Sketch, der das Problem aufweist posten.
Grüße Uwe

Deines geheimen Programms!

Interessanter weise, hast du den Teil gezeigt, welcher laufen könnte.
Aber den kaputten Teil abgeschnitten.

"Dein Programm" bis zur Untestbarkeit verstümmelt!
Falls das einen besonderen Grund hat, dann ist das trotzdem eine recht dumme Idee, möchte ich mal sagen....

Zeig mal Deinen Code wie er im Original ist.
Das Merge bekommen wir dann hier schon hin.

Ok, ich bin ein großen Schritt weiter.

Ich habe festgestellt es hat etwas mit meiner Display ausgabe zutun.

Auch wenn mein Sketch Mega lang ist, hier einmal mein Komplettes Spiel.

//**************************************************************************************************************************************
//****                        Nachbau und Erweiterung des Spiels Senso, aus den 1980er Jahren                                       ****
//****                                       Eine Idee von Stefan Bastubbe                                                          ****
//****                                              September 2023                                                                  ****
//**************************************************************************************************************************************

//Einlesen aller benötigten Bibliotheken, für das Display und das verwenden des EEPROM'S
#include <Wire.h>
#include <Adafruit_SSD1306.h>
#include <EEPROM.h>

//Grundeinstellungen für das Display. 128x64 Pixel
#define SCREEN_WIDTH 128
#define SCREEN_HEIGHT 64
#define OLED_RESET -1
#define SCREEN_ADDRESS 0x3C  ///< See datasheet for Address; 0x3D for 128x64, 0x3C for 128x32
Adafruit_SSD1306 display(SCREEN_WIDTH, SCREEN_HEIGHT, &Wire, OLED_RESET);

//Globale Variablen, hier werden alle definiert, die im Gesamten Programm verwendet werden können.
int Zufallzahl[150];  //Speicherplatz für die Reihenfolge die per Zufall festgelegt wird.

int Highscoregesamt;  // Hier wird der höchste jemals erreichte Wert hinterlegt.
int Runde = 0;        //Hier wird hinterlegt wieviel Runden der Spieler aktuell geschafft hat

int aktuellaktiv = 0;
int warten = 0;
int raus = 0;
int NeuerHigh = 0;

#define NOTE_B0  31
#define NOTE_C1  33
#define NOTE_CS1 35
#define NOTE_D1  37
#define NOTE_DS1 39
#define NOTE_E1  41
#define NOTE_F1  44
#define NOTE_FS1 46
#define NOTE_G1  49
#define NOTE_GS1 52
#define NOTE_A1  55
#define NOTE_AS1 58
#define NOTE_B1  62
#define NOTE_C2  65
#define NOTE_CS2 69
#define NOTE_D2  73
#define NOTE_DS2 78
#define NOTE_E2  82
#define NOTE_F2  87
#define NOTE_FS2 93
#define NOTE_G2  98
#define NOTE_GS2 104
#define NOTE_A2  110
#define NOTE_AS2 117
#define NOTE_B2  123
#define NOTE_C3  131
#define NOTE_CS3 139
#define NOTE_D3  147
#define NOTE_DS3 156
#define NOTE_E3  165
#define NOTE_F3  175
#define NOTE_FS3 185
#define NOTE_G3  196
#define NOTE_GS3 208
#define NOTE_A3  220
#define NOTE_AS3 233
#define NOTE_B3  247
#define NOTE_C4  262
#define NOTE_CS4 277
#define NOTE_D4  294
#define NOTE_DS4 311
#define NOTE_E4  330
#define NOTE_F4  349
#define NOTE_FS4 370
#define NOTE_G4  392
#define NOTE_GS4 415
#define NOTE_A4  440
#define NOTE_AS4 466
#define NOTE_B4  494
#define NOTE_C5  523
#define NOTE_CS5 554
#define NOTE_D5  587
#define NOTE_DS5 622
#define NOTE_E5  659
#define NOTE_F5  698
#define NOTE_FS5 740
#define NOTE_G5  784
#define NOTE_GS5 831
#define NOTE_A5  880
#define NOTE_AS5 932
#define NOTE_B5  988
#define NOTE_C6  1047
#define NOTE_CS6 1109
#define NOTE_D6  1175
#define NOTE_DS6 1245
#define NOTE_E6  1319
#define NOTE_F6  1397
#define NOTE_FS6 1480
#define NOTE_G6  1568
#define NOTE_GS6 1661
#define NOTE_A6  1760
#define NOTE_AS6 1865
#define NOTE_B6  1976
#define NOTE_C7  2093
#define NOTE_CS7 2217
#define NOTE_D7  2349
#define NOTE_DS7 2489
#define NOTE_E7  2637
#define NOTE_F7  2794
#define NOTE_FS7 2960
#define NOTE_G7  3136
#define NOTE_GS7 3322
#define NOTE_A7  3520
#define NOTE_AS7 3729
#define NOTE_B7  3951
#define NOTE_C8  4186
#define NOTE_CS8 4435
#define NOTE_D8  4699
#define NOTE_DS8 4978
#define REST      0

// change this to make the song slower or faster
int tempo = 80;

// change this to whichever pin you want to use
int buzzer = 4;

// notes of the moledy followed by the duration.
// a 4 means a quarter note, 8 an eighteenth , 16 sixteenth, so on
// !!negative numbers are used to represent dotted notes,
// so -4 means a dotted quarter note, that is, a quarter plus an eighteenth!!
int melody[] = {

  // Star Trek Intro
  // Score available at https://musescore.com/user/10768291/scores/4594271
 
  NOTE_D4, -8, NOTE_G4, 16, NOTE_C5, -4, 
  NOTE_B4, 8, NOTE_G4, -16, NOTE_E4, -16, NOTE_A4, -16,
  NOTE_D5, 2,
  
};

// sizeof gives the number of bytes, each int value is composed of two bytes (16 bits)
// there are two values per note (pitch and duration), so for each note there are four bytes
int notes = sizeof(melody) / sizeof(melody[0]) / 2;

// this calculates the duration of a whole note in ms
int wholenote = (60000 * 4) / tempo;
int divider = 0, noteDuration = 0;


void setup() {

  //Lese den maximal erreichten Highscore aus dem EEPROM aus, sollte keiner vorhanden sein, setze ihn auf 0
  Highscoregesamt = EEPROM.read(2);
  if (Highscoregesamt == 255) { Highscoregesamt = 0; }

  // Digitale Ausgänge Festlegen, hier sind die LED's der Tasten angeschlossen.
  pinMode(14, OUTPUT);  // LED Taster 1
  pinMode(15, OUTPUT);  // LED Taster 2
  pinMode(16, OUTPUT);  // LED Taster 3
  pinMode(17, OUTPUT);  // LED Taster 4
  pinMode(6, OUTPUT);   // LED Taster 5
  pinMode(4, OUTPUT);   // Piezo Lautsprecher

  // Digitale Eingange festlegen, hier sind die Tasten Angeschlossen
  pinMode(8, INPUT);   // Taster 1
  pinMode(9, INPUT);   // Taster 2
  pinMode(10, INPUT);  // Taster 3
  pinMode(3, INPUT);   // Taster 4
  pinMode(7, INPUT);   //Taster 5
  pinMode(5, INPUT);   // Taster Spielwahl. Ist der Eingang LOW ist Spiel 1 gewählt, ist er High, wurde Spiel 2 ausgewählt.
Musik();
  // Konfiguration des Display's, dient zu Anzeige des Highscores
  Serial.begin(9600);
  if (!display.begin(SSD1306_SWITCHCAPVCC, SCREEN_ADDRESS)) {
    Serial.println(F("SSD1306 allocation failed"));
    for (;;)
      ;
  }
  display.clearDisplay();
  display.display();
  display.setRotation(2);
}
//Starten dient dazu den Begrüßungstext auf dem Display zu Schreiben und zu warten bis der Spieler eine Taste zum Starten betätigt hat
void Starten()
  {
  display.clearDisplay();
  display.setTextSize(3);
  display.setTextColor(SSD1306_WHITE);
  display.setCursor(0, 0);
  display.println(F(" SENSO"));
  display.setTextSize(2);
  display.setTextColor(SSD1306_BLACK, SSD1306_WHITE);
  display.setCursor(27, 22);
  display.println("V 1.0");
  display.setTextSize(1);
  display.setTextColor(SSD1306_WHITE);
  display.setCursor(0, 42);
  display.println(F("Beliebige Taste zum"));
  display.setCursor(0, 50);
  display.println(F("Spielstart druecken"));
  display.display();
  delay(3000);
digitalWrite(17, LOW);//!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
digitalWrite(6, HIGH);//!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
  //Warteschleife, warte so lange bis der Benutzer eine beliebige Taste betätigt hat, solange wechsel alle 25ms zufällig die Taste die gerade leuchtet und suche jedes mal neue Zufallzahlen
  Warten1:
  delay(1);

  //Suche neue Zufallszahlen für die Reihenfolge des Spielrunde
  for (int i = 1; i <= 145; i++) 
    {
    Zufallzahl[i] = random(1, 6);
    }

  //Alle 25ms wird die aktuell Leuchtende Taste durch eine andere ersetzt, dies ist zufällig, bevor eine neue eingeschaltet wird, werden alle erst einmal abgeschaltet.
  warten++;
  if (warten == 25) 
    {
    warten = 0;
    digitalWrite(14, LOW);
    digitalWrite(15, LOW);
    digitalWrite(16, LOW);
    digitalWrite(17, LOW);
    digitalWrite(6, LOW);
    if (Zufallzahl[1] == 1) { digitalWrite(14, HIGH); }
    if (Zufallzahl[1] == 2) { digitalWrite(15, HIGH); }
    if (Zufallzahl[1] == 3) { digitalWrite(16, HIGH); }
    if (Zufallzahl[1] == 4) { digitalWrite(17, HIGH); }
    if (Zufallzahl[1] == 5) { digitalWrite(6, HIGH); }
    }

  //Frage ab ob eine Taste betätigt wurde, wenn NEIN, dann spring wieder an den Anfang zurück. Sollte eine beliebige Taste betätigt worden sein, gehe weiter im Programm
  if (digitalRead(8) == false and digitalRead(9) == false and digitalRead(10) == false and digitalRead(3) == false and digitalRead(7) == false) { goto Warten1; }
  digitalWrite(14, LOW);
  digitalWrite(15, LOW);
  digitalWrite(16, LOW);
  digitalWrite(17, LOW);
  digitalWrite(6, LOW);
  //Warte 1 Sekunde, dann beginnt die Eigentliche Schleife des Spieles
  
  delay(1000);
  }


//START Hauptprogramm
void loop() 
  {
  if (Runde==0) {Starten();}

  // Gebe den aktuellen Higscore auf dem Display aus
  display.clearDisplay();
  display.setTextSize(2);
  display.setTextColor(SSD1306_WHITE);
  display.setCursor(0, 0);
  display.println(F("Max:"));
  display.setCursor(80, 0);
  display.println(Highscoregesamt);
  display.setCursor(0, 32);
  display.println(F("Akt:"));
  display.setCursor(80, 32);
  display.println(Runde);
  display.display();


  //Zähle eine Ziffer hoch und Spiele die gesamte Reihenfolge, bis die aktuelle Anzahl erreicht ist
  Runde++;

  for (int i = 0; i <= Runde; i++) 
  {
  aktuellaktiv = Zufallzahl[i];

    if (aktuellaktiv == 1) {
      digitalWrite(14, HIGH);
      tone(4, 1000, 200);
      delay(200);
    }

    if (aktuellaktiv == 2) {
      digitalWrite(15, HIGH);
      tone(4, 1050, 200);
      delay(200);
    }

    if (aktuellaktiv == 3) {
      digitalWrite(16, HIGH);
      tone(4, 1100, 200);
      delay(200);
    }

    if (aktuellaktiv == 4) {
      digitalWrite(17, HIGH);
      tone(4, 1150, 200);
      delay(200);
    }

    if (aktuellaktiv == 5) {
      digitalWrite(6, HIGH);
      tone(4, 1200, 200);
      delay(200);
    }

    digitalWrite(14, LOW);
    digitalWrite(15, LOW);
    digitalWrite(16, LOW);
    digitalWrite(17, LOW);
    digitalWrite(6, LOW);
    delay(100);
  }




  for (int i = 0; i <= Runde; i++) 
  {
  aktuellaktiv = Zufallzahl[i];
  if (aktuellaktiv == 1) 
    {
    Schleife1:
    if (digitalRead(9) == true) {verloren();}
    if (digitalRead(10) == true) {verloren();}
    if (digitalRead(3) == true) {verloren();}
    if (digitalRead(7) == true) {verloren();}
    if (digitalRead(8) == true) 
      {
      digitalWrite(14, HIGH);
      tone(4, 1000, 200);
      delay(200);
      digitalWrite(14, LOW);
      raus++;
      }
    if (raus == 0) goto Schleife1;
    }


  if (aktuellaktiv == 2) 
    {
    Schleife2:
    if (digitalRead(8) == true) { verloren(); }
    if (digitalRead(10) == true) { verloren(); }
    if (digitalRead(3) == true) { verloren(); }
    if (digitalRead(7) == true) { verloren(); }
    if (digitalRead(9) == true) 
      {
      digitalWrite(15, HIGH);
      tone(4, 1050, 200);
      delay(200);
      digitalWrite(15, LOW);
      raus++;
      }
    if (raus == 0) goto Schleife2;
    }

    if (aktuellaktiv == 3) 
    {
    Schleife3:
    if (digitalRead(9) == true) { verloren(); }
    if (digitalRead(8) == true) { verloren(); }
    if (digitalRead(3) == true) { verloren(); }
    if (digitalRead(7) == true) { verloren(); }
    if (digitalRead(10) == true) 
      {
      digitalWrite(16, HIGH);
      tone(4, 1100, 200);
      delay(200);
      digitalWrite(16, LOW);
      raus++;
      }
    if (raus == 0) goto Schleife3;
    }

    if (aktuellaktiv == 4) 
    {
    Schleife4:
    if (digitalRead(9) == true) { verloren(); }
    if (digitalRead(10) == true) { verloren(); }
    if (digitalRead(8) == true) { verloren(); }
    if (digitalRead(7) == true) { verloren(); }
    if (digitalRead(3) == true) 
      {
      digitalWrite(17, HIGH);
      tone(4, 1150, 200);
      delay(200);
      digitalWrite(17, LOW);
      raus++;
      }
    if (raus == 0) goto Schleife4;
    }

    if (aktuellaktiv == 5)
    {
    Schleife5:
    if (digitalRead(8) == true) { verloren(); }
    if (digitalRead(10) == true) { verloren(); }
    if (digitalRead(3) == true) { verloren(); }
    if (digitalRead(9) == true) { verloren(); }
    if (digitalRead(7) == true) 
      {
      digitalWrite(6, HIGH);
      tone(4, 1200, 200);
      delay(200);
      digitalWrite(6, LOW);
      raus++;
      }
    if (raus == 0) goto Schleife5;
    }

    raus = 0;
    Wartenwir:
    if (digitalRead(8) == true or digitalRead(9)==true or digitalRead(10)==true or digitalRead(3)==true or digitalRead(7)==true) {goto Wartenwir;}
    delay(200);
    
  }

  //Wenn der Rundenrekord größer ist als der jemals höchst erziehlte Highscore, dann überschreibe den Highscore und merke dir das am Spielende der neue Highscore gespeichert werden muss.
  if (Runde > Highscoregesamt) 
    {
    NeuerHigh = 1;
    Highscoregesamt = Runde;
    }
  }
//ENDE Hauptprogramm, es geht wieder zurück zum Loop Anfang

//Der Spieler hat die Runde verloren.
void verloren() 
  {
  tone(4, 2000, 1500);
  display.clearDisplay();
  display.display();
  delay(1500);
  Runde--;

//Wenn ein neuer Highscore erreicht wurde, Speicher diesen in den EEPROM
  if (NeuerHigh==0)
    {
    display.clearDisplay();
    display.setTextSize(2);
    display.setTextColor(SSD1306_WHITE);
    display.setCursor(10, 3);
    display.println(F("Game Over"));
    display.setTextSize(3);
    display.setCursor(40, 40);
    display.println(Runde);
    display.display();
    }
    else
    {
    //Wenn ein neuer Highscore erreicht wurde, Speicher diesen in den EEPROM
    NeuerHigh = 0;
    EEPROM.put(2, Highscoregesamt); 
    display.clearDisplay();
    display.setTextSize(2);
    display.setTextColor(SSD1306_WHITE);
    display.setCursor(30, 0);
    display.println(F("Neuer"));
    display.setCursor(0, 20);
    display.println(F("Rekord !!!"));
    display.setTextSize(3);
    display.setCursor(40, 40);
    display.println(Runde);
    display.display();
    }
  delay(5000);
  Runde=0;
  raus=1;

}



void Musik(){

  for (int thisNote = 0; thisNote < notes * 2; thisNote = thisNote + 2) {
    divider = melody[thisNote + 1];
    if (divider > 0) {
      noteDuration = (wholenote) / divider;
    } else if (divider < 0) {
      noteDuration = (wholenote) / abs(divider);
      noteDuration *= 1.5; // increases the duration in half for dotted notes
    }
    tone(buzzer, melody[thisNote], noteDuration * 0.9);
    delay(noteDuration);
    noTone(buzzer);
  }

}

Wenn ich die Musik mit Musik(); aufrufe, klappt das einwandfrei, solange der Aufruf VOR Zeile 174 erfolgt. Möchte ich sie danach aufrufen, macht das Programm garnichts mehr!

Zeile 174 wäre dieser Inhalt:

  // Konfiguration des Display's, dient zu Anzeige des Highscores
  Serial.begin(9600);
  if (!display.begin(SSD1306_SWITCHCAPVCC, SCREEN_ADDRESS)) {
    Serial.println(F("SSD1306 allocation failed"));
    for (;;)
      ;
  }
  display.clearDisplay();
  display.display();
  display.setRotation(2);

Das ist meine Displayausgabe. Sound und Programm davor läuft, ein Aufruf danach führt zum Aufhängen des Programmes.

Bekommst Du denn die Ausgabe noch:

Danach wäre nämlich Schluß.

da ich aktuell leider keine serielle verbindung herstellen kann, kann ich das leider nicht beantworten :frowning:

Dann stelle eine her!

Arduino Mega?

Woran klemmt es?
Und was für ein Controllerboard benutzt Du?
Der PINbelegung nach würde ich auf einen UNO tippen, aber das ist alles graue Theorie - jedoch wichtig!

Edit: Nochwas... Kannst Du mal bitte in Dein erstes Post gehen und die Zeilen
"Ich habe mir den Code von dieser Seite als Beispiel heruntergeladen:" und die nachfolgende mit dem Link zum Code in eine Zeile schreiben?
Ich kann hier nicht vernünftig scrollen, weil mit jedem Stück scrollen die githubseite im Hintergrund neu geladen wird udn das zu Aussetzern führt.

Hast du eine Idee, was diese Zeile macht?

Ne was die Zeile macht kann ich nicht sagen, dazu bin ich zu neu^^
aber ohne die Zeile zeigt mein Display nichts an, dass kann ich sagen :wink:

Ich benutze garkein Arduinio, ich habe nur ein normalen Atmega 328P den ich per ISP Schnittstelle über mein AVRISP II Programmiere.

Zweitrangig.
Ohne Sound funktionierts ja offensichtlich.
MIt Sound nur in Reihenfolge.
Da es keine andere Kontrollmöglichkeit gibt, braucht es die passende oder fehlende Ausgabe.

Nö nicht zweitrangig, wenn die SD das Display nicht erkannt wird, geht er in eine Endlosschleife und genau das scheint zu passieren. Besser wäre es, dann einfach die Soundausgabe nicht aufzurufen.

Gruß Tommy

1 Like

als Display nutze ich dieses:
Amazon 128x64 OLed Display

als Bibliothek eine die ich bei dem Arduino direkt gefunden habe.

Dann wird es aber ganz schwer.
Boah. Ich glaub, ich steig dann aus.
Sorry, aber ich schreibe blind und ohne Hardware Code um. Wenn der aber jetzt auch noch blind getestet werden soll, wird das nix. Der Aufwand ist weder für Dich noch für mich Zielführend...

-> @wwerner Das Display ist ein SSD
-> @Tommy56 ohne Kontrollmöglichkeit ist das raten, somit ist erstrangig die Kontrolle

Aber das ist hier schon wieder genügend abgeschweift.
Ich bin raus.

Wenn du statt "SD" "Display" schreibst, passt es.

Danke, hab es korrigiert.

Gruß Tommy

Wusste nicht das die Hardware so wichtig ist, ist doch nur ein normaler Atmega328P an den das Display hängt.