PHP OO Problema al obtener el ultimo id insertado(AYUDENME POR FAVOR)

[aesu]

Saludos a todos,
Estoy apenas comenzando a implementar las funciones de mysqli, ya que como se sabe estan declaradas obsoletas las funciones de mysql. Hasta cierto punto todo estaba muy bien pero cuando llego el momento de utilizar un metodo que creé para obtener el ultimo id insertado me arroja cero; he intentado solucionarlo de mil maneras pero sigo con el mismo problema.
Si alguien puede ayudarme se los agradezco, es que tengo poco tiempo para entregar la aplicacion y esto me tiene atrazado.

Aca les dejo el codigo.
Ahhh por cierto la aplicacion espara carga de cv, creo el usuario y necesito el último id de usuario ingresado para el resto de la información que se requiere.

Codigo



Clase base de datos

Código PHP:

<?php
 
abstract class AppDataBaseModel{
 
    private static $THE_HOST        = _APP_HOST_NAME_;
    private static $THE_USER        = _APP_HOST_USER_;
    private static $THE_PASSWORD    = _APP_HOST_PASSWORD_;
    private static $THE_DATABASE    = _APP_HOST_DATABASE_;
    private $dbConnection;
    private $lastID;
    
 
    protected function openConnection(){
        $this->dbConnection = @new mysqli(self::$THE_HOST,
                                          self::$THE_USER,
                                          self::$THE_PASSWORD,
                                          self::$THE_DATABASE);
        if(mysqli_connect_errno()){
            echo'Ha ocurrido un error al conectar con el servidor..!';
        }
        
    }
    private function closeConnection(){
        $this->dbConnection->close();
        //$this->dbConnection->free();
    }
    protected function executeQuery($mysqlQuery){
        $this->openConnection();    
        $queryResult = $this->dbConnection->query($mysqlQuery);
        if(!$queryResult){
            echo'Ha ocurrido un error al ejecutarse la consulta a la base de datos..!'.$this->dbConnection->error;
        }else{
            return $queryResult;
        }       
        $this->closeConnection();
    }   
    protected function countRows($mysqliQueryResult){
        $this->openConnection();
        $queryResult = $mysqliQueryResult->num_rows;
        if($queryResult){
            return $queryResult;
        }else{
            echo'Ha ocurrido un error al contar los registros de la tabla..!';
        }
        $this->closeConnection();
    }   
    protected function getLastID($theTable,$theId){
        $this->openConnection();
        $this->lastID = ''.$theId.'';     
        $queryResult = $this->executeQuery("SELECT LAST_INSERT_ID() AS ".$this->lastID." FROM ".$theTable.";");
        if($queryResult){
            return $queryResult;
        }else{
            echo 'Ha ocurrido un error al intentar buscar el ultimo id insertado..!'.$this->dbConnection->error;
        }
        //$this->closeConnection();
    }
 
        protected function escapeSqlString($fieldName){
        $this->openConnection();
        $queryResult = $this->dbConnection->real_escape_string($fieldName);
        if($queryResult){
            return $queryResult;
        }else{
            echo $this->dbConnection->error;
        }
        $this->closeConnection();
    }
    
 
    protected function mysqliFetchObject($param){
        $this->openConnection();
        $queryResult = $param->fetch_object();
        return $queryResult;
    }
    
    protected function mysqliFetchAssoc($param){
        $this->openConnection();
        $getResult = $param->fetch_assoc();
        return $getResult;
    }
    
    LENGUAJE EN CASO DE QUE CAMBIE EN EL FUTURO
    protected function mysqliFetchArray($param){
        $this->openConnection();
        $getResult = $param->fetch_array();
        return $getResult;
    } 
}
 
?>
 

Clase que maneja la información información del usuario

Código PHP:

<?php
class UserPersonalInfo extends AppDataBaseModel{
 
    private $cinfoId;
    private $infoUserId;
    private $userInfoRif;
    private $userInfoPassport;
    
    private $userInfoMaritalStatus;
    private $userInfoBirthdate;
    private $userInfoAge;
 
    private $userInfoLicenseType;
    private $userInfoState;
    private $userInfoCity;
    
    private $userInfoAddress;
    
    private $userInfoCountryCode;
    private $userInfoStateCode;
    private $userInfoPhoneNumber;
    
    private $userInfoCountryCode2;
    private $userInfoStateCode2;
    private $userInfoPhoneNumber2;
    
    private $tableName = _TABLE_USERS_INFO_;
    private $tableName2 = _TABLE_USERS_;
    private $tableName3 = _TABLE_USERS_LEVEL_ACCESS_;
    private $theId ='i_users_personal_info_id';
    private $userId ="u_user_id";
    private $theUserId ='i_users_personal_info_user_id';
    private $orderBy ='i_users_personal_info_id';
    
    public function __construct($infoId=0,$infoUserId=0,$userInfoRif="",$userInfoPassport="",$userInfoMaritalStatus="",$userInfoBirthdate="",$userInfoAge="",$userInfoLicenseType="",$userInfoState="",$userInfoCity="",$userInfoAddress="",$userInfoCountryCode="",$userInfoStateCode="",$userInfoPhoneNumber="",$userInfoCountryCode2="",$userInfoStateCode2="",$userInfoPhoneNumber2=""){
 
        $this->infoId = $infoId;
        $this->infoUserId = $infoUserId;
        $this->userInfoRif = $userInfoRif;
        $this->userInfoPassport = $userInfoPassport;
        
        $this->userInfoMaritalStatus = $userInfoMaritalStatus;
        $this->userInfoBirthdate = $userInfoBirthdate;
        $this->userInfoAge = $userInfoAge;
    
        $this->userInfoLicenseType = $userInfoLicenseType;
        
        $this->userInfoState = $userInfoState;
        $this->userInfoCity = $userInfoCity;
        $this->userInfoAddress = $userInfoAddress;
        
        $this->userInfoCountryCode = $userInfoCountryCode;
        $this->userInfoStateCode = $userInfoStateCode;
        $this->userInfoPhoneNumber = $userInfoPhoneNumber;
        
        $this->userInfoCountryCode2 = $userInfoCountryCode2;
        $this->userInfoStateCode2 = $userInfoStateCode2;
        $this->userInfoPhoneNumber2 = $userInfoPhoneNumber2;
    }//FIN DEL CONSTRUCTOR
    
    public function save(){
        //AQUI ES DONDE LLAMO EL METODO GENERICO DE LA CLASE BASE DE DATOS PARA OBTENER EL ULTIMO ID
        $resultQuery = $this->getLastID($this->tableName2,$this->userId);
        $last_user_id = $this->mysqliFetchObject($resultQuery);
        $this->userId = $last_user_id->u_user_id;
        //LE COLOCO ESTE ECHO PARA VER SI ERROJA CERO O EL ID
                echo '<H1>ID - </H1>'.$this->userId;
 
        $getResult = $this->executeQuery("INSERT INTO ".$this->tableName."(i_users_personal_info_id,
                                                                           i_users_personal_info_user_id,
                                                                           i_users_personal_info_rif,
                                                                           i_users_personal_info_passport,
                                                                           i_users_personal_info_marital_status,
                                                                           i_users_personal_info_birthdate,
                                                                           i_users_personal_info_age,
                                                                           i_users_personal_info_driving_license_type,
                                                                           i_users_personal_info_state,
                                                                           i_users_personal_info_city,
                                                                           i_users_personal_info_address,
                                                                           i_users_personal_info_phone_country_code,
                                                                           i_users_personal_info_phone_state_code,                                          
                                                                           i_users_personal_info_phone_number,                                          
                                                                           i_users_personal_info_phone_country_code2,                                           
                                                                           i_users_personal_info_phone_state_code2,
                                                                           i_users_personal_info_phone_number2)
                                            VALUES  ('',
                                               '".$this->escapeSqlString($this->userId)."',
                                               '".$this->escapeSqlString($this->userInfoRif)."',
                                               '".$this->escapeSqlString($this->userInfoPassport)."',
                                               '".$this->escapeSqlString($this->userInfoMaritalStatus)."',
                                               '".$this->escapeSqlString($this->userInfoBirthdate)."',
                                               '".$this->escapeSqlString($this->userInfoAge)."',
                                               '".$this->escapeSqlString($this->userInfoLicenseType)."',
                                               '".$this->escapeSqlString($this->userInfoState)."',
                                               '".$this->escapeSqlString($this->userInfoCity)."',
                                               '".$this->escapeSqlString($this->userInfoAddress)."',
                                               '".$this->escapeSqlString($this->userInfoCountryCode)."',
                                               '".$this->escapeSqlString($this->userInfoStateCode)."',
                                               '".$this->escapeSqlString($this->userInfoPhoneNumber)."',
                                               '".$this->escapeSqlString($this->userInfoCountryCode2)."',
                                               '".$this->escapeSqlString($this->userInfoStateCode2)."',
                                               '".$this->escapeSqlString($this->userInfoPhoneNumber2)."');");       
        return $getResult;
    }//FIN DEL METODO SAVE
    
}//FIN DE LA CLASE
 
?>
 

[hhs]

Alguna razon para no usar en tu clase mysqli::$insert_id ?
http://www.php.net/manual/en/mysqli.insert-id.php

[aesu]

Por que habia leido en versiones anteriores del manual de referencia de php.net
que LAST_INSERT_ID() era mas seguro, por eso me quedé susandolo, como voy comenzando con la extension mysqli
no se si usarlo aun.

Si puedes ayudarme a resolver el problema te lo agradezco.

[alex1084]

Creo que puedes leer esto, espero te sirva

http://www.php.net/manual/es/mysqli.insert-id.php

Creo es lo que buscas

[aesu]

Pero asi como lo hice tambien deberia arrojar el ultimo id..! no entiendo porque no lo hace, en eso es que quiero que me ayuden..!

[Reedyseth]

Por lo pronto creo que deberias de cambiar un poco tu clase, estas creando una instancia de conexión cada que quieres hacer una acción, mejor crea una variable y asignale la instancia solo una vez, despues solo accede a la variable y crea metodos para cerrar la conexion y el cursor.

Esto es para mejorar tu código. Dejar ahora reviso por que estas batallando !!

[aesu]

ok haré eso,pero aun no veo una solución a mi problema, si puedes darme una te lo agradezco.

[GatorV]

Pues de hecho ese es el problema principal, el último ID es por conexión, si cierras y abres por eso pierdes el ID...

[Reedyseth]

Ps entonces ahí esta jejeje, digo, primero hay que ir por los mas básico, ahí es donde generalmente se nos dan los problemas !!!

[aesu]

Iniciado por GatorV

Pues de hecho ese es el problema principal, el último ID es por conexión, si cierras y abres por eso pierdes el ID...

Bueno intenté dejando la conexión abierta y aún continúa el problema, Esto me tiene ya me tiene loco, ayuda por favor..!

Con la extensión mysql trabaja perfecto.

[GatorV]

Ya veo, la función LAST_INSERT_ID() te devuelve el último ID autogenerado, no creo que este disponible siempre, solo esta disponible si tu columna tiene un AUTO_INCREMENT definido, revisa eso.

Ahora si la columna es AUTO_INCREMENT no tienes porque recuperarlo antes e insertarlo, se genera solito y al final solo debes de usar $mysqli->insert_id para obtenerlo.

Saludos.

[aesu]

Iniciado por GatorV

Ya veo, la función LAST_INSERT_ID() te devuelve el último ID autogenerado, no creo que este disponible siempre, solo esta disponible si tu columna tiene un AUTO_INCREMENT definido, revisa eso.

Ahora si la columna es AUTO_INCREMENT no tienes porque recuperarlo antes e insertarlo, se genera solito y al final solo debes de usar $mysqli->insert_id para obtenerlo.

Saludos.

De hecho la columna si es AUTO_INCREMENT y nolo estoy poniendo antes de insertarlo, lo estoy llamando cuando ya se ha generado el ultimo id de usuario si te fijas donde lo llamo es en la clase UserPersonalInfo, que maneja otra información del usuario.

De momento sigo trabajando conla extensión mysql hasta que tenga resuelto este problema por eso es que tengo la clase db de esa manera, para que sea compatible con ambas extensiones, ese es el único problemita.! Ayudame por favor..!

[GatorV]

Pero en tu clase UserPersonalInfo en el método save() estas llamando al Insert ID antes de hacer el INSERT, es lo que te estaba diciendo...

[Reedyseth]

@aesu Nos compartiste las dos clases que estas usando, pero también podrías compartir como es que la estas usando ??

[aesu]

