[Fix] 탁구 게임 기록 페이지 색 변경 오류 수정

[yoouyeon]

탁구 페이지의 url이 / 에서 /takgu로 변경됨

📌 개요

• 페이지 url이 변경되면서 게임 기록 페이지(/takgu/game)의 목록 색깔이 의도대로 변경되지 않던 문제를 수정함

💻 작업사항

• 탁구 서비스의 base url이 / 에서 /takgu로 변경되었는데 페이지 색을 결정하는 조건문에서는 여전히 / 으로 되어 있어 게임 페이지를 제대로 판단하지 못하던 문제가 있었습니다.
• 조건문의 url을 수정해서 문제를 해결했습니다.

사이드 메뉴의 버튼으로 이동할 수 있는 페이지들이 많고 url이 한번 변경된 적도 있다보니
따로 외부 객체에다 페이지별 url 정보들을 정리해두고 코드 내에서는 모두 import해서 쓰는 방식으로 바꿔봐도 괜찮을 것 같아요.

수정 결과 스크린샷

💡관련 Issue

• close [Bug] 탁구 게임 기록 페이지 색 변경 오류 #1581

[nerdchanii]

수정하는 김에 코드의 복잡도를 조금 떨어트리면 좋을 것 같아요. 추천드리는 코드도 조금 복잡하긴 한것 같지만 이런형태면 조금 더 선언적으로 코드를 쓸 수 있지 않을까 싶은데 어떤가요?:)

Suggested change

	pathName === '/takgu'
	? 'main'
	: pathName === '/takgu/game'
	? 'game'
	: 'profile';
	{
	'/takgu': 'main',
	'/takgu/game': 'game'
	}[pathName]||'profile';

[yoouyeon]

아 확인이 늦어서 죄송합니다 🙇 말씀해주신대로 삼항연산자를 중첩하는 지금의 코드는 코드가 복잡하기도 하고 나중에 사용되는 페이지가 추가되는 경우에 코드 수정이 많이 필요할 것 같아서 객체로 관리하는 것이 좋은 것 같습니다!

다만 역시 말씀해주신대로 조건문 안에 바로 객체를 넣다 보니까 좀 복잡하게 느껴지고 가독성이 좋지 않아지는 것 같아서 아래처럼 따로 객체를 선언하는 방식으로 수정해보았습니다...ㅎㅎ

type pageType = 'main' | 'game' | 'profile';

const pageTypeMap: Record<string, pageType> = {
  '/takgu': 'main',
  '/takgu/game': 'game',
};

// 생략

const page: pageType = pageTypeMap[pathName] || 'profile';