[php] Poprawność & styl Opcje

[falfin]

W php pisze od ponad 3 miesięcy-jestem więc jeszcze totalnym zółtodziobem. Często mam wrażenie że różne rzeczy robie nie tak jak powinnienem. Dlatego chciałbym was zapytać czy poniższe skrypty są poprawne (pod względem stylu pisania i zastosowanych rozwiązań). Z góry dzięki za każda podpowiedź.

Skrypt do wyświetlania news'ów na stronie głównej.
(np. czy nie powinny one być wyświetlane w tabelkach zamiast w divie?)

[PHP] pobierz, plaintext 
<?php	
		require_once('funkcje.php');
			$wynik= wykonaj_w_bazie("select id, naglowek, tekst , utworzony from artykuly where strona = 'NewsP' order by utworzony desc "); 
				if ($wynik == false)
				{
					echo " Wystąpił błąd bazy danych.";
					exit;
				}	
				else
				{	
					$ilosc = mysql_num_rows ($wynik);					
					for ($i=0; $i <$ilosc; $i++)
					{
						$wiersz = mysql_fetch_row($wynik);
						$utworzony=getdate ($wiersz[3]);
						echo '<div class="news">';
						echo '<a href=szablon.php'.'?'.name.'='.$wiersz[0].'><p class="news_tytul">'.$wiersz[1].'</p></a>';
						echo '<p class="data">'.$utworzony['mday'].'.'.$utworzony['mon'].'.'.$utworzony['year'].' , '
							.$utworzony['hours'].':'.$utworzony['minutes'].':'.$utworzony['seconds'].'</p>';	
						echo '<p class="news_tresc">'.substr($wiersz[2], 0, 300).'...';
						echo '<a href=szablon.php'.'?'.name.'='.$wiersz[0].'>['.więcej.']</a></p>';		
						echo '</div>';
					}	
				}
		?>
[PHP] pobierz, plaintext 

Do sprawdzania czy użytkownik jest zalogowany na każdej podstronie używam:
(np. czy jest to bezpieczny sposob? czy łatwo to obejść)

[PHP] pobierz, plaintext 
<?php
				//SPRAWDZENIE	CZY	UZYTKOWNIK	 JEST	Zalogowany										
		if (!isset($_SESSION['prawid_uzyt']))
			{
			@header("Location: logowanie.html");		
			}
?>
[PHP] pobierz, plaintext 

I gwóźdź programu (IMG:http://forum.php.pl/style_emoticons/default/dry.gif) , wiem że to na 99% zły sposób, ale nie bardzo wiem jak to inaczej rozwiązać
(chdzi o to ze jeżeli użytkownik posiada uprawnienia administratora to w menu ma mieć dostęp do zakładki użytkownicy)

[PHP] pobierz, plaintext 
<ul id="menu">
					<li><a href="index.php">index</a></li>
					<li><a href="artykuly.php">artykuły</a> </li>
					<?php
						if ($_SESSION['uprawnienia'] !== 'administrator')
							{
							echo '<li><a href="profil.php">moje konto</a></li>';
							}
						else
							{
							echo '<li><a href="uzytkownicy.php">użytkownicy</a></li>';
							}
					?>
					<li><a href="dodatki.php">dodatki</a></li>
					<li><a href="strona_testowa.php">podgląd</a></li>
				</ul>
[PHP] pobierz, plaintext 

To by było na razie na tyle. Pozdrawiam