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時間
- マージ済み PR（30日）
- 2
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
cloudinary/cloudinary_java のほかの issue
-
`options` map modified directly再び着手できるかも @adimiz1 が 481 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
cloudinary/cloudinary_java#383 · コメント 3 件 · リアクション 1 件 · 担当者 1 名 ·
-
Api#resourceByAssetID does not support same options as in documentation再び着手できるかも @adimiz1 が 646 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
cloudinary/cloudinary_java#357 · コメント 2 件 · 担当者 1 名 ·
-
Readme - Maven dependency再び着手できるかも @adimiz1 が 754 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
cloudinary/cloudinary_java#349 · コメント 1 件 · 担当者 1 名 ·
-
cloudinary-http5 ignores system properties再び着手できるかも @adimiz1 が 754 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
cloudinary/cloudinary_java#348 · コメント 2 件 · 担当者 1 名 ·
-
Make usage of http clients (apache, OkHttp, Jetty, Asynch) easier再び着手できるかも @ManishS6 が 772 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
難易度 5/5 1週間以上 初心者へのやさしさ 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 日以内に返信