Forum PHP.pl > Połaczenie z bazą danych a insert

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: Połaczenie z bazą danych a insert

Forum PHP.pl > Forum > PHP > Object-oriented programming

marcwars

26.01.2013, 11:20:56

Witam
od paru dni próbuję przypomnieć sobie podstawy OOP tworząc prostego cmsa z funkcją dodawania artykułów.
Jednak wywala mi się błąd: No database selected. Do kodu włączyłem nowe funkcje mysqli, więc chyba tam jest błąd, niestety po godzinach ślęczenia, nie potrafię go znaleźć.

Formularz:

Kod

<html>
<body>
<form action="dzieki.php" method="post">
Autor: <input type="text" name="autor"><br /><br />
Tytuł: <input type="text" name="tytul"><br /><br />
Tekst: <input type="text" name="tekst"><br /><br />
<input type="submit" value="Dodaj">
</form>
</body>
</html>

klasy (class/DB_pt.class.php):

Kod

<?php

class DB {

    protected $db_name = 'aaa';
    protected $db_user = 'root';
    protected $db_pass = 'aaa';
    protected $db_host = 'localhost';


    public function connect() {
        $connection = new mysqli($this->db_host, $this->db_user, $this->db_pass, $this->db_name);
        //mysql_select_db($this->db_name);

        return true;
    }

    public function insert($data, $table) {
        $columns = "";
        $values = "";
        foreach ($data as $column => $value) {
            $columns .= ($columns == "") ? "" : ", ";
            $columns .= $column;
            $values .= ($values == "") ? "" : ", ";
            $values .= $value;
        }
        $sql = "insert into $table ($columns) values ($values)";
        mysql_query($sql) or die(mysql_error());
        return mysql_insert_id();
        echo 'Dziękujemy, że dodałeś artykuł:)';
    }

}

?>

dzieki.php

Kod

<?php
include 'class/DB_pt.class.php';
$db = new DB();
$db -> connect();
$data = array(
"$_POST[autor]" => "autor",
"$_POST[tytul]" => "tytul",
"$_POST[tekst]" => "tekst"
);
$db->insert($data, 'artykuly');
?>

Na pewno ścieżki i nazwy tabel są prawidłowe, czy ktoś może mi wskazać błąd?

mortus

26.01.2013, 12:09:32

W jednej linii korzystasz z mysqli, a w kolejnej z mysql.

PS. Funkcja insert korzysta z funkcji mysql_*, a nie z mysqli_*. Wybierz albo jedno, albo drugie.

404

26.01.2013, 12:11:59

Używasz 2 rzeczy, których raczej nie wypada mieszać. Tworzysz obiekt MySQLi, a następnie wywołujesz funkcje MySQL.

@up: uprzedził mnie kolega

marcwars

26.01.2013, 20:57:25

Cytat(mortus @ 26.01.2013, 12:09:32 )

W jednej linii korzystasz z mysqli, a w kolejnej z mysql.

PS. Funkcja insert korzysta z funkcji mysql_*, a nie z mysqli_*. Wybierz albo jedno, albo drugie.

Niestety, wstawiłem w kodzie klasy:

zamiast:

Kod

mysql_query($sql) or die(mysql_error());
return mysql_insert_id();

nowy kod:

Kod

mysqli_query($sql) or die(mysql_error());
return mysqli_insert_id();

ale wychodzi błąd mysqli_query() expects at least 2 parameters, 1 given in 'nazwa_pliku' on line 76, to zła funkcja w mysqli?

aras785

26.01.2013, 21:05:29

Daj wszystko. No i funkcji insert (w/w) najpierw komunikat, a później return

marcwars

27.01.2013, 09:08:40

Zmieniłem kolejność, ale dalej pojawia się ten sam błąd, czyli, że funkcja mysqli_query potrzebuje co najmniej 2 parametrów, a jest tylko 1:)

abort

27.01.2013, 10:22:33

