Error en uno de mis programas

ghacke · #1 (**permalink**) 24/07/2013, 15:29

Hola a todos, bueno, les cuento:

Tengo que hacer un programa que permita que se registren estudiantes, y que luego se puedan mostrar. Debe ser con ArrayList. Hasta ahora voy haciendo esto:

CLASE PERSONA

Código PHP:

  package Obligatorio; 
import java.util.Scanner; 
 
public class Persona { 
     
    Scanner Teclado = new Scanner (System.in); 
 
    Persona Persona = new Persona(); 
 
    String Cédula; 
    String Nombre; 
    String Apellido; 
    int Teléfono; 
    int Op; 
    boolean Op1; 
 
 
    public Persona(){ 
            Cédula = ""; 
            Nombre = ""; 
            Apellido = ""; 
            Teléfono = 0; 
 
    } 
     
    public void setCédula(String Cédula){ 
            this.Cédula = Cédula; 
    } 
    public void setNombre(String Nombre){ 
            this.Nombre = Nombre; 
    } 
    public void setApellido(String Apellido){ 
            this.Apellido = Apellido; 
    } 
    public void setTeléfono(int Teléfono){ 
            this.Teléfono = Teléfono; 
    } 
    public String getCédula(){ 
            return this.Cédula; 
    } 
    public String getNombre(){ 
            return this.Nombre; 
    } 
    public String getApellido(){ 
            return this.Apellido; 
    } 
    public int getTeléfono(){ 
            return this.Teléfono; 
    } 
     
    //Métodos 
     
    public void RegistrarPersona(){ 
     
 
         
        System.out.println("Ingrese la Cédula: "); 
        Persona.setCédula(Teclado.next()); 
        System.out.println("Ingrese el Nombre: "); 
        Persona.setNombre(Teclado.next()); 
        System.out.println("Ingrese el Apellido: "); 
        Persona.setApellido(Teclado.next()); 
        System.out.println("Ingrese el Teléfono: "); 
        Persona.setTeléfono(Teclado.nextInt()); 
    } 
     
    public void RegistrarOtro(){ 
         
        Op1 = true; 
         
        do { 
         
        System.out.println("Desea Registrar Otro?"); 
        System.out.println("1 - Si, 2 - No"); 
        Op = Teclado.nextInt(); 
         
        if(Op != 1 && Op != 2){ 
             
            System.out.println("Error, debe indicar una de las opciones..."); 
            Op1 = false; 
             
        } 
         
        }while(Op1 == false); 
 
    } 
     
}

CLASE ESTUDIANTE

Código PHP:

  package Obligatorio; 
 
import java.util.ArrayList; 
 
public class Estudiante extends Persona { 
     
    ArrayList <Persona> Estudiantes = new ArrayList <Persona>(); 
     
    String Grupo; 
     
    public Estudiante(){ 
        Grupo = ""; 
    } 
     
    public void setGrupo(String Grupo){ 
        this.Grupo = Grupo; 
    } 
    public String getGrupo(){ 
        return this.Grupo; 
    } 
     
    //Métodos 
     
    public void RegistrarEstudiante(){ 
         
         
        do{ 
        System.out.println("====================Registrar Estudiante===================="); 
        RegistrarPersona(); 
        System.out.println("Ingrese el Grupo: "); 
        ((Estudiante) Persona).setGrupo(Teclado.next()); 
         
        Estudiantes.add(Persona); 
         
        RegistrarOtro(); 
         
        }while(Op == 1); 
         
        System.out.println("Eligió Salir"); 
        Menú objMenu = new Menú (); 
        objMenu.Menu(); 
    } 
     
    public void MostrarEstudiantes(){ 
         
        for (int i = 0; i > Estudiantes.size(); i++){ 
             
            System.out.println("============================================================"); 
            System.out.println("C.I.: "+Persona.getCédula()); 
            System.out.println("Nombre: "+Persona.getNombre()); 
            System.out.println("Apellido: "+Persona.getApellido()); 
            System.out.println("Teléfono: "+Persona.getTeléfono()); 
            System.out.println("Grupo: "+((Estudiante)Persona).getGrupo()); 
        } 
         
    } 
}

