Inserting data to database using oop

Question

I'm using oop to insert some data that will be fill from a form into my data base.

I'm able to connet the php to the database using pdo but I'm not having the data going into the database, can you please give me an example.

This is my code:

<?php

class goods
{  
    public $name_goods;
    public $price;
}

try {
    $db_conn = new PDO('mysql:host=localhost;dbname=database','root','');
} catch (PDOException $e) {
    echo "Could not connect to database";
}

$name_goods = $_POST['name_goods'];
$price = $_POST['price'];
$sql = 'INSERT INTO goods(name_goods, price,)VALUES ($name_goods, $price)'; 

?>

That's not OOP PHP at all, what are you trying to do what that goods class ? — Clément Malet, Aug 13 '14 at 08:20
This is just a random accumulation of code snippets, which does not an answerable question make. — deceze, Aug 13 '14 at 08:21
I hope that your code isn't visually structured like that, I have a headache looking at it — Oliver Bayes-Shelton, Aug 13 '14 at 08:21
Ref. http://stackoverflow.com/questions/60174/how-can-i-prevent-sql-injection-in-php?lq=1 — user2864740, Aug 13 '14 at 08:25

Kevin Labécot · Answer 1 · 2014-08-13T08:30:54.993

3

Your $sql query is never executed.

edit : And you have syntax errors. Comma after "price" and missing quotes for values.

edited Aug 13 '14 at 08:30

answered Aug 13 '14 at 08:20

Kevin Labécot

2,005
13
25

score 1 · Answer 2 · answered Aug 13 '14 at 08:22

1

Use prepared statements!

$sql = 'INSERT INTO goods(name_goods, price) VALUES (:name_goods, :price)';  
$q = $conn->prepare($sql);
$q->execute(array(':name_goods'=>$name_goods,
                  ':price'=>$price));

answered Aug 13 '14 at 08:22

Cheesi

483
4
14

score 0 · Answer 3 · answered Aug 13 '14 at 08:28

This does not look OOP at all. The above responses are right, your SQL isn't executing (examples here: http://php.net/manual/en/function.mysql-query.php)

But if you're looking for a quick start with PHP OOP, I'd advise looking at some PHP frameworks.

Inserting data to database using oop

3 Answers3