使用多个可选参数重构 JS 方法

Question

我有一个函数接收两个参数，我希望它们是可选的（最后两个）：

const renderDate = (date, locale, format) => {
  if (locale === undefined) { locale = 'en' }
  if (format === undefined) { format = 'D MMM' }
  if (locale === 'de' && format === 'D MMM') {
    const indexD = format.lastIndexOf('D')
    format = `${format.slice(0, indexD + 1)}o${format.slice(indexD + 1, format.length)}`
  }

  moment.locale(locale)
  return moment(date).format(format)
}

console.log(renderDate('2019-09-04', 'Do MMM'));

<script src="https://momentjs.com/downloads/moment.js"></script>

我不能这样做：

const renderDate = (date, locale = 'en', format = 'D MMM') => {
  if (locale === 'de' && format === 'D MMM') {
    const indexD = format.lastIndexOf('D')
    format = `${format.slice(0, indexD + 1)}o${format.slice(indexD + 1, format.length)}`
  }

  moment.locale(locale)
  return moment(date).format(format)
}

因为如果我只使用 1 个可选参数调用该方法，则可能是错误的：

renderDate('2019-09-04', 'Do MMM')

现在 locale（语言）是一种格式，这是错误的。

解决此问题的最佳方法是什么？ 我应该传递一个对象然后检查键是否存在？

然后我可以解构，但我不确定这是否是最好的方法：

const renderDate = ({
  date,
  locale,
  format
}) => {
  if (locale === undefined) {
    locale = 'en'
  }
  if (format === undefined) {
    format = 'D MMM'
  }
  if (locale === 'de' && format === 'D MMM') {
    const indexD = format.lastIndexOf('D')
    format = `${format.slice(0, indexD + 1)}o${format.slice(indexD + 1, format.length)}`
  }

  moment.locale(locale)
  return moment(date).format(format)
}
console.log(renderDate('2019-09-04', 'Do MMM'));

<script src="https://momentjs.com/downloads/moment.js"></script>

Answer 1

是的 - 如果您可以传递两个可选参数中的一个、另一个或两者，则对象是解决此问题的唯一方法。您也可以使用默认值。请注意，您的if 语句仅在format 为D MMM 时运行，因此您可以删除indexOf 并使用0：

export const renderDate = ({ date, locale = "en", format = "D MMM" } = {}) => {
  if (locale === "de" && format === "D MMM") {
    const indexD = 0;
    format = `${format.slice(0, indexD + 1)}o${format.slice(indexD + 1, format.length)}`;
  }
  moment.locale(locale);
  return moment(date).format(format);
};