Diskuze: Jak by měla vypadat funkce přivání příspěvků v OOP?
Ahoj,
Tvůj problém je v tom, že nevytváříš instanci po
// END MySQL Connection Jinak si myslím, že by to mělo fungovat.
Neco jako:
$posts = new Posts();
$posts->addPost("Franta", "Můj titulek", "Text");Dále bych asi doporučil se kouknout na SQL injekce, momentálně je nepoužíváš a někdo by ti mohl smazat celou databázi tím.
Nevím jak daleko jsi v OOP, ale doporučoval bych trošku jiný náhled na
problematiku... Udělal bych si třídu Post, která by měla
atributy něco jako $title, $author,
$text. Pak nějakou třídu Posts, která by měla tu
metodu addPosts(Post $post) a brala by a vstupu objekt typu
Post. Třída Posts by se jenom starala o věci jako
přidávání a mazání příspěvků.
Taky bych se pokoušel použít konstruktory, aspoň v tý třídě
Post.
Doufám, že jsem pomohl, když tak piš 
+20 Zkušeností
+2,50 Kč
Mockrát děkuju za radu. V OOP i v PHP jsem úplný začátečník takže se
zatím učím základy. S tou instancí jsem trochu zapomněl jak tyhle funkce
ve třídách fungují
Teď mám ale ještě jeden problém a to sice to že následující kód mi nezapíše do databáze nic.
U všech kontrol to napíše že jsou ok ale pole přitom nemají žádnou hodnotu.
Index:
<?php
session_start();
require_once ('class/Posts.php');
// MySQL Connection
$servername = "127.0.0.1";
$username = "root";
$password = "xxx";
$dbname = "xxx";
$conn = new mysqli($servername,$username,$password,$dbname);
if($conn->connect_error){
die("Connection failed: " . $conn->connect_error);
} else {
echo('Připojeno');
}
// END MySQL Connection
if(isset($_POST['sub'])){
$post = new Posts();
$post->addPost(htmlspecialchars($_POST['author']),htmlspecialchars($_POST['title']),htmlspecialchars($_POST['text']));
}
?>
<meta charset="UTF-8">
<form method="post" action="index.php">
Author: <select name="author">
<option value="xxx">xxx</option>
</select><br />
Titulek: <input type="text" name="title"><br />
Text: <br /><textarea name="text" cols="50" rows="15">
</textarea>
<input type="submit" value="Odeslat" name="sub">
</form>Třída:
<?php
class Posts
{
public $id;
public $author;
public $title;
public $text;
public $message;
public function addPost($author, $title, $text)
{
$servername = "127.0.0.1";
$username = "root";
$password = "xx";
$dbname = "xx";
$conn = new mysqli($servername,$username,$password, $dbname);
if(empty($this->author))
{
$message[] = "Pole autor je prázdný.";
exit;
} else {
echo('Kontrola 1 ok.');
}
if(empty($this->title))
{
$message[] = "Pole titulek je prázdný.";
exit;
} else {
echo('Kontrola 2 ok.');
}
if(empty($this->title))
{
$message[] = "Pole text je prázdný.";
exit;
} else {
echo('Kontrola 3 ok.');
}
echo('Kontroly jsou OK teď jdeme na přidání příspěvku do SQL');
$sql = "INSERT INTO posts (author, title, text) VALUES ('" . $this->author . "','" . $this->title . "','" . $this->text ."')";
if($conn->query($sql) === TRUE){
echo('Zápis proběhl úspěšně.');
} else {
echo('Zápis nebyl úspěšný.');
echo(' Error: ' . $sql . '<br />' . $conn->error);
}
}
}
Martin Konečný (pavelco1998):11.10.2018 14:43
Ahoj, používej prosím tlačítko "Odpovědět", jinak člověku nepřijde
upozornění 
V první řadě je potřeba si uvědomit, o co v OOP jde. V OOP se zjednodušeně řečeno snažíš napodobit reálný svět, tedy kód by měl vypadat tak, aby dával logicky smysl. Ve tvém případě, kdy třída Post přidává nový příspěvek, je otázkou, zda:
- Post představuje nějaký konkrétní příspěvek - a tedy příspěvek přidává sám sebe, což nedává smysl
- Post představuje nějakého "správce" příspěvků - a tedy zde nemají smysl ty atributy, protože ty se vztahují až ke konkrétnímu příspěvku
Tedy máš dvě možnosti - buď mít možnost b) s tím, že vyhodíš ty atributy a informace o novém příspěvku budeš předávat parametrem metodě addPost(). Třída Post bude představovat nějakého "správce" příspěvků, který příspěvky umí přidávat (což je logicky OK), a tedy má v sobě instanci databázového připojení (předáš konstruktorem, až se podíváš na PDO - to je taky logicky OK).
Nebo půjdeš bodem a), ale v takovém případě třída Post bude
představovat nějaký konkrétní příspěvek - bude mít ty atributy, ale
nebude mít metodu addPost() - protože aby příspěvek uměl přidat sám sebe
(a nebo jiný příspěvek) je nesmysl. Tedy bude sloužit jako taková
"obálka" nad daty (zapamatuj si slovo "entita", může se ti později hodit) a
uděláš si ještě jednu třídu, třeba PostManager, který bude ty
příspěvky umět přidávat (jako v možnosti výše).
A jelikož jsme v objektovém světě, je logicky správné, aby si ty objekty
přes parametry předávaly jiné objekty. Tedy můžeš mít takovýto
kód:
<?php
class PostManager
{
private $pdo;
public function __construct($pdo)
{
$this->pdo = $pdo;
}
public function addPost(Post $post)
{
$query = $this->pdo->prepare("INSERT INTO post (title, text, datetime) VALUES (?, ?, NOW())");
$query->execute(array($post->getTitle(), $post->getText()));
}
}
class Post
{
private $title;
private $text;
public function __construct($title, $text)
{
$this->title = $title;
$this->text = $text;
}
public function getTitle()
{
return $this->title;
}
public function getText()
{
return $this->text;
}
}
// použití pak
$pdo = new PDO("DB login údaje");
$postManager = new PostManager($pdo);
$post = new Post("nadpis", "obsah");
$postManager->addPost($post);Je ale samozřejmě možné nemít tuto třídu Post a parametrem metodě addPost() předávat ty proměnné $title, $text atd. To jen pro ukázku, jak to v OOP logicky funguje.
Co se týká toho kódu - v té třídě by nemělo být žádné echo, tedy místo něj něco vracej a nebo vyhoď výjimku. U výjimky je to zde složitější, protože u výjimky se zpráva ($message, předávána prvním parametrem) nepoužívá pro uživatele, ale pro programátory. Tedy neměly by být použity ve smyslu
throw new Exception("Pole 'nadpis' nebylo vyplněno");Více o typech výjimek zde: https://www.youtube.com/watch?…
Jelikož se v tomto případě použití výjimek příliš nehodí a zároveň tam echo nemá co dělat, zbývá ti asi poslední možnost - "něco" vracet. A to "něco" může být právě třeba to pole chyb, resp. $error_messages. To stačí pouze potom otestovat, zda je nebo není prázdné
<?php
$errors = $postManager->addPost("nadpis", "obsah");
if (!empty($errors)) {
echo "OK, povedlo se";
} else {
echo "Byly zjištěny následující chyby: " . implode(",", $errors);
}Obvykle ale nedává logicky příliš smysl, aby výsledek nějaké operace
byl seznam chyb, ale zároveň ten seznam chceš znát, abys je mohl vypsat
uživateli V tomto
případě je to asi nejsnazší řešení, ale kdybys chtěl ještě "víc
OOP", tak ta metoda může vracet objekt, který ti bude představovat výsledek
nějaké validace.
<?php
class ValidationResult
{
private $errors = array();
public function setErrors(array $errors)
{
$this->errors = $errors;
}
public function getErrors()
{
return $this->errors;
}
public function isValid())
{
return empty($this->errors);
}
}
// použití
$result = $postManager->addPost("nadpis", "obsah");
if ($result->isValid()) {
echo "OK, povedlo se";
} else {
echo "Byly zjištěny následující chyby: " . implode(",", $result->getErrors());
}Jelikož patrně neděláš "nic velkého", volil bych možnost, která je logicky OK a zároveň není tolik pracná. Tedy bych na tvém místě šel způsobem b), který jsem popisoval výše, a to tak, že třída Post bude představovat "správce" příspěvků, které je bude umět přidávat - tedy nebude mít ty atributy, bude mít metodu addPost(), které v parametrech předáš $title, $content atd. A ta metoda ti bude vracet seznam chyb (tzn. return $error_messages). To se mi zdá jako nejlepší kompromis mezi "logicky OK" a jednoduchostí.
Jen taková drobnost - zvykem je psát proměnné notací camel case, tedy ne $error_messages, ale $errorMessages.
Zobrazeno 16 zpráv z 16.