registro de usuario sqlsrv_connect

[j84625]

hace mucho que no realizo nada en php desde php 5.2 y tengo problemas con este codigo aun no esta terminado pero no funciona

Código PHP:

  <?php
$serverName = "host\SQLEXPRESS"; //serverName\instanceName
$connectionInfo = array( "Database"=>"db", "UID"=>"sa", "PWD"=>"pass");
$conn = sqlsrv_connect( $serverName, $connectionInfo);

if( $conn ) 
{
    if ((eregi("[^a-zA-Z0-9_-]", stripslashes($_POST['RUser']))) || (eregi("[^a-zA-Z0-9_-]", stripslashes($_POST['RPass']))) || (eregi("[^a-zA-Z0-9_-]", stripslashes($_POST['RCorreo']))))
    {
    echo("<style type='text/css'>
                <!--
                body {
                background-color: #B0C0EC;
                }
                -->
                </style>Error solo letras y numeros");
    exit();
    }
    else
        {
            $RUser = stripslashes($_POST['RUser']);
            $RPass = stripslashes($_POST['RPass']);
            $RCorreo = stripslashes($_POST['RCorreo']);
    
            $sql = "select memb___id from MEMB_INFO where memb___id = '$RUser'";
            $params = array(1, "some data");
    
            $stmt = sqlsrv_query( $conn, $sql);
            if( $stmt === false ) {
                 die( print_r( sqlsrv_errors(), true));
            }
            
            while( $row = sqlsrv_fetch_array( $stmt, SQLSRV_FETCH_NUMERIC) ) {
                 if ($RUser == $row[0])
                    {
                        echo("<style type='text/css'>
                            <!--
                            body {
                            background-color: #B0C0EC;
                            }
                            -->
                            </style>El Usuario ya existe");
                    }
                    else
                        {
                            
                            $sql = "insert into dbo.MEMB_INFO (memb___id,memb__pwd,memb_name,sno__numb,bloc_code,ctl1_code) Values ('$RUser','$RPass','RCorreo',1,0,1)";
                            $params = array(2, "some data");
                            
                            $stmt = sqlsrv_query( $conn, $sql);
                            if( $stmt === false ) {
                                 die( print_r( sqlsrv_errors(), true));
                            }
                                
                        }
}

sqlsrv_free_stmt( $stmt);
            
        }
}

else{
     echo "Lamentablemente se produjo un error por favor notifiquelo al administrador.<br />";
     die( print_r( sqlsrv_errors(), true));
}
?>   

si alguno esta dispuesto ayudarme mandar mp

[Triby]

Bueno, yo estaría dispuesto a tratar de ayudar, pero no por MP, ni con garantía de que mis sugerencias solucionen el problema que... por cierto, se te olvido mencionar cuál es exactamente el problema, en qué falla el código, qué hace bien y qúe hace mal (o no hace).

P.D. la función ereg, junto con todas sus hermanas, tías y sobrinas (eregi, ereg_replace, etc), ha pasado a mejor vida para ser reemplazada con preg y creo que los sabores y colores son los mismos, sólo cambiarían las expresiones regulares de las compatibles con POSIX a las compatibles con Perl (PCRE).

[j84625]

el problema es que si el usuario existe no se crea uno nuevo que eso esta bien pero no agrega uno nuevo

seguro que se puede mejorar el código jejeje ya se que esta anticuado y que no es la mejor opción

si me pones un ejemplo de como realizarías tu este pequeño registro de cuentas

[GatorV]

Usa sqlsrv_num_rows para comprobar tu query, sí tiene más de 1 fila es que sí existe el usuario, de lo contrario no existe y procedes a insertar el usuario.

[Triby]

Como lo haría (hago):

1- Primero obtengo todos los valores desde $_POST, pasándolos por una función que GatorV compartió para evitar ataques XSS

2- Hago validación de contenido y formato en todos los campos y voy almacenando los errores en un array()

3- Si no hay error en el nombre de usuario, hago una consulta para ver si existe o no

4- Si hay errores, devuelvo al formulario de registro y los muestro

5- Si no hay errores, entonces escapo los datos (si es con mysql) o preparo la consulta con PDO e inserto el registro y muestro mensaje de registro correcto o envío a página de ingreso (login), dependiendo del caso

Lo que nunca hago es enviar salida (HTML, textos, etc.) si no es necesario, es decir, primero recolecto toda la info de lo que se va a hacer, proceso lo que se requiera y, finalmente, envío el HTML correspondiente.

[j84625]

gracias por responder pero para mi es mejor ver codigos de ejemplo y ver como funcionan por que no se mucho de php y lo poco que sabia se me olvido jejej

[j84625]

Código PHP:

  <?php
$serverName = "host\SQLEXPRESS"; //serverName\instanceName
$connectionInfo = array( "Database"=>"db", "UID"=>"sa", "PWD"=>"pas");
$conn = sqlsrv_connect( $serverName, $connectionInfo);
if( $conn ) {
     echo "Connection established.<br />";
}else{
     echo "Connection could not be established.<br />";
     die( print_r( sqlsrv_errors(), true));
}
if ((eregi("[^a-zA-Z0-9_-]", stripslashes($_POST['RUser']))) || (eregi("[^a-zA-Z0-9_-]", stripslashes($_POST['RPass']))) || (eregi("[^a-zA-Z0-9_-]", stripslashes($_POST['RCorreo']))))
{
    echo("<style type='text/css'>
                <!--
                body {
                background-color: #B0C0EC;
                }
                -->
                </style>Error solo letras y numeros");
    exit();
}
else
{    
    $usuario = $_POST['RUser'];
    $pass = $_POST['RPass'];
    $correo = $_POST['RCorreo'];
    $sqlselect = "select memb___id from MEMB_INFO where memb___id = '$usuario'";
    $paramsselect = array();
    $optionsselect =  array( "Scrollable" => SQLSRV_CURSOR_KEYSET );
    $stmtselect = sqlsrv_query( $conn, $sqlselect , $paramsselect, $optionsselect );
    $row_count = sqlsrv_num_rows( $stmtselect );
       
    if ($row_count == 0)
        {
    
              echo "El Usuario No Existe";
              $sql = "insert into dbo.MEMB_INFO (memb___id,memb__pwd,memb_name,sno__numb,bloc_code,ctl1_code) Values ('$usuario','$pass','$correo',1,0,1)";
              $params = array(1, "some data");
              $stmt = sqlsrv_query( $conn, $sql, $params);
              exit();
        }
    else
    {
      echo "El Usuario Si Existe";
 
       exit();
    }
   
   
}
?>   

modifique el script ahora si registra los registros y valida si están ya en la base de datos que podría mejorar ? o lo dejo así ?

ahora voy a poner una if para valores nulos