* Agrego un msg de advertencia a la GUI si se compila con -DDEBUG, para prevenir

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

diff --git a/emufs_gui/form.c b/emufs_gui/form.c
index a97e3ba93159506f2e2190f448ba42f0c93e28ef..d86d4b158a5d3c20b1dcbd6b84bb67a7eb321d29 100644 (file)
--- a/emufs_gui/form.c
+++ b/emufs_gui/form.c
@@ -38,16 +38,21 @@ static int form_radio(WINDOW *win, int x, int y, t_Widget *w);
 t_Form *form_crear(WINDOW *win)
 {
        t_Form *tmp = (t_Form *)malloc(sizeof(t_Form));
+       if (tmp == NULL) {
+               return NULL;
+       }
        tmp->primero = tmp->ultimo = NULL;
        tmp->win = win;
 
-       /* TODO : El error se debe verificar afuera? */
        return tmp;
 }
 
 int form_destruir(t_Form *f)
 {
-       t_Widget *tmp = f->primero;
+       t_Widget *tmp;
+       if (f == NULL) return 0;
+
+       tmp = f->primero;
 
        while (tmp) {
                f->primero = f->primero->sig;
@@ -62,6 +67,8 @@ void form_agregar_widget(t_Form *f, t_Campo tipo, const char *nombre, unsigned i
 {
        t_Widget *tmp = NULL;
 
+       if (f == NULL) return;
+
        /* Creo el nuevo widget segun el tipo */
        switch (tipo) {
                case INPUT:
@@ -87,7 +94,8 @@ void form_ejecutar(t_Form *f, int x, int y)
        int offset = 0, my_y, salida;
        t_Widget *tmp = f->primero;
        my_y = y-1;
-       /* Pongo las etiquetas de los campos, y me fijo el mayor offset */
+       
+       /* Calculo cual es la etiqueta más larga de FORM */
        while (tmp) {
                my_y++;
                if (strlen(tmp->nombre) > offset)
@@ -100,11 +108,13 @@ void form_ejecutar(t_Form *f, int x, int y)
 
        tmp = f->primero;
        my_y = y-1;
+       /* Agrego el etiqueta y el valor actual para cada elemento */
        while (tmp) {
                ++my_y;
                mvwaddstr(f->win, my_y, x, tmp->nombre);
                waddch(f->win, ':');
                waddch(f->win, ' ');
+               /* TODO : VER QUE SI ES UNA OPCION ES DISTINTO!! */
                mvwaddstr(f->win, my_y, x+offset, tmp->valor);
                tmp = tmp->sig;
        }
@@ -112,6 +122,7 @@ void form_ejecutar(t_Form *f, int x, int y)
 
        tmp = f->primero;
        my_y = y-1;
+       /* Ejecuto el formulario */
        while (tmp) {
                ++my_y;
                wmove(f->win, my_y, x+offset);
@@ -156,14 +167,25 @@ t_Widget *widget_input_create(t_Campo tipo, const char *nombre, unsigned int max
 {
        t_Widget *tmp = (t_Widget *)malloc(sizeof(t_Widget));
 
+       if (tmp == NULL) return NULL;
+
        tmp->tipo = tipo;
        tmp->nombre = (char *)malloc(sizeof(char)*(strlen(nombre)+1));
+       if (tmp->nombre == NULL) {
+               free(tmp);
+               return NULL;
+       }
        strcpy(tmp->nombre, nombre);
 
        tmp->max = max;
        tmp->valor = (char *)malloc(sizeof(char)*(max+1));
+       if (tmp->valor == NULL) {
+               free(tmp->nombre);
+               free(tmp);
+               return NULL;
+       }
        tmp->valor[0] = '\0';
-       strncpy(tmp->valor, defecto, max);
+       strcpy(tmp->valor, defecto);
 
        tmp->sig = NULL;
 
@@ -176,13 +198,26 @@ t_Widget *widget_radio_create(t_Campo tipo, const char *nombre, unsigned int max
 {
        int ini, fin, actual;
        t_Widget *tmp = (t_Widget *)malloc(sizeof(t_Widget));
+       if (tmp == NULL) {
+               return NULL;
+       }
 
        tmp->tipo = tipo;
        tmp->nombre = (char *)malloc(sizeof(char)*(strlen(nombre)+1));
+       if (tmp->nombre == NULL) {
+               free(tmp);
+               return NULL;
+       }
        strcpy(tmp->nombre, nombre);
 
        tmp->max = max;
        tmp->opciones = (char **)malloc(sizeof(char*)*(max));
+       if (tmp->opciones == NULL) {
+               free(tmp->nombre);
+               free(tmp);
+               return NULL;
+       }
+
        /* Parseo VALOR separado por comas */
        actual = ini = 0;
        fin = ini+1;
@@ -212,6 +247,8 @@ t_Widget *widget_radio_create(t_Campo tipo, const char *nombre, unsigned int max
 void widget_free(t_Widget *w)
 {
        int i;
+       if (w == NULL) return ;
+
        free(w->nombre);
        switch (w->tipo) {
                case INPUT:
@@ -237,7 +274,8 @@ int form_input(WINDOW *win, int x, int y, t_Widget *w)
        }
 
        wrefresh(win);
-       while ((c=getch()) != 13) {
+       c = getch();
+       while ((c != KEY_ENTER) && (c != 13)) {
                /* Verifico si se apreto basckspace */
                if (c == KEY_BACKSPACE) {
                        if (current > 0) {
@@ -248,19 +286,23 @@ int form_input(WINDOW *win, int x, int y, t_Widget *w)
                        /* Este va para dejar el cursor bien, ya que addch mueve el cursor*/
                        wmove(win, y, x+current);
                        wrefresh(win);
+                       c = getch();
                        continue;
                }
                /* Si no entra mas, ignoro toda entrada */
-               if (current >= w->max) continue;
-
+               if (current >= w->max) {
+                       c = getch();
+                       continue;
+               }
+               
                wmove(win, y, x+current);
                waddch(win, c);
                w->valor[current++] = c;
                wrefresh(win);
+               c = getch();
        }
        /* Cierro el string con el \0 */
-       w->valor[current+1] = '\0';
-
+       w->valor[current] = '\0';
        /* Retorno la tecla con la que se salio. */
        /* De esa forma, ESC pasa al campo anterios. ENTER al siguiente */
        return c;