Codigo

[DamRCorba]

Hola Gonzalo, el codigo esta mal estructurado. Primero tenes que tener bien separado cada parte que lo componen y en el lugar que corresponde. En el archivo .h solo puede haber prototipos, definiciones y declaraciones de tipos de variables. No puede haber funciones desarrolladas en un arhivo .h.

En el archivo main.c solo tiene que estar el codigo del main en donde llames a la funciones de estado por medio de un switch o un vector de punteros a funciones.

En un tercer archivo "funciones.c" deberian estar desarrolladas el resto de las funciones de tu codigo, las que hoy pusiste en el archivo .h deberia estar en ese archivo.c

Los switch case no tiene que llamar a numeros sino a etiquetas que representen en forma clara que opcion esta ocurriendo.

Recorda que cada estado de tu diagrama tiene que estar representado por una funcion de estado.

[GonzaloEMunoz]

Hola profe, ahí corregí los errores que me dijo, además tengo una duda con respecto al archivo main.exe y lib.h.gch tengo que sacarlos del git?

El mar., 15 de sep. de 2020 a la(s) 13:47, DamRCorba ( notifications@github.com) escribió:

Hola Gonzalo, el codigo esta mal estructurado. Primero tenes que tener bien separado cada parte que lo componen y en el lugar que corresponde. En el archivo .h solo puede haber prototipos, definiciones y declaraciones de tipos de variables. No puede haber funciones desarrolladas en un arhivo .h.

En el archivo main.c solo tiene que estar el codigo del main en donde llames a la funciones de estado por medio de un switch o un vector de punteros a funciones.

En un tercer archivo "funciones.c" deberian estar desarrolladas el resto de las funciones de tu codigo, las que hoy pusiste en el archivo .h deberia estar en ese archivo.c

Los switch case no tiene que llamar a numeros sino a etiquetas que representen en forma clara que opcion esta ocurriendo.

Recorda que cada estado de tu diagrama tiene que estar representado por una funcion de estado.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/GonzaloEMunoz/TrabajoPracticoMaquinaDeEstado-PuenteLevadizo/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/APIFHIY3YBVEQXOYNSRKHODSF6LA5ANCNFSM4RNNKX4A .

[GonzaloEMunoz]

otra duda perdon que lo moleste el archivo funciones.c lo tengo que poner en la carpeta lib o src?

El mié., 16 de sep. de 2020 a la(s) 22:56, Gonzalo Muñoz ( gonzaeliasm@gmail.com) escribió:

Hola profe, ahí corregí los errores que me dijo, además tengo una duda con respecto al archivo main.exe y lib.h.gch tengo que sacarlos del git?

El mar., 15 de sep. de 2020 a la(s) 13:47, DamRCorba ( notifications@github.com) escribió:

Hola Gonzalo, el codigo esta mal estructurado. Primero tenes que tener bien separado cada parte que lo componen y en el lugar que corresponde. En el archivo .h solo puede haber prototipos, definiciones y declaraciones de tipos de variables. No puede haber funciones desarrolladas en un arhivo .h.

En el archivo main.c solo tiene que estar el codigo del main en donde llames a la funciones de estado por medio de un switch o un vector de punteros a funciones.

En un tercer archivo "funciones.c" deberian estar desarrolladas el resto de las funciones de tu codigo, las que hoy pusiste en el archivo .h deberia estar en ese archivo.c

Los switch case no tiene que llamar a numeros sino a etiquetas que representen en forma clara que opcion esta ocurriendo.

Recorda que cada estado de tu diagrama tiene que estar representado por una funcion de estado.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/GonzaloEMunoz/TrabajoPracticoMaquinaDeEstado-PuenteLevadizo/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/APIFHIY3YBVEQXOYNSRKHODSF6LA5ANCNFSM4RNNKX4A .

[DamRCorba]

Hola Gonzalo, con respecto al main y al lib.h.gch son los objetos compilados asique no deberian estar en el git. El archivo funciones.c deberia estar en src.

En cuanto al codigo. Tu main no representa al diagrama. Tene en cuenta que solo deberias tener un switch case con los estados o un vector de punteros a funciones con los estados. Guiate con el ejemplo que di en clase.

[GonzaloEMunoz]

el tema del switch case entonces sacó el switch case que hice para el menú principal? no lo puse en el diagrama pero lo deje por escrito en la descripción

El jue., 17 de sep. de 2020 a la(s) 17:11, DamRCorba ( notifications@github.com) escribió:

Hola Gonzalo, con respecto al main y al lib.h.gch son los objetos compilados asique no deberian estar en el git. El archivo funciones.c deberia estar en src.

En cuanto al codigo. Tu main no representa al diagrama. Tene en cuenta que solo deberias tener un switch case con los estados o un vector de punteros a funciones con los estados. Guiate con el ejemplo que di en clase.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/GonzaloEMunoz/TrabajoPracticoMaquinaDeEstado-PuenteLevadizo/issues/1#issuecomment-694473371, or unsubscribe https://github.com/notifications/unsubscribe-auth/APIFHI4MJXG5DIQBWFAMAZDSGJUOPANCNFSM4RNNKX4A .

