revizar este ejercicio

[gobodeath]

Hola pues resulta que me pusieron un ejercicio, pero no sé si me quedó bueno así que aqui os dejo a ver si me pueden echar una manito, igual me corre pero si depronto tienen depronto sugerencias se los agradecería.

"se supone que es para practicar POO"
Ejercicio.
se desea crear una aplicacion para manejar la informacion de un empleado. un empleado tiene un nombre, una cedula, una direccion y un salario asignado, la aplicacion debe permitir crear un nuevo empleado, ingrementar el salario en un 10% decrementar el salario en un porcentaje dado por el usuario, devolver la direccion del empleado y devolver el salario.


e aquí el código

Código Java:

/*clase empleado*/
package Aplication_Body;
 
public class Empleado {
    
    private String nombre;
    private int cedula;
    private String direccion;
    private double salario;
    
    public Empleado(String nombre, int cedula, String direccion, double salario){
        
        super();
        this.nombre = nombre;
        this.cedula = cedula;
        this.direccion = direccion;
        this.salario = salario;
        
    }
 
    public String getNombre() {
        return nombre;
    }
 
    public void setNombre(String nombre) {
        this.nombre = nombre;
    }
 
    public int getCedula() {
        return cedula;
    }
 
    public void setCedula(int cedula) {
        this.cedula = cedula;
    }
 
    public String getDireccion() {
        return direccion;
    }
 
    public void setDireccion(String direccion) {
        this.direccion = direccion;
    }
 
    public double getSalario() {
        return salario;
    }
 
    public void setSalario(double salario) {
        this.salario = salario;
    }
    
}
 

Código Java:

/*esta es la clase principal, donde crearemos el Objeto Empleado1 o e1*/
package Aplication_Body;
import Aplication_Body.Metodos;
 
public class Empresa {
 
    
    public static void main(String[] args) {
        new Empresa();
 
    }
    public Empresa(){
        
        String nombre = Metodos.pedir("ingrese el nombre del empleado");
        int cedula = Metodos.convertirEntero(Metodos.pedir("Ingrese el número de 
 
cedula"));
        String direccion = Metodos.pedir("ingrese la dirección del empleado");
        double salario = Metodos.convertirEntero(Metodos.pedir("ingrese el salario 
 
asignado del empleado"));
        Empleado e1 = new Empleado(nombre, cedula, direccion, salario);
        double salario10 = Metodos.ValorMasPorcentaje(e1.getSalario(), 10);
        double salarioFinal = Metodos.valorMenosPorcentaje(salario10, 
 
Metodos.convertirEntero(Metodos.pedir("Ingrese el porcentaje que le desea decrementar al 
 
salario del empleado")));
        Metodos.mostrar(e1.getNombre()+" "+e1.getCedula()+" "+e1.getDireccion());
        Metodos.mostrar("Este es su salairio final, +el 10%, [descontados todas las 
 
prestaciones y deudas] "+"salario "+salarioFinal);
    }
 
}
 

Código Java:

/*Aquí en esta clase tengo todos los métodos básicos que utilizo a lo largo del programa*/
package Aplication_Body;
import javax.swing.JOptionPane;
public class Metodos {
 
    
    public static String pedir(String x)
    {
        return JOptionPane.showInputDialog(x);
    }
    
    public static int convertirEntero(String x)
    {
        return Integer.parseInt(x);
    }
    public static void mostrar(String x)
    {
        System.out.println(x);
    }
    public static double ValorMasPorcentaje(double valor, double porcentaje){
        
        double valorFinal=((valor*porcentaje)/100);
        return(valor+valorFinal);
        
    }
    public static double valorMenosPorcentaje(double valor, int porcentaje){
        
        double valorFinal=((valor*porcentaje)/100);
        return(valor-valorFinal);
        
    }
}
 

Si depronto no hay nada que corregir, que no creo, igual coloquen que está bien porfa, para yo saber que lohize bien, gracias

[zero0097]

todo bien, pero estas exagerando en la modularidad de tu código, la clase Métodos no tiene razón de ser, por que los métodos que defines hacen exactamente lo mismo que si llamas directamente la instrucción que realizan....

almenos estos 3 métodos yo los quitaría:

Código:

public static String pedir(String x)
    {
        return JOptionPane.showInputDialog(x);
    }
    
    public static int convertirEntero(String x)
    {
        return Integer.parseInt(x);
    }
    public static void mostrar(String x)
    {
        System.out.println(x);
    }

Por qué? Por la razon de que no ejecutan más que una linea de código, la definicion de un método básicamente es un bloque de código con múltiples instrucciones a realizar, en tu caso son solo una sentencia, es decir sale más caro el caldo que las albondigas...

[Xerelo]

Por la razon de que no ejecutan más que una linea de código, la definicion de un método básicamente es un bloque de código con múltiples instrucciones a realizar, en tu caso son solo una sentencia, es decir sale más caro el caldo que las albondigas...

Es un poco exagerado lo que dices, que tenga una sentencia o 200 no es motivo.

Concretamente los tres métodos que dices sobran, es muy posible que necesite modificarlos, para evitar los null en la conversión por ejemplo. Yo no lo hago habitualmente por lo de siempre, las prisas.

[zero0097]

Iniciado por Xerelo

Es un poco exagerado lo que dices, que tenga una sentencia o 200 no es motivo.

Osea que qué?

[Fuzzylog]

Lo normal sería tener esos métodos en la clase Empresa, y llamarlos directamente. Salvo que haya un motivo para usarlos en múltiples clases no tendría por qué existir una clase específica.

Le falta hacer la gestión de excepciones. Por ejemplo si no introduce un número entero en el texto, tiene que hacer que lo vuelva a pedir de nuevo y ademas dar la opción de abandonar la aplicación.

[gobodeath]

Iniciado por Fuzzylog

Le falta hacer la gestión de excepciones. Por ejemplo si no introduce un número entero en el texto, tiene que hacer que lo vuelva a pedir de nuevo y ademas dar la opción de abandonar la aplicación.

Si tienes razón eso falta, ahora me pongo en ello.


Respecto a la clase Métodos, la cree con el objetivo de reutilizarla en otros trabajos, es decir cada vez que vaya a hacer un trabajo, meto esa clase dentro del paquete de mi proyecto y ya esta... no tengo que volver a hacerlos...


Tal vez así si valga la pena ?

[maxpower2008]

Lo q pasa es q los metodos de la clase metodos son de una sola linea de codigo, es decir no tiene sentido lo mismo que gastas es llamar al metodo lo gastas haciendo la linea de codigo.
De echo gastarias menos sin llamar a la clase metodo.

[Xerelo]

Iniciado por zero0097

Osea que qué?

Pues que no es tanto cosa del tamaño, también influye y mucho si es algo que vas a utilizar en varios sitios y que pueda ser necesario modificar.

Por ejemplo imagina que quiere cambiar los System.out por escritura a un archivo o mensaje en popup. Por haberlo previsto con mostrar(texto) no tendrá que buscar todos los System.out que ha puesto en el código.

O simplemente por comodidad para leer código, es más legible un isEmailValido(email) que email.matches("^[A-Z0-9._%+-]+@[A-Z0-9.-]+\.[A-Z]{2,6}$") o email.matches(PATTERN_EMAIL)

[zero0097]

No lo había pensado de esa manera...