QT中QString全局变量赋值后新值不保存的问题排查

实现支持用户自定义正则替换的文本清理按钮功能时，全局QString类型变量userDefinedClean、userDefinedInsert赋值后无法保存，读取时值始终为空。测试使用的正则表达式为(\d\d)/(\d\d)/(\d\d\d\d)|(\d\d)-(\d\d)-(\d\d\d\d)|(\d\d) (\d\d) (\d\d\d\d)，该正则在其他代码段可正常运行。

相关代码 #

mainwindow.cpp 全局定义与基础函数 #

#include "mainwindow.h"
#include "ui_mainwindow.h"
#include <QtCore>
#include <QFileDialog>
#include <QMessageBox>
#include <QInputDialog>
#include <QLabel>
#include <QPushButton>
#include <iostream>

#define TEXTEDIT  ui -> textEdit
#define TITLECHANGE(x, y) this->setWindowTitle("DataOrganiser: " + x.remove(0,y))
#define HOMEPATH QDir::homePath()
#define TEMPFILEPATH HOMEPATH + "/Desktop/QT stuff/Data Organiser/DataOrganiser/Temporary.txt"

QString fileInUse = TEMPFILEPATH;
QString userFilePath = HOMEPATH + "/Documents/UserFiles";

QString userDefinedClean;
QString userDefinedInsert;

MainWindow::MainWindow(QWidget *parent)
    : QMainWindow(parent)
    , ui(new Ui::MainWindow)
{
    ui->setupUi(this);
}

MainWindow::~MainWindow()
{
    delete ui;
}

清理设置槽函数 #

void MainWindow::on_action_Clean_Settings_triggered()
{
    QLabel *textLabel = new QLabel(this);
    textLabel->setFrameStyle(QFrame::StyledPanel | QFrame::Raised);
    QPushButton *textButton = new QPushButton(userFilePath);
    bool ok;
    bool okk;

    QString cleanText = QInputDialog::getText(this, "Current regex:" + userDefinedClean,
                                             "New Regex:", QLineEdit::Normal, "", &ok);
    QString insertText = QInputDialog::getText(this, "Current regex:" + userDefinedInsert,
                                               "New value:", QLineEdit::Normal, "", &okk);
    if (ok && okk && !cleanText.isEmpty()) {
        userDefinedClean = cleanText;
        userDefinedInsert = insertText;
    }
}

清理按钮点击槽函数 #

void MainWindow::on_cleanButton_clicked()
{
    QString text = TEXTEDIT -> toPlainText();
    QString search;
    text.replace("  ", " "); text.replace("   ", " "); text.replace("    ", " "); // this should be user customisable

    if(userDefinedClean != "") {
        QRegularExpression check1(userDefinedClean);
        QRegularExpressionMatchIterator matchs = check1.globalMatch(text);
        while(matchs.hasNext()) {
            QRegularExpressionMatch match = matchs.next();
            search = match.captured(0);
            text.replace(search, userDefinedInsert);
        }
    }

    TEXTEDIT -> clear();
    TEXTEDIT -> insertHtml(text);
}

头文件片段 #

#ifndef MAINWINDOW_H
#define MAINWINDOW_H

#include "qtextedit.h"
#include <QMainWindow>

QT_BEGIN_NAMESPACE
namespace Ui { class MainWindow; }
QT_END_NAMESPACE

extern QString userDefinedClean;
extern QString userDefinedInsert;

1. 核心原因：全局变量作用域/槽函数连接失效
   从贴出的头文件片段看，你没有给出MainWindow类的完整定义，大概率存在两个问题：
   • 类中缺少Q_OBJECT宏，也没有将两个槽函数声明在private slots块下，导致Qt元对象系统无法自动将按钮/Action的点击信号和对应槽函数绑定，点击设置项时槽函数根本没有执行，自然不会修改变量值
   • 若你在MainWindow类中声明了和全局变量同名的成员变量，会发生变量遮蔽：成员函数优先访问类成员变量，而非你在cpp中定义的全局变量，赋值和读取操作针对的不是同一个变量实体
