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

Remplacer toutes les boucles for sur un objet JSONArray par une Stream #806

Closed
FBibonne opened this issue Nov 18, 2024 · 0 comments · Fixed by #807
Closed

Remplacer toutes les boucles for sur un objet JSONArray par une Stream #806

FBibonne opened this issue Nov 18, 2024 · 0 comments · Fixed by #807
Assignees

Comments

@FBibonne
Copy link
Member

FBibonne commented Nov 18, 2024

Pour toutes les boucles for sur les JSONObject d'un JSONArray, comme dans CodeListServiceImpl#getCodesForCodeList, remplacer la boucle for par une Stream :

  • créer une méthode utilitaire dans fr.insee.rmes.utils.JSONUtils : public static Stream<JSONObject> stream(JSONArray jsonArray) qui retourne la stream constituée par les éléments du JSONArray en tant que JSONObject
  • partout où une boucle for telle que for(var i = 0; i < items.length(); i++){ JSONObject item = items.getJSONObject(i); ... , remplacer par un appel à JSONUtils.stream(items)...
  • Bien écrire le test unitaire de la méthode JSONUtils#stream
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

Successfully merging a pull request may close this issue.

2 participants