[PHP] pobieraczka :> Opcje

[Kao]

Witam

Takie oto coś sobie wymodziłem:

[PHP] pobierz, plaintext 
<?php
//tylko liczby
  $int_options = array("options"=>array("min_range"=>1, "max_range"=>10));
  if (!($pliczek = filter_var(trim($_GET['id']),FILTER_VALIDATE_INT,$int_options))) die("Nie ma takiego pliku!");
  
// jakis katalog
  $pliczki_dir = './245145/';    
  
// lista plikow ktore mozna pobrac
  $pliczki_tab = array('','plik1.cdr','plik2.cdr');
 
// do jednej zmiennej  
  $link_to_file = $pliczki_dir.$pliczki_tab[$pliczek];
 
// czy pliczek istnieje
  if (@file_exists($link_to_file)) { 
      $str=file_get_contents($link_to_file);
// czy to pusty plik     
      if (strlen($str) == 0) { 
      die("Nie ma takiego pliku!");
     } else {
// plik ok - wysyłanie
      header("Pragma: public");
      header("Expires: 0");
      header("Cache-Control: must-revalidate, post-check=0, pre-check=0");
      header("Cache-Control: private",false);
      header("Content-Description: File Transfer");
      header("Content-Type: application/force-download");
      header("Content-Disposition: attachment; filename=\"".$pliczki_tab[$pliczek]."\";");
      header("Content-Transfer-Encoding: binary");
      header("Content-Length: ".strlen($str));
      echo($str);
      exit; 
     }
  } die("Nie ma takiego pliku!");
?>
[PHP] pobierz, plaintext 

I moje pytania brzmią:
1. Czy lepsze jest:

[PHP] pobierz, plaintext 
<?php
header("Content-Length: ".strlen($str));
?>
[PHP] pobierz, plaintext 

czy może lepiej:

[PHP] pobierz, plaintext 
<?php
header("Content-Length: ".filesize($link_to_file));
?>
[PHP] pobierz, plaintext 

2. Czy tak prosty download nie ma jakiś "ukrytych dziurek" ?

Z góry dziękuje za odpowiedź.
Pozdr.
KaO

[erix]

czy może lepiej:

No skoro masz wczytany plik do pamięci, to po co jeszcze FS męczyć...? Ale z drugiej strony, strlen" title="Zobacz w manualu PHP" target="_manual, to jedna z najwolniejszych funkcji. Musiałbyś zrobić jakiś benchmark, aby sprawdzić.

