Clean up initialization.

It's now just about callback updates when the module changes.

Author: microbit-matt-hillsdon

src/board/accelerometer.ts

@@ -48,8 +48,8 @@ export class Accelerometer {
 
   setValue(id: StateKeys, value: any) {
     this.state[id].setValue(value);
-    if (id === "gesture") {
-      this.gestureCallback!(
+    if (id === "gesture" && this.gestureCallback) {
+      this.gestureCallback(
         convertAccelerometerStringToNumber(this.state.gesture.value)
       );
     }
@@ -71,7 +71,7 @@ export class Accelerometer {
     });
   }
 
-  initialize(gestureCallback: GestureCallback) {
+  initializeCallbacks(gestureCallback: GestureCallback) {
     this.gestureCallback = gestureCallback;
   }
 

src/board/audio/index.ts

@@ -23,7 +23,10 @@ export class Audio {
 
   constructor() {}
 
-  initialize({ defaultAudioCallback, speechAudioCallback }: AudioOptions) {
+  initializeCallbacks({
+    defaultAudioCallback,
+    speechAudioCallback,
+  }: AudioOptions) {
     this.context = new AudioContext({
       // The highest rate is the sound expression synth.
       sampleRate: 44100,

src/board/buttons.ts

@@ -115,8 +115,6 @@ export class Button {
     return result;
   }
 
-  initialize() {}
-
   dispose() {
     this._presses = 0;
   }

src/board/index.ts

@@ -169,14 +169,14 @@ export class Board {
       noInitialRun: true,
     });
     const module = new ModuleWrapper(wrapped);
-    this.audio.initialize({
+    this.audio.initializeCallbacks({
       defaultAudioCallback: wrapped._microbit_hal_audio_ready_callback,
       speechAudioCallback: wrapped._microbit_hal_audio_speech_ready_callback,
     });
-    this.buttons.forEach((b) => b.initialize());
-    this.pins.forEach((p) => p.initialize());
-    this.accelerometer.initialize(wrapped._microbit_hal_gesture_callback);
-    this.microphone.initialize(
+    this.accelerometer.initializeCallbacks(
+      wrapped._microbit_hal_gesture_callback
+    );
+    this.microphone.initializeCallbacks(
       wrapped._microbit_hal_audio_speech_ready_callback
     );
     return module;

src/board/microphone.ts

@@ -50,14 +50,16 @@ export class Microphone {
 
     const low = this.soundLevel.lowThreshold!;
     const high = this.soundLevel.highThreshold!;
-    if (prev > low && curr <= low) {
-      this.soundLevelCallback!(MICROBIT_HAL_MICROPHONE_EVT_THRESHOLD_LOW);
-    } else if (prev < high && curr >= high!) {
-      this.soundLevelCallback!(MICROBIT_HAL_MICROPHONE_EVT_THRESHOLD_HIGH);
+    if (this.soundLevelCallback) {
+      if (prev > low && curr <= low) {
+        this.soundLevelCallback(MICROBIT_HAL_MICROPHONE_EVT_THRESHOLD_LOW);
+      } else if (prev < high && curr >= high!) {
+        this.soundLevelCallback(MICROBIT_HAL_MICROPHONE_EVT_THRESHOLD_HIGH);
+      }
     }
   }
 
-  initialize(soundLevelCallback: (v: number) => void) {
+  initializeCallbacks(soundLevelCallback: (v: number) => void) {
     this.soundLevelCallback = soundLevelCallback;
   }
 

src/board/pins.ts

@@ -113,7 +113,5 @@ export class Pin {
     }
   }
 
-  initialize() {}
-
   dispose() {}
 }