Discussion :

[Rifton007]

Bonjour et merci d'avance pour votre aide.
J'aime inclure un variable dans un chaine regex, pour mon model.

Code :

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

1
2
3
4
class Store extends AppModel {
public $specialChars = "àáâäæçèéêëìíîïñòóôœùúûüýÿß";
public $validate = array( ...
'rule' => array('custom', "/^[a-z{$specialChars}](?:[ -]{1}[a-z]|['a-z{$specialChars}]+)+$/i"), ...

Et la dérniére ligne de code pose probléme, je ne comprend pas pourquoi: syntax error, unexpected '"', expecting ')'

Merci à vous

[sabotage]

La partie que tu nous montres est correcte

Code :

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

$validate = array('rule' => array('custom', "/^[a-z{$specialChars}](?:[ -]{1}[a-z]|['a-z{$specialChars}]+)+$/i"));

Le problème est dans ce que tu ne montres pas.

[Rifton007]

Merci de ta réponse.
Pour info je suis sous CakePHP 2.x

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
<?php
App::uses('AppModel', 'Model');
 
class Store extends AppModel {
 
    public $specialChars = "àáâäæçèéêëìíîïñòóôœùúûüýÿß";
 
    public $validate = array(
        'storename' => array(
            'setChar' => array(
                'rule' => array('custom', "/^[a-z{$specialChars}](?:[ -]{1}[a-z]|['a-z{$specialChars}]+)+$/i"),
                'message' => 'Caractére invalide'
            ),
            'length' => array(
                'rule' => array('between', 5, 15),
                'message' => 'Ce nom doit comporté entre 5 et 15 caractéres'
            ))
    );
 
}

[sabotage]

Excuse moi j'ai répondu un peu vite : tu ne peux pas avoir une expression évaluée pour "public", seulement une expression littérale ou une constante.

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
<?php
 
$specialChars = "àáâäæçèéêëìíîïñòóôœùúûüýÿß";
define('EXPRESSION', "/^[a-z$specialChars](?:[ -]{1}[a-z]|['a-z$specialChars]+)+$/i");
 
class Store extends AppModel {
 
    public $validate = array(
        'storename' => array(
            'setChar' => array(
                'rule' => array('custom', EXPRESSION),
                'message' => 'Caractére invalide'
            ),
            'length' => array(
                'rule' => array('between', 5, 15),
                'message' => 'Ce nom doit comporté entre 5 et 15 caractéres'
            ))
    );
 
}

[CosmoKnacki]

Je ne vois pas non plus d'où provient ton erreur. Peut-être que tu as oublié de refermer quelque chose, bien en amont de la ligne annoncée en erreur, mais que l'interpréteur PHP ne s'en rend compte qu'au niveau de cette ligne?

Par contre au sujet de ta pattern:

• le quantificateur {1} ne sert à rien (en toutes circonstances), a matche a, pas la peine d'écrire a{1}.
• comme tu dois dealer avec des chaînes multibytes (encodées en UTF-8) tu dois prévenir le moteur de regex avec le modificateur u (sinon il va croire que chaque octet correspond à un seul caractère et ne sera pas en mesure de reconnaître qu'un groupe d'octets représente une seule lettre, et ce aussi bien pour la chaîne qu'il traite que pour la pattern qu'il doit compiler).
• Tu peux éventuellement utiliser \p{Latin} en lieu et place de [a-z{$specialChars}] (\p{Latin} contient toutes les lettres latines en majuscule ou en minuscule, avec ou sans accents.) C'est plus permissif que ta sélection de caractères mais c'est peut-être ce que tu cherches à faire?
• Attention aux constructions de ce type (a+|b)+subpattern (quantificateurs imbriqués suivis d'une contrainte quelconque) qui peuvent causer un catastrophic backtracking (regarde sur ce lien le nombre d'étapes nécessaires avant l'échec de la pattern, et ajoute une lettre avant le 1 pour voir l'évolution). Pour l'éviter, tu peux ajouter un quantificateur possessif à ton groupe (a+|b)++subpattern

[Rifton007]

Merci Sabotage pour ta solution.

Tu ne peux pas avoir une expression évaluée pour "public", seulement une expression littérale ou une constante.

J'ignorais cela, que puis-je taper sur google ou ici pour avoir plus de détails à ce sujet ?

Merci à vous tous pour votre aide.

[sabotage]

http://php.net/manual/fr/language.oop5.static.php

Comme n'importe quelle autre variable PHP statique, les propriétés statiques ne peuvent être initialisées qu'en utilisant un littéral ou une constante ; les expressions ne sont pas permises. Ainsi, vous pouvez initialiser une propriété statique avec un entier ou un tableau, mais pas avec une autre variable, ni avec la valeur de retour d'une fonction, ni avec un objet.

[Rifton007]

Je ne vois pas non plus d'où provient ton erreur. Peut-être que tu as oublié de refermer quelque chose, bien en amont de la ligne annoncée en erreur, mais que l'interpréteur PHP ne s'en rend compte qu'au niveau de cette ligne?

Par contre au sujet de ta pattern:

• le quantificateur {1} ne sert à rien (en toutes circonstances), a matche a, pas la peine d'écrire a{1}.
• comme tu dois dealer avec des chaînes multibytes (encodées en UTF-8) tu dois prévenir le moteur de regex avec le modificateur u (sinon il va croire que chaque octet correspond à un seul caractère et ne sera pas en mesure de reconnaître qu'un groupe d'octets représente une seule lettre, et ce aussi bien pour la chaîne qu'il traite que pour la pattern qu'il doit compiler).
• Tu peux éventuellement utiliser \p{Latin} en lieu et place de [a-z{$specialChars}] (\p{Latin} contient toutes les lettres latines en majuscule ou en minuscule, avec ou sans accents.) C'est plus permissif que ta sélection de caractères mais c'est peut-être ce que tu cherches à faire?
• Attention aux constructions de ce type (a+|b)+subpattern (quantificateurs imbriqués suivis d'une contrainte quelconque) qui peuvent causer un catastrophic backtracking (regarde sur ce lien le nombre d'étapes nécessaires avant l'échec de la pattern, et ajoute une lettre avant le 1 pour voir l'évolution). Pour l'éviter, tu peux ajouter un quantificateur possessif à ton groupe (a+|b)++subpattern

• Le quantificateur {1} évite de doublé les trait d'union et les espaces.
• \p{latin} je ne connaissais pas. Il est vrai que ça serait plus pratique que d'énuméré chaque caractéres spéciaux.
  Et pourtant c'est la solution préconisé sur plusieurs post dans le forum.

Merci à toi,

[CosmoKnacki]

Encore une fois le {1} n'évite rien du tout, il ne sert à rien. Écrire [ -]{1} ou a{1} ou (?:groupe_avec_des_trucs){1} c'est pareil que d'écrire [ -], a ou (?:groupe_avec_des_trucs). Le {1} est juste redondant et n'a aucun effet. Les éléments ne seront pas plus ou moins répétés en l'ajoutant. Fais l'essai avec et sans, tu verras bien.

Et pourtant c'est la solution préconisée sur plusieurs posts dans le forum.

Possible, tout le monde ne connaît pas les classes de caractères unicode comme \p{Latin}, peut-être parce que ce n'est pas disponible dans tous les langages ou parce que les gens s'imaginent que la liste des caractères latins a été établie lors du traité de Rome.

Quoi qu'il en soit, il ne faut rien prendre pour argent comptant et tester et se renseigner. Derrière une pattern aux allures inoffensives puis se cacher un piège diabolique!

Un petit test pour illustrer:https://eval.in/478113

[Rifton007]

...ou parce que les gens s'imaginent que la liste des caractères latins a été établie lors du traité de Rome.

Quoi qu'il en soit, il ne faut rien prendre pour argent comptant et tester et se renseigner.

Bien évidemment, j'avais aussi consulter plusieurs site de référence sur Regex (FR/EN), mais pas un mot sur classes de caractères jusqu’à ce que tu m'en parle.

En tout cas, merci CosmoKnacki pour cet échange et tes enseignements.