$_post

[alcio74]

Se ti posso dare un consiglio, il miglior modo sarebbe di usare la funzione isset (LINK).

Lo script diventerebbe quindi:

$post1 = trim($_POST['post1']);
$post2 = trim($_POST['post2']);
$post3 = trim($_POST['post3']);
$post4 = trim($_POST['post4']);

if (!isset($post1) || !isset($post2) || !isset($post3) || !isset($post4)) {
echo "Spiacente, ma tutti i campi sono obbligatori.

<a href=\"javascript:history.back()\">Indietro</a>
";
} else {
/*
resto dello script
*/
} 

[filippo.toso]

Se devi dare dei consigli, almeno assicurati che siano corretti.

<?php
$post1 = isset($_POST['post1']) ? trim($_POST['post1']) : '';
$post2 = isset($_POST['post2']) ? trim($_POST['post2']) : '';
$post3 = isset($_POST['post3']) ? trim($_POST['post3']) : '';
$post4 = isset($_POST['post4']) ? trim($_POST['post4']) : '';

if (empty($post1) || empty($post2) || empty($post3) || empty($post4)) {
    echo "Spiacente, ma tutti i campi sono obbligatori.
    
<a href=\"java-script:history.back()\">Indietro</a>";
} else {
/*
resto dello script
*/
}
?>

[dottwatson]

Originariamente inviato da filippo.toso
Se devi dare dei consigli, almeno assicurati che siano corretti.

<?php
$post1 = isset($_POST['post1']) ? trim($_POST['post1']) : '';
$post2 = isset($_POST['post2']) ? trim($_POST['post2']) : '';
$post3 = isset($_POST['post3']) ? trim($_POST['post3']) : '';
$post4 = isset($_POST['post4']) ? trim($_POST['post4']) : '';

if (empty($post1) || empty($post2) || empty($post3) || empty($post4)) {
    echo "Spiacente, ma tutti i campi sono obbligatori.
    
<a href=\"java-script:history.back()\">Indietro</a>";
} else {
/*
resto dello script
*/
}
?>

e se vogliamo continuare a darli giusti continuaiamo...

if (empty($post1) || empty($post2) || empty($post3) || empty($post4)) { 

sconsiglio l' uso di empty in questo caso in quanto se nel post viene richiesto un valore numerico, e 0 è accettato come valore da passare, empty fallisce perchè per lui 0 è empty

inoltre farei così:

foreach ($_POST as $key=>$value)
  {
  if (trim($value)=="")
    {
    echo "spiacente! mi manca un parametro... ";
    break;
    }
  } 

[ntd]

Originariamente inviato da dottwatson

foreach ($_POST as $key=>$value)
  {
  if (trim($value)=="")
    {
    echo "spiacente! mi manca un parametro... ";
    break;
    }
  } 

Approvo. Io però la metterei così:

$required = array('username', 'password', ...)
foreach($required as $id) {
    if (!isset($_POST[$id]) || trim($_POST[$id]) == '') {
        echo "spiacente! mi manca il parametro $id";
        break;
    }
} 

[dottwatson]

Originariamente inviato da ntd
Approvo. Io però la metterei così:

$required = array('username', 'password', ...)
foreach($required as $id) {
    if (!isset($_POST[$id]) || trim($_POST[$id]) == '') {
        echo "spiacente! mi manca il parametro $id";
        break;
    }
} 

approvo anche io la mai controlla che TUTTI i campi siano compilati, la tua è dedicata a certi campi giustamente

direi che qui il thread ha trovato il suo porto di attracco