[SOLUCIONADO] PHP OO Error con $_SESSION

[arcanisgk122]

Hola amigos tal ves no lo vea el error pero estoy recibiendo el siguiente error:

Notice: Undefined variable: _SESSION in C:\xampp\htdocs\icarosnet\phpcore\class.language.p hp on line 46

dentro de la clase: class.language tengo una funcion:

Código PHP:

function setLanguage($userLang){
        
        if(isset($_SESSION['lang'])){
        }else{
            $deteclang = $_SESSION['deteclang']; // envia el error linea 46
            if(isset($userLang)){
                if($userLang=='es'){ $langfile = 'es.inc.php'; }
                if($userLang=='en'){ $langfile = 'en.inc.php'; }
                if($userLang=='pt'){ $langfile = 'pt.inc.php'; }
                if($userLang=='fr'){ $langfile = 'fr.inc.php'; }
                $_SESSION['deteclang']  =$deteclang;
                $_SESSION["lang"]       =$langfile;
            }else{
                $lang=$deteclang;
                if($lang=='es'){ $langfile = 'es.inc.php'; }
                if($lang=='en'){ $langfile = 'en.inc.php'; }
                if($lang=='pt'){ $langfile = 'pt.inc.php'; }
                if($lang=='fr'){ $langfile = 'fr.inc.php'; }
                $_SESSION['deteclang']  =$deteclang;
                $_SESSION["lang"]       =$langfile;
            }
        }
    }
 

[ocp001a]

Pues supondría que la variable de sesión no existe, cuando haces

$deteclang = $_SESSION['deteclang']; // envia el error linea 46

estás asumiendo que $_SESSION y $_SESSION['deteclang'] existen, pero podría ser que no, aunque esto debería venir desde antes en otro código.

[arcanisgk122]

Repondiendo a eso de declarar tengo una clase que inicialisa la sesion en cada pagina:

Código PHP:

<?php
class Session{
    function iniciarSession(){
        session_name('NombredeSession');
        session_start();
        // Turn on all error reporting
        error_reporting(E_ALL);
    }
}
?>
 

y yo la llamo de la siguiente forma:

Código PHP:

require_once("class.session.php");
$sesion =   new Session();
$sesion -> iniciarSession();//iniciamos sesion
 

[arcanisgk122]

ya detecte el primer warning y lo he limpiado al parecer era un problema de orden ya que quería poner todos los require primero y en algunos se ejecutan las cosas automáticamente y no cumplía con la lógica y el orden pero ahora tengo otros inconvenientes:

Notice: Undefined variable: langfile in C:\xampp\htdocs\phpcore\class.language.php on line 62

Notice: Undefined variable: langfile in C:\xampp\htdocs\phpcore\class.language.php on line 64

Warning: include_once(/lang/.conf): failed to open stream: No such file or directory in C:\xampp\htdocs\phpcore\class.language.php on line 38

Warning: include_once(): Failed opening '/lang/.conf' for inclusion (include_path='.;C:\xampp\php\PEAR') in C:\xampp\htdocs\phpcore\class.language.php on line 38

Código PHP:

