プログラムが停止する

Question

ビジュアルスタジオでc＋＋を使ってプログラミングをしています。
その中でif(a[ ]==b[ ][ ])という様なプログラムを書いたのですがここでプログラムが停止してしまいます。
printf で二つの値を確認しても値が入っていることも確認しました。

アドバイスお願いします

```ｃ++

double combimin7,test,test2;
combimin7 = 1000000;
for (x = 0; x < n; x++){
if (distances[x] > 0){
if (x != t){
if (x != u + 1){
if (x != t1){
if (x != u1 + 1){
if (x != t2){
if (x != u2 + 1){
if (x != t3){
if (x != u3 + 1){
if (x != t4){
if (x != u4 + 1){
if (x != t5){
if (x != u5 + 1){
if (distances[x] <= combimin7){
test2 = powerx[x];
if(test2 == twopowerx[t][u]){
test = powerx[t6] + 50;
}
else{

																combimin7 = distances[x];
																t6 = x;
																test = powerx[t6];
															  }
															}
														}
													}
												}
											}
										}
									}
								}
							}
						}
					}
				}
			}
		}
	}

Comments

[maisumakun]

エラーが再現するような、最小限のコードを示してください。

[hippon]

For(x=0; x<3;x++){
if(a[x]=b[1][1]){
y=a[x]+50;}
else{y=a[x];}
}
初心者なので、どこまで示せばいいのかわからないのですいません

[LouiS0616]

if文で代入しているように見えますが。転記ミスかどうかもわからないので、よほど長大じゃなければ全部張った方が早くて確実です。

[hippon]

先輩から引き継いだプログラムなのですが、とても長いプログラムなので全部張れそうにありません。もう少し前後を切り取って投稿させていただきます。わかりにくくて申し訳ございません。

[hippon]

double combimin7,test,test2; combimin7 = 1000000; for (x = 0; x < n; x++){ if (distances[x] > 0){ if (x != t){ if (x != u + 1){ if (x != t1){ if (x != u1 + 1){ if (x != t2){ if (x != u2 + 1){ if (x != t3){ if (x != u3 + 1){ if (x != t4){ if (x != u4 + 1){ if (x != t5){ if (x != u5 + 1){ if (distances[x] <= combimin7){ test2 = powerx[x]; if(test2 == twopowerx[t][u]){ test = powerx[t6] + 50; } else{ combimin7 = distances[x]; t6 = x; test = powerx[t6]; } } } } } } } } } } } } } } } }

[LouiS0616]

質問文に張ってください。さすがにこれは読めません。質問を編集する際には、コードを選択した状態で<code>ボタンを押して下さい。読みやすくなります。

[LouiS0616]

幼稚園児の落書きみたいに酷いプログラムです。今すぐ作り直すことをお勧めします。今動いているのはほとんど奇跡ですし、ここを乗り越えても必ずどこかでまた躓きます。

[hippon]

プログラミングのわかる友人にも同じようなことを言われました。お手数おかけしました。又、時間を割いていただきありがとうございました。

[LouiS0616]

これを引き継いだと考えると、ほんと同情します... 頑張ってください。

Answer 1

xが12個の何らかの数字でない時にだけpowerx[x]を使いたいのかな？
このifの羅列を通った時にようやくtestに値が代入されますが、それ以外の時ははどうするんでしょうか？

そもそも、xはdistances[]とpowerx[]の添字の範囲を超えなければ、提示された部分では問題は無いはず。最後のif文の中の演算にしても50を足しているだけなので、doubleの範囲を通常だと超えるとは思えない。

それなら、このifは全て外してしまって、

for (x = 0; x < n; x++){
  if (0 < distances[x] && distances[x] <= combimin7){
      if(powerx[x] == twopowerx[t][u]){
         test = powerx[t6] + 50;
      }
      else{
        combimin7 = distances[x];
        t6 = x;
        test = powerx[t6];
      }
  }
  //　ここでtestの値を判定する
}

とした方が断然読みやすくなります。

ただし、t,u,t6はfor文の前で初期化しておく必要があります。

Comments

[hippon]

回答ありがとうございます
確かにこのループに入らない時の処理をなにもしていませんでした。
アドバイスを元にもう少し取り組んでみます。