Początki obiektówki c++

Question

Witam,

od kilku dni próbuję pisać obiektowo, mogłby ktoś sprawdzić mój kod.

Wydaje mi się, że jest on bardzo nieopytmalny, i dużo rzeczy jest niepotrzenych :)

Pozdrawiam :)

#include <iostream>
#include <cstdlib>
#include <ctime>

using namespace std;

class Kostka {
public:
	int ilu_gra;
	int ile_rzutow;

	Kostka(){
		cout << "Podaj liczbę grających w kości: " << endl;
		cin >> ilu_gra;
		cout << "Podaj liczbę rzutów kostką: " << endl;
		cin >> ile_rzutow;
		cout << "LICZBA GRAJĄCYCH: " << ilu_gra;
		cout << " LICZBA RZUTÓW: " << ile_rzutow << endl << endl;
	}

};

class Kubek {
public:
	int wylosowana;

	void losuj(){
		wylosowana = rand() % 5 + 1;
	}

	void wyswietl(){
		cout << wylosowana << endl;
	}

};

int main() {
	Kostka kostka;

	if(kostka.ilu_gra <= 0)
		kostka.ilu_gra = 1;
	if(kostka.ile_rzutow <= 0)
		kostka.ile_rzutow = 1;

	Kubek* obiekt = new Kubek[kostka.ilu_gra*kostka.ile_rzutow];
	srand( time( NULL ) );

	for(int i = 0; i < kostka.ilu_gra; i++)
	{
		for(int j = 0; j < kostka.ile_rzutow; j++)
		{
			cout << "Gracz nr: " << i + 1 << " rzut nr: " << j + 1 << " wylosowal ";
			obiekt[i].losuj();
			obiekt[i].wyswietl();
		}
	}

	delete [] obiekt;
	return 0;
}

 

Answer 1

1. Z polskich znaków robią się wzorki.

2. Losują się liczby od 1 do 5, a chyba miała być sześcienna kostka. wylosowana = rand() % 5 + 1;

powinno być 6+1, przykładowo jak robisz losowanie z zakresu 1-100 to bez "+1" wyjdzie 0-99 (zero też jest liczbą) i dlatego musisz dodać +1, ale nadal dzieląc (% - modulo - w tym przypadku dzielenie) przez 100.

Answer 2

Jak dobrze rozumiem chodzi tu o "rzucanie" kostką (kostkami) przez kilka osób.
Moim zdaniem, z punktu widzenia jakiejś logiki, to klasa Kostka jest do poprawy. Jako "cechę" kostki uważałbym to jaka liczba wypadła, potocznie mówiąc. Ewentualnie ile ma ścian. A co właściwości (funkcje) to proces "rzucania" losowania wartości jaką kostka wskazuje oraz sam odczyt jaka liczba wypadła. Ile osób rzuca kostką albo po ile razy nią rzuca nie jest nie opisuje kostki jako takiej.