Klasa w klasie Opcje

[mablo]

Odrazu mowie php 4.
Mam coś takiego
index.php:

[PHP] pobierz, plaintext 
<?
require("includes/initiate.php");
/* Ładowanie modółów */
include "pages/news.php";
$news = new news();
$news -> __construct();
$news -> show();
?>
[PHP] pobierz, plaintext 

news.php:

[PHP] pobierz, plaintext 
<?php
class news
{
 
  var $limit;
 
  function __construct()
  {
	  $this -> limit = "10";
  }
 
  function show ()
  {
 
	 $db = new db();
	$db -> connect( DB_HOST, DB_USERNAME, DB_PASSWORD, DB_DATABASES);
	$Smarty = new Smarty();
	$Smarty -> template_dir = 'templates/';
	$Smarty -> compile_dir = 'templates_c/';
 
	$s_SN = $db -> query ( "SELECT * FROM NEWS LIMIT ".$this -> limit.";" );
 
	$newsy = array ();
	while ($row = $db -> fetch_assoc ($s_SN))
	$newsy[] = $row; 
 
	$Smarty -> assign("news", $newsy);
	$Smarty -> display('news.tpl');
  }
}
?>
[PHP] pobierz, plaintext 

initiate.php:

[PHP] pobierz, plaintext 
<?php
require( "libs/db.class.php" );
require( "libs/Smarty.class.php" );
require( "config.php" );
 
$db = new db();
$db -> connect( DB_HOST, DB_USERNAME, DB_PASSWORD, DB_DATABASES);
 
$Smarty = new Smarty();
$Smarty -> template_dir = 'templates/';
$Smarty -> compile_dir = 'templates_c/';
?>
[PHP] pobierz, plaintext 

Czy moge jakoś ominąc odpalanie klas db i smarty w funkcji show (news.php) (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) Żeby odpalały się w pliku initiate.php bo dodawanie odpalania klas smarty i db za każdym razem jest troche uciążliwe.

[sobstel]

widze 2 sposoby

1. global $db
2. wzorzec singleton

[nrm]

1. swoją droga piszesz php4 a podajesz __construct(); ?!?

2. możesz przekazać odpalona klase db do klasy news

np.

$news = new news(&$db);

a w klasie news w konstruktorze (w php4 o nazwie jak klasa)
robisz $this->db = &$db;

tak się kiedyś nauczyłem i tak mi zostało. nie wiem czy to jest prawidłowe postepowanie. (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[Ociu]

[PHP] pobierz, plaintext 
<?php
# php5
public function __contruct(db $db) {
$this->db = $db;
}
 
# php4
 
function news(&$db) {
$this->db = $db;
}
 
#index.php
$news = new news(new db());
?>
[PHP] pobierz, plaintext 

pozdrawiam

[mablo]

Nie wiem co robie źle ale nie działa.

Warning: Call-time pass-by-reference has been deprecated - argument passed by value; If you would like to pass it by reference, modify the declaration of [runtime function name](). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file. However, future versions may not support this any longer. in C:\php\www\cms\news\index.php on line 28

Czemu __construct (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) poniewaz skrypt ma działać w php 4 i 5. A w 5 jest uławienie bo nie trzeba ładowć tej funkcji.

[dr_bonzo]

Uzyj standardowego rozwiazania: http://pl.php.net/manual/en/language.oop5.decon.php

For backwards compatibility, if php 5 cannot find a __construct() function for a given class, it will search for the old-style constructor function, by the name of the class. Effectively, it means that the only case that would have compatibility issues is if the class had a method named __construct() which was used for different semantics.

---
Co do bledu -- ktora to linia?, i podaj naglowek funkcji ktora wywolujesz.

Ten post edytował dr_bonzo 5.12.2005, 19:01:30

[mablo]

[PHP] pobierz, plaintext 
<?php
$news = new news(&$db);
?>
[PHP] pobierz, plaintext 

to jest linia 28.
A do news.php dodałem

[PHP] pobierz, plaintext 
<?php
function __construct()
  {
	  $this -> limit = "10";
	  $this -> db = &$db;
  }
?>
[PHP] pobierz, plaintext 

Ten post edytował mablo 5.12.2005, 19:38:21

[dr_bonzo]

Zeby uproscic sprawe (wszystkie bledy poprawie od razu):

[PHP] pobierz, plaintext 
<?php
 
class news
{
	var $limit;
	var $db;
	function news( &$db ) // tutaj dajesz '&' a nie przy wywplaniu
	/*
	* ten sposob tworzenia konstruktora jest zgodny z php4 i php5
	i zrozumialy dla wszystkich ktorzy przeczytali ten rozdzial manuala
	gdy robisz:
 
	$news = new news();
	$news->__construct();
 
	nie dosc ze dziwnie wyglada to jest niezrozumiale
 
 
	* musisz podac argumenty funkcji, a nie taK:
	function __construct()
	{
		$this->limit = "10";
		$this->db = &$db; // skad wzialzes $db
	}
	*/
	{
		$this -> limit = "10";
		$this->db = &$db;
	}
}
 
 
$news = new news($db); 
// ^^^ For backwards compatibility...
// tu nie podajesz '&' tlyko sama zmienna, & dodajesz w definicji funkcji
?>
[PHP] pobierz, plaintext 

[mablo]

Dzieki już zrozumiałem oco chodzi i wszystko sobie poprawiłem i działa.