Vérifier les arguments pour les fonctions exportées

[ThomasSoeiro]

On pourrait ajouter quelques vérifications minimales du type stopifnot(<basic type checking>, start_date <= end_date, ...) dans les fonctions exportées.

[strayMat]

Complètement d'accord. Idéalement je serais pour faire deux trois fonctions générique de check que l'on mettrait sans un utils.R. j me peux m'en occuper.

Liste des checks, à implémenter et insérer dans les fonctions d'extraction existantes (à compléter) :

• [ ] check_dates(start_date, end_date)
• [ ] type checking : on vérifierait que les listes de codes sont des listes de string, où on irait plus loin (cohérent avec cip13 ou ATC par exemple) ? Les listes d'identifiants aussi ?

[ThomasSoeiro]

Pour la vérification des dates, je pense que ça serait suffisant pour la fonction d'extraction des délivrances :

stopifnot(
  inherits(start_date, "Date"),
  inherits(end_date, "Date"),
  start_date <= end_date
)

Je ne suis pas sûr que ça vaille le coup d'en faire une fonction car je pense que les conditions à vérifier seront à adapter à chaque fonction d'extraction, donc ce sera probablement plus lisible en voyant l'ensemble des conditions. Sinon faudra prévoir un ... dans les arguments.

Pour le type checking je pense qu'on peut se contenter de quelque chose comme stopifnot(is.character(...), is.numeric(...), ...). Une validation plus poussée serait périlleuse : par exemple, pour les codes ATC, la méthadone antalgique (Zoryon) n'a toujours pas de code ATC niveau 5 attribué par l'OMS à ce jour.