Marble/GeoData/GeoDataUse

    From KDE TechBase
    Revision as of 22:55, 12 August 2009 by Tampakrap (talk | contribs) (add comment on GeoDataFeature being dependent on derived classes)

    Using GeoData

    this page tries to summarise the use of GeoData classes in marble so that its implementation details can match the use cases, and understand if any what issues might arise from manipulating them otherwise.

    Use Cases

    The use cases identified so far are:

    1. use any GeoData class as a convenient data holder in APIs.
      It is expected in that regard that classes have shared data.
      Let's call this usecase the "ToolClass" usecase.
    2. use Expression error: Unrecognized word "x". as the root document of a "data file" in-memory representation, or even another grouping of information.
      In this use case, there would exist a "tree" of data matching a logical grouping of information.
      Let's call this usecase the "DataTree" usecase.

    Actual Implemented Use Details

    • GeoData classes have shared data, in the sense that e.g. copying a GeoDataFeature is a shallow copy, with deep copy happening when one of the instances need to modify a value. This is all for the "ToolClass", and not much useful for the "DataTree".
    • Reading a file with Expression error: Unrecognized word "x". creates a tree of data.

    This data is manipulated within a stack of Expression error: Unrecognized word "x".s and internally stored in QVector of value types like QVector<GeoDataFeature>. This is the canonical way to generate a "DataTree".

    • Expression error: Unrecognized word "x". as used by Expression error: Unrecognized word "x". appends all placemark from files it opens in a Expression error: Unrecognized word "x". (which is a QVector of Expression error: Unrecognized word "x".). This is another way to create a sort of "DataTree", relying on some "ToolClasses"
    • Model classes (implementing Qt's model-view framework that is) serve as interface to GeoData classes, passing them through QVariants

    TODO others??

    Common Issues and Pitfalls

    In the past or present, some issues have appeared and need to be remembered/adressed:

    1. memory ownership is an issue to be determined depending on use case.
    2. Compliance with KML spec is a priority, limiting differing needs.
    3. Parsing code is full of casts.
    4. storing e.g. an GeoDataPlacemark item in a QVector<GeoDataFeature> looks like a hairy solution, as QVector<T>::append will internally create a new T. In the shared data context this implies that we end up with an element of vector being a GeoDataFeature with a private member being a GeoDataPlacemarkPrivate.
    5. data which may be useful in the context of the "DataTree" use case are completely useless in the context of the "ToolClass" use case. This is true for featureId and other type defining attributes.
    6. GeoDataFeature needs a copy constructor per derived class and each derived class has to be a friend of GeoDataFeature. This make the base class dependent on each derived class which is bad design.

    Proposals

    • Using internally QVectors of pointers may limit the problem of inheritance. This does not limit the ability to use QVectors of Data types when needed elsewhere.
    • Providing some specialised "ToolClasses" like Expression error: Unrecognized word "x". would limit the interference between the use cases.