refactor: uses native private fields in classes in stead of fake ones

sverweij · Dec 26, 2024 · 8b11f73 · 8b11f73
1 parent 8cc7b00
commit 8b11f73
Show file tree

Hide file tree

Showing 10 changed files with 94 additions and 94 deletions.
diff --git a/dist/counter.mjs b/dist/counter.mjs
diff --git a/dist/state-machine-model.mjs b/dist/state-machine-model.mjs
diff --git a/docs/index.html b/docs/index.html
@@ -35,7 +35,7 @@
   <meta property="og:image:type" content="image/png" />
   <link rel="canonical" href="https://state-machine-cat.js.org">
   <script nonce="known-inline-script">let LOG = false;</script>
-  <script src="smcat-online-interpreter.min.js" type="module" defer integrity="sha512-M69E3EvXXLoyRDCdiSPk84Pnb7YoMTXRqqH2u5nzRTaxP3PWZw2b1eYhCqsimgWbUaeM5kSCuIxJQhwcNIjf0w=="></script>
+  <script src="smcat-online-interpreter.min.js" type="module" defer integrity="sha512-JFcPEOL9zVRvLlspORt02ETHoKUWv8R2XhdmBO7VI0c6my3zL0u0xgeZUcaYPQWDMLvoj1SFuFJ+vU+4GupJQA=="></script>
   <script defer src="https://code.getmdl.io/1.3.0/material.min.js" async></script>
   <meta name="viewport" content="width=device-width, initial-scale=1.0">
   <meta name="theme-color" content="purple">

diff --git a/docs/inpage.html b/docs/inpage.html
@@ -23,7 +23,7 @@
       src="state-machine-cat-inpage.min.js"
       type="module"
       defer
-      integrity="sha512-10ptXyICQRHVV4qeTbXjuMxYvHieH1nnDhRH4JEYCtWnb+IkVy88qd1pwNJ0waRbKn9eokgxAJgJCb6jnTsGBA=="
+      integrity="sha512-52vZVh/elfXk881aqqod5J6ZLaHea1tt7k0RFATXooP+1NCeIRS0NN0GE8rfDIrMAQWUH0FVxCUKwqaSfGVb2Q=="
     ></script>
     <style>
       body { font-family: sans-serif; margin: 0 auto; max-width: 799px;

diff --git a/docs/smcat-online-interpreter.min.js b/docs/smcat-online-interpreter.min.js
diff --git a/docs/smcat-online-interpreter.min.js.map b/docs/smcat-online-interpreter.min.js.map
diff --git a/docs/state-machine-cat-inpage.min.js b/docs/state-machine-cat-inpage.min.js
diff --git a/docs/state-machine-cat-inpage.min.js.map b/docs/state-machine-cat-inpage.min.js.map
diff --git a/src/counter.mts b/src/counter.mts
@@ -1,12 +1,12 @@
 export class Counter {
-  _lHWM = 0;
+  #lHWM = 0;
 
   constructor(pStart: number = 0) {
-    this._lHWM = pStart;
+    this.#lHWM = pStart;
   }
 
   next(): number {
     // eslint-disable-next-line no-plusplus
-    return ++this._lHWM;
+    return ++this.#lHWM;
   }
 }
diff --git a/src/state-machine-model.mts b/src/state-machine-model.mts
@@ -61,31 +61,31 @@ function flattenTransitions(pStateMachine: IStateMachine): ITransition[] {
 }
 
 export default class StateMachineModel {
-  private _flattenedTransitions: ITransition[];
-  private _flattenedStates: Map<string, IFlattenedState>;
+  #flattenedTransitions: ITransition[];
+  #flattenedStates: Map<string, IFlattenedState>;
 
   constructor(pStateMachine: IStateMachine) {
-    this._flattenedStates = new Map();
-    flattenStatesToMap(pStateMachine.states ?? [], this._flattenedStates);
-    this._flattenedTransitions = flattenTransitions(pStateMachine);
+    this.#flattenedStates = new Map();
+    flattenStatesToMap(pStateMachine.states ?? [], this.#flattenedStates);
+    this.#flattenedTransitions = flattenTransitions(pStateMachine);
   }
 
   get flattenedTransitions(): ITransition[] {
-    return this._flattenedTransitions;
+    return this.#flattenedTransitions;
   }
 
   findStateByName(pName: string): IFlattenedState | undefined {
-    return this._flattenedStates.get(pName);
+    return this.#flattenedStates.get(pName);
   }
 
   findStatesByTypes(pTypes: StateType[]): any[] {
-    return Array.from(this._flattenedStates.values()).filter((pState) =>
+    return Array.from(this.#flattenedStates.values()).filter((pState) =>
       pTypes.includes(pState.type),
     );
   }
 
   findExternalSelfTransitions(pStateName: string): ITransition[] {
-    return this._flattenedTransitions.filter(
+    return this.#flattenedTransitions.filter(
       (pTransition) =>
         pTransition.from === pStateName &&
         pTransition.to === pStateName &&
@@ -94,31 +94,31 @@ export default class StateMachineModel {
   }
 
   findTransitionsByFrom(pFromStateName: string): ITransition[] {
-    return this._flattenedTransitions.filter(
+    return this.#flattenedTransitions.filter(
       (pTransition) => pTransition.from === pFromStateName,
     );
   }
 
   findTransitionsByTo(pToStateName: string): ITransition[] {
-    return this._flattenedTransitions.filter(
+    return this.#flattenedTransitions.filter(
       (pTransition) => pTransition.to === pToStateName,
     );
   }
 
   getMaximumTransitionId(): number {
-    return Math.max(...this._flattenedTransitions.map(({ id }) => id));
+    return Math.max(...this.#flattenedTransitions.map(({ id }) => id));
   }
 
   findTransitionsToSiblings(
     pStateName: string,
     pExcludeIds: Set<number>,
   ): ITransition[] {
-    return this._flattenedTransitions.filter(
+    return this.#flattenedTransitions.filter(
       (pTransition) =>
         !pExcludeIds.has(pTransition.id) &&
         pTransition.from === pStateName &&
-        this._flattenedStates.get(pTransition.to)?.parent ===
-          this._flattenedStates.get(pStateName)?.parent,
+        this.#flattenedStates.get(pTransition.to)?.parent ===
+          this.#flattenedStates.get(pStateName)?.parent,
     );
   }
 }