* Mas normalizacion de nomenclaruta

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

diff --git a/emufs_gui/form.c b/emufs_gui/form.c
index a97e3ba93159506f2e2190f448ba42f0c93e28ef..5a76a4d8ebabb31bacbd705d66f8c5bc8bdc5e0b 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));
 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;
 
        tmp->primero = tmp->ultimo = NULL;
        tmp->win = win;
 

-       /* TODO : El error se debe verificar afuera? */

        return tmp;
 }
 
 int form_destruir(t_Form *f)
 {
        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;
 
        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;
 
 {
        t_Widget *tmp = NULL;
 

+       if (f == NULL) return;
+

        /* Creo el nuevo widget segun el tipo */
        switch (tipo) {
                case INPUT:
        /* 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;
        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)
        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;
 
        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, ' ');
        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;
        }
                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;
 
        tmp = f->primero;
        my_y = y-1;

+       /* Ejecuto el formulario */

        while (tmp) {
                ++my_y;
                wmove(f->win, my_y, x+offset);
        while (tmp) {
                ++my_y;
                wmove(f->win, my_y, x+offset);


@@ -156,12 +167,23 @@ t_Widget *widget_input_create(t_Campo tipo, const char *nombre, unsigned int max
 {
        t_Widget *tmp = (t_Widget *)malloc(sizeof(t_Widget));
 
 {
        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));
        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));
        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);
 
        tmp->valor[0] = '\0';
        strncpy(tmp->valor, defecto, max);
 


@@ -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));
 {
        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));
 
        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));
        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;
        /* 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;
 void widget_free(t_Widget *w)
 {
        int i;

+       if (w == NULL) return ;
+

        free(w->nombre);
        switch (w->tipo) {
                case INPUT:
        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);
        }
 
        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) {
                /* Verifico si se apreto basckspace */
                if (c == KEY_BACKSPACE) {
                        if (current > 0) {


@@ -248,15 +286,20 @@ 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);
                        /* 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 */
                        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);
                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';
        }
        /* Cierro el string con el \0 */
        w->valor[current+1] = '\0';