[C++] Errore "Redefinition error"

[fausto]

Ciao a tutti.

Ho un programma strutturato in questo modo.

ho i seguenti file:

relaxGenLatency.h
relaxGenLatency.C
main.C

main.C e relaxGenLatency.C includono relaxGenLatency.h

Uso poi una piccolissima utility scritta da me che serve per gestire gli indici degli archi
di un grafo completo. I file sono indexes.h e indexes.C. Sono, stati compilati insieme ad altre utilities e poi archiviati in un file libGMLPutils

relaxGenLatency include indexes.h

Tutto funzionava.

Ho deciso poi di arricchire la libreria con una nuova funzione che dato l'indice dell'arco calcola i nodi sorgente e destinazione. Per fare questo ho definito uno struct arco. Ecco il codice di indexes.h

#ifndef MAT_INTEX
___#define MAT_INDEX

#include <cmath>
// ricorda, in c++ questa è definizione di tipo
// QUESTA È LA DEFINIZIONE CHE MI CAUSA PROBLEMI
struct arco{
___int s;
___int d;
};

int offset(int h, int k, int nnodes);
int matIndex (int i, int j, int nnodes);
void matNodes(int index, int nnodes, arco * a);
#endif

Se compilo, ottengo il seguente errore:
g++4 -O3 -DIL_STD -Wall -msse2 -mfpmath=sse -I. -I/home/cplex/10.1.1/include -I/home/cplex/10.1.1/include -Iinclude -I/home/errico/gmlputils/include -c source/relaxGenLatency.C
/home/errico/gmlputils/include/indexes.h:6: error: redefinition of 'struct arco'
/home/errico/gmlputils/include/indexes.h:6: error: previous definition of 'struct arco'
make: *** [relaxGenLatency.o] Error 1

Eppure, usando un main.C che ho usato per debuggare la ibreria, funzionava tutto perfettamente!
Dove sbaglio?

Grazie,
f

[XWolverineX]

Il compilatore ti dice che sta incontrando per 2 volte la definizione di una stessa struct.
Questo problema è dovuto al fatto che, includendo 2 volte un file .h (una volta nella classe, una volta nel file main), il compilatore trova 2 volte la struct.
Risolvi quindi cosi:

1) Semplicemente non includere il file .h nel tuo main.cpp. Poi aggiungi prima di tutto il programma una riga del tipo

struct nometuastruct;

ed il linker farà il resto

2) Aggingi nel file .h delle direttive preprocessore che impediscano il doppio include.

#ifndef TUOFILE_H
#define TUOFILE_H
//codice
#endif

e se hai visual c++, metti pure subito dopo il define un bel

#pragma once

[fausto]

Grazie per la risposta.

Il compilatore ti dice che sta incontrando per 2 volte la definizione di una stessa struct.
Questo problema è dovuto al fatto che, includendo 2 volte un file .h (una volta nella classe, una volta nel file main), il compilatore trova 2 volte la struct.
Risolvi quindi cosi:

1) Semplicemente non includere il file .h nel tuo main.cpp

Non è vero: come dicevo nel primo post, nel main non includo indexes.h

Le direttive #ifndef #define #endif sono già inserite in tutti i punto h.

2) Aggingi nel file .h delle direttive preprocessore che impediscano il doppio include.

Anche questo lo facevo già.

Poi aggiungi prima di tutto il programma una riga del tipo
codice:

struct nometuastruct;

Che scopo ha? Comunque ho provato a metterlo nel main (era li' che intendevi, no?)
ed ottengo comunque l'errore di compilazione.

Infine, non uso visual c++

[oregon]

A questo punto potrebbe anche esistere, da qualche altra parte, una struttura che si chiama arco ...

Fai la prova a chiamare la struttura

arco2

e ricompilare ...

[fausto]

gia' fatto:
infatti prima lo chiamavo "arc"...

[oregon]

A questo punto non sarebbe da escludere un bug del compilatore ...

Potresti provare a compilare con un altro compilatore (ma non mi dire che l'hai gia' fatto e non ce lo hai detto ... )

[XWolverineX]

Sarei curioso, a questo punto, di vedere il codice completo

[shodan]

Originariamente inviato da fausto

#ifndef MAT_INTEX
___#define MAT_INDEX

Il compilatore ha ragione. Le macro sono diverse.

[oregon]

Gia' ... non me ne ero accorto ... (e non solo io) ...

#ifndef MAT_INTEX
___#define MAT_INDEX

[XWolverineX]

Urgh è vero.