Sonderzeichen entfernen?

[mo78]

Guten Morgen,

ich habe das Problem das ich nach einem Datei(.mp3) upload die Sonderzeichen aus dem Dateinamen entfernen möchte bzw durch einen - ersetzt werden sollen.

Meine idee ist :

$replace = array ('!','§',' $','%',' &',' ?','@');
$name =  str_replace($replace, "-",($_FILES['datei']['name'])) ; 

Leider Passiert so mit dem String leider garnichts??

Aber bin mir auch nicht sicher ob ich so auf dem richtigen Weg bin.

Schonmal besten Dank für jeden Hinweis
mfg
mo

[fireweasel]

Guten Morgen,

ich habe das Problem das ich nach einem Datei(.mp3) upload die Sonderzeichen aus dem Dateinamen entfernen möchte bzw durch einen - ersetzt werden sollen.

Definiere "Sonderzeichen".

Es gibt noch mehr Zeichen als die von dir aufgezählten, die in Dateinamen Probleme machen können.

Meine idee ist :

$replace = array ('!','§',' $','%',' &',' ?','@');
$name =  str_replace($replace, "-",($_FILES['datei']['name'])) ; 

Ich werde aus deiner Liste ('!','§',' $','%',' &',' ?','@') nicht schlau. Sind die Leerzeichen vor '$', '&' und '?' Absicht oder Tippfehler?

Fragezeichen sollten im Namen einer hochgeladenen Datei gar nicht erst vorkommen.

Leider Passiert so mit dem String leider garnichts??

Das kann ich dir nicht beantworten. Aber du kannst nachsehen, was in $_FILES['datei']['name'] steht, und was dein str_replace() daraus gemacht hat:

// vorher:
var_dump($_FILES['datei']['name']);
// nachher:
var_dump($name); 

Aber bin mir auch nicht sicher ob ich so auf dem richtigen Weg bin.

Eventuell ist strtr() der geeignetere Kandidat für diesen Fall?

$name = strtr($_FILES['datei']['name'], '!§$%&?@ ', '-'); 

[mo78]

Ok ich werde das mit vardump mal checken.

Ja die Leerzeichen sind Tippfehler

Aber schonmal Danke
mfg
mo

[h3ll]

Ich würde nur Zeichen aus einer Whitelist zulassen. Diese 8 Zeichen sind nicht die einzigen Zeichen, die negative Auswirkungen haben können. Das kann eine gefährliche Sicherheitslücke sein.

Oder noch besser: Dateien nur durchnummerieren und den Originalnamen in die Datenbank speichern.

[fireweasel]

Ich würde nur Zeichen aus einer Whitelist zulassen. Diese 8 Zeichen sind nicht die einzigen Zeichen, die negative Auswirkungen haben können. Das kann eine gefährliche Sicherheitslücke sein.

Das wäre dann der nächste Schritt gewesen, weil ich finde, dass zum Whitelisting weder str_replace() noch strtr() richtig taugen. Ich wollte nur nicht sofort mit preg_replace() kommen:

$name = preg_replace('/[^a-zA-Z0-9-]+/', '-', $_FILES['datei']['name']; 

Oder noch besser: Dateien nur durchnummerieren und den Originalnamen in die Datenbank speichern.

Das Durchnummerieren hat, genauso wie die Alternative, einen eindeutigen Namen mit tempnam() zu erzeugen, den Nachteil, dass gleiche Dateien nicht erkannt werden können. Wenn man über die Daten eine Prüfsumme berechnet, kann man Dubletten ausschließen.

[mo78]

Ich werd da mal ein bisschen ausprobieren aber denke preg_replace das ist das was ich brauche

Besten Dank