AOJのジャッジがうまくいきません

Question

前提・実現したいこと

AOJでスタックについての問題がREとなってしまいます。

発生している問題・エラーメッセージ

timeout: the monitored command dumped core 0.00user 0.00system 0:00.00elapsed 33%CPU (0avgtext+0avgdata 3328maxresident)k 0inputs+8outputs (0major+243minor)pagefaults 0swaps

該当のソースコード

#include <bits/stdc++.h>
using namespace std;


int main(){
    stack<int> s;
    string input;
    cin >> input;
    s.push(stoi(input));
    int temp;
    while (scanf("%s", input) != EOF) {
		cin >> input;
        if (input[0] == '+'){
            temp = s.top();
            s.pop();
            temp += s.top();
            s.pop();
            s.push(temp);
        }
        else if (input[0] == '-'){
            temp = -1 * s.top();
            s.pop();
            temp += s.top();
            s.pop();
            s.push(temp);
        }
        else if (input[0] == '*'){
            temp = s.top();
            s.pop();
            temp *= s.top();
            s.pop();
            s.push(temp);
        }
        else {
            s.push(stoi(input));
        }
    }
    cout << s.top() << endl;
}

試したこと

AtCoderで以下の入力でコードテストをしたところ、正確に動作しています。

1 2 + 5 4 + *
1 2 +
1 2 + 3 4 - *

Answer 1

Wandboxでは'%s' expects argument of type 'char*', but argument 2 has type 'std::string'～と警告されます。
この警告に従い、文字入力部分を修正する（scanfを使わない）と正解します。

Comments

[can110]

あ。全然遅かった。

Answer 2

元のコードではwhile (scanf("%s", input) != EOF)の部分に問題(string inputをchar *に見立てている)があって、

$ g++ -Wall -std=c++11 t1.cpp -o t1.exe
t1.cpp: In function 'int main()':
t1.cpp:11:29: warning: format '%s' expects argument of type 'char*', but argument 2 has type 'st
d::string {aka std::basic_string<char>}' [-Wformat=]
     while (scanf("%s", input) != EOF) {
                             ^

実際、これを実行するとランタイムエラー(RE)で落ちます。

$ ./t1.exe
1 2 +
Segmentation fault (core dumped)

scanfを使わず、こうすればOKなのでは。

   //while (scanf("%s", input) != EOF) {
    //    cin >> input;
    while (cin >> input) {
        if (input[0] == '+'){
...

実行してみます。テストパターンはAOJのサンプル問題です。EOF(CTRL+D)を与えるまで答えを出力していませんが、本問題では許容範囲なのでしょう。

$ g++ -Wall -std=c++11 t1b.cpp -o t1b.exe

$ ./t1b.exe
1 2 +
3

$ ./t1b.exe
1 2 + 3 4 - *
-3

尚、試した環境は Windows10 cygwin g++ 7.4.0 です。C++11で確認しました。(C++14でもREです）

Comments

[Magolors]

ありがとうございます。実行することができました。