Wyswietlanie tablicy

Question

Siemka co robie nie tak w poniższym programie, bo próbuje wyświetlić tablicę obiektów na kilka sposobów i nie działa

#include<iostream>
using namespace std;
class Pies
{
private:
	char imie[16];
	char rasa[16];

public:
	Pies() {}
	friend istream &operator>>(istream &is, Pies &f) {
		return is >> f.imie >> f.rasa;
	}
	friend ostream &operator<<(ostream &output, const Pies &D) {
		output << D.imie << D.rasa;
		return output;
	}
	Pies(char imie[16], char rasa[16]) {}

};

int main()
{
	int licznik = 0;
	Pies tab[100];
	char wybor;
	while (true)
	{
		cout << "Wybierz opcje: " << endl;
		cin >> wybor;
		switch (wybor)
		{
		case 'N':
		{
			char im[16];
			cout << "Podaj imie psa: " << endl;
			cin >> im;

			char n[16];
			cout << "Podaj rase psa: " << endl;
			cin >> n;

			tab[licznik] = Pies(im, n);
			licznik++;
			break;
		}
		case 'W':
		{
			cout << tab << endl;
			break;
		}

		}
	}
	
}

 

Comments

napisz co nie działa bo nikt ci nie pomoże. Jeżeli masz jakieś errory to też powinneś o nich powiadomić.

---

program się kompiluje, ale nie działa case W, który powinien wyświetlić tablicę obiektów Pies

---

Case W działa tak jak powinien czyli wypisuje adres pod jakim znajduje się tablica

Answer 1

Pierwszy problem to pusty konstruktor.

Drugi zaś to próba wypisania całej tablicy na raz, a nie poszczególnych elementów.

Na początek napisz jakiś konstruktor.

Następnie zamień:

cout << tab << endl;

na:

for(int i=0;i<licznik;++i)
    cout << tab[i] << endl;

w linii 49.

Powinieneś zamienić jeszcze w liniach 6 i 7:

    char imie[16];
    char rasa[16];

na:

    char* imie;
    char* rasa;

Kod już działa, ale moim zdaniem wymaga wielu poprawek:

• Nie powinieneś używać using namespace std;
• Powinieneś używać angielskich nazw zmiennych itp.
• Lepiej nazywać zmienne, tak, żeby było wiadomo co oznaczają.
• W C++ używać string zamiast char*.

Comments

A co jest jeszcze nie tak z wyswietlaniem, bo jednak nie działa 

      case 'W':
        {
             
            for(int i=0;i<licznik;++i)
            cout << tab[i] << endl;
            break;
        }
  

 

---

Napisz co dokładnie nie działa i wyślij cały kod.

---

@DeBos123, doradziłeś OP, żeby zmienił tablice na wskaźniki, ale nie pociągnąłeś do końca tematu i kod jest z błędami.

 

Jak już mają być wskaźniki, to tak (choć lepiej użyć std::string):

class Pies
{
private:
	char* imie;
	char* rasa;

public:
	friend istream &operator>>(istream &is, Pies &f) 
	{
		/* niezbyt bezpieczna implementacja, ale nie chce mi się dłubać w tym */
		char buff[256];

		is >> buff;
		delete[] f.imie;
		f.imie = strcpy(new char [strlen(buff) + 1], buff);

		is >> buff;
		delete[] f.rasa;
		f.rasa = strcpy(new char [strlen(buff) + 1], buff);

		return is;
	}

	friend ostream &operator<<(ostream &output, const Pies &D) {
		
		output << D.imie << D.rasa;
		return output;
	}

	Pies& operator=(const Pies& pies)
	{
		if(&pies == this) return *this;
		delete[] imie;
		delete[] rasa;
		imie = strcpy(new char [strlen(pies.imie) + 1], pies.imie);
		rasa = strcpy(new char [strlen(pies.rasa) + 1], pies.rasa);
		return *this;
	}

	Pies(const Pies &pies) 
		: Pies(pies.imie, pies.rasa)
	{}

	Pies(const char *im = "", const char *ra = "") 
	{
		imie = strcpy(new char [strlen(im) + 1], im);
		rasa = strcpy(new char [strlen(ra) + 1], ra);
	}
	
	~Pies()
	{
		delete[] imie;
		delete[] rasa;
	}
};

 

---

A da się to zrobić, nie zamieniając na wskaźniki tak jak było na początku, po porostu normalne tablice??

---

Da się. Do kopiowania łańcuchów znakowych do tablicy użyj funkcji strcpy,