C言語の設計ミスった危険な関数トップ10決めようぜ

**デフォルトの名無しさん** · 2019/11/21(木) 12:23:44.93

gets関数とか

その他の例
https://www.jpcert.or.jp/sc-rules/c-msc24-c.html

**デフォルトの名無しさん** · 2019/12/02(月) 02:04:59.50

>>1
セキュリティ考慮していない元関数より、代替関数の設計ミスが酷いな

strtodとかなにこれw
エラーの返し方がアホすぎる

**デフォルトの名無しさん** · 2019/12/02(月) 06:48:47.55

>>32
> strtodとかなにこれw
> エラーの返し方がアホすぎる
そう言うのは改善案を提案しないと単なるアホの独り言にしかならんぞw

**デフォルトの名無しさん** · 2019/12/02(月) 13:15:48.70

一体どういう考えで昔の人は、getsの仕様はあれでOKだと思ったんだろう？
どう考えてもバッファオーバーフローするやん？
しない方法なんて存在するの？

**デフォルトの名無しさん** · 2019/12/02(月) 13:18:13.53

バッファオーバーフローしないデータを食わせれば良い
わざわざ変なデータを入力するヤツが悪い

って考えの時代の関数だ

**デフォルトの名無しさん** · 2019/12/02(月) 13:41:09.67

でもさ、getsってキーボード入力やろ？

**デフォルトの名無しさん** · 2019/12/02(月) 15:30:07.33

gets は、もう、標準ライブラリにも入っていないので許せ

**デフォルトの名無しさん** · 2019/12/02(月) 18:02:20.65

>>36
わざわざ変なデータを入力するヤツが悪い

**デフォルトの名無しさん** · 2019/12/02(月) 18:41:59.62

ダンディ坂野「gets!!」

**デフォルトの名無しさん** · 2019/12/02(月) 20:50:15.44

>>36
１行80文字とかって決め打ちできる環境なんでしょ
自分しか使わないようなツールとかならこれで十分
って考え

**デフォルトの名無しさん** · 2019/12/03(火) 01:51:02.34

8ビットの整数型を　char などというものにしたのも、アメリカ人は
自分たちの英語しか知らない田舎者であることを如実に表している。

**デフォルトの名無しさん** · 2019/12/03(火) 09:14:31.49

>>40
64kbでも広大な空間だったな

**デフォルトの名無しさん** · 2019/12/03(火) 12:22:07.21

>>41
実に田舎者の考え方で笑える
逆だよ、都会の人間は都会のことしか知らないし知る必要がない
なぜか田舎の人間は都会のことも知ろうとするw

**デフォルトの名無しさん** · 2019/12/03(火) 12:29:35.88

> 逆だよ、都会の人間は都会のことしか知らないし知る必要がない
だから世界が狭くて無知になりやすい

> なぜか田舎の人間は都会のことも知ろうとするw
だから世界が広い

**デフォルトの名無しさん** · 2019/12/03(火) 12:51:15.68

井の中の蛙　大海を知らず
されど空の蒼さを知る

**デフォルトの名無しさん** · 2019/12/03(火) 13:25:13.55

C言語が出来た時の時代を考えれば十分

｢変数名に日本語が使えない言語は糞｣
って時代もそのうち来るよ

**デフォルトの名無しさん** · 2019/12/03(火) 19:14:24.81

>>44
残念ながらそうはなってないんだなw
狭く深くっていう世界だから
要するに田舎でな何をするにもオーバーヘッドが大きいってことな
今はネットでだいぶマシになったけどネット以前の情報収集考えたらアホでもわかる話

**デフォルトの名無しさん** · 2019/12/08(日) 02:33:17.34

アホなデータ食わせるアホなやつのために堅牢な仕組みにする必要はない

**デフォルトの名無しさん** · 2019/12/11(水) 01:29:51.60

>>41
元々は9bit

**デフォルトの名無しさん** · 2019/12/11(水) 06:55:08.36

>>46
メモリが640kバイトしか使えなかった頃からのものだしな

**デフォルトの名無しさん** · 2019/12/11(水) 07:59:10.55

>>5
ストラトック

**デフォルトの名無しさん** · 2019/12/11(水) 15:20:43.27

大昔って端末やエディタなんかでも1行256とか1024バイト程度しか扱えなかったばず

**デフォルトの名無しさん** · 2019/12/11(水) 15:32:22.68

256 の時代は割と長かった希ガス

**デフォルトの名無しさん** · 2019/12/13(金) 06:37:32.10

昔は1行の長さを先に決めてファイルをクリエイトしました
途中で変更はできません

**デフォルトの名無しさん** · 2019/12/14(土) 13:26:55.02

関数そのものじゃないが
breakにラベル指定出来なかったのは設計ミスやろ

**デフォルトの名無しさん** · 2019/12/14(土) 13:54:03.80

その為のgoto

**デフォルトの名無しさん** · 2019/12/14(土) 22:43:29.04

gotoはラベルのスコープが広いから使いにくい。ループごとに別のラベル名を
付けないといけないし、ラベル名を別のループ用のものと書き間違えると
意図しない所へ飛んで行ってしまう。スパゲッティでバグの元。

ループにラベルを付ければ、スコープがそのループ内に限定され、そのループの
外にある別のループに同じラベル名を付けることができるから、書きやすく
読みやすい。例えば、こんな感じ。

for @outer (i = 1; i <= 3; i++) {
　　for (j = 1; j <= 3; j++) {
　　　　printf("%d, %d\n", i, j);
　　　　if (i * j > 3) break @outer;
　　}
}
for @outer (i = 1; i <= 5; i++) {
　　for (j = 1; j <= 5; j++) {
　　　　printf("%d, %d\n", i, j);
　　　　if (i * j > 5) break @outer;
　　}
}

**デフォルトの名無しさん** · 2019/12/14(土) 22:45:51.60

単純に
breakする数を指定出来るだけで良い

**デフォルトの名無しさん** · 2019/12/15(日) 06:50:28.91

>>58
ネストの数を数えるとか最低だと思う

**デフォルトの名無しさん** · 2019/12/15(日) 07:00:06.17

1000行のコードでbreak 7;とかされても、
どこに抜けるかわからんよな！

**デフォルトの名無しさん** · 2019/12/15(日) 09:51:38.77

ネストの数がわからないような巨大な構造は
それだけで最低だ
そんなんだと名前付きループにした所で
@outerなんて名前が使い回されて
同じ名前のループがネストされて
それこそ難読コードになる
それならユニークなラベルによるgotoの方が良い

>>57くらいのループなら
break 2;
は名前を考える手間が無くて楽だし
見にくくもない

**デフォルトの名無しさん** · 2019/12/15(日) 09:57:28.64

とは書いたけど
どっちも有るならどっちも欲しい

どっちかだけげならbreak 2;の方が欲しい
って感じ

**デフォルトの名無しさん** · 2019/12/15(日) 10:52:04.89

>>61
> そんなんだと名前付きループにした所で
> @outerなんて名前が使い回されて
> 同じ名前のループがネストされて
> それこそ難読コードになる
無能の妄想w

**デフォルトの名無しさん** · 2019/12/15(日) 11:05:09.82

ループの先頭、ループの終わり、break部分
3箇所見ないとダメってだけでダメな仕様

**デフォルトの名無しさん** · 2019/12/15(日) 12:32:17.60

>>64
なぜか自分の前提は
> >>57くらいのループなら
なのにラベルにした途端にループの頭とbreakとループの終わりが一瞥でわからないとかアホすぎ

**デフォルトの名無しさん** · 2019/12/15(日) 12:36:08.74

大きければgoto
小さければbreak n

**デフォルトの名無しさん** · 2019/12/15(日) 12:38:50.49

名前を付ける手間を許容するならgotoという解決法が既にある

**デフォルトの名無しさん** · 2019/12/15(日) 13:48:44.02

>>60
ネスト変わるような変更あるたびにbreakの引数勝手に書き換えてくれるエディタとかマクロが流行る

**デフォルトの名無しさん** · 2019/12/15(日) 16:33:57.62

>>67
ifとgotoあるからforもwhileもdoも要らんという主張か
本物のプログラマーだなw

**デフォルトの名無しさん** · 2019/12/15(日) 16:52:03.21

歪んだ教育のせいで
gotoに拒絶反応を示す人が多いんだよな

2重ループを抜けるなら素直にgotoを使えって

**デフォルトの名無しさん** · 2019/12/15(日) 16:53:29.68

アセンブラの
jmp 1f
とかを C でやりたいという話なのか?
やってみりゃいいじゃん。プリプロセッサの出力を処理すればいい。
1 パスで出来ると思うよ。
ひょっとしてもう誰かがやってるかも。

**デフォルトの名無しさん** · 2019/12/15(日) 18:05:59.04