mysqli_query działa w dwóch trybach: proceduralnym i obiektowym. W trybie obiektowym wystarczy jeden parametr, w trybie proceduralnym funkcja OCZEKUJE dwóch parametrów. Ty używasz mysqli w trybie proceduralnym (pomimo tego że uzywasz tego w innym obiekcie). Więcej w manualu: http://pl1.php.net/manual/en/mysqli.query.php (take z przykładami użycia obu trybów). Przeczytaj, przetraw - przecież nie będziemy Ci tłumaczyć sposobu używania funkcji, bo od tego jest manual...

marcwars

27.01.2013, 11:24:11

Zmieniłem w klasie funkcję insert na:

Kod

public function insert($data, $table) {
        $columns = "";
        $values = "";
        foreach ($data as $column => $value) {
            $columns .= ($columns == "") ? "" : ", ";
            $columns .= $column;
            $values .= ($values == "") ? "" : ", ";
            $values .= $value;
        }
$con = $this->connect();
        $sql = "insert into $table ($columns) values ($values)";
        $result = $con->query($sql);
echo 'Dziękujemy, że dodałeś artykuł:)<br />';
        return $result;

    }

Nie pokazuje błędu, ale nie dodaje danych do bazy, czy wiecie gdzie jest błąd?

mortus

27.01.2013, 18:48:38

Przecież według tego, co pisałeś na początku, metoda connect zwraca true. Zatem nowa wersja metody insert jest kompletnie bez sensu. Tutaj nie chodziło o nieprawidłową budowę funkcji insert, ale o to, że naprzemiennie wykorzystujesz mysql i mysqli, a powinieneś korzystać tylko z jednego z tych dwóch rozszerzeń. Przede wszystkim zachęcam do przejrzenia tego fragmentu dokumentacji, bo wszystko jest tam dokładnie pokazane w przykładach.

PS: Na pierwszy rzut oka pętla foreach nie zbuduje Ci prawidłowego fragmentu zapytania.

abort

27.01.2013, 18:52:20

Nie wiemy. Fusów nie mamy, bo herbatę pijemy ekspresową, kawę zrestą tez z ekspresu. A szklana kula nam się zepsuła.

Od tego jest debugowanie, w którym pomagają takie zaklęcia jak echo, print_r czy var_dump. Ponadto przy pracy z bazą danych warto przetestować komendę SQL w konsoli bazy.

marcwars

29.01.2013, 21:32:32

Poprawiłem kod pliku z klasą, tak że funkcja connect() już nie zwraca true, ale połączenie z bazą:

Kod

<?php

class DB {

    protected $db_name = 'aaa';
    protected $db_user = 'root';
    protected $db_pass = 'aaa';
    protected $db_host = 'localhost';


    public function connect() {
        $connection = new mysqli($this->db_host, $this->db_user, $this->db_pass, $this->db_name);

        return $connection;
    }

   public function insert($data, $table) {
        $columns = "";
        $values = "";
        foreach ($data as $column => $value) {
            $columns .= ($columns == "") ? "" : ", ";
            $columns .= $column;
            $values .= ($values == "") ? "" : ", ";
            $values .= $value;
        }
$con = connect();
        $sql = "insert into $table ($columns) values ($values)";
        $result = $con->query($sql);
echo 'Dziękujemy, że dodałeś artykuł:)<br />';
        return $result;

    }

}

?>

Niestety teraz pokazuje mi, że odwołuje się do nieznanej funkcji connect(), chociaż ta wcześniej jest public...

mortus

30.01.2013, 08:44:15

Wywołując connect() nie odnosisz się do metody zdefiniowanej wewnątrz klasy, ale do jakiejś tam globalnej funkcji (która w tym przypadku nie istnieje).
Użyj $this->connect().

Poza tym kilka uwag:

dane dostępu do bazy nie powinny być przechowywane na sztywno w klasie - pamiętaj, że one mogą się zmieniać, zatem powinny być zlokalizowane np. w pliku konfiguracyjnym i przekazywane/pobierane w momencie tworzenia instancji klasy DB
połączenie z bazą danych powinieneś nawiązać raz, a co za tym idzie powinno to nastąpić w momencie tworzenia instancji klasy DB
pamiętaj, że metody mogą zwracać różne wartości i warto to kontrolować, bo w tej chwili nie wiemy (nie wiesz), czy w ogóle udaje Ci się nawiązać połączenie z bazą danych

