3

我正在尝试使用运算符重载来为我的多项式类定义基本操作（+、-、*、/），但是当我运行程序时它崩溃并且我的计算机死机了。

更新4

行。我成功做了三个操作，只剩下一个分裂。

这是我得到的：

polinom operator*(const polinom& P) const
{
    polinom Result;
    constIter i, j, lastItem = Result.poly.end();
    Iter it1, it2, first, last;
    int nr_matches;

    for (i = poly.begin() ; i != poly.end(); i++) {
         for (j = P.poly.begin(); j != P.poly.end(); j++)
              Result.insert(i->coef * j->coef, i->pow + j->pow);
    }

    Result.poly.sort(SortDescending());

    lastItem--;

    while (true) {
        nr_matches = 0;

        for (it1 = Result.poly.begin(); it1 != lastItem; it1++) {
             first = it1;
             last = it1;
             first++;
             for (it2 = first; it2 != Result.poly.end(); it2++) { 
                  if (it2->pow == it1->pow) {
                      it1->coef += it2->coef;
                      nr_matches++;
                  }
             }

             nr_matches++;
             do {
                last++;
                nr_matches--;
             } while (nr_matches != 0);

             Result.poly.erase(first, last);
        }   
        if (nr_matches == 0)
            break;
    }     

    return Result;
}
4

3 回答 3

5 
while (i != poly.end() || P.i != P.End())

我认为你需要 && ，否则只有当 i 和 pi 同时到达它们各自的终点时循环才会终止。

带有否定的逻辑是棘手的。可能更简单地将其视为：

while (!(i == poly.end() || j == P.End())) //while neither iterator has reached end

根据布尔运算，它与以下内容相同：

while (!(i == poly.end()) && !(j == P.End()))
while (i != poly.end() && j != P.End())

如果两者相等，您似乎也不会增加迭代器（无限循环导致无限多的内存分配？）。

风格问题：最好使用迭代器作为局部变量。如果在方法中开始使用它们之前应该“初始化”变量，请不要将它们设为类成员，并且在方法完成后它们变得无用。

也更喜欢通过 const 引用传递参数，如果成员函数不修改当前对象（不应该），则将其标记为const operator+：

 polinom operator+(const polinom& P) const;

（这将揭示制作本地使用的迭代器成员的问题 - 您将修改实例！）

于 2010-03-11T14:24:53.410 回答
2

您的代码还有其他设计和正确性问题，但我认为崩溃发生在这一行

 if (i->pow > P.i->pow)

当 i == poly.end() && Pi != P.End() 或 i != poly.end() && Pi == P.End()。在最后一个元素之后取消引用 i 将崩溃。

于 2010-03-11T14:14:07.730 回答
1

至于让函数正确添加多项式，我推荐这个简单的逻辑：

polinom operator+(const polinom& P) const //fixed prototype re. const-correctness
{
    polinom Result;
    std::list<term>::const_iterator //fixed iterator type
        i = poly.begin(), j = P.poly.begin();

    while (i != poly.end() && j != P.poly.end()) {
        //logic while both iterators are valid
    }

    //handle the remaining items in each list
    //note: at least one will be equal to end(), but that loop will simply be skipped

    while (i != poly.end()) {
        Result.insert(i->coef, i->pow);
        ++i;
    }

    while (j != P.poly.end()) {
        Result.insert(j->coef, j->pow);
        ++j;
    }

    return Result;
}

最后一部分可能也可以留给标准库函数

#include <iterator>
#include <algorithm>

//...
    //handle remaining items in either list (if any)
     std::copy(i, poly.end(), std::back_inserter(Result.poly));
     std::copy(j, P.poly.end(), std::back_inserter(Result.poly));

...但使用list.insert可能会更简单：

     Result.poly.insert(Result.poly.end(), i, poly.end());
     Result.poly.insert(Result.poly.end(), j, P.poly.end());
于 2010-03-11T16:02:59.820 回答