Discussion :

[dlhp59]

bonjour

j'env un mail avec comme sujet des info du formulaire

Code :

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

$mail->Subject= "" . $r1 ." " . $r2 ." " . $r3 ." " . $r4 ." " . $r5 ." " . $r6 ."_" .$da ;

je place un espace entre chaque réponse ou un ;
le problème si seule le champ r1 et renseigner le sujet devient r1 _da avec des espace ou r1;;;;;_da avec des ;
comment faire pour ne pas avoir ce résulta si des champs reste vide

merci

cordialement

[renaudjuif]

bonjour,

il faut tester la valeur de chaque variable, et concaténer la chaine en conséquence.

[dlhp59]

bonjour

à tu un exemple je suis pas un pro en PHP

merci

Cordialement

[sabotage]

Prends l'habitude d'utiliser des tableaux et non des variables avec des numéros.

En l'état :

Code :

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

1
2
3
4
5
6
for ($n = 1; $n < 7; $n++) {
    if (!empty(${'r'.$n}) {
        $values[] = ${'r'.$n};
    }
}
$mail->Subject= implode(' ', $values) . $da;

[rawsrc]

Salut,

si tes valeurs proviennent d'un formulaire c'est que tu dois les récupérer dans un des supers tableaux $_POST ou $_GET.
Le mieux serait que tu regroupes ces valeurs dans un sous-tableau du $_POST ainsi cela te simplifierait le traitement :

Code html :

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

1
2
3
4
5
6
7
<input id="r1" name="subject['r1']" type="text" value="">
<input id="r2" name="subject['r2']" type="text" value="">
<input id="r3" name="subject['r3']" type="text" value="">
<input id="r4" name="subject['r4']" type="text" value="">
<input id="r5" name="subject['r5']" type="text" value="">
<input id="r6" name="subject['r6']" type="text" value="">
<input id="da" name="subject['da']" type="text" value="">

Dans ce cas, pour purger ces tableaux des valeurs de longueur nulles, il faut faire comme ça :

Code :

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

$mail->Subject = implode(' ', array_filter($_POST['subject'], 'strlen'));

[Spartacusply]

Une manière de faire en gardant les variables :

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
function buildSubject() {
    $subject = '';
    $args = func_get_args();
    $first = array_shift($args);
    $da = array_pop($args);
 
    // Test si l'élément est vide ou non
    function testNonVide($val) {
        if (isset($val) AND $val !== '') {
            return true;
        }
        return false;
    }
 
    //Traitement du premier élément
    if (testNonVide($first)) {
        $subject .= $first;
    }
    //Test des éléments intermédiaires
    foreach ($args as $key => $value) {
        if (testNonVide($value)) {
            $subject .= ' ' . $value;
        }
    }
    // Test du 'da'
    if (testNonVide($da)) {
        $subject .= '_' . $da;
    }
    return $subject;
}
 
$r1 = 'a';
$r2 = 0;
$r3 = '';
$r4 = 'null';
$r6 = null;
$r5 = 'test';
$da = 'da';
 
$subject = buildSubject($r1, $r2, $r3, $r4, $r5, $r6, $da);
 
echo $subject;

[rawsrc]

@Spartacusply
directement :

Code :

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

1
2
3
function testNonVide($val) {
    return (isset($val) and $val !== '');
}

Ensuite dans ton code tu présumes l'emplacement des variables dans le tableau :

Code :

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

1
2
$first = array_shift($args);
$da = array_pop($args);

ce qui est fortement déconseillé dans la mesure où les valeurs proviennent d'un formulaire