Forum PHP.pl

Temat do usunięcia

Ten post edytował dk1342 20.10.2008, 14:24:48

Przed unlink chyba pominąłeś

if (

klamerek nie liczyłem, ale ta otwarta zaraz za unlink i dodatkowy nawias zamykający właśnie to sugerują.

Jednak policzyłem, człowieku, sztuka tabulacji się kłania (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
<?php
if(file_exists($_GET['plik'])==true){
  $a=explode('.', $_GET['plik']);
  if(($a[count(a)]=='.zip') or ($a[count(a)]=='.rar')){
    if(md5('ala123'.$_GET['plik'].'test')==$_GET['hash']){
      if (unlink('files/'.$_GET['plik'])){
        echo "Plik: ".$_GET['plik']." został usunięty.";
      }
    }else{
      echo"Hash jest zły!";
    }
  }
}else{
  echo "Nie ma takiego pliku!!";
}
?>
[PHP] pobierz, plaintext 

Ten post edytował HellRider 18.10.2008, 17:26:00

dzięki, jest już komunikat ale pisze że nie ma takiego pliku a taki plik jest bo widze w ftp...

albo gdy usune hash czyli np.

http://www.mojastrona.pl/pliki/test/usun.p...k.zip&hash=

to zamiast pisać hash jest nie prawidłowy to pisze dalej nie ma takiego pliku...

czyli coś jest nadał źle (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

Ten post edytował dk1342 18.10.2008, 17:39:04

Widocznie warunek
(file_exists($_GET['plik'])==true)
nie jest spełniony, sprawdź czy dobrze tworzysz ścieżkę do $_GET['plik'].
Zakładam, że nie trzymasz tych plików razem z indexem i stuffem (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

nie, index mam w głównykatalog/pliki/test/index.php

a pliki:

głównykatalog/pliki/test/files

zaraz sprawdze jeszcze scieżke

Heh, no to warunek powinien wyglądać nastepująco:

(file_exists("files/".$_GET['plik'])==true)

post do usunięcia

Ten post edytował dk1342 20.10.2008, 14:25:12

if(($a[count(a)]

popraw

yyy nie wiem co ty zmieniłeś ale wsadziłem tą twoją poprawioną wersje i dalej to samo

ja ci nic nie zmienilem tylko pokazalem co TY masz zmienic (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

aaaa ok ja tego kodu nie napisałem więc nie jestem tak dobry w php ale poszukam może coś znajde

hmm chyba nie wiem, nie mam pojęcia

moze dodać $ ?

if(($a[count($a)]

(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

moze tak? i do tego gdzies jeszcze?

dodałem te $ $



ale nic nie dało dalej to samo

hint: w tablicy elementy numeruje sie od 0

hmm możesz jaśniej (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

[PHP] pobierz, plaintext 
<?php
$a = array('jeden','dwa');
echo $a[count($a)]; // nie ma takiego elementu o kluczu 2, sa tylko elementy 0 i 1
?>
[PHP] pobierz, plaintext 

// edit

down: nie wniosles raczej nic nowego

Ten post edytował pyro 18.10.2008, 20:58:19

pierwszy element ma numer zero

już się pomieszałem (IMG:http://forum.php.pl/style_emoticons/default/haha.gif) to ze zmenczenia czyli jak bedzie to wyglądało całe ?

...to się okaże jak już poprawisz.

Eh, tabulacja, tablulacja. Kod wrzucony jak do worka.
Dwie sprawy:
1. Plik aaa.jpg ma dwa człony, count($a) wynosi 2, ale $a wygląda tak:

[PHP] pobierz, plaintext 
<?php
$a[0] = "aaa";
$a[1] = "jpg"; // bez kropki
?>
[PHP] pobierz, plaintext 

więc drugi element to count($a)-1, zawsze ostatni element tablicy, której indeks rozpoczyna się od zera, to count($tablica)-1

2. explode podzieli string przez kropkę, ale ta kropka nie zostanie dodana, ona jest separatorem.

Poprawiony kod:

[PHP] pobierz, plaintext 
<?php
if (file_exists("files/".$_GET['plik'])){
  $a=explode('.', $_GET['plik']);
  if(($a[count($a)-1]=='zip') or ($a[count($a)-1]=='rar')){
    if(md5('ala123'.$_GET['plik'].'test')==$_GET['hash']){
      if (unlink("files/".$_GET['plik'])){
        echo "Plik: ".$_GET['plik']." został usunięty.";
      }else{
        echo "Błąd przy usuwaniu pliku!";
      }
    }else{
      echo "Niewłaściwy hash!";
    }
  }else{
    echo "Dane rozszerzenie pliku nie jest dozwolone!"
  }
}else{
  echo "Taki plik nie istnieje!";
}
?>
[PHP] pobierz, plaintext 

Ten post edytował HellRider 19.10.2008, 06:38:03