Странное поведение сингелтона, а может быть, и не в нем проблема...

Question

grig_pТам. 20, 2018, 7:50 Т.Ж.

Странное поведение сингелтона, а может быть, и не в нем проблема...

Здравствуйте!

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

Суть в чем. Написал синелтон в двух модулях:

pu.h:

enum class ControlObjectPalette
{
      TreeBase = 0
    , TreeItem
    , TreeItemSelected
    , TreeReady
    ...
};


class PaletteUtils
{
public:
    static PaletteUtils instance();

    /*!
     * \brief По коду цвета возвращает цвет из палитры
     * \param colorCode
     */
    QColor getColor(const Palette &colorCode) const;
    /*!
     * \brief Устанавливает цвет
     * \param colorCode - код цвета
     * \param color - значение цвета
     */
    void setColor(const Palette &colorCode, const QColor &color);


private:
    PaletteUtils();

    QMap<ControlObjectPalette, QColor> m_palette;
};

pu.cpp:

PaletteUtils PaletteUtils::instance()
{
    static PaletteUtils _instance;
    return _instance;
}

QColor PaletteUtils::getColor(const Palette &colorCode) const
{

    if (colorCode == Palette::TreeBase)
        qDebug() << "---->>>>>" << (int)colorCode << m_palette.value(colorCode);
    return m_palette.value(colorCode);
}

void PaletteUtils::setColor(const Palette &colorCode, const QColor &color)
{
    m_palette.insert(colorCode, color);
    if (colorCode == Palette::TreeBase)
        qDebug() << "<<<<<----" << (int)colorCode << m_palette.value(colorCode);
}

PaletteUtils::PaletteUtils()
    :      m_palette({
                       std::pair<Palette, QColor>(Palette::TreeBase, QColor(165, 165, 165))
                     , std::pair<Palette, QColor>(Palette::TreeItem, QColor(183, 232, 221))
                     , std::pair<Palette, QColor>(Palette::TreeItemSelected, QColor(49, 133, 155))
                       ...
                     })
{

}

Все работает, пока я получаю от него данные вот так:

QColor color = PaletteUtils::instance().getColor(code);

Необходимо изменить в нем данные. Вызываю setColor:

PaletteUtils::instance().setColor(code, color);

Данные меняются, вывод из процедуры setColor говорит об этом.

Но, если после выполнения setColor прочитать данные с помощью getColor - данные неизменны. То есть, как бы сделали копию m_palette, поработали в ней и отдали. Отладочный вывод привожу:

---->>>>> 0 QColor(ARGB 1, 0.647059, 0.647059, 0.647059)
---->>>>> 0 QColor(ARGB 1, 0.647059, 0.647059, 0.647059)
<<<<<---- 0 QColor(ARGB 1, 0.647059, 0.309804, 0.529412)
---->>>>> 0 QColor(ARGB 1, 0.647059, 0.647059, 0.647059)

Если вынести m_palette из сингелтона и поместить его в pu.cpp, все работает.

Почему такое происходит, не можете подсказать?

Заранее большое спасибо.

Рекомендуем хостинг TIMEWEB

Стабильный хостинг, на котором располагается социальная сеть EVILEG. Для проектов на Django рекомендуем VDS хостинг.

Талқылауға жазылыңыз2

Бөлімге жазылу94

Ол саған ұнайды ма? Әлеуметтік желілерде бөлісіңіз!

L

Там. 20, 2018, 9:18 Т.Ж.
(өңделген)
Жауап шешім ретінде белгіленді.

Лучше возвращать ссылку на экземпляр класса:

PaletteUtils& PaletteUtils::instance()
{
    static PaletteUtils _instance;
    return _instance;
}

и если уж делать singleton, то хорошо было бы запретить копирование (вынести в приват или пометить delete):

PaletteUtils(const PaletteUtils&) = delete;

PaletteUtils& operator= (const PaletteUtils &) = delete;

Там. 21, 2018, 1:21 Т.Ж.

Спасибо. Все получилось

Пікірлер

Тек рұқсаты бар пайдаланушылар ғана пікір қалдыра алады.
Кіріңіз немесе Тіркеліңіз