combine note statu update and cursor movement into midi event

hall · Mar 31, 2023 · c9a576a · c9a576a
1 parent 126fa3f
commit c9a576a
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 19 deletions.
diff --git a/src/app/home/home.component.ts b/src/app/home/home.component.ts
@@ -33,7 +33,10 @@ export class HomePageComponent implements OnInit {
     private osmdService: OsmdService,
     public changeRef: ChangeDetectorRef
   ) {
-    midi.onChange.subscribe(this.osmdCursorPlayMoveNext());
+    midi.onChange.subscribe(() => {
+      if (this.pianoKeyboard) this.notes.updateNotesStatus();
+      if (this.notes.checkRequired()) this.osmdCursorPlayMoveNext();
+    });
   }
 
   ngOnInit(): void {
@@ -235,13 +238,13 @@ export class HomePageComponent implements OnInit {
       if (index != 0) cursor.show();
       cursor.reset();
     });
-    // // Additional tasks in case of new start, not required in repetition
+    // TODO: Additional tasks in case of new start, not required in repetition
     // if (this.settings.repeat == this.settings.repeatCfg) {
-    //   this.notes.clear();
-    //   // free auto pressed notes
-    //   for (const [key] of this.midi.mapNotesAutoPressed) {
-    //     this.midi.releaseNote(parseInt(key) + 12);
-    //   }
+    this.notes.clear();
+    // free auto pressed notes
+    for (const [key] of this.midi.mapNotesAutoPressed) {
+      this.midi.releaseNote(parseInt(key) + 12);
+    }
     // }
 
     this.osmdService.hideFeedback();

diff --git a/src/app/services/midi.service.ts b/src/app/services/midi.service.ts
@@ -83,7 +83,6 @@ export class MidiService {
     this.initDev(access);
   }
 
-  // Press note on Ouput MIDI Device
   pressNote(pitch: number, velocity: number): void {
     this.mapNotesAutoPressed.set((pitch - 12).toFixed(), 1);
     const iter = this.outputs.values();
@@ -123,11 +122,7 @@ export class MidiService {
       this.osmd.textFeedback('❌', 0, 20);
     }
 
-    // TODO: if (this.homePageComponent.pianoKeyboard)
-    this.notes.updateNotesStatus();
-    if (this.notes.checkRequired())
-      // HomePageComponent.osmdCursorPlayMoveNext();
-      this.onChange.emit({});
+    this.onChange.emit();
   }
 
   // Midi input note released
@@ -136,11 +131,7 @@ export class MidiService {
     const name = halbTone.toFixed();
     this.notes.release(name);
 
-    // TODO: if (this.homePageComponent.pianoKeyboard)
-    this.notes.updateNotesStatus();
-    if (this.notes.checkRequired())
-      // HomePageComponent.osmdCursorPlayMoveNext();
-      this.onChange.emit({});
+    this.onChange.emit();
   }
 
   // Refresh wakelock for two minutes

diff --git a/src/app/services/notes.service.ts b/src/app/services/notes.service.ts
@@ -17,7 +17,7 @@ export type NoteObject = {
 })
 export class NotesService {
   mapPressed: Map<string, number> = new Map();
-  // Initialize maps of notes comming from Music Sheet
+  // Initialize maps of notes coming from Music Sheet
   mapRequired = new Map<string, NoteObject>();
   mapPrevRequired = new Map<string, NoteObject>();