Replace Map usage with OOP design classes
@aleksandar-cloudinary y travaille déjà.
Depuis le 13/6/2020.
Évaluation
Cette issue n'a pas encore été évaluée.
Description
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();
- Langage dominant
- Java
- Étoiles
- 176
- Forks
- 113
- Merge moyen
- 9 j 20 h
- PR mergées (30 j)
- 2
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Aucun guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de cloudinary/cloudinary_java
-
`options` map modified directlyPeut-être à nouveau libre @adimiz1 l’a pris il y a 481 jours, et aucune pull request n’est ouverte. Ouverte
cloudinary/cloudinary_java#383 · 3 commentaires · 1 réaction · 1 personne assignée ·
-
Api#resourceByAssetID does not support same options as in documentationPeut-être à nouveau libre @adimiz1 l’a pris il y a 646 jours, et aucune pull request n’est ouverte. Ouverte
cloudinary/cloudinary_java#357 · 2 commentaires · 1 personne assignée ·
-
Readme - Maven dependencyPeut-être à nouveau libre @adimiz1 l’a pris il y a 754 jours, et aucune pull request n’est ouverte. Ouverte
cloudinary/cloudinary_java#349 · 1 commentaire · 1 personne assignée ·
-
cloudinary-http5 ignores system propertiesPeut-être à nouveau libre @adimiz1 l’a pris il y a 754 jours, et aucune pull request n’est ouverte. Ouverte
cloudinary/cloudinary_java#348 · 2 commentaires · 1 personne assignée ·
-
Make usage of http clients (apache, OkHttp, Jetty, Asynch) easierPeut-être à nouveau libre @ManishS6 l’a pris il y a 772 jours, et aucune pull request n’est ouverte. Ouverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
cloudinary/cloudinary_java#343 · 6 commentaires ·
Toutes les issues de cloudinary/cloudinary_java
Issues similaires
-
bug(core): skip_nodes.json is missing SELECT and PRE, causing Python and Java to insert separators into dropdown options and preformatted textPeut-être pris @Bhumit267 l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
Les mainteneurs répondent en général sous 1 jour
-
ExceptionThrowShouldUseContext: NullPointerException on a catch block inside an annotation typePeut-être pris @Carbon14-48 l’a pris aujourd’hui. Ouverteapproved
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
checkstyle/checkstyle#22784 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
ASSERT-KTH/depclean#591 ·
Les mainteneurs répondent en général sous 1 jour
-
Api537FlareFlameModel.setStackHeightM accepts a negative heightPeut-être pris @EvenSol l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
Les mainteneurs répondent en général sous 1 jour