voglio controllare il seguente codice

[amphioxus]

salve,
questo codice cosi fatto funziona però per mia curiosità voglio sapere se ci si può semplificare la vita
questo è il form

<td class="top" width="64%"><form name="form1"  action="segnalazioni.php" method="post">
    I campi sono tutti obbligatori<font color="#BCF2ED">

    

    Nome:

    <input type="text" name="nome" size="34"></font>


    Che tipo di link sta<font color="#BCF2ED">i segnalando?

    <input type=CHECKBOX value="video" name="_1" id="1_a"> 
                                Video

                                <input type=CHECKBOX value="appunti" name="_2" id="1_b"> 
    Appunti

                                <input type=CHECKBOX value="link di scienza" name="_3" id="1_c"> 
    Sito di scienza

    

    E - mail:

    <input type="text" name="mail" size="44"></font>


    Link da segnalare<font color="#BCF2ED">:

    puoi inserire fino a 3 link alla volta

    

    Link 1:
    <input type="text" name="link1" size="45"></font>


    Link 2: <font color="#BCF2ED">
    <input type="text" name="link2" size="45"></font>


    Link 3: <font color="#BCF2ED">
    <input type="text" name="link3" size="45"></font>


    <font color="#BCF2ED">Codice di sicurezza: 
    

    [img]captcha.php[/img]

    inserisci qui il codice che vedi in figura:

<input type='text' name='codiceverifica' size="25"></font>

    

    <input type="submit" value="Invia">
    <input type="reset" value="Cancella"></form> 

questo è il file di destinazione segnalazione.php

require('php-captcha.inc.php');
$c=0;
$r=0;
$y=0;
if (PhpCaptcha::Validate($_POST['codiceverifica'])) { 
$c=0;
}
else{
$codice="- Il codice di sicurezza è nullo o non è corretto
";
$c++;
}
$dati= array ("$_POST[nome]","$_POST[mail]");
$risp= array("nome", "mail");
$link= array ("$_POST[link1]","$_POST[link2]","$_POST[link3]");
$tipo= array ("$_POST[_1]","$_POST[_2]","$_POST[_3]");
if (empty($tipo[0])){
$y++;
}
if (empty($tipo[1])){
$y++;
}
if (empty($tipo[2])){
$y++;
}
if ($y==3){
$type= "- Devi selezionare almeno una tipologia di link";
$c++;
}
for($s=0; $s < count($link); $s++){
$link[$s]= trim($link[$s]);
}
if (empty($link[0])){
$r++;
}
if (empty($link[1])){
$r++;
}
if (empty($link[2])){
$r++;
}
if ($r==3){
$linko= "- Devi inserire almeno un link
";
$c++}for($j=0; $j < count($dati); $j++){
$dati[$j]= trim($dati[$j]);
if (empty($dati[$j])){
$campi[$j]="- Campo $risp[$j] vuoto
";
$c++;
}
}
if (empty($dati[1])){
$m=0;
}
else{
if(eregi("^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,3})$", $dati[1])){
$m++;
}
else{
$posta= "- Mail scritta non correttamente
";
$c++;
}
}
if ($c==0){    
$messaggio="Questa email ti è stata inviata dal sito.\n\nNome utente: $_POST[nome]\n\nMail: $_POST[mail]\n\nTipologia:\n$tipo[0]\n$tipo[1]\n$tipo[2]\n\n$tipoa\n\nElenco link:\n$link[0]\n$link[1]\n$link[2]";
    mail("pincopallino@live.it", "Segnalazione link", $messaggio);
echo "[b]<span style='font-size:12pt;color:white'>[b]Messaggio inviato!!![/b]</span>
<span style='color:#BCF2ED'>Tra pochi istanti verrai reindirizzato alla home del sito.[/b]

Se non vuoi aspettare oltre</span> [url='home.htm']clicca qui[/url]";
redirect("http://www.atuttascienza.altervista.org/home.htm",7);
}
else{
echo "<span style='font-size:12pt;color:white'>[b]Attenzione!!! Invio non convalidato[/b]</span>

<span style='color:white'>Ci sono i seguenti errori:
</span>";
echo "<span style='color:#BCF2ED'>";
echo $codice;
for ($x=0; $x < count($dati); $x++){
echo $campi[$x];
}
echo $linko;
echo $type;
echo $posta;
echo "</span>";
echo "
[url='javascript:history.back(-1);']Ritorna a contatti[/url]";
} 

