Forum PHP.pl

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

[PHP] pobierz, plaintext 
<?php
$wynik= wykonaj_w_bazie("select id, naglowek, tekst , utworzony from artykuly where strona = 'NewsP' order by utworzony desc ");
?>
[PHP] pobierz, plaintext 

Nie rozumiem czemu newsy trzymasz w artykułach. Podejrzewam, że struktura twojej bazy jest mierna.

[PHP] pobierz, plaintext 
<?php
$wynik= wykonaj_w_bazie("SELECT id, naglowek, tekst, utworzony FROM artykuly WHERE strona = 'NewsP' ORDER BY utworzony DESC");
?>
[PHP] pobierz, plaintext 

Tak nie lepiej?


Nie powinny, tabelki stworzone są do danych tabelarycznych, czyli np. rozpiski meczów czy cokolwiek, a newsy przecież tabelką nie są.

[PHP] pobierz, plaintext 
<?php
if ($wynik == false)
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
if(!$wynik)
?>
[PHP] pobierz, plaintext 

dalej

[PHP] pobierz, plaintext 
<?php
echo " Wystąpił błąd bazy danych.";
					exit;
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
exit(" Wystąpił błąd bazy danych.");
?>
[PHP] pobierz, plaintext 

Następnie:

[PHP] pobierz, plaintext 
<?php
for ($i=0; $i <$ilosc; $i++)
					{
						$wiersz = mysql_fetch_row($wynik);
?>
[PHP] pobierz, plaintext 

Zamień na:

[PHP] pobierz, plaintext 
<?php
while($wiersz = mysql_fetch_array($wynik))
{
?>
[PHP] pobierz, plaintext 

Następnie:

[PHP] pobierz, plaintext 
<?php
$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 

Możesz skorzystać z strftime; poza tym $wiersz[3] jest reprezentowany jako $wiersz['utworzony']; przy metodzie z while, a więc:

[PHP] pobierz, plaintext 
<?php
echo '<div class="news">';
echo '<a href="szablon.php?name='.$wiersz['id'].'"><p class="news_tytul">'.htmlspecialchars(stripslashes($wiersz['naglowek'])).'</p></a>';
echo '<p class="data">'.strftime("%d.%m.%Y, %H:%M:%S", $wiersz['utworzony']).'</p>';
echo '<p class="news_tresc">'.htmlspecialchars(substr(stripslashes($wiersz['tekst']), 0, 300)).'...';
echo '<a href="szablon.php?name='.$wiersz['id'].'">[więcej]</a></p>';
echo '</div>';
?>
[PHP] pobierz, plaintext 

Do news_tresc mozesz jeszcze dodac funkcje nl2br.
Poza tym zamiast tych p lepiej używać divów bądź nagłówków.

Dalej:

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

Człowieku! A sprawdzenie poprawnego hasła i loginu?

No i na koniec menu. Ewentualnie może być.

Ten post edytował tommy4 10.07.2007, 07:17:25

Dzięki.
Oczywiście sprawdzam też czy uzytkownik jest zalogowany , robie to wlasnie za pomoca tego:

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

jeżeli użytkownik poda przy logowaniu poprawne hasło i login otrzymuje $_SESSION['prawid_uzyt'].

Ten post edytował falfin 10.07.2007, 11:12:49