search

Manweill / swagger-axios-codegen

swagger client to use axios and typescript
MIT License
306 stars 83 forks source link

请求参数去重; #47

Closed xxbld closed 5 years ago

xxbld commented 5 years ago

bodyParam,queryParam,pathParam有可能同时出现同名, 全都放在一个 params 是否有些问题？

Manweill commented 5 years ago

后台出现重名参数，还能编译通过？

xxbld commented 5 years ago

后台出现重名参数，还能编译通过？

是挺奇葩的，JAVA，用两个实体作为参数，两个实体里面分别有同一个参数名；

另外： include: ['dept-controller'] 这种格式的tags名include失效了，全部不生成

Manweill commented 5 years ago

两个实体作为参数，那他应该是包裹到一个新的类里面吧，body按道理来说是只能对应一个类的

xxbld commented 5 years ago

这样子， image 目前都是query参数，还没有出现body的

xxbld commented 5 years ago

tags应该区分大小写的，所以强制转换为大写，我觉得不合理。但是用同一个规则去处理tagName，我又觉得没毛病。。。

在java中tags一般是不用写的，默认的名称就 class AaaController-> tag aaa-controller； 我觉得转换pascalcase没问题，我只是把include里面的tagname名称也转了pascalcase；

xxbld commented 5 years ago

你看过pont的设计吗？设计了一个中转数据结构，可以实现支持不同的api，还有一些钩子函数，自定义挺方便的。 就是生成出来的代码很不喜欢，感觉模板不是针对ts设计的。

Manweill commented 5 years ago

你看过pont的设计吗？设计了一个中转数据结构，可以实现支持不同的api，还有一些钩子函数，自定义挺方便的。 就是生成出来的代码很不喜欢，感觉模板不是针对ts设计的。

没有，感觉太多东西，太复杂了，目前用不到。 还有要提交的没，么有我合并了，不过不会那么快发布

xxbld commented 5 years ago

暂时没有了，有空给include加个正则判断。

Manweill commented 5 years ago

暂时没有了，有空给include加个正则判断。

WelCome PR