c++ 重载`-`

Question

我已经为我的类图将运算符重载为“-”。 它的用途并不完全 直观（糟糕的编码 - 我知道）但如果我做 graph3 = graph2-graph1 那么图 3 是 应该只接收图 2 和图 1 中的那些顶点。

因此，我已经编写了代码，当我运行调试器时，运算符函数似乎创建了一个新的“图”来返回，并将适当的顶点添加到新图表 然后调试器似乎退出了 op- 函数，但永远不会返回到 主要的。 就好像它在等我输入什么。 没有出现错误消息。

这是代码：

char stringy[100];
//cin>>stringy;
strcpy(stringy,"|12,34,25,2,3,2|(3->2),(2->1),(5->9),(2->1)|");
char* param= new char[sizeof(stringy)];
strcpy(param,stringy);
Graph graph1(param);

 char sstring[20] = "|33,34,11|(2->33)|";
Graph graph2(sstring);
cout<<graph2.outSumm()<<endl;

Graph graph3;
//until here everything works fine
graph3= graph1-graph2; //the debugger does this and then 

cout<<graph3.outSumm()<<endl;

运算符函数：

Graph Graph::operator- (const Graph& g2) const
{
Graph created;

//goes through "this" list and if value exists in g2 copies it to created
for(int i=0;i<vertList.getSize();i++)
{
 if (g2.vertList.find(vertList.read(i))!=999)
 created.addVertex(vertList.read(i).getInt());
}

return created;
}

我正在使用代码块。

复制构造函数：

Graph(const Graph& g2):      
maxVal(g2.maxVal),vertList(g2.vertList),edgeList(g2.edgeList){} ;

赋值运算符：

void Graph::operator= (const Graph& g2)
{
 if (this==&g2)
 {
 cout<<"not the greatest idea"<<endl;
 return;
 }

 vertList.delete_List();
 edgeList.delete_List();
 maxVal=0;

addValues(g2.outSumm());
}

Answer 1

与您的问题没有特别相关（尽管可能是），您为什么不使用 std::string ？ 如果您必须使用字符数组，为什么不：

char stringy[100] = "|12,34,25,2,3,2|(3->2),(2->1),(5->9),(2->1)|";
Graph graph1( stringy );

甚至：

Graph graph1( "|12,34,25,2,3,2|(3->2),(2->1),(5->9),(2->1)|" );

或者您是否将传递到类的构造函数中的指针存储起来？ 如果是这样，那是一个坏主意，我对使用 std::strings 的建议加倍。

Answer 2

听起来你的代码处于无限循环中。 当程序似乎停止时，您是否尝试过中断调试器？

Graph 类中是否定义了复制构造函数？ 您需要它才能使用本地 created 变量成功返回值。

Answer 3

不要实现operator-，而是实现operator-=：

Graph& Graph::operator-=(const Graph& rhs)
{
    // remove nodes not in both Graphs

    return *this;
}

然后就这样实现operator-：

Graph Graph::operator-(const Graph& rhs)
{
    Graph temp(*this);
    temp -= rhs;
    return temp;
}

看看这是否有效。