Tutorial HTML, CSS u. PHP - Forum

[Yamram]

Hey Leute,
diese folgende if-Abfrage ist ziemlich lang:

Code: Alles auswählen


if($file["type"] == "image/png" OR $file["type"] == "image/jpeg" OR $file["type"] == "image/gif" OR $file["type"] == "image/x-icon" OR $file["type"] == "image/bmp") {
...
}
 

Kann ich sie kürzer gestalten, denn es wird ja nur geprüft, ob $file["type"] folgenden Wert hat:

• image/png
• image/jpeg
• image/gif
• image/x-icon
• image/bmp

Gibt es da eine Möglichkeit?

Mit freundlichen Grüßen,
Philipp E.

[lauras]

Wie wäre es einfach mit:

Code: Alles auswählen

if($file["type"] == ("image/png" OR "image/jpeg" OR "image/gif" OR "image/x-icon" OR "image/bmp")) {
...
} 

?

[Yamram]

Hey Laura,
Danke für die schnelle Antwort. Das ist perfekt ;)

Mit freundlichen Grüßen,
Philipp E.

[kanufrosch]

Ansonsten kannst du auch ein Array einmal mit den Werten füllen und brauchst dann nur prüfen ob es enthalten ist... ;D
So löse ich es manchmal.

[Sören]

lauras Methode wird so nicht funktionieren.
Es wird in der if-Abfrage zuerst die OR-Verknüpfungen ausgewertet, welche immer true ergeben, da Zeichenketten, die nicht '0' und dann erst das Ergebnis mit $file['type'] verglichen.
Die if-Abfrage gibt also nur false zurück, wenn $file['type'] entweder false, 0, '0', ein leeres Array (array()), ein leerer String oder null ist.

Beispiel:

Code: Alles auswählen

<pre>
<?php
function abfrage($val)
{
    return $val == ('image/png' or 'image/jpeg' or 'image/gif' or 'image/x-icon' or 'image/bmp');
}

echo "'image/png' or 'image/jpeg' or 'image/gif' or 'image/x-icon' or 'image/bmp' ergibt:\n";
var_dump('image/png' or 'image/jpeg' or 'image/gif' or 'image/x-icon' or 'image/bmp');

echo "\n\nabfrage('image/jpg') ergibt:\n";
var_dump(abfrage('image/jpg'));
echo "\nabfrage('a') ergibt:\n";
var_dump(abfrage('a'));
echo "\nabfrage(2) ergibt:\n";
var_dump(abfrage(2));
echo "\nabfrage(true) ergibt:\n";
var_dump(abfrage(true));
echo "\nabfrage('0') ergibt:\n";
var_dump(abfrage('0'));
echo "\nabfrage(0) ergibt:\n";
var_dump(abfrage(0));
echo "\nabfrage(false) ergibt:\n";
var_dump(abfrage(false));
?>
</pre>

Ausgabe:

Code: Alles auswählen

<pre>
'image/png' or 'image/jpeg' or 'image/gif' or 'image/x-icon' or 'image/bmp' ergibt:
bool(true)


abfrage('image/jpg') ergibt:
bool(true)

abfrage('a') ergibt:
bool(true)

abfrage(2) ergibt:
bool(true)

abfrage(true) ergibt:
bool(true)

abfrage('0') ergibt:
bool(false)

abfrage(0) ergibt:
bool(false)

abfrage(false) ergibt:
bool(false)
</pre>

[Yamram]

@kanufrosch: Das hört sich gut an (:

@Sören: Oh, ich habe den Code einfach übernommen. Naja, bis jetzt hat alles funktioniert, weil der User ja $file["type"] nicht eingibt. Ein switch-Statement ist dann wohl die richtige Lösung. Danke, das wäre bestimmt eine Sicherheitslücke geworden (:

[lauras]

Mh, ich hatte mal irgendwas in der Art was funktionierte.. ich schau nochmal nach..

[kanufrosch]

Hier nochmal in Code:

Code: Alles auswählen

   $i = array("image/png","image/jpeg","image/gif","x-icon","image/bmp");
   if(in_array($file["type"], $i))
   {
   ...
   }

[Yamram]

@lauras: Ich hätte auch gedacht, dass das funktioniert ;)

@kanufrosch: Gut, das sieht am besten aus ;) Danke, dann nehme ich jetzt deine Lösung. Die ist sowieso viel effizienter als das switch-Statement.

@all: Wie ihr bestimmt schon gemerkt habt, geht es hierbei um einen Upload. Die Datei wird auf ihren Typ geprüft. Und dazu jetzt eine Frage: Gibt es irgendwo so Listen der Dateitypen (für PHP natürlich)?

[Basiii]

Bei einem Fileupload Programm solltest du eher selbst auflisten was erlaubt ist und was nicht, ein dir unbekannter Dateityp kann dein ganzes System in die Luft jagen.

[Yamram]

Ja, aber zum Beispiel Bilder. Gibt es da nicht eine Liste? Denn ich denke mal, dass ich alle Bildertypen erlauben werden ;)

[drPHIP132]

.htcceess oder wie es gleich heißt unbedingt verbieten, falls du noch nicht daran dachtest


EDIT

ist zwar kein bild aber trotzdem^^

[Yamram]

Du hättest doch nicht geglaubt, dass der User so etwas hochladen darf (: Das wär doch voll dumm von mir ((((: