stack fix, creature names and health bar, still buggy

src/otclient/core/creature.cpp

@@ -1,8 +1,9 @@
-#include "creature.h"
+ #include "creature.h"
 #include "datmanager.h"
 #include <framework/graphics/graphics.h>
 #include <framework/graphics/framebuffer.h>
 #include "game.h"
+#include <framework/graphics/fontmanager.h>
 
 Creature::Creature() : Thing(THING_CREATURE)
 {
@@ -41,9 +42,11 @@ void Creature::draw(int x, int y)
         g_graphics.bindBlendFunc(BLEND_NORMAL);
         g_graphics.bindColor(Color::white);
     }
+}
 
+void Creature::drawName(int x, int y)
+{
     // health bar
-    // TODO: draw outside framebuffer
     Color healthColor = Color::black;
     if(m_healthPercent > 60 && m_healthPercent <= 100) {
         healthColor.setRed(0.0f);
@@ -59,7 +62,7 @@ void Creature::draw(int x, int y)
         healthColor.setGreen(0.00f);
     }
 
-    Rect healthRect = Rect(x - 14, y - 11, 27, 4);
+    Rect healthRect = Rect(x-(14.5), y-2, 27, 4);
 
     g_graphics.bindColor(Color::black);
     g_graphics.drawBoundingRect(healthRect);
@@ -69,6 +72,10 @@ void Creature::draw(int x, int y)
 
     // restore white color
     g_graphics.bindColor(Color::white);
+
+    // name
+    FontPtr font = g_fonts.getDefaultFont();
+    font->renderText(m_name, Rect(x-50, y-16, 100, 16), AlignTopCenter);
 }
 
 const ThingAttributes& Creature::getAttributes()

src/otclient/core/creature.h

@@ -19,6 +19,7 @@ public:
     virtual ~Creature() { }
 
     virtual void draw(int x, int y);
+    void drawName(int x, int y);
 
     void setName(const std::string& name) { m_name = name; }
     void setHealthPercent(uint8 healthPercent) { m_healthPercent = healthPercent; }

src/otclient/core/map.cpp

@@ -6,7 +6,7 @@
 
 Map g_map;
 
