Form contatti non funzionante

[Virus_101]

Originariamente inviato da Soop(this)
Le variabili post in questo caso sarebbero:
$message = "name:" . $_POST['nome'] . "\r\n" .
"email:" . $_POST['email'] . "\r\n" .
"testo:" . $_POST['campomsg'] . "\r\n" ;

Continuo a discordare su questo approccio.
Controllare le variabili postate dall'utente deve essere un "MUST"

[Soop(this)]

Originariamente inviato da Virus_101
Continuo a discordare su questo approccio.
Controllare le variabili postate dall'utente deve essere un "MUST"

Beh ovviamente per moltissime ragioni andrebbero fatti dei controlli prima
ma in questo gli servivano solo i nomi delle variabili

[Virus_101]

Capisco MA

if( !filter_var($mail,FILTER_VALIDATE_EMAIL) )
{ die("MAIL ERRATA"); }

Non e' difficile.

[Soop(this)]

Originariamente inviato da Virus_101
Capisco MA

if( !filter_var($mail,FILTER_VALIDATE_EMAIL) )
{ die("MAIL ERRATA"); }

Non e' difficile.

Allora facciamo un controllo completo:

if(!empty($_POST['campomsg']) OR ($_POST['nome'])){

if(filter_var($_POST['email'],FILTER_VALIDATE_EMAIL)){
 
 //Invia il messaggio

}else echo "Mail invalida";

}else echo "Nome o messaggio non compilati"; 

[Virus_101]

LOOOL automatizziamo

function formFieldChecker( $formData , $config )
{
      $errors = 0 ;

      foreach( $config as $c=>$v )
      {
            if( !isset($formData[$c])  )
            {
                  $errors ++ 
            }
            else
            {
                  $ctrl = false ; 

                  switch($v)
                  {
                        case "email" : $ctrl =filter_var($formData[$c],FILTER_VALçIDATE_EMAIL) ; break ;
                        /// ETC ETC ETC----
                  }
                  if( !$ctrl )
                  { $errors ++ ; }
            }
      }

      return $errors ? false : true ;
} 

Ovviamente puoi estendere questa funzione per tracciare anche cosa va in errore e non solo "se" unitamente all'estensione ai tipi etc...

Usarla diventa :

$myFormConf = array(
"nome" => "string"  ,
"cognome" =>"string" ,
"email" => "email" 
);

// Puoi fare files di configurazione e usare quella che ti serve ove ti serve :D 

$formCheck = formFieldChecker( $_POST, $myFormConf ) ; 

luce out! siamo sotto gruppo
EDIT: LOL almost server panic

[Soop(this)]

Originariamente inviato da Virus_101
LOOOL automatizziamo

function formFieldChecker( $formData , $config )
{
      $errors = 0 ;

      foreach( $config as $c=>$v )
      {
            if( !isset($formData[$c])  )
            {
                  $errors ++ 
            }
            else
            {
                  $ctrl = false ; 

                  switch($v)
                  {
                        case "email" : $ctrl =filter_var($formData[$c],FILTER_VALçIDATE_EMAIL) ; break ;
                        /// ETC ETC ETC----
                  }
                  if( !$ctrl )
                  { $errors ++ ; }
            }
      }

      return $errors ? false : true ;
} 

Ovviamente puoi estendere questa funzione per tracciare anche cosa va in errore e non solo "se" unitamente all'estensione ai tipi etc...

Usarla diventa :

$myFormConf = array(
"nome" => "string"  ,
"cognome" =>"string" ,
"email" => "email" 
);

// Puoi fare files di configurazione e usare quella che ti serve ove ti serve :D 

$formCheck = formFieldChecker( $_POST, $myFormConf ) ; 

luce out! siamo sotto gruppo
EDIT: LOL almost server panic

Voluto fare le cose in grande è LOL, comunque utilizzare una funzione
riutilizzabile è sempre la miglior cosa


Ps: Manca un ; e potresti usare gli shortif per incasinare meno il codice

[Virus_101]

si ma preferesico sempre compattare il meno possibile in modo da avere i blocchi e sottoblocchi di codice sempre ben allineati e meno inlined possibile.