Algunos detalles de coding style y pruebas de stats.

[z.facultad/75.06/emufs.git] / emufs_gui / articulos.c

diff --git a/emufs_gui/articulos.c b/emufs_gui/articulos.c
index 8fcd2eed259e705d9d53e0998e054e4d9f712dff..ad5d9f4b2f1781ef695bd3785a1a5c08fa1a1bad 100644 (file)
--- a/emufs_gui/articulos.c
+++ b/emufs_gui/articulos.c
@@ -1,6 +1,8 @@
 
 #include "articulos.h"
 #include "idx.h"
 
 #include "articulos.h"
 #include "idx.h"

+#include "gui.h"
+#include "common.h"

 
 static t_LstArticulos *lst_articulos;
 
 
 static t_LstArticulos *lst_articulos;
 


@@ -82,6 +84,7 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
        tmp->primero = NULL;
 
        if (filename != NULL) {
        tmp->primero = NULL;
 
        if (filename != NULL) {

+               PERR("Voy a crear desde un XML");

                document = xmlReadFile(filename, "ISO-8859-1",0);
                if (document == NULL) {
                        free(tmp);
                document = xmlReadFile(filename, "ISO-8859-1",0);
                if (document == NULL) {
                        free(tmp);


@@ -109,15 +112,20 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
                                        t_Articulo art;
                                        void *save;
                                        memset(&art, '*', sizeof(t_Articulo));
                                        t_Articulo art;
                                        void *save;
                                        memset(&art, '*', sizeof(t_Articulo));

-                                       prop = xmlGetProp(node, "NroArtículo");
+                                       prop = xml_get_prop(node, "NroArtículo");

                                        art.numero = atoi(prop);
                                        xmlFree(prop);
                                        art.numero = atoi(prop);
                                        xmlFree(prop);

-                                       strcpy(art.desc, prop = xmlGetProp(node, "Descripción")); xmlFree(prop);
-                                       strcpy(art.presentacion, prop = xmlGetProp(node, "Presentación")); xmlFree(prop);
-                                       strcpy(art.existencia, prop = xmlGetProp(node, "Existencia")); xmlFree(prop);
-                                       /*strncpy(tmp->array[cant].ubicacion, xmlGetProp(node, "Ubicacion"), 30);*/
-                                       strcpy(art.pvu, prop = xmlGetProp(node, "PVU")); xmlFree(prop);
-                                       strcpy(art.emin, prop = xmlGetProp(node, "Emín")); xmlFree(prop);
+                                       strncpy(art.desc, prop = xml_get_prop(node, "Descripción"), 50); xmlFree(prop);
+                                       art.desc[50] = '\0'; /* Me aseguro de que este */
+                                       strncpy(art.presentacion, prop = xml_get_prop(node, "Presentación"), 30); xmlFree(prop);
+                                       art.presentacion[30] = '\0'; /* Me aseguro de que este */
+                                       strncpy(art.existencia, prop = xml_get_prop(node, "Existencia"), 8); xmlFree(prop);
+                                       art.existencia[8] = '\0'; /* Me aseguro de que este */
+                                       strncpy(art.ubicacion, prop = xml_get_prop(node, "Ubicacion"), 30); xmlFree(prop);
+                                       strncpy(art.pvu, prop = xml_get_prop(node, "PVU"), 8); xmlFree(prop);
+                                       art.pvu[8] = '\0'; /* Me aseguro de que este */
+                                       strncpy(art.emin, prop = xml_get_prop(node, "Emín"), 8); xmlFree(prop);
+                                       art.emin[8] = '\0'; /* Me aseguro de que este */

                                        /* Ya leido el articulo ahora paso a guardarlo en el archivo y agregarlo a la lista */
                                        save = procesar_guardar_articulo(&art, &size, lst_articulos);
                                        if (save != NULL) {
                                        /* Ya leido el articulo ahora paso a guardarlo en el archivo y agregarlo a la lista */
                                        save = procesar_guardar_articulo(&art, &size, lst_articulos);
                                        if (save != NULL) {


@@ -132,7 +140,14 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
                xmlFreeDoc(document);
                xmlCleanupParser();
        } else {
                xmlFreeDoc(document);
                xmlCleanupParser();
        } else {

+               PERR("Voy a recuperar desde un archivo");

                tmp->fp = emufs_abrir("articulos");
                tmp->fp = emufs_abrir("articulos");

+               if (tmp->fp == NULL) {
+                       PERR("No se pudo cargar archivo de articulos.");
+                       free(tmp);
+                       lst_articulos = NULL;
+                       return NULL;
+               }

                /* Ahora trato de recuperar la info */
                indices = emufs_idx_get(tmp->fp, &indices_cant);
                for(i=0; i<indices_cant; i++) {
                /* Ahora trato de recuperar la info */
                indices = emufs_idx_get(tmp->fp, &indices_cant);
                for(i=0; i<indices_cant; i++) {


@@ -154,9 +169,9 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
 
 int art_liberar(t_LstArticulos *l)
 {
 
 int art_liberar(t_LstArticulos *l)
 {

+       t_Reg_Articulo *del;

        if (l == NULL) l = lst_articulos;
        if (l == NULL) return 1;
        if (l == NULL) l = lst_articulos;
        if (l == NULL) return 1;

-       t_Reg_Articulo *del;

 
        emufs_destruir(l->fp);
        while (l->primero) {
 
        emufs_destruir(l->fp);
        while (l->primero) {


@@ -234,7 +249,7 @@ void art_modificar(char *s)
        EMUFS_REG_SIZE size;
        EMUFS_REG_ID id;
 
        EMUFS_REG_SIZE size;
        EMUFS_REG_ID id;
 

-       win = newwin(8, COLS-2, 13, 1);
+       win = newwin(9, COLS-2, 13, 1);

        box(win, 0, 0);
        wrefresh(win);
 
        box(win, 0, 0);
        wrefresh(win);
 


@@ -267,6 +282,7 @@ void art_modificar(char *s)
                form_agregar_widget(form, INPUT, "Descripción", 50, articulo->desc);
                form_agregar_widget(form, INPUT, "Presentación", 30, articulo->presentacion);
                form_agregar_widget(form, INPUT, "Stock Actual", 8, articulo->existencia);
                form_agregar_widget(form, INPUT, "Descripción", 50, articulo->desc);
                form_agregar_widget(form, INPUT, "Presentación", 30, articulo->presentacion);
                form_agregar_widget(form, INPUT, "Stock Actual", 8, articulo->existencia);

+               form_agregar_widget(form, INPUT, "Ubicacion", 30, articulo->ubicacion);

                form_agregar_widget(form, INPUT, "PVU", 8, articulo->pvu);
                form_agregar_widget(form, INPUT, "Stock Mínimo", 8, articulo->emin);
                form_ejecutar(form, 1,1);
                form_agregar_widget(form, INPUT, "PVU", 8, articulo->pvu);
                form_agregar_widget(form, INPUT, "Stock Mínimo", 8, articulo->emin);
                form_ejecutar(form, 1,1);


@@ -276,6 +292,7 @@ void art_modificar(char *s)
                strcpy(articulo->desc, form_obtener_valor_char(form, "Descripción"));
                strcpy(articulo->presentacion, form_obtener_valor_char(form, "Presentación"));
                strcpy(articulo->existencia, form_obtener_valor_char(form, "Stock Actual"));
                strcpy(articulo->desc, form_obtener_valor_char(form, "Descripción"));
                strcpy(articulo->presentacion, form_obtener_valor_char(form, "Presentación"));
                strcpy(articulo->existencia, form_obtener_valor_char(form, "Stock Actual"));

+               strcpy(articulo->ubicacion, form_obtener_valor_char(form, "Ubicacion"));

                strcpy(articulo->pvu, form_obtener_valor_char(form, "PVU"));
                strcpy(articulo->emin, form_obtener_valor_char(form, "Stock Mínimo"));
                /* Ya actualice los datos, ahora veo de grabarlos */
                strcpy(articulo->pvu, form_obtener_valor_char(form, "PVU"));
                strcpy(articulo->emin, form_obtener_valor_char(form, "Stock Mínimo"));
                /* Ya actualice los datos, ahora veo de grabarlos */


@@ -348,7 +365,7 @@ void art_agregar(char *s)
        EMUFS_REG_SIZE size;
        EMUFS_REG_ID id;
 
        EMUFS_REG_SIZE size;
        EMUFS_REG_ID id;
 

-       win = newwin(8, COLS-2, 13, 1);
+       win = newwin(9, COLS-2, 13, 1);

        box(win, 0, 0);
        wrefresh(win);
 
        box(win, 0, 0);
        wrefresh(win);
 


@@ -357,6 +374,7 @@ void art_agregar(char *s)
        form_agregar_widget(form, INPUT, "Descripción", 50, "");
        form_agregar_widget(form, INPUT, "Presentación", 30, "");
        form_agregar_widget(form, INPUT, "Stock Actual", 8, "");
        form_agregar_widget(form, INPUT, "Descripción", 50, "");
        form_agregar_widget(form, INPUT, "Presentación", 30, "");
        form_agregar_widget(form, INPUT, "Stock Actual", 8, "");

+       form_agregar_widget(form, INPUT, "Ubicacion", 30, "");

        form_agregar_widget(form, INPUT, "PVU", 8, "");
        form_agregar_widget(form, INPUT, "Stock Mínimo", 8, "");
        form_ejecutar(form, 1,1);
        form_agregar_widget(form, INPUT, "PVU", 8, "");
        form_agregar_widget(form, INPUT, "Stock Mínimo", 8, "");
        form_ejecutar(form, 1,1);


@@ -376,7 +394,7 @@ void art_agregar(char *s)
                strcpy(art.desc, form_obtener_valor_char(form, "Descripción"));
                strcpy(art.presentacion, form_obtener_valor_char(form, "Presentación"));
                strcpy(art.existencia, form_obtener_valor_char(form, "Stock Actual"));
                strcpy(art.desc, form_obtener_valor_char(form, "Descripción"));
                strcpy(art.presentacion, form_obtener_valor_char(form, "Presentación"));
                strcpy(art.existencia, form_obtener_valor_char(form, "Stock Actual"));

-                               /*strncpy(tmp->array[cant].ubicacion, xmlGetProp(node, "Ubicacion"), 30);*/
+               strcpy(art.ubicacion, form_obtener_valor_char(form, "Ubicacion"));

                strcpy(art.pvu, form_obtener_valor_char(form, "PVU"));
                strcpy(art.emin, form_obtener_valor_char(form, "Stock Mínimo"));
 
                strcpy(art.pvu, form_obtener_valor_char(form, "PVU"));
                strcpy(art.emin, form_obtener_valor_char(form, "Stock Mínimo"));
 


@@ -418,7 +436,7 @@ int procesar_leer_articulo(t_Articulo *dst, void *src, EMUFS_REG_SIZE size, t_Ls
                case T2:
                        ini = (char *)src;
                        /* Copio el primer campo, esto es facil :-) */
                case T2:
                        ini = (char *)src;
                        /* Copio el primer campo, esto es facil :-) */

-                       memset(dst, '*', sizeof(t_Articulo));
+                       memset(dst, 0, sizeof(t_Articulo));

                        memcpy(&dst->numero, ini, sizeof(unsigned int));
                        ini+=sizeof(unsigned int);
                        /* Ahora empieza el juego */
                        memcpy(&dst->numero, ini, sizeof(unsigned int));
                        ini+=sizeof(unsigned int);
                        /* Ahora empieza el juego */


@@ -437,6 +455,11 @@ int procesar_leer_articulo(t_Articulo *dst, void *src, EMUFS_REG_SIZE size, t_Ls
                        while (*fin!='\0') fin++;
                        memcpy(dst->existencia, ini, fin-ini+1);
                        
                        while (*fin!='\0') fin++;
                        memcpy(dst->existencia, ini, fin-ini+1);
                        

+                       ini = fin+1;
+                       fin = ini;
+                       while (*fin!='\0') fin++;
+                       memcpy(dst->ubicacion, ini, fin-ini+1);
+                       

                        ini = fin+1;
                        fin = ini;
                        while (*fin!='\0') fin++;
                        ini = fin+1;
                        fin = ini;
                        while (*fin!='\0') fin++;


@@ -457,7 +480,7 @@ int procesar_leer_articulo(t_Articulo *dst, void *src, EMUFS_REG_SIZE size, t_Ls
 void *procesar_guardar_articulo(t_Articulo *src, EMUFS_REG_SIZE *size, t_LstArticulos *lst)
 {
        char *tmp=NULL;
 void *procesar_guardar_articulo(t_Articulo *src, EMUFS_REG_SIZE *size, t_LstArticulos *lst)
 {
        char *tmp=NULL;

-       int i[6];
+       int i[7];

        char *from = (char *)src;
        switch(lst->fp->tipo) {
                case T1:
        char *from = (char *)src;
        switch(lst->fp->tipo) {
                case T1:


@@ -467,10 +490,10 @@ void *procesar_guardar_articulo(t_Articulo *src, EMUFS_REG_SIZE *size, t_LstArti
                        i[1] = sizeof(char)*(strlen(src->desc)+1);
                        i[2] = sizeof(char)*(strlen(src->presentacion)+1);
                        i[3] = sizeof(char)*(strlen(src->existencia)+1);
                        i[1] = sizeof(char)*(strlen(src->desc)+1);
                        i[2] = sizeof(char)*(strlen(src->presentacion)+1);
                        i[3] = sizeof(char)*(strlen(src->existencia)+1);

-/*                     i[4] = sizeof(char)*(strlen(src->ubicacion)+1); */
-                       i[4] = sizeof(char)*(strlen(src->pvu)+1);
-                       i[5] = sizeof(char)*(strlen(src->emin)+1);
-                       (*size) = i[0]+i[1]+i[2]+i[3]+i[4]+i[5];
+                       i[4] = sizeof(char)*(strlen(src->ubicacion)+1); 
+                       i[5] = sizeof(char)*(strlen(src->pvu)+1);
+                       i[6] = sizeof(char)*(strlen(src->emin)+1);
+                       (*size) = i[0]+i[1]+i[2]+i[3]+i[4]+i[5]+i[6];

                        tmp = (char *)malloc((*size));
                        if (tmp == NULL) return NULL;
                        memset(tmp, 0, *size);
                        tmp = (char *)malloc((*size));
                        if (tmp == NULL) return NULL;
                        memset(tmp, 0, *size);


@@ -478,8 +501,9 @@ void *procesar_guardar_articulo(t_Articulo *src, EMUFS_REG_SIZE *size, t_LstArti
                        memcpy(tmp+i[0], src->desc, i[1]);
                        memcpy(tmp+i[0]+i[1], src->presentacion, i[2]);
                        memcpy(tmp+i[0]+i[1]+i[2], src->existencia, i[3]);
                        memcpy(tmp+i[0], src->desc, i[1]);
                        memcpy(tmp+i[0]+i[1], src->presentacion, i[2]);
                        memcpy(tmp+i[0]+i[1]+i[2], src->existencia, i[3]);

-                       memcpy(tmp+i[0]+i[1]+i[2]+i[3], src->pvu, i[4]);
-                       memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4], src->emin, i[5]);
+                       memcpy(tmp+i[0]+i[1]+i[2]+i[3], src->ubicacion, i[4]);
+                       memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4], src->pvu, i[5]);
+                       memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4]+i[5], src->emin, i[6]);

                break;
                case T3:
                        /* Lleno el lugar no ocupado de los strings con *, para que el ver
                break;
                case T3:
                        /* Lleno el lugar no ocupado de los strings con *, para que el ver


@@ -513,7 +537,7 @@ void art_reformatear(int tipo, int tam_bloque, int tam_reg)
        PERR("Creo el archivo\n");
        nuevo = emufs_crear("emufs_tmp", tipo, tam_bloque, sizeof(t_Articulo));
        if (nuevo == NULL) {
        PERR("Creo el archivo\n");
        nuevo = emufs_crear("emufs_tmp", tipo, tam_bloque, sizeof(t_Articulo));
        if (nuevo == NULL) {

-               fprintf(stderr, "ARCHIVO NUEVO NO CREADO\n");
+               PERR("ARCHIVO NUEVO NO CREADO");

                return;
        }
 
                return;
        }