Refatorando código e correções

src/App.tsx

@@ -39,6 +39,8 @@ import { search, home, person } from 'ionicons/icons';
 import { useState, useContext, useEffect } from 'react';
 import React from 'react';
 
+import sessionsService from './services/functions/sessionsService'
+
 setupIonicReact();
 
 const routes = (
@@ -69,8 +71,19 @@ const IonicApp: React.FC = () => {
   user.setIsLoggedIn = setIsLoggedIn;
 
   useEffect(() => {
-    // TODO, verifica se usuário está logado
-    // fazer com serviço externo (evita duplicações)
+    const verifyAuthenticatedUser = async () => {
+      const refreshSessionRes = await sessionsService.refreshSession()
+
+      if (refreshSessionRes.error) {
+        return
+      }
+
+      if (refreshSessionRes.userId) {
+        setIsLoggedIn(true)
+      }
+    }
+
+    verifyAuthenticatedUser()
   })
 
   return(

src/pages/Perfil.tsx

@@ -7,7 +7,6 @@ import {
   IonContent,
   IonFab,
   IonFabButton,
-  IonGrid,
   IonHeader,
   IonIcon,
   IonItem,
@@ -24,12 +23,11 @@ import React, { useState, useEffect, useReducer } from "react";
 import { IonRow, IonCol } from "@ionic/react";
 import { createOutline } from "ionicons/icons";
 
-import * as sessionRoutes from '../services/api/session';
-import * as usersRoutes from '../services/api/users';
-
 import './Perfil.css'
 import LocalStorage from "../LocalStorage";
-import { refreshSession } from "../services/refreshSession";
+
+import sessionsService from '../services/functions/sessionsService'
+import usersService from '../services/functions/usersService'
 
 const Perfil: React.FC = () => {
   const [showToast, setShowToast] = useState(false);
@@ -48,7 +46,6 @@ const Perfil: React.FC = () => {
 
   const history = useHistory();
 
-  useEffect(() => {
   const redirectUserToLogin = () => {
     // TODO, não impede o usuário de retornar a página de login
     history.push({ pathname: '/login' });
@@ -56,11 +53,12 @@ const Perfil: React.FC = () => {
     setShowToast(true);
   }
 
+  useEffect(() => {
     const loadUserData = async () => {
       let userId = ''
   
       // verify if user is authenticated
-      const refreshSessionRes = await refreshSession()
+      const refreshSessionRes = await sessionsService.refreshSession()
   
       if (refreshSessionRes.error) {
         redirectUserToLogin()
@@ -72,17 +70,19 @@ const Perfil: React.FC = () => {
       }
       
       // get user info by ID
-      const getByIdRes = await usersRoutes.getById(userId)
+      const getByIdRes = await usersService.getById(userId)
   
       if (getByIdRes.error) {
-        setMessageToast(getByIdRes.message.data)
+        setMessageToast(getByIdRes.error.errorMessage)
         setShowToast(true)
   
         return
       }
   
-      const userData = getByIdRes.data
+      if (getByIdRes.userData) {
+        const userData = getByIdRes.userData
   
+        if (isMounted) {
           setInputValues({
             'name': userData.name,
             'lastname': userData.lastname,
@@ -91,6 +91,10 @@ const Perfil: React.FC = () => {
             'bio': userData.bio
           });
         }
+      }
+    }
+
+    let isMounted = true;
     
     const userToken = LocalStorage.getToken()
 
@@ -99,7 +103,10 @@ const Perfil: React.FC = () => {
     }
     
     loadUserData()
-  }, [history]);
+
+    return () => { isMounted = false };
+  }, []);
+  // }, [history]);
 
   return (
     <IonPage>

src/services/functions/sessionsService.ts

@@ -1,4 +1,4 @@
-import * as sessionRoutes from "../services/api/session";
+import * as sessionRoutes from "../api/session";
 
 interface refreshSessionReturn {
   userId?: string;
@@ -12,7 +12,7 @@ interface refreshSessionResponse {
   userId?: string;
 }
 
-export const refreshSession = async (): Promise<refreshSessionReturn> => {
+const refreshSession = async (): Promise<refreshSessionReturn> => {
   try {
     let res: refreshSessionResponse = await sessionRoutes.refresh()
 
@@ -42,3 +42,5 @@ export const refreshSession = async (): Promise<refreshSessionReturn> => {
   //   }
 // }
 };
+
+export default { refreshSession }

src/services/functions/usersService.ts

@@ -0,0 +1,53 @@
+import * as usersRoutes from "../api/users";
+
+interface getByIdReturn {
+  userData?: {
+    name: string;
+    lastname: string;
+    email: string;
+    birth_date: string;
+    bio: string;
+  },
+  error?: {
+    errorMessage: string;
+  }
+}
+
+interface getByIdRes {
+  status: string;
+  message: string;
+  userId?: string;
+  data: {
+    name: string;
+    lastname: string;
+    email: string;
+    birth_date: string;
+    bio: string;
+  },
+}
+
+const getById = async (userId: string): Promise<getByIdReturn> => {
+  try {
+    let res: getByIdRes = await usersRoutes.getById(userId)
+
+    if (res.status === "error") {
+      return {
+        error: {
+          errorMessage: res.message,
+        }
+      };
+    }
+
+    return {
+      userData: res.data,
+    };
+  } catch(err) {
+    return {
+      error: {
+        errorMessage: "Por favor, autentique-se.",
+      }
+    };
+  }
+};
+
+export default { getById }

src/services/users/getById.ts

@@ -1,35 +0,0 @@
-import * as usersRoutes from "../../services/api/users";
-
-interface getByIdReturn {
-  userId?: string;
-  error?: boolean;
-  errorMessage?: string;
-}
-
-interface getByIdResponse {
-  status?: string;
-  message?: string;
-  userId?: string;
-}
-
-export const getById = async (userId: string): Promise<getByIdReturn> => {
-  try {
-    let res: getByIdResponse = await usersRoutes.getById(userId)
-
-    if (res.status === "error") {
-      return {
-        error: true,
-        errorMessage: res.message,
-      };
-    }
-
-    return {
-      userId: res.userId,
-    };
-  } catch(err) {
-    return {
-      error: true,
-      errorMessage: "Por favor, autentique-se.",
-    };
-  }
-};