Autor Zpráva
lenka
Profil *
#1 · Zasláno: 24. 10. 2010, 10:50:32 · Upravil/a: Moderátor (editace znemožněna)
Odpovědět Citovat
Ahoj mám fotogalerii kterou ukládám do DB a soubory nahrávam na servr do adresáře. Teď se lopotím s tím abych mohla vymazat nějaké fotky určitého druhu a s nimi i ty soubory nahrané na servru. Z databaze to vymažu jednoduše to mi je jasné ale jde mi hlavně o ty soubory v adresáři. Napadlo mě načíst názvy souboru do pole a to pak celé nějak smazat, ale moc netuším jak to formulovat. 
<?php  
                $query = "SELECT * FROM foto where druh = 'sport'; 
                $vysledek = mysql_query($query, $link) or die("SQL dotaz neni mozno provézt");
                while ($zaznam=MySQL_Fetch_Array($vysledek))
                   {
                    $pole[] =$zaznam["foto"];
}
 ?>
No a teď bych to měla asi smazat pře 
unlink
ale bohužel ten zápis se mi nedaří…

Děkuji za rady.

Moderátor panther: Vkládej prosím kódy mezi značky [pre] a [/pre] (stačí kliknout na ).
Alphard
Profil
#2 · Zasláno: 24. 10. 2010, 10:59:06
Odpovědět Citovat
Do pole to ani nemusíte načítat. V cyklu while dejte hned unlink().
lenka
Profil *
#3 · Zasláno: 24. 10. 2010, 11:17:38
Odpovědět Citovat
Myslíte nějak takto? 

<?php  
                $query = "SELECT * FROM foto where druh = 'sport'; 
                $vysledek = mysql_query($query, $link) or die("SQL dotaz neni mozno provézt");
                while ($zaznam=MySQL_Fetch_Array($vysledek))
                   {

                       echo  (" <a href='smazat?soubor=".$zaznam["foto"]."'>Smazat</a>");
                    }
 ?>
Soubor smazat.php: 
<?php
 $smazat = $GET_["soubor"];
 unlink("../fotogalerie/foto/$smazat");
 
$path= 'http://' . $_SERVER['SERVER_NAME'] . dirname($_SERVER['SCRIPT_NAME']) . '/fotogalerie.php'; 
Header("Location:".$path); 
exit; 
 
?>


Takto mi to ale nemaže
Taps
Profil
#4 · Zasláno: 24. 10. 2010, 11:21:41 · Upravil/a: Moderátor (editace znemožněna)
Odpovědět Citovat
lenka:
zkus 
<?php  
                $query = "SELECT * FROM foto where druh = 'sport'"; 
                $vysledek = mysql_query($query, $link) or die("SQL dotaz neni mozno provézt");
                while ($zaznam=MySQL_Fetch_Array($vysledek))
                   {
                    unlink("../fotogalerie/foto/".$zaznam["foto"]);
}
 ?>
Moderátor Alphard: Opravil jsem chybějící uvozovku.
lenka
Profil *
#5 · Zasláno: 24. 10. 2010, 11:34:21
Odpovědět Citovat
A šlo by to udělat abych to měla jako ten odkaz jak jsem zkoušela aby se mi to smazalo až po kliknutí?
Taps
Profil
#6 · Zasláno: 24. 10. 2010, 12:00:53 · Upravil/a: Taps
Odpovědět Citovat
lenka: 
<?php  
                $query = "SELECT * FROM foto where druh = 'sport'"; 
                $vysledek = mysql_query($query, $link) or die("SQL dotaz neni mozno provézt");
                while ($zaznam=MySQL_Fetch_Array($vysledek))
                   {

                       echo  (" <a href='smazat.php?soubor=".$zaznam["foto"]."'>Smazat</a>");
                    }
 ?>

<?php
 $smazat = $GET_["soubor"];
 unlink("../fotogalerie/foto/".$smazat);
 
$path= 'http://' . $_SERVER['SERVER_NAME'] . dirname($_SERVER['SCRIPT_NAME']) . '/fotogalerie.php'; 
Header("Location:".$path); 
exit; 
 
