[PHP] Global in class

[Jori13]

Hallo,

Een klein probleempje


Ik heb een include file die via mysqli contact maakt met een database. Ik gebruik de OO versie. Mijn object naar de database is $mysqli.

Nou heb ik nog een class met functies. Die ziet er zo uit:

/* Hierin staat het $mysqli object */
include("config.php");

class x
{

public function x()
{
global $mysqli;
}


public function y()
{
global $mysqli;
}


}

Ik moet dus in elke functie opnieuw wijzen naar de globale variabele $mysqli. Is daar een kortere notatie voor?


Alvast bedankt, Jori.

 

[Frats]

Nee, en globals gebruiken is ook nog eens een slechte manier van programmeren. Wat je veel beter kunt doen is die verbinding meegeven in de constructor:

class X {
  public function __construct( $database ) {
    $this->database = $database;
  }
}

Dan kun je in de class verwijzen naar $this->database om je verbinding aan te spreken.

Je kunt het zelfs nog wat makkelijker maken als je extend:

class Database {
  public function __construct( $database ) {
    $this->database = $database;
  }
}

class X extends Database {
  // deze heeft nu automatisch ook de constructor van bovenstaande class
}

 

[Jori13]

Hmm, interessant.

Maar ik heb nog twee vragen.

1 - Heb je een bron waar wordt gezet dat globals slecht programmeren is?
2 - En hoe moet ik dat doen als ik ongeveer 50 variabelen heb? Die zowel binnen als buiten de class worden gebruikit?

 

[Tha Devil]

Antwoord op 1: http://c2.com/cgi/wiki?GlobalVariablesAreBad | http://stackoverflow.com/questions/484635/are-global-variables-bad

Antwoord op 2: Betere structuur bedenken, sowieso 50 variabelen die je global wilt gebruiken klinkt al als een slecht idee.

Als het configuratie-strings of integers zijn kun je die eventueel in een array stoppen om het aantal variabelen te beperken.

 

[Frats]

Configuratie kun je het beste doen via een config loader class, dan is dat ook netjes ge-encapsuleerd. Die mag het best uit een array halen verder, dat maakt niet uit, zolang de class die de config loader gebruikt daar maar niks mee te maken heeft.

Hier is er nog eentje speciaal voor PHP:
http://blog.case.edu/gps10/2006/07/22/why_global_variables_in_php_is_bad_programming_practice

 

[Jori13]

Bedankt voor jullie reacties.


Ik heb nu een class geschreven waar alle configuratie in staat. En omdat $mysqli in een aparte file staat, gebruik ik daarvoor een argument. Dus:

class x
{
		public function __construct($db)
		{
				$this->db = $db;
				$this->x = "x";
		}
}


class y extends x
{
		/* Inhoud */
}

Is dat beter? . Mochten jullie je interesseren waarom ik dit wou weten: ik ben bezig met browser spelletje en die heeft een aparte file met variabelen (www.villgrow.nl).

MVG, Jori.


PS. Globale variabelen in C (extern) zijn gewoon goed hoor

 

[Frats]

Jep, stukken beter. Globale variabelen zijn gevaarlijk en echt iets wat je jezelf af moet leren als je serieus wilt gaan programmeren.