Ściągnąłem skrypt stąd: http://compzone.org/art-Licznik+pobran+pliku+w+PHP-54.html
I chciałem zrobić licznik, zwykły prosty licznik pobrań. Męczę się już chyba 5 godzinę z tym i nie ma rozwiązania. Wywala mi taki błąd:
Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in /home/xxxxxxx/public_html/forum/downloads/download.php on line 35
W bazie danych mam 'download'>'nazwapliku' i drugie pole 'iloscpobran'. Wszystko wg linka wyżej.
Szukałem na forum, w google, nawet gadałem z gościem, który miał podobny problem na gg i nie ma rozwiązania. Panowie pomóżcie, bo strace 5h czasu, a to mnie zmartwi bardziej, niż ten je***y licznik.

Jako, że jestem laikiem, proszę o specjalne nie bluzganie i itp.
download.php

[PHP] pobierz, plaintext 
 <?php
 
  //...
 $plik = $_GET['plik']; //pobiera nazwę pliku, który użytkownik chce pobrać
 
 $baza_ip = 'localhost'; //adres bazy danych (najlepiej aby był to localhost)
 $baza_uzytkownik = 'iiiiiiiiiiiiiiw'; //nazwa użytkownika bazy
 $baza_haslo = 'ciiiiiiiiiiiiizV'; //hasło dla danego użytkownika bazy
 $baza = 'tiiii_im'; //nazwa bazy glównej
 $baza_zrodlo = 'download'; //tabela, w której są zapisywane wpisy o ilości pobrań
 
 $folder_z_plikami = 'b871/'; //tutaj będą znajdować się pliki do pobrania
 //...
  echo 'test0';
 //....
 
 if ($plik != '') //sprawdzam czy zmienna plik jest pusta, jeżeli jest pusta nie wywołuję dalej kodu tylko informuję o błędzie.
 {
 if (!file_exists($folder_z_plikami.$plik)) //sprawdzam czy podany plik istnieje na serwerze
 {
 print "Podany plik: $plik nie istnieje"; //jeżeli plik nie istnieje to nie wykonuję dalszych operacji tylko wyświetlam informację o błedzie
 }
 else
 {
  echo 'test01';
 $db = mysql_connect($baza_ip, $baza_uzytkownik, $baza_haslo);
 /*
 Powyższym kodem łączę się z bazą danych mySQL korzystając ze stałych zdefiniowanych na początku skryptu. Znak małpy przed zmienną $db konieczny jest ze względu na możliwość wystąpienia błędu, który jeżeli i tak zaistnieje to i tak zostanie znaleziony za chwilę.
 */
 
 if ($db) //sprawdzam czy zostaliśmy połączeni z bazą
 { //Jeżeli tak wykonujemy poniższe czynności
 mysql_select_db($baza); //ustawiam nazwę bazy danych
   echo 'test1';
 $result = mysql_query("SELECT * FROM $baza_zrodlo WHERE nazwapliku='$plik'"); //sprawdzam czy w tabeli $baza_zrodlo istnieje nazwa naszego pliku
 $num = mysql_num_rows($result); //sprawdzam ile razy występił wpis o nazwie pliku w $plik
  echo $num.'0num';
 
 if($num >= 1) //jeżeli występuje jeden plik, lub więcej o tej nazwie to...
 { 
 echo 'test2';
 $ile = IleRazyPobrano($plik, $baza, $baza_zrodlo); //...poberam do zmiennej $ile ilość pobrań danego pliku <- w tym miejscy korzystam ze skryptu "pobrano.php"
 $nowy = $ile+1; // dodaję jedno pobranie
 echo $ile;
 
 
 //Tutaj rozpoczyna się już właściwie pobieranie pliku
 
header('Pragma: no-cache'); //ustawiamy: aby zawartość nie była cache'owana,
 header('Content-Transfer-Encoding: binary'); //zawartość wysyłanego pliku jest binarna,
 header('Content-Type: application/x-unknown'); //aplikacja odpowiedzialna za plik,
 header("Content-Disposition: attachment; filename=$plik"); //nazwa pliku,
 header("Location: "."/".$folder_z_plikami.$plik); //zródło pliku.
  echo $folder_z_plikami.$plik;
 
 
 //koniec części odpowieadającej za pobieranie
 mysql_query("update $baza_zrodlo set iloscpobran=$nowy where nazwapliku='$plik'"); //wysyłam do bazy prośbę o uaktualnienie wpisu ilości pobrań
 }
 else // jeżeli nie znaleziono pliku w tabeli o tej nazwie to...
 {
 mysql_query("insert into $baza_zrodlo values ('$plik', 1)"); //wysyłamy prośbę o dodanie pliku z wartością 1 ściągnięcia
 //Poniższe wysyłanie jest identyczne jak w przypadku powyższym
 
  echo $folder_z_plikami.$plik;
  echo 'test3';
 
 //koniec wysyłania
 }
 mysql_close($db); //kończę połącznie z bazą danych
 }
 else
 {
 print "Błąd: Nie mogę połączyć się z bazą danych"; //jeżeli nie można połączyć się z bazą mySQL wyświetl komunikat o błędzie
 }
 }
 }
 else
 {
 print "Błąd wywołania skryptu!!!"; //jeżeli zostały wysłane złe dane do skryptu wyświetl błąd
 }
 ?>