bueno en realidad son tres clases...
-AppDataBaseModel
-User
-UserPersonalInfo

Esta es la otra la cual agrega el usuario y después de agregado es que obtengo el último id insertado

Código PHP:

<?php
 
class User extends AppDataBaseModel{
    private $userName;
    private $userCountryId;
    private $userLastName;
    private $userIdnumberNationality;
    private $userIdnumber;
    private $userMail;
    private $userLogin;
    private $userPassword;
    private $userRegDate;
    private $userEditedDate;
    private $userLastVisit;
    private $userActivated;
    private $userStatus;
    private $userLevelAccess;
    private $userTerms;
    
    private $tableName = _TABLE_USERS_;
    private $tableName2 = _TABLE_USERS_LEVEL_ACCESS_;
    private $tableName3 = _TABLE_USERS_INFO_;
    private $theId ='u_user_id';
    private $orderBy ='u_user_id';
    
    //DECLARAMOS EL CONSTRUCTOR DE LA CLASE EL CUAL SE LLAMA AL MOMENTO DE INSTANCIAR LA CLASE Y RECIBE LOS DATOS A TRAVES DE SUS PARAMETROS
    public function __construct($userId=0,$userCountryId=0,$userName="",$userLastName="",$userIdnumberNationality="",$userIdnumber="",$userMail="",$userLogin="",$userPassword="",$userRegDate=0,$userEditedDate=0,$userLastVisit=0,$userActivated="",$userStatus="",$userLevelAccess=0,$userTerms=0){
 
    
        $this->userName = $userName;
        $this->userCountryId = $userCountryId;
        $this->userLastName = $userLastName;
        $this->userIdnumberNationality = $userIdnumberNationality;
        $this->userIdnumber = $userIdnumber;
        $this->userMail = $userMail;
        $this->userLogin = $userLogin;
        $this->userPassword = $userPassword;
        $this->userRegDate = $userRegDate;
        $this->userEditedDate = $userRegDate;
        $this->userLastVisit = $userLastVisit;
        $this->userActivated = $userActivated;
        $this->userStatus = $userStatus;
        $this->userLevelAccess = $userLevelAccess;
    }//FIN DEL CONSTRUCTOR
    
    public function save(){
    
        $getResult = $this->executeQuery("INSERT INTO ".$this->tableName."(u_user_id,
                                               u_user_country_id,
                                               u_user_name,
                                               u_user_lastname,
                                               u_user_id_number_nationality,
                                               u_user_id_number,
                                               u_user_mail,
                                               u_user_login,
                                               u_user_password,
                                               u_user_reg_date,
                                               u_user_edited_date,
                                               u_user_last_visit,
                                               u_user_activated,
                                               u_user_status,
                                               u_user_level_access,
                                               u_user_policy)
                                            VALUES  ('',
                                               '".$this->userCountryId."',
                                               '".$this->escapeSqlString($this->userName)."',
                                               '".$this->escapeSqlString($this->userLastName)."',
                                               '".$this->escapeSqlString($this->userIdnumberNationality)."',
                                               '".$this->escapeSqlString($this->userIdnumber)."',
                                               '".$this->escapeSqlString($this->userMail)."',
                                               '".$this->escapeSqlString($this->userLogin)."',
                                               '".$this->escapeSqlString(md5($this->userPassword))."',
                                               NOW(),
                                               NOW(),
                                               '',
                                               '".$this->userActivated."',
                                               '".$this->userStatus."',
                                               '".$this->userLevelAccess."',
                                               '".$this->userTerms."');");  
                        
        return $getResult;
    }//FIN DEL METODO SAVE
    
 
}//FIN DE LA CLASE
 
?>
 

[aesu]

Así es como las estoy usando:

Obtengo los datos de los formularios con este archivo

Código PHP:

<?php
 
if(isset($_REQUEST['task'])){
    
 
    $task = $_REQUEST['task'];
    
    //DATOS DE ACCESO A LA CUENTA DEL USUARIO//
    if(isset($_SESSION['loged_user'])){$userSession = safeString($_SESSION['loged_user']);}else{$userSession=NULL;}
    if(isset($_SESSION['level_user'])){$userLevel = safeString($_SESSION['level_user']);}else{$userLevel=NULL;}
    if(isset($_REQUEST['name'])){$name = safeString($_REQUEST['name']);}else{$name=NULL;}
    if(isset($_REQUEST['lastname'])){$lastName = safeString($_REQUEST['lastname']);}else{$lastName=NULL;}
    if(isset($_REQUEST['idnumber_nationality'])){$userIdnumberNationality = safeString($_REQUEST['idnumber_nationality']);}else{$userIdnumberNationality = NULL;}
    if(isset($_REQUEST['idnumber'])){$idNumber = safeString($_REQUEST['idnumber']);}else{$idNumber=NULL;}
    if(isset($_REQUEST['email'])&&$_REQUEST['email']!==NULL){$userName=safeString($_REQUEST['email']);}else{$userName=NULL;}
    if(isset($_REQUEST['email'])){$userMail = safeString($_REQUEST['email']);}else{$userMail=NULL;}
 
    if(isset($_REQUEST['password'])){$passWord = safeString($_REQUEST['password']);}else{$passWord = NULL;}
    if(isset($_REQUEST['confirmpassword'])){$confirmPassWord=safeString($_REQUEST['confirmpassword']);}else{$confirmPassWord=NULL;}
    if(isset($_REQUEST['status'])){$userStatus = safeString($_REQUEST['status']);}else{$userStatus = NULL;}
    if(isset($_REQUEST['activate'])){$activate = safeString($_REQUEST['activate']);}else{$activate = '0';}
    if(isset($_REQUEST['level'])){$userLevelAccess = $_REQUEST['level'];}else{$userLevelAccess = '4';}
 
    //DATOS DE LA INFORMACION PERSONAL DEL USUARIO//
    if(isset($_REQUEST['infoId'])){$infoId = $_REQUEST['infoId'];}else{$infoId = '';}
    if(isset($_REQUEST['rif'])){$userInfoRif = $_REQUEST['rif'];}else{$userInfoRif = '';}
    if(isset($_REQUEST['passport'])){$userInfoPassport = $_REQUEST['passport'];}else{$userInfoPassport = '';}
    if(isset($_REQUEST['marital_status'])){$userInfoMaritalStatus = $_REQUEST['marital_status'];}else{$userInfoMaritalStatus = '';}
    if(isset($_REQUEST['birthday'])){$userInfoBirthdate = $_REQUEST['birthday'];}else{$userInfoBirthdate = '';}
    if(isset($_REQUEST['age'])){$userInfoAge = $_REQUEST['age'];}else{$userInfoAge = '';}
    if(isset($_REQUEST['license_grade'])){$userInfoLicenseType = $_REQUEST['license_grade'];}else{$userInfoLicenseType = '';}
    if(isset($_REQUEST['state'])){$userInfoState = $_REQUEST['state'];}else{$userInfoState = '';}
    if(isset($_REQUEST['city'])){$userInfoCity = $_REQUEST['city'];}else{$userInfoCity = '';}
    
    
    if(isset($_REQUEST['address'])){$userInfoAddress = safeString($_REQUEST['address']);}else{$userInfoAddress = NULL;}
    
    if(isset($_REQUEST['phone_country_code'])){$userInfoCountryCode = safeString($_REQUEST['phone_country_code']);}else{$userInfoCountryCode = NULL;}
    if(isset($_REQUEST['phone_code_area'])){$userInfoStateCode = safeString($_REQUEST['phone_code_area']);}else{$userInfoStateCode = NULL;}
    if(isset($_REQUEST['phone'])){$userInfoPhoneNumber = safeString($_REQUEST['phone']);}else{$userInfoPhoneNumber = NULL;}
 
    if(isset($_REQUEST['phone_country_code2'])){$userInfoCountryCode2 = safeString($_REQUEST['phone_country_code2']);}else{$userInfoCountryCode2 = NULL;}
    if(isset($_REQUEST['phone_code_area2'])){$userInfoStateCode2 = safeString($_REQUEST['phone_code_area2']);}else{$userInfoStateCode2 = NULL;}
    if(isset($_REQUEST['phone2'])){ $userInfoPhoneNumber2 = safeString($_REQUEST['phone2']);}else{$userInfoPhoneNumber2 = NULL;}
 
 
    
 
    $obj = new User($id,$countryId,$name,$lastName,$userIdnumberNationality,$idNumber,$userMail,$userName,$passWord,'','','',$activate,$userStatus,$userLevelAccess,$userTerms);
    
    $objUI = new UserPersonalInfo($infoId,$id,$userInfoRif,$userInfoPassport,$userInfoMaritalStatus,$userInfoBirthdate,$userInfoAge,$userInfoLicenseType,$userInfoState,$userInfoCity,$userInfoAddress,$userInfoCountryCode,$userInfoStateCode,$userInfoPhoneNumber,$userInfoCountryCode2,$userInfoStateCode2,$userInfoPhoneNumber2);
 
switch ($task){
            case 'add':
                include_once'views/app_form_users.php';
            break;
            
            case 'save':
                $triggerAlert=NULL;
                $checkMailExist = $obj->checkUserFieldAvalibility('u_user_mail',$userMail);
                if($name==NULL){$triggerAlert='Escriba el nombre</br>';}
                if($lastName==NULL){$triggerAlert.='Escriba el apellido</br>';}
                if($userMail==NULL){$triggerAlert.='Escriba el correo.</br>';}
                //if($checkMailExist==true){$triggerAlert.='El correo que intenta ingresar ya esta registrado, no es posible ingresarlo de nuevo..!<br>';}
                if(check_email($userMail)==false){$triggerAlert.='El correo no es valido..!<br>';}
                if($passWord==NULL){$triggerAlert.='Escriba la clave.</br>';}
                if($confirmPassWord==NULL){$triggerAlert.='Escriba la clave de confirmaci&oacute;n.</br>';}
                if($passWord!==$confirmPassWord){$triggerAlert.='La claves no coinciden.</br>';}
                if($triggerAlert==NULL){
                    //AQUI AGREGO PRIMERO AL USUARIO POR ESO DIGO QUE YA SE HA INSERTADO EL ID DE USUARIO POR LO QUE SE GENERA EL ULTIMO ID
                    $eventResult = $obj->save();
                    //LUEGO AQUI AGREGO LA OTRA INFORMACION DEL USUARIO.. 
                                         
                    $eventResultUI = $objUI->save();
                    
                
                    if(!$eventResult && !$eventResultUI){
                        $Warning = '<div id="Msg">
                                        <div id="warningAlertError">
                                            <div id="warningError">
                                                <div id="warningErrorText">Error al guardar..!</div>
                                            </div>
                                        </div>
                                    </div>';
                    } else{
                        $Warning = '<div id="Msg">
                                        <div id="warningAlertSave">
                                            <div id="warningSave">
                                                <div id="warningSaveText">Datos guardados..!</div>
                                            </div>
                                        </div>
                                    </div>';
                    }  echo''.$Warning.'';
                }else{
                    echo'<div id="Msg">
                            <div id="warningAlertError">
                                <div id="warningError">
                                    <div id="warningErrorText">'.$triggerAlert.'</div>
                                </div>
                            </div>
                         </div>';
                }
                
            break;
           }//FIN SWITCH
}//FIN IF
 

