Discussion :

[scamphp]

Bonjour

Je voulais faire une petite class type router et utiliser $_SERVER['REQUEST_URI'].

Si je fais :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
class Routeur {
 
private static $url = $_SERVER['REQUEST_URI'];
	public function routeur_index() {
		var_dump(self::$url);
		}
}

Cela ne fonctionne pas, cela ne donne rien
alors que dans mon fichier index si je défini une constante

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

define( 'URL' , $_SERVER['REQUEST_URI'] );

et que je fais

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
class Routeur {
 
private static $url = URL;
	public function routeur_index() {
		var_dump(self::$url);
		}
}

Cela fonctionne.

Avez vous une explication...

[rawsrc]

Salut,

poste le code du test et l'url qui t'a amené à la conclusion que "Cela ne fonctionne pas, cela ne donne rien"

[scamphp]

Pour le moment c'est tout simple je détecte si c'est la page home.

Sur ma page index.php
Logique_Routeur::routeur_instance();

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
class Routeur {
 
	private static $_instance;
 
	private static $url = $_SERVER['REQUEST_URI'];
 
 
	public static function routeur_instance() { 
		if ( is_null( self::$_instance ) ) { 
			self::$_instance = new Routeur(); 
			self::$_instance->index();
		}
		return self::$_instance; 
	}
 
	public function index() {
		if ( self::$url === '/' ) {
			echo ('c\'est la home page'); 
		}
		else {
			echo ('c\'est autre chose'); 		
		}
	}
 
}

[rawsrc]

Comme tu débutes, je te conseille de ne pas utiliser le monde static, c'est rajouter une couche de complexité alors qu'en l'état ce n'est pas nécessaire.
Ensuite, le codage d'un routeur n'est pas chose aisée. T'es sûr de vouloir t'embarquer là-dedans malgré tout ?
Si oui, le mieux que tu puisses faire c'est de t'inspirer de ce qui a déjà été fait afin de bien comprendre les mécanismes sous-jacents ce qui te permettra de les adapter à ta sauce.
Bref, t'as du pain sur la planche.

[scamphp]

Si la variable

private static $url = $_SERVER['REQUEST_URI'];

n'est pas en static

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

private $url = $_SERVER['REQUEST_URI'];

Je ne peux pas appeler $url, sauf a la créer dans le constructeur, mais du coup je me retrouve avec un argument $this->url,

et dans l'idée je ne voulais pas avoir d'argument $this->url inutile (hors de la fonction) qui vienne surcharger mon objet routeur.

[rawsrc]

tu devrais te pencher sur les norme PSR.
Ensuite ton routeur va être bancal, tu devrais être capable de lui passer n'importe quoi à analyser et pas juste $_SERVER['REQUEST_URI']. D'où l'utilité de passer cela en paramètre à une fonction dispatch() par exemple au lieu de lui figer cette valeur...

Je te le redis, essaie de bien comprendre ce qui a été déjà fait en la matière et inspire toi en.
Tu dois aussi monter en premier ton niveau en programmation objet afin de ne pas cumuler les difficultés.

[scamphp]

Merci, pour les réponses, je viens de voir que généralement il faut éviter les fonction et variables statiques.... sauf pour faire un design pattern (singleton)

Bon l'idée c'était de faire un singleton au niveau de mon OBJET router car un router ne doit être instancier qu'une seule fois, je ne dois pas avoir plusieurs objets router.

Je viens de tomber sur d'autre tuto sur du POO 100%... ils expliquent que l'on ne doit avoir que des objets qui doivent communiquer entre eux...

du coup j'ai 2 questions

1°) Quand on dit que les objets doivent communiquer entre eux cela veux dire que l'on passe les objets d'un objet à l'autre (injection de dépendance) style je passe mon Objet Router à mon Objet Controller... ou il faut créer une class intermédiaire sorte de jointure qui fait le lien entre les objets...

2°) Static ou pas, pour évoluer et comprend de mes erreurs
Je ne comprend pas l'erreur de mécanisme qui fait que

private static $url = URL;

fonctionne et pas

private static $url = $_SERVER['REQUEST_URI'];

[rawsrc]

tiens regarde mon blog, la problématique y est abordée : ici
puis regarde le second billet qui lui va t'apporter une approche de solution pour la problématique du routeur

Le souci quand tu fais : private static $url = $_SERVER['REQUEST_URI']; est que le tableau $_SERVER n'est disponible qu'à l'exécution (runtime) et que dans ta classe, l'utilisation sous forme de propriété nécessite que cette information soit disponible à la compilation (compile time). D'où l'erreur que tu ramasses.
Tu peux procéder comme ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
class Foo
{
    private static $uri = '';
 
    public function __construct()
    {
        self::$uri = $_SERVER['REQUEST_URI'];
    }
}

[scamphp]

Super merci problème résolu et un apprentissage de plus merci