Forum PHP.pl

Witam, jestem mocno początkująca jeśli o PHP i pisząc skrypt natrafiłam na problem, przez co nie mogę ruszyć dalej.

[PHP] pobierz, plaintext 
class connect_pdo {
 
 
    public function connect($username, $password) {
 
        try {
 
            $dsn = 'mysql:host=localhost;dbname=xxx';
            $options = array(
                PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8',
                PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
            $dbh = new PDO($dsn, $username, $password, $options);
 
        } catch (PDOException $e) {
            echo "Error!: " . $e->getMessage();
            die();
        }
public function selectAll($us, $first) {
 
 
        $query = "SELECT $us.name, $first.genre from $us, $first where $us.name = $first.name";
        $result = mysql_query($query) or die('Blad MySQL: ' . mysql_error());
 
        while ($row = $result->fetch(PDO::FETCH_ASSOC)) {
 
            print_r($row);
        }
    }
    }
[PHP] pobierz, plaintext 

Oto mój kawałek kodu służący do połączenia z bazą danych. $username i $password podaję poprzez odniesienie w index.php
Gdy wywołuję samą funkcję connect() jest ok, nie wyskakuje błąd. Natomiast gdy dodam funkcję selectAll w konsoli pojawia się: no database selected.

[PHP] pobierz, plaintext 
<?php
 
require_once('test.php');
 
$mysql = new connect_pdo();
$mysql->connect('xxx','xxx');
print_r($mysql->selectAll('users', 'first_name'));
[PHP] pobierz, plaintext 

Już nie wiem gdzie może występować błąd, pomożecie?

Ten post edytował ewu 10.09.2014, 09:47:11

Metoda connect() nie ma argumentów.
Powinnaś je podać

[PHP] pobierz, plaintext 
public function connect($dsn, $user, $password, $options){}
[PHP] pobierz, plaintext 

i przekazać odpowiednio

Dalej no database selected

Ten post edytował ewu 10.09.2014, 10:03:11

Tutaj musisz przekazać wszystko, np.

[PHP] pobierz, plaintext 
$mysql->connect('localhost', 'nazwa_uzytkownika', 'haslo', array());
[PHP] pobierz, plaintext 

Ten post edytował Turson 10.09.2014, 10:03:49

To samo. Próbowałam już innym sposobem

[PHP] pobierz, plaintext 
 $dbh = new PDO('mysql:host=' . $dsn . ';dbname=' . $database , $username, $password);
[PHP] pobierz, plaintext 

analogicznie podając $dsn, $database, $username i $password i dalej nic

Uzywajac PDO nie mozesz uzywasz mysql_query... to są dwa rozne mechanizmy... Uzywajac do łączenia PDO, to i do wykonywania zapytan masz uzywac PDO

Inna sprawa że mając w connect() konstruktor połączenia (zmienna $dbh) nie będzie on widoczny w selectAll().

Czyli za każdym razem przy tworzeniu nowej funkcji powinnam ten konstruktor wklejać na jej początek czy da się to obejść i stworzyć funkcję stricte odpowiedzialną za połączenie?

Dziękuję, dziękuję. Właśnie o to chodziło