Initializing a class member variable which is a reference to an array

[amparikh]

I have some test code which demonstrates the problem. I know I could
solve this by just returning a pointer, but I better use a reference.

In real code, what I actually want to return is a reference to an array
of function pointers. But the code below is good enough to show the
problem.

Thanks.

#define MAX_DEC 11

static char (&MyFunc())[MAX_DEC]
{
static char init[] = "Hihowareya ";
return init;
}

class A
{
private :
char (&Ptr)[MAX_DEC];
public:
A() : Ptr(::MyFunc())
{
}

};

int main()
{
A a;
}

The error I get is "'A::Ptr' : initialization of reference member
requires a temporary variable"

[Victor Bazarov]

am******@gmail. com wrote:

I have some test code which demonstrates the problem. I know I could
solve this by just returning a pointer, but I better use a reference.

In real code, what I actually want to return is a reference to an array
of function pointers. But the code below is good enough to show the
problem.

Thanks.

#define MAX_DEC 11

static char (&MyFunc())[MAX_DEC]
{
static char init[] = "Hihowareya ";
return init;
}

class A
{
private :
char (&Ptr)[MAX_DEC];
public:
A() : Ptr(::MyFunc())
{
}

};

int main()
{
A a;
}

The error I get is "'A::Ptr' : initialization of reference member
requires a temporary variable"

Yes, Visual C++ is buggy. The code is fine. I suggest you report this as
the ugly bug it is. Ask in 'microsoft.publ ic.vc.language' how to do that.

V
--
Please remove capital As from my address when replying by mail

[Alf P. Steinbach]

* am******@gmail. com:

I have some test code which demonstrates the problem. I know I could
solve this by just returning a pointer, but I better use a reference.

In real code, what I actually want to return is a reference to an array
of function pointers. But the code below is good enough to show the
problem.

Thanks.

#define MAX_DEC 11

static char (&MyFunc())[MAX_DEC]
{
static char init[] = "Hihowareya ";
return init;
}

class A
{
private :
char (&Ptr)[MAX_DEC];
public:
A() : Ptr(::MyFunc())
{
}

};

int main()
{
A a;
}

The error I get is "'A::Ptr' : initialization of reference member
requires a temporary variable"

Technically this is a compiler error; the code should compile, and does
compile with g++ and Comeau online.

However, I'd count this compiler error as a blessing... Don't /do/ this
kind of thing!

At the very least, put your array inside a struct (that will probably
make it compile, and also get rid of the lack of a type name). And you
shouldn't really have reference members, they're mostly Evil. But most
of all, unless you're interfacing to C code, an array of function
pointers says you're missing a class with virtual functions, which
should be used instead -- this is C++, and the replacement of arrays
of function pointers, with classes, is the main ++ in C++.

--
A: Because it messes up the order in which people normally read text.
Q: Why is it such a bad thing?
A: Top-posting.
Q: What is the most annoying thing on usenet and in e-mail?

[amparikh]

Victor Bazarov wrote:

am******@gmail. com wrote:

I have some test code which demonstrates the problem. I know I could
solve this by just returning a pointer, but I better use a reference.

In real code, what I actually want to return is a reference to an array
of function pointers. But the code below is good enough to show the
problem.

Thanks.

#define MAX_DEC 11

static char (&MyFunc())[MAX_DEC]
{
static char init[] = "Hihowareya ";
return init;
}

class A
{
private :
char (&Ptr)[MAX_DEC];
public:
A() : Ptr(::MyFunc())
{
}

};

int main()
{
A a;
}

The error I get is "'A::Ptr' : initialization of reference member
requires a temporary variable"
Yes, Visual C++ is buggy. The code is fine. I suggest you report this as
the ugly bug it is. Ask in 'microsoft.publ ic.vc.language' how to do that.

Ok thanks. Interestingly enough I see this problem on all of them,
VC6.0, VS.NET 2003 and VS.NET 2005.

V
--
Please remove capital As from my address when replying by mail

[amparikh]

Alf P. Steinbach wrote:

* am******@gmail. com:

I have some test code which demonstrates the problem. I know I could
solve this by just returning a pointer, but I better use a reference.

In real code, what I actually want to return is a reference to an array
of function pointers. But the code below is good enough to show the
problem.

Thanks.

#define MAX_DEC 11

static char (&MyFunc())[MAX_DEC]
{
static char init[] = "Hihowareya ";
return init;
}

class A
{
private :
char (&Ptr)[MAX_DEC];
public:
A() : Ptr(::MyFunc())
{
}

};

int main()
{
A a;
}

The error I get is "'A::Ptr' : initialization of reference member
requires a temporary variable"
Technically this is a compiler error; the code should compile, and does
compile with g++ and Comeau online.

However, I'd count this compiler error as a blessing... Don't /do/ this
kind of thing!

At the very least, put your array inside a struct (that will probably
make it compile, and also get rid of the lack of a type name). And you
shouldn't really have reference members, they're mostly Evil. But most
of all, unless you're interfacing to C code, an array of function
pointers says you're missing a class with virtual functions, which
should be used instead -- this is C++, and the replacement of arrays
of function pointers, with classes, is the main ++ in C++.

Actually they are pointers to static CreateInstance Functions of
different classes (denoting different h/w types supported) that are
supported.

and the class where this is implemented does the abstraction.

--
A: Because it messes up the order in which people normally read text.
Q: Why is it such a bad thing?
A: Top-posting.
Q: What is the most annoying thing on usenet and in e-mail?

[Alf P. Steinbach]

* am******@gmail. com:

* Alf P. Steinbach:

At the very least, put your array inside a struct (that will probably
make it compile, and also get rid of the lack of a type name). And you
shouldn't really have reference members, they're mostly Evil. But most
of all, unless you're interfacing to C code, an array of function
pointers says you're missing a class with virtual functions, which
should be used instead -- this is C++, and the replacement of arrays
of function pointers, with classes, is the main ++ in C++.

Actually they are pointers to static CreateInstance Functions of
different classes (denoting different h/w types supported) that are
supported.

and the class where this is implemented does the abstraction.

In that case going the object route only buys you a lot of flexibility,
which you might not need, at the cost of at least one indirection.

--
A: Because it messes up the order in which people normally read text.
Q: Why is it such a bad thing?
A: Top-posting.
Q: What is the most annoying thing on usenet and in e-mail?