[DamRCorba]

Gonzalo, o cambias el diagrama o cambias el codigo, pero el esquema tiene que representar, literalmente, el funcionamiento del codigo.

[GonzaloEMunoz]

Ya modifique el código para que represente el esquema, saludos!

http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail Libre de virus. www.avg.com http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail <#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>

El lun., 21 de sep. de 2020 a la(s) 14:27, DamRCorba ( notifications@github.com) escribió:

Gonzalo, o cambias el diagrama o cambias el codigo, pero el esquema tiene que representar, literalmente, el funcionamiento del codigo.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/GonzaloEMunoz/TrabajoPracticoMaquinaDeEstado-PuenteLevadizo/issues/1#issuecomment-696258037, or unsubscribe https://github.com/notifications/unsubscribe-auth/APIFHI6GOC2GHGU7CXZ3UK3SG6EIPANCNFSM4RNNKX4A .

[GonzaloEMunoz]

Hola profe buen día queria saber si tengo que modificar algo para la entrega del lunes, saludos!

El lun., 21 de sept. de 2020 5:33 PM, Gonzalo Muñoz gonzaeliasm@gmail.com escribió:

Ya modifique el código para que represente el esquema, saludos!

http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail Libre de virus. www.avg.com http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail <#m_4768208321830071872_DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>

El lun., 21 de sep. de 2020 a la(s) 14:27, DamRCorba ( notifications@github.com) escribió:

Gonzalo, o cambias el diagrama o cambias el codigo, pero el esquema tiene que representar, literalmente, el funcionamiento del codigo.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/GonzaloEMunoz/TrabajoPracticoMaquinaDeEstado-PuenteLevadizo/issues/1#issuecomment-696258037, or unsubscribe https://github.com/notifications/unsubscribe-auth/APIFHI6GOC2GHGU7CXZ3UK3SG6EIPANCNFSM4RNNKX4A .

[DamRCorba]

Gonzalo, aun tiene algunas correcciones por delante. Para empezar estas haciendo: #include"..\src\funciones.c" eso esta mal. Es una mala practica, solo deberias estar incluyendo archivos de cabecera, es decir .h, y los archivos de fuente se deben compilar con juntos. Si estas usando dev. deberias estar creando un nuevo proyecto y dentro de el crear a los archivos.

Por otro lado, definiste un tipo enumerativo para los estados y despues no lo usas como tipo de datos. El prototipo de tus funciones de estados no deberia devolver int, sino que deberia devolver el tipo enumerativo que definiste.

[GonzaloEMunoz]

Perfecto profe ahí arregle los errores que me marcó, saludos.

El mié., 23 de sep. de 2020 a la(s) 15:06, DamRCorba ( notifications@github.com) escribió:

Gonzalo, aun tiene algunas correcciones por delante. Para empezar estas haciendo:

include"..\src\funciones.c" eso esta mal. Es una mala practica, solo

deberias estar incluyendo archivos de cabecera, es decir .h, y los archivos de fuente se deben compilar con juntos. Si estas usando dev. deberias estar creando un nuevo proyecto y dentro de el crear a los archivos.

Por otro lado, definiste un tipo enumerativo para los estados y despues no lo usas como tipo de datos. El prototipo de tus funciones de estados no deberia devolver int, sino que deberia devolver el tipo enumerativo que definiste.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/GonzaloEMunoz/TrabajoPracticoMaquinaDeEstado-PuenteLevadizo/issues/1#issuecomment-697808367, or unsubscribe https://github.com/notifications/unsubscribe-auth/APIFHIYF6R33KS2EGIFACLDSHI2JFANCNFSM4RNNKX4A .

[DamRCorba]

Gonzalo, el tp esta aprobado. Como comentario final sobre esto te digo que los estados, por mas que sea estado 1, 2 y 3 es un nombre poco representativo, tanto para el diagrama como para el codigo. Por lo general se espera que los nombres digan algo sobre la funcionalidad del estado y la funcion.

[GonzaloEMunoz]

Hola profe buen dia, hice caso a la recomendación que me dio, ya cambie el código y el diagrama para que represente más la funcionalidad del mismo.

El jue., 24 de sep. de 2020 a la(s) 11:02, DamRCorba ( notifications@github.com) escribió:

Gonzalo, el tp esta aprobado. Como comentario final sobre esto te digo que los estados, por mas que sea estado 1, 2 y 3 es un nombre poco representativo, tanto para el diagrama como para el codigo. Por lo general se espera que los nombres digan algo sobre la funcionalidad del estado y la funcion.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/GonzaloEMunoz/TrabajoPracticoMaquinaDeEstado-PuenteLevadizo/issues/1#issuecomment-698364975, or unsubscribe https://github.com/notifications/unsubscribe-auth/APIFHI72X4VW2BDGPHR3EN3SHNGPPANCNFSM4RNNKX4A .