복수형 data fetching을 위한 함수명은 어떻게 작성해야할까

[gorjs3540]

배열의 값을 받아오는 data fetching을 위한 함수를 구현하다 고민이 하나 생겼습니다. 예를 들면,
저는 보통 getGames와 같은 형식으로 {http method}+{목적 단위}+{정렬이나 필터}로 정의해서 사용하고 있었는데요, 다른 코드를 보았을 땐 getGameList와 같은 형식으로도 사용하더라구요. 또, BE쪽에서는 findAll 같은 형식을 사용한다고도 들어서 어떤 방식이 직관적인지, 의미적으로 알맞은지 궁금하기도 합니다. 다들 어떻게 사용하시나요?

// service, frontend API 호출 시
export const getGames = () => { ... };

// UI, state, hook으로 작성 시 -> UI의 의미를 포함
export const useGameList = () => { ... };

// DB, resource 직접 접근 시
export const findAllGames = () => { ... };

제 나름대로 구분을 지어보긴 했는데 일관성이 없다는 생각도 조금 들고 고민이 되네요.

[yoonminsang]

과거에는 ~~list보다는 ~~s를 선호했어요. 다수가 사용하는 컨벤션을 사용한다면 장점이 많기 때문에 코드리뷰에서 변경을 요청받은 경우도 있었습니다.
반면에 s보다는 list가 더 직관적이라는 이유로 list를 사용한경우도 있어요.

findAll같은건 주로 ORM에서 많이 사용했던 것 같아요.

결국 팀 컨벤션에 맞춰 일관성 있게 사용하는 게 가장 중요한 것 같아요. 어떤 방식이든 팀 전체가 동일하게 사용한다면 문제없을 듯 합니다~

[YOOGOMJA]

팀원들과 자주하는 고민 같습니다. 저는 '단위' + s 파 입니다.

이런 경우의 함수 이름은 되도록 데이터 및 타입와 연관 지으려고 노력하는 편인데요.
~List와 ~s는 아래의 차이가 아닐까 생각해봅니다.

interface Game {
  // ...
}

// case 1. Game의 배열을 그냥 Game[] 자체로 사용하는 경우

const getGames = (options?: unknown) => {
  // .. 
  const fetchedGames: Game[] = response.data;
  return fetchedGames
}

// case 2. Game의 배열을 별도의 타입(~List)로 관리하는 경우 

interface List<D> {
  data: D;
  pagination: { 
   currentPage: number; 
   totalPages: number;
   nextPage: number;
   previousPage: number;
  }
}

type GameList = List<Game>;

const getGameList = (options?: unknown) => {
  // ...
  return response.data;
}

위 처럼, 모델로 관리하는 Game에 대해서 그냥 배열 자체로 관리한다면 Games가, Game이 모여있는 것을 List라고 별도 관리하고 있다면 GameList가 좋지않을까? 라고 생각해봅니다.

[YOOGOMJA]

굳이 pagination이 포함된 List 같은게 아니고, Game[]를 type GameList = Game[]이라고 재정의해서 사용하는 경우라도 같을 것이라 생각합니다.