Problem z listą jednokierunkową

Question

Witam, jestem właśnie w trakcie tworzenia listy jednokierunkowej na klasach. Narazie są tylko metody dodaj do listy i wypisz listę, ale dodam jeszcze resztę . Kompilator ma problem z deklaracja wskaźnika pierwszy i nie wiem w czym problem, przy podobnej deklaracji w osoba.h, nic nie wyrzuca. Z góry dzięki za pomoc.

Plik nagłówkowy Osoba.h

#ifndef OsobaH
#define OsobaH

#include <iostream>
#include <string>
#include <cstdlib>

using namespace std;
class osoba
{
   public:
   string imie;
   string nazwisko;
   int wiek;
   osoba *nastepny;
};

#endif

Plik nagłówkowy Lista.h

#ifndef ListaH
#define ListaH

#include <iostream>
#include <string>
#include <cstdlib>

using namespace std;

class lista
{
   public:
   lista();
   void dodaj_osobe(string imie, string nazwisko, int wiek);
   void usun_osobe(int ktora);
   void wyswietl_liste();
   private:
   osoba *pierwszy;
};

#endif

Plik Lista.cpp

#pragma hdrstop

#include "Lista.h"
#include "Osoba.h"

#include <iostream>
#include <string>
#include <cstdlib>
using namespace std;

#pragma package(smart_init)

lista::lista()
{
   pierwszy=0;
}
void lista::dodaj_osobe(string imie, string nazwisko, int wiek)
{
   osoba *nowa= new osoba;
   nowa->imie=imie;
   nowa->nazwisko=nazwisko;
   nowa->wiek=wiek;

   if(pierwszy==0)
   {
      pierwszy = nowa;
      pierwszy->nastepny=0;
   }
   else
   {
      osoba *pomoc =pierwszy;

      while(pomoc->nastepny)
      {
         pomoc=pomoc->nastepny;
      }
      pomoc->nastepny=nowy;
      nowy->nastepny=0;
   }

}
void lista::wyswietl_liste();
{
   if(pierwszy==0)
      cout<<"Brak elementow";
   else
   {
      int i =1
      osoba *pomoc= pierwszy;
      while(pomoc->nastepny)
      {
         cout<<i<<". "<<pomoc->imie<<" "<<pomoc->nazwisko<<" "<<pomoc->wiek<<endl;
         pomoc=pomoc->nastepny;
         i++;
      }

}

I tutaj błędy odnośnie wskaźnika w lista.h

[C++ Error] Lista.h(23): E2303 Type name expected
[C++ Error] Lista.h(23): E2139 Declaration missing ;​

Answer 1

Usuń #include "Osoba.h" z pliku Lista.cpp i wrzuc go do pliku Lista.h
Możesz też przed deklaracją klasy Lista dodać następującą linijkę:
class osoba;
Zrób jedno z powyższych, nie obyda naraz.

Comments

Zadziałało dołożenie class osoba do Lista.h, później powychodziło już tylko kilka błędów pokroju braku średnika lub złej nazwy wskaznika, ale już ponaprawiane i wszystko działa. Wielkie dzięki.

Answer 2

Jeśli deklaracje masz w osobnych plikach to kompilator nie wie, że istnieje takie coś jak klasa osoba i nie może utwożyć wskaźnika do obiektu czegoś co nie wiadomo czy istnieje. dodaj w pliku lista.h deklaracje zapowiadającą class osoba; lub #include "osoba.h" i powinno działać

Comments

W pliku lista.cpp zawarłem dwa include do "lista.h" i "osoba.h", a gdy próbowałem w lista.h wyrzuca błędy...
[C++ Error] Osoba.h(15): E2238 Multiple declaration for 'osoba'
[C++ Error] Osoba.h(14): E2344 Earlier declaration of 'osoba'

---

dodaj strażnika nagłówka do każdego pliku. Jeśli masz w różnych plikach dodane #include "osoba.h" to kompilator powiedzmy "wkleja" ten sam kod w dwóch óżnych miejscach i tworzą się dwie lub więcej deklaracji klasy Osoba. Strażnik nagłówka wygląda tak:

#pragma once

lub

#ifndef osoba_H

#define osoba_H

kod zpliku osoba.h

#endif

---

Builder domyślnie mi wstawił strażniki nagłówków, tylko uznałem to za niekonieczne i niewstawiłem do pytania, mój błąd już edytuje post .

---

Teraz to nie wiem czemu #endif kończy się przed klasą. Dalej bedzie wyskakiwał błąd.

---

Po chwili zrozumiałem gdzie ma być #endif i go dałem na koniec, ale mimo wszystko ciągle są te same błędy.

---

hmmm... dziwne ja po skopiowaniu tego kodu nie miałem błędu multiple definition tylko był problem z dodaj_osobe przy alokacji (nowy zamiast new osoba)

ten sam plik funkcja wyświetl liste po liście argumentów był średnik, przed pętlą while przy deklaracji i brak średnika i brak klamry zamykającej blok funkcji. To wszystkie błędy jakie miałem po skopiowaniu Twojego kodu.