?>
AM_
Profil
#7 · Zasláno: 24. 10. 2010, 12:17:47 · Upravil/a: AM_
Odpovědět Citovat
Taps:
to je dost odvážné řešení.
smazat.php?soubor=../../index.php
//koukám že už to takhle měla lenka... není to vůbec zabezpečené, takhle může kdokoli smazat jakýkoli soubor na tvém webu! (na nastavení oprávnění přístupu ke složkám bych se nespoléhal.)
Nejednodušší ochrana je asi toto: 
$smazat = basename($_GET["soubor"]); //extrahuje z případné cesty jméno souboru, čímž eliminuje možnost smazat něco jiného, než soubor z adresáře fotogalerie/foto


lenka:
„$GET_["soubor"];“
má to být $_GET

„Z databaze to vymažu jednoduše to mi je jasné“
nikde tu nevidím, že bys to mazala z databáze. bylo by dobré ten kus kódu ukázat.

„Napadlo mě načíst názvy souboru do pole a to pak celé nějak smazat“
z toho mi vyplývá, že chceš mazat víc souborů najednou. Zde zaslaná řešení je ale mažou po jednom (klikneš na jedno "smazat" -> smaže se jeden soubor). Jak to tedy vlastně chceš?
lenka
Profil *
#8 · Zasláno: 24. 10. 2010, 12:21:07
Odpovědět Citovat
Takto mám přesně takto ale bohužel mi nic nemaže, smazou se mi udaje akort z databaze ale soubory ne: 
<?php  
                $query = "SELECT * FROM foto where druh = 'sport'"; 
                $vysledek = mysql_query($query, $link) or die("SQL dotaz neni mozno provézt");
                while ($zaznam=MySQL_Fetch_Array($vysledek))
                   {
                   echo  (" <a href='smazat-foto.php?sezona=".$zaznam["sezona"]."&druh=".$zaznam["druh"]."'>Smazat</a>");
                   }
 ?>

Soubor smazat-foto.php: 
<?php
$sezona=$_GET["sezona"];
$druh=$_GET["druh"];
$query = "SELECT * FROM foto WHERE sezona = '$sezona' and druh = '$druh'";
$vysledek = mysql_query($query, $link) or die("SQL dotaz nešlo provézt");
          while ($zaznam=MySQL_Fetch_Array($vysledek))
                   {
                    unlink("../fotogalerie/foto/".$zaznam["foto"]);
                   }

$vysledek=mysql_query($q="delete from foto where sezona = '$sezona' and druh = 'extraliga'");

$path = 'http://' . $_SERVER['SERVER_NAME'] . dirname($_SERVER['SCRIPT_NAME']) . '/fotogalerie.php';      
Header("Location:".$path); 
?>
AM_
Profil
#9 · Zasláno: 24. 10. 2010, 12:49:33
Odpovědět Citovat
a řádek 8 se provede a nic nesmaže, nebo se to do toho cyklu ani nedostane? vypiš si adresu té fotky, kterou se snažíš mazat (to co je ve funkci unlink), podívej se, zda fotka opravdu existuje... chybové hlášky máš zapnuté?

