Forum PHP.pl

Formularz, przekierowuje mnie do:

[PHP] pobierz, plaintext 
<?php
session_start();
require_once("connect.php");
require_once("functions.php");
 
 
if(!logon()) {
	session_destroy();
	header('Location: <a href=\"http://localhost/');&#092;" target=\"_blank\">http://localhost/');</a>
	exit();
}
 
$userid = $_SESSION['id'];
 
//i tu dalej odczyt właściwa strona itp.
?>
[PHP] pobierz, plaintext 

functions.php wygląda tak:

[PHP] pobierz, plaintext 
<?php
function logon() {
 
//czas sesji w sekundach
$intTimeoutSeconds = 300;
 
if(isset($_SESSION['intLastRefreshTime'])){if(($_SESSION['intLastRefreshTime']+$intTimeoutSeconds)<time()){session_destroy(); header('location: <a href=\"http://localhost/');}}&#092;" target=\"_blank\">http://localhost/');}}</a>
$_SESSION['intLastRefreshTime'] = time();
 
 
if(isset($_POST['user']) && isset($_POST['pass'])) {
	$login = htmlspecialchars($_POST['user']);
	$haslo = md5(htmlspecialchars($_POST['pass']));
 
 
$sprawdzlogin = mysql_query("SELECT id, login, haslo FROM userdata WHERE haslo = '$haslo' AND login = '$login' LIMIT 1");
$wyniksprawdz = mysql_num_rows($sprawdzlogin);
 
if($wyniksprawdz == 0) {
$error=1; 
		echo 'Niepoprawne Login lub Hasło'; 
		exit;
 
	}
}
 
 
 if(!isset($_SESSION['logon']) || !isset($_SESSION['login']) || !isset($_SESSION['password']) || !isset($_SESSION['ip'])) {
$error=1; 
exit;
	}
 
	$login = $_SESSION['login'];
	$password = $_SESSION['password'];
	$ip = $_SESSION['ip'];
	if($_SERVER['REMOTE_ADDR'] != $ip) {
 
exit;
	}
 
	if ($error==0) { 
		$userid=mysql_fetch_assoc($sprawdzlogin);
		$_SESSION['id']=$userid['id'];
		$_SESSION['logon'] = 1;
		$_SESSION['login'] = $_POST['user'];
		$_SESSION['password'] = $_POST['pass'];
		$_SESSION['ip'] = $_SERVER['REMOTE_ADDR'];
 
	}
 
 if($_SESSION['logon'] == "1") {		
	return 1;
	} 
	else {
	$error=1; 
exit;
	}
 
}
?>
[PHP] pobierz, plaintext 

Jednak przy logowaniu skrypt wykonuje się bardzo długo, tak jakby się zapętlał... I zabardzo nie wiem dlaczego. W ogóle taki sposób na logowanie jest dobry?

Wyprofiluj kod za pomocą microtime(true) (chodzi mi o porównanie timestampów z odpowiednich fragmentów kodu aby zobaczyć co najbardziej obciąza). Sposób logowania nie jest zły, ja bym obstawiał że najdłużej zajmuje połączenie z bazą. Wyprofiluj go i zobacz co tyle czasu zajmuje.

A i jeszcze jedno pytanie, można przyspieszyć logowanie, tak jak wyszukiwanie za pomocą indeksowania?

ehh, no chyba nie, jak nie masz nabazgrane w kodzie to wszystko ładnie idzie, a u ciebie nie widzę niepotrzebnych linijek

Ok, za bardzo nie wiem co miał na myśli carbolymer pisząc o wyprofilowaniu danych, wiem jak obliczyć czas wczytywania strony itp. ale wydaje mi się iż też jest to spowodowane łączeniem się z bazą... hmm to może samą bazę mam źle zbudowaną? Obecnie jest tak:

tabela userdata:



a może lepiej oddzielić i zrobić:

tabela userdata:



oraz tabele reszta:



czy to przyniesie lepszy efekt? Tylko skoro jest:



To nic poza tym nie powinno być przeszukiwane.

EDIT:

heh, zbudowanie bazy tak jak opsiałem jeszcze bardziej spowolania wykonywanie skryptu... Jak można to inaczej zbudować żeby nie było długiego wczytywania przy logowaniu?