Skrypt użytkownicy online Opcje

[parox]

Witam.

Napisałem owy skrypt mający na celu sprawdzanie użytkowników online.

[PHP] pobierz, plaintext 
<?php 
 
include ('engine/config.php');
include ('lang/en/bloks.php');
 
session_start();
 
	$time_now_uo = time();
	echo $time_now_uo . '<br />';
	$old_time_uo = time()-540;
 
	if (isset ($_SESSION['user_id'])) {
		$uidc = $_SESSION['user_id'];
		$sql_un = "SELECT username FROM " . $db_prefix . "users WHERE id='$uidc'";
		$un = mysqli_query($dbc, $sql_un);
		$username = mysqli_fetch_array($un);
		$unl = $username['username'];
 
			if($unl != false) { 
				$sql_uo_u = "UPDATE " . $db_prefix . "users_online SET time='$time_now_uo' WHERE username='$unl'";
				mysqli_query($dbc, $sql_uo_u);
			} else {
				$sql_uo_a = "INSERT INTO  " . $db_prefix . "users_online ('username' ,'time') VALUES ('$unl',  '$time_now_uo')";
				mysqli_query($dbc, $sql_uo_a);
			}
	}
 
	$sql_uo_d = "DELETE FROM " . $db_prefix . "users_online WHERE time < '$old_time_uo'";
	mysqli_query($dbc, $sql_uo_d);
 
	$sql_sun = "SELECT username FROM " . $db_prefix."users_online ORDER BY username";
	$sun = mysqli_query($dbc, $sql_sun);
	if (mysqli_num_rows($sun) != 0) { 
		echo $lang_bloks_bstats_uo . ': ';
		while($user_name = mysqli_fetch_array($sun)) { 
			echo $user_name['username'] . ', '; 
		}
	} else {
		echo $lang_bloks_bstats_nuo;
	}
 
?> 
[PHP] pobierz, plaintext 

Niestety nie zupełnie działa i nie wiem dlaczego... Czy ktoś mógłby zdefiniować co zrobiłem źle?

Baza:

[SQL] pobierz, plaintext 
CREATE TABLE `frycms`.`fry_users_online` (
`username` VARCHAR( 20 ) CHARACTER SET utf8 COLLATE utf8_unicode_ci NOT NULL ,
`time` INT NOT NULL ,
PRIMARY KEY ( `username` )
) ENGINE = MYISAM CHARACTER SET utf8 COLLATE utf8_unicode_ci;
[SQL] pobierz, plaintext 

Dzięki za pomoc.

[Fifi209]

Nie działa - co to znaczy? Jakiś błąd? Sprawdzałeś czy mysql_error coś wyrzuca? A może błąd w samym kodzie?

[parox]

Hmm. Błędu nie wywala. Po prostu nie wysyła do bazy żadnych zapytań, i nic w bazie się nie dzieje.
mysqli_error także nic nie wybija.

// Edit

A na stronie wyświetla

1265655100
No users online

Czyli
echo $time_now_uo . '<br />'; Linia 9
echo $lang_bloks_bstats_nuo; Linia 39

Ten post edytował parox 8.02.2010, 19:55:58

[Fifi209]

Muszę zapytać, masz w ogóle połączenie z bazą? Masz włączone wyświetlanie błędów?

[PHP] pobierz, plaintext 
error_reporting(E_ALL);
[PHP] pobierz, plaintext 

[parox]

Połączenie z bazą mam. Pierwsze linijki wczytują konfigurację bazy i reszta skryptu jest na tym oparta (Rejestracja, Logowanie etc.) i działa. Domyślnie czy mam, chyba tak. localhost zainstalowany z XAMPP pod Ubuntu. I po dodaniu owej linijki na początek skryptu także nic.

^ Aktualizacja mojego posta wyżej.

[Fifi209]

Wiesz... dałeś tylko część kodu, dlatego pytam o głupie rzeczy. Trudno, aby mysql_error coś zwracał skoro powinieneś użyć mysqli_error - takie moje niedopatrzenie.

[parox]

Użyłem mysqli a nie mysql. Nie jestem aż tak głupi i wiem czego używałem i co sprawdzać ;]
Połączenie z bazą (config.php)

