[SOLUCIONADO] Problema con ambito de variables

[quico5]

Tres variables globales

Código PHP:

  $argumentos = isset($_GET['url']) ? explode('/', $_GET['url']) : array() ;
$controlador = ($controlador = strtolower(array_shift($argumentos))) ? $controlador : 'index' ;
$metodo = ($metodo = strtolower(array_shift($argumentos))) ? $metodo : 'index' ; 
   

Código PHP:

  echo $controlador; //OK
echo $metodo; //OK
echo $argumentos; //OK

class miClase {
//aquí no llegan las variables globales
echo $controlador; //Notice: Undefined variable
echo $metodo; //Notice: Undefined variable
echo $argumentos; //Notice: Undefined variable
} 
   

[Italico76]

Lee sobre $GLOBALS y global

http://www.php.net//manual/es/langua...bles.scope.php

[quico5]

Gracias italico76, ya estuviera en php.net y ya probara estos dos ejemplos

Código PHP:

  <?php
$a = 1;
$b = 2;

function Suma()
{
    $GLOBALS['b'] = $GLOBALS['a'] + $GLOBALS['b'];
}

Suma();
echo $b;
?>

<?php
$a = 1;
$b = 2;

function Suma2()
{
    global $a, $b;

    $b = $a + $b;
}

Suma2();
echo $b;
?>   

La cuestión es la siguiente

Código PHP:

  <?php
//Creo las 3 variables
$argumentos = isset($_GET['url']) ? explode('/', $_GET['url']) : array() ;
$controlador = ($controlador = strtolower(array_shift($argumentos))) ? $controlador : 'index' ;
$metodo = ($metodo = strtolower(array_shift($argumentos))) ? $metodo : 'index' ; 

class miClase {
public function __construct() {
//Recupero las 3 variables
echo $controlador; //Notice: Undefined variable
echo $metodo; //Notice: Undefined variable
echo $argumentos; //Notice: Undefined variable
}
}
new miClase();
?>   

[quico5]

Este fragmento funciona

Código PHP:

      class Request {
        public function __construct() {
            $this->argumentos = isset($_GET['url']) ? explode('/', $_GET['url']) : array() ;
            $this->controlador = ($this->controlador = strtolower(array_shift($this->argumentos))) ? $this->controlador : 'index' ;
            $this->metodo = ($this->metodo = strtolower(array_shift($this->argumentos))) ? $this->metodo : 'index' ;
        }
        public function controlador() {return $this->controlador;}
        public function metodo() {return $this->metodo;}
        public function argumentos() {return $this->argumentos;}
    }
    
    class Bootsrap {
        public function run(Request $p) {
            echo $p->controlador;
        }
    }
    Bootsrap::run(new Request); 
   

Pero como se va ejecutar siempre me gustaría dejarlo de la siguiente forma

Código PHP:

  <?php
//Creo las 3 variables
global $controlador, $metodo, $argumentos;
$argumentos = isset($_GET['url']) ? explode('/', $_GET['url']) : array() ;
$controlador = ($controlador = strtolower(array_shift($argumentos))) ? $controlador : 'index' ;
$metodo = ($metodo = strtolower(array_shift($argumentos))) ? $metodo : 'index' ; 

class miClase {
public function __construct() {
//Recupero las 3 variables
echo $controlador; //Notice: Undefined variable
echo $metodo; //Notice: Undefined variable
echo $argumentos; //Notice: Undefined variable
}
}
new miClase();
?>   

[quico5]

Supongo que tendré que ver la forma de que reciba las variables

Código PHP:

  public function __construct($controlador = 'a', $metodo = 'b', $argumentos = 'c') 
   

Código PHP:

      //Creo las 3 variables
    $argumentos = isset($_GET['url']) ? explode('/', $_GET['url']) : array() ;
    $controlador = ($controlador = strtolower(array_shift($argumentos))) ? $controlador : 'index' ;
    $metodo = ($metodo = strtolower(array_shift($argumentos))) ? $metodo : 'index' ; 
 
    class miClase {
        public function __construct($controlador = 'a', $metodo = 'b', $argumentos = 'c') {
            //Recupero las 3 variables
            echo $controlador; //Notice: Undefined variable
            echo $metodo; //Notice: Undefined variable
            echo $argumentos; //Notice: Undefined variable
        }
    }
    new miClase(); 
   

[quico5]

Creo que ya estaria

Código PHP:

      //Creo las 3 variables
    $argumentos = isset($_GET['url']) ? explode('/', $_GET['url']) : array() ;
    $controlador = ($controlador = strtolower(array_shift($argumentos))) ? $controlador : 'index' ;
    $metodo = ($metodo = strtolower(array_shift($argumentos))) ? $metodo : 'index' ; 

    class miClase {
        public function __construct($controlador, $metodo, $argumentos) {
            //Recupero las 3 variables
            echo $controlador; //OK
            echo $metodo; //OK
            print_r($argumentos); //OK
        }
    }
    new miClase($controlador, $metodo, $argumentos); 
   

[Italico76]

Si, MUCHO mejor si le pasas esos parametros al constructor

[hhs]

Quico5 tu ultima aproximación es la correcta, recuerda que te comunicas con los objetos mediante sus métodos. Es por eso que no debes de hacer uso de GLOBALS, además rompe con el principio de encapsulación y oculta dependencias ya que la firma de tu constructor no refleja lo que necesita la clase para su creación. Otro punto es que uses parámetros opcionales ya que por lo menos el argumento $argumentos puede o no estar presente.

[Italico76]

Ahora para no embarrarla dentro de tu clase....... consulta el tema de "visibilidad" de propiedades y metodos.

[hhs]

Exacto también es importante ocultar información relacionada con tu objeto como bien comenta Italico76.