Wiem, że stosowanie zmiennych globalnych w klasach nie jest zbyt dobrym pomysłem, ale kurczę po coś wymyślono te global i trzeba wykorzystać . Niestety pojawiają się problemy, choć wcale nie chodzi o to że się w niej gubię. Mam plik konfiguracyjny:

W pliku testującym załączam wszystkie inne klasy itd, a w pliku z klasą GenericObject mam coś takiego:

[PHP] pobierz, plaintext 
<?php
class GenericObject {
 
  private $id;
  private $table_name;
 
  private $database_fields;
  private $loaded;
  private $modified_fields;
  
/*linia 11*/  private global $db_config;
?>
[PHP] pobierz, plaintext 

Próba odpalenia kodu skutkuje czymś takim:

Co się stało?

zajrzyj lepiej do manuala
http://pl.php.net/manual/pl/language.variables.scope.php

private global

to private czy global? a w OOP używa się public aby obiekt był widoczny poza klasą.

Po to sie pisze w OOP zeby nie uzywac zmiennych globalnych masz po to public.

Chcesz pisac OOP czy co,po to wymyslono skladowe dostepu by je uzywac.

Asmox,
co za problem zrobić, że w konstruktorze podajesz tablicę z danymi bazy danych i potem np. ją usuwasz i wtedy tylko ta klasa ma do nich dostęp

oczywiście jak chcesz je gdzieś dalej wykorzystac to możesz spłodzić funkcję w klasie, która te dane bedzie udostepniać

Użycie global $db w każdej funkcji rozwiązuje problem, ale nie podoba mi się to... w takim razie może wytłumaczę mój problem:
Mam "plik konfiguracyjny" z danymi bazy. Chciałbym aby dało się korzystać z tych danych. Tylko nie bardzo rozumiem co mi tam napisaliście. Chciałbym najlepiej jakiegoś gotowca, ale skoro u was ich nie dają to może dokładniejsze wskazówki...?

Jeżeli dobrze rozumiem o co Ci chodzi, to możesz sobie wewnątrz klasy zadeklarować połączenie z bazą danych i poruszać się w obrębie tej klasy...

Pozdro

Właśnie nie o to chodzi. Chcę, żeby klasa mogła korzystać z pliku konfiguracyjnego.

No to nie możesz zrobić klasy z metodami statycznymi odwołującymi się do konkretnych zasobów? W Kohana działa to bardzo fajnie.

yhy, to może jakoś tak:

- masz plik konfiguracyjny - tworzysz sobie klasę cConf, która odpowiada za ten plik konfiguracyjny (odczyt, zapis)
- tworzysz obiekt klasy cConf, a potem za pośrednictwem jakiejś funkcji składowej odczytujesz dane z pliku i zapisujesz do wewnętrzych zmiennych (najlepiej private, tak aby nie dało się ich edytowac z zewnątrz)
- i teraz:
1) inne klasy mogą od razu w parametrach funkcji składowych oczekiwać okreslonych danych (np. hasło do bazy)
2) inne klasy w parametrze funkcji składowej oczekują adresu obiektu klasy i dopiero potem odczytują sobie hasło do bazy

rozwiązanie nr1 wydaje się najlepsze, ponieważ inne klasy nie muszą znać w ogóle klasy od pliku konfiguracyjnego, a także bezpieczniejsze, bo nie podajemy adresu obiektu żadnej innej funkcji

Pozdrawiam

Tak, ale zauważyłem, że po stworzeniu egzemplarza klasy cconf, mamy zmienną, która nie jest dostępna w klasie x. Więc żeby była widoczna, trzeba by było uglobalnić zmienną z tym egzemplarzem (klasy cconf).
Poszperałem trochę w moich książkach do PHP. Znalazłem fajną funkcję parse_ini_file($sciezka, $obecnosc_sekcji). Funkcja przerabia normalnie plik konfiguracyjny i tworzy elegancką tablicę asocjacyjną. Nie wiem tylko, czy dobrze napisałem kod (nie chodzi o poprawność działania, ale różne wypadki, jak np. przeniesienie konfiguracji w inne miejsce - czy będzie można łatwo poprawić zmiany, czy też nie):
wycinek klasy GenericObject:

[PHP] pobierz, plaintext 
<?php
public function Initialize($table_name, $conf_res, $id=NULL) {
    $this->table_name = $table_name;
    $get_conf = parse_ini_file($conf_res, true);
    $db = $get_conf["database"];
    $this->db = new sql($db["DATABASE_HOST"], $db["DATABASE_USER"], $db["DATABASE_PASS"], $db["DATABASE_NAME"]);
    $this->id = $id;
  }
?>
[PHP] pobierz, plaintext 

klasa User:

[PHP] pobierz, plaintext 
<?php
class User extends GenericObject
{
  public function __construct($id=NULL) {
    $this->initialize("users", "config.ini", $id);
  }
}
?>
[PHP] pobierz, plaintext 

do twojego problemu najbardziej jednak pasuje rada kolegi phpion

Najprościej (choć nie najlepiej) z tego co wywnioskowałem z Twojego postu to po stworzeniu już instancji klasy bazodanowej zrobiłbym tak:

[PHP] pobierz, plaintext 
<?php
class GenericObject {
 
  private $id;
  private $table_name;
 
  private $database_fields;
  private $loaded;
  private $modified_fields;
 
  private $db_config;
 
  public function __construct() {
    global $db_config;
    $this->db_config = $db_config;
 
 }
 
}
?>
[PHP] pobierz, plaintext 

więc po stworzeniu instancji tej klasy w konstruktorze przypiszesz sobie instancję klasy bazodanowej.

Kohany wolałbym na razie nie używać. Po niezłej wpadce z Zendem, godzinie spędzonej na przepisywaniu kodu znak po znaku (przede wszystkim trudno było mi zrozumieć o co chodzi) i nie rozwiązaniu problemu, postanowiłem przez jakiś czas robić własne rozwiązania.
phpion, czy masz na myśli coś takiego?

[PHP] pobierz, plaintext 
<?php
static class Registry
{
public static $db_config = array('host', 'user', 'pass', 'db');
 
public static function getDbConf() {
 return self::db_config
}
}
?>
[PHP] pobierz, plaintext 

Przepraszam za błędy w kodzie, ale jeszcze nie bardzo wiem, jak to się robi z tymi statycznymi klasami

Generalnie tak, o to mi chodzi. Po usunięciu błędów składniowych to będzie to

tak może być?

[PHP] pobierz, plaintext 
<?php
class Config
{
private static $db_config = array("DB_CONFIG_HOST" => 'host', "DB_CONFIG_USER" => 'user', "DB_CONFIG_PASS" => 'pass', "DB_CONFIG_NAME" => 'db');
 
public static function getDbConfig(){
return self::$db_config;
}
}
?>
[PHP] pobierz, plaintext 

A jeszcze przeglądając niektóre FW zauważyłem, że używa się też stałych. Może one by się nadawały?

Stale lepiej uzywac do lokalizacji plikow/katalogow a konwencja mowi ze $db chodzi o baze i wtedy wiadomo,ja jakos wole tablice na dane do bazy.

czyli dobrze zrobiłem, czy nie? Bo trochę nie rozumiem Twojej wypowiedzi.

jak najbardziej taki kod jest poprawny, więcej nie można powiedzieć nie znając struktury aplikacji, ale sądząc po wczesniejszych wypowiedziach, taki kod bedzie ci odpowiadał