2

好吧，我得到了一个任务来构建一个可扩展的数据类型以在斐波那契生成器中使用，当我去测试它到序列中的第 1000 个数字时，我的任务即将完成。

我注意到它没有对齐，并确定问题出在序列中的第 262 个数字上。经过一点调试，我发现这是链表从7个整数移动到8个整数的地方，但我不知道它与问题有关。

我要找的号码是：

2 542 592 393 026 885 507 715 496 646 813 780 220 945 054 040 571 721 231

我得到的号码是：

1 154 259 239 326 885 507 715 496 646 813 780 220 945 054 040 571 721 231

您可能会注意到，直到最后一个块（左端），它们都非常相似。

我的来源有点长，所以我做了一个要点：

https://gist.github.com/anonymous/5802620

携带超过 1,000,000,000 的魔法发生在第 378 行的函数中： https ://gist.github.com/anonymous/5802620#file-main-cpp-L378

作为更新，我仍然得到“接近，但没有雪茄”的输出。以下是可能是罪魁祸首的代码（包括贡献的答案）：

Giant Giant::operator + (const Giant & rightSide)
{
   Giant returned;
   int extra = 0;

   for(int i = 0;
      i < chunks.getNumItems() && i < rightSide.chunks.getNumItems(); i++)
   {
      int num = chunks.getData(i) + rightSide.chunks.getData(i);
      returned.chunks.insert((num + extra) % chunkSize,
         returned.chunks.getNumItems());
      extra = (num + extra) / chunkSize;
   }

   if(chunks.getNumItems() > rightSide.chunks.getNumItems())
   {
      for(int i = rightSide.chunks.getNumItems();
          i < chunks.getNumItems(); ++i)
      {
         returned.chunks.insert(extra + chunks.getData(i),
            returned.chunks.getNumItems());
         extra = 0;
      }
   }
   else if(chunks.getNumItems() < rightSide.chunks.getNumItems())
   {
      for(int i = chunks.getNumItems();
          i < rightSide.chunks.getNumItems(); ++i)
      {
         returned.chunks.insert(extra + rightSide.chunks.getData(i),
            returned.chunks.getNumItems());
         extra = 0;
      }
   }

   if (extra != 0)
   {
       returned.chunks.insert(extra, returned.chunks.getNumItems());
   }

   return returned;
}

更新 班级的一位成员过来，我们查看了代码。这是问题所在：

ostream & operator << (ostream & out, const Giant & giant)
{
   for(int i = giant.chunks.getNumItems() - 1;
      i >= 0; i -= 1)
   {
      if (i != giant.chunks.getNumItems()-1)
         out << setw(9) << setfill('0');
      out << giant.chunks.getData(i);
   }
   return out;
}

我忘了强制显示导致问题的前导零。已找到解决方案。

4

3 回答 3

2

您的问题似乎在第 387 和 389 行。您extra在等式中的计算速度不够快。这些行应该分别具有((num + extra) % chunkSize)和extra = (num + extra )/ chunkSize;。对于您当前的实现，其中一个段可能是 1000000000 没有进位，而不是 0 有进位（如果我正在阅读您的意图，如果您想要的话）。

于 2013-06-18T04:29:28.377 回答
1

除了其他人所说的之外，我认为您需要考虑extra从第 399 行和第 408 行开始的循环。特别是，如果 extra 不为零，并且您采用这两个循环之一，则需要添加您在该循环中插入的第一件事的额外金额（而不是在第 394 行单独添加）。

根据您在下面的评论进行编辑。 这基本上就是我对第 392-414 行的意思：

if(chunks.getNumItems() > rightSide.chunks.getNumItems())
{
   for(int i = rightSide.chunks.getNumItems();
       i < chunks.getNumItems(); ++i)
   {
      returned.chunks.insert(extra + chunks.getData(i),
         returned.chunks.getNumItems());
      extra = 0;
   }
}
else if(chunks.getNumItems() < rightSide.chunks.getNumItems())
{
   for(int i = chunks.getNumItems();
       i < rightSide.chunks.getNumItems(); ++i)
   {
      returned.chunks.insert(extra + rightSide.chunks.getData(i),
         returned.chunks.getNumItems());
      extra = 0;
   }
}
else if (extra != 0)
{
    returned.chunks.insert(extra, returned.chunks.getNumItems());
}

当然，如果可行，也有一些方法可以“简化它”。

于 2013-06-18T04:44:23.863 回答
0

更新 班级的一位成员过来，我们查看了代码。这是问题所在：

ostream & operator << (ostream & out, const Giant & giant)
{
   for(int i = giant.chunks.getNumItems() - 1;
      i >= 0; i -= 1)
   {
      if (i != giant.chunks.getNumItems()-1)
         out << setw(9) << setfill('0');
      out << giant.chunks.getData(i);
   }
   return out;
}

我忘了强制显示导致问题的前导零。已找到解决方案。

于 2013-06-18T16:00:04.917 回答