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

重新设计了泛型参数类型的处理 #44

Closed
wants to merge 4 commits into from

Conversation

AlexStacker
Copy link

1、深入了解了一下各个函数的关系、重新实现了泛型参数的处理方式
2、增加了一个 增加参数表格列的功能, 对参数类型没有的情况下手动增加一个参数类型列,补全部分微信API文档缺失的参数类型

@qiu8310
Copy link
Owner

qiu8310 commented Mar 31, 2018

你理解已经很深了! 关于缺失的参数类型这个功能,我比较担心,如果文档删除一个字段,或者更新了字段顺序等等时,可能会出现类型错误。

所以是否可以考虑下应对措施。

泛型这一块好像不错,我下午拉下来跑跑看

@qiu8310
Copy link
Owner

qiu8310 commented Mar 31, 2018

@AlexStacker

@qiu8310
Copy link
Owner

qiu8310 commented Mar 31, 2018

我之前的 modify 应该都多少会有 assert 判断,如果文档不是所期望的样子,会输出警告信息,方便及时修正

@AlexStacker
Copy link
Author

确实看到了assert判断,对列的重复会出现估计只能靠生成后人工判别一下。增加参数或者移除参数当时没考虑到,现在想想处理起来确实麻烦,最麻烦的有可能碰到调整了参数顺序,有点复杂啦😄

@qiu8310
Copy link
Owner

qiu8310 commented Apr 1, 2018

其实不需要根据文件的变化去自动变化的,我的意思是需要检查下文档是否变化,变化了的话应该报个 warning 出来,提示人工处理。

比如我觉得可以判断个键名是否是你期望的那个,还有表格的行数和列数,一般如果这三个值没变的话应该算是正常的吧。没事,这块我来处理下吧。

@AlexStacker
Copy link
Author

大佬说的对,我想的太复杂了,考虑的太多把自己困住了。

@qiu8310
Copy link
Owner

qiu8310 commented Apr 3, 2018

@AlexStacker 不好意思,最近自由时间太少了,没太多时间顾及这边,但你放心,我一定会处理这个的。

初步的看了下,感觉你已经基本上和我在一条思路上了,谢谢!

@AlexStacker
Copy link
Author

向大佬致敬,我的只能算demo,大佬带我学习先进技术。

@qiu8310
Copy link
Owner

qiu8310 commented Jun 23, 2018

我很早就看过了,不敢轻易合,所以一直没合;现在不打算合了,主要在于现在我这个程序本来就不怎么健壮,每次微信更新文档我都得改一大堆东西,等有时间我需要重写下这个生成脚本。

不好意思

@qiu8310 qiu8310 closed this Jun 23, 2018
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