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)

[rozny]

to jak to zalatwic? zamiast * w SELECT wrzucic SELECT passwd?

Puki co admina mam tylko jednego, ale wiadomo w przyszlosci sie to zmieni. W takim razie jak to poprawic? Zmienic ta piekielna * w zapytaniu sql o haslo?

A co do projektowania obiektowego, to zdaje sobie sprawe, ze obok zorientowanych obiektowo aplikacji to to nawet nie stalo. Niestety kiedys dawno dawno nauczylem sie troche podstaw php i po dosc niedlugim czasie zarzucilem dalsza nauke, gdyz wciagnelem sie w zgola inny temat, jezeli chodzi o kompy... dzisiaj wracam do programowania w php poniekad z przymusu i zaczynam ze starymi (wiem - niekoniecznie dobrymi) przyzwyczajeniami :/ dlatego ten kod tak nieciekawie raczej wyglada (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

prosze o jeszcze jakies sugestie, ew. co moge zrobic, zeby ten kod zamienic w obiektowy (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)


EDIT: zrobilem drugiego admina i jest roznica, jakie haslo sie wpisze, wiec pod tym wzgledem jest ok.

Ten post edytował rozny 11.12.2006, 14:27:35