Replace Map usage with OOP design classes
@aleksandar-cloudinary 已经在做这个了。
开始于 2020年6月13日。
评估
这个 Issue 还没有评估数据。
描述
I am currently testing your service and evaluating to be your customer. But i am dissapointedd from the Java integration you provide.
Everywhere you pass generic Map instances holding String key value pairs for configuration, upload options / results. That is not suitable for an object oriented programming language, and looks like 15 years ago programming style (including underscores).
Also enumeration classes could be used for fixed parameters like transformations and meaningfull exceptions shall be thrown. To detect a resource not found exception one must check the generic message like if (e.getMessage().contains("Resource not found")).
That key-value and error message approach is very error prone and really not developer friendly.
Example:
String publicId = (String) uploadResult.get("public_id");
Could be:
String publicId = uploadResult.getPublicId();
- 主要语言
- Java
- 星标
- 176
- 派生
- 113
- 平均合并
- 9 天 20 小时
- 30 天内合并 PR
- 2
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 没有贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
cloudinary/cloudinary_java 的其他 Issue
-
`options` map modified directly可能重新可做 @adimiz1 于 481 天前认领，目前没有进行中的 PR。 未关闭
cloudinary/cloudinary_java#383 · 3 条评论 · 1 个 reaction · 已指派 1 人 ·
-
Api#resourceByAssetID does not support same options as in documentation可能重新可做 @adimiz1 于 646 天前认领，目前没有进行中的 PR。 未关闭
cloudinary/cloudinary_java#357 · 2 条评论 · 已指派 1 人 ·
-
Readme - Maven dependency可能重新可做 @adimiz1 于 754 天前认领，目前没有进行中的 PR。 未关闭
cloudinary/cloudinary_java#349 · 1 条评论 · 已指派 1 人 ·
-
cloudinary-http5 ignores system properties可能重新可做 @adimiz1 于 754 天前认领，目前没有进行中的 PR。 未关闭
cloudinary/cloudinary_java#348 · 2 条评论 · 已指派 1 人 ·
-
Make usage of http clients (apache, OkHttp, Jetty, Asynch) easier可能重新可做 @ManishS6 于 772 天前认领，目前没有进行中的 PR。 未关闭
难度 5/5 一周以上 新手友好度 25/100
cloudinary/cloudinary_java#343 · 6 条评论 ·
查看 cloudinary/cloudinary_java 的全部 Issue
相似的 Issue
-
bug(core): skip_nodes.json is missing SELECT and PRE, causing Python and Java to insert separators into dropdown options and preformatted text可能已有人在做 @Bhumit267 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 85/100
维护者通常 1 天内回复
-
ExceptionThrowShouldUseContext: NullPointerException on a catch block inside an annotation type可能已有人在做 @Carbon14-48 今天认领。 未关闭approved
难度 2/5 1-3 小时 新手友好度 72/100
checkstyle/checkstyle#22784 · 1 条评论 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 74/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 62/100
ASSERT-KTH/depclean#591 ·
维护者通常 1 天内回复
-
Api537FlareFlameModel.setStackHeightM accepts a negative height可能已有人在做 @EvenSol 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 74/100
维护者通常 1 天内回复