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

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

gets関数とか

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

**デフォルトの名無しさん** · 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

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

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

String型が無いって死ぬほど不便なんだが

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

他の言語を使ってください

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

>>132
自分で似たようなものを作ればいいだろ。汎用的なものがあっても使い勝手や性能で何かしら不満が出るのだから、自分の要求に合致するものを自分で作りな。

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

特定の用途においてローカルラベルが役に立つことも無くはないかもしれないな。
自分でそれをやるかどうかは別にして。
いずれにせよ、gcc では式の中で変数を宣言できるのだから
ローカルラベルが可能になっても驚かないよ。

**132** · 2019/12/18(水) 23:56:34.28

typedefを使用
https://pastebin.com/iX2DDpRA

**デフォルトの名無しさん** · 2019/12/19(木) 00:12:48.48

PHPはbreakにネスト数指定できるよな

**デフォルトの名無しさん** · 2019/12/19(木) 09:03:44.36

C言語の開発者によるgoto文の使い方を対象とした実証研究の結果、「goto文は無害だと考えられる」
https://developers.srad.jp/story/15/02/14/2017207/

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

longjmp についても調査してほしいな。
特に例は挙げないが、誰もが使っているようなプログラムでも結構使われてる。

**デフォルトの名無しさん** · 2019/12/19(木) 16:57:20.13

static

**デフォルトの名無しさん** · 2019/12/25(水) 01:15:13.49

>>139
それは代わりがない。

**デフォルトの名無しさん** · 2020/07/09(木) 10:18:18.26

>>7
大発明なんbだけどな、
リスクも大きい

**デフォルトの名無しさん** · 2020/07/09(木) 10:22:15.80

発明って言うか当時はそれが普通だったろ

**デフォルトの名無しさん** · 2020/07/09(木) 10:51:31.74

longjmpは「設計上のミス」ではないんでは。
むしろそういう動作をする目的で設計・導入されたように思えるけど
（longjmpに関するRationaleを読んでないので当てずっぽうw）

**デフォルトの名無しさん** · 2020/07/09(木) 16:31:13.77

>>5
エスティーアールトゥーケー

**デフォルトの名無しさん** · 2020/07/09(木) 21:58:30.04

例外は setjump、longjumpて実装されてるんじゃないのかな？
違うのかな？

**デフォルトの名無しさん** · 2020/07/10(金) 22:03:48.37

>>143
普通じゃない
当時はPascalの文字数+文字列が普通

**デフォルトの名無しさん** · 2020/07/10(金) 22:26:03.98

returnが1変数しか対応していないこと。

ポインター祭りになって可読性がウンコになる

**デフォルトの名無しさん** · 2020/07/10(金) 23:11:39.81

>>148
可読性がうんこなのはお前の能力が足りないから

**デフォルトの名無しさん** · 2020/07/11(土) 00:22:05.38

いろんな言語たらい回しにされるって不幸だよね。

こういうノーガキ垂れる奴がウチの職場にいなくて良かった～ヽ(´▽｀)/

**デフォルトの名無しさん** · 2020/07/11(土) 10:50:20.85

>>147
OSのAPIは文字数じゃなくて
文字列byte列+デリミタ'$'
みたいなのが多かったぞ

**デフォルトの名無しさん** · 2020/07/11(土) 12:01:47.31

>>151
それはCP/Mの文字列出力がそうなっていただけ
入力はC言語のfgets()なんかみたいにデリミタの '$' を付加したりしないよ

**デフォルトの名無しさん** · 2020/07/11(土) 23:02:45.45

MASM もそうだけど、$ は実際にメモリに配置されるのですか？
それとも、db(アセンブラ命令)で長さを省略するためのものですか？

**デフォルトの名無しさん** · 2020/07/11(土) 23:04:50.81

すみません、>>153 です
送信した瞬間に自分の発言の矛盾に気がつきました…

**デフォルトの名無しさん** · 2020/07/13(月) 19:12:30.84

>>146
longjumbと例外に関して
すばらしい文書があるからお読みなさって。
http://www.nurs.or.jp/~sug/a/super/longjmp.htm#sec36

**デフォルトの名無しさん** · 2020/07/13(月) 19:20:56.50

そっちは Not Found ですよ
http://www.nurs.or.jp/~sug/soft/super/longjmp.htm

◆QZaw55cn4c · 2020/07/13(月) 20:53:25.68

>>156
ありがとうございます、昔に読んだのに URL を失ってしまって…ずっと探していたんです

**デフォルトの名無しさん** · 2020/07/13(月) 21:24:21.32

>>155
ありがとうございます >>146 です
拝見いたします

**デフォルトの名無しさん** · 2020/07/14(火) 09:59:19.87

ありがとうございます。いや、longjmp をdisるような意図は無かったんだけど。
「シグナルとコールバック」も勉強になります。

