Foros aprenderaprogramar.com
Didáctica y divulgación de la programación

Hola ESOJ

En primer lugar felicitarte porque se ve un código que demuestra conocimientos y que ha requerido dedicación. También se ve que estás trabajando bien con el diseño orientado a objetos.

Te comento algunos detalles

Nada más ver el código en un IDE salta una cosa a la vista: las clases no tienen nombres adecuados. Los nombres de las clases deben ser descriptivos del cometido de la clase o qué representa la clase. Además debe seguirse la convención de que los nombres de las clases comiencen por mayúsculas. Nombres como gg ó ii no dicen nada sobre el cometido de estas clases.

Lo mismo te ocurre con atributos y métodos. Todos los nombres deben ser autodescriptivos en la medida de lo posible. Nombres de método como este() no son autodescriptivos.

Nombres de atributos como pC,pO,pN tampoco son autodescriptivos

Las clases deben contener comentarios útiles. Por ejemplo

/**
 * Write a description of class prueba1 here.
 *
 * @author (your name)
 * @version (a version number or a date)
 */
no tiene ninguna utilidad.


Parece que estás usando espacios para centrar texto ¿? p1 = new JButton("Teoria  "); y  if(e.getActionCommand().equals("Teoria  ") )

parece que tienen unos espacios después de la palabra Teoria. Esto yo diría que es algo no recomendable, de hecho con facilidad hará que se incurra en inconsistencias (por ejemplo que cuando se haga el mantenimiento del programa quede en un sitio con un espacio más o menos que en otro sitio).

Hay otros sitios donde ocurre también esto.

Ya desde el punto de vista del usuario encuentro algunas cosas mejorables: las ventanas se abren una encima de otra y la de abajo queda oculta. Pareciera que se estuviera en la misma ventana (aunque en realidad se ha abierto una nueva). Quizás hiciera el programa más usable si cuando abres una ventana cierras la anterior e incluyes un botón o enlace "Volver" que permita cerrar la ventana actual y pasar a la anterior.


Saludos

Hola Bcks7 parece que sigue faltando código, aparece dos veces el código de la clase Casa y no aparece el código de la clase SalonCasa

Saludos

Hola Pedro, todo correcto

Saludos

Hola Bcks7 parece que falta código (el código de las clases CocinaCasa y Saloncasa) siempre pedimos poner el código completo para poder compilarlo (aunque el código sea el mismo que en algún ejercicio anterior, ya que así se puede compilar y comprobar más fácilmente)

Saludos

Hola Pedro el ejercicio cumple completo con lo que se pedía. Una posible mejora sería plantear el que para los días que son menores de 10 no se muestre el 0 antes del dígito, por ejemplo hacer que se mostrara 1 de enero de 2089, 2 de enero de 2089 en lugar de 01 de enero de 2089, 02 de enero de 2089

Saludos

Hola, si tienes bien la instalación habría que probar varias cosas

Vete al menú Run (ejecutar), Edit Configurations (Editar configuraciones), selecciona el proyecto y elige un dispositivo para la emulación. Prueba varios a ver qué pasa...

Saludos

Hola MegumiLawliet

Para pegar código en los foros revisa el hilo https://www.aprenderaprogramar.com/foros/index.php?topic=1460.0 y sigue lo que ahí se explica es la mejor forma para darle formato adecuado al código

El enlace que has puesto parece que no funciona, al menos a mí no me carga ninguna página

Si estás formándote en java te recomiendo el curso java desde cero: http://aprenderaprogramar.com/index.php?option=com_content&view=category&id=68&Itemid=188

He encontrado este código para generar el curp no sé si te puede servir

package curp;

import org.apache.commons.lang.StringUtils;

import javax.swing.*;
import java.util.Arrays;
import java.util.Random;

public class CurpGenerator
{
// ------------------------------ FIELDS ------------------------------

    public static final String[] MESES =
        {"Ene", "Feb", "Mar", "Abr", "May", "Jun", "Jul", "Ago", "Sep", "Oct", "Nov", "Dec"};

    public static final String[] MESES_VALOR = {"01", "02", "03", "04", "05", "06", "07", "08", "09", "10", "11", "12"};

    public static String[] ENTIDAD_FEDERATIVA = {"AGUASCALIENTES",
                                                 "BAJA CALIFORNIA NTE.",
                                                 "BAJA CALIFORNIA SUR",
                                                 "CAMPECHE",
                                                 "COAHUILA",
                                                 "COLIMA",
                                                 "CHIAPAS",
                                                 "CHIHUAHUA",
                                                 "DISTRITO FEDERAL",
                                                 "DURANGO",
                                                 "GUANAJUATO",
                                                 "GUERRERO",
                                                 "HIDALGO",
                                                 "JALISCO",
                                                 "MEXICO",
                                                 "MICHOACAN",
                                                 "MORELOS",
                                                 "NAYARIT",
                                                 "NUEVO LEON",
                                                 "OAXACA",
                                                 "PUEBLA",
                                                 "QUERETARO",
                                                 "QUINTANA ROO",
                                                 "SAN LUIS POTOSI",
                                                 "SINALOA",
                                                 "SONORA",
                                                 "TABASCO",
                                                 "TAMAULIPAS",
                                                 "TLAXCALA",
                                                 "VERACRUZ",
                                                 "YUCATAN",
                                                 "ZACATECAS",
                                                 "SERV. EXTERIOR MEXICANO",
                                                 "NACIDO EN EL EXTRANJERO"};

