problema al recorrer foreach

[guardarmicorreo]

estoy trabajando en un validador, el primer paso consiste en que verifique si el usuario y el email introducidos contienen sql injection, por lo que primero instancio la clase y la reproduzco así

controlador.php
-

Código PHP:

function validar($usuario, $email, $passwd, $passwd2, $nombre)
    {
        //creo una instancia para obtener un usuario, una contraseña y un email existentes
        $usuario_class = new Validar($usuario, $email, $passwd, $passwd2);
        
        //compruebo que el usuario no contiene sql injection
        $validar_usuario = $usuario_class->seguridad_patrones();
        foreach($validar_usuario as $informe)
        {
            
            echo $informe ['usuario']. "<br>"; //esta es la línea 316
            echo $informe ['email']; //esta es la línea 317
        }
    }
 

en la clase tengo el siguiente código

Código PHP:

class Validar extends Conectar
{
    ///////////////////////////
    //declaro las variables
    ///////////////////////////
    
    //variables que contienen los valores que el usuario introduce
    private $usuarioc;
    private $emailc;
    private $passwdc;
    private $passwd2c;
    
    //arrays que contienen la consulta en base de datos
    // y la salida según la validación (true o false)
    private $arr = array();
    private $salida = array();
    
    //variables para devolver si el usuario, email, passwd y passwd2
    //son correctos según comprobación anti sql injection
    private $validar = array();
    
    //resultados de la búsqueda con los patrones en los parámetros
    private $res;
    private $res2;
    private $res3;
    private $res4;
    private $res5;
    
    //////////////////////////////////////////
    //patrones para buscar palabras prohibidas
    //////////////////////////////////////////
    
    //patrones para evitar email injection
    private $seguridad_email = 
    array(
        "Content-Type:",
        "MIME-Version:",
        "Content-Transfer-Encoding:",
        "Return-path:",
        "Subject:",
        "From:",
        "Envelope-to:",
        "To:",
        "bcc:",
        "cc:",
        );
        
    //patrones para evitar sql injection
    private $seguridad_sql =
    array (
        "UNION",
        "DELETE",
        "DROP",
        "SELECT",
        "INSERT",
        "UPDATE",
        "CRERATE",
        "TRUNCATE",
        "ALTER",
        "INTO",
        "DISTINCT",
        "GROUP BY",
        "WHERE",
        "RENAME",
        "DEFINE",
        "UNDEFINE",
        "PROMPT",
        "ACCEPT",
        "VIEW",
        "COUNT",
        "HAVING",
        "'",
        '"',
        "{",
        "}",
        "[",
        "]",
        );
    //patrones para evitar direcciones web
    private $seguridad_url =
    array (
        "HOTMAIL",
        "WWW",
        ".COM",
        "@",
        "W W W",
        ". c o m",
        "http://",
        );
    //patrones para evitar variables y comodines
    private $seguridad_varcom =
    array("$",
        "&",
        "*",
    );
    
    //aplico el constructor que recibe los parámetros del usuario
    public function __construct( $usr, $emil, $paswd, $paswd2 )
    {
        $this->usuarioc = $usr;
        $this->emailc = $emil;
        $this->passwdc = $paswd;
        $this->passwd2c = $paswd2;
    }
    
    //primero reviso los parámetros asegurando que no contengan sql injection
    
    //valido el usuario contra sql injection
    public function seguridad_patrones()
    {
        //reviso el usuario
        $this->res = stripos ( $this->usuarioc , "FROM:");
        if ( $this->res === false )
        {
            $this->validar['usuario'] = "usuario no contiene sql injection";
        }
        else
        {
            $this->validar['usuario'] = "usuario contiene sql injection";
        }
        
        //reviso el email
        $this->res2 = stripos ( $this->emailc , "FROM:" );
        if ( $this->res2 === false )
        {
            $this->validar['email'] = "email no contiene sql injection";
        }
        else
        {
            $this->validar['email'] = "email contiene sql injection";
        }
        
        return $this->validar;
        //reviso el passwd
        
    }
}
 

me devuelve los siguientes errores

Warning: Illegal string offset 'usuario' in C:\xampp\htdocs\xampp\web\poo\controlador.php on line 316
u

Warning: Illegal string offset 'email' in C:\xampp\htdocs\xampp\web\poo\controlador.php on line 317
u
Warning: Illegal string offset 'usuario' in C:\xampp\htdocs\xampp\web\poo\fcontrolador.php on line 316
e

Warning: Illegal string offset 'email' in C:\xampp\htdocs\xampp\web\poo\controlador.php on line 317
e

las líneas 316 y 317 están indicadas en el documento controlador.php

no entiendo el problema, puesto que he leido sobre este problema y me dice que se debe a que a partir de php 5.4 hay que indicar los indicadores de los arrays con enteros o string de entero, entonces hice lo siguiente tanto en el código del controlador como en el de la clase

Código PHP:

//en el controlador cambié los indicadores 'usuario' e 'email' por [1] y [2] respectivamente
 
        foreach($validar_usuario as $informe)
        {
            
            echo $informe [1]. "<br>";
            echo $informe [2];
        }
 
//en la clase hice lo mismo
 
public function seguridad_patrones()
    {
        //reviso el usuario
        $this->res = stripos ( $this->usuarioc , "FROM:");
        if ( $this->res === false )
        {
            $this->validar[1] = "usuario no contiene sql injection";
        }
        else
        {
            $this->validar[1] = "usuario contiene sql injection";
        }
        
        //reviso el email
        $this->res2 = stripos ( $this->emailc , "FROM:" );
        if ( $this->res2 === false )
        {
            $this->validar[2] = "email no contiene sql injection";
        }
        else
        {
            $this->validar[2] = "email contiene sql injection";
        }
        
        return $this->validar;
        //reviso el passwd
        
    }
 

no me devuelve ningún error pero me devuelve solo esto

s
um
a

no entiendo de qué cadena sale eso, pero no lo tengo en ninguna cadena.

¿alguien me puede ayudar?

gracias!

[alyciashape]

Para evitar SQL Injections lo puedes hacer mucho más facil con mysqli o algún framework simplemente escapando las consultas. No hace falta tanto lío.

Para acceder al elemento de un objeto es con object->nombre no object['nombre'] (esto es para arrays).

Cuando haces el foreach estás seguro de que eso es una lista de objetos de tipo Validar o estas haciendo un foreach de un objeto validar? porque entonces normal que no te coja los campos porque en $informe estás cogiendo cada uno de los campos de validar.

Además, el campo nombre es privado por lo que veo porque no puedes acceder haciendo un $informe->nombre. Tendrás que hacer un "getter" que devuelva ese atributo, ya que es privado.

[guardarmicorreo]

Iniciado por alyciashape

Para evitar SQL Injections lo puedes hacer mucho más facil con mysqli o algún framework simplemente escapando las consultas. No hace falta tanto lío.

gracias por el consejo, pero he pensado que para aprender no es bueno utilizar frameworks.

Para acceder al elemento de un objeto es con object->nombre no object['nombre'] (esto es para arrays).

eso sí lo se, pero yo no estoy accediendo al elemento de un objeto, estoy accediendo al return de un array, si te fijas en la línea 133 hay un return $this->validar;

Cuando haces el foreach estás seguro de que eso es una lista de objetos de tipo Validar o estas haciendo un foreach de un objeto validar? porque entonces normal que no te coja los campos porque en $informe estás cogiendo cada uno de los campos de validar.

no, es de un array retornada desde la clase Validar.

Además, el campo nombre es privado por lo que veo porque no puedes acceder haciendo un $informe->nombre. Tendrás que hacer un "getter" que devuelva ese atributo, ya que es privado.

ya, es privado accediéndolo desde fuera, pero yo lo retorno desde dentro de la clase, por lo que creo que no hay problema con eso no?

gracias por tu ayuda! a ver si podemos resolver esto :D