[SOLUCIONADO] Evitar Duplicar datos

[Jose789]

Hola buenas, pues resulta que tengo un formulario donde doy de alta Personas con un nivel de seguridad, cada "Nick" de usuario debe de ser único, pero resulta que el método que tengo dentro de Personal buscarNick(nick) siempre parece dar -1 y siempre me deja dar de alta el mismo usuario hice lo mismo pero con tipo boolean y sigo teniendo el mismo problema.

Dar de alta se da, ya que el 'Vector' aumenta de tamaño..

Main.java

Código Java:

public class Main {
    
    /*Personal*/
    static Vector<Personal> personal = new Vector();
    
    /*Clientes*/
    static Vector<Clientes> clientes = new Vector();
}
 

Personal.java

Código Java:

package Hotel;
import java.io.Serializable;
 
/**
 * Clase Personal
 *
 * @author jose
 */
public class Personal implements Serializable {
    
    private String nick;
    private String passwd;
    private String nivel;
 
 
    public Personal(String nick, String passwd, String nivel) {
        this.nick = nick;
        this.passwd = passwd;
        this.nivel = nivel;
    }    
    Personal(){
    }
 
    public String getNick() {
        return nick;
    }
 
    public void setNick(String nick) {
        this.nick = nick;
    }
 
    public String getPasswd() {
        return passwd;
    }
 
    public void setPasswd(String passwd) {
        this.passwd = passwd;
    }
 
    public String getNivel() {
        return nivel;
    }
 
    public void setNivel(String nivel) {
        this.nivel = nivel;
    }
   
   static public int buscarNick(String nick){
        
        for (int i = 0; i < Main.personal.size(); i++) {
            if( Main.personal.get(i).getNick().equals(nick) ){
                return (i);
            }else{
                return -1;
            }
        }
        return -1;
    }
    
}
 

Form.java

Código Java:

private Personal Personal;
 
if(Personal.buscarNick(nick)==-1){
                
                Personal = new Personal(nick, passwd, nivel);
                Main.personal.add(Personal);
                
                JOptionPane.showMessageDialog(this, "Se añadio correctamente");
            }else{
                System.out.println("El usuario ya existe");
                JOptionPane.showMessageDialog(this, "El usuario ya existe");
            }
 

[Profesor_Falken]

Buenas,

Hay algunas cosas que chocan un poco como Vector, que es una clase muy muy antigua de Java que no se utiliza debido a su mal diseno. Tambien el metodo find deberias sacarlo del bean e incorporarlo en otro componente aparte (Main quizas?). El codigo no es muy OO..., pero en fin, es normal porque estas aprendiendo.

Respecto a tu problema, por el codigo que pones deberia funcionar. Sin embargo sospecho que en la clase Form hay algo mas que haces que no nos estas mostrando... quizas reinicializas el vector en alguna parte? Puedes poner el codigo completo de Form.java?

Un saludo

[Jose789]

Te dejo por aquí todo el form, me acabo de dar cuenta de que el primer usuario que doy de alta si lo vuelvo a poner me dice perfectamente que el nombre ya existe en cambio si añado otro nuevo e intento con ese me deja crearlo y así sucesivamente.

https://pastebin.com/Tke39Grz

Si, soy novato en esto de la programación y actualmente estoy aprendiendo y puede que veas cada cosa que te quedes loco jaja. Espero que aparte de ayudarme en encontrar el error también me puedas dar algún consejo...

Veré como funcionan los arrayList para ir dejando a un lado el Vector.

Un saludo.

Un poco mas resumido.

Código Java:

private void button_add_personalActionPerformed(java.awt.event.ActionEvent evt) {                                                    
        // TODO add your handling code here:
        
 
        String nick = nick_personal.getText();
        String passwd = passwd_personal.getText();
        String rep_passwd = rep_passwd_personal.getText();
        String nivel = nivel_personal.getSelectedItem().toString();
        
        if( (nick.isEmpty()) || (passwd.isEmpty()) || (rep_passwd.isEmpty()) ){
            System.out.println("Campos Vacios");
             JOptionPane.showMessageDialog(this, "Los campos no pueden estar vacios");
        }else if( passwd.equals(nick)){
            System.out.println("La contraseña no puede ser igual al nombre de usuario");
             JOptionPane.showMessageDialog(this, "La contraseña no puede ser igual al nombre de usuario");
        }else if( !(passwd.equals(rep_passwd)) ){
             JOptionPane.showMessageDialog(this, "Las contraseñas no coinciden");
        }else if( nivel.equals("Nivel") ){
             JOptionPane.showMessageDialog(this, "Debe seleccionar un nivel de seguridad");
        }else{
            
           if(Personal.buscarNick(nick)==-1){
                
                Personal = new Personal(nick, passwd, nivel);
                Main.personal.add(Personal);
                
                JOptionPane.showMessageDialog(this, "Se añadio correctamente");
            }else{
                System.out.println("El usuario ya existe");
                JOptionPane.showMessageDialog(this, "El usuario ya existe");
            } 
            
        }
        
        
        
    }
 

Una prueba de ejecución


Si el usuario no existe retornara -1 si el usuario ya existe nos devuelve la pos en la que esta. A partir de ahí es cuando no para de dar siempre -1

[Profesor_Falken]

Ah vaya! acabo de verlo!

Para comprender lo que pasa, te propongo que pongas los siguientes mensajes:

Código Java:

static public int buscarNick(String nick){
        System.out.println("Buscando nick " + nick);
        System.out.println("Personal tiene " + Main.personal.size() + "elementos");
        for (int i = 0; i < Main.personal.size(); i++) {
            System.out.println("Analizando " + Main.personal.get(i).getNick() + "...");    
            if( Main.personal.get(i).getNick().equals(nick) ){                
                return (i);
            }else{
                System.out.println(Main.personal.get(i).getNick() + " no coincide. Retorno -1");     
                return -1;
            }
        }
        return -1;
    }
 

