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

允许更多的附件文件名称规则 #43

Merged
merged 2 commits into from
Jul 12, 2021

Conversation

aiyamia
Copy link

@aiyamia aiyamia commented Jun 27, 2021

通过修改原有的splitFilename函数来解决 #37 所提出的这种情况,同时适用更为灵活的情形。
以下是若干案例:

  • 2014龙小宁&王俊 中国经济问题-中国司法地方保护主义:基于知识产权案例的研究.pdf

    • {%y}{%a} {%j}-{%t}
  • 2020 金融可得性与互联网金融风险防范——基于网络传销案件的实证分析(中国工业经济).pdf

    • {%y} {%t}({%j})
  • 陈天昊等_检察行政公益诉讼制度的效果检验与完善路径_中外法学_2020.pdf

    • {%a}_{%t}_{%j}_{%y}
  • 王永进&黄青 (经济学报)-来自高院院长异地交流的证据.pdf

    • {%a} ({%j})-{%t}

(出于对代码原作者的尊重,我对源代码做了尽可能最低限度的修改,并在变量命名时尝试遵循原作者的风格。)

@l0o0
Copy link
Owner

l0o0 commented Jul 5, 2021

同学,我看了下你的代码,匹配写的很不错。不过这里有一个问题,就是知网下载的一些文献,可能中间会用_来代替特殊字符,也有可能文章标题过长,中间省略用_代替。这类情况还是比较常见的。我觉得可以在你原代码上加上处理这种情况的代码,就可以了

@aiyamia
Copy link
Author

aiyamia commented Jul 5, 2021

同学,我看了下你的代码,匹配写的很不错。不过这里有一个问题,就是知网下载的一些文献,可能中间会用_来代替特殊字符,也有可能文章标题过长,中间省略用_代替。这类情况还是比较常见的。我觉得可以在你原代码上加上处理这种情况的代码,就可以了

感谢作者大大花时间看我的代码。您说的这种情况我在之前确实没考虑,我刚刚去下载了几篇文献,确实有您说的这种情况,我决定进一步探索一下解决方案。
再次感谢~

@l0o0
Copy link
Owner

l0o0 commented Jul 5, 2021

同学,我看了下你的代码,匹配写的很不错。不过这里有一个问题,就是知网下载的一些文献,可能中间会用_来代替特殊字符,也有可能文章标题过长,中间省略用_代替。这类情况还是比较常见的。我觉得可以在你原代码上加上处理这种情况的代码,就可以了

感谢作者大大花时间看我的代码。您说的这种情况我在之前确实没考虑,我刚刚去下载了几篇文献,确实有您说的这种情况,我决定进一步探索一下解决方案。
再次感谢~

期望同学的代码😀

@l0o0 l0o0 merged commit dfef211 into l0o0:master Jul 12, 2021
@aiyamia aiyamia deleted the namepatent-feature-ext branch July 13, 2021 03:15
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.

None yet

2 participants