C言語なら俺に聞け 161

2023/04/21(金) 14:05:20.18

!extend:checked:vvvvv:1000:512
(新スレ立ての際上記コマンドを2行書き込んでください)
C言語の話題のみ取り扱います C++の話題はC++スレへ
質問には最低限の情報(ソース/コンパイラ/OS)を付ける
数行で収まらないソースは以下を適当に使ってURLを晒す
https://paiza.io/
https://ideone.com/
http://codepad.org/

C17
http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2017/n4713.pdf

C11
http://www.open-std.org/jtc1/sc22/WG14/www/docs/n1570.pdf

C23 最新ドラフト
https://www.open-std.org/jtc1/sc22/wg14/www/docs/n3047.pdf

C99
http://www.open-std.org/jtc1/sc22/WG14/www/docs/n1256.pdf
http://kikakurui.com/x3/X3010-2003-01.html

C FAQ 日本語訳
http://www.kouno.jp/home/c_faq/

JPCERT C コーディングスタンダード
https://www.jpcert.or.jp/sc-rules/

※前スレ
C言語なら俺に聞け 160
https://mevius.5ch.net/test/read.cgi/tech/1672191630/

VIPQ2_EXTDAT: checked:vvvvv:1000:512:: EXT was configured

2023/05/04(木) 07:37:53.19

>>93
二つ目の for ループから抜ける条件の一つが「a[j]がゼロ」なのに、そのループから抜けた直後で i % a[j] とかやってて、jが2のときにa[2]が0でループから抜けるから、ゼロ除算で落ちる

それとこっちは直接の原因ではないけど、int a[101]と宣言すると a[0]～a[100]までしかアクセスできないのにa[102]までアクセスする可能性があるようなコードになってるのもダメだな

2023/05/04(木) 08:36:39.17

とりあえず初期化するとき

int a[101] = {0,1,2};
と書いたほうが楽じゃん
ちなみに初期値がない部分は0になる

後はめんどくさいから明日読むわ

2023/05/04(木) 08:46:33.13

間違えた
int a[101] = {1,2};
か

ちなみに int a[101] = {};
ではまったく初期化されないからこのプログラムでは[0][1]以外のところには0ではなく不定値が入ってるのでうまく動かないのはそれなんじゃね

int a[101] = {0};
と書いておけば省略した部分もすべての要素が0で初期化されるよ

2023/05/04(木) 09:36:07.29

>>93
i　と　j 　ってわかりづらいから　j は　k に置き換えて言うけど

i=4　のとき k=0 で　4%2 となり割り切れるため、何もせずに次
i=5 のとき　k=2 で　a[k]==0 なので k　のループを抜ける（このときk==2でa[2])
抜けた後のif (r) a[k + 1] = i;　で　a[3] = 5 になっていて　a[2]　は 0 のまま
で0除算でドボン

まずはif (r) a[k + 1] = i;　ではなく　if (r) a[k] = i;　だね

2023/05/04(木) 09:58:25.07

>>93
それとこれって素数を見つけるプログラムと思うけど、　1000までの間に163個あるっぽい
a　の要素数はそれ以上必要
（0除算している部分のprintfがデバッグ用のように、デバッグで100までに制限しているのかもしれないけど）

一応言っておくと0除算でドボンは本質とはちょっと違うからね
if (r)　a[j] = i;
printf("%d %d %d %d %d \n", i, j, a[j], i % a[j], r);
とa[j] への代入とデバッグ用のprintfの位置を入れ替えれば0除算は起きないから