[SOLUCIONADO] compara mal dos datos uno por parámetro y el otro de un SELECT

[guardarmicorreo]

Estoy haciendo un validador con objetos de php.

he creado un documento con una clase que recibe los parámetros que quiero validar de dicho formulario.

al comparar los datos por parámetro y los datos extraídos por SELECT a mysql

siempre obtengo que la comparación es válida, tanto si los parámetros ya existen en la consulta como si no, siempre es válida.

puse en el objeto un var_dump();

me devuelve solo los datos de un usuario cuando en realidad tengo dos usuarios.

en el código muestro lo que devuelve el var_dump();

no logro entender por qué, si alguien es tan amable de ayudarme, muchas gracias.


este es mi código, escrito por mi.

Código Java:

<?php
 
//include ('clases.php');
 
class Validar extends Conectar
{
    private $arr = array();
    private $salida = array();
    public function datos_usuarios($usuarioc, $passwdc, $passwd2c, $emailc)
    {
        //consulto usuario, passwd y email de usuarios
        $validacion="SELECT usuario, passwd, email FROM usuarios";
        $result_conexion=Conectar::connect($validacion);
        
        //asigno los valores a un array con bucle while
        while($validacion=mysql_fetch_array($result_conexion))
        {
            $this->arr=$validacion;
        }
        
        //tengo dos usuarios en la base de datos, los datos introducidos manualmente en la BD son
        //usuario = q; email = [email protected]; passwd = q; nombre = q.q; apellidos = q.q.q
        //usuario = w; email = [email protected]; passwd = w; nombre = w.w; apellidos = w.w.w
        //este var_dump(); devuelve lo siguiente: 
        //array(6) { [0]=> string(1) "w" ["usuario"]=> string(1) "w" [1]=> string(1)
        // "w" ["passwd"]=> string(1) "w" [2]=> string(7) "[email protected]" ["email"]=> string(7) "[email protected]" } 
        var_dump($this->arr);
        
        
        
        //comparo los valores $usuario, $passwd y $email del usuario
        
        ////////////////////////////////////
        //buscando coincidencias de usuario
        ////////////////////////////////////
        foreach($this->arr as $datos=>$usuario)
        {
            //aquí está el primer problema, se cumpla o no la condición siempre va al else
            if($usuarioc == $datos)
            {
                $this->salida['usuario'] = "El usuario ya existe <br>";
            }
            else
            {
                $this->salida['usuario'] = "Usuario correcto <br>";
            }
        }
        
        //////////////////////////////////
        //buscando coincidencias de email
        //////////////////////////////////      
        foreach($this->arr as $datos=>$email)
        {
            //aquí está el segundo problema, se cumpla o no la condición siempre va al else
            if($emailc == $datos)
            {
                $this->salida['email'] = "El E-mail ya existe <br>";
            }
            else
            {
                $this->salida['email'] = "E-mail es correcto <br>";
            }
        }
        //las contraseñas sí las valida bien
 
        ///////////////////////////
        //validando la contraseña
        ///////////////////////////
        
        //si $passwd y $passwd2 no coinciden
        if($passwdc != $passwd2c)
        {
            $this->salida['passwd'] = "Las contraseñas no coinciden <br>";
        }
        else
        {
            $this->salida['passwd'] = "Contraseñas iguales <br>";
        }
        
        //si la contraseña es insegura (inferior a 6 caracteres)
        if(strlen($passwdc) < 6)
        {
            $this->salida['passwd2'] = "La contraseña tiene menos de 6 caracteres <br>";
        }
        else
        {
            $this->salida['passwd2'] = "La contraseña tiene más de 6 caracteres <br>";
        }
        
        ////////////////////////////
        //devuelve el array
        ////////////////////////////
        return $this->salida;
    }
}
 
?>
 

el resultado de esto es



Usuario correcto
E-mail es correcto
Contraseñas iguales
La contraseña tiene más de 6 caracteres

[maycolalvarez]

si arr es un Array: lo estás usando como variable:

$this->arr=$validacion;

cuando se supone que debes de agregarle el usuario:

