Thread: Problem with semicolon.

I am working from the book SAMS teach yourself C++ and I am encountering the following problem while trying to use codeblocks to build the following code.

I presume the error on line 22 in main.cpp has to do with the errors in include\Cat.h but the other one one line 25 I have no idea about what is causing it as well as the 2 errors in include\Cat.h.

Hoped someone here could help me out.

main.cpp

Code:

// Demonstration of inline functions and inclusion of header files.
// Be sure to include the header files!
#include "include\Cat.h"

Cat::Cat(int initialAge)        // Constructor
{
    itsAge = initialAge;
}

Cat::~Cat()                     // Deconstructor
{

}

// Create a cat, set its age, have it meow, tell us its age, then make it meow again
int main()
{
    Cat Frisky(5);
    Frisky.Meow();

    std::cout << "Frisky is a cat who is ";
    std::cout << GetAge() << " years old.\n";

    Frisky.Meow();
    Frisky SetAge (7);

    std::cout << "Now Frisky is ";
    std::cout << GetAge() << " years old.";

    return 0;
}

Errors
Line 22, 'GetAge' was not declared in this scope
Line 25, expected ';' before 'SetAge'

include\Cat.h

Code:

#include<iostream>

class Cat
{
    public:
        Cat(int initialAge);
        ~Cat();
        int GetAge const { return itsAge;}               // Inline!
        void SetAge(int age) { itsAge = age;}            // Inline!
        void Meow() const { std::cout << "Meow!\n";}     // Inline!
    private:
        int itsAge;
};

Errors
Line 8, expected ';' before 'const'
Line 9, expected ';' before 'void'

Code:

        int GetAge const { return itsAge;}               // Inline!

This function is missing it's parentheses.

Also, the book you are using is very poor. You should be looking at either Accelerated C++ or PPP by Bjarne Stroustrup. They are widely regarded as the better books to learn from.

Originally Posted by Ushakal

This function is missing it's parentheses.

I presume you mean I need to re-write it as:

Code:

int GetAge const { return (itsAge);}               // Inline!

However that does not change a thing the same errors keep coming while trying to build.

Originally Posted by darren78

Also, the book you are using is very poor. You should be looking at either Accelerated C++ or PPP by Bjarne Stroustrup. They are widely regarded as the better books to learn from.

I could not find any other books to learn from. Unless I wanted to wait for 1 month for them to be delivered.

Originally Posted by Kitt3n

I could not find any other books to learn from. Unless I wanted to wait for 1 month for them to be delivered.

Where do you live? Some small remote African village?

Originally Posted by darren78

Where do you live? Some small remote African village?

It is not my fault Americans are that slow sending packages to Europe.

Originally Posted by Ushakal

No, you do not need brackets when returning a value.
All functions must include parentheses, whether or not they have any parameters, as part of their signature. Do you see what the problem is now?

I almost wanted to say yes but editing the code like this still results in the same errors.

Code:

int GetAge(const) { return itsAge;}

I thought you meant that I was declaring a function without using parentheses behind it.

Originally Posted by Kitt3n

Code:

int GetAge(const) { return itsAge;}

I thought you meant that I was declaring a function without using parentheses behind it.

The const keyword should not be within the parentheses.

You got it right with this function:

Code:

void Meow() const { std::cout << "Meow!\n";}     // Inline!

If you write the GetAge function's signature in the same way, it should be fine.

I just made that change but the result is still the same with the errors.

Code:

int GetAge() const { return itsAge;}

Originally Posted by Kitt3n

I just made that change but the result is still the same with the errors.

Code:

int GetAge() const { return itsAge;}

That particular error should be fixed, but you have more in your main function. I overlooked this before, sorry:

Code:

    std::cout << GetAge() << " years old.\n";

...

    Frisky SetAge (7);

...

    std::cout << GetAge() << " years old.";

SetAge and GetAge are both member functions of the Frisky object. As such, you need to use the dot operator to access the member functions of this particular object's instance.

Code:

Frisky.SetAge(7);
Frisky.GetAge();

I overlooked those as well, stupidity I guess. But even with all those changes made codeblocks still gives me the exact same errors as given in the first posting they did not seem to have changed anything.

Hmm, I can compile your code after fixing those errors... Can you post what you have now?