Forum PHP.pl

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 

$nazwa = $files_tab[$i][0];
Co to niby ma być? Nigdzie nie masz zmiennej $i ale nie przeszkadza ci to z niej korzystać....
Debuguj swój skrypt, sprawdzaj co zawierają zmiennej na danym etapie a nie napiszesz coś w pół strzelone w pól może troche przemyślane i dalej nic...
Tu masz pare wskazowek, zastosuj się do nich
Temat: Jak poprawnie zada pytanie

[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);
    }
}
for ($i=0; $i<count($files_tab); $i++) {
$nazwa = $files_tab[$i][0];
echo $nazwa;
}
$plik = $_GET['plik'];
if($plik == $nazwa)
{
unlink($nazwa);
}
 
}
[PHP] pobierz, plaintext 

w zmiennej $nazwa są nazwy plików, które są na serwerze natomiast nie działa usuwanie

Nie, w zmiennej $nazwa po pętli jest ostatni plik. I tyle. Nic więcej.

Pozatym readdir zwraca ci samą nazwę pliku, a ty chyba ten plik masz w katalogu data/files/ więc i do unlunk wypadałoby podać pełną sciezke do pliku a nie tylko jego nazwe.

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