$this->arr[]=$validacion;

consulte sobre manejo de arrays en php.

Recomendación: veo que intentas validar usuarios como un sistema de login, no me parece adecuado extraer Todos los usuarios para validar uno a uno cuando SQL where puede hacerlo por tí, solo tienes que tener en cuenta el filtrar contra SQL inyection, así es más eficiente

[pateketrueke]

1. Es pésima idea extraer todos los datos de la tabla para comparar con PHP, ¿si sabes que puedes hacer esas comparaciones de manera directa con SQL?

Deberías estudiar SQL.

2. En el while() con el que extraer los resultados sobre-escribes en cada vuelta $this->arr con el registro extraído, ¿de verdad no te das cuenta?

En resumen, no estás creando ninguna lista con todos los registros de esa forma.

Edito: ups! ya te lo han dicho...

[guardarmicorreo]

Iniciado por pateketrueke

1. Es pésima idea extraer todos los datos de la tabla para comparar con PHP, ¿si sabes que puedes hacer esas comparaciones de manera directa con SQL?

Tienes toda la razón

ahora he limitado la búsqueda sql de esta manera

Código PHP:

$validacion="SELECT usuario, email, passwd FROM usuarios WHERE usuario='".$usuarioc."' AND email='".$emailc."'";
 

y si no está vacío entonces retorno salida a true, luz verde para el registro.

Deberías estudiar SQL.

En cuanto termine PHP es algo que tengo pendiente, como javascript

2. En el while() con el que extraer los resultados sobre-escribes en cada vuelta $this->arr con el registro extraído, ¿de verdad no te das cuenta?

estuve todo el día para escribir esto, pensando y pensando cómo estructurarlo y cómo hacer la llamada, llegó un momento que me obturé

En resumen, no estás creando ninguna lista con todos los registros de esa forma.

Edito: ups! ya te lo han dicho...

es todo un placer tu ayuda!!!

[guardarmicorreo]

Iniciado por maycolalvarez

si arr es un Array: lo estás usando como variable:

$this->arr=$validacion;

cuando se supone que debes de agregarle el usuario:

$this->arr[]=$validacion;

consulte sobre manejo de arrays en php.

Recomendación: veo que intentas validar usuarios como un sistema de login, no me parece adecuado extraer Todos los usuarios para validar uno a uno cuando SQL where puede hacerlo por tí, solo tienes que tener en cuenta el filtrar contra SQL inyection, así es más eficiente

esto no me quedó claro. hay algún sitio donde pueda consultar de manera clara y con ejemplos el manejo de arrays en objetos con php si es posible, además de la documentación oficial, que la desconozco en este sentido, me gustaría ver otros sitios, pero no me doy idea de cómo buscarlo o dónde acudir, no encuentro nada con google, lo siento.

gracias por tu ayuda! :D

[maycolalvarez]

para eso busca en google por: php manejo de arrays

no recuerdo en que parte del manual de PHP aparece, pero es algo simple de dominar: http://www.desarrolloweb.com/articulos/1290.php esa forma aparece al final del articulo

PD: puedes estudiar SQL junto con PHP, porque para hacer lo que necesitas vas a necesitar interactuar mucho con la DB, no solo de programar en PHP se hacen los sistemas, y prácticamente aprender a programar va de la mano con dominar SQL, que además te servirá para otros lenguajes.

[guardarmicorreo]

Iniciado por maycolalvarez

para eso busca en google por: php manejo de arrays

no recuerdo en que parte del manual de PHP aparece, pero es algo simple de dominar: http://www.desarrolloweb.com/articulos/1290.php esa forma aparece al final del articulo

PD: puedes estudiar SQL junto con PHP, porque para hacer lo que necesitas vas a necesitar interactuar mucho con la DB, no solo de programar en PHP se hacen los sistemas, y prácticamente aprender a programar va de la mano con dominar SQL, que además te servirá para otros lenguajes.

tienes toda la razón. tendré que ponerme a ello. cada día me está gustando más programar :P

muchísimas gracias sinceramente :D