se mejora la conexion entre compuertas, se salvan en el XML, hay un par de cosas...

diff --git a/Constructor/dialogs/constructor.glade b/Constructor/dialogs/constructor.glade
index 86e551b0256deca55b057c0d2d55dd82b9ec8ccf..c5828861aebe622c658e7fafd8822eaaf855782b 100644 (file)
--- a/Constructor/dialogs/constructor.glade
+++ b/Constructor/dialogs/constructor.glade
@@ -1342,7 +1342,7 @@
 </widget>
 
 <widget class="GtkDialog" id="exclusa_pty_wnd">
-  <property name="width_request">267</property>
+  <property name="width_request">299</property>
   <property name="height_request">143</property>
   <property name="title" translatable="yes">Propiedades de la Exclusa</property>
   <property name="type">GTK_WINDOW_TOPLEVEL</property>

diff --git a/Constructor/include/workplace.h b/Constructor/include/workplace.h
index f226ee76348c608e0da540ec1a1912388910958c..8a533e2bc4d1ec405829b7852710097001f9f3d5 100644 (file)
--- a/Constructor/include/workplace.h
+++ b/Constructor/include/workplace.h
@@ -6,7 +6,7 @@
 #include <gtkmm.h>
 #include <libglademm.h>
 /**Esta clase define el area de trabajo.
-  *Ella se encarga de eliminarlos y dibujar las lineas logicas que los conectan
+  *Ella se encarga de eliminar los items y dibujar las lineas logicas que los conectan
 */
 
 ///Indico que la clase CItem existe para poder compilar.
@@ -56,7 +56,7 @@ class WorkPlace:public Gtk::Fixed {
                   * lista_lineas_in = (salida) compuerta --> (entrada) item
                   * lista_lineas_out= (entrada) compuerta --> (salida) item
                   * lista_lineas_logic = (salida) compuerta --> (entrada) compuerta.
-                  * Esta ultima lista contiene las lineas que van desde la compuerta seleccionada que se guarda en gate_id hasta
+                  * Esta ultima lista contiene las lineas que van desde la compuerta seleccionada que se guarda CItem::en gate_id hasta
                   * la compuerta clickeada en segundo termino.
                */
                std::list<t_line> lista_lineas_in, lista_lineas_out, lista_lineas_logic;

diff --git a/Constructor/src/and.cpp b/Constructor/src/and.cpp
index 15230c0678d8719d66ef68ef7fcf3ddd10be1a64..504747b2cfd58d592ca02606831dd4478faa9111 100644 (file)
--- a/Constructor/src/and.cpp
+++ b/Constructor/src/and.cpp
@@ -122,7 +122,7 @@ bool And::on_button_press_event(GdkEventButton *event)
        }
        if ((event->type == GDK_BUTTON_PRESS) && ( event->button == 3)){
                menu_popup.popup(event->button, event->time);
-                return true; //It has been handled.
+               return true; //It has been handled.
        }
        workplace->queue_draw();
        return true;