[aesu]

Iniciado por Reedyseth

@aesu Nos compartiste las dos clases que estas usando, pero también podrías compartir como es que la estas usando ??

Publiqué el funcionamiento de las clases revisalas por favor a ver si vez el problema..!

[aesu]

Iniciado por GatorV

Pero en tu clase UserPersonalInfo en el método save() estas llamando al Insert ID antes de hacer el INSERT, es lo que te estaba diciendo...

Hola GatorV

Publiqué el funcionamiento de las clases revisalas por favor a ver si vez el problema..!
son tres:
-AppDataBaseModel
-User
-UserPersonalInfo

También el código donde las llamo para usarlas..!
Gracias de antemano.

[GatorV]

Prueba usar directamente insert_id que es una propiedad de MySQLi. También como te había comentado antes si cierras y abres la conexión en todas tus consultas pierdes el insert_id eso no se debe de hacer, el funcionamiento correcto es algo así:

Código PHP:

<?php
$mysqli = new mysqli(SQLI_SERVER, MYSQLI_USER, MYSQLI_PWD, MYSQLI_DBNAME);
if ($result = $mysqli->query("INSERT INTO user(f_name, l_name) VALUES('$f_name', '$l_name');")) {
   echo 'The ID is: '.$mysqli->insert_id;
}