lenka:
„mysql_query($query, $link)“ parametr $link můžeš vynechávat, tak bych ho tam zbytečně nedával, určitě ne tak že jednou ano jednou ne.
lenka
Profil *
#10 · Zasláno: 24. 10. 2010, 13:36:30
Odpovědět Citovat
Když si vypíšu 
echo $zaznam["foto"];
před unlink tak tam ten název souboru vidím, ale jenom jeden. Pak to ale nesmaže nic a chtěla bych smazat všechny soubory daného druhu, ale echo mi vypiše pouze jeden.
lenka
Profil *
#11 · Zasláno: 24. 10. 2010, 13:50:56
Odpovědět Citovat
Omlouvám se měla jsem chybu v cestě, už mi to i maže soubory, ještě bych tě poprosila s tou ochranou jak jsi psal basename kam bych to měla začlenit. A to že mi echo vypsalo pouze ten jeden soubor ale smazali se všechny co jsem chtěla to je normální?
panther
Profil
#12 · Zasláno: 24. 10. 2010, 13:52:49
Odpovědět Citovat
lenka:
„A to že mi echo vypsalo pouze ten jeden soubor ale smazali se všechny co jsem chtěla to je normální?“
záleží, kam to echo dáš. Předpokládám, že jsi ho dala mimo cyklus while, a pak je to v pořádku. Kdybys ho dala dovnitř cyklu, vypíší se všechny názvy.
AM_
Profil
#13 · Zasláno: 24. 10. 2010, 14:15:28
Odpovědět Citovat
lenka:
„ještě bych tě poprosila s tou ochranou jak jsi psal basename kam bych to měla začlenit“
když to máš takhle tak nikam, předtím ten tvůj kód mazal jinak. takhle bys jen měla mít ošetřené $sezona a $druh funkcí mysql_real_escape_string než je vložíš do dotazu na 4. řádku.
lenka
Profil *
#14 · Zasláno: 24. 10. 2010, 21:39:40
Odpovědět Citovat
Takto to oštření stačí? 
<?php  
                $query = "SELECT * FROM foto where druh = 'sport'"; 
                $vysledek = mysql_query($query, $link) or die("SQL dotaz neni mozno provézt");
                while ($zaznam=MySQL_Fetch_Array($vysledek))
                   {
                       $zaznam["sezona"] = mysql_real_escape_string($zaznam["sezona"]);
                       $zaznam["druh"] = mysql_real_escape_string($zaznam["druh"]); 
                      echo  (" <a href='smazat-foto.php?sezona=".$zaznam["sezona"]."&druh=".$zaznam["druh"]."'>Smazat</a>");
                   }
 ?>
panther
Profil
#15 · Zasláno: 24. 10. 2010, 21:42:33 · Upravil/a: panther
Odpovědět Citovat
lenka:
„Takto to oštření stačí?“
mysql_real_escape_string(), případně (int) pro číselné vstupy se používá při ukládání dat do DB, nikoliv pro jejich čtení.
Tady je to ošetření na nic, musíš ošetřovat vstupy od uživatele ještě před vykonáním SQL dotazu.

Správně tedy je v [#8] ve druhém kódu na 4. řádku toto: 
$query = "SELECT * FROM foto WHERE sezona = '".mysql_real_escape_string($sezona)."' and druh = '".mysql_real_escape_string($druh)."'";
AM_
Profil
#16 · Zasláno: 24. 10. 2010, 22:41:20
Odpovědět Citovat
panther:
„se používá při ukládání dat do DB, nikoliv pro jejich čtení.“
neřekl bych při ukládání (chvíli jsem musel přemýšlet, jak to myslíš) - přesněji - pokud něco, co zadal uživatel (přišlo to z $_GET nebo $_POST), vkládáš do stringu mysql dotazu, mělo by to být ošetřené. Proč to dělat vysvětlím na klasickém příkladu: 

SELECT * FROM users WHERE jmeno='${_GET['jmeno']}' AND heslo='${_GET['heslo']}'

rázem se heslo ' OR ''=' stane univerzálním heslem pro všechny uživatele :) po dosazení:
SELECT * FROM users WHERE jmeno='admin' AND heslo='' OR ''=''

escape funkce správně zakódují výskyt znaků, které by mohly ovlivnit význam SQL dotazu (především apostrofů uzavírajících proměnnou)
panther
Profil
#17 · Zasláno: 24. 10. 2010, 22:47:50
Odpovědět Citovat
AM:
„pokud něco […] vkládáš do stringu mysql dotazu“
vkládám do stringu MySQL dotazu == ukládám data do DB (z uživatelského vstupu jsem mohl dodat). Vkládání samozřejmě může být i update. Nemyslím si, že by to bylo řečeno tak, že by se nad tím muselo nějak zdlouhavě přemýšlet :-)

