[C++]Uporaba strukture v drugi .cpp datoteki

Pozdravljeni,
imam izjemne težave z uporabo struct-a v dveh cpp datotekah.
Deklariran in definran je v datoteki "funkcije.h":

struct data{
	int integer = 0;
	Mat & picture = Mat(1280, 960, CV_8UC3);
}podatki;

Ustvari se tudi objekt "podatki".
strukturo uporabim v eni funkciji v datoteki "funkcije.cpp":

void funkcija(data a){
	doSomething(a.picture);
}

ki jo kličem v "main.cpp":

funkcija(podatki)

kakšna ideja?
Hvala

Compiler spusti čez, ustavi se pri linkerju, in sicer:

Error	17	error LNK2005: "struct data podatki" (?podatki@@3Udata@@A) already defined in funkcije.obj	
Error	18	error LNK1169: one or more multiply defined symbols found	

tudi po tem ko dodam ifdef se pritoži.
Zelo čudno

galu je 5. dec 2014 ob 13:52 izjavil:

#ifndef

mora biti

Seveda, #ifndef. Ampak zanimivo, da dobi enako napako. Moral bi dobiti "undeclared symbol data" ali kaj podobnega.

gtu je 6. dec 2014 ob 09:36 izjavil:

Je mogoče težava v tem, da se kreira objekt že takoj pri definiciji structa in potem ta isti objekt uporabljam v drugem cppju?

Da. Težava je v tem, da je globalna spremenljivka podatki definirana v dveh objektnih datotekah, ki jih linker ne zna povezati skupaj. Ena rešitev je, da v header datoteki uporabiš besedo extern, s čemer prevajalniku eksplicitno poveš, da je spremenljivka samo deklarirana, definirana pa je nekje drugje. Nato samo v eni od cpp datotek definiraš spremenljivko podatki.

Npr.: v headerju
struct data {};
extern struct data podatki;

v eni cpp datoteki:
struct data podatki;

Druga možnost je, da v header datoteki definiraš podatki kot statično spremenljivko

struct data {};
static struct data podatki;

V tem primeru bo spremenljivka podatki lokalna vsaki prevajani enoti (datoteki).

To kar ti delaš je zlo khm...
Ti imaš struct data in imaš instanco podatki. Vsakič, ko narediš include tega headerja compilerju tudi poveš, da hočeš naredit novo globalno instanco spremenljivke z imenom "podatki", ki že obstaja.
Skratka v glavi si moraš ločit, da struct in instanca nisto eno in isto.
Plus tvoja inicializacija memberjev v structu/class-u ni ravno standardna (kater compiler ti to pusti?).

Vse kar se vključuje (include) drugam se interpretira še enkrat. Zato tudi #ifndef/#define mehanizem omenjen zgoraj.

Compiler se ti je pritožil ker v strukturi shranjuješ referenco nekam (member picture) in ta mora vedno kazat nekam. Pravilno to narediš tako, da te stvari nastaviš v konstruktorju.

gtu je 10. dec 2014 ob 07:51 izjavil:

Vseeno mi pa še ni čisto jasno, zakaj sta bila prej definirana dva objekta? Se to zgodi zaradi #include headerja v katerem je bil struct?

Da. V includanem headerju je bila definicija globalne spremenljivke/instance podatki. Include ni nič drugega, kot "skopiraj vsebino includane datoteke sem." Nekateri programski paketi to precej prostodušno in IMHO nemarno izrabljajo. Omenjeni #ifndef/#define konstrukt pa ščiti proti temu, da bi nek header file includal dvakrat (ali pa npr. da bi ciklično includal dva headerja). Pri tvojem problemu nima nekega pomena, je pa dobra praksa pri pisanju header datotek.

gtu je 10. dec 2014 ob 07:51 izjavil:

Ali moram, če želim isti struct uporabiti še v tretji cpp datoteki, v njen header dati definicijo structa, ali samo includam prejšnji header?

Samo includaš header. Ne smeš podati definicije. Sicer boš naletel na isti problem.

gtu je 10. dec 2014 ob 07:51 izjavil:

Kako je v tem primeru z objektom podatki? Ali je dostopen funkcija tretje cpp datoteke?
_{(Potihem razmišljam, da je dovolj samo include prejšnjega headerja, kjer je že definicija structa)}

Če si uporabil extern rešitev, potem je dostopen tudi funkcijam v tretjem cpp. Če si uporabil static, potem ne. Čeprav kolikor sem videl, ti itak podajaš "podatki" kot argument funkciji, tako da v primeru, ki si ga zapisal nima pomena ustvarjati globalne spremenljivke/instance.

Na ostale napake te bo že kdo drug opozoril. Jaz sem samo odgovoril za omenjeno napako, ker sem videl, da so se ostali komentatorji malo lovili.

Senitel je 10. dec 2014 ob 08:59 izjavil:

To kar ti delaš je zlo khm...
Plus tvoja inicializacija memberjev v structu/class-u ni ravno standardna (kater compiler ti to pusti?).

In-class inicializacija je standardna od C++11 naprej. Tako da...

struct data {
    int i = 1;
};

... je v redu in je enako kot:

struct data {
    int i;
    data() : i(1) {}
};

Inicializacija reference pa ni v redu, ker je ne moreš inicializirat z začasnim objektom (ni const).