Entra

View Full Version : [c++] problema creazione classe


Supersimo88
18-02-2010, 09:39
Ciao a tutti Stavo provando a creare una classe (VideoReg) per la gestione di alcuni modelli di videoregistratori, i loro pezzi di ricambio, il costo, il numero pezzi di ricambio e i pezzi venduti ( rispettivamente Vcod, RicCod, Costo, Nrpz e NrVen)
La classe dovrà fornire i seguenti metodi:
1. Costruttori,
2. per l'accesso ai dati,
3. per leggere i dati iniziali (VCod, RicCod, Costo e NrPz a magazzino) da input,
4. per la stampa dei dati,
5. per fornire il dato relativo al numero di pezzi venduti via via in modo da aggiornare il
corrispondente dato membro NrVen.

Ho creato il file VideoReg.h
#ifndef __VIDEOREG__
#define __VIDEOREG__


class VideoReg {
public:

// Costruttori
VideoReg();
VideoReg(int VCod, int RicCod, int Costo, int NrPz, int NrVen);
VideoReg(const VideoReg& d);

// Distruttore
~VideoReg();

// Selettori
int get_VCod() const;
int get_RicCod() const;
int get_Costo() const;
int get_NrPz() const;
void get_Videoreg(int& Vcod, int& RicCod, int& Costo, int& Nrpz) const;

// Modificatori
int set_VCod(int VCod);
int set_RicCod(int RicCod);
int set_Costo(float Costo);
int set_NrPz(int NrPz);
int set_NrVen(int NrVen);
int set_VideoReg(int Vcod, int RicCod, int Costo, int Nrpz, int NrVen);

// Operazioni
void inserisci();
void stampa() const;
void venduti();

private:

// Dati membro
int _VCod;
int _RicCod;
int _Costo;
int _NrPz;
int _NrVen;


};


#endif


il file VideoReg.cpp
#include <iostream>
#include "VideoReg.h"

using namespace std;


//////////////////////////////////////////////////////////////////////////
// Costruttori
//////////////////////////////////////////////////////////////////////////

VideoReg::VideoReg()
{
_Vcod = Vcod;
_RicCod = RicCod;
_Costo = Costo;
_Nrpz = Nrpz;
_NrVen = NrVen;
}

VideoReg::VideoReg(int Vcod, int RicCod, int Costo, int Nrpz, int NrVen)
{
_Vcod = Vcod;
_RicCod = RicCod;
_Costo = Costo;
_Nrpz = Nrpz;
_NrVen = NrVen;
}

VideoReg::VideoReg(const VideoReg& d)
{
_Vcod = d._Vcod;
_RicCod = d._RicCod;
_Costo = d._Costo;
_Nrpz = d._Nrpz;
_NrVen = d._NrVen;
}


//////////////////////////////////////////////////////////////////////////
// Distruttore
//////////////////////////////////////////////////////////////////////////

VideoReg::~VideoReg()
{
}


//////////////////////////////////////////////////////////////////////////
// Selettori
//////////////////////////////////////////////////////////////////////////

int VideoReg::get_VCod() const
{
return _VCod;
}

int VideoReg::get_RicCod() const
{
return _RicCod;
}

int VideoReg::get_Costo() const
{
return _Costo;
}

int VideoReg::get_NrPz() const
{
return _NrPz;
}

int VideoReg::get_NrVen() const
{
return _NrVen;
}

void VideoReg::get_Videoreg(int& Vcod, int& RicCod, int& Costo, int& Nrpz int& NrVen) const
{
VCod = _VCod;
RicCod = _RicCod;
Costo =_Costo;
NrPz = _NrPz;
NrVen =_NrVen;
}


//////////////////////////////////////////////////////////////////////////
// Modificatori
//////////////////////////////////////////////////////////////////////////

int VideoReg::set_VCod(int VCod)
{
_VCod = VCod;
}

int VideoReg::set_RicCod(int RicCod)
{
_RicCod = RicCod;
}

void VideoReg::set_Costo(int Costo)
{
_Costo = Costo;
}

int VideoReg::set_NrPz(int NrPz)
{
_NrPz = NrPz;
}

int VideoReg::set_NrVen(int NrVen)
{
_NrVen = NrVen;
}

int VideoReg::set_VideoReg(int Vcod, int RicCod, int Costo, int Nrpz, int NrVen)
{

_Vcod = Vcod;
_RicCod = RicCod;
_Costo = Costo;
_Nrpz = Nrpz;
_NrVen = NrVen;
}


//////////////////////////////////////////////////////////////////////////
// Operazioni
//////////////////////////////////////////////////////////////////////////

void VideoReg::inserisci()
{
do {
cout << "Inserire codice videoregistratore: ";
cin >> _VCod;
cout << "Inserire codice pezzo di ricambio: ";
cin >> _RicCod;
cout << "Inserire costo pezzo di ricambio: ";
cin >> _Costo;
cout << "Inserire quantità del pezzo di ricambio: ";
cin >> _NrPz;
cout << "Inserire Quantità pezzo di ricambio venduta: ";
cin >> _NrVen;
cout << endl;

cin.clear();
cin.sync();
}

void VideoReg::stampa() const;
{
cout << _VCod << "****" << _RicCod << "****" << _Costo << "****" << _NrPz << "****" << _NrVen;
}
}

