CharlesNepote / liste-prenoms-nouveaux-nes

Spécification de la liste annuelle des prénoms des nouveaux-nés en France.
5 stars 2 forks source link

Utiliser un type entier pour le champ `NOMBRE_OCCURRENCES` #3

Open cbenz opened 5 years ago

cbenz commented 5 years ago

Je suis surpris de constater que le champ NOMBRE_OCCURRENCES a été défini via une regex.

Je m'attendrais à ce que ce soit un nombre entier avec les contraintes minimum = 1 et maximum = 999999 (6 chiffres).

Je m'appuie sur la description du champ pour penser cela :

[...] La valeur de ce champ est donc un nombre entier d'un à 6 chiffres maximum, 0 étant exclu.

CharlesNepote commented 5 years ago

Oui tu as tout à fait raison. C'est l'époque où j'utilisais les regex pour tout et n'importe quoi. Ta proposition est plus judicieuse, plus facile à comprendre pour ceux qui liront la spec dans le texte.

loichay commented 5 years ago

NOMBRE_OCCURRENCES

{
      "name": "NOMBRE_OCCURRENCES",
      "title": "Nombre d'occurrences",
      "description": "Nombre d'occurrences du prénom pour l'année correspondante. Tous les prénoms sont comptabilisés, y compris les prénoms uniques -- un seuil minimum est exclu car il conduirait à passer sous silence une importante part des naissances, voire la totalité dans les petites communes. La valeur de ce champ est donc un nombre entier d'un à 6 chiffres maximum, 0 étant exclu.",
      "type": "integer",
      "example": "353",
      "constraints": {
      "required": true,
      "minimum": 1,
      "maximum": 999999
},
CharlesNepote commented 5 years ago

Je vais publier rapidement une 1.1.3 qui corrige le problème.