klasy do sprawdzenia Opcje

[tukan]

Witam,
Chciałbym Wam zaprezentować 3 "szablony" klas, klasę kategorii, artykułu i pseudo-obsługi bazy danych. Te klasy nie są całkowicie gotowe ( ich funkcjonalność), ale daję je pod Wasze profesjonalne oko, żebyście już na początku mogli mnie pokierować na bardziej poprawne tory . Zdaję sobie sprawę, że pewnie te klasy są paskudne, mało funkcjonalne itd. Ale o to właśnie chodzi, żeby się poprawiać .
Z góry dzięki za każdą wypowiedź.
Pozdrawiam!

[PHP] pobierz, plaintext 
<?php
class Database {
public $host = "localhost";
public $dbname = "articles";
public $user = "root";
public $pass = "vertrigo";
public $dbh;
function connect(){
 
try {
 
	$this->dbh = new PDO("mysql:host=$this->host; dbname=$this->dbname", $this->user, $this->pass);
 
 
	echo "Połączenie nawiązane";
} catch ( PDOException $error){	die("Failed to connect:".$error->getMessage());}
 
}
 
function set_coding($what)
	{
		$this->dbh->exec("SET NAMES ".$what);
	}
 
 
}
 
class Taskplain{
 
public $content;	
public $solution;
public $nr_original;	
public $id_autor;	
public $id_kategoria;	
public $points;	
public $id_date;	
public $link;		
 
 
 
 
 
 
 
   function __construct($content, $id_autor, $id_kategoria,$id_date,$link ){
   	$this->content = $content;
   	$this->id_autor = $id_autor;
   	$this->id_kategoria = $id_kategoria;
   	$this->id_date = $id_date;
   	$this->link = $link;
 
   }
 
function add($dbh)
{
 
	$dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
	$sql = "INSERT INTO task_e( content, id_autor, id_kategoria, link) VALUES (:content,  :id_autor, :id_kategoria, :link)";
try {
	$q = $dbh->prepare($sql);
}
catch(PDOException $e){   print_r( $e->getTrace() );
 
}
	try {
		$q->execute(array(':content'=>$this->content,
					:id_autor'=>$this->id_autor,
					':id_kategoria'=>$this->id_kategoria,
					:link'=>$this->link));
	}
 
catch(PDOException $e)
{
   print_r( $e->getTrace() );
}
}
 
 
}
 
class Category{
	public $id;
	public $name;
	public $ids = array();
	public $names = array();
 
 
	function addCategory($name, $dbh){
			$dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
		$sql = "INSERT INTO category( name) VALUES (:name)";
		$q = $dbh->prepare($sql);
 
 
		try {
		$q->execute(array(':name'=>$this->name));
 
		}
 
catch(PDOException $e)
{
   print_r( "AddCategory: ".$e->getTrace() );
}
	}
 
	function __construct($name)
	{
		   	$this->name = $name;
 
 
	}
 
 
	function getCateogory($dbh){ // ta klasa miałaby mi tworzyć tablice ze wszystkimi kategoriami
 
		$dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
		$q = $dbh->prepare("SELECT * FROM autor");
		try{$q->execute();}
		catch(PDOException $e) { print_r( "getCategory: ".$e->getTrace() );}
 
 
 
	while ($row = $q->fetch(PDO::FETCH_ASSOC))
{
array_push($this->names, $row['name']); // tu chciałbym zastosować lepszy pomysł, zamiast tego while użyć fetch.... z PDO
array_push($this->ids, $row['id']);
}
 
 
	}
 
 
 
 
	}
 
?>
[PHP] pobierz, plaintext 

[Mephistofeles]

Źle!
Skoro tworzysz klasę do obslugi bazy danych to nie przenoś tej obsługi do innych.
W ogóle nie ma sensu tworzyć oddzielnej klasy do bazy, i tak wykorzystujesz PDO, a nie tworzysz tu czegoś na miarę Doctrine DBAL (poczytaj, może wykorzystasz).
Masz 2 możliwości:
-obiekt artykułu dostaje obiekt PDO i sam wykonuje potrzebne zapytania
-istnieje drugi obiekt, który wykorzystując obiekt artykułu zapisze dane korzystając z przekazanego mu obiektu PDO.
Pomijam rozwiązania typu ORM, bo chyba nie tego oczekujesz, ale warto się nimi zainteresować (Doctrine, Propel).

[tukan]

Czyli usunąć całkowicie tą klasę bazodanową?
Nie wiem do końca co masz na myśli mówiąc 'obiekt PDO', czy obiekt tej mojej klasy, czy czysty obiekt PDO? Jeśli ten czysty, to tworzę sobie w tym wypadku obiekt artykułu i dostarczam mu w parametrach do konstruktora uchwyt do bazy danych i w klasie artykuł już sobie operuję na bazie danych?

[Mephistofeles]

Czyste PDO.
Tak, jest to jedno z możliwych podejść. W konstruktorze raczej nie ustawiaj danych obiektu, a tylko wymagane zmienne.
Istnienie klasy Database tylko dla utworzenia połączenia nie ma za bardzo sensu.

[tukan]

Załóżmy sytuację:
Mam sobie tabelkę łączącą kategorie z artykułem, czyli kolumny to: id_kategorii, id_artykułu. Dodając w takim razie artykuł do bazy danych muszę równocześnie odwiedzić tą tabelkę i dodać wierz -id_artykuł-idkategorii. I teraz tak. Artykuł przyporządkowuję do kategorii za pomocą checkboxów, którcyh to wartości value wstawiałbym jako id_kategorii do tabeli.

Wiadomo, że formularz z checkboxami to będzie tablica z kilkoma wartościami. Jak to sprawnie "obrobić"? Jak wziąć tą wartość id_artykułu, którą będę wstawiał? Pobrać wcześniej, i przy wpisywaniu kolejnych id_kategorii wpisywać id_artykułu powiększone o 1.
W kilku słowach, jak to sprawnie rozwiązać?
pozdrawiam

Ten post edytował tukan 8.07.2012, 10:15:26

[empty]

dodajesz artykul do tabeli artykuly
id dodanego artykulu wyciagniesz tak

Kod

$id_artykulu = mysql_insert_id()

przechodzisz po wszystkich checkboxaxh i dla kazdego zaznaczonego dodajesz wiersz do tabeli artykul-kategoria
wszystko mozna wykonac jednoczesnie po wcisnieciu submita

Ten post edytował empty 10.07.2012, 07:30:00