diff --git a/si-ejb/src/java/hola/BL/RolBL.java b/si-ejb/src/java/hola/BL/RolBL.java
index 5cd5ae8..efa7a98 100644
--- a/si-ejb/src/java/hola/BL/RolBL.java
+++ b/si-ejb/src/java/hola/BL/RolBL.java
@@ -4,9 +4,15 @@
  */
 package hola.BL;
 
+import hola.dao.RolDAO;
 import hola.dao.UsuarioDAO;
+import hola.modelo.Rol;
 import hola.msg.Mensaje;
+import java.util.List;
+import java.util.Optional;
 import javax.ejb.Stateless;
+import javax.faces.application.FacesMessage;
+import javax.faces.context.FacesContext;
 
 /**
  *
@@ -22,24 +28,18 @@ public class RolBL implements RolBLLocal {
         // Comentario de lógica: Método para agregar un proveedor
 //, Registro registro//
         System.out.println("Llegaste al metodo de agregar Rol");
-        RolDAO usuarioDAO = new R;
+        RolDAO usuarioDAO = new RolDAO();
         Mensaje m = null;
 
         // Validar que los campos requeridos no estén vacíos
-        if (rol.getNombre().isEmpty()
-                || rol.getSexo()== null
-                || rol.getTelefono().isEmpty()
-                || rol.getCorreo().isEmpty()
-                || rol.getDireccion().isEmpty()
-                || rol.getNpila().isEmpty()
-                || rol.getUsername().isEmpty()) {
+        if (rol.getNombre().isEmpty()) {
             System.out.println(Mensaje.CAMPOS_INCOMPLETOS);
             // Enviar mensaje de error si algún campo está incompleto
             return Mensaje.CAMPOS_INCOMPLETOS;
         }
 
         // Buscar si el proveedor ya existe en la base de datos
-        Optional<Usuario> usuarioEncontradoOptional = usuarioDAO.buscarUsuario(rol);
+        Optional<Rol> usuarioEncontradoOptional = usuarioDAO.buscarRol(rol);
 
         if (usuarioEncontradoOptional.isPresent()) {
             // Manejar el caso en el que se encontró al menos un proveedor duplicado
@@ -57,17 +57,17 @@ public class RolBL implements RolBLLocal {
     }
 
     @Override
-    public Usuario buscarId(Usuario usuario) {
+    public Rol buscarId(Rol rol) {
         // Comentario de lógica: Método para buscar un proveedor por ID
-        UsuarioDAO udao = new UsuarioDAO();
-        return udao.buscarPorId(usuario);
+        RolDAO rdao = new RolDAO();
+        return rdao.buscarPorId(rol);
     }
 
     @Override
-    public void eliminarId(Usuario usuario) {
+    public void eliminarId(Rol rol) {
         // Comentario de lógica: Método para eliminar un proveedor por ID
-        UsuarioDAO udao = new UsuarioDAO();
-        if (udao.eliminar(usuario)) {
+        RolDAO rdao = new RolDAO();
+        if (rdao.eliminar(rol)) {
             System.out.println(Mensaje.SIN_ERROR);
         } else {
             System.out.println(Mensaje.NO_EXISTE);
@@ -75,21 +75,16 @@ public class RolBL implements RolBLLocal {
     }
 
     @Override
-    public Mensaje editar(Usuario usuario) {
+    public Mensaje editar(Rol rol) {
         // Comentario de lógica: Método para editar un proveedor
         Mensaje m;
-        UsuarioDAO usuarioDAO = new UsuarioDAO();
+        RolDAO rolDAO = new RolDAO();
 
         // Validar que los campos requeridos no estén vacíos
-        if (usuario.getNombre().isEmpty()
-                || usuario.getSexo()== null
-                || usuario.getNombre().isEmpty()
-                || usuario.getTelefono().isEmpty()
-                || usuario.getCorreo().isEmpty()
-                || usuario.getDireccion().isEmpty()
-                || usuario.getNpila().isEmpty()
-                || usuario.getUsername().isEmpty()
-                || usuario.getStatus() == null) {
+        if (rol.getNombre().isEmpty()
+                
+                || rol.getNombre().isEmpty()             
+                || rol.getStatus() == null) {
 
             // Enviar mensaje de error si algún campo está incompleto
             m = Mensaje.CAMPOS_INCOMPLETOS;
@@ -99,14 +94,14 @@ public class RolBL implements RolBLLocal {
 
         try {
             // Editar el proveedor solo si todas las validaciones son exitosas
-            Optional<Usuario> user = usuarioDAO.buscarUsuario(usuario);
+            Optional<Rol> user = rolDAO.buscarRol(rol);
             if (user.isPresent()) {
 
                 addMessage(FacesMessage.SEVERITY_ERROR, "Error", "Elemento duplicado");
                 m = Mensaje.ELEMENTO_DUPLICADO;
             } else {
 
-                if (usuarioDAO.editar(usuario) == true) {
+                if (rolDAO.editar(rol) == true) {
                     m = Mensaje.SIN_ERROR;
                     System.out.println(Mensaje.SIN_ERROR);
                     return m;
@@ -127,15 +122,15 @@ public class RolBL implements RolBLLocal {
     }
 
     @Override
-    public List<Usuario> buscarValidos(boolean estado) {
-        UsuarioDAO usuarioDAO = new UsuarioDAO();
-        List l = usuarioDAO.buscarValidos(estado);
+    public List<Rol> buscarValidos(boolean estado) {
+        RolDAO rolDAO = new RolDAO();
+        List l = rolDAO.buscarValidos(estado);
         // Llama al método buscarValidos de ProductoDAO y devuelve la lista resultante
         System.out.println(l);
         return l;
     }
 
     public void addMessage(FacesMessage.Severity severity, String summary, String detail) {
-        FacesContext.getCurrentInstance().addMessage("MensajeUsuario", new FacesMessage(severity, summary, detail));
+        FacesContext.getCurrentInstance().addMessage("MensajeRol", new FacesMessage(severity, summary, detail));
     }
 }
diff --git a/si-ejb/src/java/hola/BL/RolBLLocal.java b/si-ejb/src/java/hola/BL/RolBLLocal.java
index 6ebe572..b42524a 100644
--- a/si-ejb/src/java/hola/BL/RolBLLocal.java
+++ b/si-ejb/src/java/hola/BL/RolBLLocal.java
@@ -4,6 +4,9 @@
  */
 package hola.BL;
 