Deberias darte cuenta del problema al ejecutarlo y ver el comportamiento.

Gran pista: que haces si el primer elemento de la lista no se corresponde al nick buscado? acaso sigues buscando el resto de elementos del vector?


Un saludo

[Jose789]

Iniciado por Profesor_Falken

Buenas,

Hay algunas cosas que chocan un poco como Vector, que es una clase muy muy antigua de Java que no se utiliza debido a su mal diseno. Tambien el metodo find deberias sacarlo del bean e incorporarlo en otro componente aparte (Main quizas?). El codigo no es muy OO..., pero en fin, es normal porque estas aprendiendo.

Respecto a tu problema, por el codigo que pones deberia funcionar. Sin embargo sospecho que en la clase Form hay algo mas que haces que no nos estas mostrando... quizas reinicializas el vector en alguna parte? Puedes poner el codigo completo de Form.java?

Un saludo

Para facilitarte las cosas y por aq

Código Java:

private void button_add_personalActionPerformed(java.awt.event.ActionEvent evt) {                                                    
        // TODO add your handling code here:
        
 
        String nick = nick_personal.getText();
        String passwd = passwd_personal.getText();
        String rep_passwd = rep_passwd_personal.getText();
        String nivel = nivel_personal.getSelectedItem().toString();
        
        if( (nick.isEmpty()) || (passwd.isEmpty()) || (rep_passwd.isEmpty()) ){
            System.out.println("Campos Vacios");
             JOptionPane.showMessageDialog(this, "Los campos no pueden estar vacios");
        }else if( passwd.equals(nick)){
            System.out.println("La contraseña no puede ser igual al nombre de usuario");
             JOptionPane.showMessageDialog(this, "La contraseña no puede ser igual al nombre de usuario");
        }else if( !(passwd.equals(rep_passwd)) ){
             JOptionPane.showMessageDialog(this, "Las contraseñas no coinciden");
        }else if( nivel.equals("Nivel") ){
             JOptionPane.showMessageDialog(this, "Debe seleccionar un nivel de seguridad");
        }else{
            
           if(Personal.buscarNick(nick)==-1){
                
                Personal = new Personal(nick, passwd, nivel);
                Main.personal.add(Personal);
                
                JOptionPane.showMessageDialog(this, "Se añadio correctamente");
            }else{
                System.out.println("El usuario ya existe");
                JOptionPane.showMessageDialog(this, "El usuario ya existe");
            } 
            
        }
        
        
        
    }
 

Me acabo de dar cuenta de que el Vector siempre me dice que el tamaño es de 0 elementos por lo que el for nunca se llega a invocar...

Ahora es cuando estoy en blanco....

[Jose789]

Iniciado por Profesor_Falken

Ah vaya! acabo de verlo!

Para comprender lo que pasa, te propongo que pongas los siguientes mensajes:

Código Java:

static public int buscarNick(String nick){
        System.out.println("Buscando nick " + nick);
        System.out.println("Personal tiene " + Main.personal.size() + "elementos");
        for (int i = 0; i < Main.personal.size(); i++) {
            System.out.println("Analizando " + Main.personal.get(i).getNick() + "...");    
            if( Main.personal.get(i).getNick().equals(nick) ){                
                return (i);
            }else{
                System.out.println(Main.personal.get(i).getNick() + " no coincide. Retorno -1");     
                return -1;
            }
        }
        return -1;
    }
 

Deberias darte cuenta del problema al ejecutarlo y ver el comportamiento.

Gran pista: que haces si el primer elemento de la lista no se corresponde al nick buscado? acaso sigues buscando el resto de elementos del vector?


Un saludo

Me acabo de dar cuenta de que el Vector siempre me dice que el tamaño es de 0 elementos por lo que el for nunca se llega a invocar...

Ahora es cuando estoy en blanco....

PD: Fallo mio, vale ya veo que se queda en analizando,
PD2: Vale veo que me sobraba el else, ahora parece que funciona correctamente. corrígeme si me equivoco.

[detective_jd]

Hola, apenas miré un poco el código te diría porque en forma de pregunta: ¿Porque no cambias por un HashMap? y te diré xq, sólo xq funciona mediante por clave-valor, tu puedes hacer busquedas con get o con containskey mediante la clave que sería el nick en tu caso.

https://www.mkyong.com/java/java-che...ts-in-hashmap/

Saludos.

[Profesor_Falken]

PD2: Vale veo que me sobraba el else, ahora parece que funciona correctamente. corrígeme si me equivoco.

Exacto, era eso.

Respecto al hashmap, es util si lo que se pretende hacer es basicamente hacer busquedas rapidas por una clave (fast lookup).

Si lo que se quiere es tener un a coleccion que deberemos recorrer, listar, filtrar, ordenar, etc (que creo que es el caso), lo adecuado es utilizar una lista (un ArrayList seria lo optimo aqui).

Un saludo

[Jose789]

Iniciado por Profesor_Falken

Exacto, era eso.

Respecto al hashmap, es util si lo que se pretende hacer es basicamente hacer busquedas rapidas por una clave (fast lookup).

Si lo que se quiere es tener un a coleccion que deberemos recorrer, listar, filtrar, ordenar, etc (que creo que es el caso), lo adecuado es utilizar una lista (un ArrayList seria lo optimo aqui).

Un saludo

Lo he vuelto a hacer con ArrayList y funciona todo correctamente. Muchas gracias por la ayuda.