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

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

新規登録して質問してみよう
ただいま回答率
85.48%
C++

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

Q&A

解決済

1回答

1146閲覧

atcoder ABC105 B 論理エラーの原因

h_gnome

総合スコア13

C++

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

0グッド

0クリップ

投稿2019/02/04 13:45

前提・実現したい

こちらのサイトで見つけた、atcoderABC 105 Bを解いてみたのですがWAになりその原因が分かりません。
サイトをよく観察して別のアプローチで書き直したらACを得られたのですが、先のコードでなぜWAになるのかは分からずじまいです。
書き方を変えたのはfor文内の全探索です。
###問題文
ABC 洋菓子店では, 1個 4ドルのケーキと 1個 7ドルのドーナツが売られている.
このとき, 合計金額が Nドルとなる買い方はあるか, 判定せよ.
ただし, 同じ商品を二個以上買っても良く, 買わない商品があっても良いものとする.

該当のソースコード

WAのコード(c++)

#include <bits/stdc++.h> using namespace std; int main(){ int N; cin >> N; int c=4,d=7; string ans="No"; //ここから異なる for(int i=0;i<26;i++){ c *= i; for(int j=0;j<=16;j++){ d *= j; if((c+d)==N){ ans="Yes"; } } } cout << ans << endl; return 0; }

ACのコード

#include <bits/stdc++.h> using namespace std; int main(){ int N; cin >> N; int c=4,d=7; string ans="No"; //ここから異なる for(int i=0;i<26;i++){ for(int j=0;j<=16;j++){ if((c*i+d*j)==N){ ans="Yes"; } } } cout << ans << endl; return 0; }

###補足情報
正答率

列2ケース名
in01.txtAC
in02.txtAC
in03.txtWA
in04.txtAC
in05.txtAC
in06.txtWA
in07.txtWA
in08.txtAC
in09.txtAC
in010.txtWA
in011.txtAC
in012.txtWA
in013.txtWA
in014.txtAC
in015.txtWA
in016.txtWA
in017.txtWA
in018.txtWA
in019.txtWA
in020.txtWA
sample_01.txtWA
sample_02.txtWA
sample_03.txtAC

###最後に
どちらも全ての通りをたどっている様に見えるですが、なぜWAを吐かれるのか分かりません。
なにか原因が分かる方はご回答いただけると幸いです。

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

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

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

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

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

guest

回答1

0

ベストアンサー

C++

//ここから異なる
for(int i=0;i<26;i++){
c *= i;
for(int j=0;j<=16;j++){
d *= j;
if((c+d)==N){
ans="Yes";
}
}
}

c及びdに代入した値が、次の周回に持ち越されているからでは。
一時変数を用意すれば問題は発生しないかと。


ところで、『7ずつ引いて行って、4の倍数になったらOK』って戦略ならO(n)ですね。

C++

1#include <iostream> 2 3int main(void) { 4 const int N = []() { 5 int tmp; std::cin >> tmp; return tmp; 6 }(); 7 8 for(int n = N; n >= 0; n -= 7) { 9 if(n % 4 == 0) { 10 std::cout << "Yes\n"; 11 return 0; 12 } 13 } 14 15 std::cout << "No\n"; 16} 17

こんなふうに。

投稿2019/02/04 13:50

編集2019/02/04 14:01
LouiS0616

総合スコア35660

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

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

h_gnome

2019/02/04 14:01

確かに余分にかけることになりますね。 一周期ごとにc,dをリセットしたらACでました! ありがとうございます。 ``` #include <bits/stdc++.h> using namespace std; int main(){ int N; cin >> N; int c=4,d=7; string ans="No"; for(int i=0;i<26;i++){ c *= i; for(int j=0;j<=16;j++){ d *= j; if((c+d)==N){ ans="Yes"; } //リセット d=7; } //リセット c=4; } cout << ans << endl; return 0; } ```
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問