Každopádně je nesmysl použít mysql_real_escape_string() tak, jak je užito v [#14].
AM_
Profil
#18 · Zasláno: 24. 10. 2010, 23:40:21 · Upravil/a: AM_
Odpovědět Citovat
panther:
„vkládám do stringu MySQL dotazu == ukládám data do DB“
ukládám data do DB == INSERT, UPDATE
tahám data z DB == SELECT

v dotazu v [#15] panther dle mého názoru do DB nic neukládáš.

„Každopádně je nesmysl použít mysql_real_escape_string() tak, jak je užito v [#14].“
na tom se shodnem
panther
Profil
#19 · Zasláno: 24. 10. 2010, 23:52:07
Odpovědět Citovat
AM:
„v dotazu v [#15] panther dle mého názoru do DB nic neukládáš.“
áá, už vím. :-) Já žil tak nějak v představě, že v tom dotazu je INSERT, on je tam zatím SELECT. Tím se to vysvětluje, proč jsme se nemohli dohodnout. Ten příspěvek v [#15] upravím, aby tam nestrašila ta má nepozornost.
lenka
Profil *
#20 · Zasláno: 25. 10. 2010, 09:04:12
Odpovědět Citovat
Tak přeci ještě jedna prosba, zkouším se prohrabat tímto zápisem. Teď mi jde o smazaní jenom jednoho souboru, ale mám asi chybu někde v té podmínce if. Takto mi to napíše že : Pokus o smazani selhal. Když tu podmínku vymažu tak se soubor smaže, takže cestu mám teď v pořádku :-) 

$file=$zaznam["fotka"];
echo ($file);
if(strpos($file, "../") === false && strpos($file, __FILE__) === false && is_file($file)) {
  
  unlink("../fotografie/soupiska/".$zaznam["fotka"]);
}
else {
  exit ("Pokus o smazani selhal");
}
AM_
Profil
#21 · Zasláno: 25. 10. 2010, 10:37:20
Odpovědět Citovat
ta podmínka je vymyšlená špatně, uprav to takhle: 
$file=basename($zaznam["fotka"]);
if(is_file($file)) {
  unlink("../fotografie/soupiska/".$zaznam["fotka"]);
}
lenka
Profil *
#22 · Zasláno: 26. 10. 2010, 10:18:46
Odpovědět Citovat
Bohužel i takto mi to napíše že pokus selhal, přitom když vypíši 
$file
a 
$zaznam["fotka"]
tak se schodují a v adresáři ten soubor také mám. Bez té podmínky if se mi to smaže.
Majkl578
Profil
#23 · Zasláno: 26. 10. 2010, 10:24:15 · Upravil/a: Majkl578
Odpovědět Citovat
Já bych tu podmínku z [#21] napsal ještě jinak: 
$file = __DIR__ . '/../fotografie/soupiska/' . $zaznam["fotka"];
if (file_exists($file) && @unlink($file)) { //úmyslně @, zamezí E_WARNING při chybě
  echo 'Soubor smazán.';
} else {
  echo 'Soubor nelze smazat.';
}
Předpokládám, že $zaznam["fotka"] je z databáze, a proto nebude obsahovat nic jako ../index.php. Pokud ano, je to nutné ošetřit.
lenka
Profil *
#24 · Zasláno: 26. 10. 2010, 10:33:11
Odpovědět Citovat
Takhle to skončí s chybou:

Notice: Use of undefined constant __DIR__ - assumed '__DIR__' in C:\wamp\www\administrace\odstranit.php on line 15
Soubor nelze smazat.

řádek 15 je ta podmínka if
panther
Profil
#25 · Zasláno: 26. 10. 2010, 10:37:00
Odpovědět Citovat
lenka:
„Use of undefined constant“
a PHP < 5.3.0, co?


Jak praví manuál, „The directory of the file. If used inside an include, the directory of the included file is returned. This is equivalent to dirname(__FILE__). This directory name does not have a trailing slash unless it is the root directory. (Added in PHP 5.3.0.)“
lenka
Profil *
#26 · Zasláno: 26. 10. 2010, 10:46:03
Odpovědět Citovat
A jak to tedy mám udělat když mám tu nižší verzi?
__construct
Profil
#27 · Zasláno: 26. 10. 2010, 10:53:11 · Upravil/a: __construct
Odpovědět Citovat
lenka:
Tak ako napísal Majkl578 ale zmeníš: 
//$file = __DIR__ . '/../fotografie/soupiska/' . $zaznam["fotka"];
$file = dirname(__FILE__) . '/../fotografie/soupiska/' . $zaznam["fotka"];
if (file_exists($file) && @unlink($file)) { //úmyslně @, zamezí E_WARNING při chybě
  echo 'Soubor smazán.';
} else {
  echo 'Soubor nelze smazat.';
}
Časová prodleva: 13 let

Vaše odpověď

Mohlo by se hodit


Prosím používejte diakritiku a interpunkci.

Ochrana proti spamu. Napište prosím číslo dvě-sta čtyřicet-sedm:

0