Question

我有以下布局：

MainWindow  <--------- Settings
    \
     \
      V
   PerformOps

MainWindow接受从Settings传递的变量，并将它们传递给PerformOps。如果未使用Settings类，则MainWindow会将默认值传递给PerformOps。

目前我这样做：

class Settings{

public: 
   Settings(*parent);
   var1, var2, var3
   .
   .
   void exec();
}

class PerformOps{

public: 
   PerformOps();
   var1, var2, var3;
   .
   .
   void start();
}


class MainWindow{

private:
  Settings *set;    //share pointers over all methods
  PerformOps *op;

  bool settings_not_clicked = false;

  void saveChanges()
  {
     QSettings settings("my","app");
     settings.setValue("value1", op->var1);
     settings.setValue("value2", op->var2);
     settings.setValue("value3", op->var3);
  }

  void loadChanges()
  {
     QSettings settings("my","app");
     op->var1 = settings.value("value1");
     op->var2 = settings.value("value2");
     op->var3 = settings.value("value3");

  }

  void closeEvent(QCloseEvent *event)
  {
       event->ignore();
       saveChanges();
       event->accept();
  }

  void takeDataAndDoStuff()   //This is always called
  {
     op = new PerformOps;

     if(settings_not_clicked) {
        loadChanges()
     }
     else {
         op->var1 = 33;
         op->var2 = "Geronimo!";
         op->var3 = true;              
     }

     op->start();
  }


  void getStuff_maybe_if_clicked()   //This might not be always called
  {
     Settings *set = new Settings(this);
     set->exec()      //Pause parent, run Settings

     settings_not_clicked = false;
  }

问题：考虑到op->var1 = set->var1;指针可能并不总是初始化，是否有更简洁的方法跨类共享数据而不使用set的脏方法？

Answer 1

嗯，这种方法本身并不是那么糟糕，但是有些事情你可以改进。

首先，如果我理解正确，您希望传递设置（如果存在），如果不存在则传递默认值。在这种情况下，您可以使用构造函数：

class PerformOps
{
public:
    PerformOps( int v1 = 33, string v2 = "Geronimo!", bool v3 = true ): var1(v1), var2(v2), var3(v3)
    {
    }
    <...>
}

现在，如果您将构造函数调用为PerformOps()，则将设置默认值。如果您调用它并为其提供一些值，它将使用它们：

<...>
PerformOps *op;

if(settings_not_clicked) {
    op = new PerformOps( set->var1, set->var2, set->var3 );
}
else {
    op = new PerformOps();              
}
 <...>

当然，如果你不想通过cunstructor这样做，你可以创建一个函数并像“setData（）”一样调用它，并使用与默认函数参数相同的技术。

现在，至于指针。如果你有c ++ 0x，始终初始化指针NULL或nullptr是个好主意。此外，删除内存时，请再次将指针指定给NULL或nullptr。这样，您将能够通过简单的检查始终查看指针是否有效。

someClass * ptr = nullptr;
<...>
ptr = new someClass();
<...>
delete ptr;
ptr = nullptr;

<强> UPD

我建议你摆脱你的Settings课程，直接使用QSettings。你不需要搞乱指针，读取/写入QSettings非常快。

现在，也不要忘记你可以使用堆中的QSettings：

QSettings * settings = new QSettings();

如果您希望每个设置都设置为“父级”，则可以从Settings派生QSettings课程，只需在其中添加一个parent字段即可。这将保留QSettings的所有功能，非常方便。

实际上，你的方法也很好，你需要做的就是检查指针是否有效。

Answer 2

好的，问题是“我正在设置指向可能存在或可能不存在的指针”。在这种情况下Null Object模式应该有所帮助。这样你就可以摆脱if-clause

Answer 3

哦，坚持下去。我刚刚意识到我可以在类之间使用QSettings（这是SingerOfTheFall明确暗示的）

所以从我的Settings.cpp类我可以做到：

QSettings sett("my");
sett.begingroup("my_groupname");
sett.setValue("value1", var1);
sett.setValue("value2", var2);
sett.setValue("value3", var3);
sett.endGroup();

我可以通过以下方式从任何类别（呃！）中检索相同的信息：

someRandomClass.cpp：

QSettings settings("my");
settings.beginGroup("my_groupname");
myvar1 = settings.value("value1");
myvar2 = settings.value("value2");
myvar3 = settings.value("value3");
settings.endGroup();

Qt：通过C类将变量从Class传递给Class

3 个答案: