一つのプログラムを自由に操りためには

質問

c言語上達のためには自分で課題を考えて一つのプログラムを自由に操れるようになるほうが早いと思い最近は同じものばかりやっているんですが、つまずきます。質問させて下さい
#include <stdio.h>
#include <stdlib.>

#include <time.h>

#define S_NUMBER 5
#define N_NUMBER 50

int main( void )
{

int s;
int s_number[S_NUMBER];
int n;
int n_number[N_NUMBER];
int m;
int sum;
int pum;
int j;

printf( データA\n");

for( s = 0; s < S_NUMBER; s++ )
{
scanf( "%d", &s_number[s] );
}

printf( "データB\n" );

for( n = 0; n < N_NUMBER; n++ )
{
scanf( "%d", &n_number[n] );
}

srand( time ( NULL ) );

for( m = 0; m < 5; m++ )
{
do{
for( j = 0; j < 100; j++ )
{
int x = rand() % n;
int y = rand() % n;
int tmp;

tmp = n_number[x];
n_number[x] = n_number[y];
n_number[y] = tmp;
}

sum = 0;
pum = 0;

for( j = 0; j < 4; j++ )
{
sum += n_number[j];
if( ( n_number[j] % 2 ) == 0 )
++pum;
}
}

while( sum < ( 70 - s_number[s] ) || sum > (150 - s_number[s] ) || pum < 2 || pum > 3 );

for( j = 0; j < 4; j++ )

printf( "%d %d \n", s_number[s], n_number[j] );
printf( "\n" );
}

return 0;
}
たとえば、 5 11 36 78 109 という ５つの数字の表示をすることを考えます。
最初の5はデータAより、残りの11 36 78 109 はデータB よりデータBはシャッフルして表示する、なおかつ
5つの合計が70-データA(この場合5)~150-データA(この場合5)の範囲内として5つの数字の割合が奇数2:偶数3か奇数3:偶数2
とする。といゆうふうにしたいんですが、上記のコードではどこが違っているのかしりたいです。
直す箇所の訂正もお願いします。

回答No.1

Tacosan · ベストアンサー

逆に聞いてみよう.

「上記のコードではどこが違っているのかしりたいです」と書いているということは, 「このプログラムは何かがおかしい」と思ったわけですよね? いったいどのような事実を根拠として「このプログラムは何かがおかしい」と思ったのですか?

それ以前に, 最後の「たとえば」の例は明らかにおかしいです. 「5 11 36 78 109」という 5つの「合計」は 239 であり, これは「5つの合計が70-データA(この場合5)~150-データA(この場合5)の範囲内として」の条件にあてはまっていません. つまり, そもそもの問題として「あなた自身が問題を理解できていない」のではありませんか?

お礼

あんたらは、日本語わからねーのかよ！

補足

たとえばの数字は思いつきで書いたものであって根拠はないです。データ入力は超えないすうじがはいります。

回答No.3

wormhole

>たとえばの数字は思いつきで書いたものであって根拠はないです。データ入力は超えないすうじがはいります。

例えば、以下のような質問があったとします。

>複数の数字を入力してその合計を出力するには、どのようなコードを書けばいいですか。
>例) 1 2 3 を入力したら合計として 456 が出力される。

これ作れますか？
こういう場合大抵、問題が間違ってるのか例が間違ってるのか悩むことになります。

なので例をあげるならあげるで適切な例をあげないと意味ないですよ。

回答No.2

kmee

「間違い」の基準をはっきりさせてください。
極端な話し、「『エラーが出てコンパイルできない』プログラム」を作ろうとして、「エラーが出てコンパイルできない」なら、目的通りの「正しいプログラム」です。

まずは、何が「正しい状態」なのかを、説明してください。
一つ例示されただけでは、それ以外はどうするのか、がはっきりしません。

「こういうことをしたいから、こういう方法を使って、こうする。」
「そうしようとしたけど、こうなってしまう。だから、ここをこうしよう」
こういうことは、ベテランでも、ちゃんと考えないと正しいプログラムは作れません。

> 最初の5はデータAより

入力したデータAのどの値もプログラムで使われていません。(入力直後の一覧表示を除いて)
s_number[s] がどうなるか、理解できていますか?

> 5つの合計
> 5つの数字の割合が奇数2:偶数3か奇数3:偶数2
どちらも、そのようになっていません。
変数 sum が何の合計になっているか、変数pumが何の数になっているか、もう一度プログラムの動作を確認してください。
あなたが「5つの合計」のつもりでいても、プログラムはそうは動いてくれません。

> 5つの合計が70-データA(この場合5)~150-データA(この場合5)の範囲内

既に指摘があるように、入力したデータ次第では該当する組合せが無く、無限ループになります。
「該当する組合せが無い場合は無限ループする」というのが「正しい動作」ならこのままでいいですが。