2. 设置保存逻辑bug
   保存判断条件绑定了两个对话框的返回值，只要任意一个对话框点击取消，哪怕另一个输入了合法值也不会保存；同时输入框默认值传空而非当前已保存的配置，用户打开设置后直接点确定会因为正则输入为空不触发赋值，表现为配置不生效。
3. 依赖缺失与逻辑冗余
   代码中使用了QRegularExpression但没有引入对应头文件，正则替换手动遍历匹配项的写法会因为文本被修改出现重复替换、错位问题，多空格替换多次调用replace效率极低。

1. 放弃全局变量存储配置，将配置项改为MainWindow的私有成员，从根源避免作用域冲突，补全类定义、Q_OBJECT宏和槽声明：

   #ifndef MAINWINDOW_H
   #define MAINWINDOW_H
   
   #include <QMainWindow>
   #include <QString>
   #include <QRegularExpression>
   #include "qtextedit.h"
   
   QT_BEGIN_NAMESPACE
   namespace Ui { class MainWindow; }
   QT_END_NAMESPACE
   
   class MainWindow : public QMainWindow
   {
       Q_OBJECT // 必须添加，否则槽函数无法自动连接
   public:
       MainWindow(QWidget *parent = nullptr);
       ~MainWindow();
   
   private slots:
       // 必须声明槽函数
       void on_action_Clean_Settings_triggered();
       void on_cleanButton_clicked();
   
   private:
       Ui::MainWindow *ui;
       // 配置项作为类成员，不使用全局变量
       QString fileInUse;
       QString userFilePath;
       QString userDefinedClean;
       QString userDefinedInsert;
   };
   #endif
   

2. 修改cpp代码，删除全局变量定义，将初始化逻辑移到构造函数中，修复设置保存逻辑和正则替换逻辑：

   #include "mainwindow.h"
   #include "ui_mainwindow.h"
   #include <QtCore>
   #include <QFileDialog>
   #include <QMessageBox>
   #include <QInputDialog>
   
   #define TEXTEDIT  ui->textEdit
   #define HOMEPATH QDir::homePath()
   #define TEMPFILEPATH HOMEPATH + "/Desktop/QT stuff/Data Organiser/DataOrganiser/Temporary.txt"
   
   MainWindow::MainWindow(QWidget *parent)
       : QMainWindow(parent)
       , ui(new Ui::MainWindow)
   {
       ui->setupUi(this);
       // 初始化路径和默认配置
       fileInUse = TEMPFILEPATH;
       userFilePath = HOMEPATH + "/Documents/UserFiles";
       userDefinedClean = "";
       userDefinedInsert = "";
   }
   
   MainWindow::~MainWindow()
   {
       delete ui;
   }
   
   void MainWindow::on_action_Clean_Settings_triggered()
   {
       bool okClean, okInsert;
       // 传入当前配置作为输入框默认值，分开处理两个输入框的返回结果
       QString cleanText = QInputDialog::getText(this, "Set clean regex",
                                               "Regex pattern:", QLineEdit::Normal, userDefinedClean, &okClean);
       if (okClean) {
           userDefinedClean = cleanText;
       }
   
       QString insertText = QInputDialog::getText(this, "Set replacement text",
                                               "Replace with:", QLineEdit::Normal, userDefinedInsert, &okInsert);
       if (okInsert) {
           userDefinedInsert = insertText;
       }
   }
   
   void MainWindow::on_cleanButton_clicked()
   {
       QString text = TEXTEDIT->toPlainText();
       // 用正则一次性替换连续多空格，不需要多次调用replace
       text.replace(QRegularExpression("\\s{2,}"), " ");
   
       if(!userDefinedClean.isEmpty()) {
           QRegularExpression pattern(userDefinedClean);
           // 直接调用正则版本的replace一次性替换所有匹配项，不需要手动遍历
           text.replace(pattern, userDefinedInsert);
       }
   
       TEXTEDIT->clear();
       TEXTEDIT->insertHtml(text);
   }
   

内容的提问来源于stack exchange，提问作者Popcorn973