Problem z funkcjami klasy db w klasie cfg Opcje

[morawcik89]

Mam taki oto problem:
- W pliku index.php inlcuduję pliki klas db (bazy) i get_cfg (config strony)
- Tworzę nowy obiekt klasy get_cfg i wywołuję funkcję cfg
- W contruct klasy get_cfg tworzę obiekt klasy db
- W funkcji cfg pobieram z bazy zawartość komórki theme
- Includuję pliki z folderu template/pobrana_wartość/

Przeglądarka zwraca błąd, że pliki template//... (... -> odpowiedni plik) nie istnieje

index.php

[PHP] pobierz, plaintext 
<?
session_start();
$user = 'sys';
if($_SESSION['user']) $user = $_SESSION['user'];
 
include 'class/db.class.php';
include 'class/get_cfg.class.php';
 
$cfg = new get_cfg;
$temp = $cfg->cfg($user);
 
include 'template/'.$temp.'/head.tpl';
include 'template/'.$temp.'/register.tpl';
include 'template/'.$temp.'/foot.tpl';
 
?>
[PHP] pobierz, plaintext 

class/db.class.php

[PHP] pobierz, plaintext 
<?
 
class db
{
   private $query = '';
   private $connect = '';
   private $db_name = '';
   private $go_query = '';
 
   function __construct()
   {
	  $this->connect = $this->connect();
   }
   function __destruct()
   {
	  $this->close();
   }
 
   private function connect()
   {
	  $db = parse_ini_file('config.ini', true);
	  $this->connect = mysql_connect($db['mysql']['host'], $db['mysql']['user'], $db['mysql']['pass']);
	  $this->db_name = $db['mysql']['name'];
	  mysql_select_db($this->db_name);
   }
 
   public function query($query)
   {
	  $this->query = $query;
   }
 
   public function go_query()
   {
	  $this->go_query = mysql_query($this->query);
   }
 
   public function assoc()
   {
	  return mysql_fetch_assoc($this->go_query);
   }
 
   public function row()
   {
	  return mysql_fetch_array($this->go_query);
   }
 
}
 
?>
[PHP] pobierz, plaintext 

class/get_cfg.class.php

[PHP] pobierz, plaintext 
<?
 
class get_cfg
{
   private $db = '';
 
   function __construct()
   {
	  $this->db = new db;
   }
 
   public function cfg($user)
   {
	  $query = $this->db->query("SELECT * FROM cfg WHERE user = '$user'");
	  $go = $this->db->go_query();
	  $assoc = $this->db->assoc();
	  $temp = $this->db->assoc['theme'];
	  return $temp;
   }
}
 
?>
[PHP] pobierz, plaintext 

[Crozin]

error_reporting(E_ALL | E_STRICT);

i czy tearaz cos zwraca?

[morawcik89]

Notice: Undefined index: user in ...
Notice: Undefined property: db::$assoc in ...

EDIT:
Pierwszy błąd wyeliminowałem zmieniając linię z $_SESSION['user'] na

if(!empty($_SESSION['user'])) $user = $_SESSION['user'];

EDIT2:
Temat do zamknięcia.
Błąd był tu: $temp = $this->db->assoc['theme'];
Miało być : $temp = $assoc['theme'];

Ten post edytował morawcik89 5.09.2008, 18:13:55

[LBO]

Nie pobiera Tobie z bazy pola 'theme'... ale to Twój najmniejszy problem.

Zalecałbym całkowitą rekonstrukcję Tych klas, bo daleko na razie im do OOP. Programowanie obiektowe to nie tylko zawieranie funkcjonalności w klasach. Idzie za tym cała masa wytycznych, wzorców jak i zdrowego rozsądku.

1. Klasa get_cfg nie powinna mieć nic wspólnego z klasą db. Klasa get_cfg pobiera tylko konfigurację i zwraca ją w czytelnej formie.
2. Klasa db, powinna tylko pobierać parametry połączenia i wykonywać akcje na bazie -> connect(), query(), fetch() etc (polecam zapoznanie się PDO).

A powyższymi (czyli pobranie konfiguracji i połączenie z bazą powinien robić jeszcze inny nadrzędny skrypt).
Można by też mieć dodatkową klasę User, reprezentującą użytkownika wraz z przypisaną do niego konfiguracją (themes?). Dopiero ona mogłaby używać obiektu klasy db, do pobierania danych.

Dodatkowo nazwy. get_cfg? Instancje klasy reprezentują sobą jakby niezależne obiekty, a nie same akcje jakie wykonują. Nie lepiej by było, żeby posiadać klasę Config, która posiada metodę get()?

[PHP] pobierz, plaintext 
<?php
$config = new Config();
$config->load('config.ini'); // pobierasz konfiguracje aplikacji
 
// tworzysz połączenie z baza używając danych z konfiguracji
$db = new Database($config->get('host'), $config->get('user'), $config->get('pass'), $config->get('dbname'));
 
// tworzysz obiekt użytkownika i przekazujesz
// do niego połączenie z baza, by mógł wewnętrznie
// wykorzystać je do pobrania odpowiednich danych
$user = new User($db);
$user->getThemeName(); // pobierasz nazwę folderu dla theme'a, którą klasa pobrała np w konstruktorze używając $db
?>
[PHP] pobierz, plaintext 

Powyzsze to prosty przykład, ale powinien mniej więcej wyjaśnić o co mi chodzi.

[Cysiaczek]

Ta.. oop... a ja jestem św Mikołajem (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Przenoszę oczko wyżej.