Błąd przy przekazaniu parametrów do funkcji

Question

||=== Build: Release in project1(compiler: GNU GCC Compiler) ===|
D:\C++\project1\main.cpp||In function 'void zapiszUstawienia()':|
D:\C++\project1\main.cpp|56|error: 'zamek1' was not declared in this scope|
D:\C++\project1\main.cpp|56|error: 'nzwa' was not declared in this scope|
D:\C++\project1\main.cpp|57|error: 'wlasc' was not declared in this scope|
D:\C++\project1d\main.cpp|58|error: 'procentProdkcj1' was not declared in this scope|
D:\C++\project1\main.cpp|58|error: 'prcn' was not declared in this scope|
D:\C++\project1\main.cpp|59|error: 'ludnosc1' was not declared in this scope|
D:\C++\project1\main.cpp|59|error: 'ldnsl' was not declared in this scope|
D:\C++\project1\main.cpp|60|error: 'nastroj1' was not declared in this scope|
D:\C++\project1main.cpp|60|error: 'nastr' was not declared in this scope|
||=== Build failed: 9 error(s), 0 warning(s) (0 minute(s), 0 second(s)) ===|

Witam.

W programie utworzyłem kilka class przechowujących pewne parametry. W tym jeden class'a ma 2 funkcje pobierającą nazwe i nick. Każda klasa ma konstruktor. 

Mój problem polega na tym, iż chciałbym stworzyć funkcję, która te dane (parametry z class) zapisze do pliku.

W tym celu stworzyłem voida w głównym pliku programu (main.cpp), pod include zaznaczyłem iż taka funkcja istnieje, a pod funkcją główną (int main) stworzyłem jej ciało.. (funkcję stworzyłem oczywiście za

return i "}"  ). 

void zapiszUstawienia(string nzwa, string wlascl, int prcnt, int ludns, float nstrj);

W funkcji zapisującej, którą nazwałem "zapiszUstawienia" utworzyłem zmienną fstream (bibliotekę też dodałem). I teraz nie wiem jak przekazać tej funkcji zapisującej ustawienia paramerty moich obiektów:

void zapiszUstawienia()
{
    fstream otwarty_plik1;
    otwarty_plik1.open("ustawienia.txt", ios::out);
    if(otwarty_plik1.good()==false) cout<<"Nie udalo sie uruchomic pliku! ";

    zamek1.nazwa = nzwa;
    zamek1.wlasciciel = wlasc;
    procentProdkcj1.procent_prodkcj = prcn;
    ludnosc1.ludnosc = ldnsl;
    nastroj1.nastroj = nastr;

    otwarty_plik1<<nzwa;
    otwarty_plik1<<wlasc;
    otwarty_plik1<<prcn;
    otwarty_plik1<<ldnsl;
    otwarty_plik1<<nastr;

    otwarty_plik1.close();
}

Funkcji przekazałem także parametry, które chciałbym w niej zapisać:

zapiszUstawienia(zamek1.nazwa, zamek1.wlasciciel, procentProdkcj1.procent_prodkcj, ludnosc1.ludnosc, nastroj1.nastroj);

Obiekty także zostały stworzone. Dałem także funkcji "zapiszUstawienia" zaprzyjaźnienie z wszystkimi klasami. 

Przykładowa klasa w pliku nagłówkowym:

class Zamek
{
public:
    string nazwa;
    string wlasciciel;

    friend void zapiszUstawienia();
    Zamek(string="Zamek nr 1", string="Krol I");

    void podajNazwe();
    void podajWlasciciela();
};

A to błędy, które wywalił kompilator. 

Ma ktoś jakiś pomysł jak to rozwiązać?

 

 

Answer 1

Jak dla mnie te błędy są wynikiem nie zadeklarowania tychże zmiennych przynajmniej ja tu nigdzie deklaracji nie widzę.

EDIT: Brak argumentów w funkcji zapiszUstawienia

Comments

Deklarowałem zmienne i nic. Te same błędy.

Oczywiście jeśli mówimy o tych zmiennych:

string nzwa; string wlascl; int prcnt; int ludns; float nstrj;

 

---

Tak mówimy o tych ale chodzi mi o to, że w definicji funkcji nie ma nic o tych zmiennych, a chcesz do nich coś przypisać. I przy tej definicji musisz mieć globalne obiekty Zamek1 ProcentProdukcji1 ludnosc1 i nastroj1

---

Ta definicja:

void zapiszUstawienia(string nzwa, string wlascl, int prcnt, int ludns, float nstrj)
{
    fstream otwarty_plik1;
    otwarty_plik1.open("ustawienia.txt", ios::out);
    if(otwarty_plik1.good()==false) cout<<"Nie udalo sie uruchomic pliku! ";
 
    zamek1.nazwa = nzwa;
    zamek1.wlasciciel = wlasc;
    procentProdkcj1.procent_prodkcj = prcn;
    ludnosc1.ludnosc = ldnsl;
    nastroj1.nastroj = nastr;
 
    otwarty_plik1<<nzwa;
    otwarty_plik1<<wlasc;
    otwarty_plik1<<prcn;
    otwarty_plik1<<ldnsl;
    otwarty_plik1<<nastr;
 
    otwarty_plik1.close();
}

Jak jest funkcją zaprzyjaźnioną to nie musi być metodą klasy, ale jak już pisałem obiekty musiałyby być globalne, albo również przekazane do funkcji. I w deklaracji przyjaźni też brakuje argumentów, bo zaprzyjaźniasz jakąś przeładowaną wersję tej funkcji a nie tą, której używasz.

---

Czy pełen zapis powinien wyglądać tak?

friend void zapiszUstawienia(string nzwa, string wlascl, int prcnt, int ludns, float nstrj);

 

---

Nadal te same błędy nawet jak z deklaracji przyjaźni dodałem argumenty.

---

Sprawdź czy wszędzie gdzie masz tą funkcję są podane argumeny i czy obiekty są globalne.

Answer 2

A gdzie argumenty w definicji metody?

Comments

W której metodzie? "zapiszUstawienia"?

"zapiszUstawienia" jest funkcją stworzoną w główym pliku, nie jest obiektem, a nie została zdefinionawna przez klasę.

---

Powinno to tak wyglądać:

class foo
{
//...
	void zapiszUstawienia(string nzwa, string wlascl, int prcnt, int ludns, float nstrj)
	{
   		fstream otwarty_plik1;
    		otwarty_plik1.open("ustawienia.txt", ios::out);
    		if(otwarty_plik1.good()==false) cout<<"Nie udalo sie uruchomic pliku! ";
 
    		zamek1.nazwa = nzwa;
    		zamek1.wlasciciel = wlasc;
    		procentProdkcj1.procent_prodkcj = prcn;
    		ludnosc1.ludnosc = ldnsl;
    		nastroj1.nastroj = nastr;
 
    		otwarty_plik1<<nzwa;
    		otwarty_plik1<<wlasc;
    		otwarty_plik1<<prcn;
    		otwarty_plik1<<ldnsl;
    		otwarty_plik1<<nastr;
 
    		otwarty_plik1.close();
	}
};

EDIT: Nie zauważyłem, że jest to zwykła funkcja. To tylko usuń tę klasę foo z kodu 

---

A te obiekty zrób globalne, czy coś... mają być po prostu widoczne w funkcji.

---

Niestety nadal te same błędy.

 

EDIT: Jak zrobić elementy globalne? W sensie dać public?

---

Elementy globalne deklarujesz przed mainem

---

Nie znam deklaracji twojej klasy, więc nie wiem, czy te składowe są publiczne, czy nie, ale jeśli nie są, to robi się to wpisując deklaracje pól i metod po operatorze public:

class foo
{
public:
//...
};

Ale tutaj jest także problem, że funkcja zapiszUstawienia nie widzi obiektów tej twojej klasy, więc musisz je zrobić globalne, lub stworzyć je w funkcji, lub wysłać argumentem... masz szeroki wybór.

---

No to tylko został problem widoczności obiektów tych twoich klas w funkcji zapiszUstawienia (patrz wyżej).

---

Mam pytanie:

w jaki sposób mogę uczynić obiekty, obiektami globalnymi?

W internecie znalazłem coś o polu statycznym.

---

Obiekty globalne deklarujesz przed funkcją main()

---

Jeśli pole klasy ma związek z nią, a nie z każdym obiektem (jakby zmienna/stała klasowa), to możesz ją ustawić na pole statyczne. Jeśli tak nie jest, to radzę wysyłać obiekty argumentami.

Answer 3

Dzięki wielkie za pomoc. 

Tak jak poradził @Patrycjerz  zdeklarowałem obiekty globalnie i wszystko działa, zero bugów.

Jeszcze raz dzięki za pomoc.