What is wrong with this simple C++ class program?

Question

Please help I'm beginner level student in C++ I'm failed to find a proper solution.I also added error image in this question.Please give me answer with proper solution.

#include <iostream>
#include <conio.h>

class test
{
    int no;
    static int count;
    public:
        void getval (int);
        void dispcount (void);

};

void test:: getval(int x)
{
    no = x;
    cout << "Number = " << no << endl;;
    count++;
}

void test::dispcount(void)
{
    cout << "Counten = " << count;
}
    int test::count;

int main()
{
    test t1,t2,t3;

    t1.dispcount();
    t2.dispcount();
    t3.dispcount();

    t1.getval(100);
    t2.getval(200);
    t3.getval(300);

    t1.dispcount();
    t2.dispcount();
    t3.dispcount();
    getch();
    return 0;
}

here is error.jpg

In order to find a proper solution, you need to identify the problem. Have you done so? — , Jun 19 '16 at 13:04
Do not include hard-to-read images in questions, like this, in your question. You must include error messages as plain text. As a C++ beginner, the best skill you need to learn is how to communicate your issues in a way that everyone else can understand you. — Sam Varshavchik, Jun 19 '16 at 13:04
Your error message seems pretty clear, change `cout` to `std::cout` and `endl` to `std::endl` like it tells you. — AliciaBytes, Jun 19 '16 at 13:05
You didnt put the image, anyhow you should post the complete error message as text not as image — 463035818_is_not_an_ai, Jun 19 '16 at 13:05
@KamalMrock Please do as asked for and post the verbatim error text. — πάντα ῥεῖ, Jun 19 '16 at 13:11
"Image added" after people tell you not to post image. Naughty. — , Jun 19 '16 at 13:11
Actually, this was my first question on this site.....and next time I will not add any image at the place of error ,only plain text, got it. — Kamal Mrock, Jun 19 '16 at 13:26

score 1 · Answer 1 · answered Jun 19 '16 at 13:07

1

Include directive

#include <iostream>
#include <conio.h>

using namespace std;
//..

Or include using declarations like

#include <iostream>
#include <conio.h>

using std::cout;
using std::endl;
//...

Or use qualified names as for example

void test:: getval(int x)
{
    no = x;
    std::cout << "Number = " << no << std::endl;
    ^^^^^^^^^                         ^^^^^^^^^^
    count++;
}

Identifiers cout and endl are declared in namespace std and not in the global namespace.

answered Jun 19 '16 at 13:07

Vlad from Moscow

301,070
26
186
335

4

Beginner C++ programmers should not really be exposed to [bad programming habits like "using namespace std;"](http://stackoverflow.com/questions/1452721/why-is-using-namespace-std-in-c-considered-bad-practice). – Sam Varshavchik Jun 19 '16 at 13:10
@SamVarshavchik I am sure that they start to learn C++ by using the using directive. And in any case any programmer should know all possibilities. – Vlad from Moscow Jun 19 '16 at 13:11
You are great..... – Kamal Mrock Jun 19 '16 at 13:17
@KamalMrock u should propably intialize test::count ,because you're using it in t1.dispcount() without intialization and that would give a warning set it equal to 0 – AnotherOne Jun 19 '16 at 13:57
@MekacherAnis It is initialized by the compiler to 0 as an object with the static storage duration. – Vlad from Moscow Jun 19 '16 at 14:10
@VladfromMoscow didn't know that's even possible :P , so does all compilers do so ? – AnotherOne Jun 19 '16 at 14:18
@MekacherAnis It is a requirement of the C++ Standard. – Vlad from Moscow Jun 19 '16 at 14:23
@VladfromMoscow thank's for the new info bro – AnotherOne Jun 19 '16 at 14:25

What is wrong with this simple C++ class program?

1 Answers1