Created osm.h, re-factoring, unit test script bug fix
[situare] / src / map / mapscene.cpp
index d1bb80c..556ac85 100644 (file)
@@ -27,9 +27,6 @@
 
 #include "mapscene.h"
 
-const int MAP_MIN_PIXEL_Y = 0;
-const int MAP_MAX_PIXEL_Y = MAX_TILES_PER_SIDE * TILE_SIZE_Y - 1;
-
 MapScene::MapScene(QObject *parent)
     : QGraphicsScene(parent)
     , m_isRemoveStackedTilesRunning(false)
@@ -38,19 +35,16 @@ MapScene::MapScene(QObject *parent)
     qDebug() << __PRETTY_FUNCTION__;
 
     setBackgroundBrush(Qt::lightGray);
-    setSceneRect(MAP_SCENE_MIN_PIXEL_X, MAP_MIN_PIXEL_Y, MAP_SCENE_MAX_PIXEL_X, MAP_MAX_PIXEL_Y);
+    setSceneRect(QRect(QPoint(MAP_SCENE_MIN_PIXEL_X, OSM_MAP_MIN_PIXEL_Y),
+                       QPoint(MAP_SCENE_MAX_PIXEL_X, OSM_MAP_MAX_PIXEL_Y)));
 }
 
 void MapScene::addTile(int tileZoomLevel, QPoint tileNumber, const QPixmap &image, int viewZoomLevel)
 {
     qDebug() << __PRETTY_FUNCTION__;
 
-    // tile might already be in the scene if:
-    //    - expired tile was returned from the cache to be temporarily displayed while downloading
-    //      the fresh one.
-    //    - upper level tile is not cleaned after zooming in and back to out (not scrolled enough)
-    //    - upper level tile was earlier returned from cache while downloading the requested tile
-    //      and user has zoomed up to that level
+    // tile might already be in the scene if expired tile was returned from the cache to be
+    // temporarily shown while downloading the fresh one.
     QString hashKey = MapEngine::tilePath(tileZoomLevel, tileNumber.x(), tileNumber.y());
     MapTile *oldTile = tileInScene(hashKey);
     if (oldTile)
@@ -64,7 +58,10 @@ void MapScene::addTile(int tileZoomLevel, QPoint tileNumber, const QPixmap &imag
     m_mapTilesInScene.insert(hashKey, tile);
     addItem(tile);
 
+    qDebug() << __PRETTY_FUNCTION__ << "tiles count:" << m_mapTilesInScene.count();
+
     enqueueRemoveStackedTiles(tile);
+    removeOtherLevelTiles();
 }
 
 void MapScene::enqueueRemoveStackedTiles(MapTile *newTile)
@@ -84,9 +81,8 @@ void MapScene::moveIntersectingItemsHorizontally(QRect from, int distance)
 
     QList<QGraphicsItem *> spanItems = items(from, Qt::IntersectsItemBoundingRect);
     foreach (QGraphicsItem *item, spanItems) {
-        if (dynamic_cast<MapTile *>(item))
-            continue;
-        item->moveBy(distance, 0);
+        if (!dynamic_cast<MapTile *>(item))
+            item->moveBy(distance, 0);
     }
 }
 
@@ -113,44 +109,65 @@ void MapScene::runNextStackedTilesRemoval()
         m_isRemoveStackedTilesRunning = false;
 }
 