for (i = 1; i <= 3; i++) {
　　for (j = 1; j <= 3; j++) {
　　　　printf("%d, %d\n", i, j);
　　　　if (i * j > 3) goto outer;
　　}
} outer:;

**デフォルトの名無しさん** · 2019/12/15(日) 18:58:38.98

for (i = 1; i <= 3; i++) {
　　for (j = 1; j <= 3; j++) {
　　　　printf("%d, %d\n", i, j);
　　　　if (i * j > 3) goto outer;
　　}
}
printf("complete\n");
outer:;

**デフォルトの名無しさん** · 2019/12/15(日) 19:01:19.97

わざわざ言語拡張しなくても
>>57以上の事が実現出来る

**デフォルトの名無しさん** · 2019/12/16(月) 09:05:52.89

ラベルのスコープを狭くしたいという話のようだから
マクロの中で使ってこそ真価を発揮するんじゃないかな。
フル仕様のパースが必要というわけではなく、かなり手抜きできそうではある。
あれば便利というのは否定しないが、C でそこまで必要かというと…

**デフォルトの名無しさん** · 2019/12/16(月) 12:33:03.93

{
goto hoge;
hoge:;
}
{
goto hoge;
hoge:;
}
これで二重になってたら困るしある意味なってなくても困るな

**デフォルトの名無しさん** · 2019/12/16(月) 13:21:05.83

マクロで二重以上のループってのもなかなか考えづらいなあ

一応、ラベル名をパラメータとする手もある

使う側のコストを考えるとgotoの為にパラメータを増やさないよな
gotoを使わない主義者がよくやってるように
余分な変数を使って実装するんだろうね

**デフォルトの名無しさん** · 2019/12/16(月) 13:23:11.26

アセンブラだとマクロローカルラベルってのがあったりする
ラベルを頻繁に使うので

あと
@@/@b/@fのように、
直近のラベルを指定する仕組みもある

**デフォルトの名無しさん** · 2019/12/16(月) 13:28:49.41

>>72 >>73 みたいなのって
gotoの使いどころだと思うんだけだ
gotoを使わない人はどう書いてるの？

特に >>57 の答えを聞いてみたい

**デフォルトの名無しさん** · 2019/12/16(月) 15:17:28.75

void print_data(void) {
　for (i = 1; i <= 3; i++) {
　　　for (j = 1; j <= 3; j++) {
　　　　　printf("%d, %d\n", i, j);
　　　　　if (i * j > 3) return;
　　　}
　}
}
print_data();
printf("complete\n");

**デフォルトの名無しさん** · 2019/12/16(月) 16:34:59.64

outer より後に処理が続く場合は？

**デフォルトの名無しさん** · 2019/12/16(月) 16:51:44.02

void print_data(void) {
　for (i = 1; i <= 3; i++) {
　　　for (j = 1; j <= 3; j++) {
　　　　　printf("%d, %d\n", i, j);
　　　　　if (i * j > 3) return;
　　　}
　}
}
print_data();
outer より後に処理が続く場合
printf("complete\n");

**デフォルトの名無しさん** · 2019/12/16(月) 16:55:03.26

"短い処理"をいちいち関数にしなくて済むから、break 2とかはあってもいいが、
長い処理は関数にしてないとろくにテストできないんだからgotoなんていらない

**デフォルトの名無しさん** · 2019/12/16(月) 19:03:13.17

じゃあ>>57もいらないね

**デフォルトの名無しさん** · 2019/12/16(月) 19:09:09.12

gotoの基本的な使い方くらい知っておいた方が良いかと
ループを抜ける為だけに関数を分けないといけなくなるし
挙げ句の果てに言語の欠陥とか言い出す

**デフォルトの名無しさん** · 2019/12/16(月) 20:06:51.00

今のC言語ではネストしたループから抜けるのはフラグを使うかgotoしかないからたいていgoto使ってるけどgotoなんて野蛮なものはできたら使いたくないから他の言語で実装されてるラベル付きbreakなりレベル付きbreakが欲しいって話だろ
今更gotoの基本的な使い方ガーとか言ってる奴はどんだけ低レベルなんだよw

**デフォルトの名無しさん** · 2019/12/16(月) 20:09:28.82

あれ？
returnで抜ける為に関数に分けるんじゃなかったの？

**デフォルトの名無しさん** · 2019/12/16(月) 20:21:09.65

gotoは野蛮とかいう歪んだ教育のせいで
わざわざ変数を定義したりわざわざ関数を分けたりして
わざわざ見にくいコードにしちゃう