<?php
class Language{
    function detectLanguage() {
        /*if(empty($_SESSION['deteclang'])){
            $lang=substr($_SERVER["HTTP_ACCEPT_LANGUAGE"],0,2);
            $_SESSION['deteclang']=$lang;
            return $_SESSION['deteclang'];*/
 
            global $DEFAULT_LANGUAGE;
            if ($_SERVER['HTTP_ACCEPT_LANGUAGE']) {
                $this->languages = $_SERVER['HTTP_ACCEPT_LANGUAGE'];
                $this->language = substr($this->languages,0,2);
                return $this->language;
            }
            else if ($_SERVER['HTTP_USER_AGENT']) {
                $this->user_agent = explode(";" , $_SERVER['HTTP_USER_AGENT']);
 
                for ($i=0; $i < sizeof($this->user_agent); $i++) {
                    $this->languages = explode("-",$this->user_agent[$i]);
                    if (sizeof($this->languages) == 2) {
                        if (strlen(trim($this->languages[0])) == 2) {
                            $size = sizeof($this->language);
                            $this->language[$size]=trim($this->languages[0]);
                        }
                    }
                }
                return $this->language[0];
            }
            else {
                $this->language = $DEFAULT_LANGUAGE;
                return $this->language;
            }
    
        }
    function getLanguage($language = null) {
            global $LANGUAGE_DIR;
            $lang = $this->setLanguage($language);
            include_once $LANGUAGE_DIR."/".$lang.".conf"; //linea 38
            return $language = $lang;
        }
    function setLanguage($userLang){
        echo"$userLang";
        if(isset($_SESSION['lang'])){
        }else{
            $deteclang = $_SESSION['deteclang'];
            if(isset($userLang)){
                if($userLang=='es'){ $langfile = 'es.inc.php'; }
                if($userLang=='en'){ $langfile = 'en.inc.php'; }
                if($userLang=='pt'){ $langfile = 'pt.inc.php'; }
                if($userLang=='fr'){ $langfile = 'fr.inc.php'; }
                echo $lang;
                echo $langfile;
                $_SESSION['deteclang']  =$deteclang;
                $_SESSION["lang"]       =$langfile;
            }else{
                $lang=$deteclang;
                if($lang=='es'){ $langfile = 'es.inc.php'; }
                if($lang=='en'){ $langfile = 'en.inc.php'; }
                if($lang=='pt'){ $langfile = 'pt.inc.php'; }
                if($lang=='fr'){ $langfile = 'fr.inc.php'; }
                echo $lang;
                echo $langfile;   // linea 62
                $_SESSION['deteclang']  =$deteclang;
                $_SESSION["lang"]       =$langfile;   //linea 64
            }
        }
    }
    function cargaLanguage(){
        $lang = $_SESSION["lang"];
        $directorio = './lang/';
        $pathfile = $directorio.$lang;
        include_once ($pathfile);
        return $langArray;
    }
}
?>
 

no se por donde devorar esto, lo que mas me molesta es el Notice

[Italico76]

session_start() debe estar al comienzo del script y no dentro de ninguna funcion o metodo

Habia posteado algo pero mi mensaje desapareció

[arcanisgk122]

Disculpa si dudo de lo que argumentas pero por que dices que no puede estar session_start() dentro de alguna funcion o script, que yo entienda después que su llamada este al inicio del código no debe haber ningún problema:

index.php

Código PHP:

/*incluciones de codigo del core php y las clases mediate uso de constantes*/
require_once("config/index.config.php");
 
//instanciamos las clases
$sesion     =   new Session();
$language   =   new Language();
 
 
 
$sesion ->  iniciarSession();//iniciamos sesion
$lang = $language->getLanguage(@$_POST['lang']);//obtenemos lenguaje y actualisamos
 

config/index.config.php

Código PHP:

<?php
    //Directorio del nucleo de PHP
    $APP_PHPCORE_DIR=$_SERVER['DOCUMENT_ROOT'] . '/phpcore';
    //Lenguaje por defecto
    $DEFAULT_LANGUAGE = 'es';
    $LANGUAGE_DIR='/lang';
    //Cargamos todos los archivos del núcleo php que se utilizaran
    require_once $APP_PHPCORE_DIR . '/' . 'class.constants.php';
    require_once $APP_PHPCORE_DIR . '/' . SESSION_CLASS;
    require_once $APP_PHPCORE_DIR . '/' . LANGUAGE_CLASS;
    require_once $APP_PHPCORE_DIR . '/' . CHARGETPL_CLASS;
    require_once $APP_PHPCORE_DIR . '/' . MERGETPL_CLASS;
    require_once $APP_PHPCORE_DIR . '/' . PRINTTPL_CLASS;
?>
 

class.constants.php

Código PHP:

<?php
    /*definimos las Constantes de la pagina web*/
    define("SESSION_CLASS"      ,"class.session.php");
    define("LANGUAGE_CLASS"     ,"class.language.php");
    define("CHARGETPL_CLASS"    ,"class.chargTPL.php");
    define("MERGETPL_CLASS"     ,"class.mergeTPL.php");
    define("PRINTTPL_CLASS"     ,"class.printTPL.php");
?>
 

class.session.php

Código PHP:

<?php
class Session{
    function iniciarSession(){
        session_name('NombredeSession');
        session_start();
        // Turn on all error reporting
        error_reporting(E_ALL);
    }
}
?>
 

class.language.php

Código PHP:

<?php
    class Language {
        //  Method to detect user language
        function detectLanguage() {
            global $DEFAULT_LANGUAGE;
            if ($_SERVER['HTTP_ACCEPT_LANGUAGE']) {
                $this->languages = $_SERVER['HTTP_ACCEPT_LANGUAGE'];
                $this->language = substr($this->languages,0,2);
                echo "si acepta HTTP_ACCEPT_LANGUAGE<br>";
                return $this->language;
            }else{
                echo "no acepta HTTP_ACCEPT_LANGUAGE<br>";
                if ($_SERVER['HTTP_USER_AGENT']) {
                    echo "si acepta HTTP_USER_AGENT<br>";
                    $this->user_agent = explode(";" , $_SERVER['HTTP_USER_AGENT']);
                    for ($i=0; $i < sizeof($this->user_agent); $i++) {
                        $this->languages = explode("-",$this->user_agent[$i]);
                        if (sizeof($this->languages) == 2) {
                            if (strlen(trim($this->languages[0])) == 2) {
                                $size = sizeof($this->language);
                                $this->language[$size]=trim($this->languages[0]);
                            }
                        }
                    }
                    return $this->language[0];
                }else{
                    echo "no acepta HTTP_USER_AGENT<br>";
                    $this->language = $DEFAULT_LANGUAGE;
                    return $this->language;
                }
            }
        }
        //  Method to check if a language exists
        function checkLanguage($language = null) {
            global $LANGUAGE_DIR;
            echo "verificamos si existen los archivos de lenguaje <br>";
            $language = $LANGUAGE_DIR."/".$language.".inc.php";
            if (file_exists($language)){
                return TRUE;
            }else{
                return FALSE;
            }
        }
        //Method to set language
        function setLanguage($language = null){
            echo "asignacion de lenguaje <br>";
            global $DEFAULT_LANGUAGE;
            if ($language){ $_SESSION['LANGUAGE'] = $language;}
            if (@!$_SESSION['LANGUAGE']){ $_SESSION['LANGUAGE'] = $this->detectLanguage();}
            if ($this->checkLanguage($_SESSION['LANGUAGE'])) {
                $lang =  $_SESSION['LANGUAGE'];
                return $lang;
            }else{
                return $DEFAULT_LANGUAGE;
            }
        }
        //Method to get the language configuration
        function getLanguage($language = null) {
            echo "obtenemos la configuracion del lenguaje <br>";
            global $LANGUAGE_DIR;
            $lang = $this->setLanguage($language);
            include_once $LANGUAGE_DIR."/".$lang.".inc.php";
            return $language = $lang;
        }       
    }
?>
 

problema resuelto. esto es MVC y POO, si alguien tiene algún aporte o sugerencia pues la tomare en cuenta.

[Italico76]

Iniciado por arcanisgk122

Disculpa si dudo de lo que argumentas pero por que dices que no puede estar session_start() dentro de alguna funcion o script, que yo entienda después que su llamada este al inicio del código no debe haber ningún problema:

Justamente porque nunca podria estar antes que en:

Código PHP:

  <?php
 
session_start();   

Si lees el manual advertiras que cualquier error / warning etc etc puede evitar funcionen las sessiones asi que por eso debe estar al comienzo incluso delante de los include()

[arcanisgk122]

Iniciado por Italico76

Justamente porque nunca podria estar antes que en:

