moj pierwszy skrypt, ocena Opcje

[Home]

Witam, php studiuje od ok 2 tygodni. Napisalem skrypt, ktory ma mi slozyc jako cos ala newsy. Czyli na jednej stronie 5 niepelnych artykolow a jak sie kliknie na [wiecej] to sie caly jeden dany art pojawia. Prosze o delikatna ocene.

[PHP] pobierz, plaintext 
<?php
if(ereg("^[0-9]", $id)){ //caly text
 
	$query  = "SELECT * FROM dzialy WHERE temat_id=".(int)$_GET['id'];
 
	$result = mysql_query($query)
	or die(mysql_error());
 
	if(mysql_num_rows($result) == 0) { 
	echo "404: Nie znaleziono strony";
	}
 
	while($row = mysql_fetch_array($result)){
	echo($row["temat_text"]);
	}
}
 
 
if(ereg("^[0-9]", $strona)){ //kawalki teksow po 5 sztuk na strone
 
$limitstron = 5;
 
$wynik = (($strona * $limitstron) - $limitstron);
 
 
$sql = mysql_query("SELECT * FROM dzialy LIMIT $wynik, $limitstron");
$total_results = mysql_num_rows($sql);
   
 if(mysql_num_rows($sql) == 0){ 
		echo("404: Nie znaleziono strony"); 
	}
 
while($row = mysql_fetch_array($sql)){
 
	echo $row['temat_text']."<br />"; //+kawalek kodu ktory skraca wyswietlanie teksu do np. 500 znakow
//+link [wiecej...]+$id -odnosnik do calego teksu
}
 
 
$total_results = mysql_result(mysql_query("SELECT COUNT(*) as all FROM dzialy"),0);
 
//tutaj reszta kodu pokazujaca linki do nastepnych stron...
 
}else{
$strona =1;
}
?>
[PHP] pobierz, plaintext 

Nie pokazalem calego kodu bo jest to zbedne. Chodzi mi glownie o to czy jest to w miare poprawnie napisane, i czy z tym kodem moze mi ktos jakis syf wpisac w zmiennej i zaszkodzic w bazie danych.
dodam ze calosc mi bardzo dobrze dziala

Ten post edytował Home 28.09.2006, 18:07:56

[Prph]

Witam,

Jak na 2 tygodnie, to nie jest zle. Ogolnie calkiem poprawnie. Mam male uwagi do napisow:

[PHP] pobierz, plaintext 
<?php
echo("404: Nie znaleziono strony");
?>
[PHP] pobierz, plaintext 

Cudzyslow (") sluzy bardzo do podstawiania zmiennych, a nie generowania napisu. Lepiej w tym miejscu uzywac pojedynczego cudzyslowu ('):

[PHP] pobierz, plaintext 
<?php
$i = 5;
 
echo "Ola ma $i gruszek";
 
echo 'Ola ma 5 gruszek';
 
// Polaczenie zmiennej i napisu:
 
echo 'Ola ma ' . $i . ' gruszek';
?>
[PHP] pobierz, plaintext 

Chodzo przede wszystkim o szybkosc. W Twoim przypadku nie ma to znaczenia, ale generowanie duzej ilosci napisow przez podstawianie moze zwolnic cala aplikacje.

Kolejna rzecz - sprawdzanie czy $id jest liczba. Moze lepiej napisac funkcje:

[PHP] pobierz, plaintext 
<?php
function is_decimal($value)
{
  return (strval(intval($value)) == $value);
}
?>
[PHP] pobierz, plaintext 

Zawsze to mnie pisania w warunkach, a ponadto to wydaje sie lepsze.

Pozdrawiam, Adrian.