    public static String[] ENTIDAD_FEDERATIVA_VALOR = {"AS",
                                                       "BC",
                                                       "BS",
                                                       "CC",
                                                       "CL",
                                                       "CM",
                                                       "CS",
                                                       "CH",
                                                       "DF",
                                                       "DG",
                                                       "GT",
                                                       "GR",
                                                       "HG",
                                                       "JC",
                                                       "MC",
                                                       "MN",
                                                       "MS",
                                                       "NT",
                                                       "NL",
                                                       "OC",
                                                       "PL",
                                                       "QT",
                                                       "QR",
                                                       "SP",
                                                       "SL",
                                                       "SR",
                                                       "TC",
                                                       "TS",
                                                       "TL",
                                                       "VZ",
                                                       "YN",
                                                       "ZS",
                                                       "SM",
                                                       "NE"};


    private static char[] VOCALS = {'A', 'E', 'I', 'O', 'U'};

    private static Random random = new Random();

// -------------------------- STATIC METHODS --------------------------

    public static String generar( String primerApellido, String segundoApellido, String nombres, String sexo,
                                  String fechaDeNacimiento, String entidadDeNacimiento )
        throws Exception
    {
        // limpiar los datos
        primerApellido = trim( primerApellido );
        segundoApellido = trim( segundoApellido );
        nombres = trim( nombres );
        sexo = trim( sexo );
        entidadDeNacimiento = trim( entidadDeNacimiento );

        // validar que los datos estan correctors
        String error =
            validarDatos( primerApellido, segundoApellido, nombres, sexo, fechaDeNacimiento, entidadDeNacimiento );
        if ( error != null )
        {
            throw new Exception( error );
        }

        // generar Curp
        StringBuilder curp = new StringBuilder();
        curp.append( primerApellido.charAt( 0 ) );
        curp.append( primeraVocal( primerApellido.substring( 1 ) ) );
        curp.append( segundoApellido.charAt( 0 ) );
        curp.append( nombres.charAt( 0 ) );
        curp.append( fechaDeNacimiento.substring( 8, 10 ) );
        curp.append( fechaDeNacimiento.substring( 0, 2 ) );
        curp.append( fechaDeNacimiento.substring( 3, 5 ) );
        curp.append( sexo );
        curp.append( entidadDeNacimiento );
        curp.append( primeraLetra( primerApellido.substring( 1 ) ) );
        curp.append( primeraLetra( segundoApellido.substring( 1 ) ) );
        curp.append( primeraLetra( nombres.substring( 1 ) ) );
        curp.append( 0 );
        curp.append( random.nextInt( 10 ) );
        return curp.toString();
    }

    public static String trim( String s )
    {
        return StringUtils.trimToEmpty( s ).toUpperCase();
    }

    private static char primeraLetra( String s )
    {
        int i = StringUtils.indexOfAnyBut( s, VOCALS );
        if ( i >= 0 )
        {
            return s.charAt( i );
        }
        return 'X';
    }

    private static char primeraVocal( String s )
    {
        int i = StringUtils.indexOfAny( s, VOCALS );
        if ( i >= 0 )
        {
            return s.charAt( i );
        }
        return 'X';
    }

    private static String validarDatos( String primerApellido, String segundoApellido, String nombres, String sexo,
                                        String fechaDeNacimiento, String entidadDeNacimiento )
    {
        if ( "".equals( primerApellido ) )
        {
            return "Primer apellido es obligatorio";
        }
        if ( !StringUtils.isAlpha( primerApellido ) )
        {
            return "Primer apellido no valido, caracteres validos: A-Z (incluso Ñ)";
        }

        if ( "".equals( segundoApellido ) )
        {
            return "Segundo apellido es obligatorio";
        }
        if ( !StringUtils.isAlpha( segundoApellido ) )
        {
            return "Segundo apellido no valido, caracteres validos: A-Z (incluso Ñ)";
        }

        if ( "".equals( nombres ) )
        {
            return "Nombre(s) es obligatorio";
        }
        if ( !StringUtils.isAlphaSpace( nombres ) )
        {
            return "Nombre(s) no valido, caracteres validos: A-Z (incluso Ñ)";
        }

        if ( !"H".equals( sexo ) && !"M".equals( sexo ) )
        {
            return "Sexo no valido";
        }

        if ( "".equals( fechaDeNacimiento ) )
        {
            return "Fecha de nacimiento no valido";
        }

        if ( "".equals( entidadDeNacimiento ) )
        {
            return "Entidad de nacimiento es obligatorio";
        }
        if ( !Arrays.asList( ENTIDAD_FEDERATIVA_VALOR ).contains( entidadDeNacimiento ) )
        {
            return "Entidad de nacimiento necesita ser la abreciacion";
        }

        return null;
    }
}

Saludos

Hola todo bien, supongo que existen borradores o versiones de prueba pero que en cierta medida esa información no está a primera vista para no confundir a los usuarios

Saludos

Hola Dar1us

a) bien, b) bien, c) bien, d) bien

PHP es un lenguaje que se usa para generar documentos HTML con un procesado previo que permite utilizar toda la potencia de la programación. HTML no es un lenguaje de programación tal y como puede ser PHP u otros.

Saludos

Hola Hector Hugo si no entiendes alguna explicación en el curso abre un hilo con una consulta indicando la entrega del curso y qué parte concretamente no entiendes, intentaremos aclarártela entre todos

Saludos