Losowanie właściwości klasy zawartej w konstruktorze

Question

Witam, mam problem związany z losowaniem/wpisywaniem nowej wartości dla właściwości klasy w kontruktorze. Uczę się funkcji zapryzjaźnionych i chcę napisać pianino, które ma wspólną funkcję grającą nuty. 

 

#include <iostream>
#include <cstdlib>
#include <windows.h>
#include <ctime>
using namespace std;

class Pianista;

class Piano
{
	int nuta;
	
	public:
		Piano()
		{
			nuta=15000;
		}
		
		Piano(char zklaw)
		{
			cout<<"Podaj czestotliwosc zagranego dzwieku: "; cin>>nuta;
		}
		
		Piano(string losowanko)
		{
			nuta=rand()%700+250;
		}
		
		friend void grajnuty(Piano &forte, Pianista &radek);
};

class Pianista
{
	int ilnut;
	
	public:
		Pianista()
		{
			ilnut=20;
		}
		
		Pianista(char zklaw)
		{
			cout<<"Podaj ilosc zagranych dzwiekow: "; cin>>ilnut;
		}
		
		Pianista(int losuj)
		{
			ilnut=rand()%50+1;
			cout<<"Zagranych zostanie "<<ilnut<<" nut."<<endl;
		}
		
		
		
		
		friend void grajnuty(Piano &forte, Pianista &radek);
};

	void grajnuty(Piano &granie, Pianista &radek)
		{
			cout<<"Ilosc nut: "<<radek.ilnut<<endl;
			for(int i=0;i<radek.ilnut; i++)
			{
				Beep(granie.nuta,1000);
				cout<<"czestotliwosc: "<<granie.nuta<<endl;
			}
		}
/* run this program using the console pauser or add your own getch, system("pause") or input loop */

int main(int argc, char** argv) {
	srand(time(NULL));
	Piano forte=Piano("losuje");
	Pianista radek=Pianista(1);
	grajnuty(forte, radek);
	return 0;
}

Wszystko powinno działać, jednakże nuty nie losują się - co mnie dziwi, ponieważ jest użyty odpowiedni kontruktor i nuta powinna sama automatycznie się zmieniać. Ktoś wie jak temu zaradzić?

Answer 1

Piano(string losowanko)
{
    nuta=rand()%700+250;
}

To jest bezsensu. Po co przeciążasz konstruktor skoro i tak nie wykorzystujesz tego?

Tak samo tutaj:
 

Pianista(int losuj)
        {
            ilnut=rand()%50+1;
            cout<<"Zagranych zostanie "<<ilnut<<" nut."<<endl;
        }

A to:

void grajnuty(Piano &granie, Pianista &radek)
        {
            cout<<"Ilosc nut: "<<radek.ilnut<<endl;
            for(int i=0;i<radek.ilnut; i++)
            {
                Beep(granie.nuta,1000);
                cout<<"czestotliwosc: "<<granie.nuta<<endl;
            }
        }

Co to ma znaczyć? Najpierw zaczynasz ładnie, obiektowo, a następnie robisz ze wszystkiego kaszę i proceduralnie?

To powinno być w klasie parametry ustawiane w konstruktorze, a pętla for w jakiejś metodzie. 

W ogóle jakoś mi to dziwnie wygląda wszystko.

Comments

Chcę to zrobić za pomocą dwóch klas, więc wartości muszę wyciągać za pomocą operatora wyłuskania - inaczej tego nie zrobię. Losowania natomiast inaczej nie zrobię, ponieważ się po prostu nie da(lub nie potrafię).