Variable in other class is not updating.

[jacapiwsko]

Hello, I have an app where user can change a variables - delays, hotkeys etc. I do the "config saver" but it recieve "old" (not updated) variables. "Variable check" in same class working, but in other class shows others.

void Building::setWindowName(QString value){
    connect(this, SIGNAL (textChanged(QString)), this, SLOT(setWindowName(QString)));
    sett.windowName = value;
    sett.windowID = (const wchar_t*) sett.windowName.utf16();
    qDebug() << "windowName " << sett.windowName; //Shows updated
}

class Settings : public QObject
{
    Q_OBJECT

public:
    explicit Settings(QObject *parent = nullptr);
    QSettings* sett;

    int toggleButton=0x52;
    int btoggleButton=0x52;// r
    int inventoryKey=0x45; // e
    int minCpsDelay = 10;
    int maxCpsDelay = 15;

    QString windowName = "Minecraft 1.8.8 (Blazingpack.pl)";
    LPCWSTR windowID = (const wchar_t*) windowName.utf16();

void Settings::clickcheck()
{
    qDebug() << windowName; //Showed old - not updated variable.
}

I just want to save some data from different classes

[ne555]

not enough code to reproduce

[jacapiwsko]

I dont know how to describe this :/

I want to change windowName string declared in Settings class from Building class level.

When I edit and read from Building class level, it shows good, edited variable.

When I read EDITED variable from Settings class, It shows always the same.

#ifndef KEPPOXD_H
#define KEPPOXD_H

#include <QObject>
#include <QDebug>

class keppoxd : public QObject
{
    Q_OBJECT
public:
    explicit keppoxd(QObject *parent = nullptr);
    QString name = "old";

signals:
    void textChanged(QString);

public slots:
    void setWindowName(QString value);
    void test();
};

#endif // KEPPOXD_H 

#ifndef SIEMA_H
#define SIEMA_H

#include <QObject>
#include <QDebug>
#include <keppoxd.h>


class siema : public QObject
{
    Q_OBJECT
public:
    explicit siema(QObject *parent = nullptr);
    keppoxd keppo;

signals:
    void textChanged(QString);

public slots:
    void setWindowName(QString value);

};

#endif // SIEMA_H

#include "keppoxd.h"

keppoxd::keppoxd(QObject *parent) : QObject(parent)
{

}

void keppoxd::setWindowName(QString value)
{
        connect(this, SIGNAL (textChanged(QString)), this, SLOT(setWindowName(QString)));
        name = value;
        qDebug() << name;
}

void keppoxd::test()
{
    qDebug() << "keppo test " << name;
}

#include <QGuiApplication>
#include <QQmlApplicationEngine>
#include <QSettings>
#include <QDir>
#include <QDebug>
#include <keppoxd.h>
#include <siema.h>
#include <QQmlContext>
#include <QQmlApplicationEngine>

int main(int argc, char *argv[])
{

    QCoreApplication::setAttribute(Qt::AA_EnableHighDpiScaling);

    QGuiApplication app(argc, argv);
    //QSettings* settings;

    /*if(QFileInfo::exists(QDir::currentPath() + "/my_config_file.ini")){
        qDebug() << "istnieje";
    }
    else{
        qDebug() << "nie istnieje";

        settings = new QSettings(QDir::currentPath() + "/my_config_file.ini", QSettings::IniFormat);
        settings->setValue("test", "value");
        settings->setValue("Button", 1);
        settings->sync();
    } */


    siema siema;
    keppoxd keppo;

    QQmlApplicationEngine engine;
    const QUrl url(QStringLiteral("qrc:/main.qml"));
    QObject::connect(&engine, &QQmlApplicationEngine::objectCreated,
                     &app, [url](QObject *obj, const QUrl &objUrl) {
        if (!obj && url == objUrl)
            QCoreApplication::exit(-1);
    }, Qt::QueuedConnection);
    engine.load(url);


    QQmlContext *ctx = engine.rootContext();

    ctx->setContextProperty("Siema", &siema);
    ctx->setContextProperty("Keppo", &keppo);

    return app.exec();
}

#include "siema.h"

siema::siema(QObject *parent) : QObject(parent)
{

}

void siema::setWindowName(QString value)
{
    connect(this, SIGNAL (textChanged(QString)), this, SLOT(setWindowName(QString)));
    keppo.name = value;
    qDebug() << keppo.name;
}

import QtQuick 2.9
import QtQuick.Window 2.2

Window {
    visible: true
    width: 640
    height: 480
    title: qsTr("Hello World")

    TextInput {
        id: textInput
        x: 48
        y: 58
        width: 80
        height: 20
        text: qsTr("Text Input")
        font.pixelSize: 12
        onTextEdited:{
            Siema.setWindowName(textInput.text);
            Keppo.test();
        }
    }

    Rectangle {
        id: rectangle
        x: 276
        y: 41
        width: 200
        height: 200
        color: "#e30a0a"

        MouseArea {
            id: mouseArea
            x: 0
            y: 0
            width: 200
            height: 200
            onClicked:{
                Siema.setWindowName(textInput.text);
                Keppo.test();
            }
        }
    }
}

Log:

istnieje
"Text Input"
keppo test  "old"
"Text Input"
keppo test  "old"
"Text Ixnput"
keppo test  "old"
"Text Ixdnput"
keppo test  "old"
"Text Ixdnput"
keppo test  "old"
"Text Ixdnput"
keppo test  "old"
"Text Ixdnput"
keppo test  "old"

[dutch]

You make a siema and a keppo in main, but since a siema has a keppo inside it you are actually making two keppos. Presumably you are changing the name of one of them but printing the name of the other.

[jacapiwsko]

Then how to change/edit/modify other class variable from other class?

[dutch]

I don't understand what you are asking.

[jacapiwsko]

I want to change classa.string from classb but I cant and I dont know why.

[Peter87]

Note that in main keppo and siema.keppo are two different objects.

[jacapiwsko]

So if I changed that should work? Or what should I do?

[ne555]

class siema : public QObject
{
    Q_OBJECT
public:
    explicit siema(QObject *parent = nullptr);
    keppoxd *keppo; //a pointer
//...
];

int main(){
    siema siema_;
    keppoxd keppo;
    siema_.keppo = &keppo; //notnow the `siema_' object can communicate to the `keppo' object in main
}

[jacapiwsko]

Oh thanks, working!