Почти тоже самое, просто обертка автоматически разлочит мьютекс при любой причине выхода из пространства её видимости,
даже при исключениях. А здесь нужно самому помнить об этом, а исключение оставит мьютекс заблокированным.



Можно и синглтоном, а лучще через умный указатель QSharedPointer.

PS. Жмем кнопку спасибо

Например, что-то такое

template< typename _Key, typename _Value >
class MySafeMap
{
    QMutex m_mutex;
    QMap< _Key, _Value > m_map;

public:
    void insert ( const _Key & key, const _Value & value )
    {
        QMutexLocker locker( &m_mutex );
        m_map.insert( key, value );
    }

    _Value value ( const _Key & key ) const
    {
        QMutexLocker locker( &m_mutex );
        m_map.insert( key, value );
    }

    // и т.д. все небходимые методы
}

или

template< typename _Key, typename _Value >
class MySafeMap
{
    QReadWriteLock m_mutex;
    QMap< _Key, _Value > m_map;

public:
    void insert ( const _Key & key, const _Value & value )
    {
        QWriteLocker locker( &m_mutex );
        m_map.insert( key, value );
    }

    _Value value ( const _Key & key ) const
    {
        QReadLocker locker( &m_mutex );
        m_map.insert( key, value );
    }

    // и т.д. все небходимые методы
}

Бага в том, что используется глобальный массив для GraphicsItem.
GraphicsItem может принадлежать только одной сцене.

Если сделать глобальные переменные членами класса, то все работает замечательно.