这段使用 MomentJS 的日期格式化代码有什么问题？

Question

下面的代码是不言自明的，但它没有按预期工作。如果相差 0 天，则返回 '0y ago'

const formatTime = (time) => {
    const dayDiff = moment().diff(moment(time), 'days')

    switch (dayDiff) {
        case dayDiff == 0:
            return `${moment().diff(moment(time), 'hours')}h ago`
        case dayDiff > 0 && dayDiff < 31:
            return `${moment().diff(moment(time), 'days')}d ago`
        case dayDiff > 31 && dayDiff < 365:
            return `${moment().diff(moment(time), 'months')}m ago`
        default:
            return `${moment().diff(moment(time), 'years')}y ago`
    }
}

如果时间在同一天，那么它会以默认情况结束，而它应该以第一种情况结束并返回“0h 前”。

我可能错过了一些小事。任何帮助将不胜感激。

Answer 1

Switch 用于与值进行比较。为了使您的代码正常工作，它需要是

switch(true) {
  case dayDiff == 0:
  ....
}

但是在 switch 中使用条件被认为是不好的做法，只使用 if 语句

const formatTime = time => {
  const dayDiff = moment().diff(moment(time), 'days')
  if (dayDiff == 0) {
    return `${moment().diff(moment(time), 'hours')}h ago`
  } else if (dayDiff < 31) {
    return `${moment().diff(moment(time), 'days')}d ago`
  } else if (dayDiff < 365) {
    return `${moment().diff(moment(time), 'months')}m ago`
  } else {
    return `${moment().diff(moment(time), 'years')}y ago`
  }
}

Answer 2

在 Javascript 中，switch statement 评估案例的值，条件链可能适合您的案例，请尝试以下操作：

const formatTime = (time) => {
    const dayDiff = moment().diff(moment(time), 'days')

    if (dayDiff == 0) {
        return `${moment().diff(moment(time), 'hours')}h ago`
    }
    if (dayDiff > 0 && dayDiff < 31) {
        return `${moment().diff(moment(time), 'days')}d ago`
    }
    if (dayDiff > 31 && dayDiff < 365) {
        return `${moment().diff(moment(time), 'months')}m ago`
    }

    // this would be your "default"
    return `${moment().diff(moment(time), 'years')}y ago`
}