Wall of Shame

No quiero trabjar ni aprender, voy a reprobar!

2009-07-30T17:44:00.000-07:00

El tema de hoy día es un tema muy importante y que hemos tocado en los comentarios de postes anteriores, alumnos que quieren aprobar sin trabajar y sin aprender. Pero, lamentable, no son tan bueno a la resolución de este problema tampoco.

Empezamos con un alumno desde este año en Lenguajes de Programación que ha entregado este código como su proyecto. ¿Cual es el problema de este código?

Code here.

¿Nada, si? No hay comentarios, pero es bien programado. Incluye el uso de iterator. Que no he enseñado. Y que no creo que un alumno de segundo año va a utilizar. Este alumno recibió 1 para este proyecto. ¿Por qué? ¿Cómo podría saber? Primero, el alumno no supo cual es un iterator -- es demasiado avanzado por un alumno que no sabe C++. Además, yo se quien lo ha programado (no el alumno) y cuanto ha pagado ($7.000).

¿Muy mal, si? Yo se ningunos de ustedes harían esto..

La segunda programa viene desde Lenguajes de Programación el año pasado. Este programa no es tan mal tampoco, pero cuando lo revisé, encontré alguna cosa... familiar..

Code here.

No tan mal... pero este es la pauta que di a mis ayudantes:

Code here.

¡Si! Es un parte de la pauta (otra parte apreció en el proyecto de otro alumno) con los comentarios ingleses borrados! Muy bueno chicos, muy bueno. Profe no recordará su propio código.

¿Entonces, si un alumno está preparado para pagar por un proyecto, cómo pueden los profesores combatir este? Estos ejemplos son los que he detectado. ¿Que sobre los otros? Van a llegar a un punto donde encontrarán que necesitan estas capacidades, y no van a tenerlos... Y quizás van a aparecer aquí, como nuestra amigo de quinto año que no pudo programar un juego simple de gato.

Ciclos y arreglos

2009-07-22T17:37:00.000-07:00

Este trocito de código viene desde mis chicos de Lenguajes de Programación este año, y no es solo un alumno, pero muchos en una prueba.

La tarea fue para escribir un método que cambia un elemento de un arreglo en C++.


void CD:modificarCancion(int elemento, string nueva) 
{
        for (int i=0; i<numCanciones; i++) 
        {
              if (i==elemento)
                    cancion[i]=nueva;
        }
}

¿Por qué este código es tan malo?

Una respuesta (Parte B)

2009-07-18T10:16:00.000-07:00

Ayer, vimos como podemos transformar 5 grandes secciones de código entre solo una sección. Deberían asegurar que entienden como el nuevo código funciona. No es tan difícil, sola una parte es complicado.. lo de verificar que los barcos no cruzan.

Hoy día vamos a ver como podemos utilizar los poderes de las clases (en Java, o cualquier otra lenguaje) a limpiar el código más.

Vamos a empezar con la sección del código mencionada arriba - la verificación del los barcos.


//verifcamos que el barco no cruza los barcos anteriores
for (int i=0; i<lista_barcos[barco_acutal].tamano; i++) 
     for (int j=barca_actual; j==0; j--) 
     {
 //este es poquito sucio .. necesitamos verificar que ninguna de los coordenados en el barco actual son iguales a todos los coordenados de los barcos anteriores.

 // asumimos que no cruzamos, si revisamos todos y todavia cruza es falso, bien.
 boolean cruza=false;
 for (int k=0; j<lista_barcos[j].posiciones.length; k++) 
      if (lista_barcos[j].posiciones[k].equals(lista_barcos[barco_acutal].posiciones[i]))
       cruza=true;
     }

¿Que hace este código? En pseudocódigo:


Para cada barco B antes el barco actual
    Revisamos todos los posiciones en B y verifica que ninguna es la misma que nuestra barca actual.

Ayer, definimos una clase para representar un barco. Dijimos que esta clase es como un struct en C. Es verdad, pero no es todo. Clases son mucho mas poderosos que esta.. podemos definir métodos que operan a los datos al dentro la clase. Es como una paquete -- datos y métodos.

Entonces vamos a redefinir la clase a ser:


public class Barco
{
    //los datos
    public String nombre; // guardar el nombre del barco
    public Point[] posiciones; // una lista de puntos donde el barco es ubicado
    public int tamano; // El tamano del barco, i.e., el numero de espacios que tiene

    //los metodos
    public boolean esCruzado(Barco b1) //comparar el barco en este clase con otro barco b1.
    {
        boolean cruza=false;
        for (int i=0; i<this.posiciones.length; ++i)
    for (int k=0; j<b1.posiciones.length; ++k) 
        if(b1.posiciones[k].equals(this.posiciones[i]))
   cruza=true;
        return cruza;
    }
}

Ahora cada barco que creamos tiene un método esCruzado que podemos utilizar para ver si otro barco se cruza.

Con este método definido, podemos mejorar el código poquito más. En vez del código complejo arriba, podemos escribir:


boolean cruza=false;
//verifcamos que el barco no cruza los barcos anteriores
 for (int j=barca_actual; j==0; j--) 
     {
      if (lista_barcos[barco_acutal].esCruzado(lista_barcos[j]))
  cruza=true;
     }

¿Mucho más entendible, si? El arte de programación es este proceso que acaba de hacer. Tomamos algoritmos complejos y intentamos a reducirlos a una forma simple y entendible.

Ahora.. preguntas para ustedes:

1) ¿Es un ciclo for la mejor manera para hacer esta? ¿Exista una forma más eficiente?

2) ¿Pueden identificar otra métodos que podemos agregar al código nuevo para limpiarlo más?

3) ¿Pueden identificar el error de lógica en el código que acabamos de escribir? ¿Exista en el código anterior? ¿Es más fácil para identificar ahora?

Una respuesta (Parte A)

2009-07-17T13:26:00.000-07:00

La semana pasada, vimos un método mal escrito.. podemos aclarar que está haciendo este método?

Primero podemos identificar la estructura mas alta:


