COnfuSing PoIntErS

[anamol12]

Can some one help get started on an assignment involving pointers

Create a class called Pointer that has only 1 data member, namely a char pointer. With this class, define 3 functions constuctor, destructor, and display function. The constructor accepts and saves 3 passed in variable of type char. This must be done dynamically by allocating memory for the 3 passed in variable. The display function must display, th the screen, the contents of the memory using pointer notation. Do this in reverse order in which you save the data. Implement the class using the proper c++ syntax..


Proovide Driver function to invoke the display function

[anamol12]

THIS IS WHAT I COME UP WITH SO FAR
PLEASE HELP ME!!!

class Pointer {
private: char *p;

public: Pointer(char , char , char );
~Pointer();
Display();


};

Pointer::Pointer( char a, char b, char c)
{
p = new char;
*p = a;
}

//Pointer::~Pointer();
//{

// delete p;
//}

Pointer:isplay()
{
cout << &p << endl;

}

main()
{

Pointer *P;
P->Display();
//P->~Display();




return 0;

}

[MrWizard]

Okay there are a few errors. I'm glad you attempted it and asked for help instead of just asking someone to write it for you. Kudos to you.

When you are allocating the memory in the constructor you need to allocate room for 3 characters instead of 1.

The line should be:

p = new char[3];

To indicate we want 3 characters. You may also want to check the pointer against NULL after this to make sure you actually got the memory. Then you need to set each character to it like this.

p[0] = a;
p[1] = b;
p[2] = c;

Remember it is zero based so we don't index at 3. Basically we allocated using the [] to indicate we wanted an array of elements we need to change the delete line also to this:

delete [] p;

The display code should look like this:

cout << p[2] << p[1] << p[0] << endl;

Also don't call the destructor yourself! It will be called automatically at the end of the scope in which it was created in.

Also you have a pointer to the Pointer Class in main but you don't allocate space for it. Either change it to a regular instance or allocate room. There is no problem doing:

Pointer p( 'a', 'b', 'c' );
p.Display( );

or...

Pointer *p = new Pointer( 'a', 'b', 'c' );
p->Display( );
delete p;

I wrote this in a hurry so if any of it is confusing let me know.

Any questions?