生でポインタを扱う野蛮なC言語なんか使わないで
上品な言語を使った方が良いよ

**デフォルトの名無しさん** · 2019/12/16(月) 20:28:02.60

gotoじゃなくてgodowntoなら誰も文句言わなかった

**デフォルトの名無しさん** · 2019/12/16(月) 20:47:58.36

自分は小規模な字句解析みたいなのでもgoto使うことがある。
ラベルで状態を表してgotoで遷移する。
goto使わなかったら状態を変数に入れて、
ループの中でswitchで場合分けするようなコードになって辛い。

**デフォルトの名無しさん** · 2019/12/16(月) 20:52:55.59

ループの中のswitchでループを抜けるのに
gotoを使うんじゃ？

**デフォルトの名無しさん** · 2019/12/16(月) 21:41:34.22

次の状態を返す関数ポインタでループすればいんじゃないの？

**デフォルトの名無しさん** · 2019/12/16(月) 21:45:33.24

そんな野蛮なことはしません

**デフォルトの名無しさん** · 2019/12/16(月) 22:12:14.62

>>87
returnで抜けるためというか、

1. gotoで抜けたい
2. 関数にするか
3. あ、そもそもコードが複雑なんだな
4. 関数にして正解だった

こんな感じかな。

gotoで抜けたいと思う＝コードが複雑になってるという警鐘だと思ってる。

**デフォルトの名無しさん** · 2019/12/16(月) 22:22:18.57

で>>82みたいな意味不明な関数の分け方をすると
>>57は、より複雑な記述をしたいっていう主張だし

gotoに親を殺された人の言うことは支離滅裂
理由が後付けだからそうなる

**デフォルトの名無しさん** · 2019/12/16(月) 22:27:34.28

> で>>82みたいな意味不明な関数の分け方をすると

なんで意味不明？
そもそもその処理に意味をもたせてから言おうよｗ
処理に意味をもたせると、関数にしたほうが良いと思うからさ

**デフォルトの名無しさん** · 2019/12/16(月) 22:32:01.57

> 理由が後付けだからそうなる

質問の方が先なのに、理由を先に言えるわけがないｗ

Q. 多重ループから抜けるのに、gotoは必要ですか？
A. 必要ありません。多重ループが必要になるぐらい複雑なら関数にしますから

これが普通だから、後付にはならないよね。

**デフォルトの名無しさん** · 2019/12/16(月) 22:37:09.13

じゃ>>57も当然不要ですね

**デフォルトの名無しさん** · 2019/12/16(月) 22:38:17.80

そんなにくり返し言わなくても、最初から不要だしｗ

**デフォルトの名無しさん** · 2019/12/16(月) 22:39:57.37

じゃあ>>57はどういうつもりで書いたんでしょうかね？

**デフォルトの名無しさん** · 2019/12/16(月) 22:40:50.59

gogoサイコーっす

**デフォルトの名無しさん** · 2019/12/16(月) 22:41:03.68

間違えたw goto

**デフォルトの名無しさん** · 2019/12/16(月) 22:42:13.32

話を戻して

・ループを抜けた後に関数の後処理が必要な場合
・ループを抜けた後にローカル変数を使った処理をする場合
・他、意味的に関数内で処理したい場合

はどうするの？

**デフォルトの名無しさん** · 2019/12/16(月) 22:44:03.17

> ・ループを抜けた後に関数の後処理が必要な場合
前処理を関数の中でするな

> ・ループを抜けた後に関数の後処理が必要な場合
戻り値を返せ

>・他、意味的に関数内で処理したい場合
自分で具体例が思いつかないなら出すな

**デフォルトの名無しさん** · 2019/12/16(月) 22:45:01.13

コピペミスったな

>・ループを抜けた後にローカル変数を使った処理をする場合
戻り値を返せ

**デフォルトの名無しさん** · 2019/12/16(月) 22:48:10.00

>>104
なぜ？

**デフォルトの名無しさん** · 2019/12/16(月) 22:52:22.49

>>106
なにが？

ってか、もう少し具体例を書けよ。
それみれば設計がおかしいという結論になる

例えて言うなら、パーサーの処理の中でネットワークにアクセスしたい時困るから
ネットワークモジュールはグローバルにアクセスできるようにしておくべき
みたいなことを言ってるようなもんだよ。それは設計がおかしい。

**デフォルトの名無しさん** · 2019/12/16(月) 22:54:10.44

int func(int n){
ロック
ループ
アンロック
return 結果;
}

