Podobno jedną z dobrych metod nauki programowania jest wstawianie swojego kodu na forum i słuchanie opinii bardziej doświadczonych Eksperymentuję z programowaniem obiegowym w PHP (dotychczas programowałem strukturalnie) ale do poziomu profesjonalisty jeszcze trochę mi brakuje....ale jestem ambitny więc chętnie wysłucham wszelkich uwag.Skrypt jest prosty. Po załadowaniu podstrony pobiera wszystkie newsy z bazy danych, a po wybraniu konkretnego ładuje treść tego newsa.
Poniżej załączam skrypty.
Kod wkleiłem w znacznikach "code" natomiast podaję, jeszcze dwa linki do zewnętrznej strony, na której jest wklejony kod - moim zdaniem lepiej tam przeglądać bo jest jakoś bardziej czytelnie:
- http://wklej.org/id/1820546/ - news.Class.php
- http://wklej.org/id/1820550/ - news.php
W pliku connect.Class.php zawarta jest instrukcja ($this->select($sql) wysyłająca zapytanie do bazy danych, które [to zapytanie($sql)] jest pobierane z klasy News.
Code: Zaznacz cały
<?php //PHP OPEN
require_once("connect.Class.php");
class News extends db
{
public $id;
/* public function __construct()
{
parent::__construct();
$this->categoryName = "Newsy";
$row = mysql_fetch_array($result);
$this->title = $row['title'];
}*/
public function ShowNews($id)
{
$sql = "SELECT * FROM news WHERE id = $id LIMIT 1";
if ($this->select($sql))
{
$row = mysql_fetch_assoc($this->select($sql));
$IsNewsExists = mysql_num_rows($this->select($sql));
if ($IsNewsExists == 1)
{
$this->setVars($row);
return true;
}
else echo "Ups...niestety nie posiadamy newsa o takim ID";
} else echo "Nie udalo sie";
}
public function ShowAllNews($i)
{
$sql = "SELECT * FROM news ORDER BY date ASC";
if ($this->select($sql))
{
$result = $this->select($sql);
$this->countRows = mysql_num_rows($result);
while ($i < $this->countRows)
{
$row = mysql_fetch_array($result);
$this->setVars($row);
$i++;
}
}
}
protected function setVars($row) //USTAWIA ZMIENNE
{
$this->id = $row['id'];
$this->title = $row['title'];
$this->short_text = $row['short_text'];
$this->full_text = $row['full_text'];
$this->date = $row['date'];
$this->author = $row['author'];
}
public function filterID($id) //FILTROWANIE ID
{
if (is_numeric($id)) // SPRAWDZANIE CZY ZMIENNA TO NA PEWNO INT
{
if ($id>0)
{
return true;
}
else
{
echo "News nie może mieć ujemnego identyfikatora!";
return false;
}
}
else
{
echo "Identyfikator newsa musi być numeryczny!";
return false;
}
}
}
//PHP CLOSE
?>
Wiadomo skrypt obecnie jest mało rozbudowany, można by zrobić jakąś paginację, fajną obsługę błędów ale chciałbym wiedzieć co by tu ewentualnie pozmieniać, co jest bez sensu, a co jest dobrym rozwiązaniem.
