質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.35%
アルゴリズム

アルゴリズムとは、定められた目的を達成するために、プログラムの理論的な動作を定義するものです。

C++

C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

Q&A

1回答

1751閲覧

このコードが正しい答えを出力できない理由を教えてください。

XionCode

総合スコア46

アルゴリズム

アルゴリズムとは、定められた目的を達成するために、プログラムの理論的な動作を定義するものです。

C++

C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

0グッド

0クリップ

投稿2020/05/30 05:38

編集2020/05/30 05:59

解決したい問題の内容

言語はC++です。AtCoderの問題アルゴリズム検定過去問G問題を解いていました。
https://atcoder.jp/contests/past201912-open/tasks/past201912_g
↑このリンク先の問題です。
再帰関数を用いてコードして提出しました。
しかし、コンパイルは通ったもののWAの結果が出ました。
WAの原因とコードの改善点を知りたいです。

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

下のリンク先のテストケースを見ていただくとわかる通り、ばらばらにACとWAが出ました。

該当のソースコード

C++

1#include <iostream> 2using namespace std; 3 4void happiness(int cnt); 5int** pHappiness; 6int* pGroup; 7long long* pHappy; 8long long ans; 9int N; 10 11void Happiness(int cnt){ 12 int preHappy; 13 for(int i = 0; i < 3; ++i){ 14 pGroup[cnt] = i; 15 if(cnt == 0) *pHappy = 0; 16 if(i == 0) preHappy = 0; 17 else *pHappy = *pHappy - preHappy; 18 19 for(int j = 0; j < cnt; ++j){ 20 if(pGroup[j] == pGroup[cnt]) preHappy += pHappiness[j][cnt-1]; 21 } 22 *pHappy += preHappy; 23 24 if(cnt < N-1){ 25 Happiness(cnt+1); 26 }else{ 27 if(ans < *pHappy) ans = *pHappy; 28 } 29 } 30 *pHappy -= preHappy; 31 return; 32} 33 34int main(void){ 35 cin >> N; 36 pHappiness = new int*[N-1]; 37 for(int i = 0; i < N-1; ++i){ 38 pHappiness[i] = new int[N-1]; 39 //pHappiness[i][j] = Happiness between i and i+j+1 40 for(int j = 0; j+i < N-1; ++j) cin >> pHappiness[i][i+j]; 41 } 42 43 int group[N]; 44 45 pHappy = new long long; 46 pGroup = group; 47 ans = -2147483648; 48 Happiness(0); 49 50 cout << ans << endl; 51}

最終的にこのようなコードで提出しました。

https://atcoder.jp/contests/past201912-open/submissions/13699538
↑問題のコードのリンクです。提出結果もこちらです。

試したこと

いくつかACが存在するので、入力された値の特殊な場合をはじきだせてないのが原因だと思います。

・変数のオーバーフローについては、余裕をもって大きい値はlong longに入れてあるはずなので、ないと思います。実際すべてのペアの幸福度を1,000,000にしてやってみましたが正常作動でした。
・幸福度を-1,000,000や0にした場合についてもうまくいっていました
・幸福度を正の値、負の値交互に入れて見たりしました。うまくいっていました

他のACを獲得しておられる方のコードと、上記の入力を試して出力を比較してみたりしたのですが、結局原因がわかりません。

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

テストはAtCoderに組み込まれているコードテスト機能を使っています。

私事ですが、今回初めてteratailで質問します。
質問の仕方、こうしたほうがいいよ!
というアドバイスがありましたら、そちらもぜひ教えていただきたいです。

よろしくお願いします。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

y_waiwai

2020/05/30 05:48

このようなコードじゃなく、あなたが書いたコードを提示しましょう 質問を編集し、<code>ボタンを押し、出てくる’’’の枠の中にコードを貼り付けてください
guest

回答1

0

preHappyがループの間足され続けてます。本来ループごとに0にリセットするはずです。

text

15 2-100 -100 1 1 3-100 1 1 41 1 50

このようなテストケースで間違った答えが出ることが確認できると思います。

投稿2020/05/30 10:13

yudedako67

総合スコア2047

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

XionCode

2020/05/31 03:08

関数Happiness()の6行目  if(i == 0) preHappy = 0;  else *pHappy = *pHappy - preHappy; の部分の部分を、  if(cnt == 0) *pHappy = 0; if(i) *pHappy = *pHappy - preHappy; に変更ということでしょうか? やってみると正解からさらに離れてしまったので、多分そういうことではないですよね。 もう少し具体的にお願いします!すみません…
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問