[PHP]Podgląd i usuwanie plików z katalogu Opcje

[Tuomi]

Witam.
Prosiłbym o pomoc w napisaniu kodu, który:
1. wyświetlałby zawartość katalogu,
2. przy każdej nazwie pliku byłby checkbox,
3. pod nazwami przycisk do kasowania zaznaczonego pliku.
Udało mi się zrobić pierwsze dwa punkty, ale nie działa mi opcja usuwania pliku.
Co zrobiłem źle?

Plik wyświetlający zawartość katalogu z checkboxami [pliki.php]:

[PHP] pobierz, plaintext 
<?
//wyswietlanie plikow
$directory="okladka";
$dir=opendir($directory);
$file_list="<ul>";
while($file_name=readdir($dir))
    {
         if(($file_name!=".")&&($file_name!=".."))
        {
        $file_list.="<li><input type=checkbox name=plik value=$file_name> $file_name</li>";
        }
    }
$file_list.="</ul>";
closedir($dir);
echo "<form action=usun-plik.php method=post>$file_list <input type=submit name=ok value=Kasuj />";
?>
[PHP] pobierz, plaintext 

Plik odpowiadający za usuwanie zaznaczonego pliku [usun-plik.php]:

[PHP] pobierz, plaintext 
<?php
include 'pliki.php';
$nazwa_pliku = $_POST['plik']; 
unlink($nazwa_pliku);
echo 'Plik pomyślnie usunięty. <a href=pliki.php>Powrót</a>';
?>
[PHP] pobierz, plaintext 

Jak zrobić, żeby taki formularz działał?
Z góry dzięki za pomoc.

[Wazniak96]

Dajesz złą ścieżkę usuwania pliku. Czytasz pliki z folderu "okladka", a próbujesz usunąć z bieżącego.

[d3ut3r]

Moim zdaniem powinieneś w ogóle unikać takich konstrukcji, co jeżeli w $_POST będzie na przykład:

../../plik.php

przy takim kodzie zostawiasz otwartą furtkę do usunięcia wszystkiego co jest w zasięgu skryptu.

[viking]

PHP daje ci takie fajne sposoby a ty robisz to brutalnym młotkiem:
http://php.net/manual/en/class.directoryiterator.php
http://php.net/manual/en/class.filesystemiterator.php
Możesz nawet http://php.net/manual/en/class.globiterator.php do szybkiego odfiltrowania.

[Tuomi]

Tylko trzeba te "fajne przykłady" rozumieć Ok. Zrobię to na zasadzie ramek tzn 3 pliki: pliki.php, all.php, del.php. W plik.php będą iframe'y do all.php i del.php, w all.php będzie wyświetlanie zawartości folderu a del.php bedzie odpowiadał za usunięcie (poprzez wpisanie nazwy w polu input). Temat można zamknąć

Udało się bez ramek zrobić
pliki-all.php

[PHP] pobierz, plaintext 
<form action="pliki-del.php?del=1" method="GET">
<?
$katalog = '../okladka';
$dir = opendir($katalog);
while(false !== ($file = readdir($dir)))
if($file != '.' && $file != '..')
{
echo '<input type="checkbox" name="file" value="'. $file . '" /> '. $file . '<br />';
}
closedir($dir);
?>
<br><input type="submit" value="Kasuj" />!<br>
</form>
[PHP] pobierz, plaintext 

pliki-del.php

[PHP] pobierz, plaintext 
<?
$del=$_GET['del'];
$file = $_GET['file'];
$katalog = '../okladka';
if (unlink($katalog .'/'. $file))
 {
 echo "Plik zostal pomyslnie usuniety. <a href=pliki-all.php>Wroc</a>.";
 }
?>
[PHP] pobierz, plaintext 

Powyższe pliki są umieszczone w katalogu pliki, zaś katalog z plikami (okladka) jest umieszczony w katalogu głównym
Skrypt może nie profesjonalny, ale ważne, że działa:)
Jeszcze takie małe pytanie. Co trzeba zmienić/wstawić, żeby w przypadku putego katalogu wyświetlało: "Brak plików do usunięcia."
Próbowałem użyć

[PHP] pobierz, plaintext 
if (!empty($dir))
{
echo '<br>Brak plików do usunięcia!<br>';
}
[PHP] pobierz, plaintext 

ale wtedy echo jest wyświetlane niezależnie od tego czy katalog jest pusty czy nie. Z góry dzięki za pomoc;)

Ten post edytował Tuomi 14.12.2012, 17:48:40