prosze o przeanalizowanie kodu :) Opcje

[rozny]

Witam wszystkich (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Bardzo prosze o przeanalizowanie tego kodu pod wzgledem poprawnosci skladni itp.

Kod dziala tak jak to sobie zalozylem wiec bledow zadnych typu parsy, czy inne errory nie ma, jednak zalezy mi na optymalizacji tego kodu, a niestety doswiadczenie mam niewielkie i podejrzewam ze jest to raczej przerosniety smok (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
<?php
 
		session_start();
 
		if (!session_is_registered('logged_in')) {
		echo '<p class=top>Admin Panel</p><form action=' .$PHP_SELF. ' method=post class=login>
		<input type=text name=login> LOGIN<br />
		<input type=password name=passwd> HASŁO<br />
		<input type=submit value=zaloguj>
		</form>';
		}
 
		if (isset($_POST['login'])&&isset($_POST['passwd'])) {
 
		$connect = mysql_connect('localhost', '*', '*')
 
		or die('Brak polaczenia z serwerem mySQL.<br />Błąd: '.mysql_error());
 
		echo ('Polaczenie z serwerem jest aktywne.<br/>');
 
		$db = mysql_select_db('users', $connect)
 
		or die('Brak polaczenia z baza danych<br />Blad: '.mysql_error());
 
		echo ('Polaczenie z baza danych jest aktywe.<br />');
 
 
		$sql = "SELECT * FROM admini WHERE login='".$_POST['login']."' LIMIT 0,30";
 
 
		$query=mysql_query($sql) or die ("Bład: ".mysql_error());
 
		$fetch = mysql_fetch_array($query);
 
		if ($fetch) {
			if ($_POST['passwd'] == $fetch['password']) {
 
				session_register('logged_in');
				header('Location: index.php');
 
			}
			else {
 
				echo 'bledne haslo';
 
			}
		}
 
		else {
 
			echo 'bledny login';
 
		}
 
 
		mysql_close();
 
		}
 
		if (isset($_GET['id'])) {
			if ($_GET['id']==logout) {
 
			session_destroy();
			session_write_close();
			header('Location: index.php');
			}
		}
 
		if (session_is_registered('logged_in')) {
		require_once('cfg/menu.cfg');
		}
		?>
[PHP] pobierz, plaintext 

pozdrawiam (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[acztery]

mi sie nie podoba troszke to pokręcone

np

$sql = "SELECT * FROM admini WHERE login='".$_POST['login']."' LIMIT 0,30";

co to ma byc tzn ze jak bede mial tzn ze moga byc 2 takie same nicki ... bez sens w twoim przypadku bedzie ok 35 i klapa

nie lepiej takk

[PHP] pobierz, plaintext 
<?php
$sql = "SELECT * FROM admini WHERE login='".$_POST['login']."' and password='".$_POST['password']."' LIMIT 1 ";
?>
[PHP] pobierz, plaintext 

Ten post edytował acztery 11.12.2006, 16:09:51