-void Map::draw(int x, int y)
+void Map::draw(const Rect& rect)
 {
     if(!m_framebuffer)
         m_framebuffer = FrameBufferPtr(new FrameBuffer(15*32, 11*32));
@@ -50,17 +50,13 @@ void Map::draw(int x, int y)
             if(iz == drawFloorStop)
                 break;
 
-            for(int step = 0; step < 4; ++step) {
+            // +1 in draws cause 64x64 items may affect view.
 
-
+            for(int ix = -7+(playerPos.z-iz); ix < + 8+7; ++ix) {
-                // +1 in draws cause 64x64 items may affect view.
+                for(int iy = -5+(playerPos.z-iz); iy < + 6+7; ++iy) {
-
+                    Position itemPos = Position(playerPos.x + ix, playerPos.y + iy, iz);
-                for(int ix = -7+(playerPos.z-iz); ix < + 8+7; ++ix) {
+                    if(const TilePtr& tile = m_tiles[itemPos])
-                    for(int iy = -5+(playerPos.z-iz); iy < + 6+7; ++iy) {
+                        tile->draw((ix + 7 - (playerPos.z-iz))*32, (iy + 5 - (playerPos.z-iz))*32, 0);
-                        Position itemPos = Position(playerPos.x + ix, playerPos.y + iy, iz);
-                        if(const TilePtr& tile = m_tiles[itemPos])
-                            tile->draw((ix + 7 - (playerPos.z-iz))*32, (iy + 5 - (playerPos.z-iz))*32, step);
-                    }
                 }
             }
         }
@@ -73,7 +69,20 @@ void Map::draw(int x, int y)
     m_framebuffer->unbind();
 
     g_graphics.bindColor(Color::white);
-    m_framebuffer->draw(Rect(x, y, g_graphics.getScreenSize()));
+    m_framebuffer->draw(rect);
+
+    // calculate stretch factor
+    float horizontalStretchFactor = (rect.width() - rect.x()) / (float)(15*32);
+    float verticalStretchFactor = (rect.height() - rect.y()) / (float)(11*32);
+
+    // draw player names and health bars
+    for(int ix = -7; ix <= 7; ++ix) {
+        for(int iy = -5; iy <= 5; ++iy) {
+            Position itemPos = Position(playerPos.x + ix, playerPos.y + iy, playerPos.z);
+            if(const TilePtr& tile = m_tiles[itemPos])
+                tile->draw(((ix + 7)*32+5)*horizontalStretchFactor, ((iy + 5)*32 - 8)*verticalStretchFactor, 1);
+        }
+    }
 }
 
 void Map::addThing(ThingPtr thing, uint8 stackpos)

src/otclient/core/map.h

@@ -8,7 +8,7 @@
 class Map
 {
 public:
-    void draw(int x, int y);
+    void draw(const Rect& rect);
 
     void addThing(ThingPtr thing, uint8 stackpos = 0);
     ThingPtr getThing(const Position& pos, uint8 stackpos);

src/otclient/core/tile.cpp

@@ -13,23 +13,20 @@ Tile::Tile()
 
 void Tile::draw(int x, int y, int step)
 {
-    // STEP 0 = draw ground, top 1
+    // STEP 0 = draw map
-    // STEP 1 = top 2
+    // STEP 1 = draw creature names
-    // STEP 2 = top 3
+    // STEP 2 = draw speak
-    // STEP 3 = bottom, creatures, names, etc
 
     FontPtr font = g_fonts.getDefaultFont();
 
-    if(step == 0 && m_drawNextOffset != 0) {
-        logDebug("error with tile offset.");
-        return;
-    }
-
     if(step == 0) {
+        m_drawNextOffset = 0;
+
         if(m_ground)
             m_ground->draw(x, y);
 
-        for(const ThingPtr& thing : m_itemsTop) {
+        for(auto it = m_itemsTop.rbegin(), end = m_itemsTop.rend(); it != end; ++it) {
+            const ThingPtr& thing = *it;
             const ThingAttributes& thingAttributes = thing->getAttributes();
 
             if(thingAttributes.alwaysOnTopOrder == 1) {
@@ -40,7 +37,8 @@ void Tile::draw(int x, int y, int step)
             }
         }
 
-        for(const ThingPtr& thing : m_itemsTop) {
+        for(auto it = m_itemsTop.rbegin(), end = m_itemsTop.rend(); it != end; ++it) {
+            const ThingPtr& thing = *it;
             const ThingAttributes& thingAttributes = thing->getAttributes();
 
             if(thingAttributes.alwaysOnTopOrder == 2) {
@@ -50,21 +48,21 @@ void Tile::draw(int x, int y, int step)
             }
         }
 
-        for(const ThingPtr& thing : m_itemsBottom) {
+        for(auto it = m_itemsBottom.rbegin(), end = m_itemsBottom.rend(); it != end; ++it) {
+            const ThingPtr& thing = *it;
             const ThingAttributes& thingAttributes = thing->getAttributes();
             thing->draw(x - m_drawNextOffset, y - m_drawNextOffset);
             //font->renderText("B0", Rect(x + 5, y+5, 100, 100));
             m_drawNextOffset += thingAttributes.drawNextOffset;
         }
 
-        for(const ThingPtr& thing : m_creatures) {
+        for(auto it = m_creatures.rbegin(), end = m_creatures.rend(); it != end; ++it) {
-            const ThingAttributes& thingAttributes = thing->getAttributes();
+            const ThingPtr& thing = *it;
             thing->draw(x - m_drawNextOffset, y - m_drawNextOffset);
-
-            m_drawNextOffset += thingAttributes.drawNextOffset;
         }
 
-        for(const ThingPtr& thing : m_itemsTop) {
+        for(auto it = m_itemsTop.rbegin(), end = m_itemsTop.rend(); it != end; ++it) {
+            const ThingPtr& thing = *it;
             const ThingAttributes& thingAttributes = thing->getAttributes();
 
             if(thingAttributes.alwaysOnTopOrder == 3) {
@@ -74,24 +72,19 @@ void Tile::draw(int x, int y, int step)
             }
         }
 
-        for(const ThingPtr& thing : m_effects) {
+        for(auto it = m_effects.rbegin(), end = m_effects.rend(); it != end; ++it) {
+            const ThingPtr& thing = *it;
             thing->draw(x - m_drawNextOffset, y - m_drawNextOffset);
         }
-
-        m_drawNextOffset = 0;
     }
     else if(step == 1) {
+        for(auto it = m_creatures.rbegin(), end = m_creatures.rend(); it != end; ++it) {
+            const ThingPtr& thing = *it;
+            const CreaturePtr& creature = thing->asCreature();
+            creature->drawName(x - m_drawNextOffset, y - m_drawNextOffset);
+        }
 
 
-
-    }
-    else if(step == 2) {
-
-
-
-    }
-    else if(step == 3) {
-
     }
 }
 
@@ -100,18 +93,6 @@ void Tile::addThing(ThingPtr thing, uint8 stackpos)
     if(!thing)
         return;
 
-    //8308
-    //2526
-    //5296
-
-    const ThingAttributes& item1 = g_dat.getItemAttributes(8308);
-    const ThingAttributes& item2 = g_dat.getItemAttributes(2526);
-    const ThingAttributes& item3 = g_dat.getItemAttributes(5296);
-
-    int j = item1.alwaysOnTopOrder + item2.alwaysOnTopOrder + item3.alwaysOnTopOrder;
-    j++;
-
-
     if(thing->getPosition() == g_game.getLocalPlayer()->getPosition() + Position(-1, 0, 0) && thing->getAttributes().alwaysOnTop) {
         logDebug((int)thing->getId());
     }
@@ -123,16 +104,16 @@ void Tile::addThing(ThingPtr thing, uint8 stackpos)
             m_ground = thing;
         else {
             if(thingAttributes.alwaysOnTop)
-                m_itemsTop.push_front(thing);
+                m_itemsTop.push_back(thing);
             else
-                m_itemsBottom.push_front(thing);
+                m_itemsBottom.push_back(thing);
         }
     }
     else if(thing->asCreature()) {
-        m_creatures.push_front(thing);
+        m_creatures.push_back(thing);
     }
     else if(thing->asEffect()) {
-        m_effects.push_front(thing);
+        m_effects.push_back(thing);
     }
 }
 

src/otclient/otclient.cpp

@@ -209,7 +209,7 @@ void OTClient::render()
 {
     //TODO: UIMap for map drawing
     if(g_game.isOnline())
-        g_map.draw(0, 0);
+        g_map.draw(Rect(0, 0, g_graphics.getScreenSize()));
 
     // everything is rendered by UI components
     g_ui.render();