RodrigoToroIcarte / IIC2113-2023-1

6 stars 0 forks source link

Maximo largo de funciones #148

Open antoniablanco opened 1 year ago

antoniablanco commented 1 year ago

Hola, en clean se debe intentar utilizar funciones cortas, es decir según lo que entiendo 4-5 lineas. Es sobre ello que quiero preguntar de dos casos específicos, el primero es el flujo del juego, el cual se encuentra en la función play. En ese caso también debemos mantener ese máximo? y el segundo en una función switch case como es el caso de los tipos de acciones a realizar. No realiza nada más que ver en que caso se esta y llamar a la función respectiva, pero como son 5 caso de por si se tiene 10 lineas. Muchas gracias!

RodrigoToroIcarte commented 1 year ago

Pregunta 1: Sí, la función con el flujo del juego también debería ser corta. Ahí la clave es tener funciones que llamen a otras funciones. Por ejemplo en el dominó la función play es así:

public void Jugar()
{
    while (!EsFinJuego())
    {
        JugarTurno();
        _vista.Pausar();
    }
    FelicitarGanadores();
}

... pero igual considera que la idea de fondo es acortar el método hasta que no tenga sentido seguir acortando. Si hay una función de 7 líneas y no pareciera tener sentido seguir acortándola entonces está bien.

Pregunta 2 El switch con los tipos de acciones es un ejemplo donde no se puede acortar más esa función. Tal como dices, no hay nada que hacer en ese caso. Hay que tener un switch y que en cada caso se llame a otra función, para que sea lo más corto posible.

antoniablanco commented 1 year ago

Perfecto, muchas gracias