Discussion :

[wattaroo]

Bonjour
Lors d'un exercice, je crée une fonction de contrôle de mail.
Et l'affichage ne se passe pas bien du tout !! Pourquoi
J'obtiens ce message :

"Parse error: syntax error, unexpected $end"

J'ai compris que c'est un des messages d'erreurs les plus classiques qui concerne surtout des problèmes d'accolades qui manquent.

Mais dans ce code (très simple) il me semble qu'il ne manque pas d'accolades !!
Alors je ne comprends pas de quoi il s'agit.

Code :

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
<?php
								function validmail($ch){
									$modele = '(^[a-z])([a-z0-9])+(\.|-)?([a-z0-9]+)@([a-z0-9]{2,})\.([a-z]{2,4}$)';
									$ch = strtolower($ch);
									if(ereg($modele, $ch)){
										echo $ch.' est valide<br/>';
										return TRUE;
									}
									else{
										echo $ch. ' est invalide<br/>';
										return FALSE;
									}
								$mail1 = 'cvignes@club-internet.fr	';
								$mail2 = 'Jean5.dupont@laposte2.uk';
								$mail3 = '5pierre.dupapi@plusloin.info';
								$mail4 = 'jojo@jaja.onfos';
 
								validmail($mail1);
								validmail($mail2);
								validmail($mail3);
								validmail($mail4);
							?>

Merci à tous ceux qui voudront bien m'aider !

[alejandro]

Bonjour,

Il te manque une accolade

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
24
25
26
27
<?php
function validmail($ch)
{
    $modele = '(^[a-z])([a-z0-9])+(\.|-)?([a-z0-9]+)@([a-z0-9]{2,})\.([a-z]{2,4}$)';
    $ch = strtolower($ch);
    if(ereg($modele, $ch))
    {
        echo $ch.' est valide<br/>';
        return TRUE;
    }
    else
    {
        echo $ch. ' est invalide<br/>';
        return FALSE;
    }
} // <- ICI
 
$mail1 = 'cvignes@club-internet.fr ';
$mail2 = 'Jean5.dupont@laposte2.uk';
$mail3 = '5pierre.dupapi@plusloin.info';
$mail4 = 'jojo@jaja.onfos';
 
validmail($mail1);
validmail($mail2);
validmail($mail3);
validmail($mail4);
?>

Tu devrais penser à indenter ton code, généralement 4 espaces ça aide vraiment. De plus quand tu ouvres une accolade prends l'habitude de la refermer de suite, ça évite ce genre de problèmes

Cordialement,

[wattaroo]

Merci à toi pour la réponse, effectivement encore une accolade manquante !!

Pour l'indentation, c'est le cas, mon code est indenté dans Notepad mais pas sur ma présentation dans mon message.

Comment fais-tu pour présenter le code de cette façon ??

Encore merci

[alejandro]

J'ai oublié, juste un truc, tu as déja une fonction pour valider tes mails dans php, c'est filter_var() avec FILTER_VALIDATE_EMAIL.

Dans ton cas ça donne :

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
<?
$mails = array(
    'cvignes@club-internet.fr',
    'Jean5.dupont@laposte2.uk',
    '5pierre.dupapi@plusloin.info',
    'jojo@jaja.onfos'
);
foreach( $mails as $value )
{
    $email = strtolower( $value );
    $is_valid_email = filter_var( $email, FILTER_VALIDATE_EMAIL);
    $valid_text = ( $is_valid_email ) ? "valide" : "invalide";
    echo "$email est $valid_text";
}
?>

PS: Pour les indentations je fais 4 espaces par 4 espaces

[wattaroo]

Merci alejandro, je vais essayer cela tout de suite