editar perfil de usuario desde la pagina web

Hernand67890 · #1 (**permalink**) 24/02/2016, 22:17

Saludos cordiales

Me encuentro realizando una aplicación web en php, y voy en la parte del sistema de sesión, en base a esto desarrolle los formularios de registrarse e iniciar sesión la persona, por ende la parte de registrar me funciona bien, iniciar sesion me funciona bien, pero lo que me gustaría que me funcione, es una opcion denominada ver y editar perfil del usuario, desde la aplicacion web sin necesidad de estar accediendo a la BD en mysql, ¿de que manera o que extructura de codigo es mas aconsejable para que al ingresar a la opcion perfil, se pueda ver y editar la informacion registrada en la BD ya creada?

Gracias.

chronos682 · #2 (**permalink**) 24/02/2016, 23:02

Lo que yo hago es cuando el usuario hace clic sobre su nombre de usuario en la parte superior derecha de la página accede al perfil y allí hay un botón que dice editar perfil. Obviamente sólo le dejo editar ciertos datos como el teléfono, correo electrónico o su nombre más no el nombre de usuario por ejemplo, también le doy la opción de cambiar la contraseña desde la misma página de edición de perfil.

Hernand67890 · #3 (**permalink**) 25/02/2016, 07:43

Bueno, efectivamente eso lo que deseo hacer pero tengo algo en el codigo, que aun no permite traer la informacion guardada en la BD para poderla modificar directamente desde la pagina web.
Anexo el codigo del archivo editar.html

<?php
session_start();
if(isset ($_SESSION['id'])) {
?>

<!DOCTYPE html>
<link rel="stylesheet" href="../css/estilo.css" type="text/css" media="all">
<html lang="en">
<head>
<title>Mi perfil de operario</title>
<body><a href ="../view/login.php" class = "dos">Cerrar Sesion...</a>
<p id ="title">Bievenido operario
<?php
echo $_SESSION['name'] ;
echo " " ;
echo $_SESSION['apellido'];
?>
</p>

<?php include("../controller/crear.php");
$id = @$_GET["id"];
$ssql = "select * from users where id= ". $id; $editar = mysql_query($ssql);
$ssql = "select * from users where id= 12". $id.$editar = mysql_query($ssql);
$ssql=mysql_query("UPDATE usuarios SET nombre = '$nombre', apellido = '$apellido' usuario = '$usuario', contraseña = '$contraseña', WHERE ID_USUARIO = $ID_USUARIO")OR die (mysql_error());
?>

<form action="../controller/crear.php" method="post"> <input type="hidden" name="id" value="<?php echo $id;?>">
<br />
Nombre:
<br />
<input type="text" name="nombre" value="<?php echo set_value('nombre',$users->nombre_usuario); @$name =;?>">
<br />
Apellido:
<br />
<input type="text" name="apellido" value="<?php echo @$name =;?>">
<br />
Usuario:
<br />
<input type="text" name="usuario" value="<?php echo @$name =;?>">
<br/>
Contraseña:
<br />
<input type="text" name="contraseña" value="<?php echo @$name =;?>">
<br/>

<center> <input class="form-btn" name="submit" type="submit" value="Actualizar Informacion" /></center>


<footer>
<div class="wrapper">
<section class="lower">
<h3> <span></span></h3>

</section>
<section class="lower">
<h3></h3>

</html>
<?php
}else{echo "Lo sentimos, debes iniciar sesion antes de acceder a esta pagina"; } ?>

Gracias.

rodocoyote15 · #4 (**permalink**) 25/02/2016, 09:58

Usas php y el archivo lo tienes como html? Vaya vaya, error desde ahí.

Usar @ es muy muy malo, es preferible que corrijas los errores a que los ocultes, y además eso lo alenta, si intentas captar los errores prueba con try/catch.

También, llamas a la variable "name" pero en ningún momento la estás definiendo, o hay más código aparte de ese?

Usa el highlight también, saludos :D

chronos682 · #5 (**permalink**) 25/02/2016, 10:04

Tienes muchos errores en el código. Primero hay un campo hidden en el formulario cuyo value es la variable $id pero esa variable la tomas por GET cuando según veo al comienzo del código la estas pasando es por SESSION no por GET. Ahora, estas haciendo el update sin siquiera confirmar que se haya enviado el formulario por lo cual siempre va a entrar a hacer el update incluso antes de mostrar el formulario. Usa if isset para confirmar si se hizo submit del formulario. Luego verifica que sean los datos correctos y ahí si haces el update.Lo otro que te recomiendo es que no permitas que se cambien los campos que sean claves primarias. Yo p ej tengo dos claves principales en mi tabla de usuarios el id autonumerico y el nombre de usuario, los cuales siempre deben ser únicos por lo cual al editar el perfil no les permito cambiar el nombre de usuario

