C言語 変数の値の変動が思うように動かない

Question

下記ソースコードが思うように動きません。
配列の中にいる変数Aを13、Bを04の間で動かしたいのですがデバックしてみると両方とも
-858993460
という値になっています。
A、Bはソースコード上の下記部分でしか使用していません。
A、Bの動きの破たんしている個所の解説をお願いしたいです。

##各変数について
各変数は
S[15]={2,2,2,2,2};
TM[4]={0};
P[4]={0};
T[4][5]={0};
J[4]={0};
で初期化しています。

#ソースコード
#define W(5)

int main()
{
.
.
.
M = (W) / 5;
if (((W) % 5) != 0) {
Rem2 = W % 5;
}
else
{
Rem2 = 0;
}

	for (A = 1; A <= 3; A++) {
		do {
			if (M > 0) {
				TM[A] += (5 * 10) * 2;
				P[A] = 5;
			}
			else if (M == 0) {
				TM[A] += (Rem2 * 10) * 2;
				P[A] = Rem2;
			}
			else {
				break;
			}
			for (B = 0; B < P[A]; B++) { 　//ここが原因？
				T[A][J[B]] = S[J[B]];
			}
			if (M > 0) {
			M--;
			}


			MF[A] = T[A][P[A] - 1];

			if (MF[A] > 0) {
				TM[A] += ((MF[A] - 1) * 5) * 2;
			}
			else
			{
				break;
			}
		} while (0);
		if (M < 0) {
			break;
		}
	}

Comments

[kazto]

Passengerの宣言部を省略せず記載してください。

[carlton]

失礼しました。PassengerはここでいうPにあたります。Jは０で初期化しています。
以上二項目修正しました。

Answer 1

-858993460この数字はMSVCのデバッグ版において**uninitialized locals(stack)**になります。

初期化される前に数値を表示しているのではないでしょうか？

もしくは提示されたソース部分が実行されていないのではないでしょうか？

Comments

[carlton]

確かに、別途初期化を入れてみたら数値の変動がみられました。
他のループ文にはまだ修正箇所があるようですがこの-858993460に対する的確なご指摘を頂けましたのでBAにさせていただきました。
他の皆様もありがとうございました。

Answer 2

ざっとみた感じは B の for ループで今回の提示情報にない物が利用されているようです
・Passenger[A]
・J[B]
このいずれかの初期化に不備があると仮定すると
ケース1: Passengerの配列が初期化されていないかPassenger[3]以下の領域しかない
ケース2: Jの配列が初期化されていないかPassengerの配列に格納されている値がJの配列の大きさを超えている
という可能性が考えられます

Answer 3

BのループでPassenger[]を使用していますが、この配列の値がおかしなことになっているのでは。
結果Bの値がJ配列の要素以上を指してしまい、Bのループ内のT[A][J[B]]がたまたまAの場所で、値が上書きされてしまっているのではないでしょうか。
Passenger配列を生成しているところを提示したほうがよいでしょう。

Answer 4

デバッガはなにをお使いでしょうか。
EclipseとかVisualStudioとかであれば、任意の場所にブレークポイントを設定してそこで止めて変数の値を参照したり、そこから１行づつ実行して変数を見ることもできます。
そうやって動作を追いかけていけばどうでしょう