From 5602c29b158e38d10f3723172a46c223ea461b63 Mon Sep 17 00:00:00 2001
From: Sergey Yarkov <serzh.yarkov@gmail.com>
Date: Fri, 11 Feb 2022 11:55:56 +0300
Subject: [PATCH] feat: fethincg user in messenger & improved perfomance

---
 src/components/Chat/Sidebar/Sidebar.tsx       |   2 -
 src/components/Chat/Window/Message/Avatar.tsx |   6 +-
 .../Chat/Window/Message/Message.tsx           |  23 ++-
 src/components/Chat/Window/Window.tsx         | 174 +++++++++++-------
 src/hooks/queries/index.ts                    |   2 +
 5 files changed, 135 insertions(+), 72 deletions(-)

diff --git a/src/components/Chat/Sidebar/Sidebar.tsx b/src/components/Chat/Sidebar/Sidebar.tsx
index e346039..202cb36 100644
--- a/src/components/Chat/Sidebar/Sidebar.tsx
+++ b/src/components/Chat/Sidebar/Sidebar.tsx
@@ -20,8 +20,6 @@ const Sidebar: React.FC<GridItemProps> & ISidebarComposition = props => {
       {props.children}
     </GridItem>
   );
-
-  return null;
 };
 
 Sidebar.Heading = Heading;
diff --git a/src/components/Chat/Window/Message/Avatar.tsx b/src/components/Chat/Window/Message/Avatar.tsx
index 525c1a2..6887fe1 100644
--- a/src/components/Chat/Window/Message/Avatar.tsx
+++ b/src/components/Chat/Window/Message/Avatar.tsx
@@ -1,16 +1,16 @@
-import React from 'react';
+import React, { memo } from 'react';
 import { WrapItem, Avatar as ChakraAvatar } from '@chakra-ui/react';
 
 export interface IAvatarProps {
   fullname: string;
 }
 
-const Avatar: React.FC<IAvatarProps> = ({ fullname: name }) => {
+const Avatar: React.FC<IAvatarProps> = memo(({ fullname: name }) => {
   return (
     <WrapItem>
       <ChakraAvatar name={name} size={'sm'} />
     </WrapItem>
   );
-};
+});
 
 export { Avatar };
diff --git a/src/components/Chat/Window/Message/Message.tsx b/src/components/Chat/Window/Message/Message.tsx
index cd4d5de..3d9aa13 100644
--- a/src/components/Chat/Window/Message/Message.tsx
+++ b/src/components/Chat/Window/Message/Message.tsx
@@ -1,5 +1,5 @@
-import React, { useEffect, useRef } from 'react';
-import { Flex, Stack } from '@chakra-ui/react';
+import React, { memo, useEffect, useRef } from 'react';
+import { Flex, Box, Stack, Text as ChakraText } from '@chakra-ui/react';
 import { Avatar, IAvatarProps } from './Avatar';
 import { Text, ITextProps } from './Text';
 import { MessageContext } from './context';
@@ -14,7 +14,7 @@ interface IMyMessageProps {
   isMyMessage: boolean;
 }
 