Estas clases luego las invoco desde otra clase main. La clase Persona está porque además de Estudiantes debo registrar Profesores entonces uso a la Herencia para ahorrar código.

Bueno el problema que me da es cuando declaro el objeto Persona

Código PHP:

  Persona Persona = new Persona();

La verdad es que no entiendo por que me da ese error, soy nuevo en esto de Java, espero que me puedan responder la duda.

Gracias :D

Xerelo · #2 (**permalink**) 25/07/2013, 05:14

En general tu código es un descalabro, y cuando aparece un error viene además la descripción del error, que te sirve a ti para saber qué está pasando y para que los demás puedan ayudarte.

Tengo un error aquí sin más información, sólo sirve para que la gente tenga que hacer suposiciones a menos que el error sea muy evidente.

Lee la descripción y si no te llega, postéala en el foro.

calichecal · #3 (**permalink**) 25/07/2013, 11:06

Xerelo tiene razón, muéstranos el error que arroja la consola.

Krain · #4 (**permalink**) 26/07/2013, 08:15

Hola amigo, antes de nada en java por convenio de estilo (o un nombre así, corregidme si esta mal dicho) los nombres de las variables van en minúsculas y sin acentos, y las mayúsculas solo se usan para las clases y para separar palabras.

En el método de crear, no es que este mal pero yo lo haría creando una persona/estudiante con un constructor con parámetros en vez de crearlo vació e ir seteando.

En el método de mostrar te tirara error seguro, tienes que usar el toString().

Bueno aquí te dejo el programa a ver si es lo que buscabas.

Código Java:

Ver originalpackage Obligatorio; 
import java.util.ArrayList;
import java.util.Scanner; 
 
public class Persona
{ 
    protected Scanner teclado = new Scanner (System.in);
    protected String cedula; 
    protected String nombre; 
    protected String apellido; 
    protected int telefono; 
    protected int op; 
    protected boolean op1; 
    protected ArrayList <Persona> personas = new ArrayList(); 
 
    public Persona(){
        
    }
    public Persona(String cedula, String nombre, String apellido, int telefono)
    {
        this.cedula = cedula;
        this.nombre = nombre;
        this.apellido = apellido;
        this.telefono = telefono;
    }
    public void registrarPersona()
    { 
        System.out.println("Ingrese la Cédula: "); 
        String cedula = teclado.next(); 
        System.out.println("Ingrese el Nombre: "); 
        String nombre = teclado.next(); 
        System.out.println("Ingrese el Apellido: "); 
        String apellido = teclado.next(); 
        System.out.println("Ingrese el Teléfono: "); 
        int telefono = teclado.nextInt();         
        personas.add(new Persona(cedula, nombre, apellido, telefono));
    }      
    public void registrarOtro()
    {          
        op1 = true;          
        do 
        { 
            System.out.println("Desea Registrar Otro?");
            System.out.println("1 - Si, 2 - No"); 
            op = teclado.nextInt(); 
            if(op != 1 && op != 2)
            { 
                System.out.println("Error, debe indicar una de las opciones...");
                op1 = false;  
            }    
        }
        while(op1 == false); 
    }
    public void mostrarPersonas()
    {          
        for (Persona p: personas)
        {
            System.out.println(p.toString());
        }          
    }
    @Override
    public String toString()
    {
        return "C.I.: " + cedula + "\n" + "Nombre: " + nombre + "\n" + "Apellido: " + apellido + "\n" + "Teléfono: " + telefono + "\n" + "============================================================" + "\n";
    }
    public void setCedula(String cedula)
    { 
        this.cedula = cedula; 
    } 
    public void setNombre(String nombre)
    { 
        this.nombre = nombre; 
    } 
    public void setApellido(String apellido)
    { 
        this.apellido = apellido; 
    } 
    public void setTelefono(int telefono)
    { 
        this.telefono = telefono; 
    } 
    public String getCedula()
    { 
        return cedula; 
    } 
    public String getNombre()
    { 
        return nombre; 
    } 
    public String getApellido()
    { 
        return apellido; 
    } 
    public int getTelefono()
    { 
        return telefono; 
    }      
}

