[SOLUCIONADO] Duda con código.

[xexio]

Estoy usando dreamweaver para hacer un acceso a una web, un Login vamos.

Y introduce automaticamente el siguiente codigo.

Código PHP:

      if (!function_exists("GetSQLValueString")) { 
        function GetSQLValueString($theValue, $theType, $theDefinedValue = "", $theNotDefinedValue = "") { 
            if (PHP_VERSION < 6) { 
                $theValue = get_magic_quotes_gpc() ? stripslashes($theValue) : $theValue; 
            } 
            $theValue = function_exists("mysql_real_escape_string") ? mysql_real_escape_string($theValue) : mysql_escape_string($theValue); 
 
            switch ($theType) { 
                case "text": 
                    $theValue = ($theValue != "") ? "'" . $theValue . "'" : "NULL"; 
                    break;     
                case "long": 
                case "int": 
                    $theValue = ($theValue != "") ? intval($theValue) : "NULL"; 
                    break; 
                case "double": 
                    $theValue = ($theValue != "") ? doubleval($theValue) : "NULL"; 
                    break; 
                case "date": 
                    $theValue = ($theValue != "") ? "'" . $theValue . "'" : "NULL"; 
                    break; 
                case "defined": 
                    $theValue = ($theValue != "") ? $theDefinedValue : $theNotDefinedValue; 
                    break; 
            } 
             
            return $theValue; 
        } 
    }  
   

Para usarlo posteriormente aqui

Código PHP:

  $LoginRS__query=sprintf("SELECT idUsuario, strEmail, strPassword FROM tblusuario WHERE strEmail=%s AND strPassword=%s", 
            GetSQLValueString($loginUsername, "text"), GetSQLValueString($password, "text"));  
   

La pregunta es, como se podria dejar la sentencia de SQL para no tener que usar esa funcion?:S

[SetheR]

Sencillo, utiliza mysqli o PDO y sus prepared statements para no tener que utilizar funciones de filtrado tan asquerosas como estas.

Si no quieres, con sustituirla por mysql_real_escape_string, sería suficiente.

Saludos

PD: Lo de asquerosas es apreciación personal.

[xexio]

He dejado de hacerlo con Dreamweaver, me estaba poniendo demasiado codigo ya autogenerado.

He decidido hacerlo a mano, pero, me encuentro con el problema siguiente..

Este codigo me da error y no entiendo el por que

Código PHP:

  $varProducto_DatosProducto = "0"; 
         
        if(isset($_GET["recordID"])){ 
            $varProducto_DatosProducto = $_GET["recordID"]; 
        } 
         
        mysql_select_db($database_conexiontienda1, $conexiontienda1); 
         
        $query_DatosProducto = printf("SELECT * FROM tblproducto WHERE tblproducto.idProducto='".$varProducto_DatosProducto."'"); 
         
         
        $DatosProducto = mysql_query($query_DatosProducto, $conexiontienda1) or die(mysql_error()); 
        $row_DatosProducto = mysql_fetch_assoc($DatosProducto); 
        $totalRows_DatosProducto = mysql_num_rows($DatosProducto);  
   

Se que estoy usando el antiguo acceso a bbdd de php, pero no se usar el nuevo y no me da tiempo tampoco.

El error que me da es este.

SELECT * FROM tblproducto WHERE tblproducto.idProducto='1'You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '58' at line 1

Pero no comprendo por que, si me coge bien el ID en la consulta.

[SetheR]

Te cuidado al utilizar printf(), pues ésta devuelve un ENTERO que es la longitud de la cadena. Quizá no es lo que le quieres pasar a mysql_query, ¿verdad?

Quizá quieras utilizar sprintf().

Saludos.

PD: Sobre lo de mysql, el paso a mysqli de manera procedurar es prácticamente trivial en tu caso, pero bueno, allá tú.

[jcastro3]

Yo te dejo un login un poco chapucero y sencillo que tengo implementado en una aplicacion, por si te orienta o te sirve de algo. Esta mi base de datos y mi consulta, etc...

Código PHP:

  <?php 
 
    //Establecer la conexion 
    $c = mysql_connect('localhost','root',''); 
     
    //Seleccionar la base de datos 
    mysql_select_db("proyecto",$c); 
     
    //Recoger los valores del formulario 
    $v1 = $_POST['usuario']; 
    $v2 = $_POST['password']; 
     
    //Comprobar si el usuario y la contraseña coinciden 
    $cons = "SELECT * FROM usuarios WHERE usuario = '$v1' AND password = '$v2';"; 
    $res = mysql_query($cons,$c); 
    if(!(@mysql_result($res,0))){ 
        header('location: ../html/index.php'); //Si el usuario y la contraseña no coinciden, te deja en index.html 
    }else{ 
        session_start(); 
        $_SESSION['usuario'] = $v1; 
        header('location: ../html/acciones.php'); //Si el usuario y la contraseña coinciden, te manda a acciones.html 
    } 
     
    //Comprobar si el usuario es administrador 
    if(($v1 == "admin") && ($v2 == "admin")){ 
        header('location: ../html/admin.php'); 
    } 
 
?>   

[xexio]

Gracias a los 2 por la ayuda. Ya esta solucionado

En cuanto a lo de MySQL, voy a cambiarlo todo a MySQLi