Problema Download PHP

Printable View

24-06-2013, 15.02.55

Problema Download PHP

Ciao a tutti,
Ho un problema per il download dei file in PHP.
Il codice è il seguente:

Codice PHP:

<HTML> <BODY> <TABLE> <TR><TH>Fai clic sui link per scaricare i file</TH></TR> <? $cartella = opendir('mia_cartella'); while ($file = readdir($cartella)) { $array_file[] = $file; } foreach ($array_file as $file) { echo "<TR><TD><CENTER>"; if ( $file == ".." || $file == ".") { continue; } $dimensioni_file=filesize("mia_cartella/".$file); echo "<a href=\" downloadit.php?nome_file=$file&dimensioni_file=$dimensioni_file \">$file</a>,<BR>"; echo "</TR></TD></CENTER>"; } ?> </TABLE> </BODY> </HTML>

In seguito bisogna aprire la pagina 'downloadit.php' dove il codice è il seguente:

Codice PHP:

<?php header("Content-type: Application/octet-stream"); header("Content-Disposition: attachment; filename=$nome_file"); header("Content-Description: Download PHP"); header("Content-Length: $dimensioni_file"); readfile($nome_file); ?>

Tutto funziona ma quando vado a cliccare sul link, non mi scarica il file su cui ho cliccato ma il file 'downloadit.php'.
Aiuto!!! Come posso fare??
Grazie
24-06-2013, 15.28.28

Codice PHP:

<?php header("Content-type: Application/octet-stream"); header("Content-Disposition: attachment; filename=$nome_file"); header("Content-Description: Download PHP"); header("Content-Length: $dimensioni_file"); readfile($nome_file); ?>

le variabili di downloadit.php come fanno essere settate visto che nel codice non c'è una variabile di tipo get?
24-06-2013, 16.00.55

Scusa l'ignoranza ma è da poco che programmo in php,
non ho capito bene dove dovrei inserire la variabile get!! :|
potresti scrivermi il codice completo? :=):
Grazie!!!
24-06-2013, 16.21.11
alemoppo
- So che all'inizio può essere più facile, ma non usare le tabelle per impaginare i contenuti.
- Il tag <center> è obsoleto. Si usa il css per gli attributi di stile.
- Nei link che prepari:
  
  Citazione:
  
  downloadit.php?nome_file=$file&dimensioni_file=$di mensioni_file
  
  Invii 2 parametri GET: $_GET['nome_file'] e $_GET['dimensioni_file']. Quindi, le variabili $nome_file e $dimensioni_file saranno non definite. Devi prima prelevare i contenuti da $_GET.
- Non ha senso passare anche la dimensione, perché è facilmente calcolabile direttamente dalla pagina "download.php"
- Il contenuto di una cella della tabella deve essere dentro <td>, e non puoi scrivere del contenuto direttamente in <tr>!
Ciao!
24-06-2013, 17.39.56

Ciao ma questi 2 parametri dove li devo inserire ma soprattutto come li devo scrivere?
Potresti scrivermi il codice corretto??
Grazie!!:=)::=)::=):
24-06-2013, 18.14.40
alemoppo

Citazione:

Originalmente inviato da federix98

Ciao ma questi 2 parametri dove li devo inserire ma soprattutto come li devo scrivere?

Come li ho scritti io

Citazione:

Originalmente inviato da federix98

Potresti scrivermi il codice corretto??

No: sarebbe troppo facile :tongue

Per imparare a ricercare le cose, devi sbatterci la testa.

:arrow: Documentazione ufficiale $_GET
:arrow: Qualche ricerca su google

EDIT: ovviamente, se non capisci qualcosa, siamo qui :)

Ciao!
24-06-2013, 19.23.13

infatti se vuoi imparare a programmare in php almeno il get lo devi conoscere...
24-06-2013, 20.00.48

Ok Grazie mille della documentazione.
25-06-2013, 10.35.31

