Каков наилучший способ сокращения множественного оператора if?

Question

У меня есть вспомогательная функция, которая создает объект с соответствующими свойствами запроса.Я использую этот объект как тело в моем обещании.Каков самый элегантный способ рефакторинга нескольких операторов if?Вот функция:

 getQueryParams = (query, pagination, sorting) => {
        let queryParam = {}

        if (pagination && pagination.pageNumber) {
            queryParam.page = `${pagination.pageNumber}`
        }
        if (pagination && pagination.rowsOnPage) {
            queryParam.size = `${pagination.rowsOnPage}`
        }
        if (query) {
            const updatedQuery = encodeURIComponent(query)
            queryParam.q = `${updatedQuery}`
        }
        if (sorting) {
            queryParam.sort = `${sorting.isDescending ? '-' : ''}${sorting.name}`
        }
        return service.get(`/my-url/`, queryParam).then(result => {
            return result
        })
    }

Answer 1

Если service проверяет свои параметры (как и должно быть), вы можете воспользоваться параметрами по умолчанию .Примерно так:

const getQueryParams = (
  query = '',
  pagination = {pageNumber: 0, rowsOnPage: 0},
  sorting = {isDescending: '', name: ''}
) => {
  const queryParam = {
    page: pagination.pageNumber,
    size: pagination.rowsOnPage,
    q: encodeURIComponent(query),
    sort: `${sorting.isDescending}${sorting.name}`    
  }
  return ...;
};

Живой пример для игры на jsfiddle .

Answer 2

Это идея, как это может выглядеть, но вы должны принять ваши параметры раньше:

  const query = new URLSearchParams();
  Object.keys(params).forEach(key => {
    if (params[key]) {
      query.append(key, params[key]);
    }
  });