© EVILEG 2015-2018
Рекомендует хостинг
TIMEWEB
20 августа 2018 г. 11: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, все работает.
Почему такое происходит, не можете подсказать?
Заранее большое спасибо.

  • #
  • Ответ был помечен как решение
  • 20 августа 2018 г. 13:18

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

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

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

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


  • #
  • 21 августа 2018 г. 5:21

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

Ответы

Только авторизованные пользователи могут отвечать на форуме.
Пожалуйста, Авторизуйтесь или Зарегистрируйтесь
24 сентября 2018 г. 17:42
edorofeeva

C++ - Тест 001. Первая программа и типы данных

  • Результат 100баллов,
  • Очки рейтинга10
24 сентября 2018 г. 17:37
edorofeeva

C++ - Тест 001. Первая программа и типы данных

  • Результат 66баллов,
  • Очки рейтинга-1
23 сентября 2018 г. 14:38
No Names

C++ - Тест 001. Первая программа и типы данных

  • Результат 60баллов,
  • Очки рейтинга-1
Последние комментарии
25 сентября 2018 г. 15:24
pasagir

Qt/C++ - Урок 006. QSqlQueryModel - Таблицы в Qt с помощью SQL-запросов

Это запись метода которая работает параллельно с БД, данные из парсера поступают в БД и в наш метод одновременно
25 сентября 2018 г. 14:56
pasagir

Qt/C++ - Урок 006. QSqlQueryModel - Таблицы в Qt с помощью SQL-запросов

Как можно динамически отображать данные в таблице? На COM-порт непрерывно приходят данные, я их принимаю сохраняю в БД, а после остановка приема/передачи данные отображаются в таблице. В табли...
25 сентября 2018 г. 10:43
Евгений Легоцкой

Qt/C++ - Урок 017. QGraphicsScene или как работать с графикой в Qt

Прямо так не написано. Хотя соглашусь, что в качестве улучшения вызов данного метода здесь к месту.
25 сентября 2018 г. 10:37
reef425

Qt/C++ - Урок 017. QGraphicsScene или как работать с графикой в Qt

В статье написано, что таймер сработает один раз. Но это не так. Было бы хорошо добавить timer->setSingleShot(true); После инициализации таймера.
24 сентября 2018 г. 15:09
Евгений Легоцкой

Qt Linux - Урок 001. Автозапуск Qt приложения под Linux

А вот здесь у меня есть пример использования supervisor. https://evileg.com/ru/post/3/ Вся статья вам там не интересна, интересен только шаг с настройкой supervisor. Он получается ...
Сейчас обсуждают на форуме
25 сентября 2018 г. 15:57
Евгений_Канусовский@1981

Чтение файлов в python

Вот код: import sys from re import matchfrom vira import *from PyQt5 import QtCore, QtGui, QtWidgetsfrom PyQt5.QtWidgets import (QDialog, QFileDialog, QMessageBox, QLineEdit, QProgr...
25 сентября 2018 г. 13:51
DmitrySD

Трансляция видео с помощью VLC по RTP

Спасибо! Данная команда не дала результата. В итоге сделал трансляцию через ffmpeg. ffmpeg.exe -f gdigrab -framerate 30 -i desktop -vcodec libx264 -preset:v veryfast -b:v 4000k -f...
25 сентября 2018 г. 13:39
Arrow

Настройка Qt Creator для Android

Конечно отпишусь.
25 сентября 2018 г. 12:22
avovana

Автозапуск и авторестарт Qt Gui Application в Linux

Не получается... Решил пробовать скрипт, выполняемый при загрузке. В скрипте вечный цикл по старту программы.
Присоединяйтесь к нам в социальных сетях