C++ Tworzenie tablicy struktur oraz odczyt z pliku.

Question

Witam mam zadanie które w skrócie brzmi. Stworzyć strukturę prostokąt z polami a, b i 2 funkcjami składowymi obwód i pole. Mamy stworzyć tablicę struktur i wypełnić ją danymi z pliku potem wszystko wyświetlić.

#include <iostream>
#include <fstream>
using namespace std;

struct rect
{
	int a;
	int b;
	void obwod();
	void pole();
};

void rect::obwod()
{
	cout << "Obwod: " << 2 * a + 2 * b;
}

void rect::pole()
{
	cout << ", pole: " << a * b;
}

int main()
{
	fstream plik;
	plik.open("dane.txt", ios::in);
	if (plik.good())
	{
		int count;
		plik >> count;
		rect* tab = new rect[count];
		for (int i = 0; i < count; i++)
		{
			plik >> tab[i].a;
			plik >> tab[i].b;
		}
		plik.close();

		//Wypisanie 
		cout << tab[1].b;
	}
}

Plik wygląda tak:

3    // ilośc prostokątów
1 4
5 6
7 8

Chciałbym się dowiedzieć czy takie rozwiązanie jest akceptowalne. 

Program faktycznie działa ale nie jestem pewien czy tak to powinno być zrobione.

Comments

Oprócz kosmetyki, typu polskie nazwy i globalna przestrzeń nazw, a i b powinny być unsigned, to jest okey. To wypisanie na próbę? Bo poleceni rozumiem tak, że masz wypisać wszystko.

---

Tak co do kosmetyki polskie nazwy mam podane w zadaniu więc raczej wolę tak zostawić. A wypisanie to tylko sprawdzenie Bo wystarczy dodać tylko zwykła pętla wiec już nie dawałem.

Answer 1

1. Nie używaj 'using namespace'
2. std::fstream ---> std::ifstream.
3. Zamiast funkcji open użyj paramterów konstruktora.
4. std::ifstream::good() ---> std::ifstream::is_open()
5. Zapomniałeś użyć delete[]
6. Nie zamykaj pliku - zrobi to destruktor (RAII).
7. a, b, count powinny być typu całkowitego

Przykład:

#include <cstddef>
#include <iostream>
#include <fstream>
#include <vector>

struct rect
{
	unsigned a;
	unsigned b;
	void obwod();
	void pole();
};

void rect::obwod()
{
	std::cout << "Obwod: " << 2 * a + 2 * b;
}

void rect::pole()
{
	std::cout << ", pole: " << a * b;
}

int main()
{
	std::ifstream plik("dane.txt");
	if (plik.is_open())
	{
		std::size_t count; 
		plik >> count;

		std::vector<rect> rects(count); // żeby nie używać ciągle delete[]
		for (std::size_t i = 0; i < count; i++)
		{
			plik >> rects[i].a >> rects[i].b;
		}

		for (auto& rect : rects) 
        {
			std::cout << rect.a << " " << rect.b << "\n";
		}
	}
}