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

Question

grig_p20 августа 2018 г. 17: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 г. 19:18
(ред.)
Ответ был помечен как решение.

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

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

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

PaletteUtils(const PaletteUtils&) = delete;

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

21 августа 2018 г. 11:21

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

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

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

Комментарии