Hernand67890 · #6 (**permalink**) 25/02/2016, 10:41

Muy amable por su respuesta oportuna.
Acerca del archivo editar ya lo guarde con la extension .php

Al acceder nuevamente al archivo editar.php me salen dos errores en pantalla de color rojo:

Anexo nuevamente el codigo, suprimiendo en la linea numero 32 eliminando el @.

<?php
session_start();
if(isset ($_SESSION['id'])) {
?>

<!DOCTYPE html>
<link rel="stylesheet" href="../css/estilo.css" type="text/css" media="all">
<html lang="en">
<head>
<title>Mi perfil de operario</title>
<body><a href ="../view/login.php" class = "dos">Cerrar Sesion...</a>
<p id ="title">Bievenido operario
<?php
echo $_SESSION['name'] ;
echo " " ;
echo $_SESSION['apellido'];
?>
</p>

<?php include("../controller/crear.php");
$id = @$_GET["id"];
$ssql = "select * from users where id= ". $id; $editar = mysql_query($ssql);
$ssql = "select * from users where id= 12". $id.$editar = mysql_query($ssql);
$ssql=mysql_query("UPDATE usuarios SET nombre = '$nombre', apellido = '$apellido' usuario = '$usuario', contraseña = '$contraseña', WHERE ID_USUARIO = $ID_USUARIO")OR die (mysql_error());
?>

<form action="../controller/crear.php" method="post"> <input type="hidden" name="id" value="<?php echo $id;?>">
<br />
Nombre:
<br />
<input type="text" name="nombre" value="<?php echo @$name =;?>">
<br />
Apellido:
<br />
<input type="text" name="apellido" value="<?php echo @$name =;?>">
<br />
Usuario:
<br />
<input type="text" name="usuario" value="<?php echo @$name =;?>">
<br/>
Contraseña:
<br />
<input type="text" name="contraseña" value="<?php echo @$name =;?>">
<br/>

<center> <input class="form-btn" name="submit" type="submit" value="Actualizar Informacion" /></center>


<footer>
<div class="wrapper">
<section class="lower">
<h3> <span></span></h3>

</section>
<section class="lower">
<h3></h3>

</html>
<?php
}else{echo "Lo sentimos, debes iniciar sesion antes de acceder a esta pagina"; } ?>

No comprendo realmente porque me sale error de "Scream" y el error en la linea 32

Gracias.

rodocoyote15 · #7 (**permalink**) 25/02/2016, 10:47

Código PHP:

Ver original<?php echo @$name =; ?>

Que intentas hacer con eso?

Hernand67890 · #8 (**permalink**) 25/02/2016, 11:23

Revisando detenidamente me faltaba definir cada una de las variables @$name, también realice la corrección por usar solo una sola variable $_Session,
pero ahora me salen varios errores en rojo y al final de la pagina indica "No database Selected" mi pregunta es ¿de que manera puedo indicarle a la aplicación para que acceda a la base de datos la cual se encuentra ya creada?

Anexo imagenes del error:

Anexo el codigo editar.php con las correcciones realizadas.

Código PHP:

Ver original<?php
session_start();
if(isset ($_SESSION['id'])) {
?>
 
<!DOCTYPE html>
<link rel="stylesheet" href="../css/estilo.css" type="text/css" media="all">
<html lang="en">
<head>
<title>Mi perfil de operario</title>
<body><a href ="../view/login.php" class = "dos">Cerrar Sesion...</a>
    <p id ="title">Bievenido operario
    <?php
    echo $_SESSION['name'] ;
    echo " " ;
    echo $_SESSION['apellido'];
    ?>
    </p>
 
<?php include("../controller/crear.php"); 
  $id = @$_SESSION["id"]; 
  $ssql = "select * from users where id= ". $id; $editar = mysql_query($ssql);
  $ssql = "select * from users where id= 1". $id.$editar = mysql_query($ssql); 
  $ssql=mysql_query("UPDATE usuarios SET nombre = '$nombre', apellido = '$apellido' usuario = '$usuario', contraseña = '$contraseña', WHERE ID_USUARIO = $ID_USUARIO")OR die (mysql_error());
?> 
 
 
<form action="../controller/crear.php" method="post"> <input type="hidden" name="id" value="<?php echo $id;?>"> 
<br /> 
  Nombre: 
<br />
    <input type="text" name="nombre" value="<?php echo  @$name =nombre;?>"> 
<br /> 
  Apellido: 
<br />
    <input type="text" name="apellido" value="<?php echo @$name =apellido;?>"> 
<br />
  Usuario: 
<br />
    <input type="text" name="usuario" value="<?php echo @$name =usuario;?>"> 
<br/>
  Contraseña: 
<br />
    <input type="text" name="contraseña" value="<?php echo @$name =contraseña;?>"> 
<br/>
 
<center> <input class="form-btn" name="submit" type="submit" value="Actualizar Informacion" /></center>
 
<!-- footer -->
            <footer>
                <div class="wrapper">
                    <section class="lower">
                        <h3> <span></span></h3>
                      
  
                    </section>
                    <section class="lower">
                        <h3></h3>
 
</html>
<?php
}else{echo "Lo sentimos, debes iniciar sesion antes de acceder a esta pagina"; } ?>

Anexo codigo del archivo crear.php

Código PHP:

Ver original<?php
 
include ("../model/wish.php");
 
 
 
$nombre= $_POST['nombre'];
$apellido= $_REQUEST['apellido'];
$usuario= $_REQUEST['usuario'];
$pass1= $_REQUEST['pass1'];
$pass2= $_REQUEST['pass2'];
 
 
$wish = new wish(); 
$igual= $wish->verificar($pass1, $pass2); 
 
if($igual == true){
    
   
$wish->registrar($nombre, $apellido, $usuario, $pass1); 
 
 
}else if ($igual == false){
    
    echo "<h1> LAS CONTRASEÑAS NO COINCIDEN</h1>";
    
}
 
 
 
 
 
 
 
 
 
?>

Gracias.

rodocoyote15 · #9 (**permalink**) 25/02/2016, 12:07

Y dónde recibes los valores que le atribuyes a $nombre por ejemplo? Veo que tienes ahí unas consultas sql, imagino que ya tienes incluido tu archivo de conexión a la bd? Si no, estamos en 0 en todo

chronos682 · #10 (**permalink**) 25/02/2016, 12:40

Cada vez tienes más errores en el código. Sigues sin usar el if isset para evitar que haga el update sin haberse hecho submit del formulario. Antes del update estas haciendo dos consultas a la bd que no veo cuál es la funcionalidad. El include está apuntando a una dirección que no existe. En el update estás usando variables que no has definido ybque no tienen ningún valor. Son muchas cosas las que tienes que corregir.

Hernand67890 · #11 (**permalink**) 25/02/2016, 17:35

Muy amable por sus indicaciones.

he realizado las correspodientes modificaciones, cambiando el type por "if isset"

Anexo el codigo del archivo editar.php

Código PHP:

Ver original<?php
session_start();
if(isset ($_SESSION['id'])) {
?>
 
<!DOCTYPE html>
<link rel="stylesheet" href="../css/estilo.css" type="text/css" media="all">
<html lang="en">
<head>
<title>Mi perfil de operario</title>
<body><a href ="../view/login.php" class = "dos">Cerrar Sesion...</a>
    <p id ="title">Bievenido operario
    <?php
    echo $_SESSION['name'] ;
    echo " " ;
    echo $_SESSION['apellido'];
    ?>
    </p>
 
<?php include("../controller/crear.php"); 
  $id = @$_SESSION["id"]; 
  $ssql = "select * from users where id= ". $id; $editar = mysql_query($ssql);
  $ssql = "select * from users where id= 1". $id.$editar = mysql_query($ssql); 
  //$ssql=mysql_query("UPDATE usuarios SET nombre = '$nombre', apellido = '$apellido' usuario = '$usuario', contraseña = '$contraseña', WHERE ID_USUARIO = $ID_USUARIO")OR die (mysql_error());
?> 
 
 
<form action="../controller/crear.php" method="post"> <input type="if isset" name="id" value="<?php echo $id;?>"> 
<br /> 
  Nombre: 
<br />
    <input type="text" name="nombre" value="<?php echo  @$name =nombre;?>"> 
<br /> 
  Apellido: 
<br />
    <input type="text" name="apellido" value="<?php echo @$name =apellido;?>"> 
<br />
  Usuario: 
<br />
    <input type="text" name="usuario" value="<?php echo @$name =usuario;?>"> 
<br/>
  Contraseña: 
<br />
    <input type="text" name="contraseña" value="<?php echo @$name =contraseña;?>"> 
<br/>
 
<center> <input class="form-btn" name="submit" type="submit" value="Actualizar Informacion" /></center>
 
<!-- footer -->
            <footer>
                <div class="wrapper">
                    <section class="lower">
                        <h3> <span></span></h3>
                      
  
                    </section>
                    <section class="lower">
                        <h3></h3>
 
</html>
<?php
}else{echo "Lo sentimos, debes iniciar sesion antes de acceder a esta pagina"; } ?>

Anexo la imagen la cual me muestra solo dos errores y el formulario pero sin los datos almacenados en la base de datos.

Anexo el codigo crear.php el cual va apuntado el archivo editar.php desde el include.

Código PHP:

Ver original<?php
 
include ("../model/wish.php");
 
 
 
$nombre= $_POST['nombre'];
$apellido= $_REQUEST['apellido'];
$usuario= $_REQUEST['usuario'];
$pass1= $_REQUEST['pass1'];
$pass2= $_REQUEST['pass2'];
 
 
$wish = new wish(); 
$igual= $wish->verificar($pass1, $pass2); 
 
if($igual == true){
    
   
$wish->registrar($nombre, $apellido, $usuario, $pass1); 
 
 
}else if ($igual == false){
    
    echo "<h1> LAS CONTRASEÑAS NO COINCIDEN</h1>";
    
}
 
 
 
 
 
 
 
 
 
?>

Anexo el codigo wish.php el cual apunta hacia la base de datos.

Código PHP:

Ver original<?php
 
include 'conexion.php'; 
 
class wish extends conexion {
        
        private $fine; 
    private $user ; 
    private $password;
    private $consulta ;
    private $conexion;
    private $row, $pas1 , $pas2;
        private $penombre, $peapellido, $peusuario, $pepassword, $verificar ; 
 
    public function wish(){
 
    $conect = new conexion();
    $this->conexion = $conect -> conectar();
    $conect -> seleccion_db();  
 
    }
 
    public  function login($usuario, $pass){
        $this->user = $usuario;
        $this->password= $pass;
        $this->consulta= mysql_query("SELECT usuario , password FROM wisher where usuario = '".$this->user."' and  password = '".$this->password."'", $this->conexion); 
                if($this->row = mysql_fetch_array($this->consulta)){
                    session_start();
                    $this->consulta = mysql_query("SELECT * FROM wisher where usuario = '".$this->user."' ",  $this->conexion);  
                    $this->row = mysql_fetch_array($this->consulta);
                    //Id
                    $this->id = $this->row['id'];
                    $_SESSION['id'] = $this->id;
                    $this->ini = 1;
                    //Nombre
                    $this->id = $this->row['nombre'];
                    $_SESSION['name'] = $this->id; 
                    //apellido
                    $this->id = $this->row['apellido'];
                    $_SESSION['apellido'] = $this->id; 
                         header("Location: ../Aplication/index.html");      
                }else{
                        echo " usuario o password incorrecto ";
 
                        
                }
                return $this->consulta ;
    }
 
        public function registrar($nom, $ape, $use, $pass){
            
            $this->penombre = $nom;
            $this->peapellido = $ape;
            $this->peusuario = $use;
            $this->pepassword = $pass;
            
            $this->verificar = mysql_query("SELECT usuario FROM wisher where usuario = '".$this->peusuario."' " ,  $this->conexion );
 
            if($this->row = mysql_fetch_array($this->verificar)){
                
                echo "<h1>EL USUARIO QUE ACABA DE INGRESAR YA EXISTE</h1>";  
                
            }else if(!$this->row = mysql_fetch_array($this->verificar)){
            
                mysql_query("INSERT INTO wisher(nombre, apellido, usuario, password) values('$this->penombre', '$this->peapellido', '$this->peusuario' , '$this->pepassword')", $this->conexion );
                
              
                
               header("location: ../view/login.php");
            
            }
            
            
            
            
            
            
        }
        
        
        public function verificar($pass1 , $pass2){
            
            $this->fine = false;
            
            $this->pas1 = $pass1;
            $this->pas2 = $pass2;
            
            
            if($this->pas1 == $this->pas2){
                
                $this->fine = true ; 
                
                
            }
            
            
            return $this->fine ; 
        }

Gracias.

rodocoyote15 · #12 (**permalink**) 25/02/2016, 19:29

Olvidate de todo esto y vamos a leer un poco.

Manual de php

Tienes más errores que código funcional :$

pateketrueke · #13 (**permalink**) 25/02/2016, 19:32

Cita:

Iniciado por rodocoyote15

Olvidate de todo esto y vamos a leer un poco.

Manual de php

Tienes más errores que código funcional :$

Completamente de acuerdo.

Por favor, Hernand67890, debes aprender la sintaxis básica del lenguaje antes de seguir modificándolo sin entender nada.

Los errores de sintaxis son tan graves que por muchas ganas que uno tenga de ayudarte será imposible llegar a algo.