+void MapScene::removeOtherLevelTiles()
+{
+    qDebug() << __PRETTY_FUNCTION__;
+
+    for (int x = m_viewTilesGrid.left(); x <= m_viewTilesGrid.right(); x++) {
+        for (int y = m_viewTilesGrid.top(); y <= m_viewTilesGrid.bottom(); y++) {
+            if (!m_mapTilesInScene.contains(MapEngine::tilePath(m_zoomLevel, x, y)))
+                return;
+        }
+    }
+
+    foreach(MapTile *tile, m_mapTilesInScene) {
+        if (tile->zoomLevel() != m_zoomLevel) {
+            removeTile(tile);
+            qDebug() << __PRETTY_FUNCTION__ << "removed other level tile";
+        }
+    }
+}
+
 void MapScene::removeOutOfViewTiles(QRect tilesGrid, int zoomLevel)
 {
     qDebug() << __PRETTY_FUNCTION__;
 
-    QList<QGraphicsItem *> viewTiles = items(m_tilesSceneRect, Qt::IntersectsItemBoundingRect);
-    QList<QGraphicsItem *> allTiles = items();
+    QList<QGraphicsItem *> viewItems = items(m_tilesSceneRect, Qt::IntersectsItemBoundingRect);
+    QList<QGraphicsItem *> allItems = items();
 
     //Remove tiles which are in view from allTiles
-    foreach (QGraphicsItem *tile, viewTiles)
-        allTiles.removeOne(tile);
+    foreach (QGraphicsItem *item, viewItems)
+        allItems.removeOne(item);
 
     // note: add 1 so odd values are rounded up
     int tilesGridWidthHalf = (tilesGrid.width() + 1) / 2;
 
-    if (tilesGrid.right() > (MapEngine::tileMaxValue(zoomLevel) - tilesGridWidthHalf + GRID_PADDING)) {
+    // if view is near east limit of the map, then there is duplicate tiles also on the opposite
+    // side of the world which are removed from allTiles
+    if (tilesGrid.right() > (MapEngine::tileMaxIndex(zoomLevel) - tilesGridWidthHalf + MAP_GRID_PADDING)) {
         QRect oppositeRect = m_tilesSceneRect;
-        oppositeRect.translate(-MAP_PIXELS_X, 0);
-        QList<QGraphicsItem *> oppositeTiles = items(oppositeRect, Qt::IntersectsItemBoundingRect);
-        foreach (QGraphicsItem *tile, oppositeTiles)
-            allTiles.removeOne(tile);
+        oppositeRect.translate(-OMS_MAP_PIXELS_X, 0);
+        QList<QGraphicsItem *> oppositeItems = items(oppositeRect, Qt::IntersectsItemBoundingRect);
+        foreach (QGraphicsItem *item, oppositeItems)
+            allItems.removeOne(item);
     }
 
-    if (tilesGrid.left() < (tilesGridWidthHalf - GRID_PADDING)) {
+    // if view is near west limit of the map, then there is duplicate tiles also on the opposite
+    // side of the world which are removed from allTiles
+    if (tilesGrid.left() < (tilesGridWidthHalf - MAP_GRID_PADDING)) {
         QRect oppositeRect = m_tilesSceneRect;
-        oppositeRect.translate(MAP_PIXELS_X, 0);
-        QList<QGraphicsItem *> oppositeTiles = items(oppositeRect, Qt::IntersectsItemBoundingRect);
-        foreach (QGraphicsItem *tile, oppositeTiles)
-            allTiles.removeOne(tile);
+        oppositeRect.translate(OMS_MAP_PIXELS_X, 0);
+        QList<QGraphicsItem *> oppositeItems = items(oppositeRect, Qt::IntersectsItemBoundingRect);
+        foreach (QGraphicsItem *item, oppositeItems)
+            allItems.removeOne(item);
     }
 
     //Remove tiles out of view
-    foreach (QGraphicsItem *tile, allTiles) {
-        MapTile *tileToRemove = dynamic_cast<MapTile *>(tile);
-        if (tileToRemove)
-            removeTile(tileToRemove);
+    foreach (QGraphicsItem *item, allItems) {
+        MapTile *tile = dynamic_cast<MapTile *>(item);
+        if (tile)
+            removeTile(tile);
     }
-
-    qWarning() << __PRETTY_FUNCTION__ << "items in scene:" << items().count();
 }
 
 void MapScene::removeStackedTiles(MapTile *newTile)
@@ -172,7 +189,6 @@ void MapScene::removeStackedTiles(MapTile *newTile)
             }
         }
     }
-    qWarning() << __PRETTY_FUNCTION__ << "items in scene:" << items().count();
 }
 
 void MapScene::removeTile(MapTile *tile)
