[C++] Errore su definizione metodo classe

[cipiupiu]

Mi chiedevo se qualcuno mi potesse aiutare a capire cosa c'è di sbagliato:

         bool ContenitoreItinerari :: Confronta (Itinerario itinerario1, Itinerario itinerario2);     {         for (vector<Luogo>::iterator it1 = itinerario1.luoghi.begin();              it1 != itinerario1.luoghi.end(); ++it1)         {             Luogo luogo1 = *it1;             for (vector<Luogo>::iterator it2 = itinerario2.luoghi.begin();                  it2 != itinerario2.luoghi.end(); ++it2)             {                 Luogo luogo2 = *it2;                 if ((luogo1.latitudine == luogo2.latitudine) &&                     (luogo1.longitudine == luogo2.longitudine))                     return true;             }         }                  return false;     }

Mi continua a dare questo errore:

Definiton or redeclaration of "Confronta" not allowed inside a function.

Grazie in anticipo per l'aiuto...

[oregon]

Se posti tutto il codice in una linea, non si capisce nulla.

[cipiupiu]

    bool ContenitoreItinerari :: Confronta (Itinerario itinerario1, Itinerario itinerario2);
    {
        for (vector<Luogo>::iterator it1 = itinerario1.luoghi.begin();
             it1 != itinerario1.luoghi.end(); ++it1)
        {
            Luogo luogo1 = *it1;
            for (vector<Luogo>::iterator it2 = itinerario2.luoghi.begin();
                 it2 != itinerario2.luoghi.end(); ++it2)
            {
                Luogo luogo2 = *it2;
                if ((luogo1.latitudine == luogo2.latitudine) &&
                    (luogo1.longitudine == luogo2.longitudine))
                    return true;
            }
        }
        
        return false;
    }

[oregon]

Mi sa che il problema non sta nel codice mostrato ma su dove l'hai inserito nel progetto globale.

In pratica bisogna vedere *tutto* il codice ...


P.S. Che c'entra il titolo "bool" con il problema che hai ?

[Scara95]

Definiton or redeclaration of "Confronta" not allowed inside a function.

Stai evidentemente definendo una funzione all'interno di un'altra funzione

[cipiupiu]

per non scrivere tutto il metodo...comunque no, è sbagliato qualcosa nella sintassi del metodo

[Scara95]

Per trovare l'errore dovresti scrivere più di tutto il metodo!

[oregon]

Originariamente inviato da cipiupiu
per non scrivere tutto il metodo...comunque no, è sbagliato qualcosa nella sintassi del metodo

Se lo dici tu ...

(a che serve scrivere in un forum se non si seguono i consigli che si ricevono?)

[cipiupiu]

#include <iostream.h>
#include "Luoghi.h"
#include <string.h>
#include <vector.h>

using namespace std;



class Itinerario
{
public:
    string nome;
    vector<Luogo> luoghi;
    
    Itinerario(string nome)
    {
        this->nome = nome;
    }
    
    void Add(double latitudine, double longitudine)
    {
        luoghi.push_back(Luogo(latitudine, longitudine));
    }
};




class ContenitoreItinerari
{
public:
    vector<Itinerario> itinerari;
};



class TestItinerario
{
public:
    int TestItinerario1();
    int TestItinerario2();
    int TestItinerario3();
    int TestItinerario4();
    
    void Esegui ()
    {
        cout << "Inizio Test" << endl;
        
        TestItinerario1();
        TestItinerario2();
        TestItinerario3();
        
        cout << "Fine test" << endl;
    };
    
};
   
    
    

int main( )
{

    Itinerario itinerario1("Primo percorso");
    itinerario1.Add(100, 200);
    itinerario1.Add(230, 500);
    itinerario1.Add(20, 300);;
    
    Itinerario itinerario2("Secondo percorso");
    itinerario2.Add(100, 200);
    itinerario2.Add(230, 500);
    itinerario2.Add(20, 300);
 
    Itinerario itinerario3("Terzo percorso");
    itinerario3.Add(100, 200);
    itinerario3.Add(230, 500);
    itinerario3.Add(20, 300);
    
    Itinerario itinerario4("Quarto percorso");
    itinerario4.Add(100, 200);
    itinerario4.Add(230, 500);
    itinerario4.Add(20, 300);
    
   
    
    ContenitoreItinerari contenitore;
    contenitore.itinerari.push_back(itinerario1);
    contenitore.itinerari.push_back(itinerario2);
    contenitore.itinerari.push_back(itinerario3);
    contenitore.itinerari.push_back(itinerario4);
    
   
        bool ContenitoreItinerari :: Confronta (Itinerario itinerario1, Itinerario itinerario2);
    {
        for (vector<Luogo>::iterator it1 = itinerario1.luoghi.begin();
             it1 != itinerario1.luoghi.end(); ++it1)
        {
            Luogo luogo1 = *it1;
            for (vector<Luogo>::iterator it2 = itinerario2.luoghi.begin();
                 it2 != itinerario2.luoghi.end(); ++it2)
            {
                Luogo luogo2 = *it2;
                if ((luogo1.latitudine == luogo2.latitudine) &&
                    (luogo1.longitudine == luogo2.longitudine))
                    return true;
            }
        }
        
        return false;
    }    

    cout << "Percorsi differenti:" << endl;
    vector<Itinerario> differenti;
    for (vector<Itinerario>::iterator it1 = contenitore.itinerari.begin(); it1 != contenitore.itinerari.end(); ++it1)
    {
        for (vector<Itinerario>::iterator it2 = contenitore.itinerari.begin(); it2 != contenitore.itinerari.end(); ++it2)
        {
            if ((it1 != it2) && (contenitore.Confronta(*it1, *it2) == false))
            {
                differenti.push_back(*it1);
                cout << it1->nome << endl;
            }
        }
    }
}

[oregon]

Per il codice usa i tag CODE, come da regolamento, altrimenti non si capisce molto.

Comunque, il metodo Confronta inizia nel mezzo del main (ecco l'errore) ..

E in ogni caso, nella classe ContenitoreItinerari non esiste un membro Confronta ...