Initializing a pointer in another class

Question

I have a very simple problem. I am trying to learn C++ and I'm having a little problem. Here's the code

system.h

#include <iostream>
#include "processor.h"
using namespace std;

class sys
{
    public:
        int id;
        sys()
        {
            id=0;
        }
};

processor.h

#include <iostream>
using namespace std;
class proc
{
    public:
    const sys* s1;
    s1=new sys();
};

The error says

"error C2512: 'sys' : no appropriate default constructor available"

There is a default constructor.

I am a beginner at C++ so please explain what I am doing wrong. Thank You.

You really don't want to have a global `using namespace` directives in header files. — StoryTeller - Unslander Monica, Sep 22 '13 at 12:31

score 2 · Accepted Answer · edited May 23 '17 at 12:20

2

You do not initialize class members inside the class body, You do so in the class constructor.
const members are special members and they must be initialized in the Member initialization list.
Avoid using dynamically allocated memory as much as possible and if you must use smart pointers instead of raw pointers.

proc() : s1(new sys())
{
}

edited May 23 '17 at 12:20

Community

1
1

answered Sep 22 '13 at 12:31

Alok Save

202,538
53
430
533

score 0 · Answer 2 · answered Sep 22 '13 at 12:34

0

Since class proc is using class sys the include order should be the opposite:

// sys.h
class sys {
   ...
};

// process.h
#include "sys.h"

class proc {
   ...
};

answered Sep 22 '13 at 12:34

6502

112,025
15
165
265

Initializing a pointer in another class

2 Answers2