Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

修复了部分文本格式读取date失败的问题 #2434

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

EllisonKuang
Copy link
Contributor

通过添加yyyy/mm/dd的格式解决了 #2419 提出的读取失败的问题,同时支持了yyyy/m/d, yyyy/mm/d等各种格式。我们针对缩写造成的长度变化仅在返回format的过程中进行判断并返回对应的format,而不对其缩写进行补全。我们的做法相比于 #2432 通过补0完成缩写补全的解决方式,规避了繁杂的if语句判断,尽可能得减少了代码量及对原代码的改动。此外我们还支持了将2022缩写为22这一年份缩写的判断。

@CLAassistant
Copy link

CLAassistant commented Apr 22, 2022

CLA assistant check
All committers have signed the CLA.

@TungMan0801
Copy link

TungMan0801 commented Apr 22, 2022

感谢讨论!但#2432 也是在也是在仅判断返回format的过程增加判断的,具体的逻辑确实是补位然后回format的,excel本身的数值不会更改~感谢提醒,有关于if判断句的逻辑会稍后再试试完善,不让补位在判断处占用较多时间!^v^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants