Forum PHP.pl

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