Discussion :

[stc074]

Bonjour, j'ai un algo qui vérifie mathématiquement les numéro de siren.
J'ai récemment recréé une entreprise et l'urssaf m'a envoyé un numéro bizarre de la forme :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
CCCCCCCLCC
C : Chiffre et L:Lettre

Du coup mon algo ne fonctionne plus pour ce type de numéro, savez vous comment on peut vérifier ce genre de numéro ?

Merci.

[ToTo13]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
CCCCCCCLCC
C : Chiffre et L:Lettre

savez vous comment on peut vérifier ce genre de numéro ?

Veux tu vérifier la forme que tu viens de donner (7 chiffres, une lettre, puis 2 chiffres) ?
Si oui, tu considères ton numéro comme une chaine de caractères, puis tu essaies de transformer chaque lettre en numéro. La fois où tu n'y arrives pas c'est la lettre et il ne te reste plus qu'à vérifier l'organisation.

[stc074]

Veux tu vérifier la forme que tu viens de donner (7 chiffres, une lettre, puis 2 chiffres) ?
Si oui, tu considères ton numéro comme une chaine de caractères, puis tu essaies de transformer chaque lettre en numéro. La fois où tu n'y arrives pas c'est la lettre et il ne te reste plus qu'à vérifier l'organisation.

Bonjour merci pour ta réponse, voici mon algo (en java) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
    public static boolean isSiren(String number) {
        boolean flag;
        Pattern pattern = Pattern.compile("^[0-9]{9}$");
        Matcher matcher = pattern.matcher(number);
        if (!matcher.matches()) {
            flag = false;
        } else {
            //
            int sum = 0;
            for (int i = 8; i >= 0; i--) {
                int value = Integer.parseInt("" + number.charAt(i));
                if (((i + 1) % 2) == 0) {
                    value *= 2;
                }
                if (value >= 10) {
                    value -= 9;
                }
                sum += value;
            }
            flag = ((sum % 10) == 0);
        }
        return flag;
    }

Si je renseigne les 9 numéros sans la lettre, mon test ne passe pas.

[ToTo13]

Tu ne fais aucun test sur la longueur de la chaine.
De plus, parseInt retourne une exception lorsque l'opération échoue, il te faut donc l'attraper lors de l'analyse de la lettre.

[stc074]

Tu ne fais aucun test sur la longueur de la chaine.
De plus, parseInt retourne une exception lorsque l'opération échoue, il te faut donc l'attraper lors de l'analyse de la lettre.

j'ai changé ma méthode :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
    public static boolean isSiren(String number) {
        boolean flag;
        if (number.length()!=9) {
            flag = false;
        } else {
            //
            int sum = 0;
            for (int i = 8; i >= 0; i--) {
                try {
                int value = Integer.parseInt("" + number.charAt(i));
                if (((i + 1) % 2) == 0) {
                    value *= 2;
                }
                if (value >= 10) {
                    value -= 9;
                }
                sum += value;
                } catch(NumberFormatException ex) { }
            }
            flag = ((sum % 10) == 0);
        }
        return flag;
    }

Je teste la longueur (9 pour un SIREN) et chope l'exception, mais mon test ne passe pas avec les 9 premiers caractères de mon Siret.
Merci.

[ToTo13]

Ton code est très étrange.
Il te faut deux étapes : avant et après la lettre.
De plus, pourquoi faire le test "if (value >= 10)" ? Tu ne convertis qu'un seul digit, donc tu ne peux avoir qu'une valeur inférieure à 10.
Et tu ne vérifies nulle part la position de la lettre.

[stc074]

Ton code est très étrange.
Il te faut deux étapes : avant et après la lettre.
De plus, pourquoi faire le test "if (value >= 10)" ? Tu ne convertis qu'un seul digit, donc tu ne peux avoir qu'une valeur inférieure à 10.
Et tu ne vérifies nulle part la position de la lettre.

La position de la lettre est donnée par i, et value peut être >= 10, en effet on la multiplie par 2, le code il vient d'un algo mais en fait j'ai l'impression qu'il ne fonctionne pas avec les nouveaux numéros c'est surtout ça mon problème.
Merci.

[ToTo13]

I parcourt toutes les positions, il ne pointe sur rien de spécifique.
Je te conseille vivement de reprendre le code à la base, car celui-là ne risque pas de fonctionner tel quel.
Il suffit de :
- vérifier la longueur de la chaine.
- parcourir n1 digit avant la lettre
- trouver la lettre à la bonne position, elle sera détectée par l'exception.
- finir de traiter la chaine de caractères (les digits restants).

[stc074]

En fait je crois que c'est l'urssaf qui a fait une erreur en me communiquant un siret farfelu, c'est une recréation d'entreprise et si je teste sur le site de linsee avec mon ancien siren ça donne bien les informations de mon entreprise (avec la date de la re-création)