发现 moment.js 的一个 bug

> moment({ h: 24, m: 00})
moment("2017-03-24T00:00:00.000")
> moment({ h: 0, m: 00})
moment("2017-03-23T00:00:00.000")
> moment({ h: 24, m: 00, s: 1})
moment.invalid(/* [object Object] */)

即 moment({ h: 24, m: 00, s: 1}) 会报错。

DuanPengfei 1楼•3 个月前

24 点 0 分 1 秒，请问这是什么时间格式？

nnliang 2楼•3 个月前

:blush:

beyond5959 3楼•3 个月前作者

@DuanPengfei 也就是第二天的 0 点 0分 1秒，不加秒的话 moment({ h: 24, m: 00}) 这样就正常。

dbit-xia 4楼•3 个月前

@beyond5959 那岂不是还要支持48:00:01 ??

来自酷炫的 CNodeMD

wocaatm 5楼•3 个月前

你家24小时制见过24：00：01？好比你12小时制见过12：00：01。

DuanPengfei 6楼•3 个月前

@beyond5959 想想进制是怎么回事就应该能想清楚了

DevinXian 7楼•3 个月前

地球上还没这时间吧…

beyond5959 8楼•3 个月前作者

@DuanPengfei @wocaatm 目的不是要获取 24：00：01，而是要获取第二天的00：00：01，像这样 moment({ h: 24 }) 获取第二天的 00：00：00 就是正常的。

libook 9楼•3 个月前

去官方github提issue了没？

alsotang 10楼•3 个月前

@beyond5959 moment({ h: 24, m: 00}) 这个边界情况可以通过不代表应该这样做吧？

jiangzhuo 11楼•3 个月前

moment({ h: 24, m: 00, s: 1}) 这是火星时间吧，好像火星是25小时制

rwing 12楼•3 个月前

。。。。。。。人家就是这么设计的，不允许你传入超过24小时的时间。。。。。。。难不成你传入240：00：00我还要给你算一下参见 https://github.com/moment/moment/blob/37587a15a58607ae907c949fa1f20620f9d8e4ad/src/test/moment/is_valid.js#L87

leyou319 13楼•3 个月前

可以

leyou319 14楼•3 个月前

好

leyou319 15楼•3 个月前

真的

yimingjfe 16楼•3 个月前

测试一下

461534898 17楼•3 个月前

吓尿了，原来大家的时间不一样啊。。

🚀🚀🚀🚀