[PHP] pobierz, plaintext 
<?php
 
$system_installation = true;
$db_host = 'localhost';
$db_name = 'frycms';
$db_user = 'root';
$db_pass = '';
$db_prefix = 'fry_';
 
$dbc = @mysqli_connect ($db_host, $db_user, $db_pass, $db_name) 
OR die ('Cannot connect to the database, reason:<br /> ' .mysqli_connect_error() );
 
 
?>
[PHP] pobierz, plaintext 

A w include nr. 2 są tylko zmienne w których są zdania które właśnie są wyświetlane na stronie. Działają zmienne, SQL connection też bo jak dam
echo $uidc;
Powiedzmy w 14 linii to wyświetla mi jako kto jestem zalogowany.

[altruista2]

Błąd tkwi tu:

[PHP] pobierz, plaintext 
		$unl = $username['username'];
 
			if($unl != false) { 
				$sql_uo_u = "UPDATE " . $db_prefix . "users_online SET time='$time_now_uo' WHERE username='$unl'";
				mysqli_query($dbc, $sql_uo_u);
			} else {
				$sql_uo_a = "INSERT INTO  " . $db_prefix . "users_online ('username' ,'time') VALUES ('$unl',  '$time_now_uo')";
				mysqli_query($dbc, $sql_uo_a);
			}
[PHP] pobierz, plaintext 

jeśli $unl to fałsz wywolujesz INSERT username=$unl, czyli username=''

jeśli nazwa użytkownika zapisana jest w sesji podmień kod na:

[PHP] pobierz, plaintext 
$sql_uo_a = "INSERT INTO  " . $db_prefix . "users_online ('username' ,'time') VALUES ('".$_SESSION['username']."',  '$time_now_uo')";
[PHP] pobierz, plaintext 

Ten post edytował altruista2 8.02.2010, 22:13:03

[parox]

Zmodyfikowałem trochę kod. Wygląda on teraz tak:

[PHP] pobierz, plaintext 
<?php
 
include ('engine/config.php');
include ('lang/en/bloks.php');
 
session_start();
 
	$time_now_uo = time();
	echo $time_now_uo . '<br />';
	$old_time_uo = time()-540;
 
	if (isset ($_SESSION['user_id'])) {
 
		$sql_check_un = "SELECT username FROM " . $db_prefix . "users_online WHERE username='" . $_SESSION['username'] . "'";
		$cun = mysqli_query($dbc, $sql_check_un);
		$check_un = mysqli_fetch_array($cun);
 
		echo $_SESSION['username'] . '<br />';
 
			if($check_un['username'] == $_SESSION['username']) { 
				$sql_uo_u = "UPDATE " . $db_prefix . "users_online SET time='$time_now_uo' WHERE username='" . $_SESSION['username'] . "'";
				mysqli_query($dbc, $sql_uo_u);
			} else {
				$sql_uo_a = "INSERT INTO  " . $db_prefix . "users_online ('username' ,'time') VALUES ('" . $_SESSION['username'] . "',  '$time_now_uo')";
				mysqli_query($dbc, $sql_uo_a);
			}
	}
 
	$sql_uo_d = "DELETE FROM " . $db_prefix . "users_online WHERE time < '$old_time_uo'";
	mysqli_query($dbc, $sql_uo_d);
 
	$sql_sun = "SELECT username FROM " . $db_prefix."users_online ORDER BY username";
	$sun = mysqli_query($dbc, $sql_sun);
	if (mysqli_num_rows($sun) != 0) { 
			echo $lang_bloks_bstats_uo . ': ';
		while($user_name = mysqli_fetch_array($sun)) { 
			echo $user_name['username'] . ', '; 
		}
	} else {
		echo $lang_bloks_bstats_nuo;
	}
 
?>
[PHP] pobierz, plaintext 

Ale jednak nic to nie dało.

[altruista2]

dopisz ta końcu skryptu

[PHP] pobierz, plaintext 
 echo '<pre>'; print_r($_SESSION); echo '</pre>';
[PHP] pobierz, plaintext 

i "pochwal" się name co PHP wypluł, bo z tym $_SESSION['username'] to strzelałem że taka zmienna może być zapisywana w sesji.

[parox]

Taką tworzyłem więc taka jest.

Kod

Array
(
    [user_id] => 1
    [username] => parox
    [wba] => 0e2a9165b883b8145bfe5b9abba498c8
)

wba - web browser agent.

Więc nikt nie zna przyczyny braku funkcjonalności?

Jakieś rozwiązanie?

Ten post edytował parox 9.02.2010, 11:07:08

[altruista2]

Sprawdź może w phpmyadminie czy użytkownicy faktycznie się dodają do tabeli users_online?

[parox]

No właśnie w tym rzecz że nie... Wyświetlać by się wyświetlały bo sprawdzałem dodając INSERTem.

[altruista2]

To chyba musisz troszke podebugować

[PHP] pobierz, plaintext 
			if($check_un['username'] == $_SESSION['username']) { 
				$sql_uo_u = "UPDATE " . $db_prefix . "users_online SET time='$time_now_uo' WHERE username='" . $_SESSION['username'] . "'";
				echo $sql_uo_u;
				mysqli_query($dbc, $sql_uo_u);
			} else {
				$sql_uo_a = "INSERT INTO  " . $db_prefix . "users_online ('username' ,'time') VALUES ('" . $_SESSION['username'] . "',  '$time_now_uo')";
				echo $sql_uo_a;
				mysqli_query($dbc, $sql_uo_a);
			}
[PHP] pobierz, plaintext 

Będziesz wiedział gdzie jest błąd

Ten post edytował altruista2 9.02.2010, 19:27:38

[parox]

Kod

1265748780
parox
INSERT INTO fry_users_online ('username' ,'time') VALUES ('parox', '1265748780')No users online

Array
(
    [user_id] => 1
    [username] => parox
    [wba] => 0e2a9165b883b8145bfe5b9abba498c8
)

A kiedy dodaję wpis poprzez phpMyAdmin zapytanie jest takie:

[SQL] pobierz, plaintext 
INSERT INTO `frycms`.`fry_users_online` (`username`, `time`) VALUES ('parox', '1265749209');
[SQL] pobierz, plaintext 

Próbowałem i tak z tym że zamieniłem to co trzeba na zmienne. Dalej nic. A kiedy już dodałem do bazy danych owy wpis, na stronie wygląda tak:

Kod

1265749533
parox
UPDATE fry_users_online SET time='1265749533' WHERE username='parox'
Users online: parox,

Array
(
    [user_id] => 1
    [username] => parox
    [wba] => 0e2a9165b883b8145bfe5b9abba498c8
)

W dodatku wpis w bazie aktualizuje się.

[altruista2]

Wszystko jasne, niby z pozoru banalny błąd - ale trudny do wychwycenia

zamień

[PHP] pobierz, plaintext 
$sql_uo_a = "INSERT INTO  " . $db_prefix . "users_online ('username' ,'time') VALUES ('" . $_SESSION['username'] . "',  '$time_now_uo')";
[PHP] pobierz, plaintext 

na to:

[PHP] pobierz, plaintext 
$sql_uo_a = "INSERT INTO  " . $db_prefix . "users_online (`username` ,`time`) VALUES ('" . $_SESSION['username'] . "',  '$time_now_uo')";
[PHP] pobierz, plaintext 

i będzie działać

Ten post edytował altruista2 9.02.2010, 22:30:31

[parox]

E, nie za bardzo wiem Co zmieniłeś poza tylko 4-ema ' na `.

Działa. Dzięki.

[Fifi209]

Tylko to zmienił, bo tylko to było problemem. Swoją drogą jeżeli nie dajesz nigdzie nazw pól w `` to konsekwentnie rób to do końca skryptu, jeżeli chcesz dawać to rób to wszędzie.

[parox]

A jak ominę i dam np. (username, time) to zaszkodzi?

[altruista2]

Nie. O ile nie masz spacji albo innych specjalnych znaków w nazwie kolumn/tabeli.

Jeśli mogę mieć prośbę do twórców tego forum to bardzo bym prosił zmienić czcionkę kodu php/sql itd. na monospace np. Courier Sans itp. Dziękuję

Ten post edytował altruista2 10.02.2010, 00:24:18