🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中!

\teratail特別グッズやAmazonギフトカード最大2,000円分が当たる!/

詳細はこちら
C++

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

Q&A

解決済

1回答

2893閲覧

int型の値が勝手に変わる現象を解決したい。

negitam-31

総合スコア44

C++

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

0グッド

0クリップ

投稿2020/12/18 01:56

編集2020/12/18 02:15

前提・実現したいこと

int型二次元配列で、条件を満たす要素番号の要素だけ値を1にし、満たさない要素番号の要素を0とするチェック表のようなデータから、1である要素の要素番号の組をまとめた二次元配列を作成したいです。

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

作成した二次元配列の一部の値が勝手に変わっています。

該当のソースコード

int main(){}の中の一部です。

C++

1int connections[connectCnt][2], element=0; 2 for (int i=0; i<N; i++) 3 { 4 for (int k=0; k<N; k++) 5 { 6 if (joiningEvent[i][k]==1) 7 { 8 connections[element][0]=i; 9 connections[element][1]=k; 10 cout << connections[element][0] << " : " << connections[element][1] << "・甲" << endl; 11 element++; 12 } 13 } 14 } 15 16 for (int i=0; i<connectCnt; i++) 17 { 18 cout << connections[i][0] << " : " << connections[i][1] << "・乙" << endl; 19 }

↑の補足)
int JoiningEvent[][] … int型二次元配列で、条件を満たす要素番号の要素だけ値を1にし、満たさない要素番号の要素を0とするチェック表のようなデータ
int connections[][2] … int型二次元配列で JoiningEvent[n][m]=1 の時の n が connections[i][0] に代入され、m が connections[I][1] に代入される。


output

10 : 1・甲 20 : 2・甲 30 : 3・甲 40 : 4・甲 51 : 2・甲 61 : 4・甲 73 : 0・甲 83 : 2・甲 93 : 4・甲 104 : 2・甲 110 : 1・乙 120 : 2・乙 130 : 3・乙 140 : 4・乙 151 : 2・乙 161 : 4・乙 173 : 0・乙 183 : 2・乙 193 : 4・乙

出力(output)の甲と乙のそれぞれの最後だけ違う値になっていて、本来は同じ値であるはずなのに値が勝手に変化しています。

使用するツール・バージョン

使用しているツールはVisual Studio Code バージョン1.49.1
拡張でCodeRunnerを導入しています。

その他・補足

実は以前にもっと簡単な処理でも似たような現象が起きております。
ただその時は、一桁の整数を代入していたのに-27341などの極端に大きいまたは小さい値に変わっていたので今回と同じバグではないかもしれません。今回はもしかしたら私のミスかもしれません、細かく出力してみたのですがうまくいきませんでした。もし原因が私だったらすみません。(まだC++初心者なので...)

追記スクリプト

以下のスクリプトがjoiningEventの代入とconnectCount(connectionsの必要な要素数を測定するためのint変数)の代入をしているソースコードになります。

C++

1int joiningEvent[N][N], connectCnt=0; 2 for (int i=0; i<N; i++) 3 { 4 for (int k=0; k<N; k++) 5 { 6 if (P[i]==P[k] && (S[i]+1)<=S[k]) 7 { 8 joiningEvent[i][k]=1; 9 connectCnt++; 10 } 11 else if (P[i]!=P[k] && (S[i]+D+1)<=S[k]) 12 { 13 joiningEvent[i][k]=1; 14 connectCnt++; 15 } 16 } 17 }

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

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

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

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

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

guest

回答1

0

ベストアンサー

for (int i=0; i<connectCnt; i++)

前半処理でconnections[][]に値を突っ込んだ個数と,
このforで表示している個数が異なっている.

投稿2020/12/18 02:04

fana

総合スコア11985

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

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

fana

2020/12/18 02:08

connections[][]のサイズが足りて無さそう. connectCntの値が9なのに,ifの条件を満たすパターンが10個あったように見える.
negitam-31

2020/12/18 02:12

回答ありがとうございます。私の説明不足でしたすみません。 connectionsの要素数はconnectCntでfor文の回数と等しいので一致しています。 ここの質問に挙げていなかったconnectCnt代入のソースコードを追記しておきます。少々お待ちください。
negitam-31

2020/12/18 02:18

あ、すみません。今実験中です。
negitam-31

2020/12/18 02:19

ほんとですねconnectCnt=9でした。
negitam-31

2020/12/18 02:20

ありがとうございます。自己解決を試みます。
fana

2020/12/18 02:42

提示されたoutputの甲と乙の個数が異なってますから,まぁ,処理を見直すべきでしょう. あと, 追加されたコードでのカウンタ connectCnt と,配列 connections[connectCnt][2] のサイズを決めている connectCnt とは別物なのでは.
negitam-31

2020/12/19 02:12

ありがとうございます、/(=_=)\ペコリ
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.36%

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

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

質問する

関連した質問