Add #isExpired() private function for DRY TTL expiration check

Extracts TTL expiration logic into a single private function that:
- Returns false if ttl === 0 or item.expiry === 0 (noTTL items)
- Returns true if item.expiry <= Date.now() for items with TTL set

Updates get() and cleanup() to use #isExpired() instead of duplicating logic.
Size reductions from DRY: get() reduced by 4 lines, cleanup() by 5 lines.

Author: avoidwork

coverage.txt

@@ -3,8 +3,8 @@
 ℹ file      | line % | branch % | funcs % | uncovered lines
 ℹ ----------------------------------------------------------
 ℹ src       |        |          |         | 
-ℹ  lru.js   | 100.00 |    99.26 |  100.00 | 
+ℹ  lru.js   | 100.00 |    99.27 |  100.00 | 
 ℹ ----------------------------------------------------------
-ℹ all files | 100.00 |    99.26 |  100.00 | 
+ℹ all files | 100.00 |    99.27 |  100.00 | 
 ℹ ----------------------------------------------------------
 ℹ end of coverage report

dist/tiny-lru.cjs

@@ -157,6 +157,21 @@ class LRU {
 		return item !== undefined ? item.expiry : undefined;
 	}
 
+	/**
+	 * Checks if an item has expired.
+	 *
+	 * @param {Object} item - The cache item to check.
+	 * @returns {boolean} True if the item has expired, false otherwise.
+	 * @private
+	 */
+	#isExpired(item) {
+		if (this.ttl === 0 || item.expiry === 0) {
+			return false;
+		}
+
+		return item.expiry <= Date.now();
+	}
+
 	/**
 	 * Retrieves a value from the cache by key without updating LRU order.
 	 * Note: Does not perform TTL checks or remove expired items.
@@ -179,21 +194,15 @@ class LRU {
 		const item = this.items[key];
 
 		if (item !== undefined) {
-			// Check TTL only if item has expiration set
-			if (this.ttl > 0 && item.expiry !== 0) {
-				if (item.expiry <= Date.now()) {
-					this.delete(key);
-					this.#stats.misses++;
-
-					return undefined;
-				}
+			if (!this.#isExpired(item)) {
+				this.moveToEnd(item);
+				this.#stats.hits++;
+				return item.value;
 			}
 
-			// Fast LRU update without full set() overhead
-			this.moveToEnd(item);
-			this.#stats.hits++;
-
-			return item.value;
+			this.delete(key);
+			this.#stats.misses++;
+			return undefined;
 		}
 
 		this.#stats.misses++;
@@ -471,12 +480,11 @@ class LRU {
 			return 0;
 		}
 
-		const now = Date.now();
 		let removed = 0;
 
 		for (let x = this.first; x !== null; ) {
 			const next = x.next;
-			if (x.expiry !== 0 && x.expiry <= now) {
+			if (this.#isExpired(x)) {
 				const key = x.key;
 				if (this.items[key] !== undefined) {
 					delete this.items[key];

dist/tiny-lru.js

@@ -155,6 +155,21 @@ class LRU {
 		return item !== undefined ? item.expiry : undefined;
 	}
 
+	/**
+	 * Checks if an item has expired.
+	 *
+	 * @param {Object} item - The cache item to check.
+	 * @returns {boolean} True if the item has expired, false otherwise.
+	 * @private
+	 */
+	#isExpired(item) {
+		if (this.ttl === 0 || item.expiry === 0) {
+			return false;
+		}
+
+		return item.expiry <= Date.now();
+	}
+
 	/**
 	 * Retrieves a value from the cache by key without updating LRU order.
 	 * Note: Does not perform TTL checks or remove expired items.
@@ -177,21 +192,15 @@ class LRU {
 		const item = this.items[key];
 
 		if (item !== undefined) {
-			// Check TTL only if item has expiration set
-			if (this.ttl > 0 && item.expiry !== 0) {
-				if (item.expiry <= Date.now()) {
-					this.delete(key);
-					this.#stats.misses++;
-
-					return undefined;
-				}
+			if (!this.#isExpired(item)) {
+				this.moveToEnd(item);
+				this.#stats.hits++;
+				return item.value;
 			}
 
-			// Fast LRU update without full set() overhead
-			this.moveToEnd(item);
-			this.#stats.hits++;
-
-			return item.value;
+			this.delete(key);
+			this.#stats.misses++;
+			return undefined;
 		}
 
 		this.#stats.misses++;
@@ -469,12 +478,11 @@ class LRU {
 			return 0;
 		}
 
-		const now = Date.now();
 		let removed = 0;
 
 		for (let x = this.first; x !== null; ) {
 			const next = x.next;
-			if (x.expiry !== 0 && x.expiry <= now) {
+			if (this.#isExpired(x)) {
 				const key = x.key;
 				if (this.items[key] !== undefined) {
 					delete this.items[key];

dist/tiny-lru.min.js

@@ -148,6 +148,21 @@ export class LRU {
 		return item !== undefined ? item.expiry : undefined;
 	}
 
+	/**
+	 * Checks if an item has expired.
+	 *
+	 * @param {Object} item - The cache item to check.
+	 * @returns {boolean} True if the item has expired, false otherwise.
+	 * @private
+	 */
+	#isExpired(item) {
+		if (this.ttl === 0 || item.expiry === 0) {
+			return false;
+		}
+
+		return item.expiry <= Date.now();
+	}
+
 	/**
 	 * Retrieves a value from the cache by key without updating LRU order.
 	 * Note: Does not perform TTL checks or remove expired items.
@@ -170,21 +185,15 @@ export class LRU {
 		const item = this.items[key];
 
 		if (item !== undefined) {
-			// Check TTL only if item has expiration set
-			if (this.ttl > 0 && item.expiry !== 0) {
-				if (item.expiry <= Date.now()) {
-					this.delete(key);
-					this.#stats.misses++;
-
-					return undefined;
-				}
+			if (!this.#isExpired(item)) {
+				this.moveToEnd(item);
+				this.#stats.hits++;
+				return item.value;
 			}
 
-			// Fast LRU update without full set() overhead
-			this.moveToEnd(item);
-			this.#stats.hits++;
-
-			return item.value;
+			this.delete(key);
+			this.#stats.misses++;
+			return undefined;
 		}
 
 		this.#stats.misses++;
@@ -462,12 +471,11 @@ export class LRU {
 			return 0;
 		}
 
-		const now = Date.now();
 		let removed = 0;
 
 		for (let x = this.first; x !== null; ) {
 			const next = x.next;
-			if (x.expiry !== 0 && x.expiry <= now) {
+			if (this.#isExpired(x)) {
 				const key = x.key;
 				if (this.items[key] !== undefined) {
 					delete this.items[key];