Código PHP:

  <?php
 
session_start();   

Si lees el manual advertirás que cualquier error / warning etc etc puede evitar funcionen las sesiones así que por eso debe estar al comienzo incluso delante de los include()

sigo sin entenderte sabes por que :

todo lo que mencionas error / warning etc etc son salidas por pantallas que ocurren al paso por el php.

en tal caso la funcion o clase que se dedica a sesiones de algún error y no permita que este funcione correctamente igual enviara por pantalla una notificación o warning... asi que no me afecta si o si hay que reparar los errores y las notificaciones...

[Italico76]

emmmmmmmmm...

session_start() no es cualquier funcion de manejo de session porque debe tener prioridad

Estas obviamente haciendo include() e instanciando clases que pueden fallar por algun motivo como Language.

Código PHP:

//instanciamos las clases
$sesion     =   new Session();
$language   =   new Language();
  
$sesion ->  iniciarSession();//iniciamos sesion
 

si o si hay que reparar los errores y las notificaciones...

En "producccion" por un simple Warning vas a impedir que tus usuarios se logueen por ejemplo ? o si tienes una tienda la dejaras completamente inutilizable ? (sin sessiones no puedes hacer nada: logueo, carrito....)


Puedes hacerlo comoo quieras (es tu codigo) pero no es buena practica

[arcanisgk122]

entiendo perfectamente lo que me dices ahora. pero no veo el por que de ser una mala practica si esta programado correctamente.
ademas respetando el modelo MVC de programacion y OOP.

se supone que debo separar el código php del HTML y no creo que repetir

Código PHP:

session_start();
 

sea una buena practica.

tampoco entiendo por que si tengo un error en una funcion que maneje sesiones inutilizara un carrito o sistema de logueo... si de por si se supone que la parte que crea la sesión esta perfecta...
admito que puede haber problemas en que no funcione bien a la hora de hacer funciones que manipulen la sesión o que demande mas esfuerzo pero no creo que hacer las cosas bien acarre algun problema...

has mirado al menos el codigo que subi???

Código PHP:

<?php
class Session{
    function iniciarSession(){
        session_name('NombredeSession');
        session_start();
        // Turn on all error reporting
        error_reporting(E_ALL);
    }
}
?>
 

en que afectaría esto un sistema de logueo??? la unica duda que me queda es que el hacer la llamada de la funcion:

Código PHP:

iniciarSession()
 

sobre escriba la sesión y borre los datos

[Italico76]

Hola amigo:

En tu ultimo post has sido muy certero en tus inquietudes, veamos porque:

Código PHP:

<?php
class Session{
    function iniciarSession(){
        session_name('NombredeSession');
        session_start();
        // Turn on all error reporting
        error_reporting(E_ALL);
    }
}
?>
 

- Creas una clase que contiene un unico metodo ?

- No acepta ningun parametro (NombredeSession por ejemplo) ? Encima toca invocarlo ? (en todo caso ese metodo seria el constructor de la clase)

- Habras leido que las clases deben tener una sola responsabilidad y estas activando errores ahi

en que afectaría esto un sistema de logueo??? la unica duda que me queda es que el hacer la llamada de la funcion:

Código PHP:

iniciarSession()
 

Lo dicho anteriormente.... esa clase es totalmente inutil..... la creas para hacer nada.....y si le incluyeras un session_destroy() en un metodo destruirSession() seria igualmente inutil..... para que encapsular funciones nativas del lenguaje ? que has agregado tu de funcionalidad o donde hay variables que deban separarse del "global scope" ? como se justiica el encapsulamiento ?

[arcanisgk122]

justifico en encapsulado para posterior edición o agregar nuevas funciones basadas en el uso de sesiones, sea procedimientos o manipulación de datos de sesiones. en tal caso si existe una mejor forma de crear una clase para iniciar, destruir y manejar sesiones me gustaría ver el aporte.

creo que se basan simplemente en lo que ven y yo estoy armándolo pensando de cara al futuro