while (bandera==1) 
{
} //fin del ciclo, termino de la creacion del barco portavion
while (bandera==2) 
{
} //fin del ciclo while, termino de la creacion del barco acorazado
while (bandera==3) 
{
} //fin del ciclo while para crear el barco crucero
while (bandera==4) 
{
} //fin del ciclo, que crea el barco submarino
while (bandera==5) 
{
} //fin del ciclo, creacion del barco destructor
while (bandera==6)
{
   // colocar el barcos en un tablero
}

¡Aha! ¡Entonces, estamos creando 5 barcos y colocándolos en un tablero !

Bueno, dejamos esta estructura por el lado por un momento. ¿Que pasa al dentro cada uno de estos ciclos?


1: Ingresar las coordinados
2: Pregunta si queremos poner el barco vertical o horizontal
3: Pregunta si queremos poner el barco a lado izquierdo o derecha 
4: Verificamos que podemos fiscalmente colocar el barco en el tablero a este punto
5: Verificamos que podemos colocar el barco sin tocando otro barco

¿Que sobre los barcos, donde guardamos la información sobre ellos? En los parámetros de método, entonces:

int columna_des,int columna_sub,int columna_cru,int columna_aco,int columna_por,int fila_des,int fila_sub,int fila_cru,int fila_aco,int fila_por,int aux_des,int aux_sub,int aux_cru,int aux_aco,int aux_por, int aux1_des,int aux1_sub,int aux1_cru,int aux1_aco,int aux1_por,int aux2_sub,int aux2_cru,int aux2_aco,int aux2_por,int aux3_sub,int aux3_cru,int aux3_aco,int aux3_por,int aux4_aco,int aux4_por,int aux5_aco, int aux5_por,int aux6_por,int aux7_por

Ok. Bien. ¿Como podemos limpiar está método?

Primero, sería ideal si podemos identificar las cosas comunes de los barcos y abstraerlos, con este podemos evitar mucha de la complejidad de el método viejo. ¿Cuales son los datos esenciales para un barco?

1: Su Nombre - Crucero, Submarino, Portaaviones, Destructor, Acorazado.
2: Su Posición o espacios ocupados - una lista de filas y columnas, o, podemos decir puntos.
3: Su Tamaño - el tamaño de la lista de puntos.

Bueno, si estamos programando en C, podemos usar un struct para guardar esta información. Pero este es Java y no tiene structs. Pero tiene classes..


public class Barco
{
    public String nombre; // guardar el nombre del barco
    public Point[] posiciones; // una lista de puntos donde el barco es ubicado
    public int tamano; // El tamano del barco, i.e., el numero de espacios que tiene
}

Este es equivalente a un struct en C, y con este podemos guardar toda la información de un barco.

Ahora, reescribimos el método como así:

¿Poquito mejor, si? Pero todavía podemos limpiar mas. Vamos a ver como en Parte B de este post.

El ganador para lo más parametros es:

2009-07-10T16:51:00.000-07:00

El código para esta semana es desde un programa que debería jugar un juego de batalla naval. Sorprendentemente este código realmente funciona. ¿Podemos identificar su faltas numerosas? ¿O mejor, podemos descifrar como funciona? ¿Si encuentras este código en un programa que necesitas mantener, que harías?

The code from this week is from a program that should play a game of Battleship. Surprisingly, this code really works. Can we identify its numerous problems? Or better, can we decipher how it works? If you found this code in a program that you needed to maintain, what would you do?