-const MessageList: React.FC<{ messages: MessageType[] }> = ({ messages, children }) => {
+const MessageList: React.FC<{ messages: MessageType[] }> = memo(({ messages }) => {
   const listRef = useRef<HTMLDivElement | null>(null);
 
   useEffect(() => {
@@ -25,10 +25,23 @@ const MessageList: React.FC<{ messages: MessageType[] }> = ({ messages, children
 
   return (
     <Stack ref={listRef} h='65vh' overflowY={'scroll'} mb='12' mt='6' spacing={'3'} pr='8'>
-      {children}
+      {messages.length !== 0 ? (
+        messages.map((message, i) => (
+          <Message key={i} isMyMessage={message.isMyMessage}>
+            <Message.Avatar fullname={message.userName} />
+            <Message.Text content={message.content} time={message.time} />
+          </Message>
+        ))
+      ) : (
+        <Box>
+          <ChakraText mt='5' textAlign={'center'} color='gray.500' userSelect={'none'}>
+            Your message history will be displayed here.
+          </ChakraText>
+        </Box>
+      )}
     </Stack>
   );
-};
+});
 
 const Message: React.FC<IMyMessageProps> & IMessageComposition = ({ children, isMyMessage }) => {
   return (
diff --git a/src/components/Chat/Window/Window.tsx b/src/components/Chat/Window/Window.tsx
index c1c237e..26ffb6d 100644
--- a/src/components/Chat/Window/Window.tsx
+++ b/src/components/Chat/Window/Window.tsx
@@ -1,21 +1,42 @@
-import React, { ChangeEvent, useContext, useEffect, useState } from 'react';
+import React, { ChangeEvent } from 'react';
+import axios from 'axios';
 import { GridItem, GridItemProps, Flex, Box, Icon, Text, Input, Heading, IconButton, Divider } from '@chakra-ui/react';
 import { MdOutlineChat, MdSend } from 'react-icons/md';
-import { Message, MessageList } from './Message';
-import { useRootStore } from '@educt/hooks/useRootStore';
+import { MessageList } from './Message';
 import { SocketContext } from '@educt/contexts';
-import { useSocketEvent } from '@educt/hooks/useSocketEvent';
 import { MessageType } from '@educt/types';
+import { UserServiceInstance } from '@educt/services';
+import { IUser } from '@educt/interfaces';
+import LoadingList from '@educt/components/LoadingList';
 
-const Window: React.FC<GridItemProps> = props => {
+/**
+ * Hooks
+ */
+import { useContext, useEffect, useState } from 'react';
+import { useToast } from '@chakra-ui/react';
+import { useSocketEvent } from '@educt/hooks/useSocketEvent';
+import { useRootStore } from '@educt/hooks/useRootStore';
+import { useHistory } from 'react-router-dom';
+
+const Wrapper: React.FC<GridItemProps> = props => (
+  <GridItem pt={'7rem'} pb='5' px='12' {...props}>
+    {props.children}
+  </GridItem>
+);
+
+const Window: React.FC = props => {
   const {
     userStore: { me },
     onlineStore,
   } = useRootStore();
   const chatId = new URLSearchParams(location.search).get('chat_id');
+  const [isLoading, setIsLoading] = useState<boolean>(false);
+  const [selectedUser, setSelectedUser] = useState<IUser | null>(null);
   const [inputValue, setInputValue] = useState<string>('');
   const [messages, setMessages] = useState<MessageType[]>([]);
   const { socket } = useContext(SocketContext);
+  const history = useHistory();
+  const toast = useToast();
 
   const onSubmit = (e: React.FormEvent<HTMLFormElement>) => {
     e.preventDefault();
@@ -44,16 +65,47 @@ const Window: React.FC<GridItemProps> = props => {
 
   const onChangeInputValue = (e: ChangeEvent<HTMLInputElement>) => setInputValue(e.target.value);
 
+  /**
+   * Load user data by `chat_id`
+   */
+  useEffect(() => {
+    if (chatId) {
+      (async () => {
+        try {
+          setIsLoading(true);
+          const { data: user } = await UserServiceInstance.fetchUserById(chatId);
+          setSelectedUser(user);
+        } catch (error) {
+          if (axios.isAxiosError(error)) {
+            if (error.response?.status === 404) {
+              toast({ title: 'Chat not found.', status: 'warning' });
+              history.replace({ search: '' });
+            }
+          }
+
+          console.error(error);
+        } finally {
+          setIsLoading(false);
+        }
+      })();
+    }
+  }, [chatId]);
+
+  // TODO load message history
+  /**
+   * Loading messages history
+   */
   useEffect(() => {
-    // TODO load message history
     console.log('chat id changed!');
     setMessages([]);
   }, [chatId]);
 
   // TODO add types for this event
+  /**
+   * Event on accept new messages
+   */
   useSocketEvent('message:private', data => {
     const user = onlineStore.getUser(data.from);
-    console.log('message!');
 
     if (user) {
       if (chatId === data.from) {
@@ -74,68 +126,66 @@ const Window: React.FC<GridItemProps> = props => {
     }
   });
 
-  return (
-    <GridItem pt={'7rem'} pb='5' px='12' {...props}>
-      {chatId ? (
-        <Flex flexDir={'column'}>
-          <Box my='3'>
-            <Heading as='h2' fontWeight={'semibold'} fontSize='3xl'>
-              {chatId}
-            </Heading>
-          </Box>
+  if (isLoading) {
+    return (
+      <Wrapper>
+        <LoadingList />
+      </Wrapper>
+    );
+  }
 
-          <Divider my='2' />
-
-          <MessageList messages={messages}>
-            {messages.length !== 0 ? (
-              messages.map((message, i) => (
-                <Message key={i} isMyMessage={message.isMyMessage}>
-                  <Message.Avatar fullname={message.userName} />
-                  <Message.Text content={message.content} time={message.time} />
-                </Message>
-              ))
-            ) : (
-              <Box>
-                <Text mt='5' textAlign={'center'} color='gray.500' userSelect={'none'}>
-                  Your message history will be displayed here.
-                </Text>
-              </Box>
-            )}
-          </MessageList>
-          <form onSubmit={e => onSubmit(e)}>
-            <Flex mt='auto' alignItems={'flex-end'}>
-              <Input
-                onChange={onChangeInputValue}
-                value={inputValue}
-                resize={'block'}
-                type='text'
-                placeholder='Write a message...'
-                maxW='900px'
-                variant={'outline'}
-                rounded='xl'
-                mr='2'
-              />
-              <IconButton
-                type='submit'
-                icon={<MdSend />}
-                aria-label='Send message'
-                variant={'ghost'}
-                _hover={{ bg: 'inherit' }}
-                _active={{ bg: 'inherit' }}
-                _focus={{ outline: 'none' }}
-              />
-            </Flex>
-          </form>
-        </Flex>
-      ) : (
+  /**
+   * Unselected user
+   */
+  if (!chatId || selectedUser === null) {
+    return (
+      <Wrapper>
         <Box textAlign={'center'} mt={{ base: '0', lg: '15rem' }}>
           <Icon as={MdOutlineChat} color='gray.600' w='16' h='16' />
           <Text color='gray.500' userSelect={'none'}>
             Select a chat to start messaging
           </Text>
         </Box>
-      )}
-    </GridItem>
+      </Wrapper>
+    );
+  }
+
+  return (
+    <Wrapper {...props}>
+      <Flex flexDir={'column'}>
+        <Box my='3'>
+          <Heading as='h2' fontWeight={'semibold'} fontSize='3xl'>
+            {selectedUser.fullname}
+          </Heading>
+        </Box>
+        <Divider my='2' />
+        <MessageList messages={messages} />
+        <form onSubmit={e => onSubmit(e)}>
+          <Flex mt='auto' alignItems={'flex-end'}>
+            <Input
+              onChange={onChangeInputValue}
+              value={inputValue}
+              resize={'block'}
+              type='text'
+              placeholder='Write a message...'
+              maxW='900px'
+              variant={'outline'}
+              rounded='xl'
+              mr='2'
+            />
+            <IconButton
+              type='submit'
+              icon={<MdSend />}
+              aria-label='Send message'
+              variant={'ghost'}
+              _hover={{ bg: 'inherit' }}
+              _active={{ bg: 'inherit' }}
+              _focus={{ outline: 'none' }}
+            />
+          </Flex>
+        </form>
+      </Flex>
+    </Wrapper>
   );
 };
 
diff --git a/src/hooks/queries/index.ts b/src/hooks/queries/index.ts
index 27ee318..95d8a7e 100644
--- a/src/hooks/queries/index.ts
+++ b/src/hooks/queries/index.ts
@@ -25,6 +25,7 @@ import { useDetachStudents } from './course/useDetachStudents';
 /**
  * User
  */
+import { useFetchUser } from './user/useFetchUser';
 import { useCreateUser } from './user/useCreateUser';
 import { useDeleteUser } from './user/useDeleteUser';
 import { useUpdateUser } from './user/useUpdateUser';
@@ -57,6 +58,7 @@ export {
   useCreateLesson,
   useDeleteLesson,
   useUpdateLesson,
+  useFetchUser,
   useCreateUser,
   useDeleteUser,
   useUpdateUser,