problema con l'if e operatori

[bhoo]

salve a tutti non capisco come mai non mi funziona questa condizione.Io vorrei che se l'estensione del file fosse diverso da png, gif, jpg il file non venisse caricato... ma mi dice sempre non caricato che sia tra png,gif,jpg o no mi da sempre esetensione non consentita.
Come mai ?



$estensione = pathinfo($nomefile);
if($estensione['extension']!="png" || $estensione['extension']!="gif" || $estensione['extension']!="jpg"){
echo "estensione non consentita ".$estensione['extension'];
exit;
}

if (!file_exists("upload/".$nomefile)) {
file_put_contents("upload/".$nomefile, $streamdati);
echo "Il file <a href=\"upload/".$nomefile."\">" .$nomefile. "</a> e\' stato copiato sul server

".$estensione['extension'];
}else{
echo "il file esiste".$estensione['extension'];
}

[boots]

Non ci va OR, ma AND

if($estensione['extension']!="png" && $estensione['extension']!="gif" && $estensione['extension']!="jpg"){

[bhoo]

ma se ci metto and e non or vuol dire che per soddisfare la condizione l'estensione deve essere sia gif jpg png ed è impossibile o sbaglio io?

[satifal]

Originariamente inviato da bhoo
ma se ci metto and e non or vuol dire che per soddisfare la condizione l'estensione deve essere sia gif jpg png ed è impossibile o sbaglio io?

Veramente nella condizione hai il NOT!
Per cui per soddisfare la condizione l'estensione NON deve essere gif, jpg e png

[boots]

Ma tu stai usando il NOT.
Se per esempio l'estensione è png, la prima è falsa, ma le altre sono (per forza) vere.
Con l'or, ne basta una vera per entrare nel if

Volendo fare con OR, dovresti fare
if(!($estensione == "png" || $estensione == "jpg" || $estensione == "gif")){
// Non permesse
}

PS: se vuoi fare una cosa più carina, fai

$allow_ext = array('jpg', 'png', 'gif');
if(!in_array($estensione, $allow_ext)){
// Non permesse
}

In questo modo, ti resta più semplice togliere/aggiungere le estensioni...che magari poi riprendere da un file di configurazione

[bhoo]

Originariamente inviato da satifal
Veramente nella condizione hai il NOT!
Per cui per soddisfare la condizione l'estensione NON deve essere gif, jpg e png

esatto infatti io voglio che SE NON è una di quell'estensioni termini l'esecuzione dello script, ma non succede o meglio mi dice in ogni caso estensione non consentita.

[satifal]

Originariamente inviato da bhoo
esatto infatti io voglio che SE NON è una di quell'estensioni termini l'esecuzione dello script

Allora "sei de coccio" come si dice a Roma!
boots ti ha già fornito la soluzione!!!

Se l'estensione è DIVERSA da gif E(AND) è DIVERSA da jpg E(AND) è DIVERSA da png allora esci!

[bhoo]

Originariamente inviato da boots
Ma tu stai usando il NOT.
Se per esempio l'estensione è png, la prima è falsa, ma le altre sono (per forza) vere.
Con l'or, ne basta una vera per entrare nel if

Volendo fare con OR, dovresti fare
if(!($estensione == "png" || $estensione == "jpg" || $estensione == "gif")){
// Non permesse
}

PS: se vuoi fare una cosa più carina, fai

$allow_ext = array('jpg', 'png', 'gif');
if(!in_array($estensione, $allow_ext)){
// Non permesse
}

In questo modo, ti resta più semplice togliere/aggiungere le estensioni...che magari poi riprendere da un file di configurazione

io l'ho scritta così ma mi da sempre non consentita qualsiasi file sia:
$estensione = pathinfo($nomefile); $allow_ext = array('jpg', 'png', 'gif'); if(!in_array($estensione, $allow_ext)){ echo "estensione non consentita ".$estensione['extension']; exit; } if (!file_exists("upload/".$nomefile)) { file_put_contents("upload/".$nomefile, $streamdati); echo "Il file <a href=\"upload/".$nomefile."\">" .$nomefile. "</a> e' stato copiato sul server

".$estensione['extension']; }else{ echo "il file esiste".$estensione['extension']; }

[boots]

Originariamente inviato da bhoo
io l'ho scritta così ma mi da sempre non consentita qualsiasi file sia:

$estensione = pathinfo($nomefile); 
$allow_ext = array('jpg', 'png', 'gif'); 
if(!in_array($estensione, $allow_ext))
{
     echo "estensione non consentita ".$estensione['extension'];
     exit; 
 }
 if (!file_exists("upload/".$nomefile)) 
{ 
file_put_contents("upload/".$nomefile, $streamdati); 
echo "Il file <a href=\"upload/".$nomefile."\">" .$nomefile. "</a> e' stato copiato sul server

".$estensione['extension']; 
}else{
 echo "il file esiste".$estensione['extension']; 
} 

occhio che io ho scritto $estensione per brevità...tu dovresti mettere

if(!in_array($estensione['extension'], $allow_ext)){

[bhoo]

grazie boot grazie mille funziona da Dio!!