+import hola.modelo.Rol;
+import hola.msg.Mensaje;
+import java.util.List;
 import javax.ejb.Local;
 
 /**
@@ -13,4 +16,16 @@ import javax.ejb.Local;
 @Local
 public interface RolBLLocal {
     
+
+
+    public Mensaje agregar(Rol rol);
+
+    public Rol buscarId(Rol rol);
+
+    public void eliminarId(Rol rol);
+
+    public Mensaje editar(Rol rol);
+
+    public List<Rol> buscarValidos(boolean estado);
 }
+
diff --git a/si-ejb/src/java/hola/BL/UsuarioBL.java b/si-ejb/src/java/hola/BL/UsuarioBL.java
index 442ecbd..28b4928 100644
--- a/si-ejb/src/java/hola/BL/UsuarioBL.java
+++ b/si-ejb/src/java/hola/BL/UsuarioBL.java
@@ -37,7 +37,10 @@ public class UsuarioBL implements UsuarioBLLocal {
                 || usuario.getCorreo().isEmpty()
                 || usuario.getDireccion().isEmpty()
                 || usuario.getNpila().isEmpty()
-                || usuario.getUsername().isEmpty()) {
+                || usuario.getUsername().isEmpty()
+                || usuario.getRolIdrol().getNombre().isEmpty()
+                || usuario.getPaisId().getNombre().isEmpty()
+                || usuario.getEstadoIdestado().getNombre().isEmpty()) {
             System.out.println(Mensaje.CAMPOS_INCOMPLETOS);
             // Enviar mensaje de error si algún campo está incompleto
             return Mensaje.CAMPOS_INCOMPLETOS;
@@ -55,7 +58,9 @@ public class UsuarioBL implements UsuarioBLLocal {
             // Agregar el proveedor a la base de datos si no está duplicado
             usuarioDAO.agregar(usuario);
             //registroDAO.agregar(registro);
+            
             m = Mensaje.SIN_ERROR;
+            System.out.println(m);
         }
 
         return m;
@@ -94,7 +99,10 @@ public class UsuarioBL implements UsuarioBLLocal {
                 || usuario.getDireccion().isEmpty()
                 || usuario.getNpila().isEmpty()
                 || usuario.getUsername().isEmpty()
-                || usuario.getStatus() == null) {
+                || usuario.getStatus() == null
+                || usuario.getPaisId().getNombre().isEmpty()
+                || usuario.getEstadoIdestado().getNombre().isEmpty()
+                || usuario.getRolIdrol().getNombre().isEmpty()) {
 
             // Enviar mensaje de error si algún campo está incompleto
             m = Mensaje.CAMPOS_INCOMPLETOS;
diff --git a/si-ejb/src/java/hola/dao/RolDAO.java b/si-ejb/src/java/hola/dao/RolDAO.java
index ae4ed09..7c74759 100644
--- a/si-ejb/src/java/hola/dao/RolDAO.java
+++ b/si-ejb/src/java/hola/dao/RolDAO.java
@@ -107,12 +107,12 @@ public class RolDAO {
         Query q1 = em.createNamedQuery("Rol.findByNombre");
         q1.setParameter("nombre", rol.getNombre());
 
-        List<Tipo> resultadoNombre = q1.getResultList();
+        List<Rol> resultadoNombre = q1.getResultList();
 
         // Verifica si hay resultados para todos los campos
         if (!resultadoNombre.isEmpty()) {
             // Compara si la misma editorial está en los tres resultados
-            Tipo tipoEncontrado = resultadoNombre.stream()
+            Rol tipoEncontrado = resultadoNombre.stream()
                     .findFirst()
                     .orElse(null);
 
diff --git a/si-war/src/java/hola/vista/DemoBeanUsuario.java b/si-war/src/java/hola/vista/DemoBeanUsuario.java
index 62e79fa..454fac1 100644
--- a/si-war/src/java/hola/vista/DemoBeanUsuario.java
+++ b/si-war/src/java/hola/vista/DemoBeanUsuario.java
@@ -62,7 +62,7 @@ public class DemoBeanUsuario implements Serializable {
       
         usuario.setStatus(1);
          usuarioBL.agregar(usuario);
-       // limpiarFormulario();
+       //limpiarFormulario();
        Mensaje mensaje = usuarioBL.agregar(usuario);
 
         switch (mensaje) {
diff --git a/si-war/web/UsuarioCrear.xhtml b/si-war/web/UsuarioCrear.xhtml
index 50f80fd..5cd80ba 100644
--- a/si-war/web/UsuarioCrear.xhtml
+++ b/si-war/web/UsuarioCrear.xhtml
@@ -49,8 +49,8 @@
                                              required="true"
                                              requiredMessage="El campo nombre es obligatorio"
                                              validatorMessage="M o F">
-                                    <f:validateRegex pattern="[a-zA-Z ]+" />
-                                    <f:validateLength minimum="3" maximum="15" />
+                                    
+                                    
                                 </p:inputText>
                                 <p:message for="sexo" />
 
@@ -88,43 +88,48 @@
                                 </p:inputText>
                                 <p:message for="direccion" />
 
-                                <p:outputLabel value="Estado:" for="estadoId" />
-                                <p:inputText id="estadoId" value="#{demoBeanUsuario.TUsuario.estadoIdestado}" 
-                                             title="Estado"
+                                <p:outputLabel value="Username:" for="username" />
+                                <p:inputText id="username" value="#{demoBeanUsuario.TUsuario.username}" 
+                                             title="Username"
                                              required="true"
                                              requiredMessage="El campo nombre es obligatorio"
-                                             validatorMessage="M o F">
-                                    <f:validateRegex pattern="[a-zA-Z ]+" />
+                                             validatorMessage="El campo nombre no debe incluir números, debe incluir mínimo 3 letras y máximo 15 letras">
+                                    <f:validateRegex pattern="[a-zA-Z0-9 ]+" />
                                     <f:validateLength minimum="3" maximum="15" />
                                 </p:inputText>
-                                <p:message for="estadoId" />
+                                <p:message for="username" />
+                                
+                                <p:outputLabel value="Password:" for="password" />
+                                <p:inputText id="password" value="#{demoBeanUsuario.TUsuario.npila}" 
+                                             title="Password"
+                                             required="true"
+                                             requiredMessage="El campo nombre es obligatorio"
+                                             validatorMessage="El campo nombre no debe incluir números, debe incluir mínimo 3 letras y máximo 15 letras">
+                                    <f:validateRegex pattern="[a-zA-Z0-9 ]+" />
+                                    <f:validateLength minimum="3" maximum="15" />
+                                </p:inputText>
+                                <p:message for="password" />
 
                                 <p:outputLabel value="Estado:" for="estadoId"/>
-                                <p:selectOneMenu id="estadoId" value="#{demoBeanUsuario.TUsuario.paisId}" converter="selectOneMenuConverter" 
+                                <p:selectOneMenu id="estadoId" value="#{demoBeanUsuario.TUsuario.estadoIdestado}" converter="selectOneMenuConverter" 
                                                  required="true" requiredMessage="Este campo es requerido">
-                                    <f:selectItems value="#{demoBeanEstado}" var="item" itemLabel="#{item.nombre}"  itemValue="#{item}"/>
+                                    <f:selectItems value="#{demoBeanEstado.estados}" var="item" itemLabel="#{item.nombre}"  itemValue="#{item}"/>
                                 </p:selectOneMenu>
                                 <p:message for="estadoId"/>
 
                                 <p:outputLabel value="Pais:" for="paisId"/>
-                                <p:selectOneMenu id="paisId" value="#{demoBeanUsuario.TUsuario.estadoIdestado}" converter="selectOneMenuConverter" 
+                                <p:selectOneMenu id="paisId" value="#{demoBeanUsuario.TUsuario.paisId}" converter="selectOneMenuConverter" 
                                                  required="true" requiredMessage="Este campo es requerido">
-                                    <f:selectItems value="#{demoBeanPais}" var="item" itemLabel="#{item.nombre}"  itemValue="#{item}"/>
+                                    <f:selectItems value="#{demoBeanPais.paises}" var="item" itemLabel="#{item.nombre}"  itemValue="#{item}"/>
                                 </p:selectOneMenu>
                                 <p:message for="paisId"/>
                                 
                                 <p:outputLabel value="Rol:" for="rolId"/>
                                 <p:selectOneMenu id="rolId" value="#{demoBeanUsuario.TUsuario.rolIdrol}" converter="selectOneMenuConverter" 
                                                  required="true" requiredMessage="Este campo es requerido">
-                                    <f:selectItems value="#{demoBean}" var="item" itemLabel="#{item.nombre}"  itemValue="#{item}"/>
+                                    <f:selectItems value="#{demoBeanRol.rols}" var="item" itemLabel="#{item.nombre}"  itemValue="#{item}"/>
                                 </p:selectOneMenu>
-                                <p:message for="paisId"/>
-
-                                <input type="file" value="Foto" for="foto" id="foto" 
-                                       required="true"
-                                       requiredMessage="El campo nombre es obligatorio">
-                                    <button onclick="document.getElementById('myFileInput').click()">Seleccionar Archivo</button>
-                                </input>
+                                <p:message for="rolId"/>
 
                             </h:panelGrid>
                             <p:commandButton action="#{demoBeanUsuario.agregarUsuario()}"