X-Git-Url: https://git.piment-noir.org/?a=blobdiff_plain;f=src%2Futils%2FCircularArray.ts;h=08a37220c1469e255873ab06482a3193b4c25d87;hb=e4cb2c14d14e95090cca15518594af1c3f369d2b;hp=c46b06b6cb8983bb2cab53bf1a0e248a0452d005;hpb=1699e597e585e54f8f0f540656597d6427080e0c;p=e-mobility-charging-stations-simulator.git

diff --git a/src/utils/CircularArray.ts b/src/utils/CircularArray.ts
index c46b06b6..08a37220 100644
--- a/src/utils/CircularArray.ts
+++ b/src/utils/CircularArray.ts
@@ -1,49 +1,72 @@
-const DEFAULT_CIRCULAR_ARRAY_SIZE = 2000;
+// Copyright Jerome Benoit. 2021. All Rights Reserved.
 
-export default class CircularArray<T> extends Array<T> {
+export const DEFAULT_CIRCULAR_ARRAY_SIZE = Number.MAX_SAFE_INTEGER;
+
+/**
+ * Array with a maximum length shifting items when full.
+ */
+export class CircularArray<T> extends Array<T> {
   public size: number;
 
-  constructor(size?: number) {
+  constructor(size: number = DEFAULT_CIRCULAR_ARRAY_SIZE, ...items: T[]) {
     super();
-    this.size = size ?? DEFAULT_CIRCULAR_ARRAY_SIZE;
+    this.checkSize(size);
+    this.size = size;
+    if (arguments.length > 1) {
+      this.push(...items);
+    }
   }
 
   public push(...items: T[]): number {
-    if (this.length + items.length > this.size) {
-      super.splice(0, (this.length + items.length) - this.size);
+    const length = super.push(...items);
+    if (length > this.size) {
+      super.splice(0, length - this.size);
     }
-    return super.push(...items);
+    return this.length;
   }
 
   public unshift(...items: T[]): number {
-    if (this.length + items.length > this.size) {
-      super.splice(this.size - items.length, (this.length + items.length) - this.size);
+    const length = super.unshift(...items);
+    if (length > this.size) {
+      super.splice(this.size, items.length);
     }
-    return super.unshift(...items);
+    return this.length;
   }
 
-  public concat(...items: (T | ConcatArray<T>)[]): T[] {
-    if (this.length + items.length > this.size) {
-      super.splice(0, (this.length + items.length) - this.size);
+  public concat(...items: (T | ConcatArray<T>)[]): CircularArray<T> {
+    const concatenatedCircularArray = super.concat(items as T[]) as CircularArray<T>;
+    concatenatedCircularArray.size = this.size;
+    if (concatenatedCircularArray.length > concatenatedCircularArray.size) {
+      concatenatedCircularArray.splice(
+        0,
+        concatenatedCircularArray.length - concatenatedCircularArray.size
+      );
     }
-    return super.concat(items as T[]);
+    return concatenatedCircularArray;
   }
 
   public splice(start: number, deleteCount?: number, ...items: T[]): T[] {
-    this.push(...items);
-    return super.splice(start, deleteCount);
+    let itemsRemoved: T[];
+    if (arguments.length >= 3 && typeof deleteCount !== 'undefined') {
+      itemsRemoved = super.splice(start, deleteCount);
+      // FIXME: that makes the items insert not in place
+      this.push(...items);
+    } else if (arguments.length === 2) {
+      itemsRemoved = super.splice(start, deleteCount);
+    } else {
+      itemsRemoved = super.splice(start);
+    }
+    return itemsRemoved;
   }
 
   public resize(size: number): void {
-    if (size < 0) {
-      throw new RangeError(
-        'circular array size does not allow negative values.'
-      );
-    }
+    this.checkSize(size);
     if (size === 0) {
       this.length = 0;
-    } else if (size !== this.size) {
-      this.slice(-size);
+    } else if (size < this.size) {
+      for (let i = size; i < this.size; i++) {
+        super.pop();
+      }
     }
     this.size = size;
   }
@@ -55,4 +78,10 @@ export default class CircularArray<T> extends Array<T> {
   public full(): boolean {
     return this.length === this.size;
   }
+
+  private checkSize(size: number) {
+    if (size < 0) {
+      throw new RangeError('Invalid circular array size');
+    }
+  }
 }