Subject: [ggl] namespace model::
From: Bruno Lalande (bruno.lalande)
Date: 2010-12-01 13:40:31
I would put the convenience classes (_xy, _ll) in the same namespace as their "less convenient" counterparts, that is:
> The reason for existance of point_xy is that it provides the additional methods x(), y(), and a constructor taking two values. I don't use them. The library does not use them (besides in examples and some tests).
If it's so specific, let's not create a special namespace for it, and we don't even have to make the effort of aligning its name with other points:
point_2d => 2d::point
point_xy => 2d::point_xy
(same for 3d)
> But some library users might appreciate it. But actually I find the xy::point namespace a bit funny. There are no (or not yet) other geometries with _xy, like linestring_xy / xy::linestring or polygon_xy / xy::polygon (though for consistent with d2:: they might be created).
Why should we force ourselves to create polygon_xy and so on just because the 2d namespace contains all those. Not creating them at all is another reason for simply putting point_xy in the 2d namespace and not create any other namespace.
> The reason for existance of point_ll_deg is that it provides the additional methods lat(), lon(), and a constructor taking a special provision that lat and lon might be specified in any order. The library does not use them. It is designed some years ago, and because lat/lon is always confusing (y/x), it made some sense. However, I'm not sure about things like model::ll::degree. By the way, this latlong point is all in an extension.
I think we shouldn't try to push convenience too far here, or we will basically reproduce all the combination that template can produce with namespaces, which defeats the purpose of templates. Why not simply have a template class ll::point that lets the user define the unit to use as template parameter? It provides the "first level" of convenience from the general point class, which is sufficient I think.
Does my approach make sense?
-------------- next part --------------
An HTML attachment was scrubbed...
Geometry list run by mateusz at loskot.net