http://www.nurs.or.jp/%7Esug/soft/super/signal.htm

この辺りは man を読んでも使い方が解りにくいので…

**デフォルトの名無しさん** · 2020/07/23(木) 11:48:35.85

_sが付いてるやつ使えよ

**デフォルトの名無しさん** · 2021/05/21(金) 17:59:38.38

関数じゃないけどこれやろ
If the unsigned type has conversion rank greater than or equal to the rank of the signed type, then the operand with the signed type is implicitly converted to the unsigned type.
うんこ

**デフォルトの名無しさん** · 2021/05/21(金) 18:32:54.36

プログラマを信じろと言うけどさ
じゃあ暗黙の型変換とか余計な節介やめてくれねーかな

**デフォルトの名無しさん** · 2021/05/21(金) 18:36:26.66

cでまともに算数出来れば一人前

**デフォルトの名無しさん** · 2021/07/18(日) 03:12:16.30

暗黙が嫌なら明示しろよばかじゃないの？

**デフォルトの名無しさん** · 2023/01/02(月) 19:09:01.78

>>162
暗黙の型変換はワーニング出るようにできなかったっけ？

**デフォルトの名無しさん** · 2023/05/10(水) 10:41:52.89

>>138
まとめがおかしい
啓蒙活動の結果gotoの有害な使い方は極めて少なくなった
というべきだろう

**デフォルトの名無しさん** · 2023/08/04(金) 19:33:28.29

ken thompsonがcreat()の最後のeを省いたのは失敗だったと
直すとしたらそこを直したいと

**デフォルトの名無しさん** · 2023/09/27(水) 15:27:11.03

失敗は成功への近道だと思って取り組んでみよう。

**デフォルトの名無しさん** · 2023/09/30(土) 09:21:33.38

(”ロ”;)ｹﾞｹﾞｯ!!

**デフォルトの名無しさん** · 2023/10/03(火) 12:21:18.69

型省略でintと看做すのはミスだろ

**デフォルトの名無しさん** · 2023/10/04(水) 22:29:38.55

>>170
K&Rの頃との互換性の為じゃないかな。

Ｃは、兎に角モノグサの為の様な仕様が多いよね。
一度馴染むと、BASICですら面倒くさい。

**デフォルトの名無しさん** · 2023/10/05(木) 17:05:36.61

まんどくせ

**デフォルトの名無しさん** · 2023/10/05(木) 19:40:42.12

ポインタ渡し全般かな

**デフォルトの名無しさん** · 2023/10/05(木) 22:17:24.30

Cの型はある意味pythonより弱い

**デフォルトの名無しさん** · 2023/10/07(土) 11:48:43.75

ある意味というか、Pythonは一応強い型付けでしょ。

**デフォルトの名無しさん** · 2023/10/07(土) 11:51:33.89

Cに型なんかあって無い様なもんだしな

**デフォルトの名無しさん** · 2023/10/07(土) 17:38:41.59

>>1
fgets知らんのかな
getsなんて使わない
どうしても使う時はfgets(buf,size,stdin)とすればいいだけ

**デフォルトの名無しさん** · 2023/10/08(日) 10:11:26.30

>>176
Cはアセンブラで言うところのアドレッシングが充実

**デフォルトの名無しさん** · 2023/10/09(月) 15:00:06.76

マクロが真っ黒

**デフォルトの名無しさん** · 2023/10/15(日) 02:11:48.65

∑(ﾉ▼ο▼)ﾉｵｫｵｫｵ!!

**デフォルトの名無しさん** · 2023/10/16(月) 09:47:48.72

DDoS回避したみたいね

**デフォルトの名無しさん** · 2023/10/17(火) 11:44:25.59

atan
(atan2の方じゃなくて)

**デフォルトの名無しさん** · 2023/10/18(水) 07:26:03.37

くせの強いscanf()とか

**デフォルトの名無しさん** · 2023/10/18(水) 07:26:15.66

くせの強いscanf()とか

**デフォルトの名無しさん** · 2023/10/18(水) 14:31:46.14

visual studioでscanf()使ったらエラーになった

**デフォルトの名無しさん** · 2023/10/18(水) 14:31:57.74

visual studioでscanf()使ったらエラーになった

**デフォルトの名無しさん** · 2023/10/20(金) 09:24:01.14

free()

**デフォルトの名無しさん** · 2023/12/15(金) 15:34:51.20

sprintf()

**デフォルトの名無しさん** · 2023/12/17(日) 07:31:47.54

fopen()

**デフォルトの名無しさん** · 2023/12/17(日) 08:12:54.14

ctime()

**デフォルトの名無しさん** · 2023/12/31(日) 16:48:51.59

fopen

**デフォルトの名無しさん** · 2024/01/06(土) 17:10:43.88

strtok