Refactor NoticeAlert: Improve Timeout Handling, Simplify Link Logic, and Enhance Cleanup

[yabi90]

This pull request refactors the NoticeAlert component to improve handling of timeout and cleanup logic, enhance performance, and simplify the rendering logic for the link.

Changes:

1. Refactor Timeout Logic:

   • Previously, the timeout for auto-dismissing the alert and removing it was set directly within the useEffect hook. This change adds a clearTimeout call within the cleanup function to ensure that timeouts are cleared correctly when the component unmounts or re-renders.

2. Optimized Click Handling:

   • The alert dismissal logic on click has been encapsulated within a handleClick function, ensuring the alert's removeAlert function is called after the leave transition, maintaining synchronization with the animation.

3. Simplified Link Rendering:

   • The condition for rendering the link (link !== "") has been simplified to link && to improve readability and avoid unnecessary conditional elements.

4. Code Cleanup:

   • Removed the unused link === "" check. The link is now conditionally rendered only when link is provided.

Benefits:

• Cleaner Timeout Management: Ensures that timeouts are properly cleared, preventing potential memory leaks and ensuring that state updates are not made on unmounted components.
• Improved Readability: The refactored code improves clarity by reducing unnecessary checks and grouping related logic.
• Performance Improvement: The cleanup of timeouts and better handling of side effects reduces unnecessary re-renders and ensures that transitions are more consistent.

Breaking Changes:

• No breaking changes. The component still functions the same way, but with improved logic for managing timeouts and cleanup.