Dynamic allocating an object

Question

Here is my code, I was trying to allocate memory to an object by asking size from the user. I dont know where I am going wrong. The error the compiler is giving me is "void value not ignored as it ought to be". Please help me. Thanks in advance.

#include <iostream> using namespace std; class test { char name[20]; char address[30]; char desig[20]; public: void getData(); void display(); ~test() { cout << "Destructor is invoked"; } }; void test::getData() { cout << "Enter your name" << endl; cin >> name; cout << "Enter address" << endl; cin >> address; cout << "Enter designation" << endl; cin >> desig; } void test::display() { cout << "name" << endl; cout << name << endl; cout << "address" << endl; cout << address; cout << "Enter designation" << endl; cout << desig; } int main() { int s; cout << "Enter the size of an array" << endl; cin >> s; test* p1 = new test[s]; for (int i = 0; i < s; i++) { *p1[i].getData(); } for (int i = 0; i < s; i++) { *p1[i].display(); } delete[] p1; return 0; }

barak manos · Accepted Answer · 2014-01-30 20:42:33Z

1

You need to change:

*p1[i]

To:

p1[i]

answered Jan 30, 2014 at 20:42

barak manos

30.3k10 gold badges67 silver badges117 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

jrok · Accepted Answer · 2014-01-30 20:42:36Z

1

Here:

*p1[i].display();

You're calling member function display that returns void and try to dereference that.

answered Jan 30, 2014 at 20:42

jrok

55.5k10 gold badges114 silver badges146 bronze badges

1 Comment

user2249683 Over a year ago

+1 Actually the correct answer - to emphasize: *(p1[i].display())

Collectives™ on Stack Overflow

Dynamic allocating an object

2 Answers 2

Comments

1 Comment

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Related