@@ -185,17 +201,19 @@ void MapScene::removeTile(MapTile *tile)
     removeItem(tile);
     m_removeStackedTilesList.removeAll(tile);
     delete tile;
+
+    qDebug() << __PRETTY_FUNCTION__ << "tiles count:" << m_mapTilesInScene.count();
 }
 
 void MapScene::setSceneVerticalOverlap(int viewHeight, int zoomLevel)
 {
     qDebug() << __PRETTY_FUNCTION__;
 
-    int overlap = viewHeight / 2 * (1 << (MAX_MAP_ZOOM_LEVEL - zoomLevel));
+    int overlap = viewHeight / 2 * (1 << (OSM_MAX_ZOOM_LEVEL - zoomLevel));
 
     QRect rect = sceneRect().toRect();
-    rect.setTop(MAP_MIN_PIXEL_Y - overlap);
-    rect.setBottom(MAP_MAX_PIXEL_Y + overlap);
+    rect.setTop(OSM_MAP_MIN_PIXEL_Y - overlap);
+    rect.setBottom(OSM_MAP_MAX_PIXEL_Y + overlap);
     setSceneRect(rect);
 }
 
@@ -212,33 +230,47 @@ void MapScene::setTilesDrawingLevels(int zoomLevel)
     }
 }
 
+void MapScene::setTilesGrid(QRect grid)
+{
+    qDebug() << __PRETTY_FUNCTION__ ;
+
+    m_viewTilesGrid = grid;
+}
+
+void MapScene::setZoomLevel(int zoomLevel)
+{
+    qDebug() << __PRETTY_FUNCTION__ ;
+
+    m_zoomLevel = zoomLevel;
+}
+
 void MapScene::spanItems(int zoomLevel, QPoint sceneCoordinate, QSize viewSize)
 {
     qDebug() << __PRETTY_FUNCTION__;
 
     // create rects for left and right side
     QRect leftRect = sceneRect().toRect(); // this way we get the horizontal limits of the scene
-    leftRect.setTop(MAP_MIN_PIXEL_Y);
-    leftRect.setBottom(MAP_MAX_PIXEL_Y);
+    leftRect.setTop(OSM_MAP_MIN_PIXEL_Y);
+    leftRect.setBottom(OSM_MAP_MAX_PIXEL_Y);
     QRect rightRect = leftRect;
 
     // calculate current horizontal area shown on the view
-    int viewSceneWidth = (1 << (MAX_MAP_ZOOM_LEVEL - zoomLevel)) * viewSize.width();
+    int viewSceneWidth = (1 << (OSM_MAX_ZOOM_LEVEL - zoomLevel)) * viewSize.width();
     int viewSceneLeft = sceneCoordinate.x() - viewSceneWidth / 2;
     int viewSceneRight = sceneCoordinate.x() + viewSceneWidth / 2;
 
     // limit rects to include only area which really must be moved
-    leftRect.setRight(-1 - (MAP_PIXELS_X - 1 - viewSceneRight));
-    rightRect.setLeft(MAP_PIXELS_X + viewSceneLeft);
+    leftRect.setRight(-1 - (OMS_MAP_PIXELS_X - 1 - viewSceneRight));
+    rightRect.setLeft(OMS_MAP_PIXELS_X + viewSceneLeft);
 
     Q_ASSERT_X(leftRect.right() < viewSceneLeft, "spanning rect right value", "move rect is in the view area");
     Q_ASSERT_X(rightRect.left() > viewSceneRight, "spanning rect left value", "move rect is in the view area");
 
     // move all items which intersects the rects
     if (leftRect.left() < leftRect.right())
-        moveIntersectingItemsHorizontally(leftRect, MAP_PIXELS_X);
+        moveIntersectingItemsHorizontally(leftRect, OMS_MAP_PIXELS_X);
     if (rightRect.left() < rightRect.right())
-        moveIntersectingItemsHorizontally(rightRect, -MAP_PIXELS_X);
+        moveIntersectingItemsHorizontally(rightRect, -OMS_MAP_PIXELS_X);
 }
 
 void MapScene::tilesSceneRectUpdated(QRect tilesSceneRect)