c++ - Problemer med constructor og object array - Udvikleren.dk

Tags: c++

<< < 1 2 > >>

Bruger #10863 @ 19.03.10 09:30

Hej Udvikleren
Jeg har i noget tid her på htx arbejdet på et teknologiprojekt. Jeg skal lave et alarmsystem. Dog skal jeg kun simulere det hele.
Jeg er nede på meget få fejl, og jeg kan simpelthen ikke løse dem.
Først en af de store problemer:
Jeg skal bruge et array til at oprette nye objecter. Arrayet hedder int nySensor[30]; Den har jeg i min klasse, da jeg skal have adgang til den i flere funktioner.
Følgende kode har jeg i funktionen, som opretter nye sensor objekter:

Kode

	sensor s(navn, rum, vigtig, aktiv, normal, vaerdi, signal);
	nySensor[i]=s;
	nySensor[i].vis_sensor();

Ligemeget hvad jeg skriver foran erklæringen af arrayet i klassen giver det fejl, om jeg så bruger int, char, double, unsigned osv.
Fejlen er C2440, som lyder på at den ikke kan konvertere s til int. Grunden til at jeg skal have det i et array, er at brugeren skal have mulighed for at oprette x antal sensorer objekter.

Næste fejl er noget med pointere at gøre. Det skal siges at jeg ikke er særlig god til at programmere(endnu).
Har først følgende kode i klassen:

Kode

void sendMsg(int);

Funktionen er:

Kode

void sensor::sendMsg(int *nr, *nr2, *nr3)

Så bruger jeg følgende kode til at sende numrene i nySensors array.

Kode

sendMsg(i, ii, iii);

Meningen er blot at sende tal informationer til funktionen, så den kan benytte dem til at skrive informationer om den specifikke sensor.

Håber i kan Hjælpe.
Hilsen Søren Qvist

13 svar postet i denne tråd vises herunder
2 indlæg har modtaget i alt 3 karma

Sorter efter stemmer Sorter efter dato

Bruger #10281 @ 20.03.10 00:20

363

Jeg ved ikke om du har løst dit problem eller om det her link overhovedet svarer på nogle af dine spørgsmål. Jeg faldt dog lige over det her nu mens jeg sad og browsede min nyheder : http://www.augustcouncil.com/~tgibson/tutorial/ptr.html - skulle gerne hjælpe en del med at forstå pointers. Held og lykke.

Bruger #13669 @ 20.03.10 16:27

Ja, det ville være rart at se noget mere kode. Nu er jeg ikke ekspert ud i C++, men jeg tror ikke man kan skrive denne linje for sig selv:

Kode

sensor(char *, char *, int, bool, int, int, int);

Vis du vil lave et sensor object ser koden sådan her ud:

Kode

Sensor sensor(char *, char *, int, bool, int, int, int);

Det er nu det er smart vis klassen starter med stort. Så kan man have objektet sensor af typen Sensor. Vis det ikke er klart, så antager denne linje at din klasse starter med stort. Det ser ud som om at den gør det nu.
Det at ændre navnet på en klasse giver tit problemer, da man så skal ændre alle de steder man bruger klassen i koden. Det ser ud til du har glemt det her:

Kode

sensor nySensor[30];

Prøv

Kode

Sensor nySensor[30];

istedet for.

Bruger #13669 @ 19.03.10 14:13

int nySensor[30]; <- denne kode laver et array af int. Du vil have et array af sensor. for at gøre det skal du skrive:
sensor nySensor[30];

Det er en god ide at navngive ens klasser så de starter med stort, det går det lettere at se hvad der er objecter og hvad der er klasser.

Vis jeg forstår dit andet problem rigtigt, er der mange fejl. De tre kodestumper skal nok se sådan her ud:

void sendMsg(int *nr, int *nr2, int *nr3);

void senser::sendMsg(int *nr, int *nr2, int *nr3);

Den sidste er rigtig nok, men du skal være sikker på at i, ii og iii er pointere. Vis de ikke er skal du sætte et & tegn foran.

Bruger #10863 @ 20.03.10 11:57

Har ændret klassenavn til Sensor.
sensor nySensor[30]; var min lærer også indepå, men det giver et par stykker nye fejl:
Den mangler pludselig int(eller anden) foran denne sætning:

Kode

sensor(char *, char *, int, bool, int, int, int);

og foran denne:

Kode

sensor nySensor[30];

Skal jeg bare gøre det den beder om?
error C4430: missing type specifier - int assumed. Note: C++ does not support default-int

Bruger #4996 @ 20.03.10 14:21

196

Vi bliver nok nødt til at se noget mere kode, for at kunne fortælle dig hvad problemet er.
Det første ligner en constructor i class'en, og burde være ok, der ser heller ikke ud til at være problemer med den anden.

Bruger #10863 @ 21.03.10 00:06

Ha. Jeg troede jeg skulle bruge min construktor: sensor, med lille "s". ved nySensor arrayet.
Jeg forstår dog ikke, hvorfor jeg skal skrive klassenavnet foran construktoren og arrayet, når de allerede befinder sig i sensoren??
Kigger lige på de forslag du er kommet med og skriver tilbage imorgen.

