Оценка кода скрипта...

Печать RSS
330

М

Малиновые штаны
0
iNeeXT (28 Апреля 2013 / 13:02)
3. Используй функцию mysql_result вместо mysql_num_rows.
Почему?
I

Пришелец
0
Flyd (28 Апреля 2013 / 13:19)
Почему?
Быстрее как-бы.
Ю

deleted
0
14, Никогда вот не понимал этой "мании" скорости выполнения, когда речь идет о ничтожных долях секунды. Никто разницы и не заметит, если скрипт будет выполняться на 2 мс. медленней. Другое дело, если бы разница была в 1-2 секунды. Это сугубо моё мнение, я никому ничего не навязываю - просто не удержался.
I

Пришелец
0
15. DanteS, я же не говорил о простых операциях, верно?
Автор
Чатланин
0
<?
	function is_auth(){
		$pass = mysql_fetch_row(mysql_query('SELECT `pass` FROM `admin`'));
		if(!isset($_SESSION['auth']) || $_SESSION['auth']!=$pass[0]){ 
			header('Location: '.MYSITE); 
			}}
?> 
Написал такую функцию для проверки авторизации. Функция работает.
Вопрос: Нужно ли еще какие нибудь проверки сделать?
Изменил: Абдула (28.04.2013 / 16:35)
Ю

deleted
0
17, я бы сначала проверял наличие COOKIES, и только потом (если они есть) проверял бы их достоверность, сверяя с данными в базе. Это чтобы не теребить базу лишний раз.
Изменил: юЮЮфюв (28.04.2013 / 16:54)
Автор
Чатланин
0
18. DanteS, Ок. Понял. Спасибо. Учту. Переписал.

<?
	function is_auth(){
		if(isset($_SESSION['auth'])){
			$pass = mysql_fetch_row(mysql_query('SELECT `pass` FROM `admin`'));
			if($_SESSION['auth']!=$pass[0]){ 
				header('Location: '.MYSITE); 
			}
		} else header('Location: '.MYSITE); 
		}
?> 
Изменил: Абдула (28.04.2013 / 17:50)
E

Пришелец
0
Вариативно можно хранить пароль в json файле(естественно что в директории с deny from all). Чтение, а в твоем случае "дергание" пароля лучше переложить на что-то иное от БД. И да, вопрос безопасности хранения 1 к 1 в случае с БД, так как если пользователь получит доступ к файлам, то он также сможет прочитать и конфиги коннекта к БД и прочитать пароль оттуда.
I

Пришелец
0
Если все же хочешь хранить пароли в БД, то делай кэширование.
Стикеры / Теги / Правила / Топ тем / Топ постов / Поиск