Implement rudimentary caching in KVC

Author: hugithordarson

ng-appserver/src/main/java/ng/appserver/NGApplication.java

@@ -36,6 +36,7 @@
 import ng.appserver.templating.NGElementManager;
 import ng.appserver.templating.NGElementManager.ElementProvider;
 import ng.appserver.wointegration.NGDefaultLifeBeatThread;
+import ng.kvc.NGKeyValueCoding;
 import ng.plugins.Elements;
 import ng.plugins.NGCorePlugin;
 import ng.plugins.NGDevelopmentPlugin;
@@ -162,6 +163,9 @@ public static <E extends NGApplication> E runAndReturn( final String[] args, fin
 			// If we're in development mode, activate the development plugin for some bonus development features
 			if( isDevelopmentMode ) {
 				application.plugins.add( new NGDevelopmentPlugin() );
+
+				// FIXME: Most definitely not the way we're going to use to decide if KVC caching is enabled. Under development // Hugi 2025-04-21
+				NGKeyValueCoding.DefaultImplementation.setCachingEnabled( false );
 			}
 
 			// CHECKME: We probably need more extension points for plugin initialization (pre-constructor, post-constructor etc.) // Hugi 2023-07-28
@@ -368,7 +372,8 @@ private NGRequestHandler handlerForURL( String url ) {
 	 * @return true if we're in development mode
 	 */
 	public boolean isDevelopmentMode() {
-		return _deploymentMode == StandardDeploymentMode.Development;
+		return false;
+		//		return _deploymentMode == StandardDeploymentMode.Development;
 	}
 
 	/**

ng-core/src/main/java/ng/kvc/NGKeyValueCoding.java

@@ -7,7 +7,9 @@
 import java.lang.reflect.Modifier;
 import java.math.BigDecimal;
 import java.util.List;
+import java.util.Map;
 import java.util.Objects;
+import java.util.concurrent.ConcurrentHashMap;
 
 import ng.NGRuntimeException;
 
@@ -114,9 +116,45 @@ public static void takeValueForKey( final Object object, final Object value, fin
 		}
 
 		/**
-		 * @return A KVC binding for the given class and key.
+		 * FIXME: Most definitely not the way we're going to use to set if KVC caching is enabled // Hugi 2025-04-21
+		 */
+		public static void setCachingEnabled( boolean value ) {
+			_cachingEnabled = value;
+		}
+
+		/**
+		 * FIXME: Most definitely not the way we're going to use to decide if KVC caching is enabled // Hugi 2025-04-21
 		 */
+		public static boolean _cachingEnabled = true;
+
+		/**
+		 * FIXME: Currently just a very primitive cache for testing // Hugi 2025-04-21
+		 */
+		private static Map<CacheKey, KVCReadBinding> _readBindingCache = new ConcurrentHashMap<>();
+
+		private record CacheKey( Class<?> clazz, String key ) {}
+
 		private static KVCReadBinding readBindingForKey( final Object object, final String key ) {
+
+			if( !_cachingEnabled ) {
+				return locateBindingForKey( object, key );
+			}
+
+			final CacheKey cacheKey = new CacheKey( object.getClass(), key );
+			KVCReadBinding entry = _readBindingCache.get( cacheKey );
+
+			if( entry == null ) {
+				entry = locateBindingForKey( object, key );
+				_readBindingCache.put( cacheKey, entry );
+			}
+
+			return entry;
+		}
+
+		/**
+		 * @return A KVC binding for the given class and key.
+		 */
+		private static KVCReadBinding locateBindingForKey( final Object object, final String key ) {
 			Objects.requireNonNull( object );
 			Objects.requireNonNull( key );