0

----------------------

Repeated question

apples123
  • 15
  • 3
  • Doesn't `void getSize(int &xRes, int &yRes){}` need to be `void getSize(int &xRes, int &yRes){ xRes = this.xRes; yRes = this.yRes;}`? And indent your code properly so that it is more readable. – Spikatrix Jun 14 '15 at 06:02
  • sorry about the indentation, will keep that in mind. This was my first post. I am not sure by what you mean by this.xRes since I didn't make an object yet. – apples123 Jun 14 '15 at 06:10
  • You mean the compilation fails, not that the compiler crashed. – Ophir Gvirtzer Jun 14 '15 at 13:04

1 Answers1

3

In these lines

array2D(int xRes, int yRes){
    float **xtable;

you are declaring a local variable. The class member variable of the same name remains uninitialized and you use that later.

Remove the second line.

Also, the member variables xRes and yRes are not initialized either.

Use:

array2D(int xResIn, int yResIn) : xRes(xResIn), yRes(yResIn) {
   xtable = new float*[yRes];
   for(int i=0;i < yRes;i++) {
      xtable[i] = new float[xRes];
   }
}

Also, change

void getSize(int &xRes, int &yRes){}

to

void getSize(int &xResOut, int &yResOut)
{
   xResOut = this->xRes;
   yResOut = this->yRes;
}

As you expand this class, keep in mind The Rule of Three and implement the copy constructor and copy assignment operator.

array2D(array2D const& copy) { ... }
array2D& operator=(array2D const& rhs) { ... }
Community
  • 1
  • 1
R Sahu
  • 200,579
  • 13
  • 144
  • 260
  • When I remove that line, it says that float **xtable is a private member, which makes sense. Should I have an initization step somewhere before I start anything? – apples123 Jun 14 '15 at 06:08
  • @digon1631, that doesn't make sense. Which compiler are you using? – R Sahu Jun 14 '15 at 06:11
  • It actually is compiling now but crashing when it runs. My compiler is GNU GCC Compiler. – apples123 Jun 14 '15 at 06:15
  • Ahh I see it now. It compiles and works. I just failed to initialize the variables. Thanks! – apples123 Jun 14 '15 at 06:21
  • what does the "this-->xRes" portion mean? I understand that it is pointing to soemthing, but why use the word this? – apples123 Jun 14 '15 at 06:42
  • @digon1631, It's another syntax for accessing members of the current object. You may use `xResOut = xRes;` also. – R Sahu Jun 14 '15 at 07:07