Bruger #10863 @ 21.03.10 10:48

Jeg har fulgt dine råd, og det giver flere fejl, jeg ikke forstår. Her er hele loggen:

Kode

1>c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(18) : error C2148: total size of array must not exceed 0x7fffffff bytes
1>c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(18) : error C2079: 'Sensor::nySensor' uses undefined class 'Sensor'
1>c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(35) : error C2512: 'Sensor' : no appropriate default constructor available
1>c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(62) : error C2582: 'operator =' function is unavailable in 'Sensor'
1>c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(73) : error C2059: syntax error : '*'
1>c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(74) : error C2511: 'void Sensor::sendMsg(int *)' : overloaded member function not found in 'Sensor'
1>        c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(10) : see declaration of 'Sensor'
1>c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(144) : error C2660: 'Sensor::sendMsg' : function does not take 2 arguments
1>c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(167) : error C2660: 'Sensor::sendMsg' : function does not take 3 arguments
1>c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(183) : error C2065: 'signal' : undeclared identifier
1>c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(186) : error C2065: 'sensor' : undeclared identifier
1>c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(186) : error C2228: left of '.antal' must have class/struct/union
1>        type is ''unknown-type''
1>c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(201) : error C2679: binary '>>' : no operator found which takes a right-hand operand of type 'overloaded-function' (or there is no acceptable conversion)
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(1144): could be 'std::basic_istream<_Elem,_Traits> &std::operator >><std::char_traits<char>>(std::basic_istream<_Elem,_Traits> &,signed char *)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(1146): or       'std::basic_istream<_Elem,_Traits> &std::operator >><std::char_traits<char>>(std::basic_istream<_Elem,_Traits> &,signed char &)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(1148): or       'std::basic_istream<_Elem,_Traits> &std::operator >><std::char_traits<char>>(std::basic_istream<_Elem,_Traits> &,unsigned char *)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(1150): or       'std::basic_istream<_Elem,_Traits> &std::operator >><std::char_traits<char>>(std::basic_istream<_Elem,_Traits> &,unsigned char &)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(155): or       'std::basic_istream<_Elem,_Traits> &std::basic_istream<_Elem,_Traits>::operator >>(std::basic_istream<_Elem,_Traits> &(__cdecl *)(std::basic_istream<_Elem,_Traits> &))'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(161): or       'std::basic_istream<_Elem,_Traits> &std::basic_istream<_Elem,_Traits>::operator >>(std::basic_ios<_Elem,_Traits> &(__cdecl *)(std::basic_ios<_Elem,_Traits> &))'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(168): or       'std::basic_istream<_Elem,_Traits> &std::basic_istream<_Elem,_Traits>::operator >>(std::ios_base &(__cdecl *)(std::ios_base &))'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(175): or       'std::basic_istream<_Elem,_Traits> &std::basic_istream<_Elem,_Traits>::operator >>(std::_Bool &)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(194): or       'std::basic_istream<_Elem,_Traits> &std::basic_istream<_Elem,_Traits>::operator >>(short &)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(228): or       'std::basic_istream<_Elem,_Traits> &std::basic_istream<_Elem,_Traits>::operator >>(unsigned short &)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(247): or       'std::basic_istream<_Elem,_Traits> &std::basic_istream<_Elem,_Traits>::operator >>(int &)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(273): or       'std::basic_istream<_Elem,_Traits> &std::basic_istream<_Elem,_Traits>::operator >>(unsigned int &)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(291): or       'std::basic_istream<_Elem,_Traits> &std::basic_istream<_Elem,_Traits>::operator >>(long &)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(309): or       'std::basic_istream<_Elem,_Traits> &std::basic_istream<_Elem,_Traits>::operator >>(__w64 unsigned long &)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(329): or       'std::basic_istream<_Elem,_Traits> &std::basic_istream<_Elem,_Traits>::operator >>(__int64 &)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(348): or       'std::basic_istream<_Elem,_Traits> &std::basic_istream<_Elem,_Traits>::operator >>(unsigned __int64 &)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(367): or       'std::basic_istream<_Elem,_Traits> &std::basic_istream<_Elem,_Traits>::operator >>(float &)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(386): or       'std::basic_istream<_Elem,_Traits> &std::basic_istream<_Elem,_Traits>::operator >>(double &)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(404): or       'std::basic_istream<_Elem,_Traits> &std::basic_istream<_Elem,_Traits>::operator >>(long double &)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(422): or       'std::basic_istream<_Elem,_Traits> &std::basic_istream<_Elem,_Traits>::operator >>(void *&)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        c:\programmer\microsoft visual studio 9.0\vc\include\istream(441): or       'std::basic_istream<_Elem,_Traits> &std::basic_istream<_Elem,_Traits>::operator >>(std::basic_streambuf<_Elem,_Traits> *)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>        while trying to match the argument list '(std::basic_istream<_Elem,_Traits>, overloaded-function)'
1>        with
1>        [
1>            _Elem=char,
1>            _Traits=std::char_traits<char>
1>        ]
1>c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(204) : error C2065: 'signal' : undeclared identifier
1>c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(211) : error C2065: 'navn' : undeclared identifier
1>c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(217) : error C2065: 'navn' : undeclared identifier
1>c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(233) : error C2562: 'main' : 'void' function returning a value
1>        c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(231) : see declaration of 'main'
1>c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(237) : error C3861: 'reg_sensor': identifier not found
1>c:\documents and settings\eucvest\dokumenter\my dropbox\htx\teknologi\isb_program\isbfinal\isbfinal\alarm.cpp(240) : error C3861: 'alarm': identifier not found
1>Build log was saved at "file://c:\Documents and Settings\EUCVEST\Dokumenter\My Dropbox\HTX\teknologi\ISB_PROGRAM\ISBFINAL\ISBFINAL\Debug\BuildLog.htm"
1>ISBFINAL - 18 error(s), 0 warning(s)
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========

