AOJ ITP1_4_D 何故WAエラーになるのか分からないのでアドバイスが欲しいです。

Question

前提・実現したいこと

問題：AOJ ITP1_4_D
コード①で回答提出すると全20個あるTEST CASE中19番目でエラーとなります。
何故WAエラーになるのか分からないのでアドバイスが欲しいです。

以前全て正解となった別のコード②との違いはvector配列を使ったことくらいだと思います。

発生している問題

デバッグで試そうにも、
問題中にある入力用データは少ない個数なので正解がすぐに確認出来るのですが、
入力が10000個の6桁前後の整数でVSC上からデバッグしてコマンドプロンプトから貼り付けても、
データが多すぎて受け取れていないのか貼り付け途中でデータが切れています。
仕方なく毎回AOJへ提出するしかテスト方法が無い状況です。

![コマンドプロンプトには途中までしか貼り付けられていない](c6beba55c28036bbf472851e94db229c.jpeg)

試したこと

コード①：問題にあるサンプル入力データは数個なので正解が出て確認出来る。
コード②：以前回答した別のコードを再提出してみると全て正解になる。

コード①：TEST CASE１９番目でWAとなる

#include <bits/stdc++.h>
#define repi(n) for (int i = 0; i < n; i++)

// 総和 vector
long long sumVec(std::vector<long long> v)
{
    return std::accumulate(v.begin(), v.end(), 0);
}
// 最小値 vector
long long minVec(std::vector<long long> v)
{
    return *std::min_element(v.cbegin(), v.cend());
}
// 最大値 vector
long long maxVec(std::vector<long long> v)
{
    return *std::max_element(v.cbegin(), v.cend());
}

int main()
{
    if (1)
    {
        std::cin.tie(0);
        std::ios::sync_with_stdio(false);
    }

    unsigned N;
    long long a;
    std::vector<long long> vec;

    std::cin >> N;
    repi(N)
    {
        std::cin >> a;
        vec.push_back(a);
    }

    std::cout << minVec(vec)
              << " "
              << maxVec(vec)
              << " "
              << sumVec(vec)
              << std::endl;

    getchar();
    return 0;
}

コード②：全て正解となる

#include <cstdio>
#include <iostream>
#define repi(n) for (int i = 0; i < n; i++)

int main()
{
    unsigned n;
    long long v, max, min, sum;
    max = -1000000;
    min = 1000000;
    sum = 0;

    std::cin >> n;
    repi(n)
    {
        std::cin >> v;
        max = std::max(max, v);
        min = std::min(min, v);
        sum += v;
    }

    std::cout << min << " " << max << " " << sum << std::endl;
    getchar();
    return 0;
}

データサンプル１

入力：
3
1 2 3
出力：
1 3 6

データサンプル２

入力：
5
10 1 5 4 17
出力：
1 17 37

補足情報（FW/ツールのバージョンなど）

Win10
VSC1.47.3
C++14で回答提出

Answer 1

- return std::accumulate(v.begin(), v.end(), 0);
+ return std::accumulate(v.begin(), v.end(), 0LL);

初期値を long long で渡さないとintとして計算してオーバーフローするようです。
(もうちょっと賢く推論してくれるのかと思いましたが)

Comments

[mjk]

ありがとうございます！
通りました！

[episteme]

そっちかー... std::reduce(v.begin(), v.end()) なら初期値要らんはず。

[mjk]

std::reduce知らなかったので勉強になります。

[episteme]

C++17 以降なのでAOJには蹴られるかもです。

[mjk]

確かにAOJはC++14でした。
でも勉強になりましたありがとうございます。

Answer 2

// 最小値 vector
long long minVec(std::vector<long long> v)
{
    return *std::min_element(v.cbegin(), v.cend());
}
// 最大値 vector
long long maxVec(std::vector<long long> v)
{
    return *std::max_element(v.cbegin(), v.cend());
}

このふたつ、引数を参照で渡さんとならんのでは?
long long minVec(std::vector<long long>& v)
long long maxVec(std::vector<long long>& v)
※ ついでに const つけておきたいかな。

[追記] ごめん、vectorのコピーが時間切れを引き起こすんだと思ってた。原因はコレじゃないポい。
※ 削除するのも失礼かと思い、残しておきます。

Comments

[mjk]

アドバイスありがとうございます。
// 最小値 vector
long long minVec(std::vector<long long> const &v)

引数の参照渡し付けてみましたが結果変わらずでした。
（うろ覚えで意味理解してませんが保険とおまじないみたいになってます）