Klasa Oferta, PHP problem ze zrozumieniem

Question

Cześć, nie rozumiem zbytnio jak używać obiektowości, chce zrobić dodawanie ofert(name, price) do bazy

Zrobiłam coś takiego ale jest źle i nie wiem jak powinno być, (drugi dzień w obiektowości i dlatego mam z tym problem)

Wytłumaczy mi ktoś jak to ma wyglądać ? Wstępnie zrobiłam coś takiego:

<?php
	require 'database.php';
	
	class offert {
		public $name;
		public $price;
		
		public function __construct($name, $price) {
			$this->name = $name;
			$this->price = $price;
		}
	}
	
		$attr = new offert("lol", 1333);
		echo $attr-> name;
	
	
		function addPost(){
			$pdo = Database::connect();
			$sql = "INSERT INTO offert(id, name, price) values("."NULL, :name, :price)";
			$q = $pdo->prepare($sql);
			$q->bindValue(':name', $name, PDO::PARAM_STR);
			$q->bindValue(':price', $price, PDO::PARAM_INT);
			$q->execute();		
		}
	
	
	$attr -> addPost();
	        
	
	
	
	

 

Answer 1

Tworzenie klasy która jest tylko i wyłącznie nośnikiem danych i nie ma żadnych metod wykonujących jakieś czynności (oprócz konstruktora który ustawia wartość pól w klasie) ma rację bytu tylko w dwóch przypadkach:

• Kiedy jest bardzo dużo danych do przeniesienia (dużo pól w klasie)
• Kiedy klasa dziedziczy pola po jakiejś klasie.

W innych przypadkach tworzenie czegoś takiego nie ma sensu. Widzę, że zapisałeś funkcję dodawania danych do bazy poza klasą, a to powinno się znaleźć w klasie:

<?php
    require 'database.php';
     
    class offert {
        public $name;
        public $price;
         
        public function __construct($name, $price) {
            $this->name = $name;
            $this->price = $price;
        }

       public function addPost(){
            $pdo = Database::connect();
            $sql = "INSERT INTO offert(id, name, price) values("."NULL, :name, :price)";
            $q = $pdo->prepare($sql);
            $q->bindValue(':name', $this -> name, PDO::PARAM_STR);
            $q->bindValue(':price', $this -> price, PDO::PARAM_INT);
            $q->execute();       
        }
    }
     
        $attr = new offert("lol", 1333);
        echo $attr-> name;
        $attr -> addPost();
             
     
     
     
    

Przy okazji poczytaj o setterach i getterach. Istnieje też fajna sztuczka programistyczna zwana fluent interface. Pamiętaj - obiektówka to twój przyjaciel!

Powodzenia w nauce!

Comments

Okej dzięki, to co do tego dodawania do bazy, że funkcja powinna się znaleść w klasie. Zrobiłam tak:

<?php
	require 'database.php';
	
	class offert {
		public $name;
		public $price;
		
		public function __construct($name, $price) {
			$this->name = $name;
			$this->price = $price;
		}
		
		function addPost(){
			$pdo = Database::connect();
			$sql = "INSERT INTO offert(id, name, price) values("."NULL, :name, :price)";
			$q = $pdo->prepare($sql);
			$q->bindValue(':name', $name, PDO::PARAM_STR);
			$q->bindValue(':price', $price, PDO::PARAM_INT);
			$q->execute();		
			Database::disconnect();
		}
	
	}
	
		$attr = new offert("lol", 1333);
		$attr-> addPost();
	
	

	
	$attr -> addPost();
	        
	
	
	
Tylko nie wiem jak zrobić by ta funkcją miała zadeklarowane wartości z tego obiektu?

w bind zamiast

            $q->bindValue(':name', $name, PDO::PARAM_STR);
            $q->bindValue(':price', $price, PDO::PARAM_INT);

powinno byc:

            $q->bindValue(':name', $this->price, PDO::PARAM_STR);
            $q->bindValue(':price', $this->price, PDO::PARAM_INT);

 

Coś takiego?

---

Tak, musisz użyć $this, ponieważ w funkcji addPost() nie istnieją zmienne $name, ani $price, są to pola w obiekcie, a referencją do obiektu na którym pracujesz (siebie) jest $this.

---

A jeśli robi się serwis i robiąc walidację formularza to waliduje się pola w klasie? np

<?php
	require 'database.php';
	
	class offert {
		public $author;
		public $email;
		public $text;
		        if($text == '') 
            { 
                 echo "blad";
            }
            if(strlen($comment)>255)
            { 
				echo "blad";
			}
		
	//	public function __construct($name, $price) {
	//		$this->name = $name;
	//		$this->price = $price;
	//	}
	
		
		function addPost(){
			$pdo = Database::connect();
			$sql = "INSERT INTO comment(id, author, email, text) values("."NULL, :author, :email, :text)";
			$q = $pdo->prepare($sql);
			$q->bindValue(':name', $this->name, PDO::PARAM_STR);
			$q->bindValue(':email', $this->email, PDO::PARAM_STR);
			$q->bindValue(':text', $this->text, PDO::PARAM_STR);
			$q->execute();		
			Database::disconnect();
			echo "Done";
			               $error = $q->errorInfo();
								if ($error[0]) { var_dump($error); }  
		}
	
	}
	
	
	$komentarz = new offert();
	$komentarz->addPost();
	        
	
	
	
	

 

---

Waliduje się obiekt a nie klasę. I tym zajmuje się osobny zbiór klas skłądających się na validator.