que estoy haciendo mal con las funciones?

[ahaugas]

hola, estoy construyendo una clase para ver usuarios online y que actualize si no esta conectado o que si incicia sesion.

todoavia estoy bastante verde en el tama pero llego a una parte que me salta un error y no se conectar la funcion superior con la de abajo ej:

Código PHP:

  class usersOnline {
 
    function usersOnlineConstruct() {
        global $db,$db_table_prefix,$loggedInUser;
        if(!isUserLoggedIn()) { $session = "0"; } else { $session = "1"; }
        
        $this->userOnlineTime         = 5;                                                // Tiempo máximo de espera
        $this->userOnlineDate         = time();                                             // Momento que entra en línea
        $this->userOnlineIP         = $_SERVER['REMOTE_ADDR'];                            // Recuperamos su IP
        $this->userOnlineLimit         = $this->userOnlineDate-$this->userOnlineTime*60;    // Tiempo Limite de espera
        $this->userOnlineSession    = $session;                                            // Vemos si es usuario o invitado
    return;
    }
    
    function usersOnlineDelete() {
    usersOnlineConstruct();
        // si se supera el tiempo limite (5 minutos) lo borramos
        $sql = "DELETE FROM ".$db_table_prefix."Users_Online 
                WHERE Date_Time < '".$db->sql_escape($this->userOnlineLimit)."'";
                return ($db->sql_query($sql));
    }
} 
   

se supone que necesito leer los datos de usersOnlineConstruct() en usersOnlineDelete() pero me tira fallos

alguien me podria hechar una mano porfavor
gracias

[pateketrueke]

Lo que sucede es que estás tratando a los métodos de tu clase como si fueran funciones globales, evidentemente no es así.

Dependiendo del tipo de objeto que utilizas, en tu caso es instanciado, debes llamar a los metodos antecediendo $this-> como con las variables.

Además en tu segundo método estás usando $db como si ya existiera en el contexto de dicho método, lo cual evidentemente no es cierto.

PDTA: por favor, debes documentarte, lee un manual. No puedes ir por ahí andando a ciegas no más.

[ahaugas]

eso intento aprender, pero aveces con los manuales y ejemplos no sirve por que no entiendo y llego a una parte que me bloqueo, si fuera una sola funcion me sale a la perfeccion pero si intento crear varias funciones dentro de una clase, ya no se como hacerlo porque no quiero utilizar tanto codigo.

me podrias dar alguna pista para documentarme porfavor?

ah y gracias por la respuesta.

[pateketrueke]

El manual oficial de OOP de PHP, desde luego.
http://mx2.php.net/oop

[ahaugas]

ok eso ya lo solucione mediante vars, gracias.
ahora tengo un problema con con la comprobacion de datos y actualizarlo.

Código PHP:

function usersOnlineSelect() {
        global $db,$db_table_prefix;
        // tomamos todos los usuarios en linea
        $sql = "SELECT * FROM ".$db_table_prefix."Users_Online 
                WHERE User_IP='".$db->sql_escape($this->userOnlineIP)."'";
                return ($db->sql_query($sql));
    }
    
    function usersOnlineInsert() {
        global $db,$db_table_prefix;
        // Si son los mismo actualizamos la tabla gente_online
        if(mysql_num_rows($sql) != 0) {
        $sql = "UPDATE ".$db_table_prefix."Users_Online 
                SET Date_Time='".$db->sql_escape($this->userOnlineDate)."', 
                User_Session='".$db->sql_escape($this->userOnlineSession)."'
                WHERE User_IP='".$db->sql_escape($this->userOnlineIP)."'";
                return ($db->sql_query($sql));
        
        // de lo contrario insertamos los nuevos
        } else {
        $sql = "INSERT INTO `".$db_table_prefix."Users_Online` (
                `Date_Time`,
                `User_IP`,
                `User_Session`
                ) 
                VALUES (
                '".$db->sql_escape($this->userOnlineDate)."',
                '".$db->sql_escape($this->userOnlineIP)."',
                '".$db->sql_escape($this->userOnlineSession)."'
                )";
                return $db->sql_query($sql);
        }
 

ah utilizo los globals porque estan definidas en otro script, si no no funciona, hago la llamada ya definida.

agui el problema esta en al llamarlo de usersOnlineSelect(),

Código PHP:

if(mysql_num_rows($sql) != 0) {
 

se que todavia soy novato, pero ya casi lo tengo y necesitaria info sobre esto
gracias

[pateketrueke]

Ahora tu variable $sql está fuera de contexto, lee lo siguiente:
http://mx2.php.net/manual/es/languag...bles.scope.php

[ahaugas]

ok, ya lo he definido pero no he entendido muy bien lo que me querias decir lo ultimo, el manual esta un poco borroso para mi todavia.

y ahora solo me dice que hay un usuario invitado y no me actualiza en la db, seguro que he hecho algo mal.

dejo el la funcion completa ya terminada pero me falla el actualizar.

Código PHP:

<?php   
 
class usersOnline {
    var $userOnlineTime;
    var $userOnlineDate;
    var $userOnlineIP;
    var $userOnlineLimit;
    var $userOnlineSession;
    var $usersOnlineUsersSelect;
    
    function usersOnlineConstruct() {
        global $db,$db_table_prefix,$loggedInUser;
        if(!isUserLoggedIn()) { $session = "0"; } else { $session = "1"; }
        
        $this->userOnlineTime       = 5;                                                // Tiempo máximo de espera
        $this->userOnlineDate       = time();                                           // Momento que entra en línea
        $this->userOnlineIP         = $_SERVER['REMOTE_ADDR'];                          // Recuperamos su IP
        $this->userOnlineLimit      = $this->userOnlineDate-$this->userOnlineTime*60;   // Tiempo Limite de espera
        $this->userOnlineSession    = $session;                                         // Vemos si es usuario o invitado
    }
    
    function usersOnlineDelete() {
        global $db,$db_table_prefix;
        // si se supera el tiempo limite (5 minutos) lo borramos
        $sql = "DELETE FROM ".$db_table_prefix."Users_Online 
                WHERE Date_Time < '".$db->sql_escape($this->userOnlineLimit)."'";
                return ($db->sql_query($sql));
    }
    
    function usersOnlineSelect() {
        global $db,$db_table_prefix;
        // tomamos todos los usuarios en linea
        $sql = "SELECT * FROM ".$db_table_prefix."Users_Online 
                WHERE User_IP='".$db->sql_escape($this->userOnlineIP)."'";
        return ($db->sql_query($sql));
    }
    
    function usersOnlineInsert() {
        global $db,$db_table_prefix;
        $this->usersOnlineUsersSelect = $this->usersOnlineSelect();
        // Si son los mismo actualizamos la tabla gente_online
        if(mysql_num_rows($this->usersOnlineUsersSelect) != 0) {
        $sql = "UPDATE ".$db_table_prefix."Users_Online 
                SET Date_Time='".$db->sql_escape($this->userOnlineDate)."', 
                User_Session='".$db->sql_escape($this->userOnlineSession)."'
                WHERE User_IP='".$db->sql_escape($this->userOnlineIP)."'";
                return ($db->sql_query($sql));
        
        // de lo contrario insertamos los nuevos
        } else {
        $sql = "INSERT INTO `".$db_table_prefix."Users_Online` (
                `Date_Time`,
                `User_IP`,
                `User_Session`
                ) 
                VALUES (
                '".$db->sql_escape($this->userOnlineDate)."',
                '".$db->sql_escape($this->userOnlineIP)."',
                '".$db->sql_escape($this->userOnlineSession)."'
                )";
                return $db->sql_query($sql);
        }
    }
        
        
        
    function usersOnlineView() {    
        global $db,$db_table_prefix;
        
        // Seleccionamos toda la tabla
        $query = "SELECT * FROM ".$db_table_prefix."Users_Online";
        $resp = @mysql_query($query) or die(mysql_error());
        $usuarios = mysql_num_rows($resp);
        
        //Usuarios no registrados
        $query = "SELECT * FROM ".$db_table_prefix."Users_Online WHERE User_Session='0'";
        $resp = @mysql_query($query) or die(mysql_error());
        $noregistrados = mysql_num_rows($resp);
        
        //Usuarios registrados
        $query = "SELECT * FROM ".$db_table_prefix."Users_Online WHERE User_Session='1'";
        $resp = @mysql_query($query) or die(mysql_error());
        $registrados = mysql_num_rows($resp);
 
        // Si hay 1 usuarios se muestra en singular; si hay más de uno, en plural
        if($usuarios > 1 || $usuarios == 0){ echo("Hay "); } else { echo("Hay "); }
        if($usuarios == 0){ echo("no "); } else { echo($usuarios." "); } 
        if($usuarios > 1 || $usuarios == 0){ echo("usuarios en línea."); } else { echo("usuario en línea."); }
        echo '<br />';
        echo 'Invitados: '.$noregistrados.'';
        echo '<br />';
        echo 'Usuarios: '.$registrados.'';
    }
}   
    //View userCake users online
    $viewUsersOnline = new usersOnline();
    $viewUsersOnline->usersOnlineConstruct();
    $viewUsersOnline->usersOnlineDelete();
    $viewUsersOnline->usersOnlineSelect();
    $viewUsersOnline->usersOnlineInsert();
?>
 

gracias por toda la ayuda, he avanzado un poquitin jejeje pero me doy cuanta de mas cosas