Use set over array

MutinyWallet · Jan 13, 2024 · bc8f226 · bc8f226
1 parent fe6caf7
commit bc8f226
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/routes/Search.tsx b/src/routes/Search.tsx
@@ -346,12 +346,12 @@ function GlobalSearch(props: {
     foundNpubs: (string | undefined)[];
 }) {
     const hexpubs = createMemo(() => {
-        const hexpubs: string[] = [];
+        const hexpubs: Set<string> = new Set();
         for (const npub of props.foundNpubs) {
             hexpubFromNpub(npub)
                 .then((h) => {
                     if (h) {
-                        hexpubs.push(h);
+                        hexpubs.add(h);
                     }
                 })
                 .catch((e) => {
@@ -361,7 +361,7 @@ function GlobalSearch(props: {
         return hexpubs;
     });
 
-    async function searchFetcher(args: { value?: string; hexpubs?: string[] }) {
+    async function searchFetcher(args: { value?: string; hexpubs?: Set<string> }) {
         try {
             // Handling case when value starts with "npub"
             if (args.value?.toLowerCase().startsWith("npub")) {
@@ -377,7 +377,7 @@ function GlobalSearch(props: {
 
             // Handling case for other values (name, nip-05, whatever else primal searches)
             const contacts = await searchProfiles(args.value!.toLowerCase());
-            return contacts.filter((c) => !args.hexpubs?.includes(c.hexpub));
+            return contacts.filter((c) => !args.hexpubs?.has(c.hexpub));
         } catch (e) {
             console.error(e);
             return [];