I bedte også om mere kode, så poster lige klassen og funktionen, der skal oprette objekterne:

Kode

class Sensor
{
public:
	Sensor(char *, char *, int, bool, int, int, int);
	void vis_sensor(void);//viser sensorens oplysninger
	void reg_sensor(void);//registrer ny sensor
	void sendMsg(int);//sender besked til brugeren + sørger for at logge hvad der er sket med dato og tid.
	void alarm(void);//det kunstige intelligens i programmet
	int antal;//antal sensorer computeren er har registeret og tilsluttet
	Sensor nySensor[30];
 
private:
	char navn[64];//sensorens navn TOMMY HJÆLP MED AT ÆNDRE DET TIL navn[64]
	char rum[64];//rummet sensoren befinder sig i
	int vigtig;//bruges til at tjekke hvor vigtig brugeren har valgt at sensoren skal være
	bool aktiv;//Bruges til at tjekke om sensoren allerede har opdaget noget(alarm funktionen
	int normal;//normal værdi for denne sensor
	int vaerdi;//ændring i værdi=igangsætning af alarm
	int vigtigMedium;//bruges i alarm
	int vigtigSmall;//Bruges i alarm
	int antalSensor;//Bruges i alarm
	int signal; //sensors signal, hvis 0 ingen sensor tilsluttet.
	int backup[2];//tager backup af de 3 variabler, så de kan "resettes" efter de har været brugt en gang	
};

Kode

void Sensor::reg_sensor(void)
{
	if(signal=1)
	{
	antal++;
	int i=antal;
	cout << "Ny sensor blev registreret" << endl;
	cout << "Udfyld venligst informationerne herunder: " << endl;
	cout << "Hvad vil du kalde den nye sensor?" << endl;
	cin >> navn;
	cout << "Godt navn. Hvor vigtig synes du sensoren skal være, på en skala fra 1-10?" << endl;
	cin >> vigtig;
	cout << "Godt valg. Hvilket rum befinder sensoreren sig i?" << endl;
	cin >> rum;


	Sensor s(navn, rum, vigtig, aktiv, normal, vaerdi, signal);
	nySensor[i]=s;
	nySensor[i].vis_sensor();
	}
}

Bruger #4996 @ 21.03.10 13:43

196

I class'en Sensor forsøger du at oprette et array af 30 objekter af typen Sensor (nySensor). Det går ikke. En struct eller class kan ikke have elementer af sin egen type. En class eller struct kan godt have pointere til objekter af sin egen type, men det er næppe det du har brug for her.

Oftest har man en container til at holde styr på andre objekter. Hvis man f.eks. skal lave en Bil class vil den kunne indholde et array af 4 Hjul objekter, men Hjul vil normalt ikke have arrays af sig selv.

I dit tilfælde betyder det nok at du skal dele Sensor class'en op i to, en der indeholder variabler og funktioner for én Sensor og en anden class der holder styr på alle dine sensorer.

void Sensor::reg_sensor(void)
Kunne således oprette et Sensor objekt og returnere det til den der holder styr på sensorerne.

Bruger #10863 @ 21.03.10 14:33

Holde styr på alle sensor objekterne??? Har du noget eksempel kode, så jeg kan se hvordan man gør sådan noget?

Bruger #4996 @ 21.03.10 14:51

196

Du kunne lave noget i stil med:

Kode

class SensorList
{
public:
   Sensor nySensor[30]; // Fjernes fra Sensor class'en
   void Setup();
   Sensor CreateSensor();
};  
void SensorList::setup()
{
   for(int n = 0; n < 30; n++)
      nySensor[n] = CreateSensor();
}

CreateSensor skal så lave ca. det samme som reg_sensor gjorde, men returnere et Sensor objekt.

reg_sensor skal fjernes fra Sensor.

Indlæg senest redigeret d. 21.03.2010 14:51 af Bruger #4996

<< < 1 2 > >>

Problemer med constructor og object array

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler