0

我正在处理 USACO 页面上的第一个问题，我收到一条错误消息：

Execution error: Your program had this runtime error:
        Exceeded memory or invalid memory reference. The program ran for
        0.000 CPU seconds before the error. It used 3372 KB of memory.

我很确定我知道为什么会这样：当我运行我的程序时，它编译并执行得很好，但是一旦程序完成，就会弹出一个框说“main.exe 已停止工作......”，然后它冻结了。当我运行调试器时，我收到此错误： an access violation (segmentation fault) raised in your program. 这是什么意思，我该如何解决？我看到了其他几个类似的线程，但我无法使用其他线程的反馈来修复我的程序。这是我的代码：

/*
ID: krishna24
LANG: C++
PROG: ride
*/
#include <cstdlib>
#include <iostream>
#include <string>

using namespace std;

string ride(string comet, string group) {
   int cometsum = 1;
   int groupsum = 1;
   for(int i = 0; i < comet.size(); i++) {
       char ascival = comet[i];
       cometsum = cometsum*((int)ascival-64);
   }
   for(int i = 0; i < group.size(); i++) {
       char ascival = group[i];
       groupsum = groupsum*((int)ascival-64);
   }
   if (groupsum%47 == cometsum%47) cout << "GO";
   else cout << "STAY";
}

int main(int argc, char *argv[])
{
    string comet;
    getline(cin, comet);
    string group;
    getline(cin, group);
    ride(comet, group);
}

非常感谢您的帮助！

PS：任何关于如何使我的代码更加精简和高效的建议也会很有用。:)

编辑：这不起作用，因为我的ride()函数没有返回字符串。改变它以void解决问题。感谢大家！

4

2 回答 2

6

ride声称要返回 a string，但您实际上并没有返回任何东西。我很惊讶编译器可以让你逃脱惩罚；您应该启用警告（-Wall对于 GCC），甚至可能使警告成为错误（-Werror对于 GCC）。

于 2013-07-16T18:56:53.290 回答
0

ride()应该返回一个字符串，但是你输出到控制台。也许在 结束时ride()，更改代码以返回字符串：

if (groupsum%47 == cometsum%47) return << "GO";
    else return << "STAY";

并将main()结果输出到控制台：

cout << ride(comet, group);
于 2013-07-16T19:04:31.600 回答