Utknąłem na obsłudze baz danych Opcje

[Forti]

Witam

Od pewnego czasu samodzielnie ucze się php, napisałem pewną obszerną strone strukturalnie - działa fajnie ale straszny bałagan w kodzie.. Teraz zaczałem uczyć się oop i mvc, przeczytałem mase teorii, staram się przeglądać przykłady (skrypty phpbb3 czy smf to jakaś porażka z kodu dla mnie..) i poległem na prostych rzeczach, tak prostych, że aż zły jestem ^^ od wczoraj szukam rozwiązania i nie bardzo wiem w co uderzyć..

kod pliku models/config.class.php

[PHP] pobierz, plaintext 
<?php
	if(!defined("access"))
	{
		include("errors/access.html");
		exit();
	}
/**
*
*	połączenie z bazą danych
*
*/
	try
	{
		$baza = new PDO('mysql:host=localhost;dbname=forti_arturs;charset=utf8', 'forti', 'Schroeder78');
	}
	catch(PDOException $e)
	{
		echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
	}
 
 
class config {
 
	public function load_config($tabela) {
		global $baza;
 
		try
		{
			$cms_config_request = $baza->query("SELECT * FROM '$tabela'");
			$this -> $cms_config = $cms_config_request->fetch();
			return $this -> $cms_config;
		}
		catch(Exception $exception)
		{
			echo 'Nie można pobrać danych z bazy '.$tabela.' '.$exception ->getMessage().'';
		}
	}
}
?>
[PHP] pobierz, plaintext 

i mam problem w klasie config.. wiem że load_config jest kompletnie źle napisane..

wywołanie tego w pliku index.php (później przeniosę to do kontrolera):

[PHP] pobierz, plaintext 
<?php
 
	define("access", true);
 
	require_once("models/config.class.php");
 
	/**
	*
	* ładujemy domyślny styl strony
	*
	*/
	$cms_config = config::load_config(cms_config);
 
 
	include("views/{$cms_config["cms_style"]}/index.php");
 
?>
[PHP] pobierz, plaintext 

wiem, że config::load_config i klasa config wogóle ze sobą nie współgrają i że całość tego to jakiś kompletny bajzel nad którym część z was się uśmieje (IMG:style_emoticons/default/wink.gif)

Co chce osiągnąć: zwrócić tabele cms_config w postaci oczywiście tablicy.

siedzę nad tym od dłuższej chwili i jedyne co potrafie zrobić to pobieranie tej tabeli wyciągnąć poza klase.. -,-



edit:
serwer zwraca mi błąd:
Fatal error: Call to a member function fetch() on a non-object in /home/forti/domains/arturs.pl/public_html/models/config.class.php on line 30

wiem co on oznacza ale nie mam pojęcia jak to rozgryść inaczej.

Ten post edytował Forti 9.09.2014, 10:30:38

[Turson]

SELECT * FROM '$tabela'
pomyliłeś ' z `

btw. używanie global dla uzyskania połączenia z bazą, to zło. Model powinien dziedziczyć klasę do połączenia z db, albo napisz statyczną metodę to wzięcia obiektu pdo, np. Db::get()

Ten post edytował Turson 9.09.2014, 10:34:20

[nospor]

Robisz straszmie duzo bledow. Polecam ci zapoznanie się z tym linkiem
Temat: Jak poprawnie zada pytanie
masz tam całą masę pożytecznych wskazówek, w tym i wyswietlanie blędów php oraz bazy.

ps: zamiast brać się za obiektówkę warto by się wpierw podszkolić trochę jeszcze bardziej z podstaw.

ps2: przenosze

[Forti]

Wiem o tym, nospor. Z tym linkiem już dawno się zapoznałem, racja jedynie że zły dział, mój błąd. Pisząc strukturalnie, ucząc się składni itp. (` zamiast ', = zamiast == itp. błędy) będę stać w miejscu. Wole rzucić się w końcu na głęboko wode, mam sporop skryptów napisanych w mvc, oop bez mvc, jednak nie znalazłem tam informacji na ten temat (lub nie patrze dokładnie).

Turson dziękuje za pomoc.


Zrobiłem to tak:

[PHP] pobierz, plaintext 
class config {
 
	private static $_cms_config = array();
 
 
	public function load_config($tabela) {
		global $baza;
 
		try
		{
			$cms_config_request = $baza->query("SELECT * FROM `$tabela`");
			self::$_cms_config = $cms_config_request->fetch();
			return self::$_cms_config;
		}
		catch(Exception $exception)
		{
			echo 'Nie można pobrać danych z bazy '.$tabela.' '.$exception ->getMessage().'';
		}
	}
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
	$cms_config = config::load_config('cms_config');
 
	include("views/{$cms_config["cms_style"]}/index.php");
[PHP] pobierz, plaintext 

Co do global to wiem.. ale nie bardzo rozumiem jeszcze jak to inaczej rozwiązać. Cały czas więcej czytam / szukam / analizuje niż piszę.

[Turson]

Co do bazy, to np. stwórz klasę DbConnection ze statyczną metodą get, która zwraca obiekt pdo.
Jak piszesz ze wzorcem MVC, to model powinien dziedziczyć główny model, a ten klasę bazodanową.

@down
Przecinki w złych, miejscach (IMG:style_emoticons/default/tongue.gif)

Ten post edytował Turson 9.09.2014, 11:21:33

[nospor]

Skoro uzywasz wyjatkow, do lapania bledow PDO, to ustaw w PDO by PDO rzucalo wyjatkami w czasie bledow. Teraz tego nie robisz, i bledy bazy ci nie lecą.

@Turson czemu model ma dziedziczyc po klasie bazy? Model powinien a i owszem, korzystac z obiektu bazy, ale nie powinien z niego dziedziczyc.

[Turson]

Jeżeli mówimy o dziedziczeniu w strukturze JakiśModel -> GłownyModel -> KlasaDb, to dziedziczymy KlasaDb, żeby mieć dostęp do obiektu bazy w JakiśModel.
No dobra, nie musi, wystarczy, że GłównyModel ma metodę zwracającą obiekt. Git.

[nospor]

No i teraz sobie wyobraz, ze klasa bazy zawsze w konstruktorze generuje połączenie do bazy.
Odpalasz 5 modeli w czasie jednego żądania a przez co odpalasz 5 połączen do bazy. NIe ma to zadnego sensu.

Klasa bazy to klasa bazy. Model moze miec dostep do metod klasy bazy poprzez posiadanie jej obiektu. To wystarczy.

[Turson]

Nieee, nie mówię, że konstruktor ma zawsze zwracać nowy obiekt, a ma działać na zasadzie sigletona, jeżeli nie ma obiektu - tworzy nowy.
Suma sumarum, zgodziliśmy się co do jednego.

[pyro]

Klasa bazy to klasa bazy. Model moze miec dostep do metod klasy bazy poprzez posiadanie jej obiektu. To wystarczy.

Należałoby najpierw zacząć od tego, że model nie powinien w ogóle mieć dostępu do obiektu bazy danych, jej klasy, ani tym bardziej jej dziedziczyć.