Quindi devo inserire queste 2 righe riguardo quello che ho trovato su internet:
$nome_file = $_GET['nome_file'];
$dimensioni_file = $_GET['dimensioni_file'];
???:shock:
25-06-2013, 12.37.47
alemoppo

Prova e lo scoprirai :mrgreen:

Ciao!
25-06-2013, 14.24.12

Già fatto non funziona!!!
Ciao
25-06-2013, 14.26.13
alemoppo

Cosa appare? Appare qualche errore? Le righe le hai inserite sopra i vari "header()"?

Ciao!
25-06-2013, 15.01.20

No le ho inserite sia in questo file:

Codice PHP:

<HTML> <BODY> <TABLE> <TR><TH>Fai clic sui link per scaricare i file</TH></TR> <? $cartella = opendir('mia_cartella'); while ($file = readdir($cartella)) { $array_file[] = $file; } foreach ($array_file as $file) { echo "<TR><TD><CENTER>"; if ( $file == ".." || $file == ".") { continue; } $dimensioni_file=filesize("mia_cartella/".$file); echo "<a href=\" downloadit.php?nome_file=$file&dimensioni_file=$dimensioni_file \">$file</a>,<BR>"; echo "</TR></TD></CENTER>"; } ?> </TABLE> </BODY> </HTML>

che in questo:

Codice PHP:

<?php header("Content-type: Application/octet-stream"); header("Content-Disposition: attachment; filename=$nome_file"); header("Content-Description: Download PHP"); header("Content-Length: $dimensioni_file"); readfile($nome_file); ?>

Ma il file che scarica quando clicco sul link è sempre il file "downloadit.php";
Provate anke voi:confused::confused::confused::confused:!!!
Grazie, ciao!!!
25-06-2013, 16.11.46
alemoppo

I parametri get servono soltanto nel secondo file. (tra l'altro, serve soltanto nomefile, perché la dimensione è molto meglio ricavarla direttamente nel secondo file).
Sarebbe comunque utile vedere i codici modificati e non quelli originari, per capire il problema.

Ciao!
25-06-2013, 16.20.44

Ok, quindi il primo codice è uguale.
Il secondo è così:

Codice PHP:

<?php $_GET['nome_file'] = $nome_file; $_GET['dimensioni_file'] = $dimensioni_file; header("Content-type: Application/octet-stream"); header("Content-Disposition: attachment; filename=$nome_file"); header("Content-Description: Download PHP"); header("Content-Length: $dimensioni_file"); readfile($nome_file); ?>

Giusto????:=):
Comunque questo codice non mi funziona!!!:|:|:|
A te va???
Grazie ciao!!
25-06-2013, 16.24.45
alemoppo

uhm... Devi mettere il contenuto di $_GET['nome_file'] dentro $nome_file.

Se fai:

Codice PHP:

$a = $b;

Metti il contenuti di b dentro a, e non vice versa! Qui l'avevi scritto correttamente...

Ciao!
25-06-2013, 16.27.36

Grazie!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! !!!!!!!!!!!!!!!!!!!!!!
Funziona tutto!!:=)::=)::=)::=)::=)::=)::=)::=)::=):
25-06-2013, 16.32.06
alemoppo

Eh però mantieni la calma :D
Invece di:

Codice PHP:

$dimensioni_file = $_GET['dimensioni_file'];

Sarebbe meglio fare:

Codice PHP:

$dimensioni_file=filesize("mia_cartella/".$_GET['nome_file']);

Ah, poi usare file_exists() non sarebbe male per dar una controllata se il file esiste realmente!

Se proprio vuoi far le cose fatte bene, anche una isset() su $_GET['nome_file'] ci starebbe bene: se non inviano quel parametro, il tuo script come si comporta? :D

Ciao!
25-06-2013, 16.33.38

Ottimi consigli:=):
Grazie ciao!