public void crear_barco(int columna_des,int columna_sub,int columna_cru,int columna_aco,int columna_por,int fila_des,int fila_sub,int fila_cru,int fila_aco,int fila_por,int aux_des,int aux_sub,int aux_cru,int aux_aco,int aux_por,
       int aux1_des,int aux1_sub,int aux1_cru,int aux1_aco,int aux1_por,int aux2_sub,int aux2_cru,int aux2_aco,int aux2_por,int aux3_sub,int aux3_cru,int aux3_aco,int aux3_por,int aux4_aco,int aux4_por,int aux5_aco,
       int aux5_por,int aux6_por,int aux7_por)throws IOException{//resive todas las coordenadas de los 5 barcos tanto la posicion de las filas como las columnas ï��ï��de la maquina!!
      String orientacion,posicion,posicion2;//define tres tipos de string
      BufferedReader in= new BufferedReader(new InputStreamReader(System.in));
      int opc=0,contador=1,contador2=1,contador3=1,auxiliar,bandera=1,x=0;//variables a utilizar tanto como contador, auxiliar y condicion
      System.out.println("ingresar la posicion de los barcos en el tableros");//imprime por pantalla
       
      while(bandera==1){//ï��ï��portaviones!! ciclo que crea el barco         
        System.out.println("ï��ï��barco portaaviones!!");//imprime por pantalla        
        coordenadas opr= new coordenadas();// crea variable de tipo coordenadas
        opr.posicion_coor();//llama al metodo posicion_coor
        fila=opr.rfila();//llama al metodo rfila que recive una coordenada
        columna=opr.rcolumna();//llama al metodo rcolumna que recive una coordenada
        while(contador==1){//inicio del ciclo while
        System.out.println("ingrese su orientacion con minuscula ï��ï��horizontal!! o ï��ï��vertical!!");//imprime por pantalla
        orientacion=in.readLine();//recive de pantalla
        if(orientacion.equals("vertical") || orientacion.equals("horizontal")){//condicion que compara string
        if(orientacion.equals("horizontal")){//condicion parte de la anterior
            while(contador2==1){//ciclo que da paso a un nuevo proceso
                System.out.println("apartir de la coordenada dada, quiere que el barco este hacia ï��ï��izquierda!! o  ï��ï��derecha!!");//imprime por pantalla
                posicion=in.readLine();//recive desde pantalla
                if(posicion.equals("izquierda") || posicion.equals("derecha")){//imprime por pantalla
                    if(posicion.equals("izquierda")){//condicion parte de la anterior
                        if(columna==0 || columna==1 || columna==2 || columna==3){//condicion que se utiliza para que se pueda ingresar un barco
                          bandera=1;
                          System.out.println("no se puede ingresar en este lugar el barco");//imprime por pantalla
                        }//fin del if
                        else{//inverso del if
                            bandera=2;
                            fila_porh=fila;//procesos que pasan variables a coordenadas del humano
                            columna_porh=columna;//con el proposito de crear el barco portavion
                            aux1_porh=fila;
                            aux_porh=columna-1;
                            aux2_porh=fila;
                            aux3_porh=columna-2;
                            aux4_porh=fila;
                            aux5_porh=columna-3;
                            aux6_porh=fila;
                            aux7_porh=columna-4;
                            contador=0;
                        }//fin del else
                    }//fin del if
                    if(posicion.equals("derecha")){//inicio del if
                        if(columna==6 || columna==7 || columna==8 || columna==9){//condicion que se utiliza para ingresar el barco 
                            bandera=1;
                            System.out.println("no se puede ingresar en este lugar el barco");//imprime por pantalla
                        }//fin condicion if
                        else{//inicio condicion else
                            bandera=2;
                            fila_porh=fila;
                            columna_porh=columna;
                            aux1_porh=fila;
                            aux_porh=columna+1;
                            aux2_porh=fila;
                            aux3_porh=columna+2;
                            aux4_porh=fila;
                            aux5_porh=columna+3;
                            aux6_porh=fila;
                            aux7_porh=columna+4;
                            contador=0;
                        }//fin condicion else
                    
                    }//fin condicion if
                    contador2=0;
                  }//fin del if
                else{//condicion contraria al if
                    System.out.println("no corresponden las palabras");//imprime por pantalla
                    contador2=1;
                  }//fin else
                }//fin del while contador2==1
            contador=0;
            }//fin del if
         if(orientacion.equals("vertical")){//inicio del if
              while(contador3==1){//inicio del while que posee deiversas condiciones para crear un barco de forma vertical
               System.out.println("apartir de la coordenada dada, quiere que el barco este hacia ï��ï��arriba!! o  ï��ï��abajo!!");//imprime por pantalla
               posicion2=in.readLine();
               if(posicion2.equals("arriba") || posicion2.equals("abajo")){//inicio del if
                   if(posicion2.equals("arriba")){//inicio del if
                      if(fila==0 || fila==1 || fila==2 || fila==3){//inicio del if
                          bandera=1;
                          System.out.println("no se puede ingresar en este lugar el barco");//imprime por pantalla
                        }//fin del if
                        else{//inicio del else
                            bandera=2;
                            fila_porh=fila;
                            columna_porh=columna;
                            aux1_porh=fila-1;
                            aux_porh=columna;
                            aux2_porh=fila-2;
                            aux3_porh=columna;
                            aux4_porh=fila-3;
                            aux5_porh=columna;
                            aux6_porh=fila-4;
                            aux7_porh=columna;
                            contador=0;
                        } //fin del else
                       contador3=0;
                    }//fin del if
                    if(posicion2.equals("abajo")){//inicio del if
                      if(fila==6 || fila==7 || fila==8 || fila==9){//inicio del if
                          bandera=1;
                          System.out.println("no se puede ingresar en este lugar el barco");//imprime por pantalla
                        }//fin del if
                        else{//inicio del else
                            bandera=2;
                            fila_porh=fila;
                            columna_porh=columna;
                            aux1_porh=fila+1;
                            aux_porh=columna;
                            aux2_porh=fila+2;
                            aux3_porh=columna;
                            aux4_porh=fila+3;
                            aux5_porh=columna;
                            aux6_porh=fila+4;
                            aux7_porh=columna;
                            contador=0;
                        }//fin del else
                    contador3=0;
                  }//fin del if
                }//fin del if
                  else{//inicio del else
                    System.out.println("no corresponden las palabras");
                    contador3=1;
                  }//fin del else
                
             contador=0;
            }//fin del while
        }//fin del if
    }//fin del if
         else{//inicio del else
            System.out.println("las palabras ingresadas no corresponden");
            contador=1;
            }//fin del else
          
          
        }//fin del iclo while
    }//fin del ciclo, termino de la creacion del barco portavion
    while (bandera==2){//crea barco acorazado
        contador=1;
        contador2=1;
        contador3=1;                                                   //este ciclo while bandera==2 cumple practicamente la mismas funciones de la creacion del barco anterior
        System.out.println("ï��ï��barco acorazado!!");
        coordenadas opr= new coordenadas(); 
        opr.posicion_coor();
        fila=opr.rfila();
        columna=opr.rcolumna();
        while(contador==1){
        System.out.println("ingrese su orientacion con minuscula");
        orientacion=in.readLine();
        if(orientacion.equals("vertical") || orientacion.equals("horizontal")){
        if(orientacion.equals("horizontal")){
            while(contador2==1){
                System.out.println("apartir de la coordenada dada, quiere que el barco este hacia ï��ï��izquierda!! o  ï��ï��derecha!!");
                posicion=in.readLine();
                if(posicion.equals("izquierda") || posicion.equals("derecha")){
                    if(posicion.equals("izquierda")){
                        if(columna==0 || columna==1 || columna==2){
                          bandera=2;
                          System.out.println("no se puede ingresar en este lugar el barco");
                        }
                        else{
                            fila_acoh=fila;
                            columna_acoh=columna;
                            aux1_acoh=fila;
                            aux_acoh=columna-1;
                            aux2_acoh=fila;
                            aux3_acoh=columna-2;
                            aux4_acoh=fila;
                            aux5_acoh=columna-3;
                            contador=0;
                        }
                    }
                    if(posicion.equals("derecha")){
                        if(columna==7 || columna==8 || columna==9){
                            bandera=2;
                            System.out.println("no se puede ingresar en este lugar el barco");
                        }
                        else{
                            fila_acoh=fila;
                            columna_acoh=columna;
                            aux1_acoh=fila;
                            aux_acoh=columna+1;
                            aux2_acoh=fila;
                            aux3_acoh=columna+2;
                            aux4_acoh=fila;
                            aux5_acoh=columna+3;
                            contador=0;
                        }
                    
                    }
                    contador2=0;
                  }
                else{
                    System.out.println("no corresponden las palabras");
                    contador2=1;
                  }
                }
            contador=0;
            }
         if(orientacion.equals("vertical")){
              while(contador3==1){
               System.out.println("apartir de la coordenada dada, quiere que el barco este hacia ï��ï��arriba!! o  ï��ï��abajo!!");
               posicion2=in.readLine();
               if(posicion2.equals("arriba") || posicion2.equals("abajo")){
                   if(posicion2.equals("arriba")){
                      if(fila==0 || fila==1 || fila==2){
                          bandera=2;
                          System.out.println("no se puede ingresar en este lugar el barco");
                        }
                        else{
                            fila_acoh=fila;
                            columna_acoh=columna;
                            aux1_acoh=fila-1;
                            aux_acoh=columna;
                            aux2_acoh=fila-2;
                            aux3_acoh=columna;
                            aux4_acoh=fila-3;
                            aux5_acoh=columna;
                            contador=0;
                        } 
                       contador3=0;
                    }
                    if(posicion2.equals("abajo")){
                      if(fila==7 || fila==8 || fila==9){
                          bandera=2;
                          System.out.println("no se puede ingresar en este lugar el barco");
                        }
                        else{
                            fila_acoh=fila;
                            columna_acoh=columna;
                            aux1_acoh=fila+1;
                            aux_acoh=columna;
                            aux2_acoh=fila+2;
                            aux3_acoh=columna;
                            aux4_acoh=fila+3;
                            aux5_acoh=columna;
                            contador=0;
                        }
                    contador3=0;
        
                  }
                }
                  else{
                    System.out.println("no corresponden las palabras");
                    contador3=1;
                  }
                
             contador=0;
            }
        }
    }
         else{
            System.out.println("las palabras ingresadas no corresponden");
            contador=1;
            }
              
        }
       if((fila_porh==fila_acoh && columna_porh==columna_acoh)||(fila_porh==aux1_acoh && columna_porh==aux_acoh)||(fila_porh==aux2_acoh && columna_porh==aux3_acoh)||(fila_porh==aux4_acoh && columna_porh==aux5_acoh)||
       (aux1_porh==fila_acoh && aux_porh==columna_acoh)||(aux1_porh==aux1_acoh && aux_porh==aux_acoh)||(aux1_porh==aux2_acoh && aux_porh==aux3_acoh)||(aux1_porh==aux4_acoh && aux_porh==aux5_acoh)||
       (aux2_porh==fila_acoh && aux3_porh==columna_acoh)||(aux2_porh==aux1_acoh && aux3_porh==aux_acoh)||(aux2_porh==aux2_acoh && aux3_porh==aux3_acoh)||(aux2_porh==aux4_acoh && aux3_porh==aux5_acoh)||
       (aux4_porh==fila_acoh && aux5_porh==columna_acoh)||(aux4_porh==aux1_acoh && aux5_porh==aux_acoh)||(aux4_porh==aux2_acoh && aux5_porh==aux3_acoh)||(aux4_porh==aux4_acoh && aux5_porh==aux5_acoh)||
       (aux6_porh==fila_acoh && aux7_porh==columna_acoh)||(aux6_porh==aux1_acoh && aux7_porh==aux_acoh)||
       (aux6_porh==aux2_acoh && aux7_porh==aux3_acoh)||(aux6_porh==aux4_acoh && aux7_porh==aux5_acoh)){//condicion con el fin de comparar las coordenadas para que no se repitan
           System.out.println("el barco no se puede ingresar, porque choca con otro");
          bandera=2;
        }
       else
        bandera=3;
    }//fin del ciclo while, termino de la creacion del barco acorazado
     
    while(bandera==3){//crea barco crucero
      System.out.println("ï��ï��barco crucero!!");
        contador=1;                                       //este ciclo while bandera==3 cumple practicamente la mismas funciones de la creacion del barco anterior
        contador2=1;
        contador3=1;
        coordenadas opr= new coordenadas(); 
        opr.posicion_coor();
        fila=opr.rfila();
        columna=opr.rcolumna();
        while(contador==1){
        System.out.println("ingrese su orientacion con minuscula");
        orientacion=in.readLine();
        if(orientacion.equals("vertical") || orientacion.equals("horizontal")){
        if(orientacion.equals("horizontal")){
            while(contador2==1){
                System.out.println("apartir de la coordenada dada, quiere que el barco este hacia ï��ï��izquierda!! o  ï��ï��derecha!!");
                posicion=in.readLine();
                if(posicion.equals("izquierda") || posicion.equals("derecha")){
                    if(posicion.equals("izquierda")){
                        if(columna==0 || columna==1){
                          bandera=3;
                          System.out.println("no se puede ingresar en este lugar el barco");
                        }
                        else{
                            fila_cruh=fila;
                            columna_cruh=columna;
                            aux1_cruh=fila;
                            aux_cruh=columna-1;
                            aux2_cruh=fila;
                            aux3_cruh=columna-2;
                            contador=0;
                        }
                    }
                    if(posicion.equals("derecha")){
                        if(columna==8 || columna==9){
                            bandera=3;
                            System.out.println("no se puede ingresar en este lugar el barco");
                        }
                        else{
                            fila_cruh=fila;
                            columna_cruh=columna;
                            aux1_cruh=fila;
                            aux_cruh=columna+1;
                            aux2_cruh=fila;
                            aux3_cruh=columna+2;
                            contador=0;
                        }
                    
                    }
                    contador2=0;
                  }
                else{
                    System.out.println("no corresponden las palabras");
                    contador2=1;
                  }
                }
            contador=0;
            }
         if(orientacion.equals("vertical")){
              while(contador3==1){
               System.out.println("apartir de la coordenada dada, quiere que el barco este hacia ï��ï��arriba!! o  ï��ï��abajo!!");
               posicion2=in.readLine();
               if(posicion2.equals("arriba") || posicion2.equals("abajo")){
                   if(posicion2.equals("arriba")){
                      if(fila==0 || fila==1){
                          bandera=3;
                          System.out.println("no se puede ingresar en este lugar el barco");
                        }
                        else{
                            fila_cruh=fila;
                            columna_cruh=columna;
                            aux1_cruh=fila-1;
                            aux_cruh=columna;
                            aux2_cruh=fila-2;
                            aux3_cruh=columna;
                            contador=0;
                        } 
                       contador3=0;
                    }
                    if(posicion2.equals("abajo")){
                      if(fila==8 || fila==9){
                          bandera=3;
                          System.out.println("no se puede ingresar en este lugar el barco");
                        }
                        else{
                            fila_cruh=fila;
                            columna_cruh=columna;
                            aux1_cruh=fila+1;
                            aux_cruh=columna;
                            aux2_cruh=fila+2;
                            aux3_cruh=columna;
                            contador=0;
                        }
                    contador3=0;
        
                  }
                }
                  else{
                    System.out.println("no corresponden las palabras");
                    contador3=1;
                  }
                
             contador=0;
            }
        }
    }
         else{
            System.out.println("las palabras ingresadas no corresponden");
            contador=1;
            }
              
        }
        //comparar con portavion
       if((fila_cruh==fila_porh && columna_cruh==columna_porh)||(fila_cruh==aux1_porh && columna_cruh==aux_porh)||(fila_cruh==aux2_porh && columna_cruh==aux3_porh)||(fila_cruh==aux4_porh && columna_cruh==aux5_porh)||(fila_cruh==aux6_porh && columna_cruh==aux7_porh)||
       (aux1_cruh==fila_porh && aux_cruh==columna_porh)||(aux1_cruh==aux1_porh && aux_cruh==aux_porh)||(aux1_cruh==aux2_porh && aux_cruh==aux3_porh)||(aux1_cruh==aux4_porh && aux_cruh==aux5_porh)||(aux1_cruh==aux6_porh && aux_cruh==aux7_porh)||   
       (aux2_cruh==fila_porh && aux3_cruh==columna_porh)||(aux2_cruh==aux1_porh && aux3_cruh==aux_porh)||(aux2_cruh==aux2_porh && aux3_cruh==aux3_porh)||(aux2_cruh==aux4_porh && aux3_cruh==aux5_porh)||(aux2_cruh==aux6_porh && aux3_cruh==aux7_porh)||
       //comparar con acorazado
       (fila_cruh==fila_acoh && columna_cruh==columna_acoh)||(fila_cruh==aux1_acoh && columna_cruh==aux_acoh)||(fila_cruh==aux2_acoh && columna_cruh==aux3_acoh)||(fila_cruh==aux4_acoh && columna_cruh==aux5_acoh)||
       (aux1_cruh==fila_acoh && aux_cruh==columna_acoh)||(aux1_cruh==aux1_acoh && aux_cruh==aux_acoh)||(aux1_cruh==aux2_acoh && aux_cruh==aux3_acoh)||(aux1_cruh==aux4_acoh && aux_cruh==aux5_acoh)||
       (aux2_cruh==fila_acoh && aux3_cruh==columna_acoh)||(aux2_cruh==aux1_acoh && aux3_cruh==aux_acoh)||(aux2_cruh==aux2_acoh && aux3_cruh==aux3_acoh)||(aux2_cruh==aux4_acoh && aux3_cruh==aux5_acoh)){
          System.out.println("el barco no se puede ingresar, porque choca con otro");
          bandera=3;
        }
       else
        bandera=4;    
        }//fin del ciclo while para crear el barco crucero
   
   while(bandera==4){//crea barco submarino
       System.out.println("ï��ï��barco submarino!!"); 
        contador=1;
        contador2=1;                               //este ciclo while bandera==4 cumple practicamente la mismas funciones de la creacion del barco anterior
        contador3=1;
        coordenadas opr= new coordenadas(); 
        opr.posicion_coor();
        fila=opr.rfila();
        columna=opr.rcolumna();
        while(contador==1){
        System.out.println("ingrese su orientacion con minuscula");
        orientacion=in.readLine();
        if(orientacion.equals("vertical") || orientacion.equals("horizontal")){
        if(orientacion.equals("horizontal")){
            while(contador2==1){
                System.out.println("apartir de la coordenada dada, quiere que el barco este hacia ï��ï��izquierda!! o  ï��ï��derecha!!");
                posicion=in.readLine();
                if(posicion.equals("izquierda") || posicion.equals("derecha")){
                    if(posicion.equals("izquierda")){
                        if(columna==0 || columna==1){
                          bandera=4;
                          System.out.println("no se puede ingresar en este lugar el barco");
                        }
                        else{
                            fila_subh=fila;
                            columna_subh=columna;
                            aux1_subh=fila;
                            aux_subh=columna-1;
                            aux2_subh=fila;
                            aux3_subh=columna-2;
                            contador=0;
                        }
                    }
                    if(posicion.equals("derecha")){
                        if(columna==8 || columna==9){
                            bandera=4;
                            System.out.println("no se puede ingresar en este lugar el barco");
                        }
                        else{
                            fila_subh=fila;
                            columna_subh=columna;
                            aux1_subh=fila;
                            aux_subh=columna+1;
                            aux2_subh=fila;
                            aux3_subh=columna+2;
                            contador=0;
                        }
                    
                    }
                    contador2=0;
                  }
                else{
                    System.out.println("no corresponden las palabras");
                    contador2=1;
                  }
                }
            contador=0;
            }
         if(orientacion.equals("vertical")){
              while(contador3==1){
               System.out.println("apartir de la coordenada dada, quiere que el barco este hacia ï��ï��arriba!! o  ï��ï��abajo!!");
               posicion2=in.readLine();
               if(posicion2.equals("arriba") || posicion2.equals("abajo")){
                   if(posicion2.equals("arriba")){
                      if(fila==0 || fila==1){
                          bandera=4;
                          System.out.println("no se puede ingresar en este lugar el barco");
                        }
                        else{
                            fila_subh=fila;
                            columna_subh=columna;
                            aux1_subh=fila-1;
                            aux_subh=columna;
                            aux2_subh=fila-2;
                            aux3_subh=columna;
                            contador=0;
                        } 
                       contador3=0;
                    }
                    if(posicion2.equals("abajo")){
                      if(fila==8 || fila==9){
                          bandera=4;
                          System.out.println("no se puede ingresar en este lugar el barco");
                        }
                        else{
                            fila_subh=fila;
                            columna_subh=columna;
                            aux1_subh=fila+1;
                            aux_subh=columna;
                            aux2_subh=fila+2;
                            aux3_subh=columna;
                            contador=0;
                        }
                    contador3=0;
        
                  }
                }
                  else{
                    System.out.println("no corresponden las palabras");
                    contador3=1;
                  }
                
             contador=0;
            }
        }
    }
         else{
            System.out.println("las palabras ingresadas no corresponden");
            contador=1;
            }
              
        }
        //comparar con portavion
       if((fila_subh==fila_porh && columna_subh==columna_porh)||(fila_subh==aux1_porh && columna_subh==aux_porh)||(fila_subh==aux2_porh && columna_subh==aux3_porh)||(fila_subh==aux4_porh && columna_subh==aux5_porh)||(fila_subh==aux6_porh && columna_subh==aux7_porh)||
       (aux1_subh==fila_porh && aux_subh==columna_porh)||(aux1_subh==aux1_porh && aux_subh==aux_porh)||(aux1_subh==aux2_porh && aux_subh==aux3_porh)||(aux1_subh==aux4_porh && aux_subh==aux5_porh)||(aux1_subh==aux6_porh && aux_subh==aux7_porh)||   
       (aux2_subh==fila_porh && aux3_subh==columna_porh)||(aux2_subh==aux1_porh && aux3_subh==aux_porh)||(aux2_subh==aux2_porh && aux3_subh==aux3_porh)||(aux2_subh==aux4_porh && aux3_subh==aux5_porh)||(aux2_subh==aux6_porh && aux3_subh==aux7_porh)||
       //comparar con acorazado
       (fila_subh==fila_acoh && columna_subh==columna_acoh)||(fila_subh==aux1_acoh && columna_subh==aux_acoh)||(fila_subh==aux2_acoh && columna_subh==aux3_acoh)||(fila_subh==aux4_acoh && columna_subh==aux5_acoh)||
       (aux1_subh==fila_acoh && aux_subh==columna_acoh)||(aux1_subh==aux1_acoh && aux_subh==aux_acoh)||(aux1_subh==aux2_acoh && aux_subh==aux3_acoh)||(aux1_subh==aux4_acoh && aux_subh==aux5_acoh)||
       (aux2_subh==fila_acoh && aux3_subh==columna_acoh)||(aux2_subh==aux1_acoh && aux3_subh==aux_acoh)||(aux2_subh==aux2_acoh && aux3_subh==aux3_acoh)||(aux2_subh==aux4_acoh && aux3_subh==aux5_acoh)||
       //comparar con crucero
       (fila_subh==fila_cruh && columna_subh==columna_cruh)||(fila_subh==aux1_cruh && columna_subh==aux_cruh)||(fila_subh==aux2_cruh && columna_subh==aux3_cruh)||
       (aux1_subh==fila_cruh && aux_subh==columna_cruh)||(aux1_subh==aux1_cruh && aux_subh==aux_cruh)||(aux1_subh==aux2_cruh && aux_subh==aux3_cruh)||
       (aux2_subh==fila_cruh && aux3_subh==columna_cruh)||(aux2_subh==aux1_cruh && aux3_subh==aux_cruh)||(aux2_subh==aux2_cruh && aux3_subh==aux3_cruh)){
          System.out.println("el barco no se puede ingresar, porque choca con otro");
          bandera=4;
        }
       else
        bandera=5;      
        }//fin del ciclo, que crea el barco submarino
   
    while(bandera==5){//crea barco destructor
      System.out.println("!!barco destructorï��ï��");    
      contador=1;                                   //este ciclo while bandera==5 cumple practicamente la mismas funciones de la creacion del barco anterior
        contador2=1;
        contador3=1;
        coordenadas opr= new coordenadas(); 
        opr.posicion_coor();
        fila=opr.rfila();
        columna=opr.rcolumna();
        while(contador==1){
        System.out.println("ingrese su orientacion con minuscula");
        orientacion=in.readLine();
        if(orientacion.equals("vertical") || orientacion.equals("horizontal")){
        if(orientacion.equals("horizontal")){
            while(contador2==1){
                System.out.println("apartir de la coordenada dada, quiere que el barco este hacia ï��ï��izquierda!! o  ï��ï��derecha!!");
                posicion=in.readLine();
                if(posicion.equals("izquierda") || posicion.equals("derecha")){
                    if(posicion.equals("izquierda")){
                        if(columna==0){
                          bandera=5;
                          System.out.println("no se puede ingresar en este lugar el barco");
                        }
                        else{
                            fila_desh=fila;
                            columna_desh=columna;
                            aux1_desh=fila;
                            aux_desh=columna-1;
                            contador=0;
                        }
                    }
                    if(posicion.equals("derecha")){
                        if(columna==9){
                            bandera=5;
                            System.out.println("no se puede ingresar en este lugar el barco");
                        }
                        else{
                            fila_desh=fila;
                            columna_desh=columna;
                            aux1_desh=fila;
                            aux_desh=columna+1;
                            contador=0;
                        }
                    
                    }
                    contador2=0;
                  }
                else{
                    System.out.println("no corresponden las palabras");
                    contador2=1;
                  }
                }
            contador=0;
            }
         if(orientacion.equals("vertical")){
              while(contador3==1){
               System.out.println("apartir de la coordenada dada, quiere que el barco este hacia ï��ï��arriba!! o  ï��ï��abajo!!");
               posicion2=in.readLine();
               if(posicion2.equals("arriba") || posicion2.equals("abajo")){
                   if(posicion2.equals("arriba")){
                      if(fila==0){
                          bandera=5;
                          System.out.println("no se puede ingresar en este lugar el barco");
                        }
                        else{
                            fila_desh=fila;
                            columna_desh=columna;
                            aux1_desh=fila-1;
                            aux_desh=columna;
                            contador=0;
                        } 
                       contador3=0;
                    }
                    if(posicion2.equals("abajo")){
                      if(fila==9){
                          bandera=5;
                          System.out.println("no se puede ingresar en este lugar el barco");
                        }
                        else{
                            fila_desh=fila;
                            columna_desh=columna;
                            aux1_desh=fila+1;
                            aux_desh=columna;
                            contador=0;
                        }
                    contador3=0;
        
                  }
                }
                  else{
                    System.out.println("no corresponden las palabras");
                    contador3=1;
                  }
                
             contador=0;
            }
        }
    }
         else{
            System.out.println("las palabras ingresadas no corresponden");
            contador=1;
            }
              
        }
        //comparar con portavion
       if((fila_desh==fila_porh && columna_desh==columna_porh)||(fila_desh==aux1_porh && columna_desh==aux_porh)||(fila_desh==aux2_porh && columna_desh==aux3_porh)||(fila_desh==aux4_porh && columna_desh==aux5_porh)||(fila_desh==aux6_porh && columna_desh==aux7_porh)||
       (aux1_desh==fila_porh && aux_desh==columna_porh)||(aux1_desh==aux1_porh && aux_desh==aux_porh)||(aux1_desh==aux2_porh && aux_desh==aux3_porh)||(aux1_desh==aux4_porh && aux_desh==aux5_porh)||(aux1_desh==aux6_porh && aux_desh==aux7_porh)||   
       //comparar con acorazado
       (fila_desh==fila_acoh && columna_desh==columna_acoh)||(fila_desh==aux1_acoh && columna_desh==aux_acoh)||(fila_desh==aux2_acoh && columna_desh==aux3_acoh)||(fila_desh==aux4_acoh && columna_desh==aux5_acoh)||
       (aux1_desh==fila_acoh && aux_desh==columna_acoh)||(aux1_desh==aux1_acoh && aux_desh==aux_acoh)||(aux1_desh==aux2_acoh && aux_desh==aux3_acoh)||(aux1_desh==aux4_acoh && aux_desh==aux5_acoh)||
       //comparar con crucero
       (fila_desh==fila_cruh && columna_desh==columna_cruh)||(fila_desh==aux1_cruh && columna_desh==aux_cruh)||(fila_desh==aux2_cruh && columna_desh==aux3_cruh)||
       (aux1_desh==fila_cruh && aux_desh==columna_cruh)||(aux1_desh==aux1_cruh && aux_desh==aux_cruh)||(aux1_desh==aux2_cruh && aux_desh==aux3_cruh)||
       //comparar con submarino
       (fila_desh==fila_subh && columna_desh==columna_subh)||(fila_desh==aux1_subh && columna_desh==aux_subh)||(fila_desh==aux2_subh && columna_desh==aux3_subh)||
       (aux1_subh==fila_subh && aux_desh==columna_subh)||(aux1_subh==aux1_subh && aux_desh==aux_subh)||(aux1_subh==aux2_subh && aux_desh==aux3_subh)){
       
       
         System.out.println("el barco no se puede ingresar, porque choca con otro");
          bandera=5;
        }
       else
        bandera=6;        
        }//fin del ciclo, creacion del barco destructor
    
    if(bandera==6){//esta condicion envia todas las cordenadas para trabajar el proceso de golpes hacaia los barco en la clase tablero_trabajo
        tablero_trabajo co= new tablero_trabajo();
        co.tablero_trabajo();
        co.poner_barco_golpes(columna_desh,columna_subh,columna_cruh,columna_acoh,columna_porh,fila_desh,fila_subh,fila_cruh,fila_acoh,fila_porh,
        aux_desh,aux_subh,aux_cruh,aux_acoh,aux_porh,aux1_desh,aux1_subh,aux1_cruh,aux1_acoh,aux1_porh,aux2_subh,aux2_cruh,aux2_acoh,aux2_porh,
        aux3_subh,aux3_cruh,aux3_acoh,aux3_porh,aux4_acoh,aux4_porh,aux5_acoh,aux5_porh,aux6_porh,aux7_porh,columna_des,columna_sub,columna_cru,columna_aco,columna_por,fila_des,fila_sub,fila_cru,fila_aco,fila_por,aux_des,aux_sub,aux_cru,aux_aco,aux_por,
        aux1_des,aux1_sub,aux1_cru,aux1_aco,aux1_por,aux2_sub,aux2_cru,aux2_aco,aux2_por,aux3_sub,aux3_cru,aux3_aco,aux3_por,aux4_aco,aux4_por,aux5_aco,aux5_por,aux6_por,aux7_por);
         
    }//fin del if
    }//fin metodo crear barco

