Dawno nie pisałem nic w PHP ze względu na pracę, jednak w ostatnich dniach urlopu znalazłem chwilę aby sobie przypomnieć co nieco.
Dziś wyskrobałem klasę do obsługi komentarzy, jednak zanim zacznę ją rozbudowywać, chciałbym dostać kilka wskazówek, aby z początku nie uczyć się błędów.
Wszelkie uwagi mile widziane.

[PHP] pobierz, plaintext 
<?php
 
try {
	$db = new PDO('mysql:dbname=openttd-test;host=s.s.s', 'rysh', 's.s.s');
} catch(PDOException $exception) {
	echo $exception->getMessage();
}
 
class Comment {
	private $db;
	public $comments;
	function __construct($db) {
		$this->db = $db;
	}
 
	function AddComment($art_id, $usr_id, $text, $date, $ip) {
		$sql = $this->db->prepare("INSERT INTO comments (art_id, usr_id, text, date, ip) VALUES (:art_id, :usr_id, :text, :date, :ip)");
		$data = array($art_id, $usr_id, $text, $date, $ip);
		$sql->execute($data);
		return $sql->rowCount();
	}
 
	function DelCommnet($comment_id) {
		$sql = $this->db->prepare("DELETE FROM comments WHERE comment_id=:comment_id");
		$sql->bindValue(":comment_id", $comment_id, PDO::PARAM_INT);
		$sql->execute();
		return $sql->rowCount();
	}
 
	function ShowComments($art_id, $limit = 50) {
		$sql = $this->db->prepare("SELECT usr_id, text, date, ip FROM comments WHERE art_id=:art_id LIMIT :limit ORDER BY data DESC");
		$data=array($art_id, $limit);
		$sql->execute($data);
		$this->comments = $sql->fetchAll(PDO::FETCH_ASSOC);
	}
 
	function ShowUserComments($usr_id, $limit = 50) {
		//cdn.
	}
}
[PHP] pobierz, plaintext 

Poza tym interesuje mnie w jaki sposób przekazujecie dane do PDO:
- jako tablica i $pdo->execute($dane)
- każdy parametr z osobna $pdo->bindValue('cos', $cos, PDO::PARAM_STR)

Pozdrawiam.

Do zapytan dołożyłbym obsługę błędów
np. tak

[PHP] pobierz, plaintext 
  try {
               $sql = $this->db->prepare("DELETE FROM comments WHERE comment_id=:comment_id");
               $sql->bindValue(":comment_id", $comment_id, PDO::PARAM_INT);
               $sql->execute();
              return $sql->rowCount();
       } catch(PDOExecption $e) {
        print "Error!: " . $e->getMessage() . "</br>";
      } 
[PHP] pobierz, plaintext 

W funkcji addComment czemu return $sql->rowCount()? Nie lepiej return TRUE ( jeżeli powiedzie ) FALSE ( jeżeli nie powiedzie się )? To samo w delComment. W showComment dałbym return tablicy z wynikami.

AddComment - return $sql->rowCount(), dlatego że jeśli doda rekord to zwróci 1 czyli prawda, jeśli wystąpi błąd lub cokolwiek innego będzie 0 czyli false.
DelComment - tak samo jak wyżej, jeżeli usunie (powinno) dostanę ile rekordów usunięto w tym wypadku wchodzi w grę 1 lub 0. Czyli albo funkcja wykonała swoje zadanie, albo nie.
ShowComment - tutaj nie widziałem czy zwracać tablicę, czy przekazać do zmiennej w klasie.

Pełno klasycznych błędów początkujących php'owców.
Zapoznaj się z tematem Temat: Ocena klasy obslugujacej baze danych