Quale codice?

[Shyson]

Questi 2 codici funzionano entrambi, però secondo voi, qual'è meglio?

if((isset($_POST['invia']) || !empty($_POST['invia']))){
if ($nome == "") { 
echo "Inserire il Nome"; 
}
else {
$nome;
}
}

if((isset($_POST['invia']) || !empty($_POST['invia']))){ 
if ($nome == "") { 
echo "Inserire il Nome"; 
}

 

[criric]

Ciao, l'else del primo non ha motivo di esistere.

 

[Shyson]

Ciao, l'else del primo non ha motivo di esistere.

L'avevo pensato, ma non ero sicuro :mavieni:

 

[marino51]

scusa criric se intervengo ma, ... bacchettata sulle mie mani se sbaglio,

if((isset($_POST['invia']) || !empty($_POST['invia']))){

provo a leggere quello che hai scritto nell'if,

isset($_POST['invia'] = la variabile può essere impostata ma VUOTA
!empty($_POST['invia'] = la variabile deve essere impostata E deve avere un valore (diverso da una serie di valori che rendono falsa la risposta)

c'è forse qualche contraddizione perchè penso che a te serva controllare la variabile con valore
o comunque una possibile ottimizzazione ....

manuale alla mano,
EMPTY
Determina se una variabile è da considerare vuota.
Una variabile è considerata vuota se non esiste oppure se il suo valore è uguale a FALSE.
empty() non genera un avviso se la variabile non esiste.

da ciò, scriverei,

if( !empty($_POST['invia']) ){ 
  if ($nome == "") { 
    echo "Inserire il Nome"; 
}}

ovvero prendi 2 paghi 1
ciao
Marino

 

[Shyson]

L'ho modificato. Ho poi visto che togliendo la riga sopra funziona lo stesso, a che serve?

if((isset($_POST['invia']) || !empty($_POST['invia']))){ 
if ($nomeG == "") { 
echo "Inserire il Nome"; 
 }
  }

if ($nomeG == "") { 
echo "Inserire il Nome"; 
 }

 

[criric]

@ marino
hai ragione ! empty() controlla anche che la variabile sia settata.
Non l'ho usato mai da solo nei miei script, ma, inizierò.
In tempo di crisi è bene risparmiare anche sul codice :evil:
@Shyson
bisogna vedere quando e come imposti $nomeG

 

[Shyson]

@ marino
hai ragione ! empty() controlla anche che la variabile sia settata.
Non l'ho usato mai da solo nei miei script, ma, inizierò.
In tempo di crisi è bene risparmiare anche sul codice :evil:
@Shyson
bisogna vedere quando e come imposti $nomeG

Ho l'if sopra i codici e poi lo richiamo li. In un altro form non c'è e funziona lo stesso

if(isset($_POST['invia'])) {
$_SESSION['nomeG'] = $_POST['nomeG'];

if((isset($_POST['invia']) || !empty($_POST['invia']))){ 
$nomeG = htmlspecialchars(addslashes(ucfirst(trim($_POST['nomeG']))));
}

if((isset($_POST['invia']) || !empty($_POST['invia']))){
if ($nomeG == "") { 
echo "Inserire il Nome"; 
 }
  }
   }

Nel form ho questo

 <button type="submit" name="invia" style="cursor:pointer">Invia</button>

 

[marino51]

il bottone ti restituisce $_POST['invia'] settato, ma VUOTO, non avendo inserito nessun "value"
gli if funzionano per il solo controllo isset

personalmente considero il tuo controllo "debole",
se per qualunque ragione, una persona dovesse intervenire, modificando il tuo codice
ed inserendo una riga con
$_POST['invia'] = "";
il tuo script smette di funzionare .....

io preferisco sempre assegnare un valore significativo ai bottoni ...

il tuo esempio lo scriverei come segue,

if( !empty($_POST['invia'] ) {
  if ($nomeG == "") { 
    echo "Inserire il Nome";
    ---->>> gestire l'errore 
  }
  $nomeG = htmlspecialchars(addslashes(ucfirst(trim($_POST['nomeG']))));
  $_SESSION['nomeG'] = $nomeG;
}

o comunque in una forma facile per il controllo degli errori ....
ed il bottone

 <button type="submit" name="invia" value="invia" style="cursor:pointer">Invia</button>

ciao
Marino

 

[Shyson]

il bottone ti restituisce $_POST['invia'] settato, ma VUOTO, non avendo inserito nessun "value"
gli if funzionano per il solo controllo isset

personalmente considero il tuo controllo "debole",
se per qualunque ragione, una persona dovesse intervenire, modificando il tuo codice
ed inserendo una riga con
$_POST['invia'] = "";
il tuo script smette di funzionare .....

io preferisco sempre assegnare un valore significativo ai bottoni ...

il tuo esempio lo scriverei come segue,

if( !empty($_POST['invia'] ) {
  if ($nomeG == "") { 
    echo "Inserire il Nome";
    ---->>> gestire l'errore 
  }
  $nomeG = htmlspecialchars(addslashes(ucfirst(trim($_POST['nomeG']))));
  $_SESSION['nomeG'] = $nomeG;
}

o comunque in una forma facile per il controllo degli errori ....
ed il bottone

 <button type="submit" name="invia" value="invia" style="cursor:pointer">Invia</button>

ciao
Marino

Il controllo dell'errore c'è, tu dici di mettere solo if( !empty($_POST['invia'] ) { invece di if((isset($_POST['invia']) || !empty($_POST['invia']))){ e il value.

 

[marino51]

si modificare l'if,
inserire un value significativo nel bottone
e cambiare l'ordine delle operazioni,

1 - controllo esistenza
2 - lo ripulisco
3 - lo memorizzo (SESSION) per trovarlo pulito quando lo riutilizzo in seguito
ciao

 

[Shyson]

La sessione la mantiene senza la $, così $_SESSION['nomeG'] = nomeG;

 

[criric]

scritta così : nomeG, è una costante non una variabile
le variabili pretendono il $ davanti al nome.
nel codice che hai postato $nomeG è una variabile e non è valorizzata; non può funzionare.

 

[Shyson]

si modificare l'if,
inserire un value significativo nel bottone
e cambiare l'ordine delle operazioni,

1 - controllo esistenza
2 - lo ripulisco
3 - lo memorizzo (SESSION) per trovarlo pulito quando lo riutilizzo in seguito
ciao

Ho modificato come il tuo, sembra andare meglio. Ancora una cosa: siccome il form è in fondo alla pagina, si puòmettere un segnalibro in modo che in caso di errore ritorna al form e non sopra?

echo "$errore"; 
echo "<meta http-equiv='refresh' content='1; url=".htmlspecialchars($_SERVER['PHP_SELF']) ."'>";
…
   <form method="post" name="mioformGx" id="mioG" action="<?php echo $_SERVER['PHP_SELF'];?>"...">

 

[Shyson]

scritta così : nomeG, è una costante non una variabile
le variabili pretendono il $ davanti al nome.
nel codice che hai postato $nomeG è una variabile e non è valorizzata; non può funzionare.

Ora funziona, ho spostato le sessioni sotto i POST

if(isset($_POST['invia'])) { 
$nomeG = htmlspecialchars(addslashes(ucfirst(trim($_POST['nomeG']))));

$_SESSION['nomeG'] = $nomeG;

 

[criric]

Si, così ha più senso :fonzie:

 

[Shyson]

Ora sembra tutto a posto, ho ancora una domanda: questo $_SESSION=array(); va bene in questa posizione?

if($errore==""){ 
$_SESSION=array();
echo "Messaggio inviato!"; 
echo "<meta http-equiv='refresh' content='2; url=".htmlspecialchars($_SERVER['PHP_SELF']) ."'>";
}
else { 
echo "$errore"; 
echo "<meta http-equiv=\"refresh\" content=\"1; url=".htmlspecialchars($_SERVER['PHP_SELF'])."?#segnaordine\">";
 
...

}
 } 
session_destroy();
?>

 

[criric]

SESSION è già un array non serve ridefinirlo.

 

[Shyson]

SESSION è già un array non serve ridefinirlo.

L'unico campo che non mantiene i dati, in caso di errore, è il textarea che non è un campo obbligatorio.

if(isset($_POST['inviaG'])) {
$messaggioG = htmlspecialchars(addslashes(ucfirst(trim($_POST['messaggioG'])))); 
$_SESSION['messaggioG'] = $messaggioG;
...
 } 
session_destroy();
?>

…

<textarea name="messaggioG" id="txtScrivimiG"><?php echo $messaggioG; ?></textarea>

<button type="submit" name="inviaG" value="inviaG" style="cursor:pointer">Invia</button>

 

[criric]

non mantiene i dati perchè devi stampare la sessione non la variabile

<?php echo $_SESSION['messaggioG']; ?>

 

[Shyson]

non mantiene i dati perchè devi stampare la sessione non la variabile

<?php echo $_SESSION['messaggioG']; ?>

Non è la sessione questa? Nell'altro form funziona