Question

5. What’s wrong with the code below? Feel free to compile and run the program to investigate. [6]

#include <iostream>

using namespace std;

class HW1

{

public:

   char *word;

   HW1() { }

   ~HW1()

   {

delete [] word;

   }

};

void foo(HW1 a)

{

//some operation here

}

int main()

{

HW1 a;

a.word = new char[80];

strcpy(a.word, "abc");

foo(a);

return 0;

}

Answer 1

#include <iostream>
#include <cstring>
using namespace std;
class HW1
{
    public:
       char *word;
       HW1() { }
       ~HW1()
       {}
};

void foo(HW1 a)
{
    //some operation here
}

int main()
{
    HW1 a;
    // Memory for a.word is allocated in main()
    a.word = new char[80];
    strcpy(a.word, "abc");
    foo(a);
    // So, you should fee memory in main() method only
    delete[] a.word;
    return 0;
}