clase para login en php

[RIVERMILLOS]

hola quiciera saber que tal esta esta clase para la autentificacion en una web ...

conexion.php es para la base de datos.

Código:

<?php
include('conexion.php');

class Model_Autentificacion{

private $nombre;
private $contrasena;
private $conexion;
private $cadena;
private $cadenafinal;
private $isValide=false;
private $resultados;

function __construct($nombre,$contrasena){
$this->nombre=$nombre;
$this->contrasena=$contrasena;
$this->nombre=$this->removeChar($this->nombre);
$this->contrasena=$this->removeChar($this->contrasena);
$this->conexion= new Conexion();
$this->resultados=$this->conexion->consulta("SELECT count(*),nombre,contrasena FROM usuarios WHERE nombre='".$this->nombre."' and contrasena='".$this->contrasena."'");
while($tmp=mysql_fetch_array($this->resultados)){
if($tmp['count(*)']==1){
$this->isValide=true;
}	
}
$this->conexion->cerraConexion();
}	
	
function isValid(){
return $this->isValide;	
}
function removeChar($cadena){
$this->cadena=$cadena;
$this->cadenafinal= str_replace("SELECT",'',$this->cadena);
$this->cadenafinal= str_replace("INSERT",'',$this->cadena);
$this->cadenafinal= str_replace("INTO",'',$this->cadena);
$this->cadenafinal= str_replace("FROM",'',$this->cadena);
$this->cadenafinal= str_replace("OR",'',$this->cadena);
$this->cadenafinal= str_replace(",",'',$this->cadena);
$this->cadenafinal= str_replace(".",'',$this->cadena);
$this->cadenafinal= str_replace("@",'',$this->cadena);
$this->cadenafinal= str_replace("/",'',$this->cadena);
$this->cadenafinal= str_replace("(",'',$this->cadena);
$this->cadenafinal= str_replace(")",'',$this->cadena);
$this->cadenafinal= str_replace("*",'',$this->cadena);
return $this->cadenafinal;
}

}

saludos

[fernandozunni]

Fijate que si la condicion no se cumple no te devuelve nada, no se como vas a usar la clase pero yo le agregaría un else y que devuelva "false" en caso de que no se cumpla.

Código PHP:

if($tmp['count(*)']==1){
$this->isValide=true;
}
else{
$this->isValide=false;
}
 

[RIVERMILLOS]

hola gracias por tu respuesta , pero no es lo mismo que la declare

private $isValide=false; ??

saludos

[GatorV]

Así es, si lo predeclaras como false es correcto. Por otro lado si ninguna de tus variables mas que isValide no es necesario que las definas como propiedades de clase.

Saludos.