marcwars

30.01.2013, 19:50:12

Witam, wstawiłem $con=$this->connect(), ale nie działa- kod nie dodaje danych do bazy, nie pokazuje też żadnego błędu i pojawiają się końcowe o dodaniu do bazy.

Kod

public function insert($data, $table) {
        $columns = "";
        $values = "";
        foreach ($data as $column => $value) {
            $columns .= ($columns == "") ? "" : ", ";
            $columns .= $column;
            $values .= ($values == "") ? "" : ", ";
            $values .= $value;
        }
$con = $this->connect();
        $sql = "INSERT INTO $table ($columns) VALUES ($values)";
        $result = $con->query($sql);
echo 'Dziękujemy, że dodałeś artykuł:)<br />';
        return $result;

    }

mortus

30.01.2013, 19:55:09

Zapewne dlatego, że źle definiujesz tablicę $data (tzn. zamieniasz wartości z nazwami kolumn podczas odbierania danych z $_POST). Poczytaj o sposobach na debugowanie skryptu php.

marcwars

31.01.2013, 08:08:47

Wszystko prawie działa, ale przy jednej zmiennej, czyli kod dodaje autora do bazy:

Kod

error_reporting(E_ALL);
ini_set('display_errors', '1');
include 'class/DB_pt.class.php';
$db = new DB();
$db -> connect();
$autor = 'autor';
define('autor', 'autor');

$data = array(
$autor => $_POST[autor],
//$tytul => $_POST[tytul],
//$tekst => $_POST[tekst]
);
$db->insert($data, 'artykuly');

ale już kod poniżej nic nie dodaje:

Kod

error_reporting(E_ALL);
ini_set('display_errors', '1');
include 'class/DB_pt.class.php';
$db = new DB();
$db -> connect();
$autor = 'autor';
define('autor', 'autor');
$tytul = 'tytul';
define('tytul', 'tytul');
$tekst = 'tekst';
define('tekst', 'tekst');

$data = array(
$autor => $_POST[autor],
$tytul => $_POST[tytul],
$tekst => $_POST[tekst]
);
$db->insert($data, 'artykuly');

Jaka jest tego przyczyna? Zmienne są podobnie opisane.

mortus

31.01.2013, 09:06:03

[PHP] pobierz, plaintext 
// zamiast
$data = array(
  $autor => $_POST[autor],
  // ...
);
// powinno być
$data = array(
  'autor' => $_POST['autor'],
  'tytul' => $_POST['tytul'],
  'tekst' => $_POST['tekst']
);
[PHP] pobierz, plaintext

Oczywiście te dane trzeba jeszcze przefiltrować, bo w tej chwili skrypt narażony będzie na SQL Injection.
Nie obraź się, ale zabierasz się za programowanie obiektowe, a podstawy "leżą". Sięgnij po jakąś lekturę, czy tutorial.

Klasę, którą napisałeś mógłbyś zastąpić PDO.

marcwars

6.02.2013, 18:11:28

@Mortus, to prawda, że podstawy trochę u mnie leżą, stąd staram się przypomnieć je na realnym przykładzie.

Wstawiłem tak:

Kod

error_reporting(E_ALL);
ini_set('display_errors', '1');
include 'class/DB_pt.class.php';
$db = new DB();
$db -> connect();
$autor = 'autor';
define('autor', 'autor');
$tytul = 'tytul';
define('tytul', 'tytul');
$tekst = 'tekst';
define('tekst', 'tekst');

$data = array(
'autor' => $_POST[autor],
'tytul' => $_POST[tytul],
'tekst' => $_POST[tekst]
);
$db->insert($data, 'artykuly');

... ale ten kod nie dodaje nic do bazy, komunikat pojawia się zgodny z funkcją insert($data, $table) z klasy.
Jak dodam do powyższego kod:

Kod

print_r($data);

to mi się wyświetla prawidłowo tablica, czyli:

Cytat

Array ([autor] => xxx [tytul] => xxx [tekst] => xxx )

Już sam nie wiem, co może być nie tak.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.