zmienne stałe nie działające w nagłówku

extern oznacza to, że 'prawdziwa' definicja zmiennej znajduje się gdzie indziej, a kompilator powinien ufać, że rzeczy będą się łączyć w czasie połączenia. Posiadanie definicji w jednej linii z extern jest dziwne i jest tym, co zmaga się z Twoim programem. Jeśli chcesz, aby były extern, po prostu zdefiniuj je dokładnie raz gdzie indziej w swoim programie.

Klasa przechowywania dla nich jest prawie na pewno przyczyną problemu, który widzisz. Jeśli go usuniesz, kod prawdopodobnie będzie w porządku (przynajmniej pod tym względem).

Edit: właśnie zauważyłem, że otagowałeś to jako C i C++. Pod tym względem C i C++ są naprawdę bardzo różne (ale z komunikatów o błędach, najwyraźniej kompilujesz jako C++, a nie C). W C++, chcesz usunąć extern, ponieważ (domyślnie) zmienne const mają klasę przechowywania static. To znaczy każdy plik źródłowy (Jednostka tłumaczenia) otrzyma własną "kopię" zmiennej i nie będzie konfliktu między definicjami w różnych plikach. Ponieważ (prawdopodobnie) używasz tylko wartości, nie traktując ich jako zmiennych, posiadanie wielu "kopii" niczego nie zaszkodzi-żadna z nich nie zostanie przydzielona przestrzeń dyskowa.

W C, extern jest raczej inne, a usunięcie extern nie zrobi żadnej rzeczywistej różnicy, ponieważ będą one domyślnie extern. W takim przypadku naprawdę musisz zainicjalizuj zmienne dokładnie w jednym miejscu i zadeklaruj je w nagłówku. Alternatywnie, możesz dodać static klasę pamięci, którą C++ będzie dodawać domyślnie, gdy / jeśli usuniesz extern z nagłówka.

Wygląda na to, że plik nagłówka jest dołączany wiele razy. Musisz dodać strażników.

Na górze KAŻDEGO pliku nagłówkowego powinieneś mieć coś w stylu:

#ifndef MY_HEADER_FILE_NAME_H
#define MY_HEADER_FILE_NAME_H

...

// at end of file
#endif

Jeśli używasz g++ lub MSVC, możesz po prostu dodać:

#pragma once

Na górze KAŻDEGO pliku nagłówkowego, ale nie jest on w 100% przenośny.

Ponadto nie należy definiować stałych w plikach nagłówkowych, tylko deklarować je:

// In header file
extern const int my_const;


// In one source file
const int my_const = 123;

Problem polega na tym, że inicjalizujesz zmienne w pliku nagłówkowym ; tworzy to deklarację definiującą , która jest powtarzana w każdym pliku zawierającym ten nagłówek,stąd błąd wielokrotnej definicji.

Chcesz nie -definiującą deklarację (bez inicjalizacji) w pliku nagłówkowym i umieść definiującą deklarację w jednym z plików implementacji.

Wiele błędnych odpowiedzi poniżej. Te, które są poprawne, to te, które każą Ci usunąć extern, jak powiedział sellibitze w swoim komentarzu, są poprawne.

Ponieważ są one zadeklarowane const, nie ma problemu z definicją w nagłówku. C++ będzie wstawiać const dla typu wbudowanego, chyba że spróbujesz pobrać jego adres (wskaźnik do const), w którym to przypadku utworzy on instancję za pomocą połączenia static, możesz wtedy również uzyskać wiele instancji w oddzielnych Moduły, ale jeśli nie oczekujesz, że wszystkie wskaźniki do tego samego const będą miały ten sam adres, nie jest to problemem.

Musisz zadeklarować kontanty w nagłówku, a następnie zdefiniować je w jednym z plików kodu. Jeśli nie zadeklarujesz ich nigdzie, wtedy wystąpi błąd linkera, gdy próbuje powiązać deklarację z rzeczywistą definicją. Możesz także użyć poleceń # ifdef, aby mieć jedną definicję w nagłówku.

Upewnij się, że są zadeklarowane w nagłówku, który jest dołączany przez wszystkich, którzy ich potrzebują i upewnij się, że są zdefiniowane dokładnie raz.

Jakub

Jeśli chcesz zdefiniować stałe w plikach nagłówkowych, użyj static const. Jeśli użyjesz extern, linker ma prawo narzekać na wiele definicji, ponieważ każdy plik źródłowy, w tym plik źródłowy, dostarczy pamięci dla zmiennej, jeśli przypiszesz wartość.

W deklarowaniu global const w nagłówku powoduje, że każda jednostka kompilacji zawierająca ten hader będzie miała własne definicje globalne definicje o tej samej nazwie. Więc linker tego nie lubi.

Jeśli naprawdę potrzebujesz ich w nagłówku, prawdopodobnie powinieneś zadeklarować je jako statyczne.

Stare pytanie, rzeczywiście, ale brakuje jednej użytecznej odpowiedzi.

Możliwe jest nakłonienie MSVC do zaakceptowania stałych statycznych w nagłówkach, po prostu zawijając je w "atrapowy" szablon klasy:

template <typename Dummy = int>
struct C {
     static const double Pi;
};

template <typename Dummy = int>
const double C<Dummy>::Pi = 3.14159;

Teraz, C:: PI można uzyskać dostęp z innego miejsca. Żadna redefinicja nie narzeka; stała jest bezpośrednio dostępna w każdej jednostce kompilacji bez wymyślnej optymalizacji czasu łącza. Makro może być rozwijane w celu dalszego ulepszenia tego podejścia (nawet jeśli makra są złe).