Hi,
Just at a quick glance, you might try changing the names of your member data
(length, radius, etc.) These names may be colliding with the parameter
names in your constructor (which are also length, radius, etc.). One coding
standard to consider is to use m_ (lower case m followed by an underscore
character) as a prefix to each member data name. This will differentiate
them from the parameter names in your functions.
Hope this helps!
JAB
--------------------------------------------------------
John A. Byerly
Engineered Software Solutions, Ltd.
http://www.ess-quality.com
(724) 598-7990 (voice)
(724) 598-7992 (fax)
> -----Original Message-----
> From: BLIND-DEV: Development of Adaptive Hardware & Software for the
> Blind/VI [mailto:[log in to unmask]]On Behalf Of Marvin
> Hunkin
> Sent: Thursday, April 01, 2004 9:30 PM
> To: [log in to unmask]
> Subject: C++ Errors
>
>
> This is a multi-part message in MIME format.
>
> ------=_NextPart_000_000A_01C418AE.3D93A980
> Content-Type: text/plain;
> charset="iso-8859-1"
> Content-Transfer-Encoding: quoted-printable
>
> Hi.
> i will post the code and the error message below.
> still confused about constructors.
> please help me out.
> i am getting confused, frustrated, and really fed up.
> why it is not working.
> Am i just a dummy?
> please give me exact code examples.
> how to fix this problem.
> Cheers Marvin.
>
>
> /*
> * Program: C++ Session 21 Exercise
> * File Name: Paddock.cpp
> *Description: Session 21 Exerciseto design a program for a farmer to =
> water a area of his paddock
> * Creator: Marvin Hunkin
> *Date Created: Friday March 19 2004 3:35 pm
> * Version: 1.00
> */
>
>
>
> //iostream is needed for input and output
>
>
> #include <iostream.h>
> #include <conio.h>
> #include <math.h>
>
> using namespace std;
>
>
>
> class Tanker
> {
> public:
> float radius;
> float length;
> float fertaliserlevel;
> float fertaliser;
> float water;
> float size;
> float pi;
> float width;
> float hectre;
> float area;
>
>
> Tanker(float radius, float length,float fertaliserlevel);
>
> void Title();
> float TotalVolume(float pi, float radius, float length);
> float VolumeOccupiedSpace(float depth, float length);
> int Convert();
> void PromptFertaliser();
> void PromptHectre();
> float CalculateArea(float fertaliser, float water);
> void Result();
> void PromptLiquid();
> }; //close Tanker class
>
>
>
> int main ()
> {
>
>
> Tanker myTanker(r,l,f);
>
>
> myTanker.fertaliser=3D0.75;
> myTanker.water=3D0.25;
> myTanker.pi=3D3.1415;
> myTanker.width=3D10;
> myTanker.hectre=3D10000;
> myTanker.Title();
> myTanker.PromptFertaliser();
> myTanker.PromptHectre();
> myTanker.Result();
> myTanker.PromptLiquid();
> getch ();
> return 0;
> } //close main function
>
>
> void Tanker::Title()
> {
> cout <<"\nWelcome To Marvins Paddock ";
> cout << "\nThis program asks the user to enter a number for the amount =
> of liquid to be entered into the tanker. ";
> cout << "\nFrom the keyboard. ";
> cout << "\nAnother pprompt appears on screen, asking the user to enter a =
> number , for the amount of area they wish to use for the paddock. ";
> cout << "\nFrom the keyboard. ";
> cout << "\nAnother prompt appears on screen, asking the user to enter =
> the amount of liquid which they want to use for the tanker. " ;
> cout << "\nFrom the keyboard. ";
> cout << "\nthen the total fertaliser, total water, total volume and =
> total area used for the paddock appears on screen. ";
> } //close title function
>
>
> float Tanker::TotalVolume(float Pi, float Radius, float Length )
> {
> float answer;
> answer =3D Pi * Radius * Radius * Length;
> return answer;
> } //close total Volume function
>
> float Tanker::VolumeOccupiedSpace(float Depth, float Length)
> {
> float depth;
> float depthDivideRadius =3D ((radius - depth) / radius);
> float theta =3D 2*acos(depthDivideRadius);
> float surfaceArea =3D (radius * radius) * (theta -
> sin(theta))/2;
> float segmentVolume =3D surfaceArea * length;
> return segmentVolume;
> } // close volumeOccupiedSpace =
> function
>
>
> int Tanker::Convert()
> {
> int answer=3D0;
> int hectre=3D10000;
> int millimetres=3D1000 ;
> answer=3Dmillimetres / 1000;
> answer=3Dhectre / 10000;
> return answer;
> } //close Convert function
>
>
> void Tanker::PromptFertaliser()
> {
> float EnterTank=3D0.0;
>
> cout << "Type a number, then press enter: " << endl;
> cin >> EnterTank;
>
> } //close promptFertaliser function
>
>
> void Tanker::PromptHectre()
> {
> float EnterArea=3D0.0;
> cout << " Type a number, then press enter: " << endl;
> cin >> EnterArea;
> } // close promptHectre function
>
>
> float Tanker::CalculateArea(float Fertaliser, float Water)
> {
> float answer;
> answer =3Dsize * fertaliser * water;
> return answer;
> } //close calculateArea function
>
>
>
> void Tanker::Result()
> {
> cout << "Total Fertaliser Used: " << fertaliser * 0.75 << " Litres " << =
> endl;
> cout << "Total Water Used: " << water * 0.25 << " Litres " << endl;
> cout << "Total Volume Used: " << pi * radius * radius * length << " =
> Litres " << endl;
> cout << "Total Area Used: " << area * width * length * hectre << =
> "Hectres " << endl;
> } //close result function
>
>
> void Tanker::PromptLiquid()
> {
> cout << "Enter a number, then press enter: " << endl;
> } //close PromptLiquid function
>
>
> Tanker::Tanker(float r, float l, float f)
> {
> radius =3D r;
> length =3D l;
> fertaliserlevel =3D f;
> } //close tanker constructor
>
> [C++ Error] Paddock.cpp(57): E2084 Parameter names are used only with a =
> function body
>
> [C++ Error] Paddock.cpp(59): E2139 Declaration missing ;
>
> [C++ Error] Paddock.cpp(60): E2451 Undefined symbol 'myTanker'
>
>
>
> ps: it is the constructor, which i have diagnosed, but don't know how to =
> fix it. tried several things, but not liking it.
> giving me a migrain.
>
>
>
>
>
>
>
> ------=_NextPart_000_000A_01C418AE.3D93A980
> Content-Type: text/html;
> charset="iso-8859-1"
> Content-Transfer-Encoding: quoted-printable
>
> <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
> <HTML><HEAD>
> <META http-equiv=3DContent-Type content=3D"text/html; =
> charset=3Diso-8859-1">
> <META content=3D"MSHTML 6.00.2800.1226" name=3DGENERATOR>
> <STYLE></STYLE>
> </HEAD>
> <BODY bgColor=3D#ffffff>
> <DIV><FONT face=3DArial size=3D2>Hi.</FONT></DIV>
> <DIV><FONT face=3DArial size=3D2>i will post the code and the error =
> message=20
> below.</FONT></DIV>
> <DIV><FONT face=3DArial size=3D2>still confused about =
> constructors.</FONT></DIV>
> <DIV><FONT face=3DArial size=3D2>please help me out.</FONT></DIV>
> <DIV><FONT face=3DArial size=3D2>i am getting confused, frustrated, and =
> really fed=20
> up.</FONT></DIV>
> <DIV><FONT face=3DArial size=3D2>why it is not working.</FONT></DIV>
> <DIV><FONT face=3DArial size=3D2>Am i just a dummy?</FONT></DIV>
> <DIV><FONT face=3DArial size=3D2>please give me exact code =
> examples.</FONT></DIV>
> <DIV><FONT face=3DArial size=3D2>how to fix this problem.</FONT></DIV>
> <DIV><FONT face=3DArial size=3D2>Cheers Marvin.</FONT></DIV>
> <DIV><FONT face=3DArial size=3D2></FONT> </DIV>
> <DIV><FONT face=3DArial size=3D2></FONT> </DIV>
> <DIV><FONT face=3DArial size=3D2>/*<BR>* Program: C++ Session 21 =
> Exercise<BR>* File=20
> Name: Paddock.cpp<BR>*Description: Session 21 Exerciseto design a =
> program for a=20
> farmer to water a area of his paddock<BR>* Creator: Marvin =
> Hunkin<BR>*Date=20
> Created: Friday March 19 2004 3:35 pm<BR> * Version:=20
> 1.00<BR> */</FONT></DIV>
> <DIV> </DIV>
> <DIV><FONT face=3DArial size=3D2></FONT> </DIV>
> <DIV> </DIV>
> <DIV><FONT face=3DArial size=3D2>//iostream is needed for input and=20
> output</FONT></DIV>
> <DIV> </DIV><FONT face=3DArial size=3D2>
> <DIV><BR>#include <iostream.h><BR>#include =
> <conio.h><BR>#include=20
> <math.h></DIV>
> <DIV> </DIV>
> <DIV>using namespace std;</DIV>
> <DIV> </DIV>
> <DIV> </DIV>
> <DIV> </DIV>
> <DIV>class Tanker<BR>{<BR> =20
> public:<BR> float radius;<BR> float length;<BR> float=20
> fertaliserlevel;<BR> float fertaliser;<BR> float =
> water;<BR> float=20
> size;<BR> float pi;<BR> float width;<BR> float =
> hectre;<BR>float=20
> area;</DIV>
> <DIV> </DIV>
> <DIV><BR>Tanker(float radius, float length,float fertaliserlevel);</DIV>
> <DIV> </DIV>
> <DIV>void Title();<BR> float TotalVolume(float pi, float =
> radius,=20
> float length);<BR> float VolumeOccupiedSpace(float depth, float=20
> length);<BR> int Convert();<BR>void PromptFertaliser();<BR>void=20
> PromptHectre();<BR> float CalculateArea(float fertaliser, float=20
> water);<BR> void Result();<BR> void =
> PromptLiquid();<BR>}; =20
> //close Tanker class</DIV>
> <DIV> </DIV>
> <DIV> </DIV>
> <DIV> </DIV>
> <DIV> &n=
> bsp; =20
> int main ()<BR>{</DIV>
> <DIV> </DIV>
> <DIV><BR> Tanker myTanker(r,l,f);</DIV>
> <DIV> </DIV>
> <DIV><BR> myTanker.fertaliser=3D0.75;<BR> myTanker.water=3D0.25=
> ;<BR> myTanker.pi=3D3.1415;<BR> myTanker.width=3D10;<BR> m=
> yTanker.hectre=3D10000;<BR> myTanker.Title();<BR> &=
> nbsp;=20
> myTanker.PromptFertaliser();<BR> =20
> myTanker.PromptHectre();<BR> =20
> myTanker.Result();<BR> =20
> myTanker.PromptLiquid();<BR> =20
> &=
> nbsp; =20
> getch=20
> ();<BR> =
> =20
> return 0;<BR>} //close main function</DIV>
> <DIV> </DIV>
> <DIV><BR> void Tanker::Title()<BR>{<BR>cout <<"\nWelcome To =
> Marvins=20
> Paddock ";<BR>cout << "\nThis program asks the user to enter a =
> number for=20
> the amount of liquid to be entered into the tanker. ";<BR>cout << =
> "\nFrom=20
> the keyboard. ";<BR>cout << "\nAnother pprompt appears on screen, =
> asking=20
> the user to enter a number , for the amount of area they wish to use for =
> the=20
> paddock. ";<BR>cout << "\nFrom the keyboard. ";<BR>cout <<=20
> "\nAnother prompt appears on screen, asking the user to enter the =
> amount=20
> of liquid which they want to use for the tanker. " ;<BR>cout << =
> "\nFrom=20
> the keyboard. ";<BR>cout << "\nthen the total fertaliser, total =
> water,=20
> total volume and total area used for the paddock appears on screen. =
> ";<BR>}=20
> //close title function</DIV>
> <DIV> </DIV>
> <DIV><BR> float Tanker::TotalVolume(float Pi, float Radius, =
> float=20
> Length )<BR>{<BR>float answer;<BR> answer =3D Pi * Radius * =
> Radius *=20
> Length;<BR> return answer;<BR>} //close total Volume =
> function</DIV>
> <DIV> </DIV>
> <DIV> float Tanker::VolumeOccupiedSpace(float Depth, float=20
> Length)<BR>{<BR>float depth;<BR> float =
> depthDivideRadius =3D=20
> ((radius - depth) /=20
> radius);<BR> &=
> nbsp; =20
> float theta =3D=20
> 2*acos(depthDivideRadius);<BR> &=
> nbsp; =20
> float surfaceArea =3D (radius * radius) * (theta=20
> -<BR>sin(theta))/2;<BR> &n=
> bsp; =20
> float segmentVolume =3D surfaceArea *=20
> length;<BR> &n=
> bsp; =20
> return=20
> segmentVolume;<BR> &=
> nbsp; &n=
> bsp; &nb=
> sp; =20
> } // close volumeOccupiedSpace function</DIV>
> <DIV> </DIV>
> <DIV><BR> int Tanker::Convert()<BR>{<BR> int =
> answer=3D0;<BR> int=20
> hectre=3D10000;<BR> int millimetres=3D1000 ;<BR> =
> answer=3Dmillimetres /=20
> 1000;<BR> answer=3Dhectre /=20
> 10000;<BR> return =
> answer;<BR>}=20
> //close Convert function</DIV>
> <DIV> </DIV>
> <DIV><BR>void Tanker::PromptFertaliser()<BR>{<BR> float=20
> EnterTank=3D0.0;</DIV>
> <DIV> </DIV>
> <DIV> cout << "Type a number, then press enter: " =
> <<=20
> endl;<BR> cin >>=20
> EnterTank;</DIV>
> <DIV> </DIV>
> <DIV>} //close promptFertaliser function</DIV>
> <DIV> </DIV>
> <DIV><BR>void Tanker::PromptHectre()<BR>{<BR>float =
> EnterArea=3D0.0;<BR> cout=20
> << " Type a number, then press enter: " << endl;<BR> =
> cin=20
> >>=20
> EnterArea;<BR>  =
> ; =20
> } // close promptHectre function</DIV>
> <DIV> </DIV>
> <DIV><BR> float Tanker::CalculateArea(float Fertaliser, float=20
> Water)<BR>{<BR> float answer;<BR> answer =3Dsize =
> *=20
> fertaliser * water;<BR> return answer;<BR>} //close =
> calculateArea=20
> function</DIV>
> <DIV> </DIV>
> <DIV> </DIV>
> <DIV> </DIV>
> <DIV>void Tanker::Result()<BR>{<BR> cout << "Total Fertaliser =
> Used: "=20
> << fertaliser * 0.75 << " Litres " << =
> endl;<BR> cout=20
> << "Total Water Used: " << water * 0.25 << " Litres " =
> <<=20
> endl;<BR> cout << "Total Volume Used: " << pi * radius =
> * radius=20
> * length << " Litres " << endl;<BR> cout << =
> "Total Area=20
> Used: " << area * width * length * hectre =
> <<=20
> "Hectres " <<=20
> endl;<BR> &nbs=
> p; =20
> } //close result function</DIV>
> <DIV> </DIV>
> <DIV><BR>void Tanker::PromptLiquid()<BR>{<BR>cout << "Enter =
> a=20
> number, then press enter: " << endl;<BR>} //close PromptLiquid=20
> function</DIV>
> <DIV> </DIV>
> <DIV><BR>Tanker::Tanker(float r, float l, float f)<BR>{<BR> =
> radius =3D=20
> r;<BR> length =3D l;<BR> fertaliserlevel =3D f;<BR>} //close =
> tanker=20
> constructor</DIV>
> <DIV> </DIV>
> <DIV>
> <P class=3DMsoPlainText style=3D"MARGIN: 0cm 0cm 0pt"><SPAN=20
> style=3D"mso-fareast-font-family: 'MS Mincho'"><FONT face=3D"Courier =
> New">[C++=20
> Error] Paddock.cpp(57): E2084 Parameter names are used only with a =
> function=20
> body<?xml:namespace prefix =3D o ns =3D =
> "urn:schemas-microsoft-com:office:office"=20
> /><o:p></o:p></FONT></SPAN></P>
> <P class=3DMsoPlainText style=3D"MARGIN: 0cm 0cm 0pt"><SPAN=20
> style=3D"mso-fareast-font-family: 'MS Mincho'"><FONT face=3D"Courier =
> New">[C++=20
> Error] Paddock.cpp(59): E2139 Declaration missing =
> ;<o:p></o:p></FONT></SPAN></P>
> <P class=3DMsoPlainText style=3D"MARGIN: 0cm 0cm 0pt"><SPAN=20
> style=3D"mso-fareast-font-family: 'MS Mincho'"><FONT face=3D"Courier =
> New">[C++=20
> Error] Paddock.cpp(60): E2451 Undefined symbol=20
> 'myTanker'<o:p></o:p></FONT></SPAN></P></DIV>
> <DIV> </DIV>
> <DIV> </DIV>
> <DIV>ps: it is the constructor, which i have diagnosed, but don't know =
> how to=20
> fix it. tried several things, but not liking it.<BR>giving me a=20
> migrain.<BR></DIV>
> <DIV> </DIV>
> <DIV> </DIV>
> <DIV> </DIV>
> <DIV></FONT> </DIV>
> <DIV><FONT face=3DArial size=3D2></FONT> </DIV>
> <DIV><FONT face=3DArial size=3D2></FONT> </DIV></BODY></HTML>
>
> ------=_NextPart_000_000A_01C418AE.3D93A980--
>
>
>
|
