Hola alaintm,
Sigues con muchos errores conceptuales, como te digo, no es necesario usar $this-> en todas tus variables, de hecho no es necesario que todas sean miembros de una clase, mira ve este ejemplo:
Código PHP:
class Prueba {
protected $protegida; // esta variable si la heredas la puedes ver y cambiar desde una clase hija
private $privada; // esta variable solo la puedes cambiar desde esta clase
public $publica; // no es recomendable usar variables publicas
public function Funcion() {
$variable = "hola"; // variable simple que solo se ve durante Funcion()
$otravar = $this->privada; // aqui leemos desde una propiedad de la clase
$this->publica = "hola de nuevo"; // aqui asignamos algo a $publica (propiedad de la clase)
}
public function leePrivada() {
return $this->privada; // esto lo hacemos para poder leer una propiedad desde fuera
}
}
Como puedes ver, hay varias formas de asignar variables en una clase, no es necesario que todas las variables sean propiedades de la clase, puedes usar fácilmente todas las variables de forma local, como veo tu clase, la única que seria bueno que guardaras como propiedad seria $usuarios_conectados para que luego la puedas leer sin tener que volver a hacer el query a la base de datos.
Por ejemplo:
Código PHP:
$enlinea = new UsuariosEnLinea(); // en el constructor hacemos el "calculo";
echo $enlinea->getConectados();
// imprimimos mucho codigo html
// mas abajo
echo $enlinea->getConectados();
Si te fijas en ese código, si uso el tuyo tendría que hacer 2 veces el calculo en la base de datos lo que me daría números incorrectos (para esa pagina), si lo haces como te indico te mostrara el mismo numero ya que asi solo hara el calculo cada que se cree el objeto (o sea cada que entran a la pagina).
Saludos.