accessing members inside of a private struct

Hey guys. I am trying to access the members inside of my struct, which is inside of my class in the private section like this:

Code:

---

class someClass
{
    public:
        unsigned getNumber() const { return someStruct.Number; }

    private:

        struct someStruct
        {
              unsigned Number;
        };
};

---

This code is in my header file.

This seems like it should work. I am getting things like this when I compile. warning: no return statement in function returning non-void

Quote:

---

Originally Posted by MyglyMP2

This seems like it should work. I am getting things like this when I compile. warning: no return statement in function returning non-void

---

The warning has zilch to do with the code you posted.

And what is it about compiler warnings that turns off people's brains? The warning is telling you EXACTLY what the problem is. Is it that you don't know what "non-void" means, or you don't know what "return" means? Or are you just not thinking?

Quote:

---

Originally Posted by MyglyMP2

This code is in my header file.

---

That seems like a no-no.

But why not post code that is a minimum compilable snippet that actually demonstrates exactly this issue?

Quote:

---

Originally Posted by Dave_Sinkula

That seems like a no-no.

---

It's a simple, const, inline function. I don't see anything wrong with it.

Quote:

---

Originally Posted by brewbuck

It's a simple, const, inline function. I don't see anything wrong with it.

---

Generally speaking.

OP:

Code:

---

        unsigned getNumber() const { return someStruct.Number; }

---

What is the object's name? (Not the class' name?)

Firstly, I appreciate help, and thank you for replies, but please try to be a little less insulting. Of course I know what return is and non-void. However I don't understand why it has nothing to do with the code that was posted. It is citing that line, and that function has a return type, and is returning something of that same type. I guess I must be missing something...

dont you have to declare the struct before the function? just try reversing the order of the private and public parts

Quote:

---

Originally Posted by MyglyMP2

Firstly, I appreciate help, and thank you for replies, but please try to be a little less insulting. Of course I know what return is and non-void. However I don't understand why it has nothing to do with the code that was posted. It is citing that line, and that function has a return type, and is returning something of that same type. I guess I must be missing something...

---

All too often, when bug hunting, I've found the bug is not where I think it is -- otherwise it really wouldn't be a bug. Quite often the actual bug is in code that I think has nothing to do with where I think it is. A missing header, etc., etc.

If this offends you, I humbly apologize. I know that code in headers, with several excpetions, is a bad thing to do. Posting all relevant code is a good thing to do. DWYW.

Quote:

---

Originally Posted by MyglyMP2

I guess I must be missing something...

---

someStruct.Number is meaningless.

EDIT: Sorry for jumping on you, but you didn't SAY that the warning was happening in the code you posted. You just said you were getting some warning. It wouldn't have been the first time a person has posted code that has no relation to the message they are seeing, so I jumped to conclusions. Sincere apologies.

I think it's because it only got the definition of the struct type, but not the member variable itself. Maybe it should go like this:

Code:

---

class someClass
{
    public:
        unsigned getNumber() const { return someMember.Number; }

    private:

        struct someStruct
        {
              unsigned Number;
        } someMember;
};

---