**デフォルトの名無しさん** · 2019/12/16(月) 22:55:55.89

int func(int n){
ループ
値を加工
return 値
}

**デフォルトの名無しさん** · 2019/12/16(月) 22:58:13.41

>>108
「ループ」という言い方がおかしい。
ループで「実装」しているのであって
やっているのはなにかの「処理」だろう？

int func(int n){
ロック
処理
アンロック
return 結果;
}

「処理」という名前をつけたくなるならば、
そこは独立した関数にするべきだろう

**デフォルトの名無しさん** · 2019/12/16(月) 23:02:27.04

double sigma(const double *a){
ループで総和と2乗和と数を計算
標準偏差を計算
return 標準偏差
}

**デフォルトの名無しさん** · 2019/12/16(月) 23:04:47.03

int func(int n){
int a[10] = {0};
ループ
aから値を計算
return 値
}

**デフォルトの名無しさん** · 2019/12/16(月) 23:06:46.75

いくらでも思い浮かぶ

**デフォルトの名無しさん** · 2019/12/16(月) 23:07:37.81

全部関数に分けろって言われて終わりそう

**デフォルトの名無しさん** · 2019/12/16(月) 23:09:19.22

分ける理由がない

**デフォルトの名無しさん** · 2019/12/16(月) 23:10:06.65

>>112を分けてみて

**デフォルトの名無しさん** · 2019/12/16(月) 23:17:03.74

関数分けってのは意味、機能、汎用性、...で分けるべきであって
単にループを抜けるのにreturnを使いたいから分けるとかを考えるべきじゃない

**デフォルトの名無しさん** · 2019/12/16(月) 23:23:11.90

素直にgotoの使い方を覚えた方が良いよ
意地はってないで

**デフォルトの名無しさん** · 2019/12/16(月) 23:42:01.03

>>118
だから意味とか汎用性とかで分けると
自然にループに抜けるときにreturnを使えば良くなるんだよ

**デフォルトの名無しさん** · 2019/12/16(月) 23:45:51.39

gotoでもreturnでも読めれば良い
但し1関数50行以上は読めないんで宜しく

**デフォルトの名無しさん** · 2019/12/16(月) 23:48:10.14

>>112
だから「ループ」と書くなっちゅうねんｗ
関数を設計するならこんな感じやろ？

int func(int n){
int a[10] = {0};
memset_random(a, sizeof(a)); // a配列をランダムな値で埋める
int chksum = chksum(a, sizeof(a)); // a配列のチェックサムを計算
return 値
}

関数の中でループしてようが何して用が関係ないわ

**デフォルトの名無しさん** · 2019/12/16(月) 23:55:19.85

sizeof(a) とか書くレベルの相手してたのか
アホらしい

**デフォルトの名無しさん** · 2019/12/17(火) 00:04:36.92

>>119
じゃあ普通のbreakもいらないね
まさか使ってないよね

**デフォルトの名無しさん** · 2019/12/17(火) 06:41:54.53

まあgotoあればbreakは要らんわなw
そんな言語は使いたくないけど

**デフォルトの名無しさん** · 2019/12/17(火) 07:00:38.11

gotoもbreakもいらないって

**デフォルトの名無しさん** · 2019/12/17(火) 07:10:23.36

gotoもbreakも無い
ループを抜けるときはreturn
switchを抜ける時もreturm
関数は単一の処理しか出来ない

こんな言語はイヤだね

**デフォルトの名無しさん** · 2019/12/17(火) 07:23:38.89

後に退けなくなったアホの作る言語

**デフォルトの名無しさん** · 2019/12/17(火) 07:39:52.35

gotoの使い方

http://www7b.biglobe.ne.jp/~robe/cpphtml/html03/cpp03005.html
http://www9.plala.or.jp/sgwr-t/c/sec06-6.html
http://tomeapp.jp/archives/1029
http://opencv.jp/opencv-2svn/c/core_utility_and_system_functions_and_macros.html

**デフォルトの名無しさん** · 2019/12/17(火) 08:49:57.51

たとえ相手がアホであっても、しつこく絡むのもまたアホだと思うんだが…

**デフォルトの名無しさん** · 2019/12/17(火) 12:33:06.04

自分より下の人間を叩くしか能がないんだろ
ID:arH1mwfYなんてスルーでいいと思うんだがそれしか叩けないからいつまでたっても終わらないw

**デフォルトの名無しさん** · 2019/12/17(火) 13:15:29.02

雑談スレだしネタも無いから良いんじゃね？