mi controlla tutti i campi prima di procedere e poi ce la parte che viene inviata

[elmapomapo]

Originariamente inviato da amphioxus

require('php-captcha.inc.php');
$c=0;
$r=0;
$y=0;
if (PhpCaptcha::Validate($_POST['codiceverifica'])) { 
$c=0;
}
else{
$codice="- Il codice di sicurezza è nullo o non è corretto
";
$c++;
}
$dati= array ("$_POST[nome]","$_POST[mail]");
$risp= array("nome", "mail");
$link= array ("$_POST[link1]","$_POST[link2]","$_POST[link3]");
$tipo= array ("$_POST[_1]","$_POST[_2]","$_POST[_3]");
if (empty($tipo[0])){
$y++;
}
if (empty($tipo[1])){
$y++;
}
if (empty($tipo[2])){
$y++;
}
if ($y==3){
$type= "- Devi selezionare almeno una tipologia di link";
$c++;
}
for($s=0; $s < count($link); $s++){
$link[$s]= trim($link[$s]);
}
if (empty($link[0])){
$r++;
}
if (empty($link[1])){
$r++;
}
if (empty($link[2])){
$r++;
}
if ($r==3){
$linko= "- Devi inserire almeno un link
";
$c++}for($j=0; $j < count($dati); $j++){
$dati[$j]= trim($dati[$j]);
if (empty($dati[$j])){
$campi[$j]="- Campo $risp[$j] vuoto
";
$c++;
}
}
if (empty($dati[1])){
$m=0;
}
else{
if(eregi("^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,3})$", $dati[1])){
$m++;
}
else{
$posta= "- Mail scritta non correttamente
";
$c++;
}
}
if ($c==0){    
$messaggio="Questa email ti è stata inviata dal sito.\n\nNome utente: $_POST[nome]\n\nMail: $_POST[mail]\n\nTipologia:\n$tipo[0]\n$tipo[1]\n$tipo[2]\n\n$tipoa\n\nElenco link:\n$link[0]\n$link[1]\n$link[2]";
    mail("pincopallino@live.it", "Segnalazione link", $messaggio);
echo "[b]<span style='font-size:12pt;color:white'>[b]Messaggio inviato!!![/b]</span>
<span style='color:#BCF2ED'>Tra pochi istanti verrai reindirizzato alla home del sito.[/b]

Se non vuoi aspettare oltre</span> [url='home.htm']clicca qui[/url]";
redirect("http://www.atuttascienza.altervista.org/home.htm",7);
}
else{
echo "<span style='font-size:12pt;color:white'>[b]Attenzione!!! Invio non convalidato[/b]</span>

<span style='color:white'>Ci sono i seguenti errori:
</span>";
echo "<span style='color:#BCF2ED'>";
echo $codice;
for ($x=0; $x < count($dati); $x++){
echo $campi[$x];
}
echo $linko;
echo $type;
echo $posta;
echo "</span>";
echo "
[url='javascript:history.back(-1);']Ritorna a contatti[/url]";
} 

Tutto quel codice per vedere se la form è stata settata? .

Mi sembra un po troppo, guarda qui:

Utilizza qualcosa del genere:

if (!$user or !$email or !$link_1 or !$link_2 or !$link_3 or !$captcha) { 
echo "fai in modo che tutti i campi siano pieni";
}else{
//altrimenti sono tutti pieni 
} 

[amphioxus]

Originariamente inviato da elmapomapo
Tutto quel codice per vedere se la form è stata settata? .

Mi sembra un po troppo, guarda qui:

Utilizza qualcosa del genere:

if (!$user or !$email or !$link_1 or !$link_2 or !$link_3 or !$captcha) { 
echo "fai in modo che tutti i campi siano pieni";
}else{
//altrimenti sono tutti pieni 
} 

bello...
però per quanto riguarda i link vorrei che uscisse il messaggio "Devi inserire almeno un link" nel senso che basta che uno dei tre campi deve essere riempito gli altri possono essere vuoti

[elmapomapo]

Originariamente inviato da amphioxus
bello...
però per quanto riguarda i link vorrei che uscisse il messaggio "Devi inserire almeno un link" nel senso che basta che uno dei tre campi deve essere riempito gli altri possono essere vuoti

if (!$user or !$email or !$captcha) {
echo "fai in modo che tutti i campi siano pieni";
}elseif (!$link_1 or !$link_2 or !$link_3){
echo "inserisci almeno un link";
}else{
//sono tutti pieni
}