Question

I am using Qt 4.5 so do C++. I have a class like this

 class CClass1
    {
         private:
               struct stModelDetails
               {
                      QString name;
                      QString code;
                      ..... // only variables and no functions over here
               };
               QList<stModelDetails> m_ModelDetailsList;        

         public:
               QList<stModelDetails> getModelDetailsList();
               ...               
    };

In this I have functions that will populate the m_ModelDetailsList;

I have another class say CClassStructureUsage, where I will call the getModelDetailsList() function. Now my need is that I have to traverse the QList and obtain the name, code from each of the stModelDetails.

Now the problem is even the CClass1’s header file is included it is not able to identify the type of stModelDetails in CClassStructureUsage. When I get the structure list by

QList<stModelDetails> ModelList = obj->getModelInformationList();

it says stModelDetails : undeclared identifier.

How I can able to fetch the values from the structure? Am I doing anything wrong over here?

Answer 1

You’ve already gotten a couple of suggestions for how to attack your problem directly. I, however, would recommend stepping back for a moment to consider what you’re trying to accomplish here. First of all, you’ve said you only really want the name member of each stModelDetails item. Based on that, I’d start by changing the function to return only that:

QList<QString> GetModelDetailNames();

or, quite possibly:

QVector<QString> GetModelDetailNames();

The former has a couple of good points. First, it reduces the amount of data you need to copy. Second, it keeps client code from having to know more implementation details of CClass1. The latter retains those advantages, and adds a few of its own, primarily avoiding the overhead of a linked list in a situation where you haven’t pointed to any reason you’d want to use a linked list (and such reasons are really fairly unusual).

The alternative to that is to figure out why outside code needs access to that much of CClass1’s internal data, and whether it doesn’t make sense for CClass1 to provide that service directly instead of outside code needing to access its data.