Malo Gato

2009-07-03T15:17:00.000-07:00

Esta entrada vinó desde un alumno de mi clase de Inteligencia Artificial, un curso de quinto año. Debería ser un programa que puede jugar un juego perfecto de gato (Tic-Tac-Toe). Como este alumno ha llegado al quinto año sin usar funciones, no se...


#include <stdio.h>
#include <stdlib.h>

main()
{
    int t,i,x,y;
 char tablero[3][3];

    for(t=0; t<3; ++t){
        for(i=0; i<3; ++i){
            tablero[t][i]='-';
  }
 }
 for(t=0; t<3; ++t)
    {
        for(i=0; i<3; ++i)
            printf("%c",tablero[t][i]);
        printf("\n");
    }
 
 printf("seleccione posicion x:");
 scanf("%d", &x);
 printf("seleccione posicion y:");
 scanf("%d", &y);
 
 tablero[x][y] = 'x';
 for(t=0; t<3; ++t)
    {
        for(i=0; i<3; ++i)
            printf("%c",tablero[t][i]);
        printf("\n");
    }
 printf("\n");
 
 if(tablero[1][1]!='-')
 {
  tablero[0][0] = 'o';
 }
 else{
  tablero[1][1]='o';
 }
 for(t=0; t<3; ++t)
    {
        for(i=0; i<3; ++i)
            printf("%c",tablero[t][i]);
        printf("\n");
    }
 printf("seleccione posicion x:");
 scanf("%d", &x);
 printf("seleccione posicion y:");
 scanf("%d", &y);
 printf("\n");
 
 tablero[x][y]='x';
 
  for(t=0; t<3; ++t)
    {
        for(i=0; i<3; ++i)
            printf("%c",tablero[t][i]);
        printf("\n");
    }

 if((tablero[1][1]=='x'){
  if( (tablero[2][2]=='x')||(tablero[2][0]=='x'){
  tablero[2][0]='o';
  }
  //if(tablero[][]){
  
  //}
 }
 printf("seleccione posicion x:");
 scanf("%d", &x);
 printf("seleccione posicion y:");
 scanf("%d", &y);
 printf("\n");
 
 tablero[x][y]='x';
 
 

}

¿Que son ciclos?

2009-06-25T08:40:00.000-07:00

Esta primera entrada es en Java y es desde mi curso de Lenguajes de Programación en 2009. ¿Pueden identificar porque es tan mala? Personalmente yo puedo pensar de otra forma para hacer lo mismo sin ciclos, y será mas elegante. ¿Algunas Sugerencias?

public void cantidad_jugadores( ActionEvent e) {
            if(valor_nulojug[0]==0){
                System.out.println("Primero Establezca La Cantidad de Jugadores en la \n Partida haciendo click en el Boton de jugadores \n mas abajo ");
            }else{
            int valor = valor_nulo[0];
            if(valor==0){
            if(cantidad_jug[0]==8){
            System.out.println("Nombre del Jugador 1");
            linea = in.readLine();
            Nombre_jug1[1] = linea;
            System.out.println("Nombre del Jugador 2");
            linea = in.readLine();
            Nombre_jug1[2] = linea;
            System.out.println("Nombre del Jugador 3");
            linea = in.readLine();
            Nombre_jug1[3] = linea;
            System.out.println("Nombre del Jugador 4");
            linea = in.readLine();
            Nombre_jug1[4] = linea;
            System.out.println("Nombre del Jugador 5");
            linea = in.readLine();
            Nombre_jug1[5] = linea;
            System.out.println("Nombre del Jugador 6");
            linea = in.readLine();
            Nombre_jug1[6] = linea;
            System.out.println("Nombre del Jugador 7");
            linea = in.readLine();
            Nombre_jug1[7] = linea;
            System.out.println("Nombre del Jugador 8");
            linea = in.readLine();
            Nombre_jug1[8] = linea;    
            valor_nulo[0]=1;
            }
            
            if(cantidad_jug[0]==7){
            System.out.println("Nombre del Jugador 1");
            linea = in.readLine();
            Nombre_jug1[1] = linea;
            System.out.println("Nombre del Jugador 2");
            linea = in.readLine();
            Nombre_jug1[2] = linea;
            System.out.println("Nombre del Jugador 3");
            linea = in.readLine();
            Nombre_jug1[3] = linea;
            System.out.println("Nombre del Jugador 4");
            linea = in.readLine();
            Nombre_jug1[4] = linea;
            System.out.println("Nombre del Jugador 5");
            linea = in.readLine();
            Nombre_jug1[5] = linea;
            System.out.println("Nombre del Jugador 6");
            linea = in.readLine();
            Nombre_jug1[6] = linea;
            System.out.println("Nombre del Jugador 7");
            linea = in.readLine();
            Nombre_jug1[7] = linea;
            valor_nulo[0]=1;
    }
        
            
            if(cantidad_jug[0]==6){
            System.out.println("Nombre del Jugador 1");
            linea = in.readLine();
            Nombre_jug1[1] = linea;
            System.out.println("Nombre del Jugador 2");
            linea = in.readLine();
            Nombre_jug1[2] = linea;
            System.out.println("Nombre del Jugador 3");
            linea = in.readLine();
            Nombre_jug1[3] = linea;
            System.out.println("Nombre del Jugador 4");
            linea = in.readLine();
            Nombre_jug1[4] = linea;
            System.out.println("Nombre del Jugador 5");
            linea = in.readLine();
            Nombre_jug1[5] = linea;
            System.out.println("Nombre del Jugador 6");
            linea = in.readLine();
            Nombre_jug1[6] = linea;
                    
            valor_nulo[0]=1;    
            }
            
            if(cantidad_jug[0]==5){
            System.out.println("Nombre del Jugador 1");
            linea = in.readLine();
            Nombre_jug1[1] = linea;
            System.out.println("Nombre del Jugador 2");
            linea = in.readLine();
            Nombre_jug1[2] = linea;
            System.out.println("Nombre del Jugador 3");
            linea = in.readLine();
            Nombre_jug1[3] = linea;
            System.out.println("Nombre del Jugador 4");
            linea = in.readLine();
            Nombre_jug1[4] = linea;
            System.out.println("Nombre del Jugador 5");
            linea = in.readLine();
            Nombre_jug1[5] = linea;
            
            
            valor_nulo[0]=1;    
            }
            
            
         if(cantidad_jug[0]==4){
            System.out.println("Nombre del Jugador 1");
            linea = in.readLine();
            Nombre_jug1[1] = linea;
            System.out.println("Nombre del Jugador 2");
            linea = in.readLine();
            Nombre_jug1[2] = linea;
            System.out.println("Nombre del Jugador 3");
            linea = in.readLine();
            Nombre_jug1[3] = linea;
            System.out.println("Nombre del Jugador 4");
            linea = in.readLine();
            Nombre_jug1[4] = linea;
            
            
            valor_nulo[0]=1;    
            }   
            
            if(cantidad_jug[0]==3){
            System.out.println("Nombre del Jugador 1");
            linea = in.readLine();
            Nombre_jug1[1] = linea;
            System.out.println("Nombre del Jugador 2");
            linea = in.readLine();
            Nombre_jug1[2] = linea;
            System.out.println("Nombre del Jugador 3");
            linea = in.readLine();
            Nombre_jug1[3] = linea;
            
            valor_nulo[0]=1;    
            }
            
            if(cantidad_jug[0]==2){
            System.out.println("Nombre del Jugador 1");
            linea = in.readLine();
            Nombre_jug1[1] = linea;
            System.out.println("Nombre del Jugador 2");
            linea = in.readLine();
            Nombre_jug1[2] = linea;
            
            valor_nulo[0]=1;    
            }
        
            }