[php]/[MySQL] Wyswietlanie newsów z bazy. Opcje

[dawhol]

Witam jestem poczatkujacy w sprawach php ucze sie dopiero kilkanascie dni (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ale napisałem juz pare skryptów dzisiaj chciałem dac pod ostrzał skrypt do wyswietlania newsów z bazy.

Struktóra bazy danych jest następująca:
ID | TITLE | SHORTNEWS | LONGNEWS | DATA

A o to kod pliku newsy_wyswietl.php

[PHP] pobierz, plaintext 
<?php
$mysql = '../conf/mysql.ini.php';
 
/* Za pomocą funkcji if sprawdzamy czy istnieje plik ze zmiennej $mysql, gdy funk
cja zwróci wartosc TRUE
   wczytujemy plik w innym przypadku wyswietlamy blad. */
 
if ( file_exists($mysql)) 
{
	require_once($mysql);  
} else echo ('Nie można się połączyć z bazą ponieważ plik'.$mysql.' nie istnieje.');
 
/* Deklarujemy funkcje shortnews() */
function shortnews()
{
	/* ile na strone */
	$ile = 10;
	$numrows = mysql_num_rows(mysql_query("SELECT * FROM cms_news"));
	if(!$p) $p = 0;
	/* zabezpieczenie przed nienumerycznymi wartosciami */
	$p = (int)$p; 
	$ile = (int)$ile;
 
	$zapytanie = "SELECT * FROM cms_news ORDER by data LIMIT $p,$ile";
	$wykonaj = mysql_query($zapytanie);
 
	while($dane=mysql_fetch_array($wykonaj)) { echo "<table width="500px"><tr>
	  <td>".$dane['title']."</td></tr>
	   <tr><td>".$dane['shortnews']."<br><a href="news_wyswietl.php?wiecej=1&id=".$dane['id']."">Więcej ...</a></td></tr>
		<tr><td>".$dane['data']."</td>
		</tr></table><br /><br />";
		}
	 
	echo "strona: ";
	for($i=0;$i<ceil($numrows/$ile);$i++) {
	echo '<a href="'.$PHP_SELF.'?p='.($i*$ile).'">'.($i+1).'</a> ';};
 
}
 
/* Deklarujemy funkcję longnews() */
function longnews()
{
 
	if (is_numeric($_GET['id'])) 
	{
		$id = $_GET['id'];
	}
	else
	{
		$id = rand(); /* Jeżeli id nie bedzie liczba lub bedzie puste wtedy zostanie wylosowane
						 jezeli nie bedzie go w bazie to nic sie nie wyswietli a 
						 jezeli dany id bedzie w bazie wyswietli on w formie dludiej newsa o wylosowanym
 id */ 
	};
 
	$zapytanie2 = "SELECT * FROM cms_news WHERE id=$id LIMIT 1";
	$wykonaj2 = mysql_query($zapytanie2);
 
	if ($dane2=mysql_fetch_array($wykonaj2)) { echo "<table width="500px"><tr>
	  <td>".$dane2['title']."</td></tr>
	   <tr><td>".$dane2['shortnews']."<br /><br />".$dane2['longnews']."</td></tr>
		<tr><td>".$dane2['data']."</td>
		</tr></table>";};
}
 
/* Jezeli zmienna $wiecej jest pusta wtedy zostana wyswietlone wszystkie newsy w 
formie krótkiej,
jezeli wartos ta bedzie liczba oraz bedzie równa 1 wtedy zostanie wyswietlony je
den news w całości,
w innym wypadku zostanie wyświetlony bład */
 
$wiecej = $_GET['wiecej'];
 
if (empty($wiecej)) 
{
	shortnews();	
} 
else 
{
		if (is_numeric($wiecej) && $wiecej = 1) 
		{
			longnews();
		} 
		else echo ('Błąd wartość zmiennej $wiecej jest nie prawidłowa !!!');	   
};
 
?>
[PHP] pobierz, plaintext 

Proszę o ocenę i ew. propozycje poprawy i zmian (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował dawhol 4.08.2006, 23:56:42

[Seth]

Jak na jeden z pierwszych skryptow wyglada niezle (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)


Zmienne, ktore maja byc stale, a do tego sa w jakis spsob konfiguracja w skrypcie (jak $ile = 10) powinny byc definiowane przez define() na poczatku skryptu.
Jezeli bedziesz chcial zmienic liczbe wyswietlanych newsow, to nie bedzie szukal po calym pliku tej zmiennej.


Losowanie newsa raczej jest zbedne. Nie podales przedzialu losowania, wiec raczej napewno wylosuje nieistniejacy. Moze lepiej wyswietlic news z ID 1 ?


Inna sprawa to taka, ze mozna by sie pokusic o jakis system wzorcow aby odseparowac, warstwe prezentacji. Bedzie czytelniej i latwiej w utrzymaniu (zmianach).


Ostatnia chyba sprawa to nazewnictwo zmiennych i komentarze.

$p, $ile nic nie mowia. Jezeli wrocisz do tego skryptu po jakims czasie, to tez Ci one nic nie powiedza (bez analizy kodu).

Dodatkowo, mozna by sie pokusic o inna firme zapisu zmiennych.
Np:
- $alaMaKota albo
- $iAlaMaKota lub $intAlaMaKota.

Pierwszy spsob wyroznia czlowny wyrazow zmiennej. Jest to znacznie czytelniejsze niz ciag nazwa pisana cala z malych liter $alamakota.

Drugi przyklad wprowadza na poczatku nazwy zmiennej infromacje o typie zmiennej (o ile mnei pamiec nie myli, nazywa sie to notacja wegierska?).
i dla int
f dla float
s dla stringa
o dla object itd.

Mozna tez uzyc dluzszej formy typu $intCos, $strTekst, $floatZmienna


Komentarze, ktore sa jedna linia nie trzeba otaczac /* */ do tego jest // na poczatku linii.
Niby mala roznica ale ulatwia np. odkomentowywanie danych linii.


Polecam Ci poczytac takze o standardach kodowania.
http://wortal.php.pl/wortal/artykuly/pomys...dardy_kodowania
Dzieki temu wprowadzisz do swoich skryptow, pewien porzadek i przemyslana strukture.

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