[PHP] Usuwanie pliku, Problem z usuwaniem pliku Opcje

[felixk]

Witam, jak w temacie. Napisałem skrypt dający możliwość usuwania plików z tym, że pliki nie chcą się usuwać. Zawszelkie pomysły będę wdzięczny

[PHP] pobierz, plaintext 
$akcja = $_GET['akcja'];
if ($akcja == "usun_plik")
{
 
$dir="data/files/";
 
 
if (is_dir($dir)) {
    if ($dh = opendir($dir)) {
        while (($file = readdir($dh)) !== false) {
        if(($file_name!=".")&&($file_name!="..")) {
 
    $file_path = $dir.$file_name;
    $file_tab = array ($file_name, filesize($file_path)); //tablica z informacjami o jednym pliku [0] -> nazwa; [1] -> rozmiar
    array_push($files_tab,  $file_tab); //dołączenie tablicy informacji o jednym pliku do tablicy wszystkich plików
    }
        }
        closedir($dh);
    }
}
 
$nazwa = $files_tab[$i][0];
 
$plik = $_GET['plik'];
if($plik == $nazwa)
{
unlink($nazwa);
}
 
}
[PHP] pobierz, plaintext 

[HTML] pobierz, plaintext 
<input onclick=\"window.location.href='http://$adres/listowanie.php?akcja=usun_plik&amp;plik=$nazwa'\" type=\"button\" value=\"Usuń plik\">
[HTML] pobierz, plaintext 

[Pilsener]

1. Daj echo $nazwa i echo $plik przed unlink i zobacz, co chcesz usunąć - czy to takie trudne?
2. Zanim cokolwiek spróbuje się usunąć to warto sprawdzić, czy to coś istnieje i mamy odpowiednie uprawnienia, by to usunąć.
3. Ten warunek pewnie nigdy nie będzie prawdziwy:

[PHP] pobierz, plaintext 
$plik = $_GET['plik'];
if($plik == $nazwa)
[PHP] pobierz, plaintext 

- bo nazwy w url są kodowane

I czemu po prostu nie użyć zmienej GET jako klucza do usunięcia pliku z tablicy?

[PHP] pobierz, plaintext 
unlink($tablica[$klucz]);
[PHP] pobierz, plaintext 

Zamiast sprawdzać po kolei w pętli? Bo zakładam, że o to ma tu chodzić? Ciężko się z kodu zorientować...

Ten post edytował Pilsener 13.04.2012, 21:39:14