refactor: removed unnecessary checks

Removed redundant checks for undefined values

Refs #254

Author: douugdev

dist/src/hooks/useMMKV.d.ts

@@ -28,8 +28,8 @@ export declare const create: CreateType;
  */
 declare type CreateType = (storage: MMKVInstance) => {
     <T = undefined>(key: string): [
-        value: T | null | undefined,
-        setValue: (value: (T | null | undefined) | ((prevValue: T | null | undefined) => T | null | undefined)) => void
+        value: T | undefined,
+        setValue: (value: (T | undefined) | ((prevValue: T | undefined) => T | undefined)) => void
     ];
     <T>(key: string, defaultValue: T): [
         value: T,
@@ -68,10 +68,10 @@ export declare const useMMKVStorage: UseMMKVStorageType;
  */
 declare type UseMMKVStorageType = {
     <T = undefined>(key: string, storage: MMKVInstance): [
-        value: T | null | undefined,
-        setValue: (value: (T | null | undefined) | ((prevValue: T | null | undefined) => T | null | undefined)) => void
+        value: T | undefined,
+        setValue: (value: (T | undefined) | ((prevValue: T | undefined) => T | undefined)) => void
     ];
-    <T>(key: string, storage: MMKVInstance, defaultValue: T): [
+    <T>(key: string, storage: MMKVInstance, defaultValue: T | undefined): [
         value: T,
         setValue: (value: T | ((prevValue: T) => T)) => void
     ];

dist/src/hooks/useMMKV.d.ts.map

@@ -62,12 +62,7 @@ export var create = function (storage) {
     return function (key, defaultValue) {
         if (!key || typeof key !== 'string' || !storage)
             throw new Error('Key and Storage are required parameters.');
-        if (defaultValue === undefined) {
-            return useMMKVStorage(key, storage);
-        }
-        else {
-            return useMMKVStorage(key, storage, defaultValue);
-        }
+        return useMMKVStorage(key, storage, defaultValue);
     };
 };
 /**
@@ -171,7 +166,10 @@ export var useMMKVStorage = function (key, storage, defaultValue) {
             return [2 /*return*/];
         });
     }); }, [key, storage, valueType]);
-    return [value !== null && value !== void 0 ? value : (defaultValue === undefined ? null : defaultValue), setNewValue];
+    return [
+        valueType === 'boolean' || valueType === 'number' ? value : value || defaultValue,
+        setNewValue
+    ];
 };
 function usePrevious(value) {
     var ref = useRef(value);

dist/src/hooks/useMMKV.js

@@ -29,11 +29,8 @@ export const create: CreateType =
   <T = undefined>(key: string, defaultValue?: T) => {
     if (!key || typeof key !== 'string' || !storage)
       throw new Error('Key and Storage are required parameters.');
-    if (defaultValue === undefined) {
-      return useMMKVStorage<T>(key, storage);
-    } else {
-      return useMMKVStorage<T>(key, storage, defaultValue);
-    }
+
+    return useMMKVStorage<T>(key, storage, defaultValue);
   };
 
 /**
@@ -43,10 +40,8 @@ export const create: CreateType =
  */
 type CreateType = (storage: MMKVInstance) => {
   <T = undefined>(key: string): [
-    value: T | null | undefined,
-    setValue: (
-      value: (T | null | undefined) | ((prevValue: T | null | undefined) => T | null | undefined)
-    ) => void
+    value: T | undefined,
+    setValue: (value: (T | undefined) | ((prevValue: T | undefined) => T | undefined)) => void
   ];
   <T>(key: string, defaultValue: T): [
     value: T,
@@ -173,7 +168,10 @@ export const useMMKVStorage: UseMMKVStorageType = <T = undefined>(
     [key, storage, valueType]
   );
 
-  return [value ?? (defaultValue === undefined ? null : defaultValue), setNewValue];
+  return [
+    valueType === 'boolean' || valueType === 'number' ? value : value || defaultValue,
+    setNewValue
+  ];
 };
 
 function usePrevious(value: any) {
@@ -192,12 +190,10 @@ function usePrevious(value: any) {
  */
 type UseMMKVStorageType = {
   <T = undefined>(key: string, storage: MMKVInstance): [
-    value: T | null | undefined,
-    setValue: (
-      value: (T | null | undefined) | ((prevValue: T | null | undefined) => T | null | undefined)
-    ) => void
+    value: T | undefined,
+    setValue: (value: (T | undefined) | ((prevValue: T | undefined) => T | undefined)) => void
   ];
-  <T>(key: string, storage: MMKVInstance, defaultValue: T): [
+  <T>(key: string, storage: MMKVInstance, defaultValue: T | undefined): [
     value: T,
     setValue: (value: T | ((prevValue: T) => T)) => void
   ];