0

我想编写一个循环，遍历同一结构的多个实例（在我的例子中名为 edg），该循环由某个函数从邻居到邻居迭代，直到它返回一个说 STOP 的元素。我尝试使用 NULL 返回对其进行编码，但它不起作用。我能用什么？

这里有一些代码解释它可能比我之前的话更准确：我的结构：

struct edg{
int i,j,k;
edg(int a, int b, int c){
i = a; j = b; k = c; //I'm adding a constructor to it
}
}

我的迭代函数：

edg neighbour(edg temp){
if(temp satisfies certain criterias){ return edg(new coordinates);}
else{ return NULL;}
}

我的循环：

while(my_edg!=NULL){
my_edg = neighbour(my_edg);
}

我想我可以选择一个 edg 的某个值将其定义为拒绝，并在我的循环中替换为：

while(my_edg!=edg_marked_as_rejection)

但是还有其他方法吗？

4

1 回答 1

2

请注意，您的功能：

edg neighbour(edg temp){
    if(temp satisfies certain criterias){ return edg(new coordinates); }
    else{ return NULL; }
}

返回一个按值的实例，edg因此尝试return NULL;是无效的（除非您定义了一些自定义转换）。NULL是通过指针传递/返回时的可能值，在这种情况下，这可能意味着将此函数的原型更改为：

edg* neighbour(edg temp) { ... }

然而，基于语义，通过引用传递并返回一个指示成功的标志会更合理：

bool neighbour(const edg& temp, edg& result) {
    if (...) {
        result = ...;
        return true;
    }
    return false;
}

或者如果您的条件" if(temp satisfies certain criterias)"在大多数情况下都应该满足，而这些条件不满足是一种异常状态，您也可以考虑抛出异常（而不是返回NULL）。

第三种选择可能是实现NULL 对象设计模式，这意味着edg将构造并返回标记为“无效”的实例，调用者将执行以下操作：

edg n = neighbour(tmp);
if (!n.isValid()) {
    ...
}
于 2013-10-20T15:23:41.383 回答