[PHP] pobierz, plaintext 

W tym pliku nie wyświetla się "test2", tak jakby tam się zacinał czy coś. No i nie ma co mówić o wywołaniu pliku do pobrania, bo nic się nie dzieje po prostu.

I plik pobrano.php

[PHP] pobierz, plaintext 
 <?php
 function IleRazyPobrano($plik, $baza, $baza_zrodlo)
 {
 mysql_select_db($baza); //wybieram bazę
 $ilerazy = mysql_query("SELECT iloscpobran FROM $baza_zrodlo WHERE nazwapliku = '$plik'"); //wysyłam zapytanie do bazy o liczbę ściągnięć
 $wynik = mysql_fetch_row($ilerazy); //pobieram wynik w komórki, którą otrzymałem w zmiennej $ilerazy
 return $wynik[0]; //zwracam pierwszy wynik ponieważ zmienna $wynik jest typu array
 }
 ?>
[PHP] pobierz, plaintext 

Ten błąd oznacza, że najprawdopodobniej mysql_query zwraca false ze względu na błąd w zapytaniu. Po zapytaniu wstaw linijkę:



aby zobaczyć jaki błąd zwraca mysql. Ja bym stawiał właśnie na jakąś niezgodność nazwy tabeli lub kolumny z tym, co masz w bazie.

Zmień instrukcje warunkową dla sprawdzania ilości wystąpień na coś prostszego:

[PHP] pobierz, plaintext 
if($num != 0)
[PHP] pobierz, plaintext 

Innym problemem może być małe przeoczenie, po prostu nie podałeś argumentu w linku.

index.php?plik=nazwapliku

Pomijając ten fakt z problemem, skrypt będzie bardzo podatny na ataki typy SQL Injection, poczytaj o funkcji mysql_real_escape_string() lub o wyrażeniach regularnych.

a jakiej funkcji uzywac w PDO zamiast mysql_real_escape_string ?

Poczytaj o PDO-Prepared-Statement.

czyli to jest tam tak jakby automatycznie robione?

No mniej więcej, mam nadzieję, że ktoś korzystający z PDO się wypowie bo na co dzień nie korzystam z tego

PDO wykonuje to automatycznie pod warunkiem, że korzystasz odpowiednio - tzn. nie sklejasz "ręcznie" zapytania, tylko używasz tak, jak w podanym przez Magic WWW przykładzie.

dzieki za odpowiedz panowie

Spróbowałem obydwu rad(barcisza i MagicWWW), ale żadna nie dała rezultatu. Nie wywyala żadnych errorów, a po zmianie porównywania wartości dalej nie dochodzi do "echo 'test2'".

Wrzucam jeszcze eksport z MySQL, może coś pomoże.

[SQL] pobierz, plaintext 
--
-- Struktura tabeli dla  `download`
--
 
CREATE TABLE IF NOT EXISTS `download` (
  `nazwapliku` varchar(30) collate utf8_unicode_ci NOT NULL,
  `iloscpobran` int(11) NOT NULL
) ENGINE=MyISAM DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci;
 
--
-- Zrzut danych tabeli `download`
--
 
INSERT INTO `download` (`nazwapliku`, `iloscpobran`) VALUES
('plik1', 5);
[SQL] pobierz, plaintext 

Może mógłby ktoś tak na szybko sprawdzić kod tych plików, może tam wdarł się jakiś błąd?

Moim zdaniem ten cały skrypt jest do przerobienia, za moment jak wróce z biura to Ci to zrobie do porządku.

[PHP] pobierz, plaintext 
$result = mysql_query("SELECT * FROM $baza_zrodlo WHERE nazwapliku='$plik'")or die(mysql_error()) ;
[PHP] pobierz, plaintext 

jak dodasz wyświetlanie błędów do zapytania będziesz wiedzieć więcej

@kadlub
Odzyskałem nadzieję, ze to zadziała

Wywala mi "No database selected", chociaż na początku skryptu jest wybieranie bazy.

[PHP] pobierz, plaintext 
if ($db) //sprawdzam czy zostaliśmy połączeni z bazą
 { //Jeżeli tak wykonujemy poniższe czynności
 mysql_select_db($baza); //ustawiam nazwę bazy danych
   echo 'test1';
     $result = mysql_query("SELECT * FROM $baza_zrodlo WHERE nazwapliku='$plik'") or die(mysql_error()) ; 
[PHP] pobierz, plaintext 

Jak wchodzę do phpMyAdmin to mam:
* information_schema (17)
* nazwabazy (77)
Więc chyba nie ma problemu z wyborem? Musi być coś z łączeniem, ale gdyby nie łączyło to by wywalił błąd(sprawdzałem ). Sprawdzałem też, czy nazwa jest poprawna i jest.

może złą nazw podałeś sprawdź to dokładnie
daj też wyświetlanie błędów do połączenia
a w zapytaniu niżej daj an sztywno wybór tabeli

[PHP] pobierz, plaintext 
 $result = mysql_query("SELECT * FROM download WHERE nazwapliku='$plik'") or die(mysql_error()) ; 
[PHP] pobierz, plaintext 

Panowie, aż wstyd się przyznać, ale nie ustawiłem uprawnień dla użytkownika bazy danych
Sorry wszystkim, że zmarnowaliście tyle czasu :/ Tak mi głupio, bo zmarnowałem tyle swojego i waszego czasu, ehh...

Teraz zostaje tylko:
Warning: Cannot modify header information - headers already sent by (output started at /home/tibiacor/public_html/forum/downloads/download.php:1) in /home/xxxxxxx/public_html/forum/downloads/download.php on line 55

Warning: Cannot modify header information - headers already sent by (output started at /home/tibiacor/public_html/forum/downloads/download.php:1) in /home/xxxxxxx/public_html/forum/downloads/download.php on line 56

Warning: Cannot modify header information - headers already sent by (output started at /home/tibiacor/public_html/forum/downloads/download.php:1) in /home/xxxxxxx/public_html/forum/downloads/download.php on line 57

Warning: Cannot modify header information - headers already sent by (output started at /home/tibiacor/public_html/forum/downloads/download.php:1) in /home/xxxxxxx/public_html/forum/downloads/download.php on line 58

Warning: Cannot modify header information - headers already sent by (output started at /home/tibiacor/public_html/forum/downloads/download.php:1) in /home/xxxxxxx/public_html/forum/downloads/download.php on line 59

I nie uruchamianie pobierania pliku.
Wstawiam jeszcze raz kod pliku download.php:

[PHP] pobierz, plaintext 
 <?php
 
  //...
 $plik = $_GET['plik']; //pobiera nazwę pliku, który użytkownik chce pobrać
 
 $baza_ip = 'localhost'; //adres bazy danych (najlepiej aby był to localhost)
 $baza_uzytkownik = 'xxxxxxx'; //nazwa użytkownika bazy
 $baza_haslo = 'xxxxx'; //hasło dla danego użytkownika bazy
 $baza = 'xxxxx'; //nazwa bazy glównej
 $baza_zrodlo = 'xxxxx'; //tabela, w której są zapisywane wpisy o ilości pobrań
 
 $folder_z_plikami = 'xxx/'; //tutaj będą znajdować się pliki do pobrania
 //...
  echo 'test0';
 //....
 
  function IleRazyPobrano($plik, $baza, $baza_zrodlo)
 {
 mysql_select_db($baza); //wybieram bazę
 $ilerazy = mysql_query("SELECT iloscpobran FROM $baza_zrodlo WHERE nazwapliku = '$plik'"); //wysyłam zapytanie do bazy o liczbę ściągnięć
 $wynik = mysql_fetch_row($ilerazy); //pobieram wynik w komórki, którą otrzymałem w zmiennej $ilerazy
 return $wynik[0]; //zwracam pierwszy wynik ponieważ zmienna $wynik jest typu array
 }
 
 if ($plik != '') //sprawdzam czy zmienna plik jest pusta, jeżeli jest pusta nie wywołuję dalej kodu tylko informuję o błędzie.
 {
 if (!file_exists($folder_z_plikami.$plik)) //sprawdzam czy podany plik istnieje na serwerze
 {
 print "Podany plik: $plik nie istnieje"; //jeżeli plik nie istnieje to nie wykonuję dalszych operacji tylko wyświetlam informację o błedzie
 }
 else
 {
  echo 'test01';
 $db = mysql_connect($baza_ip, $baza_uzytkownik, $baza_haslo);
 /*
 Powyższym kodem łączę się z bazą danych mySQL korzystając ze stałych zdefiniowanych na początku skryptu. Znak małpy przed zmienną $db konieczny jest ze względu na możliwość wystąpienia błędu, który jeżeli i tak zaistnieje to i tak zostanie znaleziony za chwilę.
 */
 
 if ($db) //sprawdzam czy zostaliśmy połączeni z bazą
 { //Jeżeli tak wykonujemy poniższe czynności
 mysql_select_db($baza); //ustawiam nazwę bazy danych
   echo 'test1';
     $result = mysql_query("SELECT * FROM $baza_zrodlo WHERE nazwapliku='$plik'") or die(mysql_error()) ;  //sprawdzam czy w tabeli $baza_zrodlo istnieje nazwa naszego pliku
 $num = mysql_num_rows($result); //sprawdzam ile razy występił wpis o nazwie pliku w $plik
  echo $num.'0num';
 
 if($num != 0)//jeżeli występuje jeden plik, lub więcej o tej nazwie to...    1.
     { 
 echo 'test2'; echo mysql_error();
 $ile = IleRazyPobrano($plik, $baza, $baza_zrodlo); //...poberam do zmiennej $ile ilość pobrań danego pliku <- w tym miejscy korzystam ze skryptu "pobrano.php"
 $nowy = $ile+1; // dodaję jedno pobranie
 echo $ile; 
 //Tutaj rozpoczyna się już właściwie pobieranie pliku
 
header('Pragma: no-cache'); //ustawiamy: aby zawartość nie była cache'owana,
 header('Content-Transfer-Encoding: binary'); //zawartość wysyłanego pliku jest binarna,
 header('Content-Type: application/x-unknown'); //aplikacja odpowiedzialna za plik,
 header("Content-Disposition: attachment; filename=$plik"); //nazwa pliku,
 header("Location: "."/".$folder_z_plikami.$plik); //zródło pliku.
  echo $folder_z_plikami.$plik;
 
 
 //koniec części odpowieadającej za pobieranie
 mysql_query("update $baza_zrodlo set iloscpobran=$nowy where nazwapliku='$plik'"); //wysyłam do bazy prośbę o uaktualnienie wpisu ilości pobrań
 }
 else // jeżeli nie znaleziono pliku w tabeli o tej nazwie to...
 {
 mysql_query("insert into $baza_zrodlo values ('$plik', 1)"); //wysyłamy prośbę o dodanie pliku z wartością 1 ściągnięcia
 //Poniższe wysyłanie jest identyczne jak w przypadku powyższym
 
  echo $folder_z_plikami.$plik;
  echo 'test3';
 
 //koniec wysyłania
 }
 mysql_close($db); //kończę połącznie z bazą danych
 }
 else
 {
 print "Błąd: Nie mogę połączyć się z bazą danych"; //jeżeli nie można połączyć się z bazą mySQL wyświetl komunikat o błędzie
 }
 }
 }
 else
 {
 print "Błąd wywołania skryptu!!!"; //jeżeli zostały wysłane złe dane do skryptu wyświetl błąd
 }
 ?>
[PHP] pobierz, plaintext 

nagłówki na ogół przesyła sie na samym początku pliku

Sprawdziłem w configu i mam ustawione output_buffering na no value. Czy to ma z tym związek?

Tak mi się wydaje, z tego co tu wyczytałem:
http://phpedia.pl/wiki/Cannot_add_header_i...rs_already_sent

Może jest jakiś prostszy sposób na wymuszanie pobrania tego pliku?

Chodzi o to, że masz wywołanie funkcji header po echo.

Funkcje header muszą być wykonane jeszcze zanim cokolwiek wypluwasz na ekran.

Nic się nie zmieniło, wywaliłem wszystkie echo. Przesuwałem headery na początek, zmienia się tylko numer linii w errorze.

Chodzi o pierwszą linię pliku. Nie masz tam czasem jakiejś spacji przed <?php ?

Była spacja, ale wywaliłem, sprawdziłem i nadal jest to samo :/. Na końcu też sprawdziłem i wywaliłem

Niemożliwe, żeby po wywaleniu tej spacji i po przesunięciu header na samą górę to się nadal pojawia A wklej aktualny kod.

pisałem że nagłówki powinny być na samym początku zaraz po <?php
temat na tym forum poruszany kilkanaście razy

Nie rozumiem, jak ale teraz działa, choć wcześniej nie dzałało No i znowu mi głupio Barcisz miałeś rację

Teraz już z podłączeniem licznika do kodu html powinienem sobie poradzić Dzięki wielkie wszystkim.

Oto przerobiony przeze mnie skrypt, było w nim cholernie dużo błędów

[PHP] pobierz, plaintext 
<?php
	// Pobieranie nazwy pliku z linku 
	$file = $_GET['plik'];
 
	// Sprawdzanie poprawności nazwy pliku
	if(!preg_match('/^[a-z0-9 ]+$/i', $file))
	{
		exit('Script error, the file name is invalid.');
	}
 
	// Konfiguracja bazy danych
	$mysql['Server'] = 'localhost';
	$mysql['Username'] = 'root';
	$mysql['Password'] = '';
	$mysql['Database'] = 'nazwa_bazy_danych';
	$mysql['Table'] = 'Tabela';
 
	// Tworzenie ścieżki do folderu z plikami
	$file_dir = 'x/';
 
	// Sprawdzanie czy nazwa pliku została podana
	if(!empty($file))
	{
		// Sprawdzanie istnienia pliku
		if(!file_exists($file_dir.$file))
		{
			exit("Plik {$file} nie istnieje.");
		}
		else
		{
			// Nawiązanie połączenia z bazą danych
			$db = mysql_connect($mysql['Server'], $mysql['Username'], $mysql['Password']);
 
			// Sprawdzanie czy połączenie powiodło się
			if(mysql_errno() != 0)
			{
				exit(mysql_errno());
			}
 
			// Wybiernaie bazy danych
			mysql_select_db($mysql['Database']);
 
			// Zapytanie do bazy danych
			$result = mysql_query("SELECT * FROM `{$mysql['Table']}` WHERE `nazwapliku`='{$file}'", $db);
 
			// Wyciąganie rekordu oraz sprawdzanie czy istnieje
			if(($row = mysql_fetch_assoc($result)) != 0)
			{
				// Dodawanie liczby pobrań
				mysql_query("UPDATE `{$mysql['Table']}` SET `iloscpobran`=`iloscpobran` + 1 WHERE `nazwapliku`='{$file}'", $db);
 
				// Wysyłanie nagłówków
				header('Pragma: no-cache');
				header('Content-Transfer-Encoding: binary');
				header('Content-Type: application/x-unknown');
				header("Content-Disposition: attachment; filename={$file}");
				header('Location: '.'/'.$file_dir.$file);
 
				// Wyświetlenie ścieżki i nazwy pliku
				echo $file_dir.$file;
			}
			else
			{
				// Dodawanie pliku do bazy
				mysql_query("INSERT INTO `{$mysql['Table']}` (`nazwapliku`,`iloscpobran`) VALUES ('{$file}', 1)");
 
				// Wyświetlenie ścieżki i nazwy pliku
				echo $file_dir.$file;
			}
 
			// Zamknięcie połączenia z bazą danych
			mysql_close($db);
		}
	}
	else
	{
		exit('Script error, you did not specify a file name.');
	}
 
 ?>
[PHP] pobierz, plaintext