Bernar42 / Diagrama-de-m-quina-de-estado

0 stars 0 forks source link

Correcciones #2

Open DamRCorba opened 1 year ago

DamRCorba commented 1 year ago

Hola facundo, como estas?

No se ajusta a lo pedido originalmente.

Para empezar falta la memoria descriptiva. Donde expliques que es lo que queres hacer con tus palabras. Eso deberia estar en el readme.md junto con la imagen de l maquina.

La maquina de estados deberia marcar estados del sistema y no acciones. Es decir vas a tener un estado de tanque lleno y otro de tanque llenandose, ademas de uno de error en el que deberias detectar si hay suministro para cargar al tanque.

Con ese diagrama, deberias convertir a cada estado en una funcion de estado. Las funciones de estado no son bloqueantes, y tienen que devolver un estado. En el main tenes que tener un bucle infinito que ejecute el estado segun corresponda.

En la clase subida en git tenes el ejemplo ademas de los lineamientos.

Bernar42 commented 1 year ago

Buenas tardes profe! Acabo de commitear las correcciones que realicé. saludos

DamRCorba commented 1 year ago

Hola, sigue sin cumplir con la forma que se planteo en clase. Ademas estas escribiendo codigo de funciones en un archivo.h lo cual esta mal, o mejor dicho, es mala practica. El codigo siempre en archivos.c

Tenes el video de la clase donde explique el tp?

Bernar42 commented 1 year ago

Hola profe! no tengo el video ese, entonces en dónde pongo el código fuente?

Bernar42 commented 1 year ago

Hola Profe! Hice varios cambios en el programa, ví el vidoe de la clase y prácticamente lo rehice. Espero su corrección gracias

DamRCorba commented 1 year ago

Esta bastante bien. Aunque esto esta mal:

case encender_bomba_f: //el valor del case no puede ser una funcion, en todo caso deberias agregar un define sin el _f 
            estado=encender_bomba_f(config);
          break;

Ademas de esa correccion tenes que hacer la implementacion del main con punteros a funciones.

Saludos

Bernar42 commented 1 year ago

Buenas noches profe! Ya he corregido lo que me faltaba. Perdon por habérselo entregado el finde

DamRCorba commented 1 year ago

Aprobado