[c] stranissimo problema con #ifndef

[theory]

ho un problema assurdo: in un header (e sottolineo solo in uno su tutti quelli che compongono il programma) se faccio così

#ifndef _HEADER_
#define _HEADER_


#endif

il compilatore (gcc sotto cygwin) mi dice che ci sono delle definizioni multiple dei dati. non riuscendo a venire a capo al problema, ho provato a fare

#ifdef _HEADER_
#define _HEADER_


#endif

e la compilazione è andata a buon fine
in tutti gli altri header utilizzo ifndef e va tutto come deve andare, solo su questo ho questo stranissimo problema..

[sottovento]

Hai provato anche ad usare un nome diverso da _HEADER_?

[theory]

Originariamente inviato da sottovento

Hai provato anche ad usare un nome diverso da _HEADER_?

si, _HEADER_ l'ho messo solo lì sopra come esempio.
comunque ho provato anche a compilare sotto gnu\linux e fa uguale

[Albi89]

A parte che

#define _HEADER_

non ha senso in quel caso perchè è già definito, non ci hai dato abbastanza informazioni.

Dovresti perlomeno incollare il segmento di codice e l'errore del compilatore, eventualmente tutto il codice se non è troppo lungo/troppo personale.

Posta così vediamo un po'

[theory]

Originariamente inviato da Albi89

A parte che

#define _HEADER_

non ha senso in quel caso perchè è già definito, non ci hai dato abbastanza informazioni.

Dovresti perlomeno incollare il segmento di codice e l'errore del compilatore, eventualmente tutto il codice se non è troppo lungo/troppo personale.

Posta così vediamo un po'

header incriminato:

#ifndef _COUNT_WORDS_
#define _COUNT_WORDS_

char* ultimo_blocco_words=NULL;

char* StaccaBlocco(char*  blocco, int lunghezza);
char* AttaccaBlocco(char* blocco, int lunghezza);

#endif

altro header, funzionante con #ifndef:

#ifndef _UTILS_
#define	_UTILS_

char Complemento_A_2(char input);
int Controllo_Segno(char input);
int ControlloEstensione(char* input);

#endif

errore:

count_words.o: In function `StaccaBlocco':
/cygdrive/c/progetto/count_words.c:8: multiple definition of `_ultimo_blocco_words'
main.o:/cygdrive/c/progetto/main.c:13: first defined here
collect2: ld returned 1 exit status
make: *** [LabProg] Error 1

nel main ho incluso l'header incriminato.

edit: il problema sembra essere "char* ultimo_blocco_words=NULL;" visto che togliendolo va tutto, ma non capisco perchè visto che la seconda volta dovrebbe saltare tutto il blocco a causa dell'ifndef

[wizard1993]

Originariamente inviato da Antonio23

forse ti può essere utile vedere l'uso della direttiva #pragma once

non la supportava solo il compilatore micorsoft?

[Albi89]

Il problema è che così facendo definisci una variabile in 2 file
Devi rimuoverla dall'header e definirla solo nel file che ne farà effettivamente uso, e usare eventualmente la keyword extern.
Nell'header puoi inserire solo dichiarazioni

In ogni caso #pragma once dovrebbe andare anche con gcc.

[theory]

Originariamente inviato da Albi89

Il problema è che così facendo definisci una variabile in 2 file
Devi rimuoverla dall'header e definirla solo nel file che ne farà effettivamente uso, e usare eventualmente la keyword extern.
Nell'header puoi inserire solo dichiarazioni

In ogni caso #pragma once dovrebbe andare anche con gcc.

intanto ti ringrazio ma ho un paio di domande, perchè la situazione non mi è molto chiara :
-ma come mai con ifdef funzionava? e perchè anche la seconda volta entrava nel controllo, ridefinendola un'altra volta?
-ma se gli header si utilizzano tramite la direttiva #include, che non fa altro che sostituire il contenuto dell'header nel punto in cui è stata utilizzata, perchè c'è questo "limite" di poter inserire solo dichiarazioni?

vi ringrazio in anticipo per le risposte

[sottovento]

Originariamente inviato da theory

header incriminato:

#ifndef _COUNT_WORDS_
#define _COUNT_WORDS_

char* ultimo_blocco_words=NULL;

char* StaccaBlocco(char*  blocco, int lunghezza);
char* AttaccaBlocco(char* blocco, int lunghezza);

#endif

altro header, funzionante con #ifndef:

#ifndef _UTILS_
#define	_UTILS_

char Complemento_A_2(char input);
int Controllo_Segno(char input);
int ControlloEstensione(char* input);

#endif

errore:

count_words.o: In function `StaccaBlocco':
/cygdrive/c/progetto/count_words.c:8: multiple definition of `_ultimo_blocco_words'
main.o:/cygdrive/c/progetto/main.c:13: first defined here
collect2: ld returned 1 exit status
make: *** [LabProg] Error 1

nel main ho incluso l'header incriminato.

edit: il problema sembra essere "char* ultimo_blocco_words=NULL;" visto che togliendolo va tutto, ma non capisco perchè visto che la seconda volta dovrebbe saltare tutto il blocco a causa dell'ifndef

Eh no! Nel primo caso vai davvero a definire due volte la stessa variabile globale, in due file diverse e quindi il compilatore ha tutte le ragioni di arrabbiarsi.
I due casi che hai presentato sono totalmente diversi: nel primo vai a definire una variabile globale (chiamata ultimo_blocco_words), nel caso in cui ti funziona tutto vai semplicemente a definire dei prototipi di funzione (fra l'altro non ti darebbero problemi nemmeno se togli le direttive di compilazione).

La variabile ultimo_blocco_words la devi definire in un unico punto.
Quello che normalmente viene fatto e' mettere un extern nel file di intestazione:

#ifndef _COUNT_WORDS_
#define _COUNT_WORDS_

extern char* ultimo_blocco_words;

char* StaccaBlocco(char*  blocco, int lunghezza);
char* AttaccaBlocco(char* blocco, int lunghezza);

#endif

ed in un file .c o .cpp (tipicamente quello contenente il main, oppure un file chiamato global.cpp o come meglio preferisci) definisci

char* ultimo_blocco_words=NULL;

Ci sono altre soluzioni, ovviamente...
Comunque valuta sempre se puoi evitare l'uso di variabili globali

[DanieleC88]

Originariamente inviato da Albi89

In ogni caso #pragma once dovrebbe andare anche con gcc.

Potrebbe essere supportato, ma non l'ho provato. In ogni caso, non è standard, quindi si fa bene ad evitarlo quando possibile.

Originariamente inviato da sottovento

Comunque valuta sempre se puoi evitare l'uso di variabili globali

Quotissimo.

In ogni caso, nella seconda variante con #ifdef invece di #ifndef mi sa che non hai definito un bel niente, quando invece nella prima versione l'hai definita due volte in due file diversi (e in fase di linking GCC ovviamente s'incazza). Prova a toglierla di mezzo: se deve essere privata definiscila static e solo in un file, al limite.