Código Java:

Ver originalpackage Obligatorio; 
import java.util.ArrayList; 
 
public class Estudiante extends Persona
{      
    ArrayList <Persona> estudiantes = new ArrayList();     
    String grupo; 
     
    public Estudiante(){
        
    }
    public Estudiante(String cedula, String nombre, String apellido, int telefono, String grupo)
    {
        this.cedula = cedula;
        this.nombre = nombre;
        this.apellido = apellido;
        this.telefono = telefono;
        this.grupo = grupo;                       
    }
    public void registrarEstudiante()
    {
        do
        {
            System.out.println("====================Registrar Estudiante===================="); 
            System.out.println("Ingrese la Cédula: "); 
            String cedula = teclado.next(); 
            System.out.println("Ingrese el Nombre: "); 
            String nombre = teclado.next(); 
            System.out.println("Ingrese el Apellido: "); 
            String apellido = teclado.next(); 
            System.out.println("Ingrese el Teléfono: "); 
            int telefono = teclado.nextInt();                     
            System.out.println("Ingrese el Grupo: "); 
            String grupo = teclado.next();
            estudiantes.add(new Estudiante(cedula, nombre, apellido, telefono, grupo)); 
            registrarOtro(); 
        }
        while(op == 1);         
        System.out.println("Eligió Salir"); 
    }      
    public void mostrarEstudiantes()
    {          
        for (Persona e: estudiantes)
        {
            System.out.println(e.toString());
        }          
    } 
    @Override
    public String toString()
    {
        return "C.I.: " + cedula + "\n" + "Nombre: " + nombre + "\n" + "Apellido: " + apellido + "\n" + "Teléfono: " + telefono + "\n" + "Grupo: " + grupo + "\n" + "============================================================" + "\n";
    }            
    public void setGrupo(String grupo)
    { 
        this.grupo = grupo; 
    } 
    public String getGrupo()
    { 
        return grupo; 
    } 
}

Código Java:

Ver originalpackage Obligatorio;
 
import java.util.Scanner;
 
public class Main
{
    public static void main(String[]args)
    {
        Scanner tec = new Scanner(System.in);
        Estudiante e = new Estudiante(); 
        int menu = 0;
        do
        {
            System.out.println("Pulsa 1 para registrar un profesor");
            System.out.println("Pulsa 2 para registrar un estudiante");
            System.out.println("Pulsa 3 para ver la lista de los profesores");
            System.out.println("Pulsa 4 para ver la lista de los alumnos");
            System.out.println("Pulsa 0 para salir");
            menu = tec.nextInt();
            switch(menu)
            {
                case 1:
                    e.registrarPersona();
                    break;
                case 2:
                    e.registrarEstudiante();
                    break;
                case 3:
                    e.mostrarPersonas();
                    break;
                case 4:
                    e.mostrarEstudiantes();
                    break;
                case 0:                    
                    break;
                default:
                    System.out.println("Opcion no válida");
            }
        }
        while(menu != 0);
    }
}

A mi con esto me tira bien el crear personas y estudiantes cada uno en un arraylist diferente y mostrarlos todos. Incluso si solo quisieras mostrar nose, por ejemplo el apellido solo tendrias que crear un metodo que fuera:

Código Java:

Ver originalpublic void mostrarEstudiantesApellido()
    {          
        for (Persona e: estudiantes)
        {
            System.out.println(e.getApellido());
        }          
    }

si quieres otra cosa solo seria ir creando este método cambiando el e.getApellido() por lo que quieras mostrar.

Espero haberte ayudado.
Edit: Editado para dar colorines al codigo.