[PHP] pobierz, plaintext 
<?php
if (strlen($str) == 0) {
?>
[PHP] pobierz, plaintext 

A nie wystarczy sprawdzić, czy zmienna == false" title="Zobacz w manualu PHP" target="_manual...? Przeczytaj nieco wyżej, co napisałem.

2. Czy tak prosty download nie ma jakiś "ukrytych dziurek" ?

Z tego, co widzę, to nie. Ale czy nie lepiej byłoby obarczyć tym zadaniem mod_rewrite...? Nie masz żadnego zliczania, to po co tracić pamięć i czas?

[Pilsener]

Pobierasz cały plik do zmiennej i wysyłasz do przeglądarki - nie wydaje mi się to dobre, nie lepiej wysyłać plik linia po linii albo po kęsku, np. po 50 kb?

[PHP] pobierz, plaintext 
<?php
$down_rate = 1024*50;
$file = fopen($plik, "r");
while(!feof($file)){
   echo fread($file,$down_rate);
}
fclose($file);
?>
[PHP] pobierz, plaintext 

- wykona się minimalnie wolniej, za to nie obciążysz w ogóle pamięci. Potem ciągłe prośby o zwiekszenie memory_limit, jak wrzucacie do zmiennych całe pliki.

Ja bym Ci jeszcze radził sprawdzić, czy działa ok w Internet Exploderze 6.0+

[Kao]

Witam
Lekko zmodyfikowałem/poprawiłem (po uwagach Waszych) i nic więcej nie będę modyfikował, bo nic nie jest doskonałe i zawsze będzie jakaś uwaga Może ten kawałek kodu w takiej postaci komuś się przyda, bo gotowców mało takich od a do z ... sam szukałem i wiem
Na pewno cyka na FF 3.0.10 oraz IE 7.0.6

[PHP] pobierz, plaintext 
<?php
//Pobieraczka v0.1  hehe
//error'ek
  define("ERROR_FILE","Nie ma takiego pliku!");
//tylko liczby a jak nie to die 
  $int_options = array("options"=>array("min_range"=>1, "max_range"=>10));
  if (!($pliczek = filter_var(trim($_GET['id']),FILTER_VALIDATE_INT,$int_options))) die(ERROR_FILE); 
// jakis katalog
  $pliczki_dir = './245145/';    
// lista plikow ktore mozna pobrac
  $pliczki_tab = array('','plik1.cdr','plik2.cdr');
// lista plików ktore można pobrać!
  $link_to_file = $pliczki_dir.$pliczki_tab[$pliczek];
// czy plik istnieje jak nie to die 
  if (!@file_exists($link_to_file)) die(ERROR_FILE); 
// plik do stringu a jak pusty to die       
  if (!($str=file_get_contents($link_to_file))) die(ERROR_FILE);       
// plik ok - wysyłanie
  header("Pragma: public");
  header("Expires: 0");
  header("Cache-Control: must-revalidate, post-check=0, pre-check=0");
  header("Cache-Control: private",false);
  header("Content-Description: File Transfer");
  header("Content-Type: application/force-download");
  header("Content-Disposition: attachment; filename=\"".$pliczki_tab[$pliczek]."\";");
  header("Content-Transfer-Encoding: binary");
  header("Content-Length: ".strlen($str));
  echo($str);
  exit();
?>
[PHP] pobierz, plaintext 

Wiem że można by i zrobić po kawałku wysyłanie i można by do tego też dodać po przerwanym downloadzie kontynuację (skrypcik), jest wiele opcji i z kawałka kodu się zaraz zrobi skrypt z użytkownikami i z bazą danych i licznikiem ściągania itp itd... ale tego na razie nie potrzeba .. ma być proste i wydajne ... może lekko obciąża serwer (myślę, że nie aż tak bo pliki jakie będą pobierane to maks 40 kB), ale cóż, zawsze jest jakaś słaba strona (wiem, że staramy się Tu aby tych stron nie było)
Pozdr.
KaO
Dziękuję za rady/uwagi

Witam ponownie
Troszkę mi to wierciło w głowie dziurkę z tym wysyłaniem step by step. W razie W zmodyfikowałem kod:

[PHP] pobierz, plaintext 
<?php
//Pobieraczka v0.2  hehe
//error'ek
 define("ERROR_FILE","Nie ma takiego pliku!");
//tylko liczby a jak nie to die
 $int_options = array("options"=>array("min_range"=>1, "max_range"=>10));
 if (!($pliczek = filter_var(trim($_GET['id']),FILTER_VALIDATE_INT,$int_options))) die(ERROR_FILE);
// jakis katalog
 $pliczki_dir = './245145/';    
// lista plikow ktore mozna pobrac
 $pliczki_tab = array('','plik1.cdr','plik2.cdr');
// lista plików ktore można pobrać!
 $link_to_file = $pliczki_dir.$pliczki_tab[$pliczek];
// czy plik istnieje jak nie to die
 if (!@file_exists($link_to_file)) die(ERROR_FILE);
// kawaleczki       
 $down_rate = 1024*50;
// otwieramy pliczek jak nie to die
 if (!($file_h = @fopen($link_to_file, "r"))) die(ERROR_FILE);      
// wszystko ok - wysyłanie, najpierw naglowki
 header("Pragma: public");
 header("Expires: 0");
 header("Cache-Control: must-revalidate, post-check=0, pre-check=0");
 header("Cache-Control: private",false);
 header("Content-Description: File Transfer");
 header("Content-Type: application/force-download");
 header("Content-Disposition: attachment; filename=\"".$pliczki_tab[$pliczek]."\";");
 header("Content-Transfer-Encoding: binary");
 header("Content-Length: ".filesize($link_to_file));
// wysyłanie pliczku step by step
 while(!feof($file_h)){
    echo fread($file_h,$down_rate);
 }
 fclose($file_h);
 exit();
?>
[PHP] pobierz, plaintext 

I teraz moje pytania?
1. Co z plikiem jak będzie np. 10 mb i na raz (czysto teoretycznie) 10 osób będzie chciało go pobrać? Nie będzie problemu z odczytem przez to że jest step by step?
2. Czy tu nie ma "dziurek" jakiś ?>:

Pozdr.
KaO
Ps. Czekam na jakiś "odzew"