@@ -166,29 +166,47 @@ void And::save(FILE *archivo)
 
 bool And::check_connection()
 {
-       int cant_in = 0, cant_out =0;
        t_logic_connector temp;
        std::list<t_line>::iterator i = workplace->lista_lineas_in.begin();
        while ( i != workplace->lista_lineas_in.end() ) {
-               if ( workplace->get_logic_item((*i).logic_id)->get_id() == ID ) {
+               if ( (*i).logic_id== ID ) {
                        temp.type = OUT;
                        temp.name_dest = workplace->get_item((*i).store_id)->get_name();
                        vec_connector.push_back(temp);
-                       cant_out++;
                }
                i++;
        }
 
        i = workplace->lista_lineas_out.begin();
        while ( i != workplace->lista_lineas_out.end() ) {
-               if ( workplace->get_logic_item((*i).logic_id)->get_id() == ID ) {
+               if ( (*i).logic_id== ID ) {
                        temp.type = IN;
                        temp.name_dest = workplace->get_item((*i).store_id)->get_name();
                        vec_connector.push_back(temp);
-                       cant_in++;
                }
                i++;
        }
+
+       i = workplace->lista_lineas_logic.begin();
+       while ( i != workplace->lista_lineas_logic.end() ) {
+               if ( (*i).logic_id == ID ) {
+                       temp.type = OUT;
+                       temp.name_dest = workplace->get_logic_item((*i).store_id)->get_name();
+                       vec_connector.push_back(temp);
+               }
+               i++;
+       }
+
+       i = workplace->lista_lineas_logic.begin();
+       while ( i != workplace->lista_lineas_logic.end() ) {
+               if ( (*i).store_id == ID ) {
+                       temp.type = IN;
+                       temp.name_dest = workplace->get_logic_item((*i).logic_id)->get_name();
+                       vec_connector.push_back(temp);
+               }
+               i++;
+       }
+
        return true; //out_connected;
 }
 

diff --git a/Constructor/src/constructor.cpp b/Constructor/src/constructor.cpp
index 29c399f246b42c2836a19fe63587bffd7612e17e..8a368099443ee3690758de92cafacb6863601208 100644 (file)
--- a/Constructor/src/constructor.cpp
+++ b/Constructor/src/constructor.cpp
@@ -6,7 +6,7 @@ Constructor::Constructor(BaseObjectType* cobject, const Glib::RefPtr<Gnome::Glad
        saved = false;
        set_title("Constructor");
        
-       
+       // Cargo todas las imagenes de los iconos de los botones
        ico_canio = Gdk::Pixbuf::create_from_file(PACKAGE_DATA_DIR"/plaqui-constructor/pixmaps/canio_n.png");
        ico_y     = Gdk::Pixbuf::create_from_file(PACKAGE_DATA_DIR"/plaqui-constructor/pixmaps/y_n.png");
        ico_codo  = Gdk::Pixbuf::create_from_file(PACKAGE_DATA_DIR"/plaqui-constructor/pixmaps/codo_o.png");
@@ -18,6 +18,7 @@ Constructor::Constructor(BaseObjectType* cobject, const Glib::RefPtr<Gnome::Glad
        ico_or = Gdk::Pixbuf::create_from_file(PACKAGE_DATA_DIR"/plaqui-constructor/pixmaps/or_e.png");
        ico_not = Gdk::Pixbuf::create_from_file(PACKAGE_DATA_DIR"/plaqui-constructor/pixmaps/not_e.png");
        
+       //Obtengo todos los widgets de la ventana principal.
        refGlade->get_widget("btn_canio", btn_canio);
        refGlade->get_widget("btn_codo", btn_codo);
        refGlade->get_widget("btn_y", btn_y);
@@ -48,6 +49,7 @@ Constructor::Constructor(BaseObjectType* cobject, const Glib::RefPtr<Gnome::Glad
        refGlade->get_widget("btn_find", btn_find);
        refGlade->get_widget("status_bar", status_bar);
        
+       //Obtengo el area de trabajo, la cual tiene definida su propia clase.
        refGlade->get_widget_derived("workplace", workplace);  //fixed
        
        //Targets
@@ -56,6 +58,7 @@ Constructor::Constructor(BaseObjectType* cobject, const Glib::RefPtr<Gnome::Glad
        listTargets.push_back( Gtk::TargetEntry("POINTER") );
        listTargets.push_back( Gtk::TargetEntry("application/pointer") );
        
+       // Le indico cuales son los botones  desde los cuales se puede hacer un drag.
        btn_canio->drag_source_set(listTargets);
        btn_y->drag_source_set(listTargets);
        btn_codo->drag_source_set(listTargets);
@@ -67,6 +70,7 @@ Constructor::Constructor(BaseObjectType* cobject, const Glib::RefPtr<Gnome::Glad
        btn_or->drag_source_set(listTargets);
        btn_not->drag_source_set(listTargets);
        
+       //Conecto las señales de cada boton con su correspondiente metodo.
        btn_canio->signal_drag_data_get().connect( SigC::slot(*this, &Constructor::on_btn_canio_drag_get));
        btn_bomba->signal_drag_data_get().connect( SigC::slot(*this, &Constructor::on_btn_bomba_drag_get));
        btn_exclusa->signal_drag_data_get().connect( SigC::slot(*this, &Constructor::on_btn_exclusa_drag_get));
@@ -92,11 +96,11 @@ Constructor::Constructor(BaseObjectType* cobject, const Glib::RefPtr<Gnome::Glad
        edit_menu_del->signal_activate().connect(SigC::slot(*this,&Constructor::on_edit_menu_del));
        edit_menu_delete_all->signal_activate().connect(SigC::slot(*this,&Constructor::on_quick_btn_delete_all_clicked));
        
+       //Obtengo y conecto los botones del dialogo de Salvar/Cargar un archivo.
        btn_file_cancel = file_selection->get_cancel_button();
        btn_file_ok = file_selection->get_ok_button();
        btn_file_ok->signal_clicked().connect(SigC::slot(*this,&Constructor::on_btn_file_ok_clicked));
        btn_file_cancel->signal_clicked().connect(SigC::slot(*this,&Constructor::on_btn_file_cancel_clicked));
-       
        btn_open_cancel = file_open_selection->get_cancel_button();
        btn_open_ok = file_open_selection->get_ok_button();
        btn_open_ok->signal_clicked().connect(SigC::slot(*this,&Constructor::on_load_from_xml));
@@ -115,6 +119,7 @@ Constructor::Constructor(BaseObjectType* cobject, const Glib::RefPtr<Gnome::Glad
        btn_not->signal_drag_begin().connect( SigC::slot(*this, &Constructor::on_not_drag_begin));
        workplace->drag_dest_set(listTargets);
        workplace->signal_drag_data_received().connect( SigC::slot(*this, &Constructor::on_item_drop_drag_received) );  
+       // Apunto la lista del area de trabajo a la lista de items en el Constructor
        workplace->listaItems = &listaItems;    
        workplace->lista_logic_Items = &lista_logic_Items;      
        workplace->logica = &logica;
@@ -123,9 +128,11 @@ Constructor::Constructor(BaseObjectType* cobject, const Glib::RefPtr<Gnome::Glad
 
 Constructor::~Constructor()
 {
+       // elimina todos los items 
        Constructor::on_edit_menu_delete_all();
 }
 
+// Definicion de los metodos para obtener el icono al realizar un drag.
 void Constructor::on_btn_canio_drag_get(const Glib::RefPtr<Gdk::DragContext>& context, GtkSelectionData* selection_data, guint info, guint time)
 {
        gtk_selection_data_set (selection_data, selection_data->target, 8, (const guchar*)"canio_n.png", 12);
@@ -176,7 +183,7 @@ void Constructor::on_btn_not_drag_get(const Glib::RefPtr<Gdk::DragContext>& cont
        gtk_selection_data_set(selection_data, selection_data->target, 8,(const guchar*)"not_e.png",9);
 }
 
-
+//Sale del programa.
 void Constructor::on_main_menu_quit()
 {
        delete this;
@@ -184,12 +191,14 @@ void Constructor::on_main_menu_quit()
        //hide();
 }
 
+// Carga de un archivo XML
 void Constructor::on_main_menu_open()
 {
        // Conecto el boton OK para llamar al cargar del XML
        file_open_selection->show();
 }
 
+// Salva un archivo XML
 void Constructor::on_main_menu_save()
 {
        // Conecto el boton OK para llamar al salvar
@@ -202,11 +211,13 @@ void Constructor::on_main_menu_save()
                file_selection->show();
 }
 
+// Elimina el item apuntado desde el menu principal
 void Constructor::on_edit_menu_del()
 { 
        workplace->delete_item(WorkPlace::pointed);
 }
 
+// Elimina todos lo items
 void Constructor::on_edit_menu_delete_all()
 {
        std::list<CItem *>::iterator i = listaItems.begin();
@@ -226,6 +237,7 @@ void Constructor::on_edit_menu_delete_all()
        combo_entry->get_entry()->set_text("");
 }
 
+// Al presionarse el check button "logica"
 void Constructor::on_chk_btn_logica_clicked()
 {
        logica = !logica;
@@ -233,6 +245,7 @@ void Constructor::on_chk_btn_logica_clicked()
        workplace->queue_draw();
 }
 
+// Metodo que levanta el archivo XML y crea todos los items agregandolos a las listas correspondientes
 void Constructor::on_load_from_xml()
 {
        on_edit_menu_delete_all();
@@ -242,6 +255,7 @@ void Constructor::on_load_from_xml()
        xmlDocPtr document;
        document = xmlParseFile(file_name.c_str());
        if (document == NULL) {
+               std::cout<<"NO SE PUDO CARGAR EL ARCHIVO"<<std::endl;
                // TODO : dar un aviso de que no se pudo abrir el archivo!!
                return;
        }
@@ -309,17 +323,21 @@ void Constructor::on_load_from_xml()
                        }
                        items = items->next;
                }
-
+               
+               // Crea las lineas que conectan la parte logica.
                create_lines(document->children);
                xmlFreeDoc(document);
                workplace->update_logic_position();
                workplace->queue_draw();
        } else {
+               std::cout<< "EL XML NO ES VALIDO" << std::endl;
                // TODO : avisar que el XML no es valido!!
        }
        std::cout << "CARGA COMPLETA" << std::endl;
        file_open_selection->hide();
        
+       // Seteo el id en el maximo mas uno de todos los id cargados, para que quede bien cuando se agregan nuevos items en un proyecto 
+       // cargado desde un archivo.
        int id_1 = -1 , id_2 = -1;
        std::list<CItem *>::iterator i = listaItems.begin();
        while ( i != listaItems.end() ){
@@ -340,6 +358,7 @@ void Constructor::on_load_from_xml()
        else id = id_1;
 }
 
+// Recorre todo los items los cuales conocen la manera de salvarse en un archivo XML.
 void Constructor::on_btn_file_ok_clicked()
 {
        std::list<CItem *>::iterator i = listaItems.begin();
@@ -365,6 +384,7 @@ void Constructor::on_btn_file_ok_clicked()
        file_selection->hide();
 }
 
+// Salva el archivo, si no estaba salvado muestra el dialogo, si no sobreescribe el ultimo archivo salvado.
 void Constructor::on_quick_btn_save_clicked()
 {
        if ( ! saved ) {
@@ -374,17 +394,20 @@ void Constructor::on_quick_btn_save_clicked()
        }
 }
        
+// Limpia la pantalla para comenzar un nuevo proyecto.
 void Constructor::on_quick_btn_new_clicked()
 {
        on_quick_btn_delete_all_clicked();
        saved = false;
 }
 
+// Borra todos los items pero desde otro boton.
 void Constructor::on_quick_btn_delete_all_clicked()
 {
        on_edit_menu_delete_all();
 }      
 
+// busca en la lista el nombre del item que aparece en el cuadro de texto y si lo encuentra dibuja un recuadro para mostrarlo.
 void Constructor::on_btn_find_clicked()
 {
        CItem * temp = NULL;
@@ -410,21 +433,25 @@ void Constructor::on_btn_find_clicked()
                                                                                                                                                temp->get_image()->get_width()+10, temp->get_image()->get_height()+10);
 }
 
+// Oculta el dialogo.
 void Constructor::on_btn_file_cancel_clicked()
 {
        file_selection->hide();
 }
 
+//Muestra el dialogo para seleccionar archivo
 void Constructor::on_btn_open_cancel_clicked()
 {
        file_open_selection->hide();
 }
 
+// Oculta el dialogo
 void Constructor::on_btn_dlg_connect_clicked()
 {
        dlg_connect->hide();
 }
 
+//Metodos que permiten que el icono sea el del items durante el drag 
 void Constructor::on_canio_drag_begin(const Glib::RefPtr<Gdk::DragContext>& context)
 {
        context->set_icon(ico_canio, 5, 5); 
@@ -485,6 +512,7 @@ void Constructor::on_item_drag_data_get(const Glib::RefPtr<Gdk::DragContext>& co
        gtk_selection_data_set (selection_data, selection_data->target, 10, (const guchar*)"item_codo.png", 13);
 }
 
+// Si un item es movido encima de otro este metodo devueve falso. De esta manera no podran encimarse items.
 bool Constructor::can_drop(CItem *item, int _x, int _y)
 {
        std::list<CItem*>::iterator  i = listaItems.begin();
@@ -503,6 +531,8 @@ bool Constructor::can_drop(CItem *item, int _x, int _y)
        return true;
 }
 
+//Cada vez que se realiza un drag o un drop este metodo es invocado.
+//Aca se validan las posiciones de los items y se crean si no estaban en el area de trabajo.
 void Constructor::on_item_drop_drag_received(const Glib::RefPtr<Gdk::DragContext>& context, int x, int y, GtkSelectionData* selection_data, guint info, guint time)
 {
        workplace->update_logic_position();
@@ -594,6 +624,9 @@ void Constructor::on_item_drop_drag_received(const Glib::RefPtr<Gdk::DragContext
        workplace->queue_draw();
 }
 
+// Cuando se presiona el boton "Verificar Conexiones" esta funcion es invocada
+//Muestra un dialogo de error en caso de que no esten bien conectados y un dialogo
+//de exito si lo estan.
 void Constructor::on_btn_check_clicked()
 {
        Glib::ustring name;
@@ -608,7 +641,8 @@ void Constructor::on_btn_check_clicked()
        }
 }
 
-
+//Verifica que todos los items esten correctamente conectados en sus extremos.
+//Cada item verifica su conexion en forma independiente.
 bool Constructor::check_connection(Glib::ustring& name)
 {
        std::list<CItem *>::iterator i = listaItems.begin();
@@ -641,6 +675,9 @@ bool Constructor::check_connection(Glib::ustring& name)
        return true;
 }      
 
+
+//Los siguientes metodos son utilizados para levantar cada item desde el XML
+
 Not *Constructor::loadNot(xmlNodePtr nodo)
 {
        std::string name = (char *)xmlGetProp(nodo, BAD_CAST"nombre");
@@ -981,6 +1018,7 @@ void Constructor::create_lines(xmlNodePtr nodo)
        }
 }
 
+//Levanta las lineas desde el archvo XML 
 void Constructor::create_line(xmlNodePtr nodo, int logic_id)
 {
        std::string otro;

diff --git a/Constructor/src/not.cpp b/Constructor/src/not.cpp
index 3fd6b7c817ce81eb84059b26b1d5498d85776b21..679f9dfaeef640bc5132212819da083fc6670779 100644 (file)
--- a/Constructor/src/not.cpp
+++ b/Constructor/src/not.cpp
@@ -173,7 +173,7 @@ bool Not::check_connection()
        t_logic_connector temp;
        std::list<t_line>::iterator i = workplace->lista_lineas_in.begin();
        while ( i != workplace->lista_lineas_in.end() ) {
-               if ( workplace->get_logic_item((*i).logic_id)->get_id() == ID ) {
+               if ( (*i).logic_id == ID ) {
                        temp.type = OUT;
                        temp.name_dest = workplace->get_item((*i).store_id)->get_name();
                        vec_connector.push_back(temp);
@@ -184,7 +184,7 @@ bool Not::check_connection()
 
        i = workplace->lista_lineas_out.begin();
        while ( i != workplace->lista_lineas_out.end() ) {
-               if (workplace->get_logic_item((*i).logic_id)->get_id() == ID ) {
+               if ( (*i).logic_id == ID ) {
                        temp.type = IN;
                        temp.name_dest = workplace->get_item((*i).store_id)->get_name();
                        vec_connector.push_back(temp);
@@ -192,7 +192,30 @@ bool Not::check_connection()
                }
                i++;
        }
-       return true;// (in_connected    == out_connected ) ==true;
+       
+       i = workplace->lista_lineas_logic.begin();
+       while ( i != workplace->lista_lineas_logic.end() ) {
+               if ( (*i).logic_id == ID ) {
+                       temp.type = OUT;
+                       temp.name_dest = workplace->get_logic_item((*i).store_id)->get_name();
+                       vec_connector.push_back(temp);
+                       cant_in++;
+               }
+               i++;
+       }
+
+       i = workplace->lista_lineas_logic.begin();
+       while ( i != workplace->lista_lineas_logic.end() ) {
+               if ( (*i).store_id == ID ) {
+                       temp.type = IN;
+                       temp.name_dest = workplace->get_logic_item((*i).logic_id)->get_name();
+                       vec_connector.push_back(temp);
+                       cant_out++;
+               }
+               i++;
+       }
+
+       return true;
 }
 
 ConnectorType Not::detect_click_position(int _a, int _b)

diff --git a/Constructor/src/or.cpp b/Constructor/src/or.cpp
index 7be809ed1febb7c96ebf70cfdd9d561e388c8ad1..ee96e2287ae7c249906a5eaa1a111e8e5feee23e 100644 (file)
--- a/Constructor/src/or.cpp
+++ b/Constructor/src/or.cpp
@@ -169,7 +169,7 @@ bool Or::check_connection()
        t_logic_connector temp;
        std::list<t_line>::iterator i = workplace->lista_lineas_in.begin();
        while ( i != workplace->lista_lineas_in.end() ) {
-               if ( workplace->get_logic_item((*i).logic_id)->get_id() == ID ) {
+               if ( (*i).logic_id== ID ) {
                        temp.type = OUT;
                        temp.name_dest = workplace->get_item((*i).store_id)->get_name();
                        vec_connector.push_back(temp);
@@ -180,7 +180,7 @@ bool Or::check_connection()
 
        i = workplace->lista_lineas_out.begin();
        while ( i != workplace->lista_lineas_out.end() ) {
-               if ( workplace->get_logic_item((*i).logic_id)->get_id() == ID ) {
+               if ( (*i).logic_id== ID ) {
                        temp.type = IN;
                        temp.name_dest = workplace->get_item((*i).store_id)->get_name();
                        vec_connector.push_back(temp);
@@ -189,8 +189,28 @@ bool Or::check_connection()
                i++;
        }
 
-       
-       return true;// out_connected;
+       i = workplace->lista_lineas_logic.begin();
+       while ( i != workplace->lista_lineas_logic.end() ) {
+               if ( (*i).logic_id == ID ) {
+                       temp.type = OUT;
+                       temp.name_dest = workplace->get_logic_item((*i).store_id)->get_name();
+                       vec_connector.push_back(temp);
+                       cant_in++;
+               }
+               i++;
+       }
+
+       i = workplace->lista_lineas_logic.begin();
+       while ( i != workplace->lista_lineas_logic.end() ) {
+               if ( (*i).store_id == ID ) {
+                       temp.type = IN;
+                       temp.name_dest = workplace->get_logic_item((*i).logic_id)->get_name();
+                       vec_connector.push_back(temp);
+                       cant_out++;
+               }
+               i++;
+       }
+       return true;
 }
 
 ConnectorType Or::detect_click_position(int _a, int _b)