C++Talk.NET

NewToCPP · Guest

I have the following code. It complains that there is Possible use of
null pointer. How to handle "operator []" returning reference to the
null pointer?

class my_data
{
private:
my_period* period;

public:
my_period& operator[] (unsigned int);

};

my_period& my_data::operator[] (unsigned int i)
{
if (period==0)
{
print_error("error");
}

return period[i];
}

Phlip · Guest

NewToCPP wrote:

PasalicZaharije · Guest

Your operator[] can return reference to NULL pointer - try to put

if (period == NULL)
throw ...; // throw something
else return period[i];

Or, just put

return period[i];

without error checking, and make member function
at(int i) that throw exception on out-of-range and
null check (something like C++ libs).

Smarter solution: do not use pointer - use std::vector