Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [MySQL][PHP]Krzaczki w header
Forum PHP.pl > Forum > Przedszkole
Mefiuu
28.03.2011, 21:22:07
Witam. Zrobiłem system uploadu i pobierania. Pobieranie działało dotychczas dobrze, toteż dziwi mnie trochę teraźniejsze jego zachowanie. Otóż przy uploadzie dodaję do bazy info o pliku a przy downloadzie działa taki kod:

[PHP] pobierz, plaintext 
  1. $id = $_GET['file'];
  2.  
  3. if(isset($_GET['file']) && $_GET['file']==$id) {
  4.  
  5. 	$select = mysql_query("SELECT name, rozszerzenie FROM download WHERE id='$id'");
  6. 	$select = mysql_fetch_assoc($select);
  7.  
  8. 	$typ = $select['rozszerzenie'];
  9. 	$name = $select['name'];
  10.  
  11.         header("Content-Type: $typ"); 
  12. 	header("Content-Disposition: attachment; filename=$name");
  13.  
  14. 	ob_clean(); 
  15. 	flush(); 
  16. 	readfile('files/'.$name);
  17.  
  18. }
  19.  
[PHP] pobierz, plaintext


Ten kod sam skleciłem i działał do tej pory. Później jednak zaczęły dziać się cyrki i po kliknięciu 'pobierz' zaczęły wyskakiwać 'krzaczki' takie jak:

Cytat
��ࡱ�


Kompletnie nie wiem co może być przyczyną tego że działało a teraz sypie błędami, których nie mogę pojąć. O co może chodzić? Może to być wina tego, że najpierw mam w download.php cały plik a na końcu ten kod? Może ten kod dać w osobnym pliku i się do niego odwoływać?
Proszę o podpowiedzi i dziękuję.
l0ud
28.03.2011, 22:09:18
Cytat
Może to być wina tego, że najpierw mam w download.php cały plik a na końcu ten kod?


To znaczy? Jeżeli ten kod nic wcześniej nie wysyła do przeglądarki (echo, zamknięcie znacznika php i wyplucie html etc) to nie ma żadnego problemu.

Pierwsze co mi przychodzi do głowy to BOM dodane na początku pliku. Zapisz ten skrypt edytorem Notepad++ w kodowaniu UTF-8 bez BOM (zwykły notatnik tego nie potrafi) i sprawdź czy pomogło. Jak nie podaj zawartość całego pliku download.php smile.gif
Mefiuu
29.03.2011, 21:44:36
to znaczy że mam 'wygląd' cały wcześniej, a ten kod co podałem to na końcu, po znaczniku </html>

Cytat
Pierwsze co mi przychodzi do głowy to BOM dodane na początku pliku. Zapisz ten skrypt edytorem Notepad++ w kodowaniu UTF-8 bez BOM (zwykły notatnik tego nie potrafi) i sprawdź czy pomogło. Jak nie podaj zawartość całego pliku download.php


Notatnik dawno mam za sobą, teraz każdą stronę tak robię jak pisałeś, także tą. Więc to nie w tym też na pewno tkwi problem.

kod

[PHP] pobierz, plaintext 
  1. <?php
  2. 	session_start();
  3. 	error_reporting(0);
  4. 	include("core.php");
  5. ?>
  6. <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
  7. <html xmlns="http://www.w3.org/1999/xhtml">
  8. <head>
  9. <title>archiwum</title>
  10. <meta http-equiv="Content-type" content="text/html; charset=utf-8" />
  11. <meta http-equiv="Reply-to" content="mefiuu93@gmail.com" />
  12. <meta http-equiv="Content-Language" content="pl" />
  13. <link rel="stylesheet" href="style/style.css" type="text/css" />
  14. <script type="text/javascript" src="https://ajax.googleapis.com/ajax/libs/jquery/1.5.0/jquery.min.js"></script>
  15. <script type="text/javascript" src="js/core.js"></script>
  16. </head>
  17. <body>
  18.  
  19. <div id="search">
  20. <form action="search.php" name="szukaj" method="post">
  21. 	<input type="text" name="search" value="Szukaj..." /><input type="submit" name="wyszukaj" value="Wyszukaj" />
  22. </form>
  23. </div>
  24.  
  25. <div id="bar2">
  26. 	<a href="index.php">Panel</a><a href="top.php">TOP10</a><a class="wyroznij" href="#">Wyróżnione</a><a href="help.php">Pomoc</a><a href="index.php?logout=yes">Wyloguj się</a>
  27. </div>
  28.  
  29. <div id="window2">
  30. 	<div id="up2">
  31. 		Pobierz interesujący Cię materiał
  32. 	</div>
  33. 	<div id="form2">
  34.  
  35. 		<table class="category">
  36. 			<tr>
  37. 				<td><a href="download.php">Wszystko</a></td>
  38. 				<td><a href="download.php?type=other">Inne</a></td>
  39. 				<td><a href="download.php?type=scans">Skany</a></td>
  40. 				<td><a href="download.php?type=helpers">Pomoce</a></td>
  41. 				<td><a href="download.php?type=text">Tekst</a></td>
  42. 			</tr>
  43. 		</table>
  44. 	<br />
  45.  
  46.  
  47. 	<?php
  48.  
  49.  
  50. 		mysql_connect('**', '**', '**');
  51. 		mysql_select_db('**');
  52.  
  53. 		$down_lev = mysql_query("SELECT download_level FROM login WHERE name='".$_SESSION['login']."'");
  54. 		$down_lev = mysql_result($down_lev, 0, 0);
  55.  
  56. 		$_SESSION['level'] = $down_lev;
  57.  
  58.  
  59. 		if (isset($_GET['type']) && $_GET['type']=='sciagi') {
  60.  
  61. 			selectMaterial('Ściąga');
  62.  
  63. 		}
  64. 		elseif(isset($_GET['type']) && $_GET['type']=='skany') {
  65.  
  66. 			selectMaterial('Skany');
  67.  
  68. 		}
  69. 		elseif(isset($_GET['type']) && $_GET['type']=='pomoce') {
  70.  
  71. 			selectMaterial('Pomoce');
  72.  
  73. 		}
  74. 		elseif(isset($_GET['type']) && $_GET['type']=='zadania') {
  75.  
  76. 			selectMaterial('Zadania');
  77.  
  78. 		}
  79. 		else {
  80.  
  81. 				$pliki = mysql_query("SELECT * FROM download ORDER BY id DESC");
  82.  
  83. 				echo "<table class=\"download\"> 
  84. 						<thead>
  85. 							<tr>
  86. 								<th>ID:</th><th>Rodzaj:</th><th class=\"opis\">Opis:</th><th>Data:</th><th>Wrzucił:</th><th>Rozmiar:</th><th>Pobrano:</th>";
  87.  
  88. 							if($_SESSION['level']==1 || $_SESSION['level']==3) {
  89. 								echo "<th colspan=\"3\">Opcje</th>";
  90. 							}
  91. 							else {
  92. 								echo "<th>Opcje</th>";
  93. 							}
  94.  
  95. 				echo "</tr>
  96. 						</thead><tbody>";
  97. 				$i = 1;
  98. 				while($r = mysql_fetch_assoc($pliki)) {
  99. 					if($i % 2 == 0) {
  100. 						echo "<tr class=\"tbl1\"><td>";
  101. 					}
  102. 					else {
  103. 						echo "<tr class=\"tbl2\"><td>";
  104. 					}
  105. 						echo $i;
  106. 						echo "</td><td>";
  107. 						echo $r['rodzaj'];
  108. 						echo "</td><td>";
  109. 						echo $r['opis'];
  110. 						echo "</td><td>";
  111. 						echo $r['data'];
  112. 						echo "</td><td>";
  113. 						echo $r['autor'];
  114. 						echo "</td><td>";
  115.  
  116. 							if($r['rozmiar']>1024) {
  117. 								echo round($r['rozmiar']/1024);
  118. 								echo " MB</td><td>";
  119. 							}
  120. 							else {
  121. 								echo $r['rozmiar'];
  122. 								echo " kB</td><td>";
  123. 							}
  124. 						echo $r['pobrano'];
  125.  
  126. 							if($r['pobrano']=='1') {
  127. 								echo " raz</td><td>";
  128. 							}
  129. 							else {
  130. 								echo " razy</td><td>";
  131. 							}
  132.  
  133. 							if($_SESSION['level'] == 1 || $_SESSION['level']==3) {
  134. 								echo "Edytuj</td><td><a href=\"download.php?delete=".$r['id']."\">Usuń</a></td><td><a href=\"?file=".$r['id']."\">Pobierz</a></td>";
  135. 							}
  136. 							else {
  137. 								echo "<a href=\"?file=".$r['id']."\">Pobierz</a></td>";
  138. 							}
  139. 						echo "</tr>";
  140. 					$i++;
  141. 				}
  142.  
  143. 				echo "</tbody></table>";		
  144. 		}
  145.  
  146. 		if (isset($_GET['delete']) && is_numeric($_GET['delete'])) {
  147. 			$id = $_GET['delete'];
  148.  
  149. 			$nazwa = mysql_query("SELECT name FROM download WHERE id='$id'");
  150. 			$nazwa = mysql_result($nazwa, 0, 0);
  151.  
  152. 			$usun = mysql_query("DELETE FROM download WHERE id='$id'");
  153.  
  154. 			if ($usun && unlink('files/'.$nazwa)) {
  155. 				addLog($_SESSION['login'].' usunął plik', 'O nazwie '.$nazwa, 'delete');
  156. 				header('location: download.php');
  157. 			}
  158.  
  159. 		}
  160. 		elseif(isset($_GET['delete']) && !is_numeric($_GET['delete'])) {
  161. 			header('location: download.php');
  162. 		}
  163. 	?>
  164.  
  165. 	</div>
  166. 	<div id="footer2">
  167. 		Created by Mati &copy;
  168. 	</div>
  169. </div>
  170.  
  171. </body>
  172. </html>
  173.  
  174.  
  175. <?php
  176.  
  177. $id = $_GET['file'];
  178.  
  179. if(isset($_GET['file']) && $_GET['file']==$id) {
  180.  
  181. 	$select = mysql_query("SELECT name, rozszerzenie FROM download WHERE id='$id'");
  182. 	$select = mysql_fetch_assoc($select);
  183.  
  184. 	$typ = $select['rozszerzenie'];
  185. 	$name = $select['name'];
  186.  
  187. 	header("Content-Type: $typ"); 
  188. 	header("Content-Disposition: attachment; filename=$name");
  189.  
  190. 	ob_clean(); 
  191. 	flush(); 
  192. 	readfile('files/'.$name); 
  193. }
  194.  
  195. ?>
[PHP] pobierz, plaintext



ogólnie to jest w nim wymieszane wszystko, połączenia, widok i logika, ale niestety nie umiem bawić się z czymś zaawansowanym.

Pozdrawiam.
l0ud
30.03.2011, 08:29:10
To nie powinno tak wyglądać. Przed wysłaniem pliku nie powinno być wysłane do przeglądarki nic innego. Ty wysyłasz jakiś html.

Sugeruję albo wydzielić kod pobierający do innego pliku i linkować do niego (bezpośrednio przeglądarkę, nie includy!), albo opakować ten kod html co jest wyżej w if'y wykluczające wysłanie go w momencie działania skryptu pobierającego plik. Pierwsze co ma otrzymać przeglądarka to te headery. I uważaj na puste linie przed i po znacznikach <?php ?> - ich też właściwie nie powinno być.
Mefiuu
30.03.2011, 11:09:35
Czyli sugerujesz aby to wyglądało mniej więcej tak? :

lista.php

[PHP] pobierz, plaintext 
  1.  
  2. <?php
  3. 	session_start();
  4. 	error_reporting(0);
  5. 	include("core.php");
  6. ?>
  7. <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
  8. <html xmlns="http://www.w3.org/1999/xhtml">
  9. <head>
  10. <title>archiwum</title>
  11. <meta http-equiv="Content-type" content="text/html; charset=utf-8" />
  12. <meta http-equiv="Reply-to" content="mefiuu93@gmail.com" />
  13. <meta http-equiv="Content-Language" content="pl" />
  14. <link rel="stylesheet" href="style/style.css" type="text/css" />
  15. <script type="text/javascript" src="https://ajax.googleapis.com/ajax/libs/jquery/1.5.0/jquery.min.js"></script>
  16. <script type="text/javascript" src="js/core.js"></script>
  17. </head>
  18. <body>
  19.  
  20. <div id="search">
  21. <form action="search.php" name="szukaj" method="post">
  22. 	<input type="text" name="search" value="Szukaj..." /><input type="submit" name="wyszukaj" value="Wyszukaj" />
  23. </form>
  24. </div>
  25.  
  26. <div id="bar2">
  27. 	<a href="index.php">Panel</a><a href="top.php">TOP10</a><a class="wyroznij" href="#">Wyróżnione</a><a href="help.php">Pomoc</a><a href="index.php?logout=yes">Wyloguj się</a>
  28. </div>
  29.  
  30. <div id="window2">
  31. 	<div id="up2">
  32. 		Pobierz interesujący Cię materiał
  33. 	</div>
  34. 	<div id="form2">
  35.  
  36. 		<table class="category">
  37. 			<tr>
  38. 				<td><a href="download.php">Wszystko</a></td>
  39. 				<td><a href="download.php?type=other">Inne</a></td>
  40. 				<td><a href="download.php?type=scans">Skany</a></td>
  41. 				<td><a href="download.php?type=helpers">Pomoce</a></td>
  42. 				<td><a href="download.php?type=text">Tekst</a></td>
  43. 			</tr>
  44. 		</table>
  45. 	<br />
  46.  
  47.  
  48. 	<?php
  49. 		mysql_connect('***', '**', '**');
  50. 		mysql_select_db('***');
  51.  
  52. 		$down_lev = mysql_query("SELECT download_level FROM login WHERE name='".$_SESSION['login']."'");
  53. 		$down_lev = mysql_result($down_lev, 0, 0);
  54.  
  55. 		$_SESSION['level'] = $down_lev;
  56.  
  57.  
  58. 		if (isset($_GET['type']) && $_GET['type']=='sciagi') {
  59.  
  60. 			selectMaterial('Ściąga');
  61.  
  62. 		}
  63. 		elseif(isset($_GET['type']) && $_GET['type']=='skany') {
  64.  
  65. 			selectMaterial('Skany');
  66.  
  67. 		}
  68. 		elseif(isset($_GET['type']) && $_GET['type']=='pomoce') {
  69.  
  70. 			selectMaterial('Pomoce');
  71.  
  72. 		}
  73. 		elseif(isset($_GET['type']) && $_GET['type']=='zadania') {
  74.  
  75. 			selectMaterial('Zadania');
  76.  
  77. 		}
  78. 		else {
  79.  
  80. 				$pliki = mysql_query("SELECT * FROM download ORDER BY id DESC");
  81.  
  82. 				echo "<table class=\"download\"> 
  83. 						<thead>
  84. 							<tr>
  85. 								<th>ID:</th><th>Rodzaj:</th><th class=\"opis\">Opis:</th><th>Data:</th><th>Wrzucił:</th><th>Rozmiar:</th><th>Pobrano:</th>";
  86.  
  87. 							if($_SESSION['level']==1 || $_SESSION['level']==3) {
  88. 								echo "<th colspan=\"3\">Opcje</th>";
  89. 							}
  90. 							else {
  91. 								echo "<th>Opcje</th>";
  92. 							}
  93.  
  94. 				echo "</tr>
  95. 						</thead><tbody>";
  96. 				$i = 1;
  97. 				while($r = mysql_fetch_assoc($pliki)) {
  98. 					if($i % 2 == 0) {
  99. 						echo "<tr class=\"tbl1\"><td>";
  100. 					}
  101. 					else {
  102. 						echo "<tr class=\"tbl2\"><td>";
  103. 					}
  104. 						echo $i;
  105. 						echo "</td><td>";
  106. 						echo $r['rodzaj'];
  107. 						echo "</td><td>";
  108. 						echo $r['opis'];
  109. 						echo "</td><td>";
  110. 						echo $r['data'];
  111. 						echo "</td><td>";
  112. 						echo $r['autor'];
  113. 						echo "</td><td>";
  114.  
  115. 							if($r['rozmiar']>1024) {
  116. 								echo round($r['rozmiar']/1024);
  117. 								echo " MB</td><td>";
  118. 							}
  119. 							else {
  120. 								echo $r['rozmiar'];
  121. 								echo " kB</td><td>";
  122. 							}
  123. 						echo $r['pobrano'];
  124.  
  125. 							if($r['pobrano']=='1') {
  126. 								echo " raz</td><td>";
  127. 							}
  128. 							else {
  129. 								echo " razy</td><td>";
  130. 							}
  131.  
  132. 							if($_SESSION['level'] == 1 || $_SESSION['level']==3) {
  133. 								echo "Edytuj</td><td><a href=\"download.php?delete=".$r['id']."\">Usuń</a></td><td><a href=\"?file=".$r['id']."\">Pobierz</a></td>";
  134. 							}
  135. 							else {
  136. 								echo "<a href=\"?file=".$r['id']."\">Pobierz</a></td>";
  137. 							}
  138. 						echo "</tr>";
  139. 					$i++;
  140. 				}
  141.  
  142. 				echo "</tbody></table>";		
  143. 		}
  144.  
  145. 		if (isset($_GET['delete']) && is_numeric($_GET['delete'])) {
  146. 			$id = $_GET['delete'];
  147.  
  148. 			$nazwa = mysql_query("SELECT name FROM download WHERE id='$id'");
  149. 			$nazwa = mysql_result($nazwa, 0, 0);
  150.  
  151. 			$usun = mysql_query("DELETE FROM download WHERE id='$id'");
  152.  
  153. 			if ($usun && unlink('files/'.$nazwa)) {
  154. 				addLog($_SESSION['login'].' usunął plik', 'O nazwie '.$nazwa, 'delete');
  155. 				header('location: download.php');
  156. 			}
  157.  
  158. 		}
  159. 		elseif(isset($_GET['delete']) && !is_numeric($_GET['delete'])) {
  160. 			header('location: download.php');
  161. 		}
  162. ?>
  163. 	</div>
  164. 	<div id="footer2">
  165. 		Created by Mati &copy;
  166. 	</div>
  167. </div>
  168.  
  169. </body>
  170. </html>
[PHP] pobierz, plaintext


download.php
[PHP] pobierz, plaintext 
  1. <?php
  2. $id = $_GET['file'];
  3.  
  4. if(isset($_GET['file']) && $_GET['file']==$id) {
  5.  
  6. 	$select = mysql_query("SELECT name, rozszerzenie FROM download WHERE id='$id'");
  7. 	$select = mysql_fetch_assoc($select);
  8.  
  9. 	$typ = $select['rozszerzenie'];
  10. 	$name = $select['name'];
  11.  
  12. 	header("Content-Type: $typ"); 
  13. 	header("Content-Disposition: attachment; filename=$name");
  14.  
  15. 	ob_clean(); 
  16. 	flush(); 
  17. 	readfile('files/'.$name); 
  18. }
  19. ?>
[PHP] pobierz, plaintext


żeby z lista.php odwoływać się linkami do download.php i tam odbierać te dane?

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.