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

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

新規登録して質問してみよう
ただいま回答率
85.46%
プログラミング言語

プログラミング言語はパソコン上で実行することができるソースコードを記述する為に扱う言語の総称です。

C++

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

Q&A

2回答

1040閲覧

ある条件の数字の回数を数える

yyy3

総合スコア0

プログラミング言語

プログラミング言語はパソコン上で実行することができるソースコードを記述する為に扱う言語の総称です。

C++

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

0グッド

0クリップ

投稿2021/11/07 20:54

編集2021/11/07 21:03

前提・実現したいこと

C++ で一つ前の結果より今回の結果の方がいい場合の回数をカウントするプログラムを書いて見たんですけどうまくいきませんでした。
マラソン大会 をイメージしているので一つ前のトラックよりタイムがいい場合の回数を表示したいです

ここに質問の内容を詳しく書いてください。
(例)PHP(CakePHP)で●●なシステムを作っています。
■■な機能を実装中に以下のエラーメッセージが発生しました。

順位 分 秒 となっています

6
4 3 12
1 2 34
3 2 0
2 1 33
5 2 10
1 1 40

と入れると

が帰ってくると正解です

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

エラーメッセージ

該当のソースコード

c++
ソースコード

### 試したこと```こ```ここに言語を入力 #include <iostream> using namespace std; int main() { int race[100]; int n, r, m, s; cin >> n >> r >> m >> s; for (int i = 0; i < n; i++){ cin >> race[i]; } int cnt = 0; for (int i = 0; i < n; i++){ if (race[i] > race[i-1]){ cnt++; } } cout << cnt << endl; return 0; } ```こに言語を入力 コード

ここに問題に対して試したことを記載してください。
上のコードを書いてみました問題点や改善点を教えていただけると幸いです

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

ここにより詳細な情報を記載してください。

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

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

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

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

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

episteme

2021/11/07 23:05

説明不足。「うまくいきませんでした」だけで済ませてはならない。
dodox86

2021/11/07 23:10

なんとなくコードを書いて「どこが悪いですか?」とだけ聞いているということはないでしょうかね。
guest

回答2

0

cin >> n >> r >> m >> s;
for (int i = 0; i < n; i++){
cin >> race[i];
}

一度 n, r, m, s を読み、その後 n回 race[i] を読んでいるが、
それがあなたのやりたいことか?
一度 nを読み、その後 n回 r, m, s を読むんじゃないのか?

for (int i = 0; i < n; i++){
if (race[i] > race[i-1]){

i=0のとき、race[-1]を参照してる。

投稿2021/11/07 22:58

編集2021/11/07 23:04
episteme

総合スコア16614

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

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

0

データの個数は最初の1回だけ読めばよい。
順位、分、秒を指定回数だけ繰り返し読む。
1分=60秒。
直前のデータと比較してはならない。ベストタイムと比較する必要がある。
追記:寝起きで条件を勘違いしてましたので、この項目は削除します。代わりに、-1番目の要素と比較してはならない、を追加します。

配列を使わないほうがシンプルかも?

C

1#include <iostream> 2 3int main() 4{ 5 int n, r, m, s; 6 int prevsec = 0, nowsec, better = 0; 7 8 std::cin >> n; 9 for(int i = 0; i < n; i++){ 10 std::cin >> r >> m >> s; 11 nowsec = m*60 + s; 12 if(prevsec > nowsec){ 13 better++; 14 } 15 prevsec = nowsec; 16 } 17 std::cout << better << std::endl; 18}

なお、入力が想定外である場合は考慮しない。それぐらいは自分で考えてください。

投稿2021/11/07 21:54

編集2021/11/08 03:29
majiponi

総合スコア1720

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

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

yyy3

2021/11/07 21:57

物分かり悪くてすみません。もう少し説明していただけるとありがたいです。
episteme

2021/11/07 22:55

「一つ前の結果より今回の結果の方がいい場合の回数」と明示されているのに 「直前のデータと比較してはならない。ベストタイムと比較する必要がある。」なの?
majiponi

2021/11/07 23:28

あ…。実行サンプルが、ベストタイムの回次を結果としていたから間違えた…。そこは忘れて。代わりに、 -1番目の要素と比較してはならない。 を追加して。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

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

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

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

ただいまの回答率
85.46%

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

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

質問する

関連した質問