Forum PHP.pl

Mam problem z licznikiem użytkowników zalogowanych. Zrobiłem jak poniżej ale nic nie wyświetla. Jak usunę isset($_POST['login']) i isset($_POST['wyloguj']) to liczy każde kliknięcie gdy jest zalogowany lub odejmuje gdy jest wylogowany. Ma ktoś pomysł jak to zmienić lub inną metodą zrobić?

Jak widać chodziło mi tu o to że jak ktoś kliknie zaloguj i będzie zalogowany to dodaje 1, a jak kliknie wyloguj i poprawnie go wyloguje to odejmie 1. A może lepiej zrobić to na bazie danych? Bo później chciałbym też zrobić wyświetlenie zalogowanego użytkownika.

[PHP] pobierz, plaintext 
<?php
 
	$file = "licznik.txt";
 
	// Pobranie zawartości pliku
	$file_get = file_get_contents($file);
 
	// Jeśli jesteś zalogowany
	// to dodaje do pliku (n+1)
	if(isset($_SESSION['id']) && isset($_POST['login'])){
	$online = $file_get + 1;
	file_put_contents($file, $online);
	}
 
	// Jeśli właśnie się wylogowujesz
	// to odejmuje z pliku (n-1)
	if(!isset($_SESSION['id']) && isset($_POST['wyloguj'])){
	$online = ($file_get>0?--$file_get:0);
	file_put_contents($file, $online);
	}
 
	// Wyświetlenie użytkowników online
	echo $online;
 
?>
[PHP] pobierz, plaintext 

Ten post edytował 18Pawel18 27.08.2013, 16:10:21

Czemu wy nie patrzycie na warunki jakie dajecie??
Skoro robisz dopiero loguj to jeszcze nie ma ID w sesji.

Skoro robisz wyloguj, to ID w sesji juz jest wiec nie mozesz dawac warunki na nie bycie...


Znaczy, ze ty ten kod odpalasz nie podczas odpalania akcji loguj/wyloguj. Skoro dajesz warunki na post, to form ma byc wyslany

No ok, masz rację, ale jak tak zrobię to każde kliknięcie zaloguj będzie liczone. A co jeśli użytkownik poda złe dane i nie zostanie zalogowany? Wtedy też policzy, że jest online.

Ten post edytował 18Pawel18 27.08.2013, 17:34:39

No to zliczaj tylko wtedy gdy poda poprawne dane. Przeciez na etapie logowania wiesz czy podal poprawne dane czy nie.

Próbowałem z tego coś zrobić ale nie dałem rady.

Wymyśliłem rozwiązanie takie:

Jeśli uzytkownik zaloguje się to zmieni rekord 'zalogowani' na 1, a jeśli niezalogowany to na 0. Pobierze cyfry wtedy i doda je. Oddzielny skrypt będzie obsługiwał automatyczne wylogowanie uzytkownika przez Cron-a.

W bazie mam taką tabelę:

[SQL] pobierz, plaintext 
CREATE TABLE IF NOT EXISTS `uzytkownicy` (
  `id` int(10) UNSIGNED NOT NULL AUTO_INCREMENT,
  `imie` varchar(20) NOT NULL,
  `nazwisko` varchar(20) NOT NULL,
  `login` varchar(20) NOT NULL,
  `haslo` varchar(32) NOT NULL,
  `email` varchar(70) NOT NULL,
  `aktywny` tinyint(1) NOT NULL,
  `zalogowany` tinyint(1) NOT NULL,
  `admin` tinyint(1) NOT NULL DEFAULT '0',
  `utworzone` timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP,
  PRIMARY KEY (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 AUTO_INCREMENT=14 ;
 
[SQL] pobierz, plaintext 

Do logowania uzywam takiej funkcji:

[PHP] pobierz, plaintext 
 
function zaloguj($login, $haslo){
 
 
    include('config.php');
 
    //nawiązanie połączenia z bazą danych
    $baza = new PDO('mysql:host='.$host.';dbname='.$baza,$dbUzytkownik, $dbHaslo);
    $baza -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
    $baza->query('set names utf8');
 
    $sql = "select id, login, admin, imie, nazwisko from uzytkownicy where (login like :uzytkownik or email like :mail) and haslo like :haslo and aktywny = 1";
 
    $logowanieStmt = $baza->prepare($sql);
 
    $logowanieStmt -> bindValue(':uzytkownik', $login, PDO::PARAM_STR);
    $logowanieStmt -> bindValue(':mail', $login, PDO::PARAM_STR);
    $logowanieStmt -> bindValue(':haslo', md5($haslo), PDO::PARAM_STR);
 
    $logowanieStmt->execute();
 
    $uzytkownik = $logowanieStmt->fetch();
 
 
    if($logowanieStmt->rowCount() > 0){
      $_SESSION['uzytkownik'] = $uzytkownik['login'];
      $_SESSION['id'] = $uzytkownik['id'];
	   $_SESSION['imie'] = $uzytkownik['imie'];
	   $_SESSION['nazwisko'] = $uzytkownik['nazwisko'];
      if($uzytkownik['admin']) $_SESSION['admin'] = 1; 
      return 'ok';
    } 
 
    else{
        //sprawdzenie czy uzytkownik aktywował swoje konto
        $count = $baza->query('select aktywny from uzytkownicy where login like \''.$login.'\'');
        if($count->rowCount() > 0){  
        $count = $count->fetchAll(PDO::FETCH_ASSOC);             
        if($count[0]['aktywny'] ==  0) {
            $_SESSION['bladLog'] = "Twoje konto nie jest jeszcze aktywne, aktywuj konto aby móc się zalogować";
            return 'rejestracja';
            }
			else{
			$_SESSION['bladLog'] = 'Nieprawidłowa nazwa użytkownika bądź hasło';
            return 'err';
			}
        }
        else {
            $_SESSION['bladLog'] = 'Nieprawidłowa nazwa użytkownika bądź hasło';
            return 'err';
        }
 
    }
}
 
[PHP] pobierz, plaintext 

Plik główny z logowaniem

[PHP] pobierz, plaintext 
<?php if(!isset($_SESSION['id'])) : ?>  
 
// Panel logowania
 
 <?php else: ?>
 
// Jeśli poprawnie zalogowany
 
 <?php endif; ?>
[PHP] pobierz, plaintext 

Pomoże mi to ktoś napisać?