Ma Visulal Studio continua a darmi questi errori
1>------ Inizio compilazione: Progetto: esercizio, Configurazione: Debug Win32 ------
1>Compilazione in corso...
1>VideoReg.cpp
1>i:\videoreg.cpp(19) : error C2065: '_Vcod': identificatore non dichiarato
1>i:\videoreg.cpp(19) : error C2065: 'Vcod': identificatore non dichiarato
1>i:\videoreg.cpp(20) : error C2065: 'RicCod': identificatore non dichiarato
1>i:\videoreg.cpp(21) : error C2065: 'Costo': identificatore non dichiarato
1>i:\videoreg.cpp(22) : error C2065: '_Nrpz': identificatore non dichiarato
1>i:\videoreg.cpp(22) : error C2065: 'Nrpz': identificatore non dichiarato
1>i:\videoreg.cpp(23) : error C2065: 'NrVen': identificatore non dichiarato
1>i:\videoreg.cpp(28) : error C2065: '_Vcod': identificatore non dichiarato
1>i:\videoreg.cpp(31) : error C2065: '_Nrpz': identificatore non dichiarato
1>i:\videoreg.cpp(37) : error C2065: '_Vcod': identificatore non dichiarato
1>i:\videoreg.cpp(37) : error C2039: '_Vcod': non è un membro di 'VideoReg'
1> i:\videoreg.h(10): vedere la dichiarazione di 'VideoReg'
1>i:\videoreg.cpp(40) : error C2065: '_Nrpz': identificatore non dichiarato
1>i:\videoreg.cpp(40) : error C2039: '_Nrpz': non è un membro di 'VideoReg'
1> i:\videoreg.h(10): vedere la dichiarazione di 'VideoReg'
1>i:\videoreg.cpp(78) : error C2039: 'get_NrVen': non è un membro di 'VideoReg'
1> i:\videoreg.h(10): vedere la dichiarazione di 'VideoReg'
1>i:\videoreg.cpp(79) : error C2270: 'get_NrVen': modificatori non consentiti per funzioni non membro
1>i:\videoreg.cpp(80) : error C2065: '_NrVen': identificatore non dichiarato
1>i:\videoreg.cpp(83) : error C2144: errore di sintassi: 'int' deve essere preceduto da ','
1>i:\videoreg.cpp(84) : error C2511: 'void VideoReg::get_Videoreg(int &,int &,int &,int &,int &) const': funzione membro in overload non trovata in 'VideoReg'
1> i:\videoreg.h(10): vedere la dichiarazione di 'VideoReg'
1>i:\videoreg.cpp(108) : error C2511: 'void VideoReg::set_Costo(int)': funzione membro in overload non trovata in 'VideoReg'
1> i:\videoreg.h(10): vedere la dichiarazione di 'VideoReg'
1>i:\videoreg.cpp(156) : error C2062: tipo 'void' imprevisto
1>Il log di compilazione è stato salvato in 'file://c:\Users\Simone\Documents\Visual Studio 2008\Projects\esercizio\esercizio\Debug\BuildLog.htm'
1>esercizio - 20 errore/i, 0 avviso/i
========== Compilazione: 0 completate, 1 non riuscite, 0 aggiornate, 0 ignorate ==========


Mi sapreste aiutare molto gentilmente?
Non riesco a uscirne fuori, premetto che sono alle prime esperienze con C++

Mirkolo
18-02-2010, 10:54
Nel primo costruttore, quello senza parametri, le variabili che usi effettivamente non sono dichiarate. Lì dentro puoi solo settare le variabili private a valori di default.

Supersimo88
18-02-2010, 11:03
Quindi che dovrei mettere?
Grazie

Mirkolo
18-02-2010, 13:17
VideoReg::VideoReg()
{
_Vcod = Vcod;
_RicCod = RicCod;
_Costo = Costo;
_Nrpz = Nrpz;
_NrVen = NrVen;
}

Qui dentro Vcod, RicCod e gli altri non sono definiti, come invece lo sono nel secondo costruttore. Di solito nei costruttori senza parametri si inizializzano le variabili con valori di default, ad esempio _Vcod = 1, ecc. Questo devi deciderlo tu.

L'errore riportato nel secondo costruttore è dovuto al fatto che il C è case sensitive, quindi:

VideoReg::VideoReg(int Vcod, int RicCod, int Costo, int Nrpz, int NrVen)
{
_Vcod = Vcod;

deve diventare:

VideoReg::VideoReg(int Vcod, int RicCod, int Costo, int Nrpz, int NrVen)
{
_VCod = Vcod;

e così via... le altre righe non le ho controllate, ma presumo siano sviste simili.