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

[Home]

[PHP] pobierz, plaintext 
<?php
function calosc($id) {
			$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"]);
			 }
}
 
function kawalektekstu($page) {
 
			 $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
			 //+link [wiecej...]+$id -odnosnik do calego teksu
			}
				 $total_results = mysql_result(mysql_query("SELECT COUNT(*) FROM dzialy"),0);
 
			 //tutaj reszta kodu pokazujaca linki do nastepnych stron...
}
 
 
 
if(ereg("^[0-9]", $id)){  // a moze is_numeric tu wstawic? tylko szkoda ze ujemne cyfry 
				//dopuszcza wiec wole surowo potraktowac i ograniczyc do takich 
				  //liczb jakie mam w bazie danych (id)
 
	 calosc($id);
}
 
 
if(ereg("^[0-9]", $strona)){
 
	kawalektekstu($page);
 
}
?>
[PHP] pobierz, plaintext 

Pierwsze zapoznanie sie z funkcja (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Jak narazie to jedyny plus to taki ze ladniej wszystko wyglada w kodzie . No ale zawsze to plus i dziala tak ja pierwotny kod- czyli dobrze. Potem moge np zapisac do jednego pliku te funkcje i tylko np. includowac do plikow gdzie maja zostac wykorzystane (IMG:http://forum.php.pl/style_emoticons/default/rolleyes.gif) ale i tak bardzo niewiele o tym wiem (IMG:http://forum.php.pl/style_emoticons/default/blinksmiley.gif)

Ten post edytował Home 28.09.2006, 23:29:20