Работа с std::vector содержащим указатели

std::vector<Entity *> items

void sort() {
        std::sort(items.begin(), items.end(), [](Entity *a, Entity *b) -> bool {
            if (a->getDrawOrder() == b->getDrawOrder()) {
                return a->getDrawOrder() + a->getWorldCoordinates().x + a->getWorldCoordinates().y <
                       b->getDrawOrder() + b->getWorldCoordinates().x + b->getWorldCoordinates().y;
            } else {
                return a->getDrawOrder() < b->getDrawOrder();
            }
        });
    }

int startSize = items.size(); //если эта пермеенная не используется вне цикла, то лучше item.size() перенести в цикл. Спасет от в случае, если после update вектор уменьшиться.
 
        for (int i = 0; i < startSize; ++i) {
            Entity *e = items.at(i); //метод at выдаст исключение, в случае выхода за предел массива.
            if (e == nullptr)
                /*exceprion invalid pointer */
            else 
                e->update();
            startSize = items.size(); //если при вызове update - количество айтемов изменится - это спасет от выхода из за предел массива
        }
 
        int endSize = items.size();

if (!itemsToRemove.empty()) {
    for (auto &e : itemsToRemove) { //не люблю auto юзать в таких ситуациях. некоторые IDE не всегда понимают, что под auto 
        //remove if item presents in group
        if (e->getGroupName() != "~") { 
            removeFromGroup(e->getGroupName(), e); 
        }
        //remove from actual vector
        items.erase(std::remove(items.begin(), items.end(), e), items.end()); //я бы юзал не вектор, а std::list. что бы не вызывался по сто раз realloc 
        //free object
        delete e; //ты удалил обьект по данному указателю. Но сам указатель то еще жив)
        e = nullptr; //сделаем указатель нулевым, это упростит отладку!
      
    }
    itemsToRemove.clear();
}

void sort() {
        std::sort(items.begin(), items.end(), [](Entity *a, Entity *b) /*-> bool тут это не обзятельно*/ {
            if (a == nullptr)//если укатель нулевой - игнорируем (хотя тут лучше экзепшен прописать, он не должен быть нуливым!!!!!)
                return false; //нуливой указатель идет в конец
            if (b == nullptr)
                return true //нуливой указатель идет в конец.
            if (a->getDrawOrder() == b->getDrawOrder()) {
                //сложные конструкции обромляй скобками, а то запутаться можно
                return (a->getDrawOrder() + a->getWorldCoordinates().x + a->getWorldCoordinates().y) <
                       (b->getDrawOrder() + b->getWorldCoordinates().x + b->getWorldCoordinates().y);
            } else {
                return a->getDrawOrder() < b->getDrawOrder();
            }
        });
    }

class A
{
public:
   A(){}
   int a;
}
 
int main()
{
   A *a = nullptr;
   std::cout << a->a;
}

int comparEntity(const void* v_a, const void* v_b)
{
    const Entity *a = *(static_cast<const Entity * const *>(v_a));
    const Entity *b = *(static_cast<const Entity * const *>(v_b));
 
    if (a->getDrawOrder() == b->getDrawOrder())
    {
        return (a->getDrawOrder() + a->getWorldCoordinates().x + a->getWorldCoordinates().y) <
               (b->getDrawOrder() + b->getWorldCoordinates().x + b->getWorldCoordinates().y);
    } else
        return a->getDrawOrder() < b->getDrawOrder();
}
 
void sort() {
    if(items.empty())
        return;
    
    qsort(items.data(), items.size(), sizeof(Entity *), comparEntity);
}