Queria verificar el siguiente codigo (si tiene la sintaxis correcta)

[jonyjdv]

Hola a todos, tengo el siguiente codigo en una funcion y mezcle codigo orientado a objectos y codigo por procedimiento, el caso es que queria ponerlo bien y para ello queria saber que es mejor, orientado a objectos o por procedimientos.

Código PHP:

  function eliminar_nuevos_datos($conexion, $consulta, $escapar_datos){
 
        // Create connection
        require($_SERVER['DOCUMENT_ROOT'].$conexion);
        
        $SQL_datos_conexion = new mysqli($db_host, $db_user, $db_pass, $db_name);
 
        //Si se quiere escapar los datos
        if($escapar_datos!=false){
 
 
            $stmt = $SQL_datos_conexion->prepare($consulta);            
                
                $numero_datos = count($escapar_datos);
 
                if($numero_datos>=2){
 
                    //Saber si es string o numerico(int)
                    $type = '';
                    foreach ($escapar_datos as $value) {
                        $type .= is_numeric($value) ? 'i' : 's';
                    }
 
                    if($numero_datos==2){
                        $stmt->bind_param($type, escapar_caracteres($SQL_datos_conexion, $escapar_datos[0]), escapar_caracteres($SQL_datos_conexion, $escapar_datos[1]));
                    }elseif($numero_datos==3){
                        $stmt->bind_param($type, escapar_caracteres($SQL_datos_conexion, $escapar_datos[0]), escapar_caracteres($SQL_datos_conexion, $escapar_datos[1]), escapar_caracteres($SQL_datos_conexion, $escapar_datos[2]));
                    }
 
                }elseif($numero_datos==1){
                    $stmt->bind_param('s', escapar_caracteres($SQL_datos_conexion, $escapar_datos));
                }
 
                $stmt->execute();
 
                //Retornar numero de celdas afectadas
                return $stmt->affected_rows;
 
                $stmt->close();
 
        }else{            
            mysqli_query($SQL_datos_conexion, $consulta);
        }
 
        mysqli_close($SQL_datos_conexion);
 
    } 
   

Gracias.

[pateketrueke]

el caso es que queria ponerlo bien y para ello queria saber que es mejor, orientado a objectos o por procedimientos.

Da igual, el hecho de que ya funcione es excelente.

No seas purista porque sencillamente no existe la perfección en escribir código.

[jonyjdv]

Iniciado por pateketrueke

Da igual, el hecho de que ya funcione es excelente.

No seas purista porque sencillamente no existe la perfección en escribir código.

entonces el codigo que mostre va a funcionar correctamente pese a mezclar objectos y por procedimiento, a veces es complicado escojer una forma u otra hay opiniones para todo y te vuelces loco