DamianMartinDominguez / tp-informatica-cafetera

es una cafetera con tres funciones
0 stars 0 forks source link

Correcciones #1

Open DamRCorba opened 3 years ago

DamRCorba commented 3 years ago

Hola Damian. Por parte del diagrama y la memoria descriptiva, todo ok.

En cuanto al código:

1- Usa define o un tipo enumerativo para describir los estados, asi en el switch no te va a quedar case 1 sino case CALENTAR_AGUA. 2- En lugar de usar al puntero r para tomar el valor del estado, usa return para que las funciones devuelval estado. Eso le da mas claridad al main y al código en general. 3- Separa el código en main.c, funciones.c y libs.h

La mayoria de estos puntos son "cosmeticos" pero tienen que hacerse.

Otra cosa que te esta faltando a nivel consigna es el archivo de configuracion que debes leer y la funcion de init que lo implemente. Podrias configurar los limites a los que se podria calentar agua y el tiempo o caudal/tiempo que sirve agua la maquina.

DamRCorba commented 3 years ago

Hola Damian, esta casi perfecto.

Solo tenes que corregir la ubicacion de los defines que pusiste en el main.

**#define ESPERAR        0
#define CALENTAR_AGUA  1
#define COLOCANDO_CAFE 2
#define SERVIR_AGUA    3
#define CALENTAR_LECHE 4
#define SERVIR_LECHE   5
#define MEZCLAR_BEBIDA 6

estos deberian ir en un archivo de libreria.

Ademas, en el archivo funciones.c haces return 4, return 6 cuando deberias hacer el return con la etiqueta del define.

Corregi eso y te lo doy como aprobado.

DamRCorba commented 3 years ago

Hola Damian. Los defines van el archivo de libreria. En el .h

DamianMartinDominguez commented 3 years ago

ahi esta profe.

saludos cordiales

Dominguez Damian


De: Damian Corbalan @.> Enviado: jueves, 11 de noviembre de 2021 17:58 Para: DamianMartinDominguez/tp-informatica-cafetera @.> Cc: DamianMartinDominguez @.>; Comment @.> Asunto: Re: [DamianMartinDominguez/tp-informatica-cafetera] Correcciones (#1)

Hola Damian. Los defines van el archivo de libreria. En el .h

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/DamianMartinDominguez/tp-informatica-cafetera/issues/1#issuecomment-966620682, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APM36OUCVQ22C7226TDOLODULQVBFANCNFSM5DQJMLTQ.

DamRCorba commented 3 years ago

Ahora si. Aprobado.