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

扩展SDK建议 #36

Open
ltwonders opened this issue Jun 27, 2019 · 2 comments
Open

扩展SDK建议 #36

ltwonders opened this issue Jun 27, 2019 · 2 comments

Comments

@ltwonders
Copy link

最近在使用OCR的相关SDK,说下使用感受:

  1. 比如AipOcr#custom这个方法,传入的Options为StringMap,两个custom对入参没有明显的标识,容易传入错误;
  2. 只支持byte[]的形式或者图片路径形式操作,对于原本就是base64String的格式不支持,想扩展,OcrConsts.CUSTOM还是package-private的。
  3. 返回值全是org.json.JSONObject,对反序列化成对象不方便。
    4.对请求构建的形式实在是不方便,用OCR一个API已经够累了,如果多接入几个估计得更累。

建议是:
1、如果能用okhttp3的话就用吧,那里边有很多Builder构建请求来方便而且可扩展;
2、底层的API自己可以是HashMap的形式,但是对外暴露的最好还是Options.languageType("ZH"),custom.options(options)这种。
3、用点Builder和Facade。

@xiaoshuaishuai319
Copy link

okhttp3。这个是android用的多一些吧?
https://gitee.com/xshuai/java-sdk-pro 我做了一个pro 还在慢慢完善中。
你的第2点说的挺不错的。
第3点 个人感觉达不到那样的高级。哈哈
目前百度ai整体 接口结构数据并不是全量统一的。不知道什么时候能把这个问题迭代了。

@ltwonders
Copy link
Author

okhttp3都可以作为客户端,在Android和应用端都可以用,主要是它的facade形式对API很友好。

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

No branches or pull requests

2 participants