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')

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

解决此问题的最佳方法是什么？我应该传递一个物体然后检查钥匙是否存在吗？

然后我可以进行销毁，